Skip to content
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
| 2026-08-14 | PR-1953 | 5f6ecbd554f91a692965f7f876807d4e9e9c2c26 | PR #1953 full review and required base sync | no PR-introduced P0-P2 defect; existing P2 verified-correct; merged latest main | manual adversarial review; current thread verification; git merge-tree; git diff --check; base-sync native ledger/docs checks; focused Playwright/Next build not run (node_modules absent) |
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
| 2026-08-14 | PR-1953 | 98b7458ba7ef4cecc5b5f5747deccf6adbab0480 | scripts/run-playwright.mjs; docs/branch-review-records | no PR-introduced P0-P2 defect; corrected prior local-only merge record and merged latest main | manual adversarial review; current thread verification; git merge-tree; git diff --check; docs links; ledger inbox; ledger guards; focused Playwright/Next build unavailable (node_modules absent) |
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
| 2026-08-14 | claude/playwright-tsconfig-isolation | a5af47d083091a44cf515bed031d7cbd1bea3ea7 | scripts/run-playwright.mjs | FIXED - isolated child tsconfig given explicit include/exclude, empirically reproduced and verified | typecheck,verify:ui-focused-repro |
27 changes: 27 additions & 0 deletions scripts/run-playwright.mjs
Original file line number Diff line number Diff line change
Expand Up @@ -261,6 +261,33 @@ try {
baseUrl: "../..",
paths: { "@/*": ["src/*"] },
},
// Declaring include/exclude here (rather than leaving them unset and
// inheriting the root tsconfig.json's) is deliberate. TypeScript resolves
// an extended config's *inherited* relative include/exclude entries
// against the repo root, so an unset include here would still resolve
// "**/*.ts" and ".next/dev/types/**/*.ts" against the shared top-level
// .next/ directory — not this isolated run's own NEXT_DIST_DIR output
// under `dist/`. That pulls stale/foreign route types from whatever the
// top-level .next happens to contain (a prior `npm run dev` or `npm run
// build`) into this run's typecheck. Excluding the repo-root .next/ and
// pointing at this run's own dist/types + dist/dev/types keeps the
// isolated build's typecheck scoped to itself (outstanding-issues #210).
include: [
"../../next-env.d.ts",
"../../**/*.ts",
Comment thread
BigSimmo marked this conversation as resolved.
"../../**/*.tsx",
"../../**/*.mts",
"dist/types/**/*.ts",
"dist/dev/types/**/*.ts",
],
exclude: [
"../../node_modules",
"../../scratch/**",
"../../supabase/functions/**",
"../../worktrees/**",
"../../scripts/archive/**",
"../../.next/**",
],
},
null,
2,
Expand Down
Loading