chore: remove old sourcemap search strategies

OPTIONS.md

@@ -11,7 +11,6 @@
 <h5>Default value:</h4><pre><code>undefined</pre></code><h4>customPropertiesGenerator</h4><p>Customize the properties shown for an object in the debugger (local variables, etc...). Samples:<br>    1. { ...this, extraProperty: &#39;12345&#39; } // Add an extraProperty 12345 to all objects<br>    2. this.customProperties ? this.customProperties() : this // Use customProperties method if available, if not use the properties in this (the default properties)<br>    3. function () { return this.customProperties ? this.customProperties() : this } // Use customDescription method if available, if not return the default properties<br><br>    Deprecated: This is a temporary implementation of this feature until we have time to implement it in the way described here: <a href="https://github.com/microsoft/vscode/issues/102181">https://github.com/microsoft/vscode/issues/102181</a></p>
 <h5>Default value:</h4><pre><code>undefined</pre></code><h4>cwd</h4><p>Absolute path to the working directory of the program being debugged. If you&#39;ve set localRoot then cwd will match that value otherwise it falls back to your workspaceFolder</p>
 <h5>Default value:</h4><pre><code>localRoot || ${workspaceFolder}</pre></code><h4>enableContentValidation</h4><p>Toggles whether we verify the contents of files on disk match the ones loaded in the runtime. This is useful in a variety of scenarios and required in some, but can cause issues if you have server-side transformation of scripts, for example.</p>
-<h5>Default value:</h4><pre><code>true</pre></code><h4>enableTurboSourcemaps</h4><p>Configures whether to use a new, faster mechanism for sourcemap discovery</p>
 <h5>Default value:</h4><pre><code>true</pre></code><h4>env</h4><p>Environment variables passed to the program. The value <code>null</code> removes the variable from the environment.</p>
 <h5>Default value:</h4><pre><code>{}</pre></code><h4>envFile</h4><p>Absolute path to a file containing environment variable definitions.</p>
 <h5>Default value:</h4><pre><code>null</pre></code><h4>localRoot</h4><p>Path to the local directory containing the program.</p>
@@ -64,7 +63,6 @@
 <h5>Default value:</h4><pre><code>undefined</pre></code><h4>customPropertiesGenerator</h4><p>Customize the properties shown for an object in the debugger (local variables, etc...). Samples:<br>    1. { ...this, extraProperty: &#39;12345&#39; } // Add an extraProperty 12345 to all objects<br>    2. this.customProperties ? this.customProperties() : this // Use customProperties method if available, if not use the properties in this (the default properties)<br>    3. function () { return this.customProperties ? this.customProperties() : this } // Use customDescription method if available, if not return the default properties<br><br>    Deprecated: This is a temporary implementation of this feature until we have time to implement it in the way described here: <a href="https://github.com/microsoft/vscode/issues/102181">https://github.com/microsoft/vscode/issues/102181</a></p>
 <h5>Default value:</h4><pre><code>undefined</pre></code><h4>cwd</h4><p>Absolute path to the working directory of the program being debugged. If you&#39;ve set localRoot then cwd will match that value otherwise it falls back to your workspaceFolder</p>
 <h5>Default value:</h4><pre><code>"${workspaceFolder}"</pre></code><h4>enableContentValidation</h4><p>Toggles whether we verify the contents of files on disk match the ones loaded in the runtime. This is useful in a variety of scenarios and required in some, but can cause issues if you have server-side transformation of scripts, for example.</p>
-<h5>Default value:</h4><pre><code>true</pre></code><h4>enableTurboSourcemaps</h4><p>Configures whether to use a new, faster mechanism for sourcemap discovery</p>
 <h5>Default value:</h4><pre><code>true</pre></code><h4>env</h4><p>Environment variables passed to the program. The value <code>null</code> removes the variable from the environment.</p>
 <h5>Default value:</h4><pre><code>{}</pre></code><h4>envFile</h4><p>Absolute path to a file containing environment variable definitions.</p>
 <h5>Default value:</h4><pre><code>null</pre></code><h4>killBehavior</h4><p>Configures how debug processes are killed when stopping the session. Can be:<br><br>- forceful (default): forcefully tears down the process tree. Sends SIGKILL on posix, or <code>taskkill.exe /F</code> on Windows.<br>- polite: gracefully tears down the process tree. It&#39;s possible that misbehaving processes continue to run after shutdown in this way. Sends SIGTERM on posix, or <code>taskkill.exe</code> with no <code>/F</code> (force) flag on Windows.<br>- none: no termination will happen.</p>
@@ -118,7 +116,6 @@
 <h5>Default value:</h4><pre><code>undefined</pre></code><h4>customPropertiesGenerator</h4><p>Customize the properties shown for an object in the debugger (local variables, etc...). Samples:<br>    1. { ...this, extraProperty: &#39;12345&#39; } // Add an extraProperty 12345 to all objects<br>    2. this.customProperties ? this.customProperties() : this // Use customProperties method if available, if not use the properties in this (the default properties)<br>    3. function () { return this.customProperties ? this.customProperties() : this } // Use customDescription method if available, if not return the default properties<br><br>    Deprecated: This is a temporary implementation of this feature until we have time to implement it in the way described here: <a href="https://github.com/microsoft/vscode/issues/102181">https://github.com/microsoft/vscode/issues/102181</a></p>
 <h5>Default value:</h4><pre><code>undefined</pre></code><h4>cwd</h4><p>Absolute path to the working directory of the program being debugged. If you&#39;ve set localRoot then cwd will match that value otherwise it falls back to your workspaceFolder</p>
 <h5>Default value:</h4><pre><code>localRoot || ${workspaceFolder}</pre></code><h4>enableContentValidation</h4><p>Toggles whether we verify the contents of files on disk match the ones loaded in the runtime. This is useful in a variety of scenarios and required in some, but can cause issues if you have server-side transformation of scripts, for example.</p>
-<h5>Default value:</h4><pre><code>true</pre></code><h4>enableTurboSourcemaps</h4><p>Configures whether to use a new, faster mechanism for sourcemap discovery</p>
 <h5>Default value:</h4><pre><code>true</pre></code><h4>env</h4><p>Environment variables passed to the program. The value <code>null</code> removes the variable from the environment.</p>
 <h5>Default value:</h4><pre><code>{}</pre></code><h4>envFile</h4><p>Absolute path to a file containing environment variable definitions.</p>
 <h5>Default value:</h4><pre><code>null</pre></code><h4>localRoot</h4><p>Path to the local directory containing the program.</p>
@@ -170,7 +167,6 @@
 <h5>Default value:</h4><pre><code>localRoot || ${workspaceFolder}</pre></code><h4>debugWebviews</h4><p>Configures whether we should try to attach to webviews in the launched VS Code instance. This will only work in desktop VS Code.</p>
 <h5>Default value:</h4><pre><code>false</pre></code><h4>debugWebWorkerHost</h4><p>Configures whether we should try to attach to the web worker extension host.</p>
 <h5>Default value:</h4><pre><code>false</pre></code><h4>enableContentValidation</h4><p>Toggles whether we verify the contents of files on disk match the ones loaded in the runtime. This is useful in a variety of scenarios and required in some, but can cause issues if you have server-side transformation of scripts, for example.</p>
-<h5>Default value:</h4><pre><code>true</pre></code><h4>enableTurboSourcemaps</h4><p>Configures whether to use a new, faster mechanism for sourcemap discovery</p>
 <h5>Default value:</h4><pre><code>true</pre></code><h4>env</h4><p>Environment variables passed to the program. The value <code>null</code> removes the variable from the environment.</p>
 <h5>Default value:</h4><pre><code>{}</pre></code><h4>envFile</h4><p>Absolute path to a file containing environment variable definitions.</p>
 <h5>Default value:</h4><pre><code>null</pre></code><h4>localRoot</h4><p>Path to the local directory containing the program.</p>

package.json

@@ -167,7 +167,6 @@
   "main": "./src/extension.js",
   "enabledApiProposals": [
     "portsAttributes",
-    "findTextInFiles",
     "workspaceTrust",
     "tunnels"
   ],

src/adapter/breakpointPredictor.ts

@@ -12,11 +12,7 @@ import { ILogger, LogTag } from '../common/logging';
 import { fixDriveLetterAndSlashes, forceForwardSlashes } from '../common/pathUtils';
 import { ISourceMapMetadata } from '../common/sourceMaps/sourceMap';
 import { ISourceMapFactory } from '../common/sourceMaps/sourceMapFactory';
-import {
-  ISearchStrategy,
-  ISearchStrategyFallback,
-  ISourcemapStreamOptions,
-} from '../common/sourceMaps/sourceMapRepository';
+import { ISearchStrategy, ISourcemapStreamOptions } from '../common/sourceMaps/sourceMapRepository';
 import { ISourcePathResolver } from '../common/sourcePathResolver';
 import { getOptimalCompiledPosition, parseSourceMappingUrl } from '../common/sourceUtils';
 import * as urlUtils from '../common/urlUtils';
@@ -80,25 +76,16 @@ export class BreakpointPredictorCachedState<T> {
 
 @injectable()
 export abstract class BreakpointSearch {
-  private readonly repo: ISearchStrategy;
-
   constructor(
-    @inject(AnyLaunchConfiguration) config: AnyLaunchConfiguration,
     @inject(OutFiles) private readonly outFiles: OutFiles,
-    @inject(ISearchStrategy) repo: ISearchStrategy,
-    @inject(ISearchStrategyFallback) fallbackRepo: ISearchStrategy,
+    @inject(ISearchStrategy) private readonly repo: ISearchStrategy,
     @inject(ILogger) protected readonly logger: ILogger,
     @inject(ISourceMapFactory) private readonly sourceMapFactory: ISourceMapFactory,
     @inject(ISourcePathResolver)
     private readonly sourcePathResolver: ISourcePathResolver | undefined,
     @inject(BreakpointPredictorCachedState)
     private readonly state: BreakpointPredictorCachedState<unknown>,
-  ) {
-    this.repo =
-      'enableTurboSourcemaps' in config && config.enableTurboSourcemaps === false
-        ? fallbackRepo
-        : repo;
-  }
+  ) {}
 
   public abstract getMetadataForPaths(
     sourcePaths: readonly string[],

src/build/generate-contributions.ts

@@ -354,11 +354,6 @@ const nodeBaseConfigurationAttributes: ConfigurationAttributes<INodeBaseConfigur
     description: refString('node.versionHint.description'),
     default: 12,
   },
-  enableTurboSourcemaps: {
-    type: 'boolean',
-    default: true,
-    description: refString('node.enableTurboSourcemaps.description'),
-  },
 };
 
 /**