Repository navigation
feat: list Codex sessions across default and Orca homes (#79) - #80
Merged
Merged
Conversation
This was referenced Sep 16, 2026
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
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.
Codex sessions in Orca/custom homes were absent from default listing unless callers already knew the home path. Default
listnow queries bounded default, CODEX_HOME, Orca and configured candidates on the destination and addscodexHome/stateDbto every Codex row. Explicit--codex-homeremains single-home, including MCP destinations.Closes #79. Stacked on #78, which is still open. Merge order: #76 → #77 → #78 → this PR; retarget this PR to main after #78 merges.
discovery.codex.homesand inventory errors, retaining successful Codex/Claude results on partial failures. Explicit/environment/configured missing homes fail; absent optional candidates do not.not_installedand exit 0. Errors still return exit 1. The legacy top-levelcodexHomeremains only for a single candidate without inventory errors; consumers should use row-level paths.Validation: 290 tests pass with MCP dependencies; standalone suite passes with two optional protocol tests skipped. New fixtures cover all candidate sources, duplicate UUIDs, aliases, explicit selection, missing/corrupt/unsupported DBs, invalid timestamps/configuration, enumeration/read/resolve failures, archived filtering, SSH source execution with destination environment, multi-host metadata and MCP isolation. Wheel/sdist build and diff checks pass.
Read-only live checks: local default home returns 19 sessions; SSH macbook returns 164 across default + two Orca homes. Explicit remote home returns only 25 sessions from that home. Adding a nonexistent local configured home preserves 19 valid rows and reports the missing home with exit 1. No messages sent, queue/state DB writes, runtime activation or release performed.