feature: record-a-skill capture flow in the panel - #678
Draft
artokun wants to merge 2 commits into
Draft
Conversation
There was a problem hiding this comment.
Copilot wasn't able to review any files in this pull request.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Owner
Author
|
Parked: feature requests on hold while the bug backlog is worked. This draft stays as the placeholder; do not pick up until unparked. |
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
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.
Status: placeholder branch — no implementation.
cluster/record-a-skillcarries one emptycommit (
e29ca85d, subject prefixedplaceholder:) plus merges ofmaintaken purely asupkeep. It contributes zero changed files:
git diff origin/main...cluster/record-a-skilllists nothing. This PR tracks the request; it does not deliver it.
Re-measured 2026-08-14 against
mainatafb8db64(v0.14.41).mainis now merged inrather than merely test-merged, and the branch tip's tree is byte-identical to
main's:both are
ca1b58006485ec65ff3f7efbbd37e52f2afe6800. (An earlier revision of this body citedtree
1a468145; that was a correct measurement against an oldermainand is superseded —the number moves every time
maindoes, so treat the equality as the claim, not the hash.)npm run test:uniton that tip: 4409 tests, 4408 pass, 0 fail, 1 todo — which is exactlymain's own result, necessarily, since the trees are identical.Parked under the feature freeze (bug/stability only). Not a merge candidate. Kept open
and in draft so the request is not dropped.
Scope: integrate a "record a skill" capture flow into the panel. Refs #350.
State of main (re-verified 2026-08-14): no capture flow exists. A search of
web/for arecord/capture entry point (
record.a.skill,recordSkill,skill.capture,captureSkill)returns nothing. Skills are surfaced only passively, as SDK slash commands pushed by the
orchestrator into the completion menu —
web/js/comfyui-mcp-panel.jsdescribes them at thetwo sites that handle them as "SDK slash commands (built-ins like
/compact, plus any loadedskills)". Reading and running a skill is all that exists; there is no way to author one from
the panel.