Repository navigation
fix: detect custom Codex homes independently of skill destination - #20
Merged
Merged
Conversation
4 of 5 tasks
ggordonhall
added a commit
to first-tree-ai/opentag
that referenced
this pull request
Sep 15, 2026
## Summary Custom `CODEX_HOME` previously redirected Context Tree installation beside the custom configuration directory, outside the account skill directory Codex scans. Pass account `HOME` and resolved `CODEX_HOME` independently, remove the basename restriction, and retain unavailable reporting for skipped installations. Bump the CLI and client Context Tree dependencies to the published `0.1.14`, which includes first-tree-ai/context-tree#20. Update the lockfile, portable-runtime dependency assertions, design documentation, and regression tests for both `.codex` and arbitrary custom names. ## Testing - Passed: `pnpm install`, `pnpm check`, `pnpm build`, `pnpm typecheck`. - Passed: all 47 focused Context Tree tests against the installed npm `0.1.14` package, with no local package override. - Passed: all 19 portable-runtime dependency tests. - Previously passed on the implementation commit: `pnpm --filter @opentag/server test:integration` — 334 tests. - `pnpm test`: Context Tree tests pass, but the client suite fails on the unchanged Codex probe fixture (`login-started` missing) and Claude probe timeout cases (1,268 client tests pass; 2 fail). Turbo stops on that failure, so this is not a full green repository run. - Previous agent runtime coverage attempts hit those same two probe cases. Those failures remain unresolved; this PR does not change either provider probe. ## Breaking changes Custom Codex homes no longer return `CODEX_HOME_UNSUPPORTED`. Skills install under account `$HOME/.agents/skills`, independently of the configuration directory. ## Checklist - [x] The change is focused and contains no unrelated work. - [x] Tests and documentation cover the changed behavior. - [ ] `pnpm check`, `pnpm build`, `pnpm typecheck`, and `pnpm test` pass. - [x] No credentials, private data, or generated build output are included. - [x] Canonical English docs and Chinese mirrors are synchronized when applicable.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
With a custom
CODEX_HOME, Codex configuration detection must be independent of skill placement. Detect Codex atCODEX_HOMEwhen set, falling back to$HOME/.codex, while continuing to install skills into$HOME/.agents/skills.Missing, non-directory, and symlinked configuration directories remain skipped. Tests cover custom
.codexand arbitrary directory names without a default configuration directory, plus invalid custom homes. No new CLI flags.This supplies the installer change needed by first-tree-ai/opentag#624. Release this change before updating OpenTag's dependency and merging that PR.
Testing
pnpm checkpnpm typecheckpnpm test— 201 tests passpnpm check:package