Skip to content

skills: build MCP and CLI variants from shared templates#76

Merged
jurassix merged 8 commits into
mainfrom
clint/subtext-skill-templates
May 29, 2026
Merged

skills: build MCP and CLI variants from shared templates#76
jurassix merged 8 commits into
mainfrom
clint/subtext-skill-templates

Conversation

@jurassix
Copy link
Copy Markdown
Collaborator

@jurassix jurassix commented May 28, 2026

Summary

  • Moves skills/ to templates/skills/ as the single source of truth
  • Adds a Go build tool (cli/hack/buildskills) that generates two output trees from those templates: skills/ (MCP plugin) and cli/skills/ (CLI binary)
  • Template tokens: {{tool "live-connect"}} expands to `live-connect` for MCP or `subtext live connect` for CLI; {{if eq .Target "cli"}} / SKILL.cli.md overrides handle deeper prose divergence
  • Committed generated outputs for both targets; CI drift check fails if templates are edited without re-running go generate ./...

Coverage:

  • 8 CLI skills: comments, docs, live, proof, recipe-sightmap-setup, shared, sightmap, tunnel
  • 14 MCP skills (all of the above + review, session, onboard, first-session, setup-plugin, using-subtext)

Key divergence handled via SKILL.cli.md overrides:

  • shared — CLI explains subtext sightmap upload instead of the Python side-band script
  • live / tunnel — removes "subtext-tunnel MCP server" references; adapts flow steps to subtext commands
  • sightmap / recipe-sightmap-setup — replaces the Python upload script with subtext sightmap upload

Stacks on #75 (the Go CLI port PR).

Test plan

  • cd cli && go generate ./... produces no git diff in skills/ or cli/skills/
  • cd cli && go test ./... passes (embed smoke test included)
  • Spot-check cli/skills/comments/SKILL.md: tool refs show subtext comment list not comment-list
  • Spot-check skills/comments/SKILL.md: tool refs show comment-list (byte-identical to pre-move content)
  • cli/skills/shared/ has no collect_and_upload_sightmap.py; skills/shared/ does

Comment thread .github/workflows/skills-drift.yml
@jurassix jurassix force-pushed the clint/subtext-skill-templates branch from 107002d to fa6bead Compare May 29, 2026 12:11
@jurassix jurassix changed the base branch from clint/subtext-cli to main May 29, 2026 12:55
@jurassix jurassix merged commit 1ac19f0 into main May 29, 2026
3 checks passed
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