Skill Feedback: generate-skill (good)

[KyleAMathews]

Skill Feedback: generate-skill

Package: @tanstack/intent
Skill version: 2.0
Rating: good

Task

Generate 9 SKILL.md files for ElectricSQL across two packages.

What Worked

• Clear body structure templates made it straightforward to write consistent skills
• The checklist/audit alternative template for the security skill produced a more useful output than the standard pattern
• The 500-line budget was a good constraint that forced reference file extraction
• Reference file extraction worked well for WHERE clause syntax and type parsers

What Failed

• domain_map.yaml had a duplicate YAML key (mechanism appeared twice in one failure mode entry) that failed prettier validation — generated YAML should be validated before writing

Missing

• No guidance on how to handle skills that reference APIs from other packages (e.g., electric-new-feature references TanStack DB APIs)
• Unclear whether to include full code examples from companion libraries or just reference them
• Post-generation steps (validate, setup, update package.json, add devDependency, copy shim, create labels) are presented as optional but are REQUIRED for skills to actually ship with the npm package — should be integrated into the scaffold flow
• The --shim flag mentioned in scaffold output doesn't exist as a separate command — setup does everything at once. Scaffold instructions should match the actual CLI API
• setup command created root-level bin/intent.mjs in monorepo root instead of per-package — required manual cleanup and mkdir -p for package bin/ directories

Self-Corrections

• Had to mkdir -p bin/ directories before copying intent.mjs shim to packages
• Had to clean up root-level bin/intent.mjs created by setup
• Fixed duplicate YAML key in domain_map.yaml

User Comments

Would like to review each skill individually before moving to the next one in future sessions.