Merge pull request #49 from iterative/fix/link-selector

yathomasi · web-flow · commit 494201178ede · 2023-07-15T19:43:05.000+05:45
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "repo-link-check",
-  "version": "0.14.0",
+  "version": "0.14.1",
   "description": "A multi-target library for ensuring all links in a file are alive",
   "main": "index.js",
   "author": "RMP",
diff --git a/src/scrapeLinks.test.js b/src/scrapeLinks.test.js
@@ -8,6 +8,9 @@ Markdown sample with a [link to Google](https://www.google.com), one [with a URL
 - Finally a few [ref] [links][links] [here][link-here]
 - BTW it shouldn't plain links like http://www.this.com or that.com shouldn't get picked up.
 
+
+) -> List[Dict]:
+\`\`\`
 this is a test [ref]:
 
 https://www.ref.com
diff --git a/src/scrapeLinks.ts b/src/scrapeLinks.ts
@@ -29,7 +29,7 @@ const scrapeFromString: (filePath: string, content: string) => string[] = (
       )
       const mdRefLinks = regexMap(
         content,
-        /\[.*\]:[^\S\r\n]*\n?[^\S\r\n]*(.*)/gm,
+        /\[.*\]:[^\S\r\n]*\n?[^\S\r\n]*([\w\d./?=#\-&_~:;,@+$!*()]+)/gm,
         x => x[2] || x[1]
       )
       const hrefLinks = regexMap(content, /href="(.*?)"/gm)

Original file line number	Diff line number	Diff line change
`@@ -1,6 +1,6 @@`
`1`	`1`	`{`
`2`	`2`	`"name": "repo-link-check",`
`3`		`- "version": "0.14.0",`
	`3`	`+ "version": "0.14.1",`
`4`	`4`	`"description": "A multi-target library for ensuring all links in a file are alive",`
`5`	`5`	`"main": "index.js",`
`6`	`6`	`"author": "RMP",`
Original file line number	Diff line number	Diff line change
`@@ -29,7 +29,7 @@ const scrapeFromString: (filePath: string, content: string) => string[] = (`
`29`	`29`	`)`
`30`	`30`	`const mdRefLinks = regexMap(`
`31`	`31`	`content,`
`32`		`- /\[.\]:[^\S\r\n]\n?[^\S\r\n](.)/gm,`
	`32`	`+ /\[.\]:[^\S\r\n]\n?[^\S\r\n]([\w\d./?=#\-&_~:;,@+$!()]+)/gm,`
`33`	`33`	`x => x[2] \|\| x[1]`
`34`	`34`	`)`
`35`	`35`	`const hrefLinks = regexMap(content, /href="(.*?)"/gm)`