address-pr-comments-cecil ยท diff
git:20260530.f6800fd to git:20260530.0f2110c
6 added, 3 removed. Audit A to A.
---
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
1. Reproduction
- Identify the exact issue described by the reviewer.
- Reproduce with the smallest possible scenario.
- Confirm expected vs. actual behavior.
2. Fix
- Implement the minimal safe change.
- Preserve existing public behavior unless the review explicitly requests a behavior change.
- Follow project conventions in `AGENTS.md` and `.github/instructions/php-src.instructions.md`.
3. 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.
4. 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
- - Validation: commands run + result
- - Notes: assumptions, limitations, or next steps
+ - 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.