<?xml version="1.0" encoding="utf-8" standalone="yes"?><rss version="2.0" xmlns:atom="http://www.w3.org/2005/Atom"><channel><title>Code-Review on vnykmshr</title><link>https://blog.vnykmshr.com/writing/tags/code-review/</link><description>Recent content in Code-Review on vnykmshr</description><generator>Hugo</generator><language>en</language><lastBuildDate>Sun, 08 Mar 2026 00:00:00 +0000</lastBuildDate><atom:link href="https://blog.vnykmshr.com/writing/tags/code-review/index.xml" rel="self" type="application/rss+xml"/><item><title>The detection trap</title><link>https://blog.vnykmshr.com/writing/detection-trap/</link><pubDate>Sun, 08 Mar 2026 00:00:00 +0000</pubDate><guid>https://blog.vnykmshr.com/writing/detection-trap/</guid><description>&lt;p&gt;Read &lt;a href="https://www.techdirt.com/2026/03/06/were-training-students-to-write-worse-to-prove-theyre-not-robots-and-its-pushing-them-to-use-more-ai/"&gt;something recently&lt;/a&gt; about students deliberately making their writing &amp;ldquo;imperfect&amp;rdquo; so AI detectors don&amp;rsquo;t flag it. Removing polish, flattening style, adding imperfections on purpose. Their work got good enough to look suspicious.&lt;/p&gt;
&lt;p&gt;We&amp;rsquo;re doing the same thing with code reviews.&lt;/p&gt;
&lt;p&gt;I&amp;rsquo;ve been on both sides of this. Written a clean abstraction, consistent naming, proper error boundaries, and watched someone in review go &amp;ldquo;this looks generated.&amp;rdquo; Years of caring about consistency and now consistency is the tell.&lt;/p&gt;</description></item><item><title>The senior who stopped coding</title><link>https://blog.vnykmshr.com/writing/senior-who-stopped-coding/</link><pubDate>Tue, 20 Jan 2026 00:00:00 +0000</pubDate><guid>https://blog.vnykmshr.com/writing/senior-who-stopped-coding/</guid><description>&lt;p&gt;The terminal closes slowly.&lt;/p&gt;
&lt;p&gt;First it&amp;rsquo;s one meeting. Then a few more. Then you&amp;rsquo;re &amp;ldquo;senior&amp;rdquo; and your calendar is the job. Code reviews replace coding. Strategy replaces shipping. You advise. You guide. You no longer build. Seen this happen. Almost happened to me.&lt;/p&gt;
&lt;p&gt;The problem is not the meetings. The problem is losing touch with the trade. Architecture diagrams don&amp;rsquo;t show you the queries that fan out under load. Sprint planning doesn&amp;rsquo;t show you the retry logic that fails silently. You can&amp;rsquo;t review what you can&amp;rsquo;t recognize.&lt;/p&gt;</description></item><item><title>Coding with LLMs</title><link>https://blog.vnykmshr.com/writing/coding-with-llms/</link><pubDate>Tue, 13 Jan 2026 00:00:00 +0000</pubDate><guid>https://blog.vnykmshr.com/writing/coding-with-llms/</guid><description>&lt;p&gt;The tool changed. The craft did not.&lt;/p&gt;
&lt;p&gt;Six months with AI assistants.&lt;/p&gt;
&lt;p&gt;They write code faster than I read it. That is the problem.&lt;/p&gt;
&lt;p&gt;Fast at CRUD. Slow at concurrency. Good at common patterns. Bad at your specific constraints.&lt;/p&gt;
&lt;p&gt;The bugs are quieter now. No syntax errors. No obvious mistakes. Just wrong assumptions buried in correct-looking code.&lt;/p&gt;
&lt;p&gt;I review more carefully than before. Code I did not write but will debug in production.&lt;/p&gt;</description></item><item><title>On Phabricator</title><link>https://blog.vnykmshr.com/writing/on-phabricator/</link><pubDate>Sun, 01 Sep 2013 00:00:00 +0000</pubDate><guid>https://blog.vnykmshr.com/writing/on-phabricator/</guid><description>&lt;p&gt;I&amp;rsquo;ve been using Phabricator for several months now. Code review happened before Phab too &amp;ndash; we weren&amp;rsquo;t shipping blind. Years of SVN, then Mercurial hosted on Unfuddle for a while. Reviews ran through email threads, commit comments, someone walking over to your desk. It worked. But the review was always a side channel. The diff wasn&amp;rsquo;t a first-class thing. You&amp;rsquo;d commit, someone would look at it, feedback would come through whatever medium was handy, you&amp;rsquo;d iterate, it would go in. Review happened &lt;em&gt;around&lt;/em&gt; the code, not &lt;em&gt;through&lt;/em&gt; a tool built for it.&lt;/p&gt;</description></item></channel></rss>