Problem/Motivation
AI Agents can execute the tools, which are on its own plugins. For add/edit form of AI Agent it would be nice to have a better way to search for the correct tool. Also, showing all of the tools can occupy a lot of space on the form page, therefore having this in a modal (like entity browser for example) looks like a proper way.
Proposed resolution
Create a special form element for tools plugin selection with a modal window where it should be possible to search for plugins, filter them and select.
| Comment | File | Size | Author |
|---|---|---|---|
| #18 | Screenshot 2025-07-01 at 18.00.13.png | 47.42 KB | a.dmitriiev |
| #18 | Screenshot 2025-07-01 at 17.59.53.png | 80.07 KB | a.dmitriiev |
| #18 | Screenshot 2025-07-01 at 17.59.39.png | 137.95 KB | a.dmitriiev |
| #18 | Screenshot 2025-07-01 at 17.59.27.png | 36.89 KB | a.dmitriiev |
Issue fork ai-3519692
Show commands
Start within a Git clone of the project using the version control instructions.
Or, if you do not have SSH keys set up on git.drupalcode.org:
Issue fork ai_agents-3519692
Show commands
Start within a Git clone of the project using the version control instructions.
Or, if you do not have SSH keys set up on git.drupalcode.org:
Comments
Comment #2
anjaliprasannan commentedComment #3
anjaliprasannan commentedChanged the project to aiagent
Comment #4
anjaliprasannan commentedComment #5
anjaliprasannan commentedComment #7
anjaliprasannan commentedWork done:
Remaining work
To continue with the remaining work, I am a bit confused with the implementation of how to take the values in the modal form on apply click to the ai agent form and save them in the ai agent form?
Will fix the pipeline error along with the remaining work.
Comment #8
anjaliprasannan commentedComment #15
a.dmitriiev commentedComment #16
yautja_cetanu commentedYes that may help, the issue is I think we need this Base Tools things figured out. So for me, I think having some Designs of how the Base Tools, Tools API will work is important first and then we want a plugin finder/ browser imo. Base tools may not change the browser that much but I think it might. I also think we want to use AI to help people find the right tool.
Also can we see some designs and screenshots of it here?
Comment #17
marcus_johansson commentedComment #18
a.dmitriiev commentedTools form element with 1 tool selected:
Modal dialog that is opened when clicking on "Select tools":
Filtering and selection of tools inside modal dialog:
Tools form element after tool selection:
Comment #19
a.dmitriiev commentedThe tools library is done in the same way the core media library is done, so this design and look should be familiar to all Drupal users.
Comment #20
yautja_cetanu commentedFrom a look and feel this is just better than what exists now however:
- It feels mostly a repaint of what exists now. One feedback I got from what exists now is that things like "Modification tools" - Sub-agents tools were confusing to people.
- We have the ability to configure the tools but it appears at the bottom of the page. Which is confusing.
- The UI for configuring the tools is really bad (just having to know the machine names of all bundles for example, know that you would have to limit the content seeder by entity type etc).
- Cannot have multiple versions of the same tool.
However, as mentioned the would get solved with Base Tools.
So given the above is just better in every way than what we have now, happy for this to go in. But it will likely need to be rethought from scratch with base tools
Comment #21
marcus_johansson commentedMoving to AI
Comment #22
marcus_johansson commentedThis is awesome work - I've tested it with and without a modeler and it works fine.
I've added some comments, not everything is obvious that it needs changes, just a discussion. One thing that I didn't mark everywhere in the MR, but that needs to be redone:
So the biggest issue I see for the UX:
Other notes:
Comment #23
marcus_johansson commentedBTW - I've rebased the AI Agents branch, but you can also feel free to create an issue specifically for this in AI Agents, so you get credit for both changes.
Comment #26
a.dmitriiev commentedI tried to address all comments. There are phpcs and phpstan issues, but they are not related to the changes from this MR.
Comment #27
marcus_johansson commentedTested, approved and remerged 1.2.x after fixes of phpcs, phpstan.Comment #29
marcus_johansson commentedComment #30
marcus_johansson commentedSeems like a minor fix is needed based on that I added DrupalPractice
Comment #31
a.dmitriiev commentedFixed phpstan problems!
Comment #32
marcus_johansson commentedI will set RTBC and merge it into AI core. The AI Agents part we might have to wait on, since we might have 1.2.0 release there before the 1.2.0 of AI core is released and the dependency there.
Comment #34
marcus_johansson commentedOk, we are on 100% track with the releases, so this is getting merged into AI Agents as well for 1.2.0 release. Thank you everyone!