feat(cli): add auto-plan-mode detection for fresh repos #33
+219
−7
Conversation
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
When running `ralphex` without arguments on master/main branch and no plans exist in docs/plans/, automatically prompt for plan description and switch to plan creation mode instead of erroring. - add errNoPlansFound sentinel error for detecting empty plans dir - add isMainBranch() helper to check for main/master branches - add promptPlanDescription() with io.Reader for testability - add tryAutoPlanMode() to orchestrate auto-plan-mode detection - treat missing plans directory same as empty directory - update documentation in README.md and llms.txt
Deploying ralphex with
|
| Latest commit: |
db71db7
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://9a1b8aa0.ralphex.pages.dev |
| Branch Preview URL: | https://auto-plan-mode-detection.ralphex.pages.dev |
There was a problem hiding this comment.
Pull request overview
This PR adds auto-plan-mode detection that automatically prompts users to create a plan when running ralphex without arguments on main/master branches with no existing plans, instead of erroring out. This improves the user experience for fresh repositories.
Changes:
- Introduces sentinel error
errNoPlansFoundfor detecting empty/missing plans directory - Adds helper functions
isMainBranch(),promptPlanDescription(), andtryAutoPlanMode()to implement auto-plan-mode logic - Updates
selectPlanWithFzf()to return the sentinel error in both missing directory and empty directory cases - Updates documentation in README.md and llms.txt to reflect the new behavior
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| cmd/ralphex/main.go | Core implementation of auto-plan-mode detection with sentinel error, branch checking, user prompting, and orchestration logic |
| cmd/ralphex/main_test.go | Comprehensive test coverage for new functions and integration tests for auto-plan-mode behavior on different branches and modes |
| README.md | Documentation update to mention auto-detection feature in plan creation methods |
| llms.txt | Updated comment to clarify that running without arguments can create plans interactively if none exist |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
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.
Summary
When running
ralphexwithout arguments on master/main branch and no plans exist indocs/plans/, automatically prompt for plan description and switch to plan creation mode instead of erroring with "no plans found".Changes
errNoPlansFoundsentinel error for detecting empty/missing plans directoryisMainBranch()helper to check for main/master branchespromptPlanDescription(io.Reader, colors)with injectable reader for testabilitytryAutoPlanMode()to orchestrate auto-plan-mode detection logicBehavior
--reviewand--codex-onlymodes → skip auto-plan-mode (plan is optional)