Conversation
Instead of passing isOnline through every component and disabling individual buttons, a single OfflineGuard component renders a semi-transparent overlay that blocks all pointer interaction when offline. Components no longer need to know about connectivity status. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Add scripts and configuration for automatic worktree setup: - worktree-init.sh detects worktrees and assigns unique ports via hash - post-checkout hook triggers init on branch checkout - Make Vite/API ports configurable via VITE_DEV_PORT and VITE_API_PORT - Fix hardcoded ports in websocket.ts and viewerUrl.ts - Update .envrc to load .env.local for secrets - Add husky prepare script, set core.hooksPath to .husky Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
…move deprecated completeHeat Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
All repository factory functions now require a DbConnection parameter.
Updated test files to pass the test database instance to factories:
- Added getTestDb() helper to test-db.ts
- Moved module-level repo creation into beforeAll/beforeEach
- Rewrote auth tests to use PGlite instead of spying on removed exports
- Removed obsolete { useTransaction: false } option from bracket tests
- Removed non-existent completeHeat from mock HeatRepository
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
…actor Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Remove redundant `const db = this.conn` intermediary in all repository implementations — use `this.conn` directly. Add module boundary enforcement design and implementation plan documents. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
…TP status codes Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
…ware maps domain errors Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
The helper was redundant because HeatService.updateWaveScore(), updateJumpScore(), and deleteScore() already validate that the heat is not completed — they throw HeatCompletedError which the domain error mapper middleware maps to 400 BAD_REQUEST. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Use CommonORPCErrorCode from @orpc/client instead of string for DOMAIN_ERROR_MAP, so typos in error codes are caught at compile time - Extract formatDomainScore() for domain-Score-to-response mapping in getHeadJudge handler, eliminating the last inline .map() transformation Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Enforce automated quality gates so agents cannot commit or merge bad code. Pre-commit hook runs biome auto-fix, typecheck, boundary checks, and all tests. CI now runs on pull requests targeting main with architecture boundary checks. Docker build and deploy only run on push to main. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.