Prompt Details
Model
(claude-5-sonnet)
Token size
733
Example input
[FEEDBACK_BRIEF]: Feedback to a direct report. Engineer, mid-level, joined 8 months ago. Pattern: their PRs are usually fine in isolation but they routinely introduce inconsistencies with patterns elsewhere in the codebase. I've mentioned it twice in 1:1s; it hasn't really changed. Want this to be the conversation that produces a real change. They're a good engineer otherwise.
Example output
**The feedback in one line:**
Your code review feedback has been about pattern consistency for several months; I want to dig into what's making the pattern-matching hard for you, because I think it's a fixable gap, but I need you to engage with it.
**The grounding:**
Three concrete examples from the last month:
- The new payment-flow PR introduced a Result-type pattern when the rest of the codebase uses exceptions.
- The auth update added a barrel file (index.ts re-export) when we'd specifically agreed against them.
- The reporting feature stored money as float dollars when the rest of the codebase uses integer cents.
In all three cases, the patterns are documented in CLAUDE.md and were called out in PR review. The work itself was solid; the inconsistency cost rework.
**The impact:**
Each instance costs reviewer time and rework. More importantly, the pattern of introducing inconsistencies signals to the team that the codebase has no canonical style — which makes future onboarding harder and gives the next mid-level engineer permission to invent their own conventions. The work you do well is being partially undone by the work I have to redo.
**My read on why:**
Best guess: pattern-matching to a codebase is a learned skill, not a skill you arrived with. You may be reading code as "this is how to solve this problem" rather than "this is how this codebase solves this problem." Or you may be reading our conventions once when you joined and not consulting them since.
**What I'm asking for:**
- In the next two PRs, explicitly call out in the PR description which existing patterns you matched and which you considered alternatives to. If you didn't think about pattern matching for a specific decision, say so. This makes the work visible to me.
- Re-read the conventions doc this week with fresh eyes. Bring questions to our next 1:1.
- I want this to be the conversation. The next time we talk about pattern consistency, it should be because something new happened, not because I'm repeating myself.
**What I'm NOT saying:**
- I'm not saying the work is bad. The engineering is solid.
- I'm not threatening your job. We're 8 months in; this is normal calibration territory.
- I'm not saying you have to mechanically follow every convention without questioning. If you think a convention is wrong, raise that — but raise it before you write the PR, not by quietly breaking it.
**Suggested phrasing for the actual conversation:**
"I want to spend most of this 1:1 on the pattern-consistency feedback, because I've raised it twice and I want to make sure we're actually addressing it rather than repeating the loop. I have three specific examples from the past month I want to walk through together. The work itself in each case was good — I'm not questioning that. What I'm trying to figure out is what's making the pattern-matching part hard. I have a hypothesis I want to share, and I want to hear your view, and then we'll agree on two or three concrete things to try."
By purchasing this prompt, you agree to our terms of service
CLAUDE-5-SONNET
Helps you give feedback that lands — specific, grounded, and kind without being soft. Built against the classic failure modes: the compliment sandwich that buries the point, and blunt "honesty" that's just unkind. It produces the one-line message, the grounding, the impact, your read on why, the specific ask, what you're NOT saying (so it doesn't balloon), and a phrasing to open with. Won't psychoanalyze or pile on five points. One of 40 in the Pemway Power User Pack on Gumroad.
...more
Added 1 week ago
