docs(hook-development): remove duplicated temporarily active hooks section

[sjnims]

Summary

• Removed duplicated "Temporarily Active Hooks" section from SKILL.md (50 lines)
• Updated patterns.md reference to "10 proven patterns including temporarily active and configuration-driven hooks"
• Word count reduced from 2,125 → 1,979 words (now within 1,500-2,000 target)

Problem

Fixes #65

The "Temporarily Active Hooks" section in SKILL.md duplicates content from references/patterns.md (Pattern 9: Temporarily Active Hooks and Pattern 10: Configuration-Driven Hooks), violating the skill-development best practice of keeping SKILL.md lean through progressive disclosure.

Solution

Removed the duplicated section and enhanced the reference to patterns.md to explicitly mention the temporarily active and configuration-driven hook patterns.

Alternatives Considered

• Keep both with cross-reference: Rejected because it still violates the single-source principle
• Move patterns.md content to SKILL.md: Rejected because patterns.md has more comprehensive examples

Changes

• plugins/plugin-dev/skills/hook-development/SKILL.md: Removed lines 531-578, updated patterns.md reference

Testing

• Markdownlint passes
• Word count verified (2,125 → 1,979)
• Content verified in patterns.md (Pattern 9 & 10)

---

🤖 Generated with Claude Code

[claude]

Claude finished @sjnims's task —— View job

---

PR Review Complete ✅

Tasks

• Get PR diff to understand changes
• Run markdownlint on changed file
• Verify content exists in patterns.md
• Review against plugin component criteria
• Post comprehensive review feedback

---

Summary

Looks good! This PR successfully removes duplicated content while maintaining complete documentation through progressive disclosure.

What's Good

✅ Perfect application of progressive disclosure pattern: The removed section (lines 531-578) duplicated Pattern 9 and Pattern 10 from references/patterns.md. Keeping detailed examples in reference files and a lean SKILL.md is exactly the right approach.

✅ Word count now in target range: Reduced from 2,125 → 1,979 words, within the 1,500-2,000 word target specified in the skill-development guidelines.

✅ Enhanced reference pointer: The updated line 635 now explicitly mentions "10 proven patterns including temporarily active and configuration-driven hooks" - this helps readers discover the detailed content.

✅ Markdown quality: Markdownlint passes with no errors.

✅ Content verified: I confirmed that references/patterns.md contains:

• Pattern 9: Temporarily Active Hooks (lines 261-299)
• Pattern 10: Configuration-Driven Hooks (lines 300-347)

Both patterns provide more comprehensive examples than what was removed from SKILL.md, making the reference file the better home for this content.

Checklist Verification

From the PR description:

• ✅ Markdownlint passes
• ✅ Word count verified (2,125 → 1,979)
• ✅ Content verified in patterns.md (Pattern 9 & 10)

Review Against Plugin Component Criteria

Skills structure (SKILL.md):

• ✅ YAML frontmatter present with name and description
• ✅ Strong trigger phrases in description
• ✅ Progressive disclosure maintained (detailed content in references/)
• ✅ Word count within target range

This change improves the skill by:

1. Eliminating redundancy between SKILL.md and reference files
2. Following the single-source principle
3. Making SKILL.md more scannable as an overview

Recommendation: Approve and merge. This is a clean documentation improvement that follows best practices.

---