Skip to content

Conversation

@MoerAI
Copy link

@MoerAI MoerAI commented Jan 27, 2026

Summary

Add post-publish verification step and fix missing NODE_AUTH_TOKEN in 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) - published
  • oh-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

  1. Add post-publish verification step: After npm publish, verify the package is actually available on npm registry

    • Polls npm registry for up to 60 seconds to account for propagation delay
    • Fails explicitly if package is not found, making the issue visible in CI
  2. Add NODE_AUTH_TOKEN env var: Ensure npm authentication is properly configured for the publish step

Changes

  • .github/workflows/publish-platform.yml:
    • Add NODE_AUTH_TOKEN: ${{ secrets.NPM_TOKEN }} to publish step
    • Add new "Verify publish" step with retry logic

Testing

This is a CI workflow change. To test:

  1. Trigger publish-platform.yml manually with a test version
  2. Verify the new verification step runs and confirms package availability

Related 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.

  • Bug Fixes
    • Add NODE_AUTH_TOKEN to npm publish step for proper authentication.
    • Add a verification step that polls the npm registry for up to 60s and fails if the package isn’t found.

Written for commit 5f34fba. Summary will update on new commits.

@github-actions
Copy link
Contributor

github-actions bot commented Jan 27, 2026

All contributors have signed the CLA. Thank you! ✅
Posted by the CLA Assistant Lite bot.

Copy link

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 1 file

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.

@MoerAI
Copy link
Author

MoerAI commented Jan 27, 2026

I have read the CLA Document and I hereby sign the CLA

@MoerAI
Copy link
Author

MoerAI commented Jan 27, 2026

recheck

- Add verification step to confirm package is available on npm after publish
- Wait up to 60 seconds for npm registry propagation
- Add NODE_AUTH_TOKEN env var for npm authentication
- Fail explicitly if package is not found after publishing

This helps catch silent publish failures like the v3.1.2 Windows binary
issue where the main package was published but platform binaries were not.

Fixes code-yeongyu#1171
@MoerAI MoerAI force-pushed the fix/windows-binary-publish-verification branch from 794f70f to 5f34fba Compare January 27, 2026 09:31
github-actions bot added a commit that referenced this pull request Jan 27, 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.

[Bug] oh-my-opencode-windows-x64@3.1.2 not published to npm

1 participant