test: wire the three orphaned suites into CI

[viktormarinho]

Summary

Three suites ran in no workflow at all — coverage that looked like coverage but couldn't catch anything:

1. Two .e2e.-named files that are actually unit tests — the filename rule excluded them from the unit tier and nothing else picked them up:

• packages/typegen/src/cli.e2e.test.ts → cli.test.ts: self-contained (in-process MCP server + the real CLI), 8 tests in 1.1s.
• apps/api/.../user-ask.e2e.test.ts → user-ask.test.ts: mocks all params, asserts tool registration/schemas in-process; 30 tests. Header comment updated to stop claiming E2E.
• Unit tier goes 680 → 686 files, all green under --parallel.

2. apps/web/ct Playwright component tests (174 tests) — new component-tests job in test.yml: chromium with browser cache, timeout-minutes: 15, gated on a new web output of the changes gate (apps/web/* or packages/ui/*), so backend-only PRs skip it. Not yet a required check — promote via branch protection once it proves stable.

3. The rot the wiring surfaced — 14 drifted ct tests, all fixed against current component behavior:

• Sortable item rows became role=button wrappers whose accessible name concatenates the inline action buttons → row locators rewritten (filter({ hasText }) helper, exact: true on the actions menu).
• rich-text/rich-text-inline render a TipTap editor now (contenteditable ProseMirror, HTML model values), not a <textarea> → tests rewritten for the editor.
• NumberField deliberately renders type=text + inputMode=decimal|numeric (preserves mid-typing states like 0.) → assertions updated.
• Every value still round-trips — none of the 14 were real regressions, so zero test.fixme.

Strays fixed in passing: storage-integration's find now scans apps/api and packages (its own comment already promised "every `*.integration.test.ts` file is picked up here" — false for packages until now); Playwright ct's vite build cache (apps/web/ct/playwright/.cache/) is gitignored — knip and oxlint were scanning bundle output.

Testing notes

• ct suite: 174/174 across 3 consecutive full runs (~12s each).
• Unit suite: 6216 pass / 0 fail, 686 files.
• bun run check ✓ · lint ✓ · knip ✓ · fmt:check ✓
• This PR touches apps/web/*, so the new component-tests job runs on this PR itself — the wiring validates itself.

🤖 Generated with Claude Code

---

Summary by cubic

Wires three previously unrun test suites into CI to make coverage real and catch regressions. Adds a Playwright component-tests job, fixes drifted CT assertions, broadens test discovery, prevents a typegen test hang on CI, and speeds up Playwright installs with cache restore-keys.

• New Features

  • Added component-tests job in .github/workflows/test.yml for apps/web/ct (Chromium cached, 15‑min timeout), gated by a new web filter for apps/web/* and packages/ui/*; not required yet.

• Bug Fixes

  • Closed the MCP server in packages/typegen/src/cli.test.ts teardown to avoid a lingering session stream that hung CI under bun test --parallel.
  • Renamed two misnamed unit tests from .e2e.test.ts to *.test.ts so the unit suite runs them (packages/typegen/src/cli.test.ts, apps/api/.../user-ask.test.ts).
  • Updated CT assertions to match current UI: sortable rows via role=button + hasText; rich-text/rich-text-inline now TipTap with HTML values; number/integer use type=text with inputmode.
  • Broadened discovery and cleanup: storage-integration.yml finds *.integration.test.ts in apps/api and packages; ignored Playwright CT Vite cache at apps/web/ct/playwright/.cache/.
  • Playwright CI: added restore-keys to browser caches in .github/workflows/e2e.yml and the CT job to avoid cold installs on key rotations; step labels now specify Chromium.

^{Written for commit b1fc316. Summary will update on new commits.}