Skip to content

fix: scope runner diagnostics to sessions#704

Merged
thymikee merged 1 commit into
mainfrom
codex/session-scoped-runner-logs
Jun 7, 2026
Merged

fix: scope runner diagnostics to sessions#704
thymikee merged 1 commit into
mainfrom
codex/session-scoped-runner-logs

Conversation

@thymikee
Copy link
Copy Markdown
Member

@thymikee thymikee commented Jun 7, 2026

Summary

Route daemon request diagnostics into session request files and Apple runner/xcodebuild output into session-scoped runner logs, while keeping the shared daemon log for lifecycle and pre-session failures.

Expose runner/request log paths in open JSON and session inventory, and update help, docs, and AGENTS guidance for concurrent agents.

Touched 20 files; scope stayed within daemon request logging/session artifacts, Apple runner log append safety, client output types, tests, and docs.

Validation

  • pnpm exec vitest run src/daemon/tests/request-execution-scope.test.ts src/daemon/tests/request-router-open.test.ts src/utils/tests/args.test.ts
  • pnpm check:quick
  • pnpm test:smoke (4 passed, 3 loopback-dependent cases skipped by environment)
  • pnpm check:unit was rerun outside the sandbox; the full sweep had one runner-xctestrun timeout, and the focused rerun of src/platforms/ios/tests/runner-xctestrun.test.ts passed.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jun 7, 2026

PR Preview Action v1.8.1
Preview removed because the pull request was closed.
2026-06-07 09:35 UTC

@thymikee thymikee force-pushed the codex/session-scoped-runner-logs branch from 1a7c35b to b6992a7 Compare June 7, 2026 09:06
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jun 7, 2026

Size Report

Metric Base Current Diff
JS raw 1.1 MB 1.1 MB +1.9 kB
JS gzip 371.4 kB 371.8 kB +394 B
npm tarball 482.0 kB 480.1 kB -1.9 kB
npm unpacked 1.6 MB 1.6 MB -10.7 kB

Startup median (7 runs, lower is better):

Scenario Base Current Diff
CLI --version 26.6 ms 26.8 ms +0.2 ms
CLI --help 41.4 ms 41.6 ms +0.2 ms

Top changed chunks:

Chunk Raw diff Gzip diff
dist/src/2415.js +1.1 kB +214 B
dist/src/cli.js -202 B -88 B
dist/src/session.js +310 B +54 B
dist/src/1352.js +115 B +51 B
dist/src/8699.js +74 B +29 B

@thymikee thymikee force-pushed the codex/session-scoped-runner-logs branch 2 times, most recently from 43235a4 to ac6dc54 Compare June 7, 2026 09:13
@thymikee thymikee force-pushed the codex/session-scoped-runner-logs branch from ac6dc54 to bf7551a Compare June 7, 2026 09:20
@thymikee thymikee merged commit 8697199 into main Jun 7, 2026
19 checks passed
@thymikee thymikee deleted the codex/session-scoped-runner-logs branch June 7, 2026 09:35
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