Skip to content

Conversation

@panbanda
Copy link
Owner

@panbanda panbanda commented Jan 6, 2026

Summary

  • Add is_remote_repo() to detect owner/repo shorthand patterns vs local paths
  • Integrate clone_remote with CLI path handling so -p facebook/react works
  • Use native git clone for reliable working tree checkout (gix's fetch_only doesn't checkout)
  • Add automatic cleanup of temporary cloned repositories after analysis
  • Add comprehensive tests for remote detection logic

Background

The README documents omen -p facebook/react score but this feature was never connected to the CLI. The clone_remote function existed in src/git/remote.rs but was never called - the -p flag just passed the path directly to FileSet::from_path.

Test plan

  • All 573 existing tests pass
  • Clippy passes with no warnings
  • Manual test: omen -p rust-lang/log --shallow score successfully clones and analyzes

Generated with Claude Code

- Add is_remote_repo() to detect owner/repo shorthand patterns
- Integrate clone_remote with CLI path handling
- Use native git clone for reliable working tree checkout
- Add automatic cleanup of temporary cloned repositories
- Add comprehensive tests for remote detection logic

Fixes the documented but non-functional remote repo feature.
Now `omen -p facebook/react score` works as expected.

Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@panbanda panbanda merged commit 4a48633 into main Jan 7, 2026
6 checks passed
@github-actions github-actions bot mentioned this pull request Jan 7, 2026
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.

2 participants