Skip to content
This repository was archived by the owner on Jun 16, 2026. It is now read-only.

Commit 243e73d

Browse files
mcp-tool-shopclaude
andcommitted
fix(ci): include node types under TS6 + bump brace-expansion
TypeScript 6 with module: Node16 and @types/node@25 no longer auto-includes node type definitions. `import { readFileSync } from "node:fs"` failed across the codebase with TS2591 "Cannot find name 'node:fs'", breaking CI on every push since 2026-04-25. Adding `"types": ["node"]` to tsconfig.json compilerOptions explicitly tells TS to include @types/node. Minimal, surgical change — no other @types/* devDependencies exist, so this doesn't exclude anything. Also picks up `npm audit fix` for brace-expansion (transitive, moderate severity GHSA-f886-m6hf-6m8v / GHSA-jxxr-4gwj-5jf2) so the audit gate passes once the build step is unblocked. Local verification (Node 22): - npm run build: clean - npm test: 93/93 pass - npm audit --audit-level=moderate: 0 vulnerabilities - npm pack --dry-run: 57 files, 823.5 kB Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
1 parent a3028d3 commit 243e73d

2 files changed

Lines changed: 5 additions & 4 deletions

File tree

package-lock.json

Lines changed: 3 additions & 3 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

tsconfig.json

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,8 @@
1010
"skipLibCheck": true,
1111
"forceConsistentCasingInFileNames": true,
1212
"declaration": true,
13-
"sourceMap": false
13+
"sourceMap": false,
14+
"types": ["node"]
1415
},
1516
"include": ["src/**/*.ts"],
1617
"exclude": ["node_modules", "dist"]

0 commit comments

Comments
 (0)