{"id":"review-implementing","name":"review-implementing","summary":"コードレビューのフィードバックを体系的に処理し実施すること。ユーザーがレビュアーコメント、PRフィードバック、コードレビューノート、またはレビューからの提案を実装するよう依頼する際に利用してください。","body":"# Review Feedback Implementation\n\nSystematically process and implement changes based on code review feedback.\n\n## When to Use\n\n- Provides reviewer comments or feedback\n- Pastes PR review notes\n- Mentions implementing review suggestions\n- Says \"address these comments\" or \"implement feedback\"\n- Shares list of changes requested by reviewers\n\n## Systematic Workflow\n\n### 1. Parse Reviewer Notes\n\nIdentify individual feedback items:\n- Split numbered lists (1., 2., etc.)\n- Handle bullet points or unnumbered feedback\n- Extract distinct change requests\n- Clarify ambiguous items before starting\n\n### 2. Create Todo List\n\nUse TodoWrite tool to create actionable tasks:\n- Each feedback item becomes one or more todos\n- Break down complex feedback into smaller tasks\n- Make tasks specific and measurable\n- Mark first task as `in_progress` before starting\n\nExample:\n```\n- Add type hints to extract function\n- Fix duplicate tag detection logic\n- Update docstring in chain.py\n- Add unit test for edge case\n```\n\n### 3. Implement Changes Systematically\n\nFor each todo item:\n\n**Locate relevant code:**\n- Use Grep to search for functions/classes\n- Use Glob to find files by pattern\n- Read current implementation\n\n**Make changes:**\n- Use Edit tool for modifications\n- Follow project conventions (CLAUDE.md)\n- Preserve existing functionality unless changing behavior\n\n**Verify changes:**\n- Check syntax correctness\n- Run relevant tests if applicable\n- Ensure changes address reviewer's intent\n\n**Update status:**\n- Mark todo as `completed` immediately after finishing\n- Move to next todo (only one `in_progress` at a time)\n\n### 4. Handle Different Feedback Types\n\n**Code changes:**\n- Use Edit tool for existing code\n- Follow type hint conventions (PEP 604/585)\n- Maintain consistent style\n\n**New features:**\n- Create new files with Write tool if needed\n- Add corresponding tests\n- Update documentation\n\n**Documentation:**\n- Update docstrings following project style\n- Modify markdown files as needed\n- Keep explanations concise\n\n**Tests:**\n- Write tests as functions, not classes\n- Use descriptive names\n- Follow pytest conventions\n\n**Refactoring:**\n- Preserve functionality\n- Improve code structure\n- Run tests to verify no regressions\n\n### 5. Validation\n\nAfter implementing changes:\n- Run affected tests\n- Check for linting errors: `uv run ruff check`\n- Verify changes don't break existing functionality\n\n### 6. Communication\n\nKeep user informed:\n- Update todo list in real-time\n- Ask for clarification on ambiguous feedback\n- Report blockers or challenges\n- Summarize changes at completion\n\n## Edge Cases\n\n**Conflicting feedback:**\n- Ask user for guidance\n- Explain conflict clearly\n\n**Breaking changes required:**\n- Notify user before implementing\n- Discuss impact and alternatives\n\n**Tests fail after changes:**\n- Fix tests before marking todo complete\n- Ensure all related tests pass\n\n**Referenced code doesn't exist:**\n- Ask user for clarification\n- Verify understanding before proceeding\n\n## Important Guidelines\n\n- **Always use TodoWrite** for tracking progress\n- **Mark todos completed immediately** after each item\n- **Only one todo in_progress** at any time\n- **Don't batch completions** - update status in real-time\n- **Ask questions** for unclear feedback\n- **Run tests** if changes affect tested code\n- **Follow CLAUDE.md conventions** for all code changes\n- **Use conventional commits** if creating commits afterward","author":"@mhattingpete","ownerProfile":null,"authorContacts":null,"sourceUrl":"https://github.com/mhattingpete/claude-skills-marketplace/tree/main/engineering-workflow-plugin/skills/review-implementing","license":"Apache-2.0","category":"coding","lang":"en","tokens":731,"stars":0,"calls30d":2,"claimed":false,"visibility":"public","origin":"crawler","version":"0.1.0","createdAt":"2026-08-22","updatedAt":"2026-08-22","files":[],"requires":{"mcp":[],"tools":[]},"safety":{"flags":[],"scannedAt":"2026-08-22","hasScripts":false,"networkEndpoints":[]}}