chore: update Claude code review wf

[ymc9]

Summary by CodeRabbit

• Chores
  • Updated code review workflow to run only for non-fork PRs.
  • Added conditional to bypass automated review for PRs titled with [WIP].
  • Removed obsolete skip-review logic to simplify workflow behavior.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 13cf901e-5a9f-4b92-a75b-e06cc9b9f127

📥 Commits

Reviewing files that changed from the base of the PR and between 3f2a7a8 and 5d51f42.

📒 Files selected for processing (1)

• .github/workflows/claude-code-review.yml

---

📝 Walkthrough

Walkthrough

Job-level condition added to run only when the PR head repository is not a fork; the Claude Code Review step now skips when the PR title contains [WIP]; an earlier skip-review conditional block at the end of the workflow was removed.

Changes

Cohort / File(s)	Summary
GitHub Actions Workflow Configuration ​.github/workflows/claude-code-review.yml	Added job-level condition to run only if github.event.pull_request.head.repo.fork == false. Added step-level conditional to skip the Claude Code Review step when PR title contains [WIP]. Removed the previous skip-review conditional block at the end of the workflow.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 I twitch my nose at stray forks in sight,
I skip the WIP and hop out of the night,
A tidy check, a gentle bound,
Reviews run only where roots are found. ✨

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title 'chore: update Claude code review wf' accurately summarizes the main change: updating the Claude code review workflow file with conditional improvements.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch chore/claude-review-ci

---

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.}

[coderabbitai]

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Inline comments:
In @.github/workflows/claude-code-review.yml:
- Around line 70-71: The YAML workflow has the `if:` key over-indented causing
parse failure; locate the `if:` line (the conditional starting with
"!contains(github.event.pull_request.title, '[WIP]')") and decrease its
indentation by two spaces so it aligns with sibling keys at the same mapping
level as other top-level workflow steps, ensuring consistent indentation
throughout the block.

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: c52bc7c9-a54f-4205-844d-402553226439

📥 Commits

Reviewing files that changed from the base of the PR and between 87a9f7f and 3f2a7a8.

📒 Files selected for processing (1)

• .github/workflows/claude-code-review.yml

[Copilot]

@ymc9 I've opened a new pull request, #2445, to work on those changes. Once the pull request is ready, I'll request review from you.