Skip to content

Fix PR comment and version PR description + related logic#8

Merged
theoephraim merged 6 commits into
mainfrom
fix/ci-comment-frog-images
Apr 15, 2026
Merged

Fix PR comment and version PR description + related logic#8
theoephraim merged 6 commits into
mainfrom
fix/ci-comment-frog-images

Conversation

@theoephraim

@theoephraim theoephraim commented Apr 15, 2026

Copy link
Copy Markdown
Member

Summary

  • Restyle the CI check PR comment to use frog images matching the version PR description
  • Filter changesets to only those added/modified in the PR (not all lingering changesets)
  • Show packages grouped by bump type with frog image section headers
  • List each changeset file with links to view the diff and edit the file on GitHub
  • Add "click here to add a changeset" link that opens GitHub's new file UI with a pre-filled template
  • Use detected package manager for CLI instructions (bunx bumpy add, npx bumpy add, etc.)
  • Use frog-neutral.png in the no-changesets comment with an "add changeset" link
  • Extract getChangedFiles to shared git utility, export detectPackageManager

Test plan

  • Verify CI check comment renders correctly on a PR with changesets
  • Verify no-changesets comment renders correctly
  • Verify "view diff" and "edit" links work
  • Verify "add changeset" link opens GitHub new file UI with template
  • Verify correct package manager runner is shown (npx/bunx/pnpm/yarn)

🤖 Generated with Claude Code

theoephraim and others added 4 commits April 14, 2026 22:22
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
- Filter changesets to only those added/modified in the PR
- Show package list with frog image headers and changeset file links
- Add view-diff and edit links for each changeset file
- Add "click to add changeset" link for GitHub UI
- Use detected package manager for CLI instructions
- Extract getChangedFiles to shared git utility

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
The gh CLI returns GraphQL node IDs in comments JSON, but the REST
API PATCH endpoint requires numeric IDs. Extract from the URL instead.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
-f treats @- as a literal string, -F reads from stdin.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@github-actions

github-actions Bot commented Apr 15, 2026

Copy link
Copy Markdown

bumpy-frog

The changes in this PR will be included in the next version bump.

patch Patch releases

  • @varlock/bumpy 0.0.1 → 0.0.2

Changesets in this PR

Click here if you want to add another changeset to this PR


This comment is maintained by bumpy.

Fetch origin base branch when not available, which happens in shallow
clones created by actions/checkout with default fetch-depth: 1.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@theoephraim theoephraim changed the title Restyle CI check PR comment with frog images Fix PR comment and version PR description + related logic Apr 15, 2026
@dmno-dev dmno-dev deleted a comment from github-actions Bot Apr 15, 2026
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@theoephraim theoephraim merged commit 437cd10 into main Apr 15, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant