Skip to content

fix: make Codex project roots writable - #22

Merged
pathscale merged 2 commits into
masterfrom
fix/codex-cwd-writable
Aug 1, 2026
Merged

fix: make Codex project roots writable#22
pathscale merged 2 commits into
masterfrom
fix/codex-cwd-writable

Conversation

@pathscale

@pathscale pathscale commented Aug 1, 2026

Copy link
Copy Markdown
Owner

Summary

  • include the requested cwd as well as additional directories in Codex app-server workspace-write roots
  • prioritize host controls over continuous app-server output so mid-turn steering and approvals are delivered promptly
  • preserve Codex assistant-message boundaries so adjacent messages cannot glue prose to a Prompt Syntax directive
  • add a real Codex regression proving a one-directory project can write inside its cwd
  • bump agent-abstraction to 0.4.6 in an isolated final commit

Verification

  • 191 library unit tests
  • cargo fmt --all --check
  • cargo clippy --all-targets -- -D warnings
  • cargo test --doc
  • cargo doc --no-deps
  • clean cargo package build
  • real codex_app_server_can_write_inside_its_cwd test
  • real codex_app_server_accepts_a_live_steer test

The unfiltered local live-test binary also reports the pre-existing machine state that Claude is logged out and Codex account quota is unavailable. The two targeted Codex feature tests pass outside the task sandbox.

@pathscale
pathscale force-pushed the fix/codex-cwd-writable branch from 4788ac7 to 37e0a22 Compare August 1, 2026 22:07
@pathscale
pathscale merged commit ba95855 into master Aug 1, 2026
2 checks passed
@pathscale
pathscale deleted the fix/codex-cwd-writable branch August 1, 2026 23:04
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