fix(cli): skip protocol imports in --install (#1464)

Author: AnnasMazhar

.size-limit.json

@@ -33,7 +33,7 @@
       "build/globals.js",
       "build/deno.js"
     ],
-    "limit": "881.40 kB",
+    "limit": "881.42 kB",
     "brotli": false,
     "gzip": false
   },
@@ -66,7 +66,7 @@
       "README.md",
       "LICENSE"
     ],
-    "limit": "944.14 kB",
+    "limit": "944.17 kB",
     "brotli": false,
     "gzip": false
   }

build/deps.cjs

@@ -62,7 +62,7 @@ function parseDeps(content) {
 }
 function parsePackageName(path) {
   var _a, _b;
-  if (!path) return;
+  if (!path || path.includes(":")) return;
   const name = (_b = (_a = nameRe.exec(path)) == null ? void 0 : _a.groups) == null ? void 0 : _b.name;
   if (name && !builtins.has(name)) return name;
 }

src/deps.ts

@@ -86,7 +86,7 @@ export function parseDeps(content: string): Record<string, string> {
 }
 
 function parsePackageName(path?: string): string | undefined {
-  if (!path) return
+  if (!path || path.includes(':')) return
 
   const name = nameRe.exec(path)?.groups?.name
   if (name && !builtins.has(name)) return name

test/deps.test.js

@@ -120,6 +120,12 @@ describe('deps', () => {
         [`require('@')`, {}],
         [`require('@/_foo')`, {}],
         [`require('@foo')`, {}],
+        // ignores protocol specifiers
+        [`import fs from 'node:fs'`, {}],
+        [`require('node:path')`, {}],
+        [`import('node:crypto')`, {}],
+        [`import { promises } from 'node:fs/promises'`, {}],
+        [`import * as assert from 'node:assert/strict'`, {}],
       ].forEach(([input, result]) => {
         assert.deepEqual(parseDeps(input), result)
       })