From 339801966ef3955a69bcdd77a7b0e6e8dd65542f Mon Sep 17 00:00:00 2001 From: "Angelo B. J. Luidens" Date: Wed, 3 Jun 2026 13:27:15 -0400 Subject: [PATCH] Restore green type-check baseline (117 tests + tsc all pass) Fixes 3 pre-existing TypeScript errors that vitest did not catch but tsc --noEmit did: - apps/mcp-linkedin/src/tools.test.ts: spread TOOL_NAMES before .sort() (readonly tuple has no mutating sort) - packages/sanitize/src/corpus.test.ts: cast through unknown to FixtureMeta - db/migrations/rehearse.ts: add 'postgres' driver dependency (also needed to run the migration itself) Also: apps/admin test script uses --passWithNoTests (Stage 5 admin app has no test files yet, vitest exited non-zero on empty). Co-Authored-By: Claude Opus 4.8 (1M context) --- apps/admin/package.json | 2 +- apps/mcp-linkedin/src/tools.test.ts | 2 +- bun.lock | 3 +++ package.json | 1 + packages/sanitize/src/corpus.test.ts | 2 +- 5 files changed, 7 insertions(+), 3 deletions(-) diff --git a/apps/admin/package.json b/apps/admin/package.json index b96abcf..511a9c0 100644 --- a/apps/admin/package.json +++ b/apps/admin/package.json @@ -8,7 +8,7 @@ "dev": "next dev -p 3002", "start": "next start -p 3002", "typecheck": "tsc --noEmit", - "test": "vitest run", + "test": "vitest run --passWithNoTests", "lint": "next lint" }, "dependencies": { diff --git a/apps/mcp-linkedin/src/tools.test.ts b/apps/mcp-linkedin/src/tools.test.ts index 54f0f7f..6b92da7 100644 --- a/apps/mcp-linkedin/src/tools.test.ts +++ b/apps/mcp-linkedin/src/tools.test.ts @@ -4,7 +4,7 @@ import { outletForAuthorUrn, validateToolCall } from "./server"; describe("Tool registry — 9 tools per Plan §3.2", () => { it("exposes exactly the 9 tools named in the plan", () => { - expect(TOOL_NAMES.sort()).toEqual( + expect([...TOOL_NAMES].sort()).toEqual( [ "linkedin_whoami", "linkedin_auth_status", diff --git a/bun.lock b/bun.lock index e0ebabc..5481822 100644 --- a/bun.lock +++ b/bun.lock @@ -7,6 +7,7 @@ "devDependencies": { "@types/node": "^22.0.0", "bun-types": "^1.3.12", + "postgres": "^3.4.5", "turbo": "^2.3.0", "typescript": "^5.7.0", }, @@ -597,6 +598,8 @@ "postcss": ["postcss@8.4.31", "", { "dependencies": { "nanoid": "^3.3.6", "picocolors": "^1.0.0", "source-map-js": "^1.0.2" } }, "sha512-PS08Iboia9mts/2ygV3eLpY5ghnUcfLV/EXTOW1E2qYxJKGGBUtNjN76FYHnMs36RmARn41bC0AZmn+rR0OVpQ=="], + "postgres": ["postgres@3.4.9", "", {}, "sha512-GD3qdB0x1z9xgFI6cdRD6xu2Sp2WCOEoe3mtnyB5Ee0XrrL5Pe+e4CCnJrRMnL1zYtRDZmQQVbvOttLnKDLnaw=="], + "process-warning": ["process-warning@5.0.0", "", {}, "sha512-a39t9ApHNx2L4+HBnQKqxxHNs1r7KF+Intd8Q/g1bUh6q0WIp9voPXJ/x0j+ZL45KF1pJd9+q2jLIRMfvEshkA=="], "quick-format-unescaped": ["quick-format-unescaped@4.0.4", "", {}, "sha512-tYC1Q1hgyRuHgloV/YXs2w15unPVh8qfu/qCTfhTYamaw7fyhumKa2yGpdSo87vY32rIclj+4fWYQXUMs9EHvg=="], diff --git a/package.json b/package.json index 2707eca..b937d19 100644 --- a/package.json +++ b/package.json @@ -19,6 +19,7 @@ "devDependencies": { "@types/node": "^22.0.0", "bun-types": "^1.3.12", + "postgres": "^3.4.5", "turbo": "^2.3.0", "typescript": "^5.7.0" }, diff --git a/packages/sanitize/src/corpus.test.ts b/packages/sanitize/src/corpus.test.ts index fd87e29..152dc6d 100644 --- a/packages/sanitize/src/corpus.test.ts +++ b/packages/sanitize/src/corpus.test.ts @@ -27,7 +27,7 @@ const parseFixture = (name: string, raw: string): Fixture => { if (!m) throw new Error(`Fixture ${name} missing frontmatter`); const fmRaw = m[1]!; const body = m[2]!; - const meta = parseSimpleYaml(fmRaw) as FixtureMeta; + const meta = parseSimpleYaml(fmRaw) as unknown as FixtureMeta; return { name, meta, body }; };