Skip to content

Fix version detection for desktop release packaging - #114

Merged
petertzy merged 2 commits into
mainfrom
fix/desktop-release-version-regex
Oct 21, 2026
Merged

petertzy merged 2 commits into
mainfrom
fix/desktop-release-version-regex

Conversation

@petertzy

Copy link
Copy Markdown
Owner

Problem

The packaging workflow used an over-escaped version regex, so it found no existing releases and assigned v0.1.0. The publishing workflow then correctly rejected the existing tag.

Fix

  • Correct the grep regex for vMAJOR.MINOR.PATCH release tags.
  • Fail safely when no matching release history is found, rather than silently selecting v0.1.0.

Validation

The new workflow has not been executed yet. Re-run Desktop packaging after merging, then Publish desktop release. Previous artifacts contain the wrong version and must not be reused.

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 21, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-21T08:35:16.827498Z 72d95e3 PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@petertzy
petertzy merged commit ac79ce1 into main Oct 21, 2026
4 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