By name: address-pr-comments-cecil description: Standard workflow to address pull request review comments in Cecil. Use when responding to review feedback, fixing requested changes, or preparing a clear reviewer reply with reproduction, fix, tests, and response.
Address PR Comments (Cecil)
Use this workflow to process review feedback consistently.
Response profile: Standard.
Scope
- Applies to PR comments and requested changes.
- Goal: produce a correct fix and a reviewer-ready response.
Required Sequence
- Reproduction
- Identify the exact issue described by the reviewer.
- Reproduce with the smallest possible scenario.
- Confirm expected vs. actual behavior.
- Fix
- Implement the minimal safe change.
- Preserve existing public behavior unless the review explicitly requests a behavior change.
- Follow project conventions in
AGENTS.mdand.github/instructions/php-src.instructions.md.
- Tests and checks
- Run the smallest relevant checks first.
- Use project commands as needed:
composer code:style,composer code:analyse,composer test,composer test:cli. - If a check is skipped, state it explicitly with reason.
- Reviewer response
- Explain what was changed and why.
- Reference files and key lines.
- Summarize validation performed and outcomes.
- Call out risks, assumptions, and follow-ups if any.
- Keep details concise but sufficient for asynchronous review (what changed, why, and proof of validation).
Output Template
Use this structure in your final PR comment response:
- Issue reproduced: yes/no + short evidence
- Fix applied: concise summary + impacted files/components
- Validation: commands run + result (passed/failed/skipped)
- Notes: assumptions, limitations, risks, or next steps
Guardrails
- Do not include unrelated refactors.
- Do not revert user changes outside the scope.
- Keep diffs focused and reviewable.