refactor: W-18488820 - last of fs and eslint rule

[CristiCanizales]

What does this PR do?

This pull request introduces several updates across the codebase, focusing on improving code quality, enforcing stricter linting rules, transitioning to asynchronous patterns, and enhancing compatibility with web extensions. Below is a summary of the most important changes grouped by theme.

Linting and Code Quality Improvements:

• Updated the linting rule for @typescript-eslint/no-floating-promises from warn to error to enforce stricter handling of unresolved promises. (eslint.config.mjs, eslint.config.mjs L133-R133)
• Added a new no-restricted-imports rule to restrict usage of Node.js fs and fs-extra modules, recommending the use of VSCode's fs API for compatibility with web extensions. (eslint.config.mjs, eslint.config.mjs R290-R301)

Asynchronous Code Transition:

• Converted synchronous methods to asynchronous ones in the ApexDebugForTest class, ensuring proper handling of asynchronous operations. (apexDebugForTest.ts, packages/salesforcedx-apex-debugger/test/unit/adapter/apexDebugForTest.ts L103-R117)
• Updated unit tests in apexDebug.test.ts and commandOutput.test.ts to use await for testing asynchronous methods, improving reliability and readability. (apexDebug.test.ts, [1]; commandOutput.test.ts, [2]

Debugger Enhancements:

• Removed unused lines and improved formatting in the ApexReplayDebug class for better readability and maintainability. (apexReplayDebug.ts, [1] [2]

Compatibility Updates:

• Introduced the packages/salesforcedx-apex-replay-debugger directory to the ESLint configuration, ensuring it adheres to the project's linting rules. (eslint.config.mjs, eslint.config.mjsR410)
• Suppressed the no-restricted-imports rule in specific files where usage of Node.js fs is necessary for integration tests. (logContextUtil.ts, [1]; goldFileUtil.ts, [2]

Miscellaneous Improvements:

• Adjusted TypeScript configuration formatting for consistency. (tsconfig.json, packages/salesforcedx-apex-replay-debugger/tsconfig.jsonL5-R7)
• Updated the dispose method in appInsights.ts to use void for clarity in asynchronous contexts. (appInsights.ts, packages/salesforcedx-utils-vscode/src/telemetry/reporters/appInsights.tsL49-R49)

What issues does this PR fix or reference?

@W-18488820@

[mshanemc]

see inline comments. I think fixing the rule to only be node:fs will make this all easier.

[mshanemc]

e2e https://github.com/forcedotcom/salesforcedx-vscode/actions/runs/16121647113