Skip to content

feat(cli): add multi-platform npm release foundation#27

Merged
Abdulmuiz44 merged 1 commit into
mainfrom
feat/codra-cli-multiplatform-release-foundation
Jun 2, 2026
Merged

feat(cli): add multi-platform npm release foundation#27
Abdulmuiz44 merged 1 commit into
mainfrom
feat/codra-cli-multiplatform-release-foundation

Conversation

@Abdulmuiz44
Copy link
Copy Markdown
Collaborator

  • Adds multi-platform binary packaging foundation for @codra/cli.
  • Adds artifact-based packaging script for linux, macOS, and Windows targets.
  • Adds manual GitHub Actions release workflow draft.
  • Keeps npm publish guarded and disabled by default.
  • Preserves current local host build/test/pack flow.
  • Does not publish to npm yet.

Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 89945aa917

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment on lines +77 to +80
- name: Set up Node.js
uses: actions/setup-node@v4
with:
node-version: 20
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Badge Configure npm registry auth before publishing

When publish: true is used, this job sets NODE_AUTH_TOKEN only on the npm publish step, but the preceding actions/setup-node invocation does not specify registry-url, so it never writes the .npmrc entry that tells npm to use that token. The setup-node docs state that registry-url creates project-level npm config with auth read from env.NODE_AUTH_TOKEN (https://github.com/actions/setup-node#readme), and I also checked npm whoami --registry=https://registry.npmjs.org/ with only NODE_AUTH_TOKEN set locally, which still reports ENEEDAUTH; as a result, the guarded publish path will build and pack successfully but fail at the final npm publish.

Useful? React with 👍 / 👎.

artifact: codra-linux-arm64
bin_name: codra
- target: darwin-x64
os: macos-13
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Badge Use an active macOS Intel runner label

The darwin-x64 matrix leg is pinned to macos-13, but GitHub announced that the macOS 13 runner image was retired by December 4, 2025 and that macos-13 is one of the removed labels (https://github.blog/changelog/2025-09-19-github-actions-macos-13-runner-image-is-closing-down/). On current GitHub-hosted runners this leg will fail before building the Intel macOS binary, so the release workflow cannot collect all required artifacts or publish the multi-platform package.

Useful? React with 👍 / 👎.

@Abdulmuiz44 Abdulmuiz44 merged commit 2d0f8d1 into main Jun 2, 2026
1 check 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