Skip to content

Add js_repl cwd and homeDir helpers#14385

Merged
fjord-oai merged 1 commit intomainfrom
fjord/codex_cwd
Mar 11, 2026
Merged

Add js_repl cwd and homeDir helpers#14385
fjord-oai merged 1 commit intomainfrom
fjord/codex_cwd

Conversation

@fjord-oai
Copy link
Contributor

Summary

This PR adds two read-only path helpers to js_repl:

  • codex.cwd
  • codex.homeDir

They are exposed alongside the existing codex.tmpDir helper so the REPL can reference basic host path context without reopening direct process access.

Implementation

  • expose codex.cwd and codex.homeDir from the js_repl kernel
  • make codex.homeDir come from the kernel process environment
  • pass session dependency env through js_repl kernel startup so codex.homeDir matches the env a shell-launched process would see
  • keep existing shell HOME population behavior unchanged
  • update js_repl prompt/docs and add runtime/integration coverage for the new helpers

@fjord-oai fjord-oai marked this pull request as ready for review March 11, 2026 20:00
@fjord-oai
Copy link
Contributor Author

@codex review

Copy link
Contributor

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: a622a9ba4e

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@fjord-oai fjord-oai merged commit 5a89660 into main Mar 11, 2026
31 checks passed
@fjord-oai fjord-oai deleted the fjord/codex_cwd branch March 11, 2026 21:44
@github-actions github-actions bot locked and limited conversation to collaborators Mar 11, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants