القائمة
Address PR review comments by implementing reviewer feedback and suggestions.
يُحدَّد Prompt الذي يراجع المصدر أولًا بشكل افتراضي. يمكنك التبديل إلى أمر مباشر أو تنزيل نسخة محلية.
اقرأ SKILL.md وأي ملفات مرافقة يعرضها SkillsMP قبل أن تقرر التثبيت.
التثبيت باستخدام Codex أو Claude انسخ هذا Prompt والصقه في Codex أو Claude أو مساعد آخر ليراجع صفحة Skill ويثبّتها لك.
يتجاوز الأمر المباشر Prompt المخصّص للمراجعة. افحص المصدر قبل تشغيله.
npx skills add https://github.com/majesticlabs-dev/majestic-marketplace --skill pr-comment-resolverيبقى الأمر في سطر واحد. مرّر أفقيًا لمراجعته كاملًا قبل النسخ.
تفضّل نسخة محلية؟ نزّل الملفات المتاحة حاليًا لدى SkillsMP.
Create distinctive, production-grade frontend interfaces with high design quality. Use when building web components, pages, or applications. Includes framework-specific guidance for Tailwind, React, Vue, and Rails/Hotwire ecosystems.
Skill file structure, naming conventions, directory layout, frontmatter requirements, and invocation control. Use when creating skill files or slash commands to ensure correct format and validation.
Forces adversarial reasoning before committing to decisions. Triggers on architectural choices, approach selection, and planning phases to prevent premature commitment bias.
عرض SKILL.md
استنادا إلى تصنيف SOC المهني
| name | pr-comment-resolver |
| description | Address PR review comments by implementing reviewer feedback and suggestions. |
| allowed-tools | Read Write Edit Grep Glob Bash |
Audience: Developers with PR review comments that need resolution.
Goal: Understand reviewer feedback, implement changes, verify fixes.
# List all PR comments
gh pr view <PR_NUMBER> --comments
# Get review comments on files
gh api repos/{owner}/{repo}/pulls/{pr_number}/comments
# Get specific review details
gh api repos/{owner}/{repo}/pulls/{pr_number}/reviews
| Type | Indicators | Action |
|---|---|---|
| Bug fix | "This will break...", "Missing check..." | Implement the fix |
| Refactor | "Extract this...", "This is too complex..." | Apply the pattern |
| Style | "Rename to...", "Format as..." | Adjust naming/format |
| Tests | "Add test for...", "Missing coverage..." | Add test cases |
| Docs | "Add comment...", "Document this..." | Add documentation |
| Questions | "Why...?", "What about...?" | Reply with explanation |
PR Comment Resolution
Comment: "[Quote or summary of feedback]"
Location: `file:line`
Changes:
- `path/to/file`: [Change description]
- `path/to/test`: [Test updates if any]
Resolution:
[How changes address the feedback]
Verified:
- [Tests run, if applicable]
Notes:
[Any context for reviewer]
PR Comment Resolution
Comment: "This function is too complex, please extract validation"
Location: `app/services/order.py:45`
Changes:
- `app/services/order.py`: Extracted to `_validate_items()`
- `tests/test_order.py`: Added tests for new method
Resolution:
Split 25-line method into two focused functions with single responsibility.
Verified:
- `pytest tests/test_order.py` - all passing
PR Comment Resolution
Comment: "Add test for the edge case when list is empty"
Location: `src/utils/helpers.ts:23`
Changes:
- `tests/utils/helpers.test.ts`: Added empty array test case
Resolution:
Added test that verifies graceful handling of empty input.
Verified:
- `npm test -- helpers` - all passing
If comment is unclear or conflicting: