fix(ci): enable platform package publishing #829
Closed
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Remove SKIP_PLATFORM_PACKAGES flag that was temporarily added as a workaround
when OIDC wasn't configured. This was preventing the 7 platform-specific
binary packages from being published to npm, causing 404 errors when users
tried to install oh-my-opencode.
The platform packages (oh-my-opencode-{darwin,linux,windows}-{arm64,x64}, etc.)
contain the Bun single-file executables for each platform and are required
for the CLI to work without Bun installed.
![cubic-dev-ai[bot]](https://avatars.githubusercontent.com/in/1082092?s=60&v=4){kind=small}
jkoelker
reviewed
Jan 16, 2026
| CI: true | ||
| GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} | ||
| NPM_CONFIG_PROVENANCE: true | ||
| SKIP_PLATFORM_PACKAGES: true |
Contributor
There was a problem hiding this comment.
This still needs an NPM_TOKEN to be injected, just FYI
Collaborator
Author
There was a problem hiding this comment.
yep, working with Q to get that done b4 merging
Owner
|
Closing as duplicate - the same change was already pushed directly in commit ec2cf22 on Jan 17. Thanks for the PR @kdcokenny! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Problem
The platform-specific npm packages (oh-my-opencode-linux-x64, oh-my-opencode-darwin-arm64, etc.) are returning 404 from npm because they were never published.
Root Cause
SKIP_PLATFORM_PACKAGES: truewas added in commit 0230e71 by @justsisyphus on Jan 16 as a temporary workaround when OIDC wasn't configured for the platform packages. The 'for now' workaround was never removed.Fix
Remove the
SKIP_PLATFORM_PACKAGES: trueflag so the publish workflow publishes all 8 packages (main + 7 platform binaries).Note
For the first publish of the platform packages, ensure
NPM_TOKENis configured as a repo secret since OIDC/Trusted Publisher can only be configured after the packages exist on npm.After the first successful publish, configure Trusted Publisher on npmjs.com for each of the 8 packages to enable OIDC authentication going forward.
Summary by cubic
Enable publishing of platform-specific npm packages by removing SKIP_PLATFORM_PACKAGES from the CI workflow. Fixes 404 installs for platform binaries and ensures the main package plus 7 platform packages are published.
Written for commit f76e909. Summary will update on new commits.