Skip to content

Add branch review records for PR #2338, #2337, #2333 sweep - #2352

Merged
BigSimmo merged 7 commits into
mainfrom
claude/unblock-prs-agents-y1ukr1
Aug 24, 2026
Merged

Add branch review records for PR #2338, #2337, #2333 sweep#2352
BigSimmo merged 7 commits into
mainfrom
claude/unblock-prs-agents-y1ukr1

Conversation

@BigSimmo

@BigSimmoBigSimmo commented Aug 24, 2026

Copy link
Copy Markdown
Owner

Summary

Test plan

  • Docs-only change (three new content-addressed record files); no code paths affected by this PR itself.
  • N/A — no build/test/lint required for this change.

🤖 Generated with Claude Code

https://claude.ai/code/session_01C4RHy24AtgPobEQQwrj7u1


Generated by Claude Code


Note

Low Risk
Docs-only ledger appends under docs/branch-review-records/; no application, CI, or security code changes.

Overview
Adds ten content-addressed rows under docs/branch-review-records/ for 2026-08-24 Run PR sweeps. No product code.

Records cover main-syncs (#2296, #2297, #2338, #2339), CI fixes (#2325 pin allowlist, #2337 tracking token, #2333 tap targets, #2347 snapshot/docs-links), a flake diagnosis plus sync (#2341), and diagnosis-only for Dependabot Docker #2326 (Node 26 vs engines pin; no fix).

#2333 and #2338 still note an open PR-policy Clinical Governance Preflight gap left to authors. #2347 was closed by the owner mid-sweep after Static PR checks went green.

Reviewed by Cursor Bugbot for commit 6f8fd91. Configure here.

claudeand others added 6 commits August 24, 2026 14:13
Records the Run-PR-style sweep check on PR #2342 (Improve Therapy
best-match visibility): review comments already fixed, branch synced
from main, CI green on completed checks.
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01C4RHy24AtgPobEQQwrj7u1
…, #2326
Immutable review records for the four-PR dependency sweep: CI fix on
#2325, clean main-syncs on #2296/#2297, and diagnosis-only on #2326
(Node 26 Docker bump incompatible with the engine-strict Node 24 pin).
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01C4RHy24AtgPobEQQwrj7u1
Records the Run-PR-style sweep check on PR #2339 (therapy comparison
mobile design mockups): already fully green, only needed a main sync.
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01C4RHy24AtgPobEQQwrj7u1
Records the Run-PR-style sweep checks on #2341 (dictionary filter
band, fixed via main sync, unrelated flake confirmed) and #2347
(browser test gate handoff, fixed stale generated file + doc-link
allowlist entries; owner closed the PR mid-sweep for unrelated reasons).
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01C4RHy24AtgPobEQQwrj7u1
Records the Run-PR-style sweep checks: #2337 fixed a design-token
ratchet failure, #2333 fixed a tap-target size regression and
resolved a concurrent-push merge, #2338 was only a main sync. #2333
and #2338 still have an open PR-policy failure (missing Clinical
Governance Preflight section) left for the PR author to fill in.
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01C4RHy24AtgPobEQQwrj7u1
@coderabbitai

coderabbitaiBot commented Aug 24, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Review was skipped due to path filters

⛔ Files ignored due to path filters (3)
  • docs/branch-review-records/68d5616bffac66b7aa334e3785ed9de6aeca711c4280a58e263d58221ec6378c.record.md is excluded by !docs/**, !**/*.md
  • docs/branch-review-records/72c5a9775b63dec1da3871de945351009d6a31809d32e80c7108d5b7d5a06c9d.record.md is excluded by !docs/**, !**/*.md
  • docs/branch-review-records/ba504bd5ee15c145f50f09fae92928bfbfa0a5b546a8af0732fc674927c67c0c.record.md is excluded by !docs/**, !**/*.md

CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including **/dist/** will override the default block on the dist directory, by removing the pattern from both the lists.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 07c9b063-889e-4225-94e8-6fa901b5547f

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

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

@supabase

supabaseBot commented Aug 24, 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 ↗︎.

cursor[bot]
cursorBot approved these changes Aug 24, 2026

@cursorcursorBot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved. Cursor Bugbot completed successfully with no findings requiring human review, and this docs-only ledger-record change has no applicable approval policy that requires extra review. No reviewers assigned.

Open in WebView Automation

Sent by Cursor Approval Agent: Pull Request Router and Approver

@chatgpt-codex-connectorchatgpt-codex-connectorBot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit:6f8fd91f79

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@BigSimmo
BigSimmo enabled auto-merge (squash) August 24, 2026 16:50
@BigSimmo
BigSimmo merged commit 9f09d03 into mainAug 24, 2026
26 checks passed
@BigSimmo
BigSimmo deleted the claude/unblock-prs-agents-y1ukr1 branch August 24, 2026 16:52
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