Skip to content

fix: harden release publication pipeline - #75

Merged
iHildy merged 1 commit into
mainfrom
ian/fix-release-pipeline
Aug 31, 2026
Merged

fix: harden release publication pipeline#75
iHildy merged 1 commit into
mainfrom
ian/fix-release-pipeline

Conversation

@iHildy

@iHildy iHildy commented Aug 31, 2026

Copy link
Copy Markdown
Owner

Summary

Follow-up to #38 and failed runs 33342806687 and 33342807173.

  • compare release-please string outputs explicitly and use the maintained, SHA-pinned action with one stable manifest configuration
  • validate canonical full commit SHAs without OIDC before checking out release sources
  • build, pack, and smoke the exact artifact without OIDC; publish only that artifact directly to the requested latest or next tag
  • synthesize unique next.<run>.<attempt> versions so prereleases do not consume the final stable version
  • keep the OIDC job checkout-free, script-free, and limited to artifact verification plus npm publish
  • smoke the exact registry version after publication and document that immutable npm versions cannot be rolled back by the post-publish smoke
  • remove mutable input setup, candidate promotion, tag resolution, and invalid reusable-job continue-on-error
  • make dependency setup frozen and add workflow regression tests
  • document that default ~/.agents sync can expose private instructions or skills and can be disabled

The design follows GitHub reusable workflow restrictions, npm trusted publishing, and release-please action outputs.

Verification

  • go run github.com/rhysd/actionlint/cmd/actionlint@v1.7.7 -color
  • bun install --frozen-lockfile
  • bun run check
  • bun test — 107 passed
  • bun run build
  • npm pack --dry-run --jsonopencode-synced@0.9.0, 43 entries
  • live API probes accepted the exact open release-please PR SHA and the commit behind v0.9.0, while an advanced untagged main SHA is rejected for latest
  • full isolated two-instance E2E passed init, push, link, second push, and pull; the ephemeral GitHub repository was deleted

The E2E retained its known warning: the second machine local AGENTS.md did not refresh without restart, but its local sync repository contained the second sentinel and replication was confirmed.

Release PR #38 follow-up

After this merges, release-please must refresh #38 against main. The generated notes still need a manual accuracy pass to:

Commit under review: 06fc12c.

@iHildy
iHildy merged commit 7e37271 into main Aug 31, 2026
2 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