@cplieger/deadset-ts 5.2.1 → 5.2.3

package/README.md

@@ -4,7 +4,7 @@
 
 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.
 
-It reads your `tsconfig.json` projects, `allowJs` JavaScript and Vue, Svelte and Astro scripts included, with the type information of TypeScript 7.1.0-dev.20261003.1, its one dependency. Your project keeps its own TypeScript version. A type error under that version skips the function or statement that holds it. deadset-ts reports and never edits your code. It is pre-release, tested only on Linux, needs Node.js 24 or later and is licensed under GPL-3.0-or-later.
+It reads your `tsconfig.json` projects, `allowJs` JavaScript and Vue, Svelte and Astro scripts included, with the type information of TypeScript 7.1.0-dev.20261006.1, its one dependency. Your project keeps its own TypeScript version. A type error under that version skips the function or statement that holds it. deadset-ts reports and never edits your code. It is pre-release, tested only on Linux, needs Node.js 24 or later and is licensed under GPL-3.0-or-later.
 
 ## Why use it
 

package/dist/src/convention-rows.js

@@ -31,6 +31,7 @@ const APP_DIRECTORY_ENTRIES = [
 ];
 const SOLIDSTART_PLUGIN = { module: "@solidjs/start/config", export: "solidStart" };
 const NUXT_CONFIG = [`nuxt.config.${ANY_MODULE}`];
