api: merge ESM and Node.js module interceptors

[jrieken]

fixes #285297

Summary

Merges the NodeModuleInterceptor class (handling ESM imports) into the NodeModuleRequireInterceptor class (handling CJS requires) so that module interception for both ESM and CJS are handled together in a single place. Additionally, an integration test is included to verify that import and require on the VS Code API module consistently yield identical object instances.

Session Context

Key decisions from the development session: - **Interceptor Merging**: We explicitly chose to merge `NodeModuleInterceptor` into `NodeModuleRequireInterceptor` rather than having two separate classes. This keeps both ESM and CJS interception rules centralized and simpler to maintain. - **Hook Cleanup**: Since the Node `registerHooks` method returns an object with a `deregister` method, we tied this hook to the interceptor's internal `DisposableStore`. It will be safely cleaned up on disposal. - **ESM Integration Test Strategy**: We wrote the integration test using a `new Function('url', 'return import(url)')` wrapper to bypass TypeScript transpiling dynamic `import()` to `require()` in the `extensions/vscode-api-tests` CommonJS build constraints. - **Test File Location**: We specifically wrote the test file adjacent to the suite rather than using the OS temp directory, because VS Code identifies calling packages and permissions using file path containment. Writing in temp would cause `import` interceptions to fail. A fallback to skip the test was added if file system writing fails, and a bold disclaimer is placed inside the code block so it can be safely removed by developers if a crash prevents automatic cleanup.

Changes

• src/vs/workbench/api/node/extHostExtensionService.ts: Merged intercepts, added cleanup disposable for hooks.deregister().
• extensions/vscode-api-tests/src/singlefolder-tests/module.test.ts: Added validation test for API singleton integrity across ESM and CJS loaders.

[Copilot]

Pull request overview

This PR consolidates VS Code API module interception for both CommonJS (require) and ESM (import) into a single interceptor implementation in the extension host, and adds an integration test to ensure import 'vscode' and require('vscode') expose the same underlying API instances.

Changes:

• Merge the ESM interception logic into NodeModuleRequireInterceptor using node:module.registerHooks.
• Add disposal wiring intended to deregister the registered hooks.
• Add an integration test that validates ESM import and CJS require return consistent API instances.

Show a summary per file

File	Description
src/vs/workbench/api/node/extHostExtensionService.ts	Merges ESM+CJS interception paths into one interceptor and attempts to deregister hooks on disposal.
extensions/vscode-api-tests/src/singlefolder-tests/module.test.ts	Adds an integration test that dynamically imports an .mjs file to compare ESM vs CJS vscode API instances.

Copilot's findings

• Files reviewed: 2/2 changed files
• Comments generated: 1