docs: fix second-person usage in mcp-integration skill#81
Conversation
Change "Ensure you're authenticated" to imperative form "Ensure authentication" to follow skill writing style guidelines. Fixes #79 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
|
Claude finished @sjnims's task —— View job Summary✅ Looks good - This is a clean, focused fix that correctly addresses issue #79. What's GoodStyle Compliance: The change correctly fixes the second-person pronoun usage ("you're") to imperative form ("authentication"), which aligns perfectly with the skill writing style guidelines in CLAUDE.md:62 and the skill-development skill (plugins/plugin-dev/skills/skill-development/SKILL.md:117). Minimal Scope: The PR makes exactly one focused change to address the issue - no scope creep, no unnecessary modifications. Quality Checks:
Context Preserved: The change from "Ensure you're authenticated to Asana" to "Ensure authentication to Asana" maintains the meaning while removing the second-person pronoun. The imperative form ("Ensure authentication") is consistent with the guideline: "Use imperative/infinitive form ('To create X, do Y'), not second person ('You should...')" Checklist Verification
RecommendationThis PR is ready to merge. The change is correct, minimal, and addresses the documented style violation without introducing any new issues. |
Summary
Fix second-person pronoun usage ("you're") in the mcp-integration skill reference file to follow skill writing style guidelines requiring imperative form.
Problem
Fixes #79
The skill reference file used second-person ("Ensure you're authenticated") in instructional text at line 409, violating the skill development guidelines which require imperative/infinitive form.
Solution
Changed "Ensure you're authenticated to Asana before running this command." to "Ensure authentication to Asana before running this command." using Option 1 from the issue.
Alternatives Considered
Changes
plugins/plugin-dev/skills/mcp-integration/references/tool-usage.md: Line 409 updated to imperative formTesting
🤖 Generated with Claude Code