+const STRYKER_CONFIG = "{stryker,.stryker}.{conf,config}.{json,js,mjs,cjs}";
 /** Every row, ordered by name and then by range. */
 export const CONVENTION_ROWS = [
     {
@@ -258,6 +259,25 @@ export const CONVENTION_ROWS = [
         ],
         moves: [],
     },
+    {
+        name: "stryker",
+        package: "@stryker-mutator/core",
+        range: ">=7.0.0 <11.0.0",
+        entries: [],
+        moves: [],
+        shortNames: [
+            {
+                property: "testRunner",
+                files: [STRYKER_CONFIG],
+                package: "@stryker-mutator/{}-runner",
+            },
+            {
+                property: "checkers",
+                files: [STRYKER_CONFIG],
+                package: "@stryker-mutator/{}-checker",
+            },
+        ],
+    },
     {
         name: "sveltekit",
         package: "@sveltejs/kit",

package/dist/src/conventions.js

@@ -11,6 +11,7 @@ import { CONVENTION_ROWS, } from "./convention-rows.js";
 import { globExpression } from "./glob.js";
 import { dirnamePath, joinPath, normalizePath, relativePath, resolvePath } from "./paths.js";
 import { satisfies } from "./version-range.js";
+const JSON_EXTENSION = ".json";
 /** The sections an installed package is required by, so its absence fails the run. */
 const REQUIRED_SECTIONS = ["dependencies", "devDependencies"];
 const DEPENDENCY_SECTIONS = [
@@ -114,8 +115,14 @@ function isModuleExports(node) {
         node.expression.text === "module" &&
         node.name.text === "exports");
 }
-/** Every expression or declaration one file exports as its default, in any module syntax. */
+/**
+ * Every expression or declaration one file exports as its default, in any module syntax.
+ * A JSON module's default export is its document's value.
+ */
 function defaultExports(file) {
+    if (file.fileName.endsWith(JSON_EXTENSION)) {
+        return file.statements.flatMap((statement) => isExpressionStatement(statement) ? [statement.expression] : []);
+    }
     const found = [];
     for (const statement of file.statements) {
         if (isExportAssignment(statement)) {

package/dist/src/inventory.js

@@ -215,6 +215,12 @@ function nameNodeOf(node) {
 export function nodeKey(file, node) {
     return `${file.fileName}:${String(node.pos)}:${String(node.end)}:${String(node.kind)}`;
 }
+/** Whether `node` is written inside `declaration`, compared by file and range. */
+function writtenInside(declaration, node) {
+    return (node.getSourceFile().fileName === declaration.getSourceFile().fileName &&
+        declaration.pos <= node.pos &&
+        node.end <= declaration.end);
+}
 /** `export default`, up to the keyword that stands where an unnamed default export's name would. */
 const DEFAULT_EXPORT = /^export\s+(?=default\b)/u;
 /**
@@ -360,6 +366,7 @@ export function inventory(project, host, targetRoot) {
             });
             containers.push({
                 owner: record,
+                declaration: node,
                 nameNode: node.name,
                 members: node.members,
                 memberKind: "class-member",
@@ -379,6 +386,7 @@ export function inventory(project, host, targetRoot) {
             });
             containers.push({
                 owner: record,
+                declaration: node,
                 nameNode: node.name,
                 members: node.members,
                 memberKind: "interface-method",
@@ -402,6 +410,7 @@ export function inventory(project, host, targetRoot) {
             if (members.length > 0) {
                 containers.push({
                     owner: record,
+                    declaration: node,
                     nameNode: undefined,
                     members,
                     memberKind: "type-member",
@@ -422,6 +431,7 @@ export function inventory(project, host, targetRoot) {
             });
             containers.push({
                 owner: record,
+                declaration: node,
                 nameNode: node.name,
                 members: node.members,
                 memberKind: "enum-member",
@@ -604,7 +614,9 @@ export function inventory(project, host, targetRoot) {
                 outside += declarations.length;
                 continue;
             }
-            for (const held of own) {
+            // A merged symbol's table holds the members every one of its declarations writes,
+            // and each member is kept once, by the declaration that writes it.
+            for (const held of own.filter((one) => writtenInside(container.declaration, one.node))) {
                 keepMember(container, held.node, seen, areStatic);
             }
         }

package/jsr.json

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

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cplieger/deadset-ts",
-  "version": "5.2.1",
+  "version": "5.2.3",
   "type": "module",
   "sideEffects": false,
   "description": "Deterministic dead-code analysis for TypeScript, class and type members included",
@@ -29,7 +29,7 @@
     "lint:fix": "eslint . --fix && prettier --write ."
   },
   "dependencies": {
-    "@typescript/native": "npm:typescript@7.1.0-dev.20261003.1"
+    "@typescript/native": "npm:typescript@7.1.0-dev.20261006.1"
   },
   "devDependencies": {
     "@eslint/js": "10.0.1",

package/src/convention-rows.ts

@@ -98,6 +98,8 @@ const SOLIDSTART_PLUGIN = { module: "@solidjs/start/config", export: "solidStart
 
 const NUXT_CONFIG = [`nuxt.config.${ANY_MODULE}`];
 
+const STRYKER_CONFIG = "{stryker,.stryker}.{conf,config}.{json,js,mjs,cjs}";
+
 /** Every row, ordered by name and then by range. */
 export const CONVENTION_ROWS: readonly ConventionRow[] = [
   {
@@ -325,6 +327,25 @@ export const CONVENTION_ROWS: readonly ConventionRow[] = [
     ],
     moves: [],
   },
+  {
+    name: "stryker",
+    package: "@stryker-mutator/core",
+    range: ">=7.0.0 <11.0.0",
+    entries: [],
+    moves: [],
+    shortNames: [
+      {
+        property: "testRunner",
+        files: [STRYKER_CONFIG],
+        package: "@stryker-mutator/{}-runner",
+      },
+      {
+        property: "checkers",
+        files: [STRYKER_CONFIG],
+        package: "@stryker-mutator/{}-checker",
+      },
+    ],
+  },
   {
     name: "sveltekit",
     package: "@sveltejs/kit",

package/src/conventions.ts

@@ -122,6 +122,8 @@ interface ConventionEntry {
 /** Parses one file's text with no program. */
 type ParseSource = (fileName: string, text: string) => SourceFile;
 
+const JSON_EXTENSION = ".json";
+
 /** The sections an installed package is required by, so its absence fails the run. */
 const REQUIRED_SECTIONS = ["dependencies", "devDependencies"] as const;
 
@@ -248,8 +250,16 @@ function isModuleExports(node: Node): boolean {
   );
 }
 
-/** Every expression or declaration one file exports as its default, in any module syntax. */
+/**
+ * Every expression or declaration one file exports as its default, in any module syntax.
+ * A JSON module's default export is its document's value.
+ */
 function defaultExports(file: SourceFile): readonly Node[] {
+  if (file.fileName.endsWith(JSON_EXTENSION)) {
+    return file.statements.flatMap((statement) =>
+      isExpressionStatement(statement) ? [statement.expression] : [],
+    );
+  }
   const found: Node[] = [];
   for (const statement of file.statements) {
     if (isExportAssignment(statement)) {

package/src/inventory.ts

@@ -208,6 +208,8 @@ interface Building {
 /** A container whose members are read from a symbol table, from the tree, or both. */
 interface Container {
   readonly owner: Building;
+  /** The declaration that writes the container, which owns the members written inside it. */
+  readonly declaration: Node;
   /** The container's name node, which is what a batch resolves to its symbol. */
   readonly nameNode: Node | undefined;
   readonly members: readonly Node[];
@@ -473,6 +475,15 @@ export function nodeKey(file: SourceFile, node: Node): string {
   return `${file.fileName}:${String(node.pos)}:${String(node.end)}:${String(node.kind)}`;
 }
 
+/** Whether `node` is written inside `declaration`, compared by file and range. */
+function writtenInside(declaration: Node, node: Node): boolean {
+  return (
+    node.getSourceFile().fileName === declaration.getSourceFile().fileName &&
+    declaration.pos <= node.pos &&
+    node.end <= declaration.end
+  );
+}
+
 /** `export default`, up to the keyword that stands where an unnamed default export's name would. */
 const DEFAULT_EXPORT = /^export\s+(?=default\b)/u;
 
@@ -642,6 +653,7 @@ export function inventory<Brand>(
       });
       containers.push({
         owner: record,
+        declaration: node,
         nameNode: node.name,
         members: node.members,
         memberKind: "class-member",
@@ -661,6 +673,7 @@ export function inventory<Brand>(
       });
       containers.push({
         owner: record,
+        declaration: node,
         nameNode: node.name,
         members: node.members,
         memberKind: "interface-method",
@@ -684,6 +697,7 @@ export function inventory<Brand>(
       if (members.length > 0) {
         containers.push({
           owner: record,
+          declaration: node,
           nameNode: undefined,
           members,
           memberKind: "type-member",
@@ -704,6 +718,7 @@ export function inventory<Brand>(
       });
       containers.push({
         owner: record,
+        declaration: node,
         nameNode: node.name,
         members: node.members,
         memberKind: "enum-member",
@@ -897,7 +912,9 @@ export function inventory<Brand>(
         outside += declarations.length;
         continue;
       }
-      for (const held of own) {
+      // A merged symbol's table holds the members every one of its declarations writes,
+      // and each member is kept once, by the declaration that writes it.
+      for (const held of own.filter((one) => writtenInside(container.declaration, one.node))) {
         keepMember(container, held.node, seen, areStatic);
       }
     }

package/src/workspace-run.ts

@@ -41,10 +41,9 @@ export interface WorkspaceRun {
   referenceOnly(configFile: string, compiled?: readonly string[]): (file: string) => boolean;
   /**
    * Whether one file a target configuration's program holds is the target's own. A
-   * workspace package's file is by its path alone, whichever import reached it: the
-   * compiler's answer for a file reached both through `node_modules` and from source
-   * depends on which import it followed first. Any other file is unless the compiler
-   * reached it as a library's.
+   * workspace package's file is by its path alone: the run reads another package from
+   * its source, so the compiler holds that package's files as the importer's own, not
+   * as a library's. Any other file is unless the compiler reached it as a library's.
    */
   ownFile(file: string, fromExternalLibrary: boolean): boolean;
   /** The target's own manifest, its targets read back to source. */