Skip to content

app-server: Return codex home in initialize response#15689

Merged
euroelessar merged 2 commits intomainfrom
ruslan/codex-home-init
Mar 24, 2026
Merged

app-server: Return codex home in initialize response#15689
euroelessar merged 2 commits intomainfrom
ruslan/codex-home-init

Conversation

@euroelessar
Copy link
Collaborator

This allows clients to get enough information to interact with the codex skills/configuration/etc.

pub struct InitializeResponse {
pub user_agent: String,
/// Absolute path to the server's Codex home directory.
pub codex_home: PathBuf,
Copy link
Collaborator

Choose a reason for hiding this comment

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

let's use AbsolutePathBuf?

#[serde(rename_all = "camelCase")]
pub struct InitializeResponse {
pub user_agent: String,
/// Absolute path to the server's Codex home directory.
Copy link
Collaborator

Choose a reason for hiding this comment

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

nit: Absolute path to the server's $CODEX_HOME directory.?

@euroelessar euroelessar merged commit 24c4eca into main Mar 24, 2026
36 checks passed
@euroelessar euroelessar deleted the ruslan/codex-home-init branch March 24, 2026 23:13
@github-actions github-actions bot locked and limited conversation to collaborators Mar 24, 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.

2 participants