-
Notifications
You must be signed in to change notification settings - Fork 1
Add ralph-orchestrator as pixi dependency #132
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
Replace previous ralph implementation with ralph-orchestrator from the blooop conda channel. Adds configuration files and documentation. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Reviewer's GuideAdds ralph-orchestrator as a Pixi-managed dependency, wires it into the project as a convenient pixi task, and introduces default configuration, prompt templates, and documentation for using Ralph within the Claude/Pixi workflow. Sequence diagram for running Ralph via PixisequenceDiagram
actor Dev
participant Shell
participant Pixi as Pixi_environment
participant Ralph as Ralph_orchestrator_CLI
participant Files as Filesystem
Dev->>Shell: execute_pixi_run_ralph_plan_"Implement_feature_X"
Shell->>Pixi: invoke_task_ralph
Pixi->>Ralph: run_ralph_plan_with_argument_Implement_feature_X
Ralph->>Files: read_ralph.yml
Ralph->>Files: read_PROMPT.md_if_needed
Ralph->>Files: create_or_update_spec_in_specs_directory
Ralph-->>Pixi: exit_status_and_output
Pixi-->>Shell: command_result
Shell-->>Dev: show_plan_output_and_spec_location
File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey - I've found 1 issue, and left some high level feedback:
- In
pyproject.toml, consider pinningralph-orchestratorto a specific version or constrained range instead of*to avoid unexpected breaking changes from future releases. - The README describes specs living under
specs/while the commented-outcore.specs_direxample inralph.ymluses.ralph/specs/; it would be good to align these paths or clarify the intended directory structure.
Prompt for AI Agents
Please address the comments from this code review:
## Overall Comments
- In `pyproject.toml`, consider pinning `ralph-orchestrator` to a specific version or constrained range instead of `*` to avoid unexpected breaking changes from future releases.
- The README describes specs living under `specs/` while the commented-out `core.specs_dir` example in `ralph.yml` uses `.ralph/specs/`; it would be good to align these paths or clarify the intended directory structure.
## Individual Comments
### Comment 1
<location> `pyproject.toml:23` </location>
<code_context>
python = ">=3.10"
shellcheck = ">=0.10.0,<0.11"
devpod = ">=0.8.0,<0.9"
+ralph-orchestrator = "*"
[tool.pixi.feature.py310.dependencies]
</code_context>
<issue_to_address>
**suggestion (bug_risk):** Consider constraining the ralph-orchestrator version instead of using a wildcard.
Using `*` means any future major version could be installed, potentially breaking reproducible environments (CI, dev machines). Prefer a validated compatible range (e.g. `>=x.y,<x+1.0`), especially for tooling that affects workflows.
Suggested implementation:
```
ralph-orchestrator = ">=0.1.0,<1.0.0"
```
You should adjust the specific version range (`>=0.1.0,<1.0.0`) to match the currently validated/known-compatible versions of `ralph-orchestrator` in your project (for example, `>=0.3.2,<0.4.0` if you only support a specific minor line).
</issue_to_address>Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
| python = ">=3.10" | ||
| shellcheck = ">=0.10.0,<0.11" | ||
| devpod = ">=0.8.0,<0.9" | ||
| ralph-orchestrator = "*" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
suggestion (bug_risk): Consider constraining the ralph-orchestrator version instead of using a wildcard.
Using * means any future major version could be installed, potentially breaking reproducible environments (CI, dev machines). Prefer a validated compatible range (e.g. >=x.y,<x+1.0), especially for tooling that affects workflows.
Suggested implementation:
ralph-orchestrator = ">=0.1.0,<1.0.0"
You should adjust the specific version range (>=0.1.0,<1.0.0) to match the currently validated/known-compatible versions of ralph-orchestrator in your project (for example, >=0.3.2,<0.4.0 if you only support a specific minor line).
Summary
ralphpixi task for convenience (pixi run ralph)ralph.ymlconfiguration andPROMPT.mdtemplate.claude/README.mdTest plan
pixi installsucceedspixi run ralph --versionreturns version 2.3.1pixi run ralph initandpixi run ralph runcommands🤖 Generated with Claude Code
Summary by Sourcery
Add Ralph Orchestrator as a Pixi-managed tool and document how to use it in the project.
New Features:
pixi run ralphtask for invoking the Ralph CLI.ralph.ymlconfiguration andPROMPT.mdtemplate for Ralph workflows.Documentation:
.claude/README.mdwith installation, usage, configuration, and reference links for Ralph Orchestrator.