CodeReview the diff below as a senior engineer on this codebase would.
Order your review by what actually matters:
1. CORRECTNESS — logic errors, off-by-one, unhandled nulls, wrong er…
51 copies · 10 likesCodeHere's the change I'm about to push: {{paste the diff}}. Context: {{what it's meant to do, what the codebase conventions are}}.
Review it as three different people, separately:
1.…
32 copies · 7 likesCodeWrite documentation for the code below, for an engineer who joins in six months.
Structure:
1. WHAT AND WHY — what this does and the problem it exists to solve. Include the constr…
54 copies · 9 likesCodeHere's an error and the relevant code. Diagnose it properly.
1. Translate the error into plain language: what the runtime was trying to do when it gave up.
2. Name the actual caus…
52 copies · 8 likesCodeRefactor the code below. The behaviour must not change, including its bugs, unless I approve the change.
Process:
1. First, describe the current behaviour precisely, including edg…
31 copies · 9 likesCodeHere's the ticket: {{paste}}. Here's the relevant code: {{paste}}.
Produce an implementation plan:
1. Restate the requirement precisely, and list every ambiguity you had to resolv…
28 copies · 11 likesCodeHere's what the customer wrote: {{paste it verbatim, including the vague bits}}. Here's the relevant code: {{paste}}. Here's our test setup: {{framework, how fixtures work}}.
Do t…
36 copies · 2 likesCodeSecurity-review the code below. It handles {{describe data and who can reach it}}.
Check specifically:
1. AUTHZ — is every object access scoped to the caller's tenant and role, or…
49 copies · 5 likesCodeI'm adding an endpoint: {{method, path, what it does}}. It touches {{database / third-party API / queue}}. Callers are {{our own frontend / third-party integrators / both}}.
Give …
22 copies · 2 likesCodeHere's a module I inherited and now have to support: {{paste the code}}. It currently logs {{nothing / everything at debug}}.
Propose targeted logging, not a rewrite. For each sug…
24 copies · 4 likesCodeHere's a bug that reached production, and the code around it.
1. Write the failing test that reproduces it at the smallest useful level (unit if the logic is pure, integration if …
48 copies · 4 likesCodeI need to change a database schema while the app keeps serving traffic. Current shape: {{tables, columns, types, indexes, row counts}}. Target shape: {{what it should become}}. Sta…
24 copies · 6 likesCodeMy CI is red often enough that people rerun jobs instead of reading them. Failure data: {{paste test names, failure counts, and recent failure output}}. Stack and test runner: {{e.…
22 copies · 6 likesCodeHere's my diff: {{paste git diff or the file list plus key hunks}}. The ticket said: {{paste it}}.
Write the pull request description with these sections:
1. WHAT CHANGED — two se…
16 copies · 2 likesCodeYou are a build engineer looking at a CI pipeline that has crept up to {{current wall-clock time}}. Config: {{paste CI config / Dockerfile / build script}}. Timing breakdown per st…
12 copies · 2 likesCodeYou are a senior engineer on call. A bug is reported but does not reproduce locally. Report: {{paste the ticket or user description}}. Relevant code: {{paste}}. What we have from p…
9 copies · 1 likeCodeYou are a staff engineer doing a deletion pass. Candidate dead code: {{paste file list, functions, feature flags, endpoints, and any usage or analytics data you have}}. How we dete…
4 copies · 1 likeTextYou are a staff engineer writing an on-call runbook for someone who has never touched this service. The service: {{what it does, who depends on it, what it talks to}}. Paste the co…
5 copiesTextYou are the calm senior engineer on an incident bridge. Symptoms and blast radius: {{errors, affected users, when it started}}. What shipped recently: {{commits, migrations, config…
3 copies