Skip to content

refactor(skill): rewrite claude skill for expert knowledge density#47

Merged
developerdavi merged 1 commit intomainfrom
refactor/claude-skill
Mar 18, 2026
Merged

refactor(skill): rewrite claude skill for expert knowledge density#47
developerdavi merged 1 commit intomainfrom
refactor/claude-skill

Conversation

@developerdavi
Copy link
Owner

Summary

  • Rewrote .claude/skills/morg/skill.md from 498 → ~165 lines (67% reduction)
  • Replaced 11 individual command sections with one compact routing table
  • Consolidated scattered anti-patterns into one authoritative NEVER list
  • Merged duplicate worktree sections into one
  • Fixed bug: morg completion zshmorg shell-init zsh
  • Removed unsupported allowed-tools frontmatter attribute
  • Expanded trigger keywords in skill description
  • Added git worktrees section to CLAUDE.md
  • Updated worktree guidance: use pnpm install instead of manual node_modules symlinks (pnpm's content-addressable store makes installs fast)
  • Reverted accidental pnpm v11 packageManager field in package.json

Test plan

  • pnpm typecheck — clean
  • pnpm test — 60/60 passing
  • pnpm build && morg install-claude-skill --yes — installs successfully

Concentrate the skill on expert knowledge instead of command reference docs.

- Replace 11 individual command sections with one routing table
- Consolidate scattered anti-patterns into one NEVER list
- Merge duplicate worktree sections into one
- Fix bug: `morg completion` → `morg shell-init`
- Remove unsupported `allowed-tools` frontmatter
- Expand trigger keywords in description
- Add git worktrees section to CLAUDE.md
- Update worktree guidance: use `pnpm install` (fast via store)
  instead of manual node_modules symlinks
- Revert accidental pnpm v11 packageManager field
@developerdavi developerdavi merged commit c0d7d68 into main Mar 18, 2026
1 check passed
@developerdavi developerdavi deleted the refactor/claude-skill branch March 18, 2026 21:32
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