ci: reuse one package and balance test jobs - #734
Conversation
# Conflicts: # .github/workflows/node-ci.yml # sdk/typescript/tests-ts/cli-cloud-publish.test.ts # sdk/typescript/tests-ts/cli-dedupe.test.ts # sdk/typescript/tests-ts/cli-publish.test.ts # sdk/typescript/tests-ts/cloud-publish.test.ts # sdk/typescript/tests-ts/component-scan.test.ts # sdk/typescript/tests-ts/publication-integration.test.ts # sdk/typescript/tests-ts/publish.test.ts # sdk/typescript/tests-ts/skeleton.test.ts
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
@codex review |
|
Codex Review: Didn't find any major issues. 🎉 Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
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". |
|
@codex review |
|
Codex Review: Didn't find any major issues. Can't wait for the next one! Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
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". |
|
@codex review |
|
Codex Review: Didn't find any major issues. Keep it up! Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
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". |
|
@codex review |
|
Codex Review: Didn't find any major issues. Can't wait for the next one! Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
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". |
Summary
CI repeats the same Bun suite across Node versions, rebuilds the package in each lane, and leaves most Windows tests in two large shards. Build one checked archive, keep the full Bun suite on every OS, and use the supported Node matrix for installed-package checks.
Changes
readdir, and the full-inventory tests still verify that every shared file runs exactly once.Testing
git diff --checkpassed.Risk and rollout
No production code changes. Real process, database, filesystem, and installed-package coverage remains intact across the supported platforms. Package artifacts are required and tied to the current workflow run; dependencies are installed separately on each platform.
The sharding helper retains invalid-count checks and tests for complete inventory coverage. The removed duplicate-input test exercised a case that cannot occur with the filesystem inventory. Refresh duration estimates after expensive tests change. MCP timing reports become available when the native MCP runner lands.
Public disclosure review