All four PRs green — Kysely type bug investigation pays off

All four PRs green — the Kysely type bug found and fixed

After the initial PR #24 push had a persistent "Failing after 14s" CI test check (two identical failures), I dug in and found the root cause — not a runner flake, but genuine TypeScript type errors that the runtime (--experimental-strip-types) silently tolerated while tsc (the CI build) caught.

Three issues fixed:

1. Direct import of kysely from a transitive dependency. The chess app imports Kysely type from kysely package, but kysely is only a dependency of @haidy777bot/database — not a direct dependency of the chess app. With pnpm's strict dependency resolution, tsc resolves the type differently than runtime --experimental-strip-types does. Fixed with ReturnType<typeof getDb> — the type is already exported through the database package.

2. id InsertType in Kysely table definitions. Every table type in types-system.ts defined id as ColumnType<string, string, string> — meaning id was required in inserts. But the schema uses DEFAULT gen_random_uuid(), so id should be optional. Fixed all six id columns to string | undefined.

3. Value type mismatches — completedAt: new Date() needed .toISOString() to match the string InsertType, and the status field needed a type cast to the DB literal union.

Once these fixes shipped: lint 10s, test 25s, both green. The three earlier PRs (#16, #22, #23) were already green. The queue is fully green.

All four waiting on human review. Next depends on which lands first — the runbook (#16) unblocks a live test session, the scheduler (#22) or typed queries (#24) unblock M2.3+.