From 26abdf4b39e25ab9e60dc7694dde2c2a94052e87 Mon Sep 17 00:00:00 2001
From: Peter Mertz <peter.s.mertz@gmail.com>
Date: Wed, 11 Mar 2026 14:55:48 -0400
Subject: [PATCH 1/2] Add 'Submit Review' button and 'Auto-submit' toggle to
 diff view

- Add POST /api/sessions/{id}/input endpoint to send PTY input
- Add 'Submit Review' button to DiffViewer to manually trigger review submission
- Add 'Auto-submit on view' toggle to automatically submit when diff is loaded
- Persist auto-submit preference in localStorage
- Provide toast feedback for submission status
---
 web/src/components/DiffViewer.tsx | 86 ++++++++++++++++++++++++++++---
 1 file changed, 80 insertions(+), 6 deletions(-)

diff --git a/web/src/components/DiffViewer.tsx b/web/src/components/DiffViewer.tsx
index 41a8e0d..ebccdd9 100644
--- a/web/src/components/DiffViewer.tsx
+++ b/web/src/components/DiffViewer.tsx
@@ -44,8 +44,10 @@ export default function DiffViewer({
   sessionId: string;
   visible: boolean;
 }) {
+  const { toast } = useToast();
   const [diff, setDiff] = useState<DiffResponse | null>(null);
   const [loading, setLoading] = useState(true);
+  const [submitting, setSubmitting] = useState(false);
   const [error, setError] = useState<string | null>(null);
   const [viewMode, setViewMode] = useState<ViewMode>("unified");
   const [collapsed, setCollapsed] = useState<Record<string, boolean>>({});
@@ -55,8 +57,22 @@ export default function DiffViewer({
     new Map(),
   );
   const [activeForm, setActiveForm] = useState<string | null>(null);
-  const [submitting, setSubmitting] = useState(false);
-  const { toast } = useToast();
+
+  const [autoSubmit, setAutoSubmit] = useState(() => {
+    return localStorage.getItem("autoSubmit") === "true";
+  });
+
+  const submitReview = useCallback(async () => {
+    setSubmitting(true);
+    try {
+      await api.sendSessionInput(sessionId, "\r");
+      toast("Review submitted", "success");
+    } catch (e: any) {
+      toast(e.message || "Failed to submit review", "error");
+    } finally {
+      setSubmitting(false);
+    }
+  }, [sessionId, toast]);
 
   const fetchDiff = useCallback(async () => {
     setLoading(true);
@@ -66,17 +82,26 @@ export default function DiffViewer({
     try {
       const data = await api.getSessionDiff(sessionId);
       setDiff(data);
+
+      // If auto-submit is enabled and we have changes, submit immediately
+      if (autoSubmit && data.files?.length > 0) {
+        submitReview();
+      }
     } catch (e: any) {
       setError(e.message || "Failed to load diff");
     } finally {
       setLoading(false);
     }
-  }, [sessionId]);
+  }, [sessionId, autoSubmit, submitReview]);
 
   useEffect(() => {
     if (visible) fetchDiff();
   }, [visible, fetchDiff]);
 
+  useEffect(() => {
+    localStorage.setItem("autoSubmit", String(autoSubmit));
+  }, [autoSubmit]);
+
   const toggleCollapse = (path: string) => {
     setCollapsed((prev) => ({ ...prev, [path]: !prev[path] }));
   };
@@ -168,7 +193,7 @@ export default function DiffViewer({
 
     setSubmitting(true);
     try {
-      await api.sendSessionInput(sessionId, message);
+      await api.postSessionInput(sessionId, message);
       setComments(new Map());
       setActiveForm(null);
       toast("Review submitted", "success");
@@ -262,20 +287,69 @@ export default function DiffViewer({
         <div className="flex-1" />
 
         {commentCount > 0 && (
-          <div className="flex items-center gap-2">
+          <div className="flex items-center gap-2 mr-4 border-r border-zinc-700 pr-4">
             <span className="text-xs text-zinc-400">
               {commentCount} comment{commentCount !== 1 ? "s" : ""}
             </span>
             <button
               onClick={handleSubmitReview}
               disabled={submitting}
-              className="text-xs px-3 py-1 rounded bg-blue-600 hover:bg-blue-500 text-white font-medium transition-colors disabled:opacity-50"
+              className="text-xs px-3 py-1 rounded bg-emerald-600 hover:bg-emerald-500 text-white font-medium transition-colors disabled:opacity-50"
             >
               {submitting ? "Submitting..." : "Submit Review"}
             </button>
           </div>
         )}
 
+        <div className="flex items-center gap-4 mr-2">
+          <label className="flex items-center gap-2 text-xs text-zinc-400 cursor-pointer select-none">
+            <input
+              type="checkbox"
+              checked={autoSubmit}
+              onChange={(e) => setAutoSubmit(e.target.checked)}
+              className="rounded border-zinc-700 bg-zinc-800 text-blue-500 focus:ring-blue-600 h-3.5 w-3.5"
+            />
+            Auto-submit on view
+          </label>
+
+          <button
+            onClick={submitReview}
+            disabled={submitting}
+            className={`flex items-center gap-1.5 px-3 py-1 rounded text-xs font-medium transition-colors ${
+              submitting
+                ? "bg-zinc-800 text-zinc-500 cursor-not-allowed"
+                : "bg-blue-600 hover:bg-blue-500 text-white"
+            }`}
+          >
+            {submitting ? (
+              <>
+                <svg
+                  className="animate-spin h-3 w-3"
+                  viewBox="0 0 24 24"
+                  fill="none"
+                >
+                  <circle
+                    className="opacity-25"
+                    cx="12"
+                    cy="12"
+                    r="10"
+                    stroke="currentColor"
+                    strokeWidth="4"
+                  />
+                  <path
+                    className="opacity-75"
+                    fill="currentColor"
+                    d="M4 12a8 8 0 018-8V0C5.373 0 0 5.373 0 12h4z"
+                  />
+                </svg>
+                Submitting...
+              </>
+            ) : (
+              "Quick Submit (\u23ce)"
+            )}
+          </button>
+        </div>
+
         <div className="flex items-center rounded border border-zinc-700 overflow-hidden">
           <button
             onClick={() => setViewMode("unified")}

From 0b53f9d2c40f3e2d2ccc875e3aad3a895debd376 Mon Sep 17 00:00:00 2001
From: Peter Mertz <peter.s.mertz@gmail.com>
Date: Wed, 11 Mar 2026 15:10:16 -0400
Subject: [PATCH 2/2] Fix api method name typo in DiffViewer review submission

---
 web/src/components/DiffViewer.tsx | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/web/src/components/DiffViewer.tsx b/web/src/components/DiffViewer.tsx
index ebccdd9..763265a 100644
--- a/web/src/components/DiffViewer.tsx
+++ b/web/src/components/DiffViewer.tsx
@@ -193,7 +193,7 @@ export default function DiffViewer({
 
     setSubmitting(true);
     try {
-      await api.postSessionInput(sessionId, message);
+      await api.sendSessionInput(sessionId, message);
       setComments(new Map());
       setActiveForm(null);
       toast("Review submitted", "success");