chore(ai): Add /create-java-pr Claude Code skill#5119
Conversation
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Semver Impact of This PR🟢 Patch (bug fixes) 📋 Changelog PreviewThis is how your changes will appear in the changelog. This PR will not appear in the changelog. 🤖 This preview updates automatically when you update the PR. |
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 3 potential issues.
Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable autofix in the Cursor dashboard.
| git add CHANGELOG.md | ||
| git commit -m "changelog" | ||
| git push | ||
| ``` |
There was a problem hiding this comment.
Skill always adds changelog entry, ignores #skip-changelog
Medium Severity
Step 6 always instructs adding a changelog entry, but the repo uses #skip-changelog in PR descriptions for meta changes (CI, tooling, skills) that should not appear in the changelog. The skill never checks for or mentions this convention, so it will add changelog entries for PRs that should skip them, contradicting Sentry's craft changelog behavior.
| ``` | ||
|
|
||
| If the push fails due to diverged history, ask the user how to proceed rather than force-pushing. | ||
|
|
There was a problem hiding this comment.
Missing check for empty branch before push
Medium Severity
The skill never verifies that the branch has changes to include in the PR. If the user runs it with no uncommitted changes and no commits ahead of main, the workflow still pushes, creates an empty PR, and adds a changelog entry for it, producing a meaningless PR and changelog pollution.
| - <Short description of the change> ([#<PR_NUMBER>](https://github.com/getsentry/sentry-java/pull/<PR_NUMBER>)) | ||
| ``` | ||
|
|
||
| Use the PR number returned by `sentry-skills:create-pr`. Match the style of existing entries — sentence case, ending with the PR link, no trailing period. |
There was a problem hiding this comment.
No validation of PR creation before changelog step
Medium Severity
Step 6 always runs after Step 5 and uses the PR number from sentry-skills:create-pr.
If create-pr fails or does not return a PR number, the skill still proceeds to Step 6 and adds a changelog entry with a missing or invalid PR number, producing a malformed changelog link.
adinauer
changed the title
/create-java-pr Claude Code skill
1 participant
📜 Description
Add a
create-java-prClaude Code skill that automates the PR creation workflow for sentry-java: branch creation, code formatting, API dump, committing, pushing, PR creation, and changelog updates.💡 Motivation and Context
Standardizes the PR workflow so Claude Code follows the same steps every time, reducing manual effort and ensuring nothing is missed (formatting, API dump, changelog, etc.).
#skip-changelog
💚 How did you test it?
Invoked
/create-java-prin Claude Code and confirmed it follows all steps.📝 Checklist
sendDefaultPIIis enabled.🔮 Next steps
🤖 Generated with Claude Code