Skip to content

fix(codex): pass session cwd to app-server#316

Merged
tiann merged 1 commit intotiann:mainfrom
ROOOO:pr/codex-session-cwd
Mar 19, 2026
Merged

fix(codex): pass session cwd to app-server#316
tiann merged 1 commit intotiann:mainfrom
ROOOO:pr/codex-session-cwd

Conversation

@ROOOO
Copy link
Contributor

@ROOOO ROOOO commented Mar 19, 2026

Summary

  • pass session.path into Codex app-server thread startup
  • pass the same cwd into follow-up turn requests
  • add tests for the generated app-server payload

Testing

  • bunx vitest run src/codex/utils/appServerConfig.test.ts

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Findings

  • None.

Summary

  • Review mode: initial. No issues found in modified lines. Residual risk: runtime behavior not exercised here; tests not run locally.

Testing

  • Not run (automation). Suggested: bunx vitest run src/codex/utils/appServerConfig.test.ts

HAPI Bot

@tiann tiann merged commit 987010f into tiann:main Mar 19, 2026
2 checks passed
@ROOOO ROOOO deleted the pr/codex-session-cwd branch March 19, 2026 13:17
Hwwwww-dev pushed a commit to Hwwwww-dev/hapi that referenced this pull request Mar 20, 2026
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