Skip to content

feat: implement cli detectors for Claude Code and Codex CLI (issue #19)#45

Merged
thundermiracle merged 2 commits intomainfrom
codex/issue-19-cli-detectors
Feb 26, 2026
Merged

feat: implement cli detectors for Claude Code and Codex CLI (issue #19)#45
thundermiracle merged 2 commits intomainfrom
codex/issue-19-cli-detectors

Conversation

@thundermiracle
Copy link
Copy Markdown
Owner

Summary

  • harden CLI detection reasoning with explicit reason codes:
    • detected
    • config_missing
    • binary_missing
    • binary_and_config_missing
    • config_override_invalid
  • enforce deterministic precedence: explicit override path is evaluated before fallback paths
  • add probe helper to test precedence behavior without runtime env mutation
  • add fixture-style Rust tests for CLI detection state transitions and path precedence
  • add Node contract tests for Claude/Codex CLI candidate ordering and reason-code presence

Commit Structure

  • feat(rust): harden cli detector status reasoning and precedence handling (issue #19)
  • test(ts): add cli detector contract tests for candidate order and reason codes (issue #19)

Validation

  • pnpm run lint
  • pnpm test
  • cargo test --manifest-path src-tauri/Cargo.toml
  • pnpm outdated

Closes #19

@thundermiracle thundermiracle merged commit 7cc0ee3 into main Feb 26, 2026
2 checks passed
@thundermiracle thundermiracle deleted the codex/issue-19-cli-detectors branch February 26, 2026 14:26
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.

P1-DETECT: Implement detectors for Claude Code and Codex CLI

1 participant