Skip to content

refactor(skill): replace always-loaded stubs with registry pickers - #15

Merged
cursor[bot] merged 2 commits into
mainfrom
cursor/pr-thin-skill-85a6
Oct 4, 2026
Merged

cursor[bot] merged 2 commits into
mainfrom
cursor/pr-thin-skill-85a6

Conversation

@twidtwid

@twidtwid twidtwid commented Oct 4, 2026

Copy link
Copy Markdown
Owner

Why

Eight of twelve shape references were 13-line stubs. Agents still loaded them. The gold HTML is the contract. SKILL.md should route, not repeat.

What changed

  • Shape and primitive stubs in SKILL.md are picker tables generated from contracts/.
  • Eight stub files under references/shapes/ are deleted.
  • Thick references stay for editorial, podcast, execution-log, and deck-review.
  • perf/baseline.json now records 7394 skill tokens.

Scope

This PR only thins the skill and the stub files. It does not add --against or the clipboard helper.

Blast Radius

Always-loaded skill tokens drop 12257 → 7394 (4863 below the shape-lint parent). Generated HTML did not change.

Verification

  • make check is green.
  • The picker tables name all twelve shapes and all ten primitives.
  • Step 7 still runs lint-artifact.mjs and still forbids hand-retyped scripts.
Open in Web Open in Cursor 

@cursor
cursor Bot force-pushed the cursor/pr-shape-lint-85a6 branch from c826ca6 to 6a1b47b Compare October 4, 2026 21:09
@cursor
cursor Bot force-pushed the cursor/pr-thin-skill-85a6 branch from 518dcf4 to accf143 Compare October 4, 2026 21:09
@cursor
cursor Bot force-pushed the cursor/pr-shape-lint-85a6 branch 2 times, most recently from d752814 to a7fc509 Compare October 4, 2026 22:20
cursoragent and others added 2 commits October 4, 2026 22:22
SKILL.md is now a router. Eight 13-line shape stubs are gone.
The gold HTML is the contract. Thick references stay for
editorial, podcast, execution-log, and deck-review.

Co-authored-by: Todd Dailey <twidtwid@users.noreply.github.com>
Thin shapes no longer list missing references/shapes files. The checker
fails if any listed path is gone, and SKILL.md pickWhen must match the
JSON. Harness complete count follows the thick-only reference rule.

Co-authored-by: Todd Dailey <twidtwid@users.noreply.github.com>
@cursor
cursor Bot changed the base branch from cursor/pr-shape-lint-85a6 to main October 4, 2026 22:22
@cursor
cursor Bot force-pushed the cursor/pr-thin-skill-85a6 branch from da41d6d to 98a7c6d Compare October 4, 2026 22:22
@cursor
cursor Bot merged commit cbd231b into main Oct 4, 2026
2 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.

2 participants