git-reviewer 2.4.0 → 2.4.1

package/README.md

@@ -3,7 +3,7 @@
 [![CI](https://github.com/dheerajjha/reviewer/actions/workflows/ci.yml/badge.svg)](https://github.com/dheerajjha/reviewer/actions/workflows/ci.yml)
 [![License: MIT](https://img.shields.io/badge/license-MIT-blue.svg)](LICENSE)
 [![Node](https://img.shields.io/badge/node-%3E%3D18-brightgreen.svg)](package.json)
-[![Tests](https://img.shields.io/badge/tests-270-brightgreen.svg)](test/)
+[![Tests](https://img.shields.io/badge/tests-271-brightgreen.svg)](test/)
 [![Dependencies](https://img.shields.io/badge/dependencies-3-brightgreen.svg)](package.json)
 
 When an agent writes the code, reading it becomes the bottleneck — and the tool
@@ -290,7 +290,7 @@ rather than a public issue.
 
 ```bash
 npm install           # 3 dependencies, no build step, ~6MB
-npm test              # 270 tests
+npm test              # 271 tests
 npm run test:watch
 npm run test:coverage
 ```

package/bin/reviewer.js

@@ -172,6 +172,25 @@ async function main() {
   // what to run next, and keeps serving.
   const handingOff = !process.stdout.isTTY;
 
+  // One newline, now, before anything else happens.
+  //
+  // This looks pointless and is not. A reader on the other end of the pipe may
+  // give up waiting for the first byte long before there is a review to send —
+  // `claude -p` waits three seconds and then proceeds without stdin at all,
+  // printing a warning and exiting 0, so the whole thing fails silently and
+  // the agent is asked to apply a review it never received. Reviewing takes
+  // minutes, not seconds. A single byte at startup holds the stream open; the
+  // rest arrives whenever the review is submitted.
+  //
+  // Measured against claude 2.1.270: producer silent for 8s then sending the
+  // instruction, the instruction was ignored and the file left unchanged; the
+  // same producer emitting one newline first had it applied. It is a
+  // first-byte timeout, not a total one.
+  //
+  // A leading blank line costs nothing — the review is Markdown, and every
+  // other consumer of a pipe is indifferent to it.
+  if (handingOff) process.stdout.write('\n');
+
   let handedOff = false;
   const stop = () => {
     server.close(() => process.exit(0));

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "git-reviewer",
-  "version": "2.4.0",
+  "version": "2.4.1",
   "description": "Review your coding agent's changes like a pull request — locally, with inline comments and threaded follow-ups, then hand the marked-up review back to the agent",
   "author": "Dheeraj Jha",
   "license": "MIT",