Skip to content

docs(issues): reconcile inbox requests from PR #2023/#2024 - #2032

Closed
BigSimmo wants to merge 9 commits into
mainfrom
claude/ledger-reconcile-issues-c1trbj
Closed

docs(issues): reconcile inbox requests from PR #2023/#2024#2032
BigSimmo wants to merge 9 commits into
mainfrom
claude/ledger-reconcile-issues-c1trbj

Conversation

@BigSimmo

Copy link
Copy Markdown
Owner

Summary

New/updated rows from the requested set

  • #212 — corrected (unsafe-cast/JSON.parse remediation status)
  • #J912J9 — new, P1: governance question on fabricated similarity: 1 on document-summary rows
  • #DP6M3G — new, P1: R1, unbudgeted strong escalation / lithium dosing route (packet S1b)
  • #6BG9X2 — new, P2: R2 + R3, claim-support strictness on verbatim-faithful restatements (packet S1c)
  • #BTVMVK — new, P2: recurring Sentry "Unhandled server request error" on /api/search
  • #ND10QT — new, P3: source_metadata pin in rag-row-contracts.ts is data-backed only
  • #TYJ0XP — new, P3: eval-canary.yml is post-merge only — record in docs/rag-behaviour
  • #0MSNT8 — new, P3: G1 implementation (governance Option B, similarity_origin: document_context)

The remaining 14 applied requests (#056, #098, #265, #314, #316, #318, #322, #237, #330, #331, #192, #162, #238, #324) were also pending in the inbox and were reconciled in the same transaction, since issues:reconcile applies the full pending batch atomically.

Verification

  • npm run check:outstanding-issuesOutstanding-issues guard passed: 348 rows (98 open, 250 archived), unique display and durable ids, collision-free allocation enabled, deprecated next-id marker ignored, no merge driver, no ids deleted from base 78fe906b8f30.
  • npm run check:ledger-write-disciplineLedger write discipline passed for 78fe906b8f30..HEAD.
  • Verification not run: npm run verify:pr-local / verify:cheap — this diff touches only docs/outstanding-issues.md and JSON inbox request files (a data move, not source/test/config), and is fully covered by the two ledger-specific gates above, which are exactly the gates this repo's own tooling designates for this transaction (check:ledger-write-discipline is what CI/pre-push run for this exact docs/outstanding-issues.md diff).
  • npm run format was run; it reported the reconciled docs/outstanding-issues.md as already Prettier-clean (no diff). It also reformatted unrelated pre-existing drift elsewhere in the tree; those changes were reverted to keep this PR docs/ledger-only per task scope.
  • UI verification not run: no UI/frontend/routing/styling code changed.

Risk and rollout

  • Risk: Low. This is a mechanical ledger-reconcile transaction (ID allocation + markdown table update), not a behavior change. check:ledger-write-discipline verifies the canonical-ledger diff exactly equals the recorded reconciliation transaction.
  • Rollback: Revert this commit; the immutable inbox request files remain available under docs/outstanding-issues-inbox/applied/ for re-reconciliation if needed.
  • Provider or production effects: None.

Notes

RAG impact: no retrieval behaviour change — ledger reconcile only (docs-only edit to docs/outstanding-issues.md and its inbox; no src/lib/rag/** or retrieval/ranking/eval surface touched).


Generated by Claude Code

… ledger
Applied after PR #2023 (440a34f) and #2024 (78fe906) landed their
immutable inbox requests. npm run issues:reconcile allocated IDs and
moved each processed request to docs/outstanding-issues-inbox/applied/.
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01D2BUayF2Z9DAxvmL5XmLcC
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, add credits to your account and enable them for code reviews in your settings.

@supabase

supabaseBot commented Aug 17, 2026

Copy link
Copy Markdown

This pull request has been ignored for the connected project sjrfecxgysukkwxsowpy because there are no changes detected in supabase directory. You can change this behaviour in Project Integrations Settings ↗︎.


Preview Branches by Supabase.
Learn more about Supabase Branching ↗︎.

@coderabbitai

coderabbitaiBot commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

Warning

Review limit reached

You’ve reached a temporary PR review limit under our Fair Usage Limits Policy.

Your recent review volume is higher than typical usage, so adaptive limits are currently applied.

Next review available in:58 minutes

Limit details: You’ve used all 1 included review currently available under your plan. You completed 95 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 @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

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 configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: be4d8e10-1ac3-4e74-923e-0960994b3375

📥 Commits

Reviewing files that changed from the base of the PR and between 92f7618 and fcf1f56.

📒 Files selected for processing (24)
  • docs/branch-review-records/65d188be1a43a7c8cb344d1587c1f8e617108bc2de777d41e308fa989bb725c3.record.md
  • docs/outstanding-issues-inbox/applied/0649db55-c4ea-4683-b51c-4c6679754f81.json
  • docs/outstanding-issues-inbox/applied/1d35d652-d833-4934-890c-84b6837581cd.json
  • docs/outstanding-issues-inbox/applied/2f5fbcde-438b-47ab-a1b7-01309802d935.json
  • docs/outstanding-issues-inbox/applied/314e5e31-7d15-46f2-adb0-c4d113bc589d.json
  • docs/outstanding-issues-inbox/applied/46750cbf-f4f4-4a00-ab93-bdac644afbde.json
  • docs/outstanding-issues-inbox/applied/55cb57e8-7e6e-42f8-bd79-81b81a01ef3a.json
  • docs/outstanding-issues-inbox/applied/55f0d5de-8642-487a-b063-44c1c0bc0921.json
  • docs/outstanding-issues-inbox/applied/64dd6966-0697-4b70-a7ee-c8d526729404.json
  • docs/outstanding-issues-inbox/applied/6c0d5004-45a4-42c8-b001-92428f76a0b0.json
  • docs/outstanding-issues-inbox/applied/739887b7-24d6-4ea8-9aad-e7700a537913.json
  • docs/outstanding-issues-inbox/applied/7ad31d10-aeef-4635-863d-d6ce9340d916.json
  • docs/outstanding-issues-inbox/applied/7c602a99-9db9-4a56-9d7b-1692db146b5d.json
  • docs/outstanding-issues-inbox/applied/7e454b66-74ae-4e67-8669-1f1fdc2c018c.json
  • docs/outstanding-issues-inbox/applied/81fdf812-4dd6-4fd6-b2e8-f3b730affd8a.json
  • docs/outstanding-issues-inbox/applied/855d03b3-61a8-4a81-ad31-80d996dd581a.json
  • docs/outstanding-issues-inbox/applied/88628da6-881d-4947-a710-56093c5503fa.json
  • docs/outstanding-issues-inbox/applied/a3ba2812-2984-4fad-b959-6dfe69fb6aa8.json
  • docs/outstanding-issues-inbox/applied/a81b68eb-6b26-46db-81fa-90279a599886.json
  • docs/outstanding-issues-inbox/applied/aa0eb6ae-8f6a-4440-8a15-6ba0d17f812d.json
  • docs/outstanding-issues-inbox/applied/c156f1c0-8d83-460a-b1b5-ba970d664405.json
  • docs/outstanding-issues-inbox/applied/e54f04f8-09bb-4887-b3e4-a5cd99958798.json
  • docs/outstanding-issues-inbox/applied/ff73ce15-6890-44a1-8ddb-2919306a0e86.json
  • docs/outstanding-issues.md

Comment @coderabbitai help to get the list of available commands.

@github-actions

github-actionsBot commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

CI triage

CI failed on this PR. Automated classification of the 2 failed job(s):

  • Static PR checksneeds investigation: inspect the failing step and uploaded diagnostics; rerun only after classifying the cause.
  • PR requiredneeds investigation: inspect the failing step and uploaded diagnostics; rerun only after classifying the cause.

Compared with main CI run #11399 (cancelled).

Classification is evidence routing, not permission to ignore a failure. Exact quarantined Playwright identities remain governed by the flake ledger.

@BigSimmo

Copy link
Copy Markdown
OwnerAuthor

Superseded by #2043: a single fresh-base reconcile over the complete pending inbox. This branch reconciled a partial base (check:ledger-write-discipline: 'moved only part of the base inbox and left 6 request(s) pending'); a merge-and-rerun would allocate IDs as two batches and fail the same check, so it is replaced rather than repaired.

@BigSimmoClaude

Copy link
Copy Markdown
OwnerAuthor

Superseded by #2045, which has merged into main (squash 14421fba9). While this PR was open, 6 more ledger-reconcile requests landed on main, which check:ledger-write-discipline correctly flagged as a partial transaction here. #2045 redid the reconciliation from a fresh base as one complete, verified transaction. Closing this in favor of #2045.


Generated by Claude Code


Generated by Claude Code

@gitguardian

Copy link
Copy Markdown

⚠️ GitGuardian has uncovered 1 secret following the scan of your pull request.

Please consider investigating the findings and remediating the incidents. Failure to do so may lead to compromising the associated services or software components.

🔎 Detected hardcoded secret in your pull request
GitGuardian idGitGuardian statusSecretCommitFilename
36221231TriggeredGeneric High Entropy Secretd439b65tests/rag-adversarial-fixtures.test.tsView secret
🛠 Guidelines to remediate hardcoded secrets
  1. Understand the implications of revoking this secret by investigating where it is used in your code.
  2. Replace and store your secret safely. Learn here the best practices.
  3. Revoke and rotate this secret.
  4. If possible, rewrite git history. Rewriting git history is not a trivial act. You might completely break other contributing developers' workflow and you risk accidentally deleting legitimate data.

To avoid such incidents in the future consider


🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request.

@BigSimmo
BigSimmo deleted the claude/ledger-reconcile-issues-c1trbj branch August 22, 2026 08:59
Sign up for freeto 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.

2 participants

@BigSimmo@claude