Skip to content

feat(test): comprehensive test coverage for core packages#63

Merged
siracusa5 merged 1 commit intomainfrom
feat/001-test-coverage
Mar 30, 2026
Merged

feat(test): comprehensive test coverage for core packages#63
siracusa5 merged 1 commit intomainfrom
feat/001-test-coverage

Conversation

@siracusa5
Copy link
Copy Markdown
Collaborator

Summary

  • Add vitest to packages/shared, packages/board-server, packages/chat-relay, and apps/cli
  • Type validation tests for types.ts and chat-types.ts (shared)
  • Integration tests for HTTP routes, yaml-store CRUD, and WebSocket hub (board-server)
  • Relay message routing and protocol serialization tests (chat-relay)
  • Validate and compile command tests with fixtures (CLI)
  • Add test:all script to root package.json
  • Update CI to run test:all on every PR with coverage reporting

Test plan

  • Run pnpm test:all locally and confirm all suites pass
  • Check CI runs the new test job on this PR
  • Review test coverage for meaningful assertions (not just smoke tests)

🤖 Generated with Claude Code

@siracusa5 siracusa5 force-pushed the feat/001-test-coverage branch 2 times, most recently from 13cdbb6 to fdf9197 Compare March 29, 2026 17:57
- Add vitest to shared, board-server, chat-relay, and CLI packages
- Type validation tests for types.ts and chat-types.ts (shared)
- Integration tests for HTTP routes, yaml-store, and WebSocket hub (board-server)
- Relay message routing and protocol serialization tests (chat-relay)
- Validate and compile command tests with fixtures (CLI)
- Add test:all script to root package.json
- Update CI workflow to run test:all on every PR
- Add coverage reporting via vitest coverage

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@siracusa5 siracusa5 force-pushed the feat/001-test-coverage branch from fdf9197 to a26766c Compare March 29, 2026 18:02
@siracusa5 siracusa5 merged commit 6a11e84 into main Mar 30, 2026
6 checks passed
@siracusa5 siracusa5 deleted the feat/001-test-coverage branch March 30, 2026 12:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant