Skip to content

app-server: use native path strings for command actions#27921

Closed
anp-oai wants to merge 1 commit into
codex/cross-platform-path-renderingfrom
codex/command-action-native-paths
Closed

app-server: use native path strings for command actions#27921
anp-oai wants to merge 1 commit into
codex/cross-platform-path-renderingfrom
codex/command-action-native-paths

Conversation

@anp-oai

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

Copy link
Copy Markdown
Collaborator

Why

Prepare command-action read paths to cross the app-server boundary without assuming the app-server host path syntax. This is stacked on #27819.

What

  • Represent CommandAction::Read.path as NativePathString and resolve it to PathUri with an explicit source convention.
  • Add host-independent POSIX, Windows drive, and UNC parsing, including dot-segment normalization.
  • Update app-server builders, transitional native-host consumers, generated schemas, and API documentation.

Validation

  • Added cross-platform native-path round-trip and command-action wire tests.
  • codex-utils-path-uri, codex-app-server-protocol, codex-app-server, and codex-analytics suites pass.
  • Bazel lockfile validation passes.
  • The TUI suite ran 2,825 tests: 2,813 passed and 12 unrelated guardian/title/status-preview baseline tests failed reproducibly.

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