Skip to content

Delegate release retry identity to shared action - #153

Open
Kralizek wants to merge 3 commits into
masterfrom
refactor/shared-release-resume
Open

Kralizek wants to merge 3 commits into
masterfrom
refactor/shared-release-resume

Conversation

@Kralizek

Copy link
Copy Markdown
Owner

Summary

Simplify SMCP's manual release recovery now that calculate-next-version@v0.3 provides operation-specific idempotency.

Changes

  • always let calculate-next-version@v0.3 resolve the manual release version/tag;
  • remove the alternate resumed-version/tag path and resumable-target checkout;
  • reduce resumable-release detection to publication state only: verify whether the calculated tag already has a workflow-owned draft release for the current commit;
  • keep GitHub/NuGet publication recovery local to SMCP;
  • keep dry-run, package-version validation, release asset upload, tag validation, and external release.published support unchanged.

The shared action now owns release identity; SMCP only owns the transaction state of an unfinished publication.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟡 Changes recommended

API failures in resumable-release detection must not be treated as missing releases.

Get a fresh assessment by requesting another Copilot review.

Pull request overview

Simplifies manual release recovery by delegating release identity to calculate-next-version@v0.3.

Changes:

  • Removes alternate resumed-version and target-checkout logic.
  • Uses the calculated tag throughout release publishing.
  • Detects matching workflow-owned draft releases.
File summaries
File Summary
.github/workflows/scripts/detect-resumable-release.sh Detects resumable draft releases for the calculated tag.
.github/workflows/release.yml Uses shared-action outputs for release version and publication.
Review details
  • Files reviewed: 2/2 changed files
  • Comments generated: 1
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread .github/workflows/scripts/detect-resumable-release.sh Outdated
@sonarqubecloud

Copy link
Copy Markdown

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.

2 participants