Skip to content

build(plugin): add Python-free musl native artifacts - #796

Open
kmbroai wants to merge 2 commits into
dev/kyleb/python-free-windows-primitivesfrom
dev/kyleb/python-free-musl-artifacts
Open

build(plugin): add Python-free musl native artifacts#796
kmbroai wants to merge 2 commits into
dev/kyleb/python-free-windows-primitivesfrom
dev/kyleb/python-free-musl-artifacts

Conversation

@kmbroai

@kmbroai kmbroai commented Sep 3, 2026

Copy link
Copy Markdown
Collaborator

Summary

Prepare native OS primitives for Linux installations that use musl, so the TypeScript migration can preserve Alpine support.

Changes

  • Distinguish GNU and musl Linux payloads using the Node diagnostic report header.
  • Add native x64 and arm64 musl builds with pinned Rust and Node container images, offline compilation, and no Python dependency.
  • Check ELF architecture and libc requirements, then test each unchanged binary on Node 20.0 and 22.13.

Testing

  • On the rebased head: TypeScript compilation, SDK/MCP types, formatting, Rust formatting, Ruff, portable source checks, and all nine checker tests passed.
  • The GNU native behavioral proof and musl ELF inspection passed locally.
  • The original implementation is unchanged by the rebase. The earlier x64/arm64 musl proofs on Node 20 and 22 remain recorded in the PR history; fresh hosted jobs must verify this head's pinned containers.

Risk and rollout

Stacked on #795. This prepares build artifacts; product commands continue using their existing helpers. Universal package assembly and platform proofs remain required before TypeScript callers replace Python.

Public disclosure review

  • No customer, partner, prospect, or user identities, data, or identifying details are included.
  • No credentials, personal data, private source, scan findings, or nonpublic links or tickets are included.
  • I reviewed the branch name, title, description, commits, changes, comments, logs, screenshots, attachments, and links for public disclosure.

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Sep 3, 2026

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
🔒 Security Review Completed 2026-09-03T19:11:30.218946Z e2cff42 New commits
ℹ️ 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" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@mldangelo-oai

Copy link
Copy Markdown
Collaborator

I see the correction in 12303d2. The validator now attributes version requirements to their providing libraries, including the libgcc_s.so.1 compatibility case.

The current hosted arm64 musl job is still running. Holding approval until this head passes the ELF check, runs the same artifact through the Node 20 and 22 proofs, and uploads the verified artifact. The previous head stopped before the runtime proofs and upload; the new native-runner result will verify the fix.

@mldangelo-oai mldangelo-oai left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Reviewed the validator correction in 12303d2. The hosted arm64 musl job now passes ELF validation, runs the same artifact successfully on Node 20.0.0 and 22.13.0, and uploads the verified artifact. Both musl architectures and the current SDK CI run pass on this head. This clears my earlier approval hold.

@kmbroai
kmbroai force-pushed the dev/kyleb/python-free-musl-artifacts branch from 12303d2 to 8871500 Compare September 3, 2026 18:27
@kmbroai
kmbroai force-pushed the dev/kyleb/python-free-musl-artifacts branch from 8871500 to e2cff42 Compare September 3, 2026 19:07
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.

2 participants