docs(skills): consolidate to one embedded source + sync README/CHANGELOG#99
Merged
Conversation
Skills lived in two hand-maintained places — the embedded internal/core/skills/builtin/ (24 files, what //go:embed actually ships) and a duplicate top-level skills/ mirror (13 files) that the README pointed at. The mirror had already drifted: the 11 skills added in #94/#95 existed only under builtin/, and the README's path ("internal/skills/ skills/") did not even exist. Two sources of truth with no guard is the same drift class we just closed for slash commands. Collapse to a single source: remove the top-level skills/ mirror (no build consumer — the embed reads builtin/ directly; the 13 files were byte- identical and remain in git history), and repoint the README at internal/core/skills/builtin/. Update the skill table to all 24 (count, path, and the 11 new operational / runtime-engine / AI-inference rows) and add CHANGELOG entries for the skill expansion and the native-scrollback TUI redesign. Signed-off-by: rlaope <piyrw9754@gmail.com>
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
1 participant
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.
Summary
Skills coexisted in two hand-maintained places that had already drifted:
internal/core/skills/builtin/— 24 files, what//go:embedactually ships.skills/— a 13-file mirror the README pointed at (and the README pathinternal/skills/skills/didn't even exist).The 11 skills from #94/#95 existed only under
builtin/, so the mirror was stale. Two sources of truth with no guard = the same drift class just closed for slash commands.Resolution — single source:
skills/mirror (no build consumer; the embed readsbuiltin/directly; the 13 files were byte-identical and stay in git history).internal/core/skills/builtin/and update the skill table to all 24 (count, path, and 11 new operational / runtime-engine / AI-inference rows).Test plan
go build ./...+go test ./...green (the embed is unaffected — it never read top-levelskills/)skills/(only the README linked it)