Retire tsc now that vp check type checks

AGENTS.md

@@ -75,18 +75,15 @@ Pass `REVIEW.md` by path to every reviewer sub-agent, not only the standards one
 
 ## Commands
 
-Run from project root. Prefer these `pnpm` scripts. `vp` subcommands (`vp test`, `vp lint`, `vp fmt`, `vp check`) are fine to run directly, since `vp` resolves the project's pinned Vite+ and its config; use `pnpm exec vp` if the global CLI isn't installed. `vp check` runs format, lint, and type checks. Never invoke `tsc`, `vitest`, `oxlint`, or `oxfmt` directly or via `pnpx`, because bare tools use the wrong version and miss project context.
+Run from project root. Prefer these `pnpm` scripts. `vp` subcommands (`vp test`, `vp lint`, `vp fmt`, `vp check`) are fine to run directly, since `vp` resolves the project's pinned Vite+ and its config; use `pnpm exec vp` if the global CLI isn't installed. `vp check` runs format, lint, and type checks. Never invoke `tsc`, `tsgo`, `vitest`, `oxlint`, or `oxfmt` directly or via `pnpx`, because bare tools use the wrong version and miss project context.
 
-- `pnpm check:types` — typecheck all packages (no per-file/per-package option; this is the granularity)
 - `pnpm checks` — all static checks (types + lint + format); default validation for small changes
 - `pnpm check:lint` / `pnpm lint` — lint / lint and fix
 - `pnpm check:format` / `pnpm format` — check / fix formatting
 - `pnpm test` — run all tests
 - `pnpm test <path>` — test files matching a path substring (file, dir, or partial)
 - `pnpm test -t "suite > name"` — test by name, with segments joined by a spaced `>` exactly as the reporter prints them; a single segment such as `-t "renders empty state"` also works (no `--` separator; pnpm forwards args verbatim, unlike npm)
 
-`pnpm check:types` runs in parallel across packages in under a minute; just run it.
-
 ## Agent skills
 
 ### Issue tracker

REVIEW.md

@@ -12,7 +12,7 @@ Reserve Important for the cases below. Everything else is Nit at most. That incl
   - Values that reach a query outside a Query Fragment, or any other break of the rules in `docs/agents/connectors.md`.
   - A limit that doesn't bound the scan.
   - A change to one connector, or to shared query code, that changes behavior in the other query languages without saying why.
-- **Runtime APIs newer than the browser floor in `packages/graph-explorer/.browserslistrc`,** such as `Error.isError` or Iterator helpers. `tsc` and oxlint catch most of these. Check by hand for what they miss, listed in `docs/agents/browser-floor.md`: DOM instance members, and built-in features that `lib` allows but the floor doesn't.
+- **Runtime APIs newer than the browser floor in `packages/graph-explorer/.browserslistrc`,** such as `Error.isError` or Iterator helpers. The type check and oxlint catch most of these. Check by hand for what they miss, listed in `docs/agents/browser-floor.md`: DOM instance members, and built-in features that `lib` allows but the floor doesn't.
 - **Production code that imports a `devDependency`.**
 - **Work that grows with graph or schema size:** one query or subscription per item, quadratic loops, or copying passes over large collections.
 - **Tests that can't fail.** Name a plausible regression of the behavior the test's name promises, and prove the verdict with a mutation: inject that regression and run the test. A test that stays green is the finding, and the fix is to tighten it until the mutation turns it red. Recommend deletion only when no regression of that behavior could reach the test. Flag these:
@@ -32,7 +32,7 @@ Reserve Important for the cases below. Everything else is Nit at most. That incl
 
 ## Do not report
 
-- Anything CI enforces: `vp lint` (oxlint, including the React Compiler rules), `vp fmt` (oxfmt), `tsc`, lockfile checks, Trivy and dependency advisories.
+- Anything CI enforces: `vp lint` (oxlint, including the React Compiler rules), `vp fmt` (oxfmt), the type check in `vp check`, lockfile checks, Trivy and dependency advisories.
 - Pre-existing issues, unless this PR makes them newly reachable.
 - The Git rules in `AGENTS.md`. PRs are squash merged, so commit messages, branch names, and how the commits are split never reach `main`.
 - `Changelog.md`. It's updated at release time.

docs/agents/testing.md

@@ -76,7 +76,7 @@ Commands are in AGENTS.md.
 
 - Small change: `pnpm checks`, plus `pnpm test <path>` for the area you touched
 - Before a PR, or after touching shared utilities, core providers, or type definitions: `pnpm test`
-- No associated tests (config or type-only change): `pnpm check:types`
+- No associated tests (config or type-only change): `pnpm checks`
 
 ## Special cases
 

package.json

@@ -14,9 +14,7 @@
     "check:format": "vp fmt --check",
     "test": "vp test run",
     "test:watch": "vp test watch",
-    "check:types": "pnpm -r --include-workspace-root --parallel --stream run typecheck",
-    "typecheck": "tsc --noEmit",
-    "checks": "vp check && pnpm run check:types",
+    "checks": "vp check",
     "start": "pnpm --filter \"graph-explorer-proxy-server\" run start",
     "clean": "pnpm --stream -r run clean",
     "clean:dep": "rm -rf node_modules && pnpm -r exec rm -rf node_modules",

packages/graph-explorer-proxy-server/package.json

@@ -6,7 +6,6 @@
   "author": "amazon",
   "type": "module",
   "scripts": {
-    "typecheck": "tsc --noEmit",
     "dev": "node src/node-server.ts",
     "start": "NODE_ENV=production node src/node-server.ts"
   },

packages/graph-explorer/package.json

@@ -6,7 +6,6 @@
   "author": "amazon",
   "type": "module",
   "scripts": {
-    "typecheck": "tsc --noEmit",
     "dev": "vp dev --force",
     "clean": "rimraf dist",
     "vite-build": "NODE_OPTIONS=--max_old_space_size=6144 vp build",

packages/shared/package.json

@@ -12,9 +12,6 @@
     "./types": "./src/types/index.ts",
     "./utils/testing": "./src/utils/testing/index.ts"
   },
-  "scripts": {
-    "typecheck": "tsc --noEmit"
-  },
   "devDependencies": {
     "vite-plus": "catalog:"
   }