fix(review): resolve promptTitle race condition

The close event fired synchronously when rl.close() was called inside
the question callback, causing resolve(null) from the close handler to
settle the Promise before resolve(answer) ran — so typed titles were
always discarded.

Drop the close event handler and resolve entirely in the question
callback. On Ctrl-C readline calls the callback with an empty string,
which is coerced to null via `answer || null`.

Author: igmagollo

src/review.ts

@@ -167,16 +167,12 @@ async function promptTitle(current: string | null): Promise<string | null> {
       (answer) => {
         rl.close();
         hideCursor();
-        // If user pressed Ctrl-C, readline emits close event before question callback
-        // We handle that via the 'close' event on rl
-        resolve(answer);
+        // On Ctrl-C, readline calls the question callback with an empty string.
+        // Treat empty string as null (cancel / clear) — callers distinguish
+        // "cancelled" from "set to empty" by receiving null either way.
+        resolve(answer || null);
       },
     );
-    // Handle Ctrl-C during input
-    rl.on("close", () => {
-      hideCursor();
-      resolve(null);
-    });
   });
 }