From 9315e5dfda9d5197e438c376c7156e05bda0fc43 Mon Sep 17 00:00:00 2001 From: Gary Basin Date: Thu, 16 Jul 2026 17:02:49 -0400 Subject: [PATCH] chore(centaur-client): give the package a typecheck script so pnpm check covers it centaur-client was the only workspace package without a typecheck script. pnpm -r typecheck skips a package that lacks the script rather than failing, so pnpm check -- the pre-PR bar in CLAUDE.md -- never typechecked this package at all. Its types were only ever checked by build (bare tsc, whose include covers *.test.ts), which runs solely in CI's Surface static job. That is how #545 shipped a red build: vitest does not typecheck, so a test-file type error was invisible to every gate a human runs locally. --noEmit because build already owns dist (outDir + declaration); a typecheck should not race it for build artifacts. --- surface/centaur-client/package.json | 1 + 1 file changed, 1 insertion(+) diff --git a/surface/centaur-client/package.json b/surface/centaur-client/package.json index a39a3221..08a7678a 100644 --- a/surface/centaur-client/package.json +++ b/surface/centaur-client/package.json @@ -32,6 +32,7 @@ ], "scripts": { "build": "tsc -p tsconfig.json", + "typecheck": "tsc -p tsconfig.json --noEmit", "test": "vitest run" }, "devDependencies": {