Conversation
| <> | ||
| <span className="inline-flex items-center gap-1 text-[11px] text-green-700 bg-green-50 px-2 py-0.5 rounded-full"> | ||
| <Icon icon={ check } size={ 12 } /> | ||
| { __( 'Installed' ) } |
apps/studio/src/modules/agent-skills/components/skills-panel.tsx
Outdated
Show resolved
Hide resolved
apps/studio/src/modules/agent-skills/components/skills-panel.tsx
Outdated
Show resolved
Hide resolved
apps/studio/src/modules/agent-skills/components/skills-panel.tsx
Outdated
Show resolved
Hide resolved
@nightnei Let's hold off with this PR for now. I think we will implement #2741 first as the base for the backend and then rebase this one to the UI based on the discussion here #2741 (comment) I will mark this as draft for now to avoid confusion. |
|
@nightnei this is now available for review, thank you! |
📊 Performance Test ResultsComparing 2833c86 vs trunk app-size
site-editor
site-startup
Results are median values from multiple test runs. Legend: 🟢 Improvement (faster) | 🔴 Regression (slower) | ⚪ No change (<50ms diff) |
|
Hm, again I have this problem when installing doesn't work 🤔 cc @sejas @epeicher
On Monday, I am planning to debug my environment and find the root cause, since it seems that we need to add error handling in some places and anyway it's good to know why it doesn't work for me second time. The code LGTM, but I would ask somebody else to test it, to ensure that it works for other folks. Feel free to not wait for my approval, most likelly it's my local issue. |


Related issues
How AI was used in this PR
I used it to get the basic idea and implementation from https://github.com/Automattic/studio/pull/2511/changes#diff-c69fa338d4e3c20084125a9fa7186cf40116a7460210922cfd4b4021935f4c18
Proposed Changes
This PR adds
Skillstab to theAI Settingsmodal which allows to install and manage diffeent skills.Testing Instructions
npm startAgents Suitefeature in the topbar menu underElectron > Feature FlagsAssistanttabAI SettingsmodalSkillstab such as installing and removing skills and confirm it works as expectedPre-merge Checklist