Surface the released version in the Cut release run UI#13
Conversation
Set a dynamic run-name so the Actions list shows 'Cut release vX.Y.Z' instead of 'Cut release #1', and add a job summary panel with the canonical resolved tag, source commit, and release link.
📝 WalkthroughWalkthroughThe ChangesRelease workflow visibility
Estimated code review effort🎯 2 (Simple) | ⏱️ ~8 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)
Comment |
There was a problem hiding this comment.
🧹 Nitpick comments (1)
.github/workflows/cut-release.yml (1)
93-101: ⚡ Quick winConsider using environment variables for consistency and to address the static analysis warning.
The static analyzer flags line 95 for template injection risk. While the value is validated by the semver regex on lines 48-51 and should be safe, this workflow already uses the defensive env-var pattern on line 44:
env: INPUT_VERSION: ${{ inputs.version }} # via env, never inlined into the shellFor consistency and to silence the static analyzer, consider applying the same pattern here.
♻️ Proposed refactor to use environment variables
- name: Summarise the release + env: + TAG: ${{ steps.tag.outputs.tag }} run: | - tag="${{ steps.tag.outputs.tag }}" + tag="${TAG}" { echo "## Released ${tag} :rocket:" echo "" echo "- **Tag:** \`${tag}\` (from commit \`${GITHUB_SHA:0:7}\` on \`main\`)" echo "- **Release:** ${GITHUB_SERVER_URL}/${GITHUB_REPOSITORY}/releases/tag/${tag}" } >> "$GITHUB_STEP_SUMMARY"🤖 Prompt for 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. In @.github/workflows/cut-release.yml around lines 93 - 101, The release summary step in the workflow inlines the tag via tag="${{ steps.tag.outputs.tag }}" which triggers a template-injection warning; change it to pass the tag into the shell via an env var (e.g. set env: RELEASE_TAG: ${{ steps.tag.outputs.tag }}) and then reference $RELEASE_TAG inside the run block (update the echo lines that use ${tag} to use $RELEASE_TAG) while leaving GITHUB_SHA and GITHUB_STEP_SUMMARY references unchanged; this mirrors the defensive pattern used for INPUT_VERSION and will silence the static analyzer.
🤖 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.
Nitpick comments:
In @.github/workflows/cut-release.yml:
- Around line 93-101: The release summary step in the workflow inlines the tag
via tag="${{ steps.tag.outputs.tag }}" which triggers a template-injection
warning; change it to pass the tag into the shell via an env var (e.g. set env:
RELEASE_TAG: ${{ steps.tag.outputs.tag }}) and then reference $RELEASE_TAG
inside the run block (update the echo lines that use ${tag} to use $RELEASE_TAG)
while leaving GITHUB_SHA and GITHUB_STEP_SUMMARY references unchanged; this
mirrors the defensive pattern used for INPUT_VERSION and will silence the static
analyzer.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: 243240ca-6e85-46d4-a976-2bd82546d98f
📒 Files selected for processing (1)
.github/workflows/cut-release.yml
What
Two changes to
.github/workflows/cut-release.ymlso the cut version is visible in the GitHub UI:run-name— the Actions run list now showsCut release vX.Y.Zinstead ofCut release #1, so runs are distinguishable at a glance.v-prefixed), the source commit, and a direct release link. This is the authoritative version, sincerun-nameechoes the raw dispatch input which may omit thev.Why
When cutting a release there was no easy way to tell which version a run produced from the Actions list view.
Notes
run-nameis evaluated at dispatch time, so existing runs keep their titles.Summary by CodeRabbit