knip 5.70.0 → 5.70.1

package/dist/typescript/pragmas/custom.js

@@ -17,7 +17,7 @@ export const collectCustomImports = (sourceFile) => {
             const isLocal = isInternal(id) || isAbsolute(id);
             const modifiers = isLocal ? IMPORT_MODIFIERS.ENTRY : IMPORT_MODIFIERS.NONE;
             const offset = match[0].length - match[2].length;
-            const specifier = isLocal ? id : getEnvSpecifier(id);
+            const specifier = isLocal || id === 'node' ? id : getEnvSpecifier(id);
             importNodes.push({ specifier, identifier: undefined, pos: comment.pos + match.index + offset, modifiers });
         }
     }

package/dist/typescript/visitors/dynamic-imports/jsDocType.js

@@ -1,5 +1,5 @@
 import ts from 'typescript';
-import { IMPORT_MODIFIERS } from '../../../constants.js';
+import { IMPORT_MODIFIERS, IMPORT_STAR } from '../../../constants.js';
 import { importVisitor as visit } from '../index.js';
 const supportsJSDocImportTag = 'isJSDocImportTag' in ts;
 const getImportSpecifiers = (node) => {
@@ -12,19 +12,33 @@ const getImportSpecifiers = (node) => {
                 imports.push({
                     specifier: importClause.literal.text,
                     identifier,
-                    pos: importClause.literal.pos,
+                    pos: node.qualifier?.getStart() ?? importClause.literal.pos,
                     modifiers: IMPORT_MODIFIERS.NONE,
                 });
             }
         }
         if (supportsJSDocImportTag && ts.isJSDocImportTag(node) && ts.isStringLiteralLike(node.moduleSpecifier)) {
-            const moduleSpecifier = node.moduleSpecifier;
-            imports.push({
-                specifier: moduleSpecifier.text,
-                identifier: undefined,
-                pos: moduleSpecifier.pos,
-                modifiers: IMPORT_MODIFIERS.NONE,
-            });
+            if (node.importClause?.namedBindings && ts.isNamedImportBindings(node.importClause.namedBindings)) {
+                const bindings = node.importClause.namedBindings;
+                if (ts.isNamespaceImport(bindings)) {
+                    imports.push({
+                        specifier: node.moduleSpecifier.text,
+                        identifier: IMPORT_STAR,
+                        pos: bindings.name.getStart(),
+                        modifiers: IMPORT_MODIFIERS.TYPE_ONLY,
+                    });
+                }
+                else {
+                    for (const element of bindings.elements) {
+                        imports.push({
+                            specifier: node.moduleSpecifier.text,
+                            identifier: String((element.propertyName ?? element.name).escapedText),
+                            pos: element.name.getStart(),
+                            modifiers: IMPORT_MODIFIERS.TYPE_ONLY,
+                        });
+                    }
+                }
+            }
         }
         ts.forEachChild(node, visit);
     }

package/dist/typescript/visitors/dynamic-imports/requireCall.js

@@ -15,7 +15,7 @@ export default visit(() => true, node => {
                 });
                 if (propertyAccessExpression) {
                     const identifier = String(propertyAccessExpression.name.escapedText);
-                    return { identifier, specifier, pos: propertyAccessExpression.name.pos, modifiers };
+                    return { identifier, specifier, pos: propertyAccessExpression.name.getStart(), modifiers };
                 }
                 const variableDeclaration = node.parent;
                 if (ts.isVariableDeclaration(variableDeclaration) &&
@@ -28,7 +28,7 @@ export default visit(() => true, node => {
                             alias,
                             symbol: isTLA ? variableDeclaration.symbol : undefined,
                             specifier,
-                            pos: node.arguments[0].pos,
+                            pos: variableDeclaration.name.getStart(),
                             modifiers,
                         };
                     }
@@ -38,7 +38,7 @@ export default visit(() => true, node => {
                             const identifier = (element.propertyName ?? element.name).getText();
                             const alias = element.propertyName ? element.name.getText() : undefined;
                             const symbol = isTLA ? element.symbol : undefined;
-                            return { identifier, specifier, alias, symbol, pos: element.pos, modifiers };
+                            return { identifier, specifier, alias, symbol, pos: element.name.getStart(), modifiers };
                         });
                     }
                     return { identifier: 'default', specifier, pos: node.arguments[0].pos, modifiers };
@@ -53,7 +53,10 @@ export default visit(() => true, node => {
                         modifiers: IMPORT_MODIFIERS.RE_EXPORT,
                     };
                 }
-                return { identifier: 'default', specifier, pos: node.arguments[0].pos, modifiers };
+                if (ts.isCallExpression(node.parent)) {
+                    return { identifier: 'default', specifier, pos: node.getEnd(), modifiers };
+                }
+                return { identifier: 'default', specifier, pos: node.getStart(), modifiers };
             }
         }
     }

package/dist/typescript/visitors/imports/reExportDeclaration.js

@@ -8,7 +8,7 @@ export default visit(() => true, node => {
                 return {
                     identifier: IMPORT_STAR,
                     specifier: node.moduleSpecifier.text,
-                    pos: node.pos,
+                    pos: node.moduleSpecifier.getStart() - 7,
                     modifiers: IMPORT_MODIFIERS.RE_EXPORT,
                 };
             }
@@ -17,7 +17,7 @@ export default visit(() => true, node => {
                     identifier: IMPORT_STAR,
                     namespace: String(node.exportClause.name.text),
                     specifier: node.moduleSpecifier.text,
-                    pos: node.exportClause.name.pos,
+                    pos: node.exportClause.name.getStart(),
                     modifiers: IMPORT_MODIFIERS.RE_EXPORT,
                 };
             }
@@ -28,14 +28,14 @@ export default visit(() => true, node => {
                         identifier: String(element.propertyName.text),
                         alias: String(element.name.text),
                         specifier: specifier.text,
-                        pos: element.pos,
+                        pos: element.propertyName.getStart(),
                         modifiers: IMPORT_MODIFIERS.RE_EXPORT,
                     };
                 }
                 return {
                     identifier: (element.propertyName ?? element.name).getText(),
                     specifier: specifier.text,
-                    pos: element.pos,
+                    pos: element.name.getStart(),
                     modifiers: IMPORT_MODIFIERS.RE_EXPORT,
                 };
             });

package/dist/version.d.ts

@@ -1 +1 @@
-export declare const version = "5.70.0";
+export declare const version = "5.70.1";

package/dist/version.js

@@ -1 +1 @@
-export const version = '5.70.0';
+export const version = '5.70.1';

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "knip",
-  "version": "5.70.0",
+  "version": "5.70.1",
   "description": "Find and fix unused dependencies, exports and files in your TypeScript and JavaScript projects",
   "homepage": "https://knip.dev",
   "repository": {