Conversation
…ent statuses Add Fastify-based HTTP API with Bearer token auth: - GET /api/status — full manifest with refreshed agent statuses and lifecycle - GET /api/worktrees/:id — single worktree detail by ID or name - GET /api/worktrees/:id/diff — branch diff in numstat format Closes #68
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
|
Warning Rate limit exceeded
⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (6)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
P0 — Security: - Use crypto.timingSafeEqual for Bearer token comparison - Return reply from auth hook to prevent handler execution after 401 P1 — Architecture: - Extract computeLifecycle to core/lifecycle.ts (fix server→commands dependency) - Re-export from commands/status.ts for backward compatibility - Remove unused WorktreeNotFoundError import - Remove redundant requireManifest call in diff route P2 — Robustness: - Add Fastify error handler mapping PpgError codes to HTTP status codes - Diff route uses readManifest (no lock/write) instead of updateManifest - Add tests: auth bypass, ManifestLockError, NotInitializedError, git diff failure, verify diff uses readManifest not updateManifest
Summary
GET /api/status— full manifest with refreshed agent statuses and computed lifecycleGET /api/worktrees/:id— single worktree detail (resolves by ID or name)GET /api/worktrees/:id/diff— branch diff in numstat format (file-level additions/deletions)projectRootandbearerTokenoptionsCloses #68
Test plan
inject()with mocked core functions