docs: keep PR titles/descriptions uncompressed under caveman mode - #1322
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (2)
📝 WalkthroughWalkthroughThe PR updates handoff and agent documentation to require verbatim PR template content, required governance and RAG fields, and awareness of strict policy parsing and validation. ChangesPR policy guidance
Estimated code review effort: 1 (Trivial) | ~5 minutes Possibly related PRs
Suggested labels: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
Comment |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 29c58693d6
ℹ️ 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".
|
Closing as superseded by clean rebase: #1335 Useful content kept: AGENTS “PR titles/descriptions are parsed input” bullet + handoff step requiring full Dropped/stale: tip’s hand-written ledger-row instruction — current |
|
This pull request has been ignored for the connected project Preview Branches by Supabase. |
…man-fix-6875bb # Conflicts: # .claude/skills/handoff/SKILL.md
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 7e96cc3791
ℹ️ 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".
Summary
pr-policy.ymlrunsscripts/pr-policy.mjsagainst the exact PR title/body text and hard-blocks the merge when a clinical-risk diff lacks a complete## Clinical Governance Preflightor a RAG-ranking-surface diff lacks a satisfyingRAG impact:line. Caveman-mode output-style compression can paraphrase/shorten that exact-parsed text and silently trip the gate. Added an explicit carve-out (AGENTS.md, handoff skill) so PR titles/descriptions are always written in full normal prose from.github/pull_request_template.md, regardless of active output style.Verification
npm run check:pr-policy(self-test + workflow guard) — passednpx prettier --check AGENTS.md .claude/skills/handoff/SKILL.md— passednpm run lint— passed, 0 warningsnpm run typecheck/npm run test/npm run check:rag:fixtures— blocked >10 min by cross-worktree heavyweight-lock contention (other worktrees running full vitest/Playwright suites). Diff is docs-only (AGENTS.md+ one.claude/skillsfile, nosrc//scripts//test files touched); hosted CI runs these independently.Risk and rollout
🤖 Generated with Claude Code
Summary by CodeRabbit