Merge pull request microsoft#185784 from microsoft/tyriar/185139

Fix :line:col in anything quick access when @ is in path

Author: Tyriar

src/vs/workbench/contrib/search/common/search.ts

@@ -157,7 +157,7 @@ export interface IFilterAndRange {
 }
 
 export function extractRangeFromFilter(filter: string, unless?: string[]): IFilterAndRange | undefined {
-	if (!filter || unless?.some(value => filter.indexOf(value) !== -1)) {
+	if (!filter || unless?.some(value => filter.indexOf(value) === 0)) {
 		return undefined;
 	}
 

src/vs/workbench/contrib/search/test/common/extractRange.test.ts

@@ -12,6 +12,7 @@ suite('extractRangeFromFilter', () => {
 		assert.ok(!extractRangeFromFilter(''));
 		assert.ok(!extractRangeFromFilter('/some/path'));
 		assert.ok(!extractRangeFromFilter('/some/path/file.txt'));
+		assert.ok(!extractRangeFromFilter('/some/@path'), 'paths like as /node_modules/@types/ should be supported');
 
 		for (const lineSep of [':', '#', '(', ':line ']) {
 			for (const colSep of [':', '#', ',']) {
@@ -44,7 +45,7 @@ suite('extractRangeFromFilter', () => {
 	});
 
 	test('unless', async function () {
-		const res = extractRangeFromFilter('/some/path/file.txt@ (19,20)', ['@']);
+		const res = extractRangeFromFilter('@/some/path/file.txt (19,20)', ['@']);
 
 		assert.ok(!res);
 	});