issues: flag #336/#222/#321 as merged-but-unresolved, #322 as in-progress - #2081
Conversation
…ress via #2069 PR #2072 merged citing #336, #222, #321 but verification against actual main content shows none are genuinely satisfied — record the gap so a future session doesn't mistake a matching PR title for resolution. Also note #2069 is open against #322 with a known remaining gap (Warfarin records still don't cross-resolve), so a future session doesn't duplicate that PR. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01RnhwRP5FsY5JktkbwAcubD
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
|
Warning Review limit reachedYou’ve reached a temporary PR review limit under our Fair Usage Limits Policy. Next review available in: 10 minutes Limit details: You’ve used all 1 included review currently available under your plan. You completed 100 included PR reviews in the past 7 days; at that activity level, included reviews refill at 1 review per hour. Your organization has reached its usage spending cap. Adjust your spending cap in the billing tab. How can I continue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews. How do review limits work?CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability. For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (4)
Comment |
|
This pull request has been ignored for the connected project Preview Branches by Supabase. |
There was a problem hiding this comment.
Pull request overview
This PR updates the outstanding-issues workflow by adding four immutable inbox request JSON files under docs/outstanding-issues-inbox/ to record status updates (merged-but-unresolved for #336/#222/#321, and in-progress context for #322) without editing the canonical docs/outstanding-issues.md ledger directly.
Changes:
- Add an inbox
updaterequest for#336noting the responsive breakpoint tokens landed but the required design-system decision record did not. - Add inbox
updaterequests for#222and#321noting PR #2072 cited them but did not implement their actual scope. - Add an inbox
updaterequest for#322flagging an in-progress PR and why it is not yet fully resolved.
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| docs/outstanding-issues-inbox/a39af37c-b4dd-4e4b-84dc-395d86937615.json | Adds an update request capturing #336 as merged-but-unresolved (decision record still missing). |
| docs/outstanding-issues-inbox/1505ba13-1b2a-47c9-b6cc-03b060f7a878.json | Adds an update request capturing #222 as merged-but-unresolved (scope not actually addressed). |
| docs/outstanding-issues-inbox/779bc0e9-c28f-40ab-abda-cfdcf93ea752.json | Adds an update request capturing #321 as merged-but-unresolved (no relevant files changed). |
| docs/outstanding-issues-inbox/8e28c783-d655-4480-852c-bafaf0b3c07b.json | Adds an update request capturing #322 as in-progress to prevent duplicate parallel work. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
PR #2081 (already merged) queued "update" flags for #222, #321, and #336 noting they were still open after PR #2072. This PR queues its own newer, more complete requests for the same three rows (two "done" decisions, one updated "PARTIAL" status), which the ledger inbox tool correctly refuses to auto-reconcile as a silent overwrite — it requires an explicit cancel decision per row. Cancels the three now-superseded #2081 flags in favor of this PR's requests, fixing the docs:check-links / ledger-inbox failure in Static PR checks. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_015qDHwBNZYyRLdJHqSkiYTg
Summary
docs/outstanding-issues-inbox/update requests (no canonical ledger edits — reconciliation applies them later), recorded from this session's open-PR review sweep onbigsimmo/Database:#336(named responsive breakpoint tokens): PR feat(ui): standardize responsive breakpoint tokens and align header/secondary controls (#336, #222, #321) #2072 merged citing this row and the tokens now exist insrc/app/globals.css, but the row's required decision — record raw-vs-named-tokens indocs/design-system/GATES.md— was never made (GATES.mduntouched since PR Keep mobile sheet headers below device safe areas #1990, no mention of breakpoints). Stays open.#222(headers surface styling convergence): feat(ui): standardize responsive breakpoint tokens and align header/secondary controls (#336, #222, #321) #2072 shipped only a cosmetic font-weight tweak and an empty-state class alignment, not the PageHeader/EmptyState convergence decision the row asks for. Stays open.#321(differentials compare/filmstrip/DocumentViewer control alignment): feat(ui): standardize responsive breakpoint tokens and align header/secondary controls (#336, #222, #321) #2072's file list never touched any of these files despite citing the row. Stays open.#322(Warfarin catalogue record dedup): PR feat(medications): reconcile Warfarin records and connect catalogue drugs into interaction graph (#322, #323) #2069 is open and targets this row — it reconciles both duplicate records to the same interaction set, but its own new coverage test still fails because the two records don't cross-resolve each other by name (a content/authoring decision, not a mechanical fix). Flagged so a future session doesn't open a duplicate PR for the same dedup work.docs/outstanding-issues-inbox/.Verification
npm run check:outstanding-issues— "Ledger inbox check passed: 20 pending request(s), 217 applied." / "Outstanding-issues guard passed: 348 rows (97 open, 251 archived)..."npm run verify:pr-local— not run: this is a docs/ledger-inbox-only change (Tier 1, static/focused perdocs/process-hardening.md);check:outstanding-issuesis the exact gate that covers request-file validity and write discipline for this change class.npm run verify:ui— not applicable, no UI/routing/styling changenpm run verify:release— not applicable, not a release/handoff gateRisk and rollout
docs/outstanding-issues.mditself (canonical edits only happen via a dedicatednpm run issues:reconcilebranch).Notes
Generated by Claude Code