docs(skills): add create-pr lifecycle rule - #1084
Conversation
Add a repo-local create-pr skill so PR workflow policy lives with the repository instead of relying on a global skill copy. The skill now owns the full PR lifecycle: branch setup, atomic commits, PR creation, AI review requests, review-thread replies, CI polling, and completion criteria. Split the operational details into focused references for branch and commit setup, PR opening, AI reviewer handling, CI handling, and final readiness. The AI review reference requires CodeRabbit mentions and documents the verified Cubic handle @cubic-dev-ai when Cubic is available.
Apply the repository formatter to native package manifests so the pre-push oxfmt check passes. This moves the existing commonjs type field to the formatter-preferred position without changing package behavior.
|
@coderabbitai review |
@ryoppippi I have started the AI code review. It will take a few minutes to complete. |
|
✅ Actions performedReview triggered.
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (7)
💤 Files with no reviewable changes (1)
✅ Files skipped from review due to trivial changes (3)
🚧 Files skipped from review as they are similar to previous changes (1)
📝 WalkthroughWalkthroughAdds a new Changescreate-pr Skill Documentation
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
Deploying with
|
| Status | Name | Latest Commit | Preview URL | Updated (UTC) |
|---|---|---|---|---|
| ✅ Deployment successful! View logs |
ccusage-guide | b55da7d | Commit Preview URL Branch Preview URL |
May 19 2026, 10:26 PM |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In @.agents/skills/create-pr/references/branch-and-commit.md:
- Line 17: Replace the nonstandard spelling "revertable" with "revertible" in
the sentence "Use the `commit` skill to create atomic, revertable Conventional
Commits." so it reads "Use the `commit` skill to create atomic, revertible
Conventional Commits." making sure only the single word is changed and
punctuation/formatting (including the inline `commit` code span and
"Conventional Commits") remain unchanged.
In @.agents/skills/create-pr/SKILL.md:
- Line 33: The commit-quality rule text currently uses the incorrect spelling
"revertable"; update that phrase to "revertible" in the string "Keep commits
atomic and independently revertable; use the `commit` skill." so it reads "Keep
commits atomic and independently revertible; use the `commit` skill." and ensure
any other occurrences of "revertable" in the same document are corrected to
"revertible" as well.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: d6e510aa-66b9-4afe-9e66-7f3b7a042a18
📒 Files selected for processing (13)
.agents/skills/create-pr/SKILL.md.agents/skills/create-pr/references/ai-review.md.agents/skills/create-pr/references/branch-and-commit.md.agents/skills/create-pr/references/ci.md.agents/skills/create-pr/references/completion.md.agents/skills/create-pr/references/open-pr.mdCLAUDE.mdpackages/ccusage-darwin-arm64/package.jsonpackages/ccusage-darwin-x64/package.jsonpackages/ccusage-linux-arm64/package.jsonpackages/ccusage-linux-x64/package.jsonpackages/ccusage-win32-arm64/package.jsonpackages/ccusage-win32-x64/package.json
ccusage
@ccusage/ccusage-darwin-arm64
@ccusage/ccusage-darwin-x64
@ccusage/ccusage-linux-arm64
@ccusage/ccusage-linux-x64
@ccusage/ccusage-win32-arm64
@ccusage/ccusage-win32-x64
commit: |
ccusage performance comparisonPR SHA: This compares the Rust PR release binary against the configured base package on the same CI runner. Package runner startupExecution setup measures any pre-benchmark package materialization used by the execution benchmark. Bunx temp cache measures one
Cached bunx execution performanceRuns the same large fixture through Fixtures: Claude
Package runtime diagnosticsCompares the PR package wrapper, the installed native optional dependency binary, and the workspace release binary on the same large fixture. This identifies whether slow package results come from JavaScript wrapper overhead, the published native binary build, or the Rust core itself. Fixtures: Claude
Committed fixture performanceCommitted small fixtures for stable PR-to-PR feedback and explicit Claude/Codex command coverage. Fixtures: Claude
Large real-world-shaped fixture performanceGenerated fixtures shaped from aggregate local log statistics: thousands of JSONL files, many small sessions, and a long tail of larger sessions. No real prompts, paths, or outputs are stored in the fixtures. Fixtures: Claude
Artifact size
Lower medians and smaller artifacts are better. CI runner noise still applies; use same-run ratios as directional PR feedback, not release guarantees. |
ccusage performance comparisonPR SHA: This compares the PR package against the configured base package on the same CI runner. Package runner startupExecution setup measures any pre-benchmark package materialization used by the execution benchmark. Bunx temp cache measures one
Cached bunx execution performanceRuns the same large fixture through Fixtures: Claude
Package runtime diagnosticsCompares the PR package wrapper, the installed native optional dependency binary, and the workspace release binary on the same large fixture. This identifies whether slow package results come from JavaScript wrapper overhead, the published native binary build, or the Rust core itself. Fixtures: Claude
Committed fixture performanceCommitted small fixtures for stable PR-to-PR feedback and explicit Claude/Codex command coverage. Fixtures: Claude
Large real-world-shaped fixture performanceGenerated fixtures shaped from aggregate local log statistics: thousands of JSONL files, many small sessions, and a long tail of larger sessions. No real prompts, paths, or outputs are stored in the fixtures. Fixtures: Claude
Artifact size
Lower medians and smaller artifacts are better. CI runner noise still applies; use same-run ratios as directional PR feedback, not release guarantees. |
Address CodeRabbit feedback by using the standard spelling revertible in the create-pr skill commit guidance.
…bbit-loop' into codex/add-create-pr-skill-coderabbit-loop
Avoid making fish the default shell in the repo-local create-pr skill. The guidance now tells agents to match the active shell and only calls out fish for the PR body stdin pattern where heredoc syntax differs.
|
@coderabbitai review |
@ryoppippi I have started the AI code review. It will take a few minutes to complete. |
🧩 Analysis chain
🏁 Script executed: Length of output: 10950 CodeRabbit Review — PR
|
| File | Status |
|---|---|
SKILL.md |
✅ Clear, self-contained, good "Always Apply" rules |
references/ai-review.md |
✅ Correct gh api graphql usage (-F for typed fields), correct reply endpoint |
references/branch-and-commit.md |
✅ |
references/ci.md |
✅ |
references/completion.md |
✅ Completion gates are precise and unambiguous |
references/open-pr.md |
✅ <<'EOF' heredoc and fish printf alternatives are correct; --body-file - guidance is solid |
package.json files |
✅ Pure field reordering, no behavioral change |
✅ Actions performed
Review triggered.
Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.
Mark create-pr as the single entry point for PR work in CLAUDE.md. This removes the apparent peer-level overlap with pr-ai-review-workflow while keeping that workflow documented as a helper used by create-pr for review-loop mechanics.
|
@coderabbitai review\n@cubic-dev-ai review\n\n@coderabbitai Addressed your latest actionable feedback in 7ea4372 by marking create-pr as the single PR-work entry point in CLAUDE.md and documenting pr-ai-review-workflow as its helper for review-loop mechanics. Validation: pre-commit hooks and pre-push hooks passed. |
|
Tip For best results, initiate chat on the files or code changes. 🧩 Analysis chain🏁 Script executed: Length of output: 541
The actionable item is resolved. 🐇 ʕ•ᴥ•ʔ If you found this review helpful, would you consider giving us a shout-out on X? Thank you for using CodeRabbit! |
Fold the repo-local pr-ai-review-workflow skill into create-pr so PR creation, AI review requests, review replies, CI handling, and completion gates have a single skill entry point. Move the reusable gh review command reference under create-pr, update skill routing and cross-skill references, and remove the old standalone review workflow skill.
|
@coderabbitai review\n@cubic-dev-ai review\n\n@coderabbitai Integrated pr-ai-review-workflow into create-pr in b55da7d: create-pr is now the single PR workflow skill, the reusable gh review commands moved under create-pr references, and old pr-ai-review-workflow routing/files were removed. Validation: targeted oxfmt check, pre-commit hooks, and pre-push hooks passed. |
|
Tip For best results, initiate chat on the files or code changes. 🧩 Analysis chain🏁 Script executed: Length of output: 4073
Here's what the scripts confirmed in the current branch state:
|
1 participant
Adds a repo-local `create-pr` skill as the single PR lifecycle rule for this repository. The skill covers branch setup, atomic commits, PR creation, AI review requests, review-thread replies, CI handling, and completion criteria, with each step split into focused references.
It requires CodeRabbit mentions for review and thread replies, and documents `@cubic-dev-ai` as the Cubic reviewer handle when Cubic is available. A follow-up formatter commit updates native package manifests so the repository pre-push `oxfmt --check` hook passes.
Testing:
Summary by cubic
Adds a repo-local
create-prskill as the single PR lifecycle rule and routes all PR work to it inCLAUDE.md. Merges the priorpr-ai-review-workflowintocreate-pr, moves the review command reference under it, and removes the old skill.New Features
.agents/skills/create-pr/SKILL.mdwith focused references:branch-and-commit,open-pr,ai-review,ci,completion,gh-review-commands.@coderabbitai; add@cubic-dev-aiwhen available.gh pr createusage: match active shell; pass bodies via stdin; for fish, pipeprintfinto--body-file -; avoid literal\nin--body.Refactors
.agents/skills/fix-ci/SKILL.mdand.agents/skills/ccusage-development/SKILL.mdto referencecreate-prand use “revertible”.gh-review-commandsexamples now mention the reviewer bot in replies and top-level comments.Written for commit b55da7d. Summary will update on new commits. Review in cubic
Summary by CodeRabbit
Documentation
Chores