@mandipadk7/kavi 0.1.2 → 0.1.3

package/dist/tui.js

@@ -2,7 +2,7 @@ import path from "node:path";
 import readline from "node:readline";
 import process from "node:process";
 import { extractPromptPathHints, routeTask } from "./router.js";
-import { pingRpc, readSnapshot, rpcEnqueueTask, rpcResolveApproval, rpcShutdown, rpcTaskArtifact, rpcWorktreeDiff, subscribeSnapshotRpc } from "./rpc.js";
+import { pingRpc, rpcAddReviewNote, rpcAddReviewReply, rpcEnqueueReviewFollowUp, readSnapshot, rpcEnqueueTask, rpcResolveApproval, rpcSetReviewNoteStatus, rpcShutdown, rpcTaskArtifact, rpcUpdateReviewNote, rpcWorktreeDiff, subscribeSnapshotRpc } from "./rpc.js";
 const RESET = "\u001b[0m";
 const ANSI_PATTERN = /\u001b\[[0-9;]*m/g;
 const SUBSCRIPTION_RETRY_MS = 1_000;
@@ -209,6 +209,12 @@ function toneLine(value, tone, selected) {
 function countTasks(tasks, status) {
     return tasks.filter((task)=>task.status === status).length;
 }
+function countOpenReviewNotes(snapshot, agent) {
+    if (!snapshot) {
+        return 0;
+    }
+    return snapshot.session.reviewNotes.filter((note)=>note.status === "open" && (agent ? note.agent === agent : true)).length;
+}
 function changedPathCount(diff) {
     return diff?.paths.length ?? 0;
 }
@@ -517,6 +523,106 @@ function selectedHunkIndex(ui, agent, review) {
     const current = ui.hunkSelections[agent] ?? 0;
     return Math.max(0, Math.min(current, hunks.length - 1));
 }
+function reviewDispositionTone(disposition) {
+    switch(disposition){
+        case "approve":
+            return "good";
+        case "concern":
+            return "bad";
+        case "question":
+            return "warn";
+        default:
+            return "muted";
+    }
+}
+function reviewDispositionLabel(disposition) {
+    switch(disposition){
+        case "approve":
+            return "Approve";
+        case "concern":
+            return "Concern";
+        case "question":
+            return "Question";
+        case "note":
+            return "Note";
+        default:
+            return disposition;
+    }
+}
+function activeReviewContext(snapshot, ui) {
+    const agent = reviewAgentForUi(snapshot, ui);
+    if (!agent) {
+        return null;
+    }
+    const review = ui.diffReviews[agent]?.review ?? null;
+    const filePath = review?.selectedPath ?? selectedDiffPath(snapshot, ui, agent);
+    if (!filePath) {
+        return null;
+    }
+    const task = ui.activeTab === "tasks" ? selectedTask(snapshot, ui) : null;
+    const hunkIndex = selectedHunkIndex(ui, agent, review);
+    const hunkHeader = hunkIndex === null ? null : parseDiffHunks(review?.patch ?? "")[hunkIndex]?.header ?? null;
+    return {
+        agent,
+        taskId: task?.owner === agent ? task.id : null,
+        filePath,
+        hunkIndex,
+        hunkHeader
+    };
+}
+function reviewNotesForContext(snapshot, context) {
+    if (!snapshot || !context) {
+        return [];
+    }
+    return [
+        ...snapshot.session.reviewNotes
+    ].filter((note)=>{
+        if (note.agent !== context.agent || note.filePath !== context.filePath) {
+            return false;
+        }
+        if (context.hunkIndex !== null) {
+            return note.hunkIndex === context.hunkIndex || note.hunkIndex === null;
+        }
+        return true;
+    }).sort((left, right)=>right.createdAt.localeCompare(left.createdAt));
+}
+function syncSelectedReviewNote(snapshot, ui) {
+    const notes = reviewNotesForContext(snapshot, activeReviewContext(snapshot, ui));
+    ui.selectedReviewNoteId = notes.some((note)=>note.id === ui.selectedReviewNoteId) ? ui.selectedReviewNoteId : notes[0]?.id ?? null;
+}
+function selectedReviewNote(snapshot, ui) {
+    const notes = reviewNotesForContext(snapshot, activeReviewContext(snapshot, ui));
+    return notes.find((note)=>note.id === ui.selectedReviewNoteId) ?? notes[0] ?? null;
+}
+function renderReviewNotesSection(notes, selectedNoteId, width) {
+    if (notes.length === 0) {
+        return [
+            "- none"
+        ];
+    }
+    return notes.flatMap((note)=>{
+        const stateLabel = note.landedAt ? `${note.status}+landed` : note.status;
+        const prefix = `${note.id === selectedNoteId ? ">" : "-"} [${reviewDispositionLabel(note.disposition)} ${stateLabel}] ${shortTime(note.createdAt)}`;
+        const followUps = note.followUpTaskIds.length > 0 ? ` | follow-ups=${note.followUpTaskIds.length}` : "";
+        const replies = note.comments.length > 1 ? ` | replies=${note.comments.length - 1}` : "";
+        const landed = note.landedAt ? ` | landed=${shortTime(note.landedAt)}` : "";
+        const detail = note.body || note.summary;
+        return wrapText(`${prefix} ${detail}${followUps}${replies}${landed}`, width).map((line)=>toneLine(line, note.status === "resolved" ? "muted" : reviewDispositionTone(note.disposition), note.id === selectedNoteId));
+    });
+}
+function renderSelectedReviewNoteSection(note, width) {
+    if (!note) {
+        return [
+            "- none"
+        ];
+    }
+    return [
+        ...wrapText(`Status: ${note.status} | Disposition: ${reviewDispositionLabel(note.disposition)} | Updated: ${shortTime(note.updatedAt)}`, width),
+        ...wrapText(`Landed: ${note.landedAt ? shortTime(note.landedAt) : "-"}`, width),
+        ...wrapText(`Follow-up tasks: ${note.followUpTaskIds.join(", ") || "-"}`, width),
+        ...note.comments.flatMap((comment, index)=>wrapText(`${index === 0 ? "Root" : `Reply ${index}`}: ${comment.body} (${shortTime(comment.updatedAt)})`, width))
+    ];
+}
 function latestPendingApproval(approvals) {
     return [
         ...approvals
@@ -610,7 +716,7 @@ function artifactForTask(ui, task) {
 function taskDetailTitle(ui) {
     return `Inspector | Task ${ui.taskDetailSection}`;
 }
-function renderTaskInspector(task, artifactEntry, loading, diffEntry, loadingDiff, ui, width, height) {
+function renderTaskInspector(task, artifactEntry, loading, diffEntry, loadingDiff, snapshot, ui, width, height) {
     if (!task) {
         return renderPanel("Inspector", width, height, [
             styleLine("No task selected.", "muted")
@@ -698,12 +804,14 @@ function renderTaskInspector(task, artifactEntry, loading, diffEntry, loadingDif
             const hunks = parseDiffHunks(review.patch);
             const hunkIndex = agent ? selectedHunkIndex(ui, agent, review) : null;
             const selectedHunk = hunkIndex === null ? null : hunks[hunkIndex] ?? null;
+            const reviewNotes = reviewNotesForContext(snapshot, activeReviewContext(snapshot, ui));
             lines.push(...section("Review", [
                 `Agent: ${review.agent}`,
                 `Selected file: ${review.selectedPath ?? "-"}`,
                 `Changed files: ${review.changedPaths.length}`,
                 `Hunks: ${hunks.length}`,
                 `Selected hunk: ${hunkIndex === null ? "-" : `${hunkIndex + 1}/${hunks.length}`}`,
+                `Notes: ${reviewNotes.length}`,
                 `Stat: ${review.stat}`
             ].flatMap((line)=>wrapText(line, innerWidth))));
             lines.push(...section("Changed Files", review.changedPaths.length ? review.changedPaths.flatMap((filePath)=>wrapText(`${filePath === review.selectedPath ? ">" : "-"} ${filePath}`, innerWidth)) : [
@@ -718,6 +826,8 @@ function renderTaskInspector(task, artifactEntry, loading, diffEntry, loadingDif
             lines.push(...section("Patch", review.patch ? wrapPreformatted(review.patch, innerWidth) : [
                 "No textual patch available."
             ]));
+            lines.push(...section("Review Notes", renderReviewNotesSection(reviewNotes, ui.selectedReviewNoteId, innerWidth)));
+            lines.push(...section("Selected Review Note", renderSelectedReviewNoteSection(selectedReviewNote(snapshot, ui), innerWidth)));
         } else {
             lines.push(...section("Diff Review", [
                 "No diff review available yet."
@@ -892,10 +1002,12 @@ function renderWorktreeInspector(snapshot, worktree, diffEntry, loadingDiff, ui,
         const hunks = parseDiffHunks(diffEntry.review.patch);
         const hunkIndex = selectedHunkIndex(ui, worktree.agent, diffEntry.review);
         const selectedHunk = hunkIndex === null ? null : hunks[hunkIndex] ?? null;
+        const reviewNotes = reviewNotesForContext(snapshot, activeReviewContext(snapshot, ui));
         lines.push(...section("Review", [
             `Selected file: ${diffEntry.review.selectedPath ?? "-"}`,
             `Hunks: ${hunks.length}`,
             `Selected hunk: ${hunkIndex === null ? "-" : `${hunkIndex + 1}/${hunks.length}`}`,
+            `Notes: ${reviewNotes.length}`,
             `Stat: ${diffEntry.review.stat}`
         ].flatMap((line)=>wrapText(line, innerWidth))));
         if (selectedHunk) {
@@ -907,6 +1019,8 @@ function renderWorktreeInspector(snapshot, worktree, diffEntry, loadingDiff, ui,
         lines.push(...section("Patch", diffEntry.review.patch ? wrapPreformatted(diffEntry.review.patch, innerWidth) : [
             "No textual patch available."
         ]));
+        lines.push(...section("Review Notes", renderReviewNotesSection(reviewNotes, ui.selectedReviewNoteId, innerWidth)));
+        lines.push(...section("Selected Review Note", renderSelectedReviewNoteSection(selectedReviewNote(snapshot, ui), innerWidth)));
     } else {
         lines.push(...section("Diff Review", [
             "No diff review available yet."
@@ -921,7 +1035,7 @@ function renderWorktreeInspector(snapshot, worktree, diffEntry, loadingDiff, ui,
 function renderInspector(snapshot, ui, width, height) {
     switch(ui.activeTab){
         case "tasks":
-            return renderTaskInspector(selectedTask(snapshot, ui), artifactForTask(ui, selectedTask(snapshot, ui)), selectedTask(snapshot, ui) ? ui.loadingArtifacts[selectedTask(snapshot, ui)?.id ?? ""] === true : false, diffEntryForAgent(ui, managedAgentForTask(selectedTask(snapshot, ui))), managedAgentForTask(selectedTask(snapshot, ui)) ? ui.loadingDiffReviews[managedAgentForTask(selectedTask(snapshot, ui)) ?? "codex"] === true : false, ui, width, height);
+            return renderTaskInspector(selectedTask(snapshot, ui), artifactForTask(ui, selectedTask(snapshot, ui)), selectedTask(snapshot, ui) ? ui.loadingArtifacts[selectedTask(snapshot, ui)?.id ?? ""] === true : false, diffEntryForAgent(ui, managedAgentForTask(selectedTask(snapshot, ui))), managedAgentForTask(selectedTask(snapshot, ui)) ? ui.loadingDiffReviews[managedAgentForTask(selectedTask(snapshot, ui)) ?? "codex"] === true : false, snapshot, ui, width, height);
         case "approvals":
             return renderApprovalInspector(selectedApproval(snapshot, ui), width, height);
         case "claims":
@@ -965,7 +1079,8 @@ function renderLane(snapshot, agent, width, height) {
             `Worktree: ${worktree ? path.basename(worktree.path) : "-"}`,
             `Branch: ${worktree?.branch ?? "-"}`,
             `Pending approvals: ${approvals.length}`,
-            `Changed paths: ${diff?.paths.length ?? 0}`
+            `Changed paths: ${diff?.paths.length ?? 0}`,
+            `Open reviews: ${countOpenReviewNotes(snapshot, agent)}`
         ].flatMap((line)=>wrapText(line, innerWidth))),
         ...section("Summary", wrapText(status.summary ?? "No summary yet.", innerWidth)),
         ...section("Tasks", tasks.length ? tasks.slice(0, 4).flatMap((task)=>wrapText(`- [${task.status}] ${task.title}`, innerWidth)) : [
@@ -986,7 +1101,7 @@ function renderHeader(view, ui, width) {
     const repoName = path.basename(session?.repoRoot ?? process.cwd());
     const line1 = fitAnsiLine(`${toneForPanel("Kavi Operator", true)} | session=${session?.id ?? "-"} | repo=${repoName} | rpc=${view.connected ? "connected" : "disconnected"}`, width);
     const line2 = fitAnsiLine(`Goal: ${session?.goal ?? "-"} | status=${session?.status ?? "-"} | refresh=${shortTime(view.refreshedAt)}`, width);
-    const line3 = fitAnsiLine(session ? `Tasks P:${countTasks(session.tasks, "pending")} R:${countTasks(session.tasks, "running")} B:${countTasks(session.tasks, "blocked")} C:${countTasks(session.tasks, "completed")} F:${countTasks(session.tasks, "failed")} | approvals=${snapshot?.approvals.filter((approval)=>approval.status === "pending").length ?? 0} | claims=${session.pathClaims.filter((claim)=>claim.status === "active").length} | decisions=${session.decisions.length}` : "Waiting for session snapshot...", width);
+    const line3 = fitAnsiLine(session ? `Tasks P:${countTasks(session.tasks, "pending")} R:${countTasks(session.tasks, "running")} B:${countTasks(session.tasks, "blocked")} C:${countTasks(session.tasks, "completed")} F:${countTasks(session.tasks, "failed")} | approvals=${snapshot?.approvals.filter((approval)=>approval.status === "pending").length ?? 0} | reviews=${countOpenReviewNotes(snapshot)} | claims=${session.pathClaims.filter((claim)=>claim.status === "active").length} | decisions=${session.decisions.length}` : "Waiting for session snapshot...", width);
     const tabs = OPERATOR_TABS.map((tab, index)=>{
         const count = buildTabItems(snapshot, tab).length;
         const label = `[${index + 1}] ${tabLabel(tab)} ${count}`;
@@ -1015,6 +1130,19 @@ function footerSelectionSummary(snapshot, ui, width) {
 }
 function renderFooter(snapshot, ui, width) {
     const toast = currentToast(ui);
+    const reviewContext = activeReviewContext(snapshot, ui);
+    if (ui.reviewComposer) {
+        const composerHeader = fitAnsiLine(styleLine(ui.reviewComposer.mode === "edit" ? "Edit Review Note" : ui.reviewComposer.mode === "reply" ? "Reply To Review Note" : "Capture Review Note", "accent", "strong"), width);
+        const composerLine = fitAnsiLine(`Disposition: ${reviewDispositionLabel(ui.reviewComposer.disposition)} | Enter save | Esc cancel | Ctrl+U clear`, width);
+        const scopeLine = fitAnsiLine(`Scope: ${reviewContext?.agent ?? "-"} | ${reviewContext?.filePath ?? "-"}${reviewContext?.hunkIndex === null || reviewContext?.hunkIndex === undefined ? "" : ` | hunk ${reviewContext.hunkIndex + 1}`}`, width);
+        const promptLine = fitAnsiLine(`> ${ui.reviewComposer.body}`, width);
+        return [
+            composerHeader,
+            composerLine,
+            scopeLine,
+            promptLine
+        ];
+    }
     if (ui.composer) {
         const composerHeader = fitAnsiLine(styleLine("Compose Task", "accent", "strong"), width);
         const composerLine = fitAnsiLine(`Route: ${ui.composer.owner} | 1 auto 2 codex 3 claude | Enter submit | Esc cancel | Ctrl+U clear`, width);
@@ -1028,7 +1156,7 @@ function renderFooter(snapshot, ui, width) {
     }
     return [
         fitAnsiLine("Keys: 1-7 tabs | h/l or Tab cycle tabs | j/k move | [ ] task detail | ,/. diff file | { } diff hunk | c compose | r refresh", width),
-        fitAnsiLine("Actions: y/Y allow approval | n/N deny approval | g/G top/bottom | s stop daemon | q quit", width),
+        fitAnsiLine("Actions: y/Y allow approval | n/N deny approval | A/C/Q/M add note | o/O select note | T reply | E edit | R resolve | F fix task | H handoff | g/G top/bottom | s stop daemon | q quit", width),
         footerSelectionSummary(snapshot, ui, width),
         fitAnsiLine(toast ? styleLine(toast.message, toast.level === "error" ? "bad" : "good") : styleLine("Operator surface is live over the daemon socket with pushed snapshots.", "muted"), width)
     ];
@@ -1187,6 +1315,76 @@ async function resolveApprovalSelection(paths, snapshot, ui, decision, remember)
     });
     setToast(ui, "info", `${decision === "allow" ? "Approved" : "Denied"} ${approval.toolName}${remember ? " with remembered rule" : ""}.`);
 }
+async function submitReviewNote(paths, view, ui) {
+    const composer = ui.reviewComposer;
+    const context = activeReviewContext(view.snapshot, ui);
+    if (!composer || !context) {
+        throw new Error("No active diff review context is available.");
+    }
+    const body = composer.body.trim();
+    if (!body) {
+        throw new Error("Review note cannot be empty.");
+    }
+    if (composer.mode === "edit" && composer.noteId) {
+        await rpcUpdateReviewNote(paths, {
+            noteId: composer.noteId,
+            body
+        });
+    } else if (composer.mode === "reply" && composer.noteId) {
+        await rpcAddReviewReply(paths, {
+            noteId: composer.noteId,
+            body
+        });
+    } else {
+        await rpcAddReviewNote(paths, {
+            agent: context.agent,
+            taskId: context.taskId,
+            filePath: context.filePath,
+            hunkIndex: context.hunkIndex,
+            hunkHeader: context.hunkHeader,
+            disposition: composer.disposition,
+            body
+        });
+    }
+    ui.reviewComposer = null;
+    setToast(ui, "info", composer.mode === "reply" ? `Added reply to review note ${composer.noteId ?? "-"}.` : `${reviewDispositionLabel(composer.disposition)} note ${composer.mode === "edit" ? "updated" : "saved"} for ${context.filePath}${context.hunkIndex === null ? "" : ` hunk ${context.hunkIndex + 1}`}.`);
+}
+async function toggleSelectedReviewNoteStatus(paths, snapshot, ui) {
+    const note = selectedReviewNote(snapshot, ui);
+    if (!note) {
+        throw new Error("No review note is selected.");
+    }
+    const status = note.status === "resolved" ? "open" : "resolved";
+    await rpcSetReviewNoteStatus(paths, {
+        noteId: note.id,
+        status
+    });
+    setToast(ui, "info", `${status === "resolved" ? "Resolved" : "Reopened"} review note ${note.id}.`);
+}
+async function enqueueSelectedReviewFollowUp(paths, snapshot, ui, mode) {
+    const note = selectedReviewNote(snapshot, ui);
+    if (!note) {
+        throw new Error("No review note is selected.");
+    }
+    const owner = mode === "fix" ? note.agent : note.agent === "codex" ? "claude" : "codex";
+    await rpcEnqueueReviewFollowUp(paths, {
+        noteId: note.id,
+        owner,
+        mode
+    });
+    setToast(ui, "info", `Queued ${mode === "fix" ? "fix" : "handoff"} follow-up for review note ${note.id} to ${owner}.`);
+}
+function cycleSelectedReviewNote(snapshot, ui, delta) {
+    const notes = reviewNotesForContext(snapshot, activeReviewContext(snapshot, ui));
+    if (notes.length === 0) {
+        ui.selectedReviewNoteId = null;
+        return false;
+    }
+    const currentIndex = Math.max(0, notes.findIndex((note)=>note.id === ui.selectedReviewNoteId));
+    const nextIndex = (currentIndex + delta + notes.length) % notes.length;
+    ui.selectedReviewNoteId = notes[nextIndex]?.id ?? notes[0]?.id ?? null;
+    return true;
+}
 async function ensureSelectedTaskArtifact(paths, view, ui, render) {
     if (!view.connected || ui.activeTab !== "tasks") {
         return;
@@ -1261,6 +1459,7 @@ async function ensureSelectedDiffReview(paths, view, ui, render) {
         };
     } finally{
         ui.loadingDiffReviews[agent] = false;
+        syncSelectedReviewNote(view.snapshot, ui);
         render();
     }
 }
@@ -1320,6 +1519,7 @@ export async function attachTui(paths) {
         selectedIds: emptySelectionMap(),
         taskDetailSection: "overview",
         composer: null,
+        reviewComposer: null,
         toast: null,
         artifacts: {},
         loadingArtifacts: {},
@@ -1338,7 +1538,8 @@ export async function attachTui(paths) {
         hunkSelections: {
             codex: 0,
             claude: 0
-        }
+        },
+        selectedReviewNoteId: null
     };
     const render = ()=>{
         process.stdout.write(renderScreen(view, ui, paths));
@@ -1346,6 +1547,7 @@ export async function attachTui(paths) {
     const syncUiForSnapshot = (snapshot)=>{
         ui.selectedIds = syncSelections(ui.selectedIds, snapshot);
         ui.diffSelections = syncDiffSelections(ui.diffSelections, snapshot, ui.activeTab === "tasks" ? selectedTask(snapshot, ui) : null);
+        syncSelectedReviewNote(snapshot, ui);
     };
     const applySnapshot = (snapshot, reason)=>{
         view.snapshot = snapshot;
@@ -1476,6 +1678,7 @@ export async function attachTui(paths) {
         const items = buildTabItems(view.snapshot, ui.activeTab);
         ui.selectedIds[ui.activeTab] = moveSelectionId(items, ui.selectedIds[ui.activeTab], delta);
         ui.diffSelections = syncDiffSelections(ui.diffSelections, view.snapshot, ui.activeTab === "tasks" ? selectedTask(view.snapshot, ui) : null);
+        syncSelectedReviewNote(view.snapshot, ui);
         render();
         void ensureSelectedTaskArtifact(paths, view, ui, render);
         void ensureSelectedDiffReview(paths, view, ui, render);
@@ -1484,6 +1687,37 @@ export async function attachTui(paths) {
         if (closed) {
             return;
         }
+        if (ui.reviewComposer) {
+            runAction(async ()=>{
+                if (key.name === "escape") {
+                    ui.reviewComposer = null;
+                    setToast(ui, "info", "Review note capture cancelled.");
+                    render();
+                    return;
+                }
+                if (key.ctrl && key.name === "u") {
+                    ui.reviewComposer.body = "";
+                    render();
+                    return;
+                }
+                if (key.name === "backspace") {
+                    ui.reviewComposer.body = ui.reviewComposer.body.slice(0, -1);
+                    render();
+                    return;
+                }
+                if (key.name === "return") {
+                    await submitReviewNote(paths, view, ui);
+                    await refresh();
+                    render();
+                    return;
+                }
+                if (input.length === 1 && !key.ctrl && !key.meta) {
+                    ui.reviewComposer.body += input;
+                    render();
+                }
+            });
+            return;
+        }
         if (ui.composer) {
             runAction(async ()=>{
                 if (key.name === "escape") {
@@ -1507,6 +1741,7 @@ export async function attachTui(paths) {
                     await refresh();
                     ui.selectedIds.tasks = buildTabItems(view.snapshot, "tasks")[0]?.id ?? null;
                     ui.diffSelections = syncDiffSelections(ui.diffSelections, view.snapshot, selectedTask(view.snapshot, ui));
+                    syncSelectedReviewNote(view.snapshot, ui);
                     render();
                     void ensureSelectedTaskArtifact(paths, view, ui, render);
                     void ensureSelectedDiffReview(paths, view, ui, render);
@@ -1539,7 +1774,7 @@ export async function attachTui(paths) {
             });
             return;
         }
-        if (key.name === "q" || key.ctrl && key.name === "c") {
+        if (input === "q" || key.ctrl && key.name === "c") {
             close();
             return;
         }
@@ -1574,6 +1809,7 @@ export async function attachTui(paths) {
             const items = buildTabItems(view.snapshot, ui.activeTab);
             ui.selectedIds[ui.activeTab] = items[0]?.id ?? null;
             ui.diffSelections = syncDiffSelections(ui.diffSelections, view.snapshot, ui.activeTab === "tasks" ? selectedTask(view.snapshot, ui) : null);
+            syncSelectedReviewNote(view.snapshot, ui);
             render();
             void ensureSelectedTaskArtifact(paths, view, ui, render);
             void ensureSelectedDiffReview(paths, view, ui, render);
@@ -1583,6 +1819,7 @@ export async function attachTui(paths) {
             const items = buildTabItems(view.snapshot, ui.activeTab);
             ui.selectedIds[ui.activeTab] = items.at(-1)?.id ?? null;
             ui.diffSelections = syncDiffSelections(ui.diffSelections, view.snapshot, ui.activeTab === "tasks" ? selectedTask(view.snapshot, ui) : null);
+            syncSelectedReviewNote(view.snapshot, ui);
             render();
             void ensureSelectedTaskArtifact(paths, view, ui, render);
             void ensureSelectedDiffReview(paths, view, ui, render);
@@ -1597,6 +1834,7 @@ export async function attachTui(paths) {
             const nextIndex = (currentIndex + delta + TASK_DETAIL_SECTIONS.length) % TASK_DETAIL_SECTIONS.length;
             ui.taskDetailSection = TASK_DETAIL_SECTIONS[nextIndex] ?? ui.taskDetailSection;
             ui.diffSelections = syncDiffSelections(ui.diffSelections, view.snapshot, selectedTask(view.snapshot, ui));
+            syncSelectedReviewNote(view.snapshot, ui);
             render();
             void ensureSelectedTaskArtifact(paths, view, ui, render);
             void ensureSelectedDiffReview(paths, view, ui, render);
@@ -1628,12 +1866,81 @@ export async function attachTui(paths) {
             });
             return;
         }
+        if (input === "A" || input === "C" || input === "Q" || input === "M") {
+            if (!activeReviewContext(view.snapshot, ui)) {
+                setToast(ui, "error", "No active diff review context is selected.");
+                render();
+                return;
+            }
+            ui.reviewComposer = {
+                mode: "create",
+                disposition: input === "A" ? "approve" : input === "C" ? "concern" : input === "Q" ? "question" : "note",
+                noteId: null,
+                body: ""
+            };
+            render();
+            return;
+        }
+        if (input === "o" || input === "O") {
+            if (!cycleSelectedReviewNote(view.snapshot, ui, input === "O" ? -1 : 1)) {
+                setToast(ui, "error", "No review notes are available in the current diff context.");
+            }
+            render();
+            return;
+        }
+        if (input === "E") {
+            const note = selectedReviewNote(view.snapshot, ui);
+            if (!note) {
+                setToast(ui, "error", "No review note is selected.");
+                render();
+                return;
+            }
+            ui.reviewComposer = {
+                mode: "edit",
+                disposition: note.disposition,
+                noteId: note.id,
+                body: note.body
+            };
+            render();
+            return;
+        }
+        if (input === "T") {
+            const note = selectedReviewNote(view.snapshot, ui);
+            if (!note) {
+                setToast(ui, "error", "No review note is selected.");
+                render();
+                return;
+            }
+            ui.reviewComposer = {
+                mode: "reply",
+                disposition: note.disposition,
+                noteId: note.id,
+                body: ""
+            };
+            render();
+            return;
+        }
+        if (input === "R") {
+            runAction(async ()=>{
+                await toggleSelectedReviewNoteStatus(paths, view.snapshot, ui);
+                await refresh();
+            });
+            return;
+        }
+        if (input === "F" || input === "H") {
+            runAction(async ()=>{
+                await enqueueSelectedReviewFollowUp(paths, view.snapshot, ui, input === "F" ? "fix" : "handoff");
+                await refresh();
+            });
+            return;
+        }
         if (input === "," || input === ".") {
             const agent = cycleDiffSelection(view.snapshot, ui, input === "," ? -1 : 1);
             if (!agent) {
                 return;
             }
             render();
+            syncSelectedReviewNote(view.snapshot, ui);
             void ensureSelectedDiffReview(paths, view, ui, render);
             return;
         }
@@ -1643,12 +1950,14 @@ export async function attachTui(paths) {
                 return;
             }
             render();
+            syncSelectedReviewNote(view.snapshot, ui);
             return;
         }
         if (key.name === "return" && ui.activeTab === "tasks") {
             const currentIndex = TASK_DETAIL_SECTIONS.indexOf(ui.taskDetailSection);
             ui.taskDetailSection = TASK_DETAIL_SECTIONS[(currentIndex + 1) % TASK_DETAIL_SECTIONS.length] ?? ui.taskDetailSection;
             ui.diffSelections = syncDiffSelections(ui.diffSelections, view.snapshot, selectedTask(view.snapshot, ui));
+            syncSelectedReviewNote(view.snapshot, ui);
             render();
             void ensureSelectedTaskArtifact(paths, view, ui, render);
             void ensureSelectedDiffReview(paths, view, ui, render);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mandipadk7/kavi",
-  "version": "0.1.2",
+  "version": "0.1.3",
   "description": "Managed Codex + Claude collaboration TUI",
   "type": "module",
   "preferGlobal": true,