@cplieger/deadset-ts 7.2.0-dev.1 → 7.2.0-dev.2

package/README.md

@@ -1,6 +1,6 @@
 # deadset-ts
 
-[![npm](https://img.shields.io/npm/v/@cplieger/deadset-ts)](https://www.npmjs.com/package/@cplieger/deadset-ts) [![JSR](https://jsr.io/badges/@cplieger/deadset-ts)](https://jsr.io/@cplieger/deadset-ts)
+[![npm](https://img.shields.io/npm/v/@cplieger/deadset-ts)](https://www.npmjs.com/package/@cplieger/deadset-ts) [![JSR](https://jsr.io/badges/@cplieger/deadset-ts)](https://jsr.io/@cplieger/deadset-ts) [![Mutation (TS)](https://img.shields.io/endpoint?url=https://raw.githubusercontent.com/cplieger/deadset-ts/badges/mutation-ts.json)](https://github.com/cplieger/deadset-ts/issues?q=label%3Astryker-tracker)
 
 deadset-ts finds dead code in TypeScript projects, down to class members, type members and dead stores inside functions, and writes a report your CI can fail on.
 

package/dist/src/dependencies.js

@@ -466,15 +466,11 @@ function skipWhitespace(text, at) {
  */
 function membersAt(text, at) {
     const found = new Map();
-    if (text.charAt(at) !== "{") {
-        return found;
-    }
     let cursor = skipWhitespace(text, at + 1);
     while (text.charAt(cursor) === '"') {
         const keyEnd = valueEnd(text, cursor);
         const key = JSON.parse(text.slice(cursor, keyEnd));
         const value = skipWhitespace(text, skipWhitespace(text, keyEnd) + 1);
-        found.delete(key);
         found.set(key, { key: cursor, value });
         cursor = skipWhitespace(text, valueEnd(text, value));
         if (text.charAt(cursor) === ",") {

package/dist/src/deprecation.js

@@ -47,7 +47,7 @@ export function deprecatedDeclarations(project, held) {
                 }
             }
             const id = held.declarations.get(nodeKey(file, node));
-            if (id !== undefined && node !== file) {
+            if (id !== undefined) {
                 named.push({ id, handle: project.handle(nameNode(node)) });
             }
             node.forEachChild(visit);

package/dist/src/findings/completion.js

@@ -145,7 +145,7 @@ function findingComponents(swept, cascade) {
                 },
             ];
     })
-        .sort((a, b) => byPosition(a.position, b.position) || (a.ref === b.ref ? 0 : a.ref < b.ref ? -1 : 1));
+        .sort((a, b) => byPosition(a.position, b.position));
     return (id) => {
         const component = held.get(id);
         return component === undefined

package/dist/src/findings/interfaces.js

@@ -10,13 +10,6 @@ const UNCALLED_MESSAGE = "no call site invokes or selects the method through the
 function positionOf(symbol) {
     return { ...symbol.position, endLine: Math.max(symbol.position.line, symbol.endLine) };
 }
-/** Two strings ordered bytewise. */
-function compare(a, b) {
-    if (a === b) {
-        return 0;
-    }
-    return a < b ? -1 : 1;
-}
 /** One run's declarations and candidates, indexed for the two kinds. */
 function indexed(input) {
     const union = input.swept.matrix.union;
@@ -31,7 +24,7 @@ function indexed(input) {
             .map(symbolOf)
             .filter((symbol) => symbol !== undefined)
             .map((symbol) => ({ ref: symbol.ref, name: symbol.name, position: positionOf(symbol) }))
-            .sort((a, b) => byPosition(a.position, b.position) || compare(a.ref, b.ref)),
+            .sort((a, b) => byPosition(a.position, b.position)),
     };
 }
 /** The finding one code makes about one declaration. */

package/dist/src/findings/unused-declarations.js

@@ -39,19 +39,17 @@ const WORDS = {
 const TEST_OF_DEAD_CODE = "every target symbol this test references is reported dead";
 /**
  * The code one candidate is reported under, or none, each symbol once under the most
- * specific code: a test of dead code first; no other test-file declaration, member of a
- * dead container or enum member nothing names; a test-support declaration test code
- * references as test-only use; then deprecation, a test reference, and the unreferenced
- * codes, an unimportable unused export being the unreachable kind's.
+ * specific code: a test of dead code first; no other test-file declaration or enum
+ * member nothing names; a test-support declaration test code references as test-only
+ * use; then deprecation, a test reference, and the unreferenced codes, an unimportable
+ * unused export being the unreachable kind's.
  */
 function codeOf(judged) {
     const { candidate, symbol } = judged;
     if (candidate.testOfDeadCode) {
         return "DS1005";
     }
-    if ((judged.test && !candidate.unreferencedTest) ||
-        judged.containerDead ||
-        NOT_THIS_FAMILY.has(symbol.kind)) {
+    if ((judged.test && !candidate.unreferencedTest) || NOT_THIS_FAMILY.has(symbol.kind)) {
         return undefined;
     }
     const referenced = candidate.productionRefs + candidate.testRefs > 0;
@@ -192,7 +190,6 @@ function familyFindings(input, fallen) {
             candidate,
             symbol,
             test: union.test[at] === true,
-            containerDead: containerDead && !fallen,
             deprecated: input.deprecated.has(symbol.id),
             unreachable: unreachable.has(symbol.id),
         };

package/dist/src/html-entries.js

@@ -102,9 +102,6 @@ function manifestDirectory(host, root, dir) {
         if (host.kindOf(joinPath(at, "package.json")) === "file") {
             return at;
         }
-        if (at === root) {
-            break;
-        }
     }
     return root;
 }

package/dist/src/intra-function-parts.js

@@ -303,7 +303,6 @@ function readsBinding(use, declared, ids) {
 export function projectParts(project, held, references, targetRoot) {
     const files = project.ownSourceFiles();
     const extended = extendedNames(files);
-    const kinds = new Map(held.symbols.map((symbol) => [symbol.id, symbol.kind]));
     const parts = [];
     const signatures = [];
     const calls = new Map();
@@ -313,12 +312,7 @@ export function projectParts(project, held, references, targetRoot) {
     const contextual = [];
     for (const file of files) {
         // A parameter property's own parts stay with the parameter when its member goes.
-        const declarationAt = (node) => {
-            const id = held.declarations.get(nodeKey(file, node));
-            return id === undefined || kinds.get(id) === "file" || isParameterProperty(node)
-                ? undefined
-                : id;
-        };
+        const declarationAt = (node) => isParameterProperty(node) ? undefined : held.declarations.get(nodeKey(file, node));
         // A part no declaration holds belongs to the file, which is judged while the file is live.
         const fileId = held.declarations.get(nodeKey(file, file));
         const holderOf = (node) => {
@@ -560,10 +554,6 @@ function valueSignature(node) {
 function parameterNames(signature) {
     const found = [];
     const patternNames = (pattern, at) => {
-        if (isIdentifier(pattern)) {
-            found.push({ name: pattern, at, pattern: true, exempt: false });
-            return;
-        }
         const rest = isObjectBindingPattern(pattern) &&
             pattern.elements.some((element) => element.dotDotDotToken !== undefined);
         for (const element of pattern.elements) {
@@ -741,9 +731,7 @@ function deadStores(file, root, block, holder, locals, ids) {
         for (const write of writes) {
             // The rest of the write's own statement runs after it: a later declarator may read the local.
             const own = statements[write.at];
-            const sameStatement = own !== undefined && write.target === local.name
-                ? usesIn(own).filter((use) => use !== local.name)
-                : [];
+            const sameStatement = own !== undefined && write.target === local.name ? usesIn(own) : [];
             if (sameStatement.length > 0) {
                 continue;
             }

package/dist/src/inventory.js

@@ -733,9 +733,7 @@ export function inventory(project, host, targetRoot) {
         });
     }
     const byId = new Map(building.map((record) => [positionKey(record.position), record]));
-    const symbols = building
-        .map(render)
-        .sort((a, b) => byPosition(a.position, b.position) || (a.ref < b.ref ? -1 : 1));
+    const symbols = building.map(render).sort((a, b) => byPosition(a.position, b.position));
     return {
         symbols,
         declarations: new Map([...byNode].map(([key, record]) => [key, positionKey(record.position)])),

package/dist/src/matrix.js

@@ -7,13 +7,6 @@ import { componentsOf } from "./components.js";
 import { graphOf, OUTSIDE } from "./graph.js";
 import { byPosition } from "./position.js";
 import { reachedWithTheTests, sweep, } from "./sweep.js";
-/** Two strings ordered bytewise. */
-function compare(a, b) {
-    if (a === b) {
-        return 0;
-    }
-    return a < b ? -1 : 1;
-}
 /**
  * What makes one reference one reference, whichever configuration made it. A consumer's
  * position is rendered against its own root, so the consumer is part of what it names.
@@ -76,7 +69,7 @@ export function matrixOf(per) {
             references.push(reference);
         }
     });
-    symbols.sort((a, b) => byPosition(a.position, b.position) || compare(a.ref, b.ref));
+    symbols.sort((a, b) => byPosition(a.position, b.position));
     return {
         configurations: per.map((one) => one.configuration),
         graphs: per.map((one) => graphOf(one.symbols, one.references, one.roots, one.testFiles, one.supportFiles)),

package/dist/src/module-declarations.js

@@ -79,9 +79,6 @@ function namesModule(specifier, name) {
  * the members that hold the export names given.
  */
 function namedBy(held) {
-    if (held.modules.size === 0) {
-        return () => [];
-    }
     const refs = new Map();
     const children = new Map();
     for (const symbol of held.symbols) {

package/dist/src/type-errors.js

@@ -84,7 +84,8 @@ function declaringManifest(host, dir, targetRoot, name) {
                 }
             }
         }
-        if (at === targetRoot || relativePath(targetRoot, at) === undefined || dirnamePath(at) === at) {
+        // The target root is not below itself, so the walk stops after reading its manifest.
+        if (relativePath(targetRoot, at) === undefined) {
             return undefined;
         }
     }

package/dist/src/workspace-resolution.js

@@ -19,7 +19,6 @@ const DEPENDENCY_MEMBERS = [
     "optionalDependencies",
 ];
 const SOURCE = /(?<!\.d)\.(?:[mc]?ts|tsx)$/u;
-const OUTPUT = /(?:\.d\.[mc]?ts|\.[mc]?js|\.jsx)$/u;
 function isRecord(value) {
     return typeof value === "object" && value !== null && !Array.isArray(value);
 }
@@ -39,7 +38,7 @@ export function packageNameOf(specifier) {
     return first;
 }
 function inside(dir, path) {
-    return path === dir || relativePath(dir, path) !== undefined;
+    return relativePath(dir, path) !== undefined;
 }
 /** The `exports` entry one subpath selects, with what a pattern's `*` matched. */
 function exportsEntry(exportsField, subpath) {
@@ -199,12 +198,10 @@ export function workspaceResolver(host, workspace) {
                 return found;
             }
         }
-        if (OUTPUT.test(path)) {
-            const beside = emittedFrom(path, { sourceDirs: [] });
-            const found = beside.find((candidate) => host.kindOf(candidate) === "file");
-            if (found !== undefined) {
-                return found;
-            }
+        const beside = emittedFrom(path, { sourceDirs: [] });
+        const found = beside.find((candidate) => host.kindOf(candidate) === "file");
+        if (found !== undefined) {
+            return found;
         }
         return compiled(member, path) && host.kindOf(path) === "file" ? path : undefined;
     };

package/dist/src/workspace.js

@@ -198,14 +198,14 @@ function memberDirectories(host, root, patterns) {
     }
     return [...included].filter((dir) => {
         const below = relativePath(root, dir);
-        const segments = below === undefined || below === "." || below === "" ? [] : below.split("/");
+        const segments = below === undefined ? [] : below.split("/");
         return dir === root || !excluded.some((pattern) => pathMatches(pattern, segments));
     });
 }
 /** Whether a workspace's patterns name one directory below its root, read without listing any. */
 function namesDirectory(root, patterns, dir) {
     const below = relativePath(root, dir);
-    if (below === undefined || below === "" || below.startsWith("..")) {
+    if (below === undefined || below.startsWith("..")) {
         return false;
     }
     const path = below.split("/");

package/jsr.json

@@ -1,6 +1,6 @@
 {
   "name": "@cplieger/deadset-ts",
-  "version": "7.2.0-dev.1",
+  "version": "7.2.0-dev.2",
   "license": "GPL-3.0-or-later",
   "exports": {
     ".": "./src/index.ts"

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cplieger/deadset-ts",
-  "version": "7.2.0-dev.1",
+  "version": "7.2.0-dev.2",
   "type": "module",
   "sideEffects": false,
   "description": "Deterministic dead-code analysis for TypeScript, class and type members included",
@@ -40,7 +40,7 @@
     "@stryker-mutator/core": "10.0.0",
     "@stryker-mutator/typescript-checker": "10.0.0",
     "@stryker-mutator/vitest-runner": "10.0.0",
-    "@types/node": "24.19.1",
+    "@types/node": "24.19.2",
     "eslint": "10.12.0",
     "fast-check": "4.10.2",
     "prettier": "3.9.9",

package/src/dependencies.ts

@@ -616,15 +616,11 @@ function skipWhitespace(text: string, at: number): number {
  */
 function membersAt(text: string, at: number): Map<string, { key: number; value: number }> {
   const found = new Map<string, { key: number; value: number }>();
-  if (text.charAt(at) !== "{") {
-    return found;
-  }
   let cursor = skipWhitespace(text, at + 1);
   while (text.charAt(cursor) === '"') {
     const keyEnd = valueEnd(text, cursor);
     const key = JSON.parse(text.slice(cursor, keyEnd)) as string;
     const value = skipWhitespace(text, skipWhitespace(text, keyEnd) + 1);
-    found.delete(key);
     found.set(key, { key: cursor, value });
     cursor = skipWhitespace(text, valueEnd(text, value));
     if (text.charAt(cursor) === ",") {

package/src/deprecation.ts

@@ -73,7 +73,7 @@ export function deprecatedDeclarations<Brand>(
         }
       }
       const id = held.declarations.get(nodeKey(file, node));
-      if (id !== undefined && node !== file) {
+      if (id !== undefined) {
         named.push({ id, handle: project.handle(nameNode(node)) });
       }
       node.forEachChild(visit);

package/src/findings/completion.ts

@@ -181,10 +181,7 @@ function findingComponents(
               },
             ];
       })
-      .sort(
-        (a, b) =>
-          byPosition(a.position, b.position) || (a.ref === b.ref ? 0 : a.ref < b.ref ? -1 : 1),
-      );
+      .sort((a, b) => byPosition(a.position, b.position));
   return (id) => {
     const component = held.get(id);
     return component === undefined

package/src/findings/interfaces.ts

@@ -19,14 +19,6 @@ function positionOf(symbol: InventorySymbol): FindingPosition {
   return { ...symbol.position, endLine: Math.max(symbol.position.line, symbol.endLine) };
 }
 
-/** Two strings ordered bytewise. */
-function compare(a: string, b: string): number {
-  if (a === b) {
-    return 0;
-  }
-  return a < b ? -1 : 1;
-}
-
 /** One run's declarations and candidates, indexed for the two kinds. */
 function indexed(input: EmitterInput): {
   readonly symbolOf: (id: string) => InventorySymbol | undefined;
@@ -49,7 +41,7 @@ function indexed(input: EmitterInput): {
         .map(symbolOf)
         .filter((symbol): symbol is InventorySymbol => symbol !== undefined)
         .map((symbol) => ({ ref: symbol.ref, name: symbol.name, position: positionOf(symbol) }))
-        .sort((a, b) => byPosition(a.position, b.position) || compare(a.ref, b.ref)),
+        .sort((a, b) => byPosition(a.position, b.position)),
   };
 }
 

package/src/findings/unused-declarations.ts

@@ -52,8 +52,6 @@ interface Judged {
   readonly symbol: InventorySymbol;
   /** Whether a test file declares it. */
   readonly test: boolean;
-  /** Whether its container is itself dead, so it falls with the container. */
-  readonly containerDead: boolean;
   readonly deprecated: boolean;
   /** Whether the unreachable-export kind reports it. */
   readonly unreachable: boolean;
@@ -61,21 +59,17 @@ interface Judged {
 
 /**
  * The code one candidate is reported under, or none, each symbol once under the most
- * specific code: a test of dead code first; no other test-file declaration, member of a
- * dead container or enum member nothing names; a test-support declaration test code
- * references as test-only use; then deprecation, a test reference, and the unreferenced
- * codes, an unimportable unused export being the unreachable kind's.
+ * specific code: a test of dead code first; no other test-file declaration or enum
+ * member nothing names; a test-support declaration test code references as test-only
+ * use; then deprecation, a test reference, and the unreferenced codes, an unimportable
+ * unused export being the unreachable kind's.
  */
 function codeOf(judged: Judged): string | undefined {
   const { candidate, symbol } = judged;
   if (candidate.testOfDeadCode) {
     return "DS1005";
   }
-  if (
-    (judged.test && !candidate.unreferencedTest) ||
-    judged.containerDead ||
-    NOT_THIS_FAMILY.has(symbol.kind)
-  ) {
+  if ((judged.test && !candidate.unreferencedTest) || NOT_THIS_FAMILY.has(symbol.kind)) {
     return undefined;
   }
   const referenced = candidate.productionRefs + candidate.testRefs > 0;
@@ -233,7 +227,6 @@ function familyFindings(input: EmitterInput, fallen: boolean): Finding[] {
       candidate,
       symbol,
       test: union.test[at] === true,
-      containerDead: containerDead && !fallen,
       deprecated: input.deprecated.has(symbol.id),
       unreachable: unreachable.has(symbol.id),
     };

package/src/html-entries.ts

@@ -131,9 +131,6 @@ function manifestDirectory(host: Host, root: string, dir: string): string {
     if (host.kindOf(joinPath(at, "package.json")) === "file") {
       return at;
     }
-    if (at === root) {
-      break;
-    }
   }
   return root;
 }

package/src/intra-function-parts.ts

@@ -529,7 +529,6 @@ export function projectParts<Brand>(
 ): ProjectParts {
   const files = project.ownSourceFiles();
   const extended = extendedNames(files);
-  const kinds = new Map(held.symbols.map((symbol) => [symbol.id, symbol.kind]));
   const parts: PartFact[] = [];
   const signatures: SignatureFact[] = [];
   const calls = new Map<string, CallUse>();
@@ -540,12 +539,8 @@ export function projectParts<Brand>(
 
   for (const file of files) {
     // A parameter property's own parts stay with the parameter when its member goes.
-    const declarationAt = (node: Node): string | undefined => {
-      const id = held.declarations.get(nodeKey(file, node));
-      return id === undefined || kinds.get(id) === "file" || isParameterProperty(node)
-        ? undefined
-        : id;
-    };
+    const declarationAt = (node: Node): string | undefined =>
+      isParameterProperty(node) ? undefined : held.declarations.get(nodeKey(file, node));
     // A part no declaration holds belongs to the file, which is judged while the file is live.
     const fileId = held.declarations.get(nodeKey(file, file));
     const holderOf = (node: Node): string | undefined => {
@@ -804,11 +799,7 @@ function valueSignature(node: Node): SignatureNode | undefined {
  */
 function parameterNames(signature: SignatureNode): ParameterName[] {
   const found: ParameterName[] = [];
-  const patternNames = (pattern: BindingName, at: number): void => {
-    if (isIdentifier(pattern)) {
-      found.push({ name: pattern, at, pattern: true, exempt: false });
-      return;
-    }
+  const patternNames = (pattern: Exclude<BindingName, Identifier>, at: number): void => {
     const rest =
       isObjectBindingPattern(pattern) &&
       pattern.elements.some((element) => element.dotDotDotToken !== undefined);
@@ -1039,10 +1030,7 @@ function deadStores(
     for (const write of writes) {
       // The rest of the write's own statement runs after it: a later declarator may read the local.
       const own = statements[write.at];
-      const sameStatement =
-        own !== undefined && write.target === local.name
-          ? usesIn(own).filter((use) => use !== local.name)
-          : [];
+      const sameStatement = own !== undefined && write.target === local.name ? usesIn(own) : [];
       if (sameStatement.length > 0) {
         continue;
       }

package/src/inventory.ts

@@ -1042,9 +1042,7 @@ export function inventory<Brand>(
   }
 
   const byId = new Map(building.map((record) => [positionKey(record.position), record]));
-  const symbols = building
-    .map(render)
-    .sort((a, b) => byPosition(a.position, b.position) || (a.ref < b.ref ? -1 : 1));
+  const symbols = building.map(render).sort((a, b) => byPosition(a.position, b.position));
 
   return {
     symbols,

package/src/matrix.ts

@@ -94,14 +94,6 @@ export interface SweepResult {
   readonly components: readonly Component[];
 }
 
-/** Two strings ordered bytewise. */
-function compare(a: string, b: string): number {
-  if (a === b) {
-    return 0;
-  }
-  return a < b ? -1 : 1;
-}
-
 /**
  * What makes one reference one reference, whichever configuration made it. A consumer's
  * position is rendered against its own root, so the consumer is part of what it names.
@@ -163,7 +155,7 @@ export function matrixOf(per: readonly Configured[]): Matrix {
       references.push(reference);
     }
   });
-  symbols.sort((a, b) => byPosition(a.position, b.position) || compare(a.ref, b.ref));
+  symbols.sort((a, b) => byPosition(a.position, b.position));
 
   return {
     configurations: per.map((one) => one.configuration),

package/src/module-declarations.ts

@@ -113,9 +113,6 @@ interface Named {
 function namedBy(
   held: Inventory,
 ): (specifier: string, names: readonly string[]) => readonly Named[] {
-  if (held.modules.size === 0) {
-    return () => [];
-  }
   const refs = new Map<string, string>();
   const children = new Map<string, InventorySymbol[]>();
   for (const symbol of held.symbols) {

package/src/type-errors.ts

@@ -151,7 +151,8 @@ function declaringManifest(
         }
       }
     }
-    if (at === targetRoot || relativePath(targetRoot, at) === undefined || dirnamePath(at) === at) {
+    // The target root is not below itself, so the walk stops after reading its manifest.
+    if (relativePath(targetRoot, at) === undefined) {
       return undefined;
     }
   }

package/src/workspace-resolution.ts

@@ -45,7 +45,6 @@ const DEPENDENCY_MEMBERS = [
 ] as const;
 
 const SOURCE = /(?<!\.d)\.(?:[mc]?ts|tsx)$/u;
-const OUTPUT = /(?:\.d\.[mc]?ts|\.[mc]?js|\.jsx)$/u;
 
 function isRecord(value: unknown): value is Record<string, unknown> {
   return typeof value === "object" && value !== null && !Array.isArray(value);
@@ -70,7 +69,7 @@ export function packageNameOf(specifier: string): string | undefined {
 }
 
 function inside(dir: string, path: string): boolean {
-  return path === dir || relativePath(dir, path) !== undefined;
+  return relativePath(dir, path) !== undefined;
 }
 
 /** The `exports` entry one subpath selects, with what a pattern's `*` matched. */
@@ -274,12 +273,10 @@ export function workspaceResolver(host: Host, workspace: Workspace): WorkspaceRe
         return found;
       }
     }
-    if (OUTPUT.test(path)) {
-      const beside = emittedFrom(path, { sourceDirs: [] });
-      const found = beside.find((candidate) => host.kindOf(candidate) === "file");
-      if (found !== undefined) {
-        return found;
-      }
+    const beside = emittedFrom(path, { sourceDirs: [] });
+    const found = beside.find((candidate) => host.kindOf(candidate) === "file");
+    if (found !== undefined) {
+      return found;
     }
     return compiled(member, path) && host.kindOf(path) === "file" ? path : undefined;
   };

package/src/workspace.ts

@@ -266,7 +266,7 @@ function memberDirectories(
   }
   return [...included].filter((dir) => {
     const below = relativePath(root, dir);
-    const segments = below === undefined || below === "." || below === "" ? [] : below.split("/");
+    const segments = below === undefined ? [] : below.split("/");
     return dir === root || !excluded.some((pattern) => pathMatches(pattern, segments));
   });
 }
@@ -274,7 +274,7 @@ function memberDirectories(
 /** Whether a workspace's patterns name one directory below its root, read without listing any. */
 function namesDirectory(root: string, patterns: readonly string[], dir: string): boolean {
   const below = relativePath(root, dir);
-  if (below === undefined || below === "" || below.startsWith("..")) {
+  if (below === undefined || below.startsWith("..")) {
     return false;
   }
   const path = below.split("/");