Skip to content

fix(push): update gateway release contract - #3838

Closed
brow wants to merge 1 commit into
mainfrom
fix-gateway-release-contract
Closed

fix(push): update gateway release contract#3838
brow wants to merge 1 commit into
mainfrom
fix-gateway-release-contract

Conversation

@brow

@brow brow commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

What changed?

Updated the push gateway chart release contract to assert the live tag-based release path. The test now pins the Create and push tag step, its resolved tag input, the refs/tags/$TAG destination, and the target SHA passed to GitHub.

Why?

Commit 0b297edbd replaced direct workflow dispatch with GitHub App-authored tags, but the contract test continued requiring the deleted dispatch code. That stale assertion broke chart validation on main and in run 30594012619.

The push gateway workflow runs only when its chart path or workflow file changes, so this contract may otherwise remain unexercised for long periods. Its publish job requires validate, which currently prevents push-chart-v* releases until this test is fixed. The chart remains at 0.1.0, and push-chart-v0.1.0 already exists.

Originating Buzz channel: e241d25e-05da-4b94-a3a4-faa41f4aec31.

How is it tested?

Updated test:

The fixed contract and shell syntax checks pass locally. Mutation checks independently break all four new tag-creation needles, delete the complete tag-creation step, and spot-check the existing tag-prefix and publisher-trigger needles. Every mutation fails, and every restored contract passes.

@brow

brow commented Jul 31, 2026

Copy link
Copy Markdown
Contributor Author

@builderbot review

@brow

brow commented Jul 31, 2026

Copy link
Copy Markdown
Contributor Author

@codex review

@chatgpt-codex-connector

Copy link
Copy Markdown

To use Codex here, create a Codex account and connect to github.

Co-authored-by: npub1tquskdu6yc4h8l7xxtceculxw600grekeq0xg2ukqfrwl7vrzg3quz3gmp <58390b379a262b73ffc632f19c73e6769ef40f36c81e642b960246eff9831222@buzz.block.builderlab.xyz>
Signed-off-by: npub1tquskdu6yc4h8l7xxtceculxw600grekeq0xg2ukqfrwl7vrzg3quz3gmp <58390b379a262b73ffc632f19c73e6769ef40f36c81e642b960246eff9831222@buzz.block.builderlab.xyz>
@brow
brow force-pushed the fix-gateway-release-contract branch from 7d0fea2 to 9548fc7 Compare July 31, 2026 01:23
@brow

brow commented Jul 31, 2026

Copy link
Copy Markdown
Contributor Author

@builderbot review

@brow

brow commented Jul 31, 2026

Copy link
Copy Markdown
Contributor Author

@codex review

@chatgpt-codex-connector

Copy link
Copy Markdown

To use Codex here, create a Codex account and connect to github.

@brow

brow commented Jul 31, 2026

Copy link
Copy Markdown
Contributor Author

Superseded: tomb directed all push work onto the single feature branch push-ios-blockers-1-2 (draft #2744). This fix landed there as 0e3389a with the stronger tag-creation needles (whole-step deletion now caught).

@brow brow closed this Jul 31, 2026
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