Local Quality Check

Run all local quality checks. Use during development to verify the current
state quickly.

First: self-review on four axes

Walk these BEFORE the commands below and before committing. A green suite says
the tests pass, not that the work is good, and none of these four is
something a runner reports.

1. Implementation gaps — the parallel change forgotten in a sibling command;
   a test or a doc not added.
2. Oddities — dead code, leftover names, a half-applied refactor.
3. Polish opportunities — small in-scope improvements; default to
   including them when they touch the same files and carry no
   behavior-break risk.
4. Regression risk — the full suite, and the call sites of anything renamed
   or removed.

Say out loud what you found, then fix it before running the checks.

Steps

Run these sequentially and report results.

0. Worktree pre-flight: [ -d node_modules ] || pnpm install --frozen-lockfile.
   git worktree add does not copy node_modules, and without it vp check
   fails with an unnamed typescript(tsconfig-error): Invalid tsconfig — Cannot find type definition file for 'node', reading like a broken fixture
   tsconfig.

1. vp check --fix (typecheck + lint + Prettier, auto-fix), then vp run check
   — the exact command CI's check-build-test job runs. They are NOT equivalent:
   vp check --fix has passed with 0 errors on a tree vp run check failed. Do
   not substitute vp run lint:fix, which does not touch Prettier, so a
   lint:fix-only run passes locally while CI fails Formatting issues found.
   Run both; CI parity comes from the second.

2. vp run typecheck:test — type-checks tsconfig.test.json (tests/**).
   vp check only type-checks tsconfig.json (src + types), which excludes
   **/*.test.ts, and vp test's Type Errors line covers only *.test-d.ts.
   A wrong import type or stale mock shape in a test file is invisible without
   this step.

3. vp run build.

   Then regenerate the generated artifacts, unconditionally:
   vp run gen:all-matrices && vp run audit:coverage:check && vp run format.
   The suite carries a byte-for-byte guard per generated matrix, and the trigger
   set is wider than any condition you would write — renaming a private provider
   method stales handled-property-wiring.json, and a verify.sh edit with no
   src change stales cli-flag-coverage.

4. vp test run — the whole unit suite. Prefer this over vp run test: nothing
   sits between the caller and the verdict.

   Read the summary's NUMBERS, not its colour, and read the Errors line, which
   is NOT Type Errors. Test Files N passed (N) is self-consistent over a
   suite that LOST files before they ran, so it cannot report its own shortfall:
   a loaded host produces [vitest-pool]: Failed to start forks worker beside an
   otherwise green summary. vp test run --maxWorkers=4 is the remedy for the
   whole [vitest-pool] / [vitest-pool-runner] family.

   Also check WHICH PROJECT the summary belongs to — the summary line cannot
   tell you, and a run launched from this worktree has been observed printing a
   sibling project's suite. Assert the root in the same command that produces the
   verdict, and read the suite's own rc, not the trailing grep's:

   # Subshell so the `exit`s are safe to paste into an interactive shell.
   (
     log=$(mktemp)                     # NOT a fixed path: concurrent lanes share /tmp.
     echo "log: $log"                  # print BEFORE the exits, or a failing run is unrecoverable.
     vp test run > "$log" 2>&1; rc=$?
     runs=$(grep -c 'RUN  v' "$log")
     [ "$runs" = 1 ] || { echo "expected 1 RUN header, found $runs -- attests to nothing; log: $log"; exit 1; }
     run_root=$(grep -m1 -oE "RUN  v[0-9.]+ .*" "$log" | sed 's/^RUN  v[0-9.]* //')
     [ "$run_root" = "$(pwd -P)" ] || { echo "WRONG PROJECT ($run_root) -- attests to nothing; log: $log"; exit 1; }
     # The alternatives are chosen against the COLOURED bytes: vitest pads the
     # label inside the dim escape, so `      Tests ` survives colouring and
     # `Tests +[0-9]` does not. Do not tighten either one.
     grep -E "Test Files|      Tests |Type Errors|Errors.*[0-9]+ error" "$log"
     if [ "$rc" != 0 ]; then
       echo "SUITE FAILED rc=$rc; log: $log"
       grep -qE '\[vitest-pool(-runner)?\]: ' "$log" \
         && echo "  the pool lost workers before their files ran, so the passing counts above cover only what survived -- re-run with --maxWorkers=4"
       exit 1
     fi
     # A run that exited 0 can still have COLLECTED fewer files than the tree
     # holds (a stray filter, a narrowed include). Unanchored `(N)`: the
     # coloured summary ends in an escape, not in `)`.
     collected=$(grep 'Test Files' "$log" | grep -oE '\([0-9]+\)' | tail -1 | tr -d '()')
     # `:(glob)` on every pathspec: git's bare `**` demands an intervening `/`
     # while vitest's does not.
     ondisk=$(git ls-files ':(glob)tests/**/*.test.ts' ':(glob)src/**/*.test.ts' \
       ':(glob)tests/**/*.test-d.ts' ':(glob)src/**/*.test-d.ts' | wc -l | tr -d ' ')
     # `-ge`, not `=`: an untracked new test file is collected but not listed.
     [ "${collected:-0}" -ge "$ondisk" ] || { echo "COLLECTED ${collected:-none} of $ondisk tracked test files over a run that exited 0 -- attests to nothing; log: $log"; exit 1; }
   )

   The wrong-project symptom is contention-dependent, not sticky — re-running
   usually lands right.

Output

Report as a table:

Check	Result
typecheck + lint + format (vp check --fix)	pass/fail
test-project typecheck (vp run typecheck:test)	pass/fail
build	pass/fail
tests (N files, M tests) (vp test run)	pass/fail

If all pass, confirm "All checks passed." If any fail, show the error output and
STOP — fix the failures and re-run before committing.

These checks attest to the working-tree state at the time of the run, so re-run
after any further edit.