SYM-208: add PR checklist local preflight

[jwalin-shah]

Summary

• Adds local PR checklist validation for generated PR body files, PR_BODY, and stdin pipelines.
• Tightens checklist parsing so every unchecked item fails, including unchecked N/A lines.
• Documents the local preflight command for Codex/Linear agents before PR handoff.
• Adds a root pnpm pr:checklist alias for the existing script.

Problem

Codex-authored PRs can fail the PR Submission Checklist workflow after handoff because unchecked template items are only discovered once CI runs. The existing parser also treated unchecked N/A checklist items as satisfied.

Solution

• scripts/check-pr-checklist.mjs now reads a PR body from a file path, stdin (-), or PR_BODY.
• scripts/lib/checklist-parser.mjs now requires every checklist item to be checked.
• docs/agent-workflows/codex-pr-checklist.md now includes local preflight commands for generated body files and gh pr view pipelines.

Submission Checklist

• Tests added or updated (happy path + at least one failure / edge case) per docs/TESTING-STRATEGY.md (N/A: script behavior validated with direct good and bad CLI fixtures)
• Diff coverage >= 80% - changed lines (Vitest + cargo-llvm-cov merged via diff-cover) meet the gate enforced by .github/workflows/coverage.yml. Run pnpm test:coverage and pnpm test:rust locally; PRs below 80% on changed lines will not merge. (N/A: docs/root script change outside app and Rust coverage harnesses)
• Coverage matrix updated - added/removed/renamed feature rows in docs/TEST-COVERAGE-MATRIX.md reflect this change (N/A: agent workflow/script tooling only)
• All affected feature IDs from the matrix are listed in the PR description under ## Related (N/A: no coverage matrix feature IDs affected)
• No new external network dependencies introduced (mock backend used per docs/TESTING-STRATEGY.md
• Manual smoke checklist updated if this touches release-cut surfaces (docs/RELEASE-MANUAL-SMOKE.md) (N/A: not a release-cut surface)
• Linked issue closed via Closes #NNN in the ## Related section (N/A: Linear issue only)

Impact

No runtime app impact. This only affects repository tooling and agent workflow documentation.

Related

• Linear: https://linear.app/symphony-test-jwalin/issue/SYM-208/relaunch-openhuman-add-pr-checklist-local-preflight
• Closes: N/A
• Follow-up PR(s)/TODOs: N/A

---

AI Authored PR Metadata (required for Codex/Linear PRs)

Linear Issue

• Key: SYM-208
• URL: https://linear.app/symphony-test-jwalin/issue/SYM-208/relaunch-openhuman-add-pr-checklist-local-preflight

Commit & Branch

• Branch: codex/SYM-208-pr-checklist-preflight
• Commit SHA: ff1d78c5fe801f29cccfc6ea3e1139f595b80494

Validation Run

• pnpm --filter openhuman-app format:check (N/A: no app workspace files changed)
• pnpm typecheck (N/A: no TypeScript app files changed)
• Focused tests: pnpm pr:checklist "$ok_file"; pnpm pr:checklist "$bad_file" expected failure; PR_BODY="$(cat "$ok_file")" pnpm pr:checklist; printf ... | pnpm pr:checklist -
• Rust fmt/check (if changed): N/A, no Rust changed
• Tauri fmt/check (if changed): N/A, no Tauri changed

Validation Blocked

• command: N/A
• error: N/A
• impact: N/A

Behavior Changes

• Intended behavior change: unchecked checklist items now fail even when the text starts with N/A; N/A items must be checked with a reason.
• User-visible effect: AI-authored PRs can validate body checklist completion before CI.

Parity Contract

• Legacy behavior preserved: existing PR_BODY workflow remains supported.
• Guard/fallback/dispatch parity checks: file input and stdin are additive input modes; no workflow changes required.

Duplicate / Superseded PR Handling

• Duplicate PR(s): N/A
• Canonical PR: this PR
• Resolution (closed/superseded/updated): N/A

[coderabbitai]

Warning

Rate limit exceeded

@jwalin-shah has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 60 minutes before requesting another review.

To continue reviewing without waiting, purchase usage credits in the billing tab.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 866e45de-84ce-4163-8a3f-0653110c5d89

📥 Commits

Reviewing files that changed from the base of the PR and between 5f3a969 and 5b42f09.

📒 Files selected for processing (5)

• app/test/checklist-parser.test.ts
• docs/agent-workflows/codex-pr-checklist.md
• package.json
• scripts/check-pr-checklist.mjs
• scripts/lib/checklist-parser.mjs

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}