docs: post-PR #41 nits — fallow snapshot refresh + drop shipped golden-queries from roadmap#42
Conversation
…lden-queries from roadmap.md "Next" - fallow.md: stamp 2026-05-02; add a one-liner to 'Adjacent — also shipped post-refresh' for PR #41 (cli/* → application/* engine lift) so the architectural prereq for HTTP serve is tracked. - roadmap.md: golden queries are operational (scripts/query-golden.ts runs in 'bun run check' + CI on every PR) — per docs-governance Rule 2, shipped items leave the forward-looking 'Next' section. Canonical home stays docs/golden-queries.md (referenced from CONTRIBUTING + benchmark.md).
|
|
Warning Rate limit exceeded
To keep reviews running without waiting, you can enable usage-based add-on for your organization. This allows additional reviews beyond the hourly cap. Account admins can enable it under billing. ⌛ 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. ℹ️ Review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (2)
✨ 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. Review rate limit: 0/1 reviews remaining, refill in 5 minutes and 57 seconds.Comment |
…llow + competitive-scan (#45) - fallow.md row B.8 was ❌ Open; PR #43 shipped it. Updated to ✅ with implementation notes (rule.id taxonomy, location auto-detection, deferred frontmatter overrides). - fallow.md MCP-server bullet said 'HTTP API stays in roadmap backlog'; PR #44 shipped it. Reworded; added a dedicated PR #44 bullet covering the new transport, shared tool/resource handlers, CSRF/DNS-rebinding guard, Zod validation at the HTTP boundary, and 404/500 status semantics. - competitive-scan-2026-04.md '4. What moved to the roadmap': replaced 'HTTP API (codemap serve) — still backlog' with shipped rows for PR #43 + PR #44. - Fixed broken anchor: cross-ref to fallow.md was #status-snapshot-as-of-2026-05-01 but the heading bumped to 2026-05-02 in PR #42.
Summary
Two small follow-ups surfaced when checking the doc set against the codebase post-PR #41:
docs/research/fallow.md— date stamp 2026-05-01 → 2026-05-02; added a one-liner under "Adjacent — also shipped post-refresh" naming PR refactor: lift cli/* engine helpers into application/* (close all layer-reversal imports) #41 (cli/* → application/* engine lift). Worth tracking there because it's the architectural prereq that unblocks the HTTP transport (serveis the next backlog item up).docs/roadmap.md— golden queries are operational (scripts/query-golden.tsruns inbun run check+ CI on every PR; canonical doc =docs/golden-queries.md). Per docs-governance Rule 2 (shipped items leave the forward-looking section), removed from "Next".Behavior change
None — docs only.
Test plan
bun run checkpassed locally.