Skip to content

Add per-site skills/instructions routes and connector plumbing - #4646

Draft
shaunandrews wants to merge 1 commit into
move-agent-instructions-into-common-libfrom
add-per-site-skills-connector-plumbing
Draft

Add per-site skills/instructions routes and connector plumbing#4646
shaunandrews wants to merge 1 commit into
move-agent-instructions-into-common-libfrom
add-per-site-skills-connector-plumbing

Conversation

@shaunandrews

Copy link
Copy Markdown
Contributor

Related issues

How AI was used in this PR

  • Fully AI-authored (Claude Code): research, implementation, and manual verification (curl against the real local server + typecheck/test suite).

Proposed Changes

Testing Instructions

  • npm run typecheck and npm test pass across all workspaces on this branch alone.
  • Verified the new routes directly: built + ran studio ui, hit GET/POST/DELETE /api/sites/:id/skills and /api/sites/:id/agent-instructions against a real local site, confirmed install/remove round-trips against the site's actual .agents/skills/ and root instruction files.

Pre-merge Checklist

  • Have you checked for TypeScript, React or other console errors?

Wires the per-site install/remove/status operations through to both
front ends, with no UI yet to surface them:

- apps/local: new /sites/:id/agent-instructions and /sites/:id/skills
  routes (GET/POST/DELETE), delegating to packages/common.
- apps/ui: new Connector methods (getSiteSkillsStatus,
  installSiteSkill, getSiteAgentInstructionsStatus, etc.) implemented
  for the ipc and local connectors; hosted throws UnsupportedError
  (no site filesystem). New siteAgentSkills capability flag and
  matching React Query hooks (use-site-ai.ts).

The Electron IPC side of this already existed and shipped
(getAgentInstructionsStatus, getWordPressSkillsStatus, etc. in
apps/studio/src/ipc-handlers.ts) — apps/ui simply never called it.

Second of a stack bringing per-site AI skills and instructions to the
agentic UI; the next PR adds the actual "AI" tab that uses this.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant