🔧 CodeRabbit CI Fix: Fix failing PR Policy CI checks#954
Conversation
|
Important Review skippedThis PR was authored by the user configured for CodeRabbit reviews. CodeRabbit does not review PRs authored by this user. It's recommended to use a dedicated user account to post CodeRabbit review feedback. ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Comment |
|
This pull request has been ignored for the connected project Preview Branches by Supabase. |
5fcf1e8
into
cursor/supabase-preview-migration-drift-7850
* fix(db): restore orphan migration versions for Supabase Preview Hosted Preview fails when schema_migrations still records versions that were renamed or renumbered away locally. Add no-op historical placeholders for those orphan versions, plus an offline guard and live history check. Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com> * fix(test): allowlist historical migration placeholder stems Unit coverage rejected the six no-op preview-alignment placeholders because duplicate stems must be explicitly allowlisted. Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com> * fix: apply CodeRabbit auto-fixes Fixed 1 file(s) based on 1 unresolved review comment. Co-authored-by: CodeRabbit <noreply@coderabbit.ai> * 🔧 CodeRabbit CI Fix: Fix failing PR Policy CI checks (#954) Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com> * fix(pr-policy): combine duplicate section headings for robust checklist validation The section() function previously used findIndex() which returns only the FIRST occurrence of a heading. When a PR body accidentally contains duplicate section headings (e.g. a prose rationale block followed by a checklist block both under ## Clinical Governance Preflight), only the first block was evaluated and all required checklist items appeared "missing". Fix: iterate over ALL heading occurrences and join their content. This makes the validator combine prose + checklist into a single searchable block, matching how a human reader treats the two blocks as one logical section. Add a self-test covering the duplicate-heading scenario. * Changes before error encountered Agent-Logs-Url: https://github.com/BigSimmo/Database/sessions/4438032f-f568-49b2-9603-947f4c5ac488 * fix: export section helper and use it in ci.yml governance sync --------- Co-authored-by: Cursor Agent <cursoragent@cursor.com> Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com> Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com> Co-authored-by: CodeRabbit <noreply@coderabbit.ai> Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
CI failure fixes was requested by @BigSimmo.
The following files were modified:
PR_POLICY_BODY.md