fix(platform): Fix unexpected connection clash on two dynamic pins link with the same keys
Background
When you connect a dynamic input pin with the same key, both pins will always be treated identically.
The example above: output is connected to #key1, but the #key1 on the other non-connected block also loses its input field indicating it's also connected
Changes ๐๏ธ
Fix connection checking by adding nodeId into the equation.
Testing ๐
[!NOTE] Only for the new autogpt platform, currently in autogpt_platform/
- 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
PR Reviewer Guide ๐
Here are some key observations to aid the review process:
| โฑ๏ธย Estimated effort to review: 2 ๐ต๐ตโชโชโช |
| ๐งชย No relevant tests |
| ๐ย No security concerns identified |
| โกย Recommended focus areas for review Unnecessary Console Log Code Duplication |
Deploy Preview for auto-gpt-docs canceled.
| Name | Link |
|---|---|
| Latest commit | 4ee44863a85447320c25f58b74ee411e659ba206 |
| Latest deploy log | https://app.netlify.com/sites/auto-gpt-docs/deploys/67069e16759ec100080aca92 |