Skip to content

Consolidate native CI and remove duplicate API checks - #211

Merged
SaladDay merged 1 commit into
mainfrom
codex/ci-cleanup-20260929
Sep 29, 2026
Merged

SaladDay merged 1 commit into
mainfrom
codex/ci-cleanup-20260929

Conversation

@SaladDay

@SaladDay SaladDay commented Sep 29, 2026 •

Copy link
Copy Markdown
Collaborator

Native daemon and installer CI previously built the same daemon, installed the same Harnesses and probed startup twice on every PR. Their checks now share one Linux/macOS/Windows matrix and test the packaged Harnesses once. Native source and dependency changes trigger that matrix; documentation-only and unrelated Web changes do not. The workflows are named native-check and api-acceptance, superseded native runs are cancelled, and the unused npm-wrapper smoke option is removed.

API acceptance keeps the isolated build, migration commands, pinned official SDK checks and container validation. Its duplicate full service test pass remains covered by core-check. The complete make check gate, workflow lint and release qualification remain intact. There are no production Runtime changes or compatibility aliases.

Validation:

  • Workflow lint, JavaScript syntax and the name guard passed.
  • All four original native race-test commands and all three platforms were retained; the consolidated Linux, macOS and Windows jobs passed.
  • Official-client and container acceptance passed.
  • A fresh independent blind review of the complete diff found no actionable issues.
  • The complete make check gate and daemon build passed in CI run 36530417085. All required repository checks are green.

@SaladDay
SaladDay marked this pull request as ready for review September 29, 2026 06:39
@SaladDay
SaladDay merged commit 9434446 into main Sep 29, 2026
7 checks 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