Update formatting, fix tests

webpro-nl · Nov 24, 2024 · 19616a9 · 19616a9
1 parent d84b0db
commit 19616a9
Show file tree

Hide file tree

Showing 3 changed files with 21 additions and 21 deletions.
diff --git a/packages/knip/src/ProjectPrincipal.ts b/packages/knip/src/ProjectPrincipal.ts
@@ -253,10 +253,15 @@ export class ProjectPrincipal {
 
     const resolve = (specifier: string) => this.backend.resolveModuleNames([specifier], sourceFile.fileName)[0];
 
-    const { imports, exports, scripts, traceRefs, jsxComponents } = _getImportsAndExports(sourceFile, resolve, typeChecker, {
-      ...options,
-      skipExports,
-    });
+    const { imports, exports, scripts, traceRefs, jsxComponents } = _getImportsAndExports(
+      sourceFile,
+      resolve,
+      typeChecker,
+      {
+        ...options,
+        skipExports,
+      }
+    );
 
     const { internal, resolved, specifiers, unresolved, external } = imports;
 

diff --git a/packages/knip/src/typescript/create-hosts.ts b/packages/knip/src/typescript/create-hosts.ts
@@ -52,23 +52,18 @@ export const createHosts = ({
         // Try to resolve @types for any non-relative, non-absolute import
         if (!moduleName.startsWith('.') && !moduleName.startsWith('/') && !moduleName.startsWith('node:')) {
           // Get the base package name (handle scoped packages and subpaths)
-          const basePackage = moduleName.startsWith('@') 
+          const basePackage = moduleName.startsWith('@')
             ? moduleName.split('/').slice(0, 2).join('/')
             : moduleName.split('/')[0];
-
-          const typeResult = ts.resolveTypeReferenceDirective(
-            basePackage,
-            containingFile,
-            options,
-            {
-              fileExists: ts.sys.fileExists,
-              readFile: ts.sys.readFile,
-              directoryExists: ts.sys.directoryExists,
-              getCurrentDirectory: () => cwd,
-              getDirectories: ts.sys.getDirectories,
-            }
-          );
-
+
+          const typeResult = ts.resolveTypeReferenceDirective(basePackage, containingFile, options, {
+            fileExists: ts.sys.fileExists,
+            readFile: ts.sys.readFile,
+            directoryExists: ts.sys.directoryExists,
+            getCurrentDirectory: () => cwd,
+            getDirectories: ts.sys.getDirectories,
+          });
+
           if (typeResult.resolvedTypeReferenceDirective?.resolvedFileName) {
             return {
               resolvedFileName: typeResult.resolvedTypeReferenceDirective.resolvedFileName,
@@ -77,7 +72,7 @@ export const createHosts = ({
             };
           }
         }
-        
+
         // Fall back to normal module resolution
         return resolveModuleNames([moduleName], containingFile)[0];
       });

diff --git a/packages/knip/test/react-props.test.ts b/packages/knip/test/react-props.test.ts
@@ -31,6 +31,6 @@ test('Find unused component props', async () => {
     processed: 3,
     total: 3,
     types: 1,
-    reactComponentProps: totalErrors,
+    componentProps: totalErrors,
   });
 });