Engineering notes / Muhammad Kashif
CodeRabbit, Claude, and GitHub: Reviews That Catch What I Miss
Code review used to be the hour I protected and the hour I resented. Wiring CodeRabbit next to Claude and a stricter GitHub habit did not make review optional — it made the boring catches automatic so my eyes could stay …
CodeRabbit, Claude, and GitHub: Reviews That Catch What I Miss
By Muhammad Kashif · Draft · AI automation & cybersecurity builder, Pakistan
Code review used to be the hour I protected and the hour I resented. Wiring CodeRabbit next to Claude and a stricter GitHub habit did not make review optional — it made the boring catches automatic so my eyes could stay on design and blast radius. Klarna's support-staff experiment sat in the same notebook as a warning: automate the wrong layer and quality drops while the dashboard still looks efficient.
The five-level generalist map, cut down
The notes sketched a five-level AI generalist roadmap — from prompting up through RAG, MCP, agents, and vibe coding. I do not climb it like a game. I pick the lowest level that ships the artifact, then stop. Most client pain dies at RAG-plus-evals or a thin agent with MCP tools, not at a multi-agent circus.
MCP as "USB-C for AI" is the metaphor I keep: one connector shape, many tools, less custom glue. I still inventory scopes per connector. USB-C did not make every cable safe; MCP does not either.
CodeRabbit as the first pass, not the verdict
CodeRabbit catches the nits I skip when I am tired — missing tests, obvious footguns, style drift. I treat its comments as a checklist, not a merge button. Claude helps me draft replies and patches; GitHub remains where humans decide.
If CodeRabbit and I disagree on a security note, CodeRabbit wins until I can explain why it is wrong in the PR. Ego is not a vulnerability management strategy.
Klarna's lesson without the hot take
Klarna replaced hundreds of support staff with AI, then quality dropped hard enough that the story became a cautionary slide. I cite it when a client wants to delete the human layer that absorbs edge cases and apology duty.
My rewrite: automate triage and drafting, keep humans on refunds, regulated advice, and anything that needs empathy with authority. Efficiency that customers can feel as neglect is not efficiency. It is churn with better unit economics for one quarter.
Ponytail and the YAGNI agent
The Ponytail plugin that forces agents to ask whether code needs to exist — roughly fifty-four percent less code on average in the notes, sometimes far more, cheaper and faster, keeping security checks — is the habit I want even without the plugin.
Before an agent writes a new abstraction, it must answer: what breaks if we delete this? If the answer is "nothing yet," we do not write it. Forty-four thousand GitHub stars in nine days is a signal of demand for restraint, not for more generators.
GitHub hygiene the agents inherit
Branch protections, required checks, CODEOWNERS, and a short CONTRIBUTING that agents can read. AGENTS.md points at those rules. An agent that can push to main is not a productivity win; it is an incident waiting for a holiday.
I also require human review on anything that touches auth, payments, or data deletion. CodeRabbit may comment. It may not approve alone.
RAG before agents when the corpus is the product
If the value is answers from a known corpus, I ship RAG with citations and evals first. Agents come later for actions. Mixing "answer from docs" and "mutate production" in one brain is how support bots invent refunds.
MCP tools for read-only search land early. Write tools land behind confirmations and role checks. The roadmap level is a diagnosis, not a destination I owe a keynote.
What I sell from this cluster
A review pipeline install, a YAGNI skill for the coding agent, and a support-automation design that keeps humans on the sharp edges. I do not sell "we removed review." I sell "we made review focus on what matters."
Workshops cover the five-level map. Retainers cover the pipeline staying honest after the workshop glow fades.
Monday merge rule
No merge if CodeRabbit security findings are unresolved, if Ponytail-or-equivalent YAGNI questions were skipped, or if a human has not looked at auth-adjacent diffs. The rule is boring. Boring is the point.
Review debt board
CodeRabbit findings older than a week become a board item with an owner. Claude may draft fixes; humans still schedule them. Ponytail-style YAGNI does not apply to unresolved security comments. Those comments are debt with interest denominated in incident risk.