Account created on 13 August 2021, about 4 years ago
  • Associate Engineer at Acquia 
#

Merge Requests

More

Recent comments

🇮🇳India kunal.sachdev

kunal.sachdev made their first commit to this issue’s fork.

🇮🇳India kunal.sachdev

Couple of small nits - overall, looks good

🇮🇳India kunal.sachdev

It’s working as expected now. What I did to test this:

  1. Created a new content type named 'tour' with following fields:
    • Tour Description - Text (formatted, long)
    • Tour Image - Media field
    • Tour Id - Number (integer)
    • Tour Tags - Reference type field referring to taxonomy term
    • Author - Reference type field referring to user
  2. Created two pieces of content of the tour type.
  3. Then created a code component using the AI with the prompt:- 'Can you create component which lists content from tour content type? I'd like to display these in a card container using tour cards'
🇮🇳India kunal.sachdev

Tested this, and AI correctly attempts data fetching only as needed.

🇮🇳India kunal.sachdev

I agree with the approach of removing the default information tool from the orchestrator, since not all agents require it and those that do already have it included.
Additionally, I tested everything and can confirm that all features are working as intended.

🇮🇳India kunal.sachdev

I've created a follow-up issue to address handling large context #3547595: Handle exception caused by context window overflow in Canvas AI

🇮🇳India kunal.sachdev

While testing, I found that metadata is still being generated for nodes, which is wrong.

🇮🇳India kunal.sachdev

I tried to fix the problem mentioned in #14 📌 Enable AI agents to generate components that fetch menus Active and the response message is now shorter. There are still occasional additional messages, but the situation is better than before. Please see the attached screenshot.

🇮🇳India kunal.sachdev

kunal.sachdev changed the visibility of the branch 3542596-enable-ai-agents to hidden.

🇮🇳India kunal.sachdev

Working on making it work for custom content and custom fields too.

🇮🇳India kunal.sachdev

I tested the prompt "Create a component to render the admin menu as a mega menu" under the following scenarios:

  1. jsonapi_menu_items module installed, linkset disabled: Uses jsonapi_menu_items code to fetch the menu.
  2. jsonapi_menu_items module installed, linkset enabled: Still uses jsonapi_menu_items code to fetch the menu.
  3. jsonapi_menu_items module uninstalled, linkset enabled: Uses linkset code to fetch the menu.
  4. jsonapi_menu_items module uninstalled, linkset disabled, user has 'administer site configuration' permission: Displays a message prompting to enable linkset configuration.
  5. jsonapi_menu_items module uninstalled, linkset disabled, user does not have 'administer site configuration' permission: Displays a message prompting to install the jsonapi_menu_items module or enable the linkset endpoint.

I also tested with a custom menu name, and it worked as expected.

🇮🇳India kunal.sachdev

kunal.sachdev changed the visibility of the branch 3542596-linkset-for-menu-fetching to hidden.

🇮🇳India kunal.sachdev

It's working as expected. I have tested it with the following prompts:

  • Create a component to show title of the latest page
  • Create a component to show title of all the articles
  • Create a component to show title and body of all the articles and pages
🇮🇳India kunal.sachdev

I’ve uploaded a video to illustrate how it’s currently working.

🇮🇳India kunal.sachdev

This provides a good foundation!! We can include additional tests for page builder tasks and other edge cases in a follow-up.

🇮🇳India kunal.sachdev

I worked on the feature allowing the AI to generate an execution plan, which is then displayed on the screen. The next step is to figure out how to check off each item in the execution plan as the corresponding tool completes its task. The main challenge here is that tool results are only provided once the entire agent called from orchestrator has finished.
For eg - In case of page builder task, AI creates a plan something like:

  • Adding components to the page:
  • component 1
  • component 2
  • component 3
  • Updating title of the page
  • Updating metadata of the page

However, all tool results become available only when the page builder agent completes, so we miss out on intermediary progress updates for each item as they're finished.

🇮🇳India kunal.sachdev

I tested by creating 4–5 components, including one from an image, and everything worked as expected each time ✅

🇮🇳India kunal.sachdev

Adding the video to show how it's working currently

Currently, only messages from executing agents are displayed. I think we should also find a way to display messages from the tools that are executed.

🇮🇳India kunal.sachdev

The issue mentioned in #39 📌 Generate/update title and metadata for pages created from components Active was occurring intermittently because $agent->solve() sometimes returned an empty string, and we were using this value as the response message. To address this, I reverted the change that used $agent->solve() for the response message.

🇮🇳India kunal.sachdev

I tested it locally and it's working as expected now!!

🇮🇳India kunal.sachdev

While testing, I discovered another issue: components added through the page builder agent are not updating correctly. Specifically, the text of the component is not being changed in the backend and remains set to its default value.

🇮🇳India kunal.sachdev

I tested the case mentioned in #26 📌 Generate/update title and metadata for pages created from components Active and it works as expected but after running 6-7 prompts, the AI started to behave unexpectedly.
I also tried similar prompts on version 1.x and observed the same issue there: the AI unnecessarily considers history/comments even when they’re not relevant. I have created a new issue 🐛 AI should use only necessary history/comments for each task Active to address this and I think we should prioritise fixing this issue first.

🇮🇳India kunal.sachdev

The case mentioned in #23 📌 Generate/update title and metadata for pages created from components Active was failing because the pageData was not getting updated with fresh values every time. I have fixed it now.

🇮🇳India kunal.sachdev

Tested it locally, I think one thing which is not working as expected is that the final message about the template is created successfully is shown before and then the message about creating the footer section is shown. Attached the screen recording showing how it's working.

🇮🇳India kunal.sachdev

The second case mentioned in #17 📌 Generate/update title and metadata for pages created from components Active wasn't working because the getEntityField tool was fetching only published data. Since the page hadn’t been published, it retrieved the title as "Untitled page," which led to a new title being created unnecessarily. To resolve this, we removed the getEntityField tool and now pass the XB page data as context directly to all relevant tools

I’ve manually tested all scenarios locally, and everything is now working as expected.

🇮🇳India kunal.sachdev

Although sometimes the title is not getting updated on the page. I checked this same issue is also there in 0.x branch so created a issue for that 🐛 Title is not update on the page when created through AI Active

🇮🇳India kunal.sachdev

Some scenarios are not working as expected so I am trying to fix it.

🇮🇳India kunal.sachdev

kunal.sachdev changed the visibility of the branch 3533881-generateupdate-title-and to hidden.

Production build 0.71.5 2024