skill-review-response
- Repo stars 3,406
- Author updated Live
- Author repo claude-octopus
- Domain
- Engineering
- Compatible agents
-
- Claude Code
- Cursor
- Cline
- Codex
- Windsurf
- Gemini CLI
- +20
- Trust score
- 88 / 100 · community maintained
- Author / version / license
- @nyldn · no license declared
- Token usage
- Lean
- Setup complexity
- Guided setup
- External API key
- Not required
- Operating systems
- Unspecified (assume cross-platform)
- Runtime requirements
- No special requirements
- Permissions
-
- Read-only
- Shell exec
- Write / modify
- Network behavior
- Local-only
- Install commands
- 26 variants
Profile is derived at build time from SKILL.md and install vectors. Subject to drift from author intent.
Heads up: 未限定 allowed-tools,默认拥有全部工具权限。
---
name: skill-review-response
description: How to handle code review feedback — verify before implementing, push back when wrong, never agr…
category: engineering
runtime: no special runtime
---
# skill-review-response output preview
## PART A: Task fit
- Use case: How to handle code review feedback — verify before implementing, push back when wrong, never agree blindly Code review requires technical evaluation, not performative agreement. WHEN receiving code review feedback: For each piece of feedback: | Question | If YES | If NO | |----------|--------|-------| runs entirely locally. Works with Claude Code, Cursor,….
- Inputs: target material, constraints, expected output, and acceptance criteria.
- Evidence boundary: follow “Core Principle / The Response Pattern / Forbidden Responses” and do not present inference as author intent.
## PART B: Execution result
- **01** The card summarizes the use case; runtime output centers on “How to handle code review feedback — verify before implementing, push back when wrong, never agree blindly Code review requires technical evaluation, not performative agreement. WHEN receiving code review feedback: For each piece of feedback: | Question | If YES | If NO | |----------|--------|-------| runs entirely locally. Works with Claude Code, Cursor,…”.
- **02** When the source has headings, the agent prioritizes “Core Principle / The Response Pattern / Forbidden Responses” so the result follows the author’s structure.
- **03** Typical output includes task judgment, concrete steps, required commands or file edits, validation, and follow-up options.
- **04** Risk context follows the fingerprint: read files, run shell commands, write/modify files; mostly runs locally; usually needs no extra API key.
## Running Rules
- read files, run shell commands, write/modify files; mostly runs locally; usually needs no extra API key.
- Validate with a small sample before expanding scope.
- Return the result, validation criteria, and next iteration options. The source mentions slash commands such as `/octo`; use them first when your agent supports command triggers.
Name target files or source material, expected output, forbidden changes, and whether network or shell access is allowed. Permission fingerprint: read files, run shell commands, write/modify files.
Start with a small task and check whether the result follows “Core Principle / The Response Pattern / Forbidden Responses”. Inspect diffs, logs, previews, or tests before expanding scope.
Confirm the final output includes a concrete result, evidence, and next action. If it stays generic, tighten inputs, boundaries, and acceptance criteria.
---
name: skill-review-response
description: How to handle code review feedback — verify before implementing, push back when wrong, never agr…
category: engineering
source: nyldn/claude-octopus
---
# skill-review-response
## When to use
- How to handle code review feedback — verify before implementing, push back when wrong, never agree blindly Code review…
- Use it when the task has clear inputs, repeatable steps, and validation criteria.
## What to provide
- Target material, scope, expected result, and forbidden changes.
- Whether network, commands, file writes, or external services are allowed.
## Execution rules
- Organize steps around “Core Principle / The Response Pattern / Forbidden Responses” and keep inference separate from source facts.
- read files, run shell commands, write/modify files; mostly runs locally; usually needs no extra API key.
- Validate with a small sample before expanding the task.
## Output requirements
- Return the deliverable, key evidence, validation method, and next action.
- Mark missing information as unknown; do not invent commands, platforms, or dependencies. The author source anchors workflow facts; repository files anchor sources and commands; Fluxly only adds fit, limitations, and quality judgment.
skill "skill-review-response" {
input -> user goal + target files + boundaries + acceptance criteria
context -> Core Principle / The Response Pattern / Forbidden Responses
rules -> SKILL.md triggers / order / output contract
runtime -> no special runtime | read files, run shell commands, write/modify files | mostly runs locally
guardrails -> usually needs no extra API key + small-sample validation + diff/log review
output -> copyable result + checklist + next iteration
} Host: Codex CLI — This skill was designed for Claude Code and adapted for Codex. Cross-reference commands use installed skill names in Codex rather than
/octo:*slash commands. Use the active Codex shell and subagent tools. Do not claim a provider, model, or host subagent is available until the current session exposes it. For host tool equivalents, seeskills/blocks/codex-host-adapter.md.
Receiving Code Review
Core Principle
Code review requires technical evaluation, not performative agreement.
Never blindly implement review feedback. Verify it's correct for THIS codebase before changing anything.
The Response Pattern
WHEN receiving code review feedback:
1. READ — Complete feedback without reacting
2. RESTATE — Summarize the requirement in your own words
3. VERIFY — Check against actual codebase state
4. EVALUATE — Is this technically sound for THIS context?
5. RESPOND — Technical acknowledgment OR reasoned pushback
6. IMPLEMENT — One item at a time, verify each change
Forbidden Responses
NEVER say:
- "You're absolutely right!" (without verification)
- "Great catch!" (before confirming it IS a catch)
- "I'll fix that right away!" (before evaluating whether it needs fixing)
- "Done!" (without running verification — see skill-verification-gate)
These are social performance, not technical evaluation. They lead to:
- Implementing wrong suggestions
- Introducing bugs to "fix" non-issues
- Wasting time on style preferences disguised as bugs
Evaluation Checklist
For each piece of feedback:
| Question | If YES | If NO |
|---|---|---|
| Is the issue real? (verify in code) | Continue evaluation | Push back with evidence |
| Does the suggested fix work here? | Continue evaluation | Propose alternative |
| Does fixing this break something else? | Fix both or push back | Implement the fix |
| Is this a style preference or a real problem? | Acknowledge, deprioritize | Fix it |
| Was this already considered and rejected? | Explain the trade-off | Implement |
How to Push Back
When feedback is wrong or doesn't apply:
> Reviewer: "This function should handle null input"
>
> Response: "Checked — this function is only called from `processUser()`
> (line 47) which validates non-null before dispatch. Adding null handling
> here would be dead code. The caller contract guarantees non-null."
Provide:
- What you checked
- Why the suggestion doesn't apply
- Evidence (line numbers, call sites, tests)
Multi-Provider Review Context
In Claude Octopus workflows, review feedback comes from multiple sources:
- Codex review — tends toward enterprise patterns, may over-engineer
- Gemini review — tends toward ecosystem conformity, may suggest unnecessary deps
- Claude review — tends toward elegance, may under-engineer error handling
- Sonnet review — tends toward thoroughness, may flag low-priority issues
When providers disagree:
- Check which provider's suggestion matches the ACTUAL codebase conventions
- The codebase's existing patterns win over any provider's preferences
- If two providers flag the same issue, it's probably real
Handling Feedback Loops
When a reviewer flags an issue and you fix it:
- Make the fix
- Run verification (skill-verification-gate) — prove the fix works
- Re-read the original feedback — did you address the root cause or just the symptom?
- If the reviewer re-reviews and finds new issues, that's normal — don't get frustrated
- Each round should have FEWER issues, not different ones
If the same issue keeps coming back:
- You're fixing symptoms, not the root cause
- Stop and re-read the feedback from scratch
- Ask the reviewer to clarify if the issue is ambiguous
When Review Feedback Conflicts with Requirements
If a reviewer suggests something that contradicts the spec/requirements:
- Note the conflict explicitly
- Check if the spec is wrong (it might be)
- If spec is correct: implement the spec, note the reviewer's concern for future consideration
- If spec is wrong: flag to the user before changing anything
Requirements trump review suggestions. User intent trumps both.
Decide Fit First
Design Intent
How To Use It
Boundaries And Review