Fix case sensitivity in package name check (resolves #869)

webpro-nl · Dec 3, 2024 · d6dab3d · boneskull · Dec 3, 2024 · webpro
1 parent 35d38a2
commit d6dab3d
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 2 deletions.
diff --git a/packages/knip/fixtures/dependencies/my-module.ts b/packages/knip/fixtures/dependencies/my-module.ts
@@ -1,4 +1,7 @@
 import has from 'has';
+import JSONStream from 'JSONStream';
+
+JSONStream;
 
 async function main() {
   return [require('ansi-regex'), import('ansi-regex')];

diff --git a/packages/knip/fixtures/dependencies/package.json b/packages/knip/fixtures/dependencies/package.json
@@ -9,7 +9,8 @@
     "@sindresorhus/is": "*",
     "fs-extra": "*",
     "has": "*",
-    "stream": "*"
+    "stream": "*",
+    "JSONStream": "*"
   },
   "peerDependencies": {
     "ansi-regex": "*",

diff --git a/packages/knip/src/util/modules.ts b/packages/knip/src/util/modules.ts
@@ -19,7 +19,7 @@ export const getPackageNameFromFilePath = (value: string) => {
 export const getPackageNameFromSpecifier = (specifier: string) =>
   isInNodeModules(specifier) ? getPackageNameFromFilePath(specifier) : getPackageNameFromModuleSpecifier(specifier);
 
-export const isStartsLikePackageName = (specifier: string) => /^(@[a-z0-9._]|[a-z0-9])/.test(specifier);
+export const isStartsLikePackageName = (specifier: string) => /^(@[a-z0-9._]|[a-z0-9])/i.test(specifier);
 
 export const stripVersionFromSpecifier = (specifier: string) => specifier.replace(/(\S+)@.*/, '$1');