-
Notifications
You must be signed in to change notification settings - Fork 100
[Planner] Planning agent based on plan.md file #424
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
| println!(" - Create a Dagger sandbox"); | ||
| println!(" - Call orchestrator.spawn_workers(llm, sandbox, prompt, validator)"); | ||
|
|
||
| /* |
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.
drop commented code, set up mocking
| } | ||
| } | ||
|
|
||
| fn is_done(&self, _response: &thread::ToolResponse) -> bool { |
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.
TODO: implement!
| Use uv package manager if you need to add extra libraries. | ||
| Program will be run using uv run main.py command. | ||
| You MUST manage your planning in a plan.md file: |
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.
duplication with planner - need to properly limit access here to plan changes
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.
extract to a seaprarte PR?
| // Create verification validators | ||
| let file_validator = FileExistsValidator::new(vec!["main.py".to_string()]); | ||
| let hello_validator = HelloWorldValidator; | ||
| let health_validator = HealthCheckValidator::new("python --version"); |
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.
replace with app healthcheck validator and make app an API
| use dabgent_sandbox::Sandbox as SandboxTrait; | ||
|
|
||
| // Create verification validators | ||
| let file_validator = FileExistsValidator::new(vec!["main.py".to_string()]); |
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.
add plan.md validator
| self.store | ||
| .push_event(&self.stream_id, &self.aggregate_id, &event, &metadata) | ||
| .await?; | ||
| dagger_sdk::connect(|client| async move { |
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.
use orchestrators instead?
| )) | ||
| } | ||
|
|
||
| async fn create_sandbox(client: &dagger_sdk::DaggerConn) -> color_eyre::Result<DaggerSandbox> { |
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.
extract
| Ok(()) | ||
| } | ||
|
|
||
| fn create_llm() -> color_eyre::Result<rig::providers::anthropic::Client> { |
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.
extract
No description provided.