Merge pull request #95365 from connorskees/issue-95364

Strip \\?\ from absolute Windows paths

src/vs/workbench/contrib/terminal/browser/links/terminalLinkManager.ts

@@ -544,6 +544,10 @@ export class TerminalLinkManager extends DisposableStore {
 						return null;
 					}
 					link = this.osPath.join(this._processCwd, link);
+				} else {
+					// Remove \\?\ from paths so that they share the same underlying
+					// uri and don't open multiple tabs for the same file
+					link = link.replace(/^\\\\\?\\/, '');
 				}
 			} else {
 				if (!this._processCwd) {

src/vs/workbench/contrib/terminal/test/browser/links/terminalLinkManager.test.ts

@@ -254,6 +254,7 @@ suite('Workbench - TerminalLinkHandler', () => {
 			assert.equal(linkHandler.preprocessPath('~/src/file3'), 'C:\\Users\\Me\\src\\file3');
 			assert.equal(linkHandler.preprocessPath('~\\src\\file4'), 'C:\\Users\\Me\\src\\file4');
 			assert.equal(linkHandler.preprocessPath('C:\\absolute\\path\\file5'), 'C:\\absolute\\path\\file5');
+			assert.equal(linkHandler.preprocessPath('\\\\?\\C:\\absolute\\path\\extended\\file6'), 'C:\\absolute\\path\\extended\\file6');
 		});
 		test('Windows - spaces', () => {
 			const linkHandler = new TestTerminalLinkManager(new TestXterm() as any, {