Skip to content

refactor: convert passive voice to imperative form in skill content#8

Merged
sjnims merged 1 commit intomainfrom
refactor/4-passive-to-imperative
Dec 7, 2025
Merged

refactor: convert passive voice to imperative form in skill content#8
sjnims merged 1 commit intomainfrom
refactor/4-passive-to-imperative

Conversation

@sjnims
Copy link
Copy Markdown
Owner

@sjnims sjnims commented Dec 7, 2025

Summary

  • Convert "Understanding X enables creating Y" passive constructions to imperative "Master X to create Y" form
  • Replace second-person possessive "your" with definite article "the" in template examples
  • Ensure consistent instructional voice across all skill content

Problem

Fixes #4

Per the skill-development best practices, SKILL.md body content should use imperative/infinitive form, not passive voice or second person. Several skills contained passive constructions and second-person pronouns that needed conversion.

Solution

Applied consistent imperative voice conversions following these patterns:

Pattern Before After
Passive gerund "Understanding X enables creating Y" "Master X to create Y"
Second-person possessive "in your project" "in the project"
Second-person possessive "Your settings" "Settings"

Changes

  • agent-development/SKILL.md: Line 11 - passive to imperative
  • command-development/SKILL.md: Line 11 - passive to imperative
  • plugin-structure/SKILL.md: Line 11 - passive to imperative
  • plugin-settings/SKILL.md: Lines 296, 307, 371 - second-person to definite article

Testing

  • All changes follow imperative/infinitive form
  • Markdown linting passes
  • Skills read naturally with consistent voice
  • No second-person pronouns in instructional content

🤖 Generated with Claude Code

Convert "Understanding X enables creating Y" patterns to imperative
"Master X to create Y" form in 3 skills. Also convert second-person
"your" to definite article "the" in plugin-settings templates.

Skills updated:
- agent-development: line 11
- command-development: line 11
- plugin-structure: line 11
- plugin-settings: lines 296, 307, 371

Fixes #4

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
@claude
Copy link
Copy Markdown

claude bot commented Dec 7, 2025

Claude finished @sjnims's task —— View job


PR Review Complete ✅

I've reviewed PR #8 and the changes look excellent. This refactoring successfully addresses issue #4 by converting passive voice constructions to imperative form across the skill content.

Summary

Approved - Changes are well-executed and fully align with the skill-development best practices documented in CLAUDE.md.

What's Good

  1. Consistent Pattern Application: All changes follow the same transformation pattern:

    • "Understanding X enables creating Y" → "Master X to create Y"
    • "in your project" → "in the project"
    • "Your settings" → "Settings"
  2. Accurate Targeting: The changes are surgical and precise:

    • Lines 11 in agent-development, command-development, and plugin-structure (overview sections)
    • Lines 296, 307, 371 in plugin-settings (template examples)
  3. Adherence to CLAUDE.md Conventions: Per line 39 of CLAUDE.md, skill body content should use "Imperative/infinitive form". These changes perfectly align with that requirement.

  4. Markdown Quality: ✅ All files pass markdownlint with no errors or warnings.

  5. Natural Flow: The imperative voice reads naturally and maintains the instructional tone:

    • agent-development:11: "Master agent structure, triggering conditions, and system prompt design to create powerful autonomous capabilities"
    • command-development:11: "Master command structure, frontmatter options, and dynamic features to create powerful, reusable workflows"
    • plugin-structure:11: "Master this structure to create well-organized, maintainable plugins"
  6. Template Consistency: The plugin-settings changes correctly remove second-person possessive pronouns from template examples, making them more neutral and professional.

Checklist Verification

  • ✅ All changes follow imperative/infinitive form
  • ✅ Markdown linting passes (verified with markdownlint)
  • ✅ Skills read naturally with consistent voice
  • ✅ No second-person pronouns in instructional content
  • ✅ Changes align with CLAUDE.md writing style conventions
  • ✅ No component structure changes (frontmatter, versions intact)

Additional Notes

This is a quality refactoring that improves the consistency and professionalism of the skill documentation. Well done!


@sjnims sjnims self-assigned this Dec 7, 2025
@claude claude bot added refactor Code restructuring without behavior change component:skill Skills layer effort:small < 1 hour labels Dec 7, 2025
@sjnims sjnims merged commit 61b1ea6 into main Dec 7, 2025
8 checks passed
@sjnims sjnims deleted the refactor/4-passive-to-imperative branch December 7, 2025 23:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

component:skill Skills layer effort:small < 1 hour refactor Code restructuring without behavior change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

refactor: convert passive voice to imperative form in skill content

1 participant