Skip to content

exec-server: add environment path refs#25121

Open
starr-openai wants to merge 3 commits into
mainfrom
starr/skills-path-ref-1-env-path-ref
Open

exec-server: add environment path refs#25121
starr-openai wants to merge 3 commits into
mainfrom
starr/skills-path-ref-1-env-path-ref

Conversation

@starr-openai
Copy link
Copy Markdown
Contributor

@starr-openai starr-openai commented May 29, 2026

Summary

  • add public codex_exec_server::EnvironmentPathRef
  • bind an absolute path to its owning executor filesystem
  • keep path operations in the next review slice

Stack

Validation

  • cd /Users/starr/code/codex-worktrees/pr-25098-restack4/codex-rs && just fmt
  • GitHub CI pending on rewritten head

@starr-openai starr-openai force-pushed the starr/skills-path-ref-1-env-path-ref branch from 2fdb7c5 to d3938c8 Compare May 29, 2026 20:41
@starr-openai starr-openai force-pushed the starr/skills-path-ref-1-env-path-ref branch from f1f663f to b444724 Compare May 29, 2026 22:29
@starr-openai starr-openai force-pushed the starr/skills-path-ref-1-env-path-ref branch from b444724 to 55ec07d Compare May 29, 2026 22:50
@starr-openai starr-openai marked this pull request as ready for review May 29, 2026 23:40
Copy link
Copy Markdown
Collaborator

@jif-oai jif-oai left a comment

Choose a reason for hiding this comment

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

lgtm

self.file_system.read_directory(&self.path, sandbox).await
}

pub fn with_path(&self, path: AbsolutePathBuf) -> Self {
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Ultra NIT, can we add some short doc-string on the pub functions?

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