fix: properly parse srcset attribute (#510)

.gitignore

@@ -2,3 +2,4 @@ node_modules/
 build/
 coverage
 .vscode
+.DS_Store

package.json

@@ -32,6 +32,7 @@
     "meow": "^10.1.1",
     "mime": "^3.0.0",
     "server-destroy": "^1.0.1",
+    "srcset": "^5.0.0",
     "update-notifier": "^6.0.0"
   },
   "devDependencies": {

src/links.ts

@@ -1,6 +1,7 @@
 import {WritableStream} from 'htmlparser2/lib/WritableStream';
 import {Readable} from 'stream';
 import {URL} from 'url';
+import {parseSrcset} from 'srcset';
 
 const linksAttr = {
   background: ['body'],
@@ -115,9 +116,7 @@ function isAbsoluteUrl(url: string): boolean {
 function parseAttr(name: string, value: string): string[] {
   switch (name) {
     case 'srcset':
-      return value
-        .split(',')
-        .map((pair: string) => pair.trim().split(/\s+/)[0]);
+      return parseSrcset(value).map(p => p.url);
     default:
       return [value];
   }

test/test.index.ts

@@ -228,6 +228,12 @@ describe('linkinator', () => {
     );
   });
 
+  it('should work with picture elements', async () => {
+    const results = await check({path: 'test/fixtures/picture'});
+    assert.ok(results.passed);
+    assert.strictEqual(results.links.length, 4);
+  });
+
   it('should not recurse by default', async () => {
     const results = await check({path: 'test/fixtures/recurse'});
     assert.strictEqual(results.links.length, 2);