Skip to content

DEV-5902: Improve ctx discovery and search - #2

Merged
ytspar merged 1 commit into
mainfrom
codex/DEV-5902-ctx-improvements
Jul 14, 2026
Merged

ytspar merged 1 commit into
mainfrom
codex/DEV-5902-ctx-improvements

Conversation

@ytspar

@ytspar ytspar commented Jul 14, 2026

Copy link
Copy Markdown

Simple Description

Improve ctx local-history discovery, search filtering, progress output, and Codex journal classification.

What Changed

  • Added default discovery for archived Codex sessions and Claude profile project roots.
  • Added --role and --exact-phrase / --exact search controls.
  • Made compact human search output focus on useful provenance instead of internal rank/signature metadata.
  • Suppressed duplicate plain progress lines, including repeated final 100% output.
  • Classified metadata-only Codex workflow journals as skipped instead of failed.
  • Improved Codex reviewer-session classification.
  • Documented the local installer PATH smoke test.

Why This Change

The ctx audit found coverage gaps and noisy output that made local history search harder to trust and operate. These changes make default indexing cover the expected fork/user locations and make search output more precise for human and automated review workflows.

Alternatives Considered

  • Leaving archived sessions and Claude profiles as explicit-only imports: rejected because these are expected default local history locations.
  • Keeping rank/signature-heavy compact output: rejected because JSON and verbose modes already preserve exact identifiers.
  • Treating empty journals as failed imports: rejected because metadata-only records are not actionable import failures.

AI observations

Implemented and verified in a dedicated worktree on codex/ctx-improvements. The branch was rebased onto current origin/main; one upstream test added after the original work required the same macOS temp-root canonicalization used for the other provider-root tests.

Tests

  • cargo check -p ctx -p ctx-history-search -p ctx-history-capture
  • cargo test -p ctx-history-search
  • cargo test -p ctx-history-capture codex_session_jsonl_skips_metadata_only_without_real_messages
  • cargo test -p ctx-history-capture codex_session_jsonl_fast_skips_metadata_only_without_real_messages
  • cargo test -p ctx-history-capture default_discovery_includes_archived_codex_and_claude_profile_roots
  • cargo test -p ctx --test search_show_locate_sql

Closes DEV-5902

@ytspar
ytspar merged commit fb588c0 into main Jul 14, 2026
1 check failed
@ytspar
ytspar deleted the codex/DEV-5902-ctx-improvements branch July 14, 2026 12:52
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.

1 participant