Skip to content

ci: deepen Node 22 packaged CLI smoke#1125

Merged
thymikee merged 1 commit into
mainfrom
codex-worker/issues1119-1120-docs-ci-20260706
Jul 6, 2026
Merged

ci: deepen Node 22 packaged CLI smoke#1125
thymikee merged 1 commit into
mainfrom
codex-worker/issues1119-1120-docs-ci-20260706

Conversation

@thymikee

@thymikee thymikee commented Jul 6, 2026

Copy link
Copy Markdown
Member

Summary

Document schema-only CLI flag owners in AGENTS.md so cdp/auth/connect/proxy/react-devtools/web route agents to SCHEMA_ONLY_CLI_COMMAND_SCHEMAS.
Extend the packaged CLI Node 22.12 job with doctor --remote --json so CI exercises a daemon-backed packaged dist path beyond --version/help.

Closes #1119
Closes #1120

Validation

pnpm check:tooling passed.
Packed CLI smoke passed locally with a temp global install and isolated state dir: agent-device doctor --remote --json returned JSON through the packaged daemon/runtime path; the temporary daemon PID was stopped afterward.

@github-actions

github-actions Bot commented Jul 6, 2026

Copy link
Copy Markdown

Size Report

Metric Base Current Diff
JS raw 1.5 MB 1.5 MB 0 B
JS gzip 491.8 kB 491.8 kB 0 B
npm tarball 597.4 kB 597.4 kB 0 B
npm unpacked 2.1 MB 2.1 MB 0 B

Startup median (7 runs, lower is better):

Scenario Base Current Diff
CLI --version 27.8 ms 28.9 ms +1.1 ms
CLI --help 52.0 ms 52.6 ms +0.6 ms

Top changed chunks: no changes in the largest emitted chunks.

@thymikee

thymikee commented Jul 6, 2026

Copy link
Copy Markdown
Member Author

Review pass: no actionable blockers found.

Checked against #1119 and #1120. The AGENTS.md additions point schema-only CLI commands at SCHEMA_ONLY_CLI_COMMAND_SCHEMAS, and the Node 22.12 packaged CLI smoke now exercises doctor --remote --json, which loads the packaged daemon/runtime path beyond --version and help.

All reported checks are green, including the 21 CI checks visible on the PR. Marking this ready for human maintainer judgment.

@thymikee thymikee added the ready-for-human Valid work that needs human implementation, judgment, or maintainer merge label Jul 6, 2026
@thymikee thymikee merged commit e4139b6 into main Jul 6, 2026
21 checks passed
@thymikee thymikee deleted the codex-worker/issues1119-1120-docs-ci-20260706 branch July 6, 2026 08:24
@github-actions

github-actions Bot commented Jul 6, 2026

Copy link
Copy Markdown
PR Preview Action v1.8.1
Preview removed because the pull request was closed.
2026-07-06 08:24 UTC

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-for-human Valid work that needs human implementation, judgment, or maintainer merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

ci: deepen Node 22.12 packaged CLI smoke test docs: AGENTS.md should cover schema-only CLI command flag owners

1 participant