Skip to content

Skip store publish when the live version matches the manifest#108

Merged
gilgold merged 1 commit into
mainfrom
ci/skip-published-version
Jun 10, 2026
Merged

Skip store publish when the live version matches the manifest#108
gilgold merged 1 commit into
mainfrom
ci/skip-published-version

Conversation

@gilgold

@gilgold gilgold commented Jun 10, 2026

Copy link
Copy Markdown
Owner

Summary

Every merge to main without a manifest version bump currently produces a red Chrome publish step (the store rejects re-uploads of the live version — PKG_INVALID_VERSION_NUMBER). Both publish steps now check the publicly available live version first and skip cleanly when it equals the manifest version:

  • Chrome: queried from the official update service (clients2.google.com/service/update2/crx)
  • Edge: queried from the store's product-details endpoint (same one the README badge uses)

Fail-open by design: if the version lookup fails or returns nothing, the step proceeds and lets the store API decide. The Edge listing lags while a submission is in review, so the guard only catches the fully-published case — an in-review duplicate still surfaces as an API error, which is the correct signal.

Verified both endpoints locally: Chrome returns 4.1.2 (live), Edge returns 3.5.9 (4.1.2 currently in review).

🤖 Generated with Claude Code

Both stores reject re-uploads of an already-published version, so the
publish steps now check the public listing version first (Chrome's
update service / Edge's product-details endpoint) and exit cleanly on
a match. Lookup failures fall through to the store API. This keeps
main green on merges that don't bump the manifest version.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@gilgold
gilgold merged commit 4ea1e08 into main Jun 10, 2026
3 checks passed
@gilgold
gilgold deleted the ci/skip-published-version branch June 10, 2026 17:25
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