Skip to content

fix: make CODEX_HOME resolution consistent#213

Merged
Dimillian merged 2 commits intoDimillian:mainfrom
hosmelq:fix/codex-home-resolution
Jan 23, 2026
Merged

fix: make CODEX_HOME resolution consistent#213
Dimillian merged 2 commits intoDimillian:mainfrom
hosmelq:fix/codex-home-resolution

Conversation

@hosmelq
Copy link
Copy Markdown
Contributor

@hosmelq hosmelq commented Jan 23, 2026

Summary

  • Centralize Codex home resolution so config, prompts, and session data all use the same CODEX_HOME/default path.
  • Ensure workspace codex home falls back to the global default when no .codexmonitor override exists.
  • Remove divergent CODEX_HOME resolvers that caused the UI and app-server to read from different locations.

Why

Locally I set CODEX_HOME to ~/.config/codex, but the app wasn’t loading it because global paths (prompts/config/session data) and workspace spawns didn’t share the same Codex home. That left the app-server reading a different location than the UI, resulting in empty project sessions. This change makes CODEX_HOME authoritative everywhere and applies it consistently so sessions and config load as expected.

Testing

  • npm run test
  • npm run lint
  • npm run typecheck

@Dimillian Dimillian changed the title Make CODEX_HOME resolution consistent fix: make CODEX_HOME resolution consistent Jan 23, 2026
@Dimillian
Copy link
Copy Markdown
Owner

@codex review

@chatgpt-codex-connector
Copy link
Copy Markdown

Codex Review: Didn't find any major issues. You're on a roll.

ℹ️ 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".

@Dimillian Dimillian merged commit 647718a into Dimillian:main Jan 23, 2026
5 checks passed
@hosmelq hosmelq deleted the fix/codex-home-resolution branch January 23, 2026 19:01
gersmann pushed a commit to gersmann/codex-monitor-web that referenced this pull request Mar 10, 2026
zip700 pushed a commit to zip700/CodexMonitor that referenced this pull request Apr 3, 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.

2 participants