feat(cli): Auto-detect explicit remote URLs in positional arguments#1145
feat(cli): Auto-detect explicit remote URLs in positional arguments#1145
Conversation
Summary of ChangesHello @yamadashy, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! This pull request streamlines the user experience for the Highlights
🧠 New Feature in Public Preview: You can now enable Memory to help Gemini Code Assist learn from your team's feedback. This makes future code reviews more consistent and personalized to your project's style. Click here to enable Memory in your admin console. Changelog
Activity
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
|
Important Review skippedAuto incremental reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the Use the checkbox below for a quick retry:
📝 WalkthroughWalkthroughThis PR implements auto-detection of explicit remote URLs passed as positional CLI arguments, routing them to Changes
Sequence DiagramsequenceDiagram
participant User
participant CLI as cliRun.ts
participant Parser as gitRemoteParse.ts
participant RemoteAction as runRemoteAction()
participant DefaultAction as runDefaultAction()
User->>CLI: Pass URL as positional argument
CLI->>CLI: Check if exactly one arg
CLI->>Parser: isExplicitRemoteUrl(arg)
Parser-->>CLI: true (for https:// or git@)
CLI->>RemoteAction: Route to remote handler
RemoteAction-->>User: Clone & process remote repo
User->>CLI: Pass shorthand (user/repo)
CLI->>Parser: isExplicitRemoteUrl(arg)
Parser-->>CLI: false
CLI->>DefaultAction: Fall back to default
DefaultAction-->>User: Process as local path
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~22 minutes Possibly related PRs
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 inconclusive)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #1145 +/- ##
==========================================
+ Coverage 87.18% 87.19% +0.01%
==========================================
Files 116 116
Lines 4385 4390 +5
Branches 1021 1022 +1
==========================================
+ Hits 3823 3828 +5
Misses 562 562 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Code Review: Auto-detect explicit remote URLs in positional argumentsOverall Assessment: ✅ Approve This is a clean, well-implemented feature that improves CLI ergonomics by allowing users to run Strengths
Potential ConsiderationsDetailsEdge Cases to Consider
DocumentationThe PR description mentions updating the help text via Premortem AnalysisPotential Failure Scenarios
Minor Suggestions (Not Blocking)
LGTM! This is a straightforward UX improvement with solid implementation. |
![devin-ai-integration[bot]](https://avatars.githubusercontent.com/in/811515?s=60&v=4){kind=small}
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
Allow users to run `repomix https://github.com/user/repo` or `repomix git@github.com:user/repo.git` without the `--remote` flag. Only explicit URL formats (https:// and git@) are auto-detected. Shorthand format (owner/repo) is not auto-detected to avoid ambiguity with local directory paths. Closes #1120
yamadashy
force-pushed
the
feat/auto-detect-remote-url
branch
from
07b518a to
540e8dd
Compare
Deploying repomix with
|
| Latest commit: |
1d5297c
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://fbb87ed6.repomix.pages.dev |
| Branch Preview URL: | https://feat-auto-detect-remote-url.repomix.pages.dev |
Code Review (Follow-up after rebase)Overall Assessment: ✅ Approve The code is unchanged from the previous review. The rebase to current One Additional ObservationLooking more closely at the interaction between
If a user passes something like Gemini's Suggestion on
|
…detection The existing --remote flag already supports ssh:// and git:// protocols via git-url-parse, so auto-detection should cover them as well.
1 participant
Allow users to run
repomix https://github.com/user/repoorrepomix git@github.com:user/repo.gitwithout the--remoteflag.Only explicit URL formats (
https://andgit@) are auto-detected. Shorthand format (owner/repo) is not auto-detected to avoid ambiguity with local directory paths.Closes #1120
Checklist
npm run testnpm run lint