fix(ci): add post-publish verification for platform binaries #1172
+24
−0
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
Add post-publish verification step and fix missing
NODE_AUTH_TOKENin the platform publish workflow to prevent silent publish failures.Problem
The v3.1.2 Windows binary was not published to npm while the main package was published successfully (#1171). This caused users to receive a version mismatch:
oh-my-opencode@3.1.2(main package) - publishedoh-my-opencode-windows-x64@3.1.2- NOT published (latest was still 3.1.1)Users still experienced the cross-compilation segfault bug (oven-sh/bun#18416) because they got the old v3.1.1 binary.
Solution
Add post-publish verification step: After
npm publish, verify the package is actually available on npm registryAdd
NODE_AUTH_TOKENenv var: Ensure npm authentication is properly configured for the publish stepChanges
.github/workflows/publish-platform.yml:NODE_AUTH_TOKEN: ${{ secrets.NPM_TOKEN }}to publish stepTesting
This is a CI workflow change. To test:
publish-platform.ymlmanually with a test versionRelated Issues
Summary by cubic
Adds a post-publish verification step for platform binaries and sets NODE_AUTH_TOKEN in the publish workflow. This prevents silent npm publish failures like the missing Windows v3.1.2 binary; fixes #1171.
Written for commit 5f34fba. Summary will update on new commits.