Skip to content

[codex] migrate ExecutorFileSystem write_file to PathUri - #27427

Closed
anp-oai wants to merge 1 commit into
codex/exec-server-path-uri-read-filefrom
codex/exec-server-path-uri-write-file
Closed

[codex] migrate ExecutorFileSystem write_file to PathUri#27427
anp-oai wants to merge 1 commit into
codex/exec-server-path-uri-read-filefrom
codex/exec-server-path-uri-write-file

Conversation

@anp-oai

@anp-oai anp-oai commented Jun 10, 2026

Copy link
Copy Markdown
Contributor

Intent: move ExecutorFileSystem writes onto PathUri without changing wire payloads.

Implementation: change write_file arguments to PathUri, converting at exec-server adapter boundaries.

Validation: focused stack validation ran affected crate tests and core regression selectors.

Stacked on #27426.

@anp-oai
anp-oai force-pushed the codex/exec-server-path-uri-write-file branch from fa28f65 to e9ccd91 Compare June 10, 2026 18:08
@anp-oai
anp-oai force-pushed the codex/exec-server-path-uri-read-file branch from 69c2758 to d86c868 Compare June 10, 2026 18:12
@anp-oai
anp-oai force-pushed the codex/exec-server-path-uri-write-file branch 2 times, most recently from d0fedda to f590226 Compare June 10, 2026 19:03
@anp-oai
anp-oai force-pushed the codex/exec-server-path-uri-read-file branch 2 times, most recently from 44d0374 to f68fa4e Compare June 10, 2026 20:20
@anp-oai
anp-oai force-pushed the codex/exec-server-path-uri-write-file branch from f590226 to 739e29f Compare June 10, 2026 20:20
@anp-oai
anp-oai force-pushed the codex/exec-server-path-uri-read-file branch from f68fa4e to df0f70f Compare June 10, 2026 22:22
@anp-oai
anp-oai force-pushed the codex/exec-server-path-uri-write-file branch from 739e29f to 3f075fc Compare June 10, 2026 22:22
@anp-oai

anp-oai commented Jun 11, 2026

Copy link
Copy Markdown
Contributor Author

Superseded by #27424, which now contains the complete mechanical ExecutorFileSystem PathUri migration as one squashed commit.

@anp-oai anp-oai closed this Jun 11, 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.

1 participant