Skip to content

chore(skills): remove redundant init skill#5

Open
brolag wants to merge 1 commit into
mainfrom
chore/remove-redundant-init-skill
Open

chore(skills): remove redundant init skill#5
brolag wants to merge 1 commit into
mainfrom
chore/remove-redundant-init-skill

Conversation

@brolag

@brolag brolag commented Jun 26, 2026

Copy link
Copy Markdown
Owner

What

Removes the init skill from the kit. It duplicated Claude Code's built-in /init and collided with it by name. Its only added value was reading the kit's CLAUDE.md template, which the built-in covers.

Why

  • Redundant: built-in /init already scans the stack and generates a tailored CLAUDE.md.
  • Name collision: a kit skill named init shadows the native command.
  • Low maintenance value for a kit whose principle is "minimum viable harness".

Changes

Area Change
skills/init/ Deleted
README / index.md / index.html Dropped from skill tables; 7 -> 6 skills, 23 -> 22 files. Usage docs that show /init now point at the built-in.
install.sh Stops installing init; rms the deprecated skill on upgrade so it no longer shadows the built-in.
uninstall.sh Keeps init listed to clean up older installs.
tests/test-install.sh Asserts 6 skills; validates spec instead of init.

Core pipeline (spec / craft / vet / exercise) and slop-scan, git-save untouched.

Verification

  • tests/test-install.sh — 22/22 pass
  • tests/test-hooks.sh — 16/16 pass
  • claude plugin validate . --strict — passed

The init skill duplicated Claude Code's built-in /init and collided with it
by name. Its only extra was reading the kit's CLAUDE.md template, which the
built-in covers well enough. Removed the skill and updated every reference:

- README / index.md / index.html: drop init from skill tables, 7->6 skills,
  23->22 files. Usage docs that show /init to generate CLAUDE.md now point at
  Claude Code's built-in.
- install.sh: stop installing init; rm the deprecated skill on upgrade so it
  no longer shadows the built-in.
- uninstall.sh: keep init in the list to clean up older installs.
- tests: assert 6 skills, validate spec instead of init.

Tests: test-install 22/22, test-hooks 16/16, plugin validate --strict passed.
brolag added a commit that referenced this pull request Jun 28, 2026
install.sh only copied SKILL.md per skill, which would leave html
broken (it ships templates/ and references/ it reads at runtime).
Copy the whole skill dir, add html to install/uninstall/test loops.

8 skills now (init, spec, craft, vet, exercise, git-save, slop-scan,
html). test-install.sh: 24/24 pass.

Note: collides with #5 (removes init -> 6); on merge, reconcile the
skill list/count to drop init, keeping html.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
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