AutoGPT
AutoGPT copied to clipboard
feat(platform): Support input nodes alongside trigger nodes
Changes ποΈ
- Wrap trigger config in
AgentPresetin"_trigger_params_{trigger_node_id}"- Update trigger execution logic to get trigger config and also pass on regular graph input
- Add
constant_inputstoTriggeredPresetSetupRequest
Checklist π
For code changes:
- [ ] I have clearly listed my changes in the PR description
- [ ] I have made a test plan
- [ ] I have tested my changes according to the test plan:
- [ ] ...
Example test plan
- [ ] Create from scratch and execute an agent with at least 3 blocks
- [ ] Import an agent from file upload, and confirm it executes correctly
- [ ] Upload agent to marketplace
- [ ] Import an agent from marketplace and confirm it executes correctly
- [ ] Edit an agent from monitor, and confirm it executes correctly
For configuration changes:
- [ ]
.env.defaultis updated or already compatible with my changes - [ ]
docker-compose.ymlis updated or already compatible with my changes - [ ] I have included a list of my configuration changes in the PR description (under Changes)
Examples of configuration changes
- Changing ports
- Adding new services that need to communicate with each other
- Secrets or environment variable changes
- New or infrastructure changes such as databases
Deploy Preview for auto-gpt-docs-dev canceled.
| Name | Link |
|---|---|
| Latest commit | 09369d2c3cd3e7da5bef1094926e761fc5422d16 |
| Latest deploy log | https://app.netlify.com/projects/auto-gpt-docs-dev/deploys/68f719bf677a06000959d89d |
[!IMPORTANT]
Review skipped
Auto reviews are disabled on this repository.
Please check the settings in the CodeRabbit UI or the
.coderabbit.yamlfile in this repository. To trigger a single review, invoke the@coderabbitai reviewcommand.You can disable this status message by setting the
reviews.review_statustofalsein the CodeRabbit configuration file.
β¨ Finishing touches
π§ͺ Generate unit tests (beta)
- [ ] Create PR with unit tests
- [ ] Post copyable unit tests in a comment
- [ ] Commit unit tests in branch
pwuts/open-2547-support-input-blocks-alongside-trigger-blocks
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 @coderabbitai help to get the list of available commands and usage tips.
Here's the code health analysis summary for commits 90af8f8..09369d2. View details on DeepSourceΒ β.
Analysis Summary
| Analyzer | Status | Summary | Link |
|---|---|---|---|
| β Β Success | π― 2 occurences resolved | View CheckΒ β | |
| β Β Success | β 2 occurences introduced | View CheckΒ β |
π‘ If youβre a repository administrator, you can configure the quality gates from the settings.
Deploy Preview for auto-gpt-docs ready!
| Name | Link |
|---|---|
| Latest commit | 09369d2c3cd3e7da5bef1094926e761fc5422d16 |
| Latest deploy log | https://app.netlify.com/projects/auto-gpt-docs/deploys/68f719bff1e32f00082d11ce |
| Deploy Preview | https://deploy-preview-11220--auto-gpt-docs.netlify.app |
| Preview on mobile | Toggle QR Code...Use your smartphone camera to open QR code link. |
To edit notification comments on pull requests, go to your Netlify project configuration.
Thank you for submitting this PR to support input nodes alongside trigger nodes!
Your changes look well-structured, including:
- Updates to the trigger execution logic
- Separation of trigger inputs from regular graph inputs
- Addition of
constant_inputsto theTriggeredPresetSetupRequestmodel - A migration script for existing presets
- Removal of UI restrictions that prevented using input blocks with webhook-triggered blocks
However, before this PR can be merged, please:
- Complete the checklist in the PR description
- Check the box confirming you've listed your changes
- Create and document a test plan
- Check the box confirming you've tested according to the plan
Once you've completed the checklist, this PR should be ready for another review. The code changes themselves look good and align well with the stated purpose of supporting input blocks alongside trigger nodes.
This pull request has conflicts with the base branch, please resolve those so we can evaluate the pull request.