feat(studio): Skills surface + controller skills routes - #974
Closed
jtenniswood wants to merge 1 commit into
Closed
jtenniswood wants to merge 1 commit into
jtenniswood wants to merge 1 commit into
Conversation
Skills browse/detail with provenance (folder skills browsable, SKILL.md inline), two-step creation, zip/folder upload, enable/disable with the .disabled holding area, and delete — all controller-mediated in managed mode, read-only in external mode. skill-upload reuses controller-security's validSkillName, deliberately the ONE name gate shared by browser and controller. local-controller.mjs grows its /skills* route family in the same diff as the surface that calls it (upload/create/body/files/enable-disable/delete into the pinned .mecatl/skills dir); the learned-skills panel reads the learned-skills transport landed with the harness tier. fflate lands with its consumer; the skills e2e spec row joins workspace.spec.ts. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
jtenniswood
added
enhancement
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Stack 9/21 — Studio: Skills surface + controller skills routes
The skills surface and, in the same diff, the controller route family that serves it — so each managed-mode write path is reviewed next to the UI that calls it.
skill-upload+tests,fflate), enable/disable with the.disabledholding area, delete; learned-skills panel over PR 5'slearned-skillsclient.skill-uploadreusescontroller-security.mjs'svalidSkillName— the same validator the controller enforces server-side (deliberately a single shared gate, from PR 3).local-controller.mjsgrows its/skills*routes (upload/create/body/files/enable-disable/delete into the pinned.mecatl/skillsdir).workspace.spec.ts; user-docs gains the skills section; llms regen.Demo at this stack point: browse/create/upload/enable/disable skills in managed mode; read-only list in external mode (the proxy's 409 table has covered these routes since PR 3).
Part of the stacked landing of #618.
🤖 Generated with Claude Code