Fix bad multiline matches for link definitions

src/languageFeatures/documentLinks.ts

@@ -33,7 +33,12 @@ function createHref(
 ): ExternalHref | InternalHref | undefined {
 	if (/^[a-z\-][a-z\-]+:/i.test(link)) {
 		// Looks like a uri
-		return { kind: HrefKind.External, uri: URI.parse(tryDecodeUri(link)) };
+		try {
+			return { kind: HrefKind.External, uri: URI.parse(tryDecodeUri(link)) };
+		} catch (e) {
+			console.warn(r`Failed to parse link ${link} in ${sourceDocUri.toString(true)}`);
+			return undefined;
+		}
 	}
 
 	const resolved = resolveInternalDocumentLink(sourceDocUri, link, workspace);
@@ -185,7 +190,7 @@ const autoLinkPattern = /(?<!\\)\<(\w+:[^\>\s]+)\>/g;
 /**
  * Matches `[text]: link`
  */
-const definitionPattern = /^([\t ]*(?<!\\)\[(?!\^)((?:\\\]|[^\]])+)\]:\s*)([^<]\S*|<(?:\\[<>]|[^<>])+>)/gm;
+const definitionPattern = /^([\t ]*(?<!\\)\[(?!\^)((?:\\\]|[^\]])+)\]:[\t ]*)([^<]\S*|<(?:\\[<>]|[^<>])+>)/gm;
 
 class InlineRanges {
 

src/test/documentLinks.test.ts

@@ -782,6 +782,16 @@ suite('Link computer', () => {
 
 		assertLinksEqual(links, []);
 	});
+
+	test('Should not crash on definition that spans multiple lines (#192)', async () => {
+		const links = await getLinksForText(joinLines(
+			`Options`,
+			`[positional parameters]:`,
+			`-R: recursively list subdirectories.`,
+		));
+
+		assertLinksEqual(links, []);
+	});
 });