Merge pull request #200919 from sparxooo/200257-fix

Fix for #200257 plus existing trailing non-numerics regex fix

src/vs/workbench/contrib/terminalContrib/links/browser/terminalLinkOpeners.ts

@@ -108,7 +108,14 @@ export class TerminalSearchLinkOpener implements ITerminalLinkOpener {
 		// - Grep output: <link>:<result line>
 		// This only happens when the colon is _not_ followed by a forward- or back-slash as that
 		// would break absolute Windows paths (eg. `C:/Users/...`).
-		text = text.replace(/:[^\\/][^\d]+$/, '');
+		text = text.replace(/:[^\\/\d][^\d]*$/, '');
+
+		// Remove any trailing periods after the line/column numbers, to prevent breaking the search feature, #200257
+		// Examples:
+		// "Check your code Test.tsx:12:45." -> Test.tsx:12:45
+		// "Check your code Test.tsx:12." -> Test.tsx:12
+
+		text = text.replace(/\.$/, '');
 
 		// If any of the names of the folders in the workspace matches
 		// a prefix of the link, remove that prefix and continue