@mandipadk7/kavi 0.1.3 → 0.1.4

package/README.md

@@ -32,73 +32,19 @@ Runtime model:
 - The operator surface talks to the daemon over a local control socket under the machine-local state root.
 - SQLite event mirroring is opt-in with `KAVI_ENABLE_SQLITE_HISTORY=1`; the default event log is JSONL under `.kavi/state/events.jsonl`.
 - The operator console exposes a task board, dual agent lanes, a live inspector pane, approval actions, inline task composition, worktree diff review with file and hunk navigation, and persisted operator review threads on files or hunks.
-
-Development:
-
-```bash
-# works in an empty folder; Kavi will initialize git and create the
-# bootstrap commit it needs for managed worktrees
-node bin/kavi.js init --home
-node bin/kavi.js doctor --json
-node bin/kavi.js paths
-node bin/kavi.js start --goal "Build the auth backend"
-node bin/kavi.js status
-node bin/kavi.js task --agent auto "Design the dashboard shell"
-node bin/kavi.js tasks
-node bin/kavi.js task-output latest
-node bin/kavi.js decisions
-node bin/kavi.js claims
-node bin/kavi.js reviews
-node bin/kavi.js approvals
-node bin/kavi.js open
-npm test
-```
+- Routing can now use explicit path ownership rules from `.kavi/config.toml` via `[routing].codex_paths` and `[routing].claude_paths`, so known parts of the tree can bypass looser keyword or AI routing.
 
 Notes:
 - `kavi init` and `kavi open` now support the "empty folder to first managed session" path. If no git repo exists, Kavi initializes one; if git exists but no `HEAD` exists yet, Kavi creates the bootstrap commit it needs for worktrees.
 - Codex runs through `codex app-server` in managed mode, so Codex-side approvals now land in the same Kavi inbox as Claude hook approvals.
 - Claude still runs through the installed `claude` CLI with Kavi-managed hooks and approval decisions.
+- `kavi doctor` now checks Claude auth readiness with `claude auth status`, and startup blocks if Claude is installed but not authenticated.
 - The dashboard and operator commands now use the daemon's local RPC socket instead of editing session files directly, and the TUI stays updated from pushed daemon snapshots rather than polling.
 - The socket is machine-local rather than repo-local to avoid Unix socket path-length issues in deep repos and temp directories.
-- The console is keyboard-driven: `1-7` switch views, `j/k` move selection, `[` and `]` cycle task detail sections, `,` and `.` cycle changed files, `{` and `}` cycle patch hunks, `A/C/Q/M` add review notes, `o/O` cycle existing threads, `T` reply, `E` edit, `R` resolve or reopen, `F` queue a fix task, `H` queue a handoff task, `y/n` resolve approvals, and `c` opens the inline task composer.
+- The console is keyboard-driven: `1-7` switch views, `j/k` move selection, `[` and `]` cycle task detail sections, `,` and `.` cycle changed files, `{` and `}` cycle patch hunks, `A/C/Q/M` add review notes, `o/O` cycle existing threads, `T` reply, `E` edit, `R` resolve or reopen, `a` cycle thread assignee, `w` mark a thread as won't-fix, `x` mark it as accepted-risk, `F` queue a fix task, `H` queue a handoff task, `y/n` resolve approvals, and `c` opens the inline task composer.
+- Review threads now carry explicit assignees and richer dispositions, including `accepted risk` and `won't fix`, instead of relying only on free-form note text.
 - Successful follow-up tasks now auto-resolve linked open review threads, landed follow-up work marks those resolved threads as landed, and replying to a resolved thread reopens it.
 
-Local install options:
-
-```bash
-# Run directly from the repo
-node bin/kavi.js help
-
-# Symlink into ~/.local/bin
-./scripts/install-local.sh
-
-# Or use npm link from this repo
-npm link
-```
-
-Publishing for testers:
-
-```bash
-# interactive publish flow: prompts for version, defaults to the next patch,
-# runs release checks, then publishes the beta tag
-npm run publish
-
-# authenticate once
-npm login
-
-# verify the package before publish
-npm run release:check
-
-# prompt for version and publish beta explicitly
-npm run publish:beta
-
-# prompt for version and publish stable
-npm run publish:latest
-
-# test the publish flow without sending anything to npm
-npm run publish -- --dry-run
-```
-
 Install commands for testers:
 
 ```bash
@@ -109,13 +55,6 @@ npm install -g @mandipadk7/kavi@beta
 npx @mandipadk7/kavi@beta help
 ```
 
-Notes on publish:
-- The package name is scoped as `@mandipadk7/kavi` to match the npm user `mandipadk7`.
-- The publish flow now builds a compiled `dist/` directory first, and the installed CLI prefers `dist/main.js`. Source mode remains available for local development.
-- Hook commands now invoke the compiled entrypoint directly when `dist/` is present, and only use `--experimental-strip-types` in source mode.
-- The interactive publish script strips `repository`, `homepage`, and `bugs` from the packaged `package.json`, so the npm page uses the bundled `README.md` instead of private GitHub links.
-- `prepublishOnly` runs the release checks automatically during publish.
-
 User-local config example:
 
 ```toml

package/dist/config.js

@@ -10,6 +10,8 @@ message_limit = 6
 [routing]
 frontend_keywords = ["frontend", "ui", "ux", "design", "copy", "react", "css", "html"]
 backend_keywords = ["backend", "api", "server", "db", "schema", "migration", "auth", "test"]
+codex_paths = []
+claude_paths = []
 
 [agents.codex]
 role = "planning-backend"
@@ -77,7 +79,9 @@ export function defaultConfig() {
                 "migration",
                 "auth",
                 "test"
-            ]
+            ],
+            codexPaths: [],
+            claudePaths: []
         },
         agents: {
             codex: {
@@ -140,7 +144,9 @@ export async function loadConfig(paths) {
         messageLimit: asNumber(parsed.message_limit, 6),
         routing: {
             frontendKeywords: asStringArray(routing.frontend_keywords, defaultConfig().routing.frontendKeywords),
-            backendKeywords: asStringArray(routing.backend_keywords, defaultConfig().routing.backendKeywords)
+            backendKeywords: asStringArray(routing.backend_keywords, defaultConfig().routing.backendKeywords),
+            codexPaths: asStringArray(routing.codex_paths, defaultConfig().routing.codexPaths),
+            claudePaths: asStringArray(routing.claude_paths, defaultConfig().routing.claudePaths)
         },
         agents: {
             codex: {

package/dist/daemon.js

@@ -13,6 +13,18 @@ import { addReviewReply, addReviewNote, autoResolveReviewNotesForCompletedTask,
 import { buildAdHocTask, buildKickoffTasks } from "./router.js";
 import { loadSessionRecord, readRecentEvents, recordEvent, saveSessionRecord } from "./session.js";
 import { loadTaskArtifact, saveTaskArtifact } from "./task-artifacts.js";
+function normalizeReviewDisposition(value) {
+    if (value === "approve" || value === "concern" || value === "question" || value === "accepted_risk" || value === "wont_fix") {
+        return value;
+    }
+    return "note";
+}
+function normalizeReviewAssignee(value) {
+    if (value === "codex" || value === "claude" || value === "operator") {
+        return value;
+    }
+    return null;
+}
 export class KaviDaemon {
     paths;
     session;
@@ -391,7 +403,8 @@ export class KaviDaemon {
         await this.runMutation(async ()=>{
             const agent = params.agent === "claude" ? "claude" : "codex";
             const filePath = typeof params.filePath === "string" ? params.filePath.trim() : "";
-            const disposition = params.disposition === "approve" || params.disposition === "concern" || params.disposition === "question" ? params.disposition : "note";
+            const disposition = normalizeReviewDisposition(params.disposition);
+            const assignee = normalizeReviewAssignee(params.assignee) ?? agent;
             const body = typeof params.body === "string" ? params.body.trim() : "";
             const taskId = typeof params.taskId === "string" ? params.taskId : null;
             const hunkIndex = typeof params.hunkIndex === "number" ? params.hunkIndex : null;
@@ -404,6 +417,7 @@ export class KaviDaemon {
             }
             const note = addReviewNote(this.session, {
                 agent,
+                assignee,
                 taskId,
                 filePath,
                 hunkIndex,
@@ -422,6 +436,7 @@ export class KaviDaemon {
                     hunkIndex: note.hunkIndex,
                     hunkHeader: note.hunkHeader,
                     disposition: note.disposition,
+                    assignee: note.assignee,
                     reviewNoteId: note.id
                 }
             });
@@ -435,7 +450,8 @@ export class KaviDaemon {
                 taskId: note.taskId,
                 filePath: note.filePath,
                 hunkIndex: note.hunkIndex,
-                disposition: note.disposition
+                disposition: note.disposition,
+                assignee: note.assignee
             });
             await this.publishSnapshot("review.note_added");
         });
@@ -443,15 +459,28 @@ export class KaviDaemon {
     async updateReviewNoteFromRpc(params) {
         await this.runMutation(async ()=>{
             const noteId = typeof params.noteId === "string" ? params.noteId : "";
-            const body = typeof params.body === "string" ? params.body.trim() : "";
+            const body = typeof params.body === "string" ? params.body.trim() : undefined;
+            const disposition = params.disposition === undefined ? undefined : normalizeReviewDisposition(params.disposition);
+            const assignee = params.assignee === undefined ? undefined : normalizeReviewAssignee(params.assignee);
             if (!noteId) {
                 throw new Error("updateReviewNote requires a noteId.");
             }
-            if (!body) {
-                throw new Error("updateReviewNote requires a note body.");
+            if ((body === undefined || body.length === 0) && disposition === undefined && assignee === undefined) {
+                throw new Error("updateReviewNote requires at least one body, disposition, or assignee change.");
+            }
+            if (body !== undefined && body.length === 0) {
+                throw new Error("updateReviewNote requires a non-empty note body.");
             }
             const note = updateReviewNote(this.session, noteId, {
-                body
+                ...body !== undefined ? {
+                    body
+                } : {},
+                ...disposition !== undefined ? {
+                    disposition
+                } : {},
+                ...assignee !== undefined ? {
+                    assignee
+                } : {}
             });
             if (!note) {
                 throw new Error(`Review note ${noteId} was not found.`);
@@ -465,7 +494,9 @@ export class KaviDaemon {
                 metadata: {
                     reviewNoteId: note.id,
                     filePath: note.filePath,
-                    hunkIndex: note.hunkIndex
+                    hunkIndex: note.hunkIndex,
+                    disposition: note.disposition,
+                    assignee: note.assignee
                 }
             });
             await saveSessionRecord(this.paths, this.session);
@@ -476,7 +507,9 @@ export class KaviDaemon {
                 reviewNoteId: note.id,
                 taskId: note.taskId,
                 agent: note.agent,
-                filePath: note.filePath
+                filePath: note.filePath,
+                disposition: note.disposition,
+                assignee: note.assignee
             });
             await this.publishSnapshot("review.note_updated");
         });
@@ -593,7 +626,7 @@ export class KaviDaemon {
                 ]
             });
             this.session.tasks.push(task);
-            linkReviewFollowUpTask(this.session, note.id, taskId);
+            linkReviewFollowUpTask(this.session, note.id, taskId, owner);
             upsertPathClaim(this.session, {
                 taskId,
                 agent: owner,
@@ -612,7 +645,8 @@ export class KaviDaemon {
                     owner,
                     mode,
                     filePath: note.filePath,
-                    sourceAgent: note.agent
+                    sourceAgent: note.agent,
+                    assignee: owner
                 }
             });
             await saveSessionRecord(this.paths, this.session);
@@ -624,7 +658,8 @@ export class KaviDaemon {
                 followUpTaskId: taskId,
                 owner,
                 mode,
-                filePath: note.filePath
+                filePath: note.filePath,
+                assignee: owner
             });
             await this.publishSnapshot("review.followup_queued");
         });

package/dist/doctor.js

@@ -1,6 +1,24 @@
 import { fileExists } from "./fs.js";
 import { runCommand } from "./process.js";
 import { hasSupportedNode, minimumNodeMajor, resolveSessionRuntime } from "./runtime.js";
+export function parseClaudeAuthStatus(output) {
+    try {
+        const parsed = JSON.parse(output);
+        const loggedIn = parsed.loggedIn === true;
+        const authMethod = typeof parsed.authMethod === "string" ? parsed.authMethod : "unknown";
+        const apiProvider = typeof parsed.apiProvider === "string" ? parsed.apiProvider : "unknown";
+        return {
+            loggedIn,
+            detail: loggedIn ? `logged in via ${authMethod} (${apiProvider})` : `not logged in (${authMethod}, ${apiProvider})`
+        };
+    } catch  {
+        const trimmed = output.trim();
+        return {
+            loggedIn: false,
+            detail: trimmed || "unable to parse claude auth status"
+        };
+    }
+}
 function normalizeVersion(output) {
     return output.trim().split(/\s+/).slice(-1)[0] ?? output.trim();
 }
@@ -37,6 +55,21 @@ export async function runDoctor(repoRoot, paths) {
         ok: claudeVersion.code === 0,
         detail: claudeVersion.code === 0 ? `${claudeVersion.stdout.trim()} via ${runtime.claudeExecutable}` : claudeVersion.stderr.trim()
     });
+    const claudeAuth = await runCommand(runtime.claudeExecutable, [
+        "auth",
+        "status"
+    ], {
+        cwd: repoRoot
+    });
+    const claudeAuthStatus = claudeAuth.code === 0 ? parseClaudeAuthStatus(claudeAuth.stdout) : {
+        loggedIn: false,
+        detail: claudeAuth.stderr.trim() || claudeAuth.stdout.trim() || "claude auth status failed"
+    };
+    checks.push({
+        name: "claude-auth",
+        ok: claudeAuth.code === 0 && claudeAuthStatus.loggedIn,
+        detail: claudeAuth.code === 0 ? claudeAuthStatus.detail : claudeAuthStatus.detail
+    });
     const worktreeCheck = await runCommand("git", [
         "worktree",
         "list"

package/dist/main.js

@@ -247,6 +247,7 @@ async function ensureStartupReady(repoRoot, paths) {
         "node",
         "codex",
         "claude",
+        "claude-auth",
         "git-worktree",
         "codex-app-server",
         "codex-auth-file"
@@ -543,6 +544,7 @@ async function commandTaskOutput(cwd, args) {
     } else {
         for (const note of artifact.reviewNotes){
             console.log(`${note.createdAt} | ${note.disposition} | ${note.status} | ${note.filePath}${note.hunkIndex === null ? "" : ` | hunk ${note.hunkIndex + 1}`}`);
+            console.log(`  assignee: ${note.assignee ?? "-"}`);
             console.log(`  comments: ${note.comments.length}`);
             for (const [index, comment] of note.comments.entries()){
                 console.log(`  ${index === 0 ? "root" : `reply-${index}`}: ${comment.body}`);
@@ -615,6 +617,7 @@ async function commandReviews(cwd, args) {
     for (const note of notes){
         console.log(`${note.id} | ${note.agent} | ${note.status} | ${note.disposition} | ${note.filePath}${note.hunkIndex === null ? "" : ` | hunk ${note.hunkIndex + 1}`}`);
         console.log(`  task: ${note.taskId ?? "-"}`);
+        console.log(`  assignee: ${note.assignee ?? "-"}`);
         console.log(`  updated: ${note.updatedAt}`);
         console.log(`  comments: ${note.comments.length}`);
         console.log(`  landed: ${note.landedAt ?? "-"}`);

package/dist/reviews.js

@@ -5,11 +5,21 @@ function trimBody(value) {
     return value.trim();
 }
 function summarizeReviewNote(disposition, filePath, hunkHeader, body) {
-    const label = disposition[0].toUpperCase() + disposition.slice(1);
+    const label = reviewDispositionSummaryLabel(disposition);
     const scope = hunkHeader ? `${filePath} ${hunkHeader}` : filePath;
     const firstLine = trimBody(body).split("\n")[0]?.trim() ?? "";
     return firstLine ? `${label} ${scope}: ${firstLine}` : `${label} ${scope}`;
 }
+function reviewDispositionSummaryLabel(disposition) {
+    switch(disposition){
+        case "accepted_risk":
+            return "Accepted Risk";
+        case "wont_fix":
+            return "Won't Fix";
+        default:
+            return disposition[0].toUpperCase() + disposition.slice(1);
+    }
+}
 function createReviewComment(body) {
     const timestamp = nowIso();
     return {
@@ -24,6 +34,7 @@ export function addReviewNote(session, input) {
     const note = {
         id: randomUUID(),
         agent: input.agent,
+        assignee: input.assignee ?? input.agent,
         taskId: input.taskId ?? null,
         filePath: input.filePath,
         hunkIndex: input.hunkIndex ?? null,
@@ -68,8 +79,10 @@ export function updateReviewNote(session, noteId, input) {
     }
     const nextBody = typeof input.body === "string" ? trimBody(input.body) : note.body;
     const nextDisposition = input.disposition ?? note.disposition;
+    const nextAssignee = input.assignee === undefined ? note.assignee : input.assignee;
     note.body = nextBody;
     note.disposition = nextDisposition;
+    note.assignee = nextAssignee;
     if (note.comments.length === 0) {
         note.comments.push(createReviewComment(nextBody));
     } else if (typeof input.body === "string") {
@@ -96,7 +109,7 @@ export function setReviewNoteStatus(session, noteId, status) {
     note.updatedAt = nowIso();
     return note;
 }
-export function linkReviewFollowUpTask(session, noteId, taskId) {
+export function linkReviewFollowUpTask(session, noteId, taskId, assignee) {
     const note = session.reviewNotes.find((item)=>item.id === noteId) ?? null;
     if (!note) {
         return null;
@@ -107,6 +120,9 @@ export function linkReviewFollowUpTask(session, noteId, taskId) {
             taskId
         ])
     ];
+    if (assignee !== undefined) {
+        note.assignee = assignee;
+    }
     note.updatedAt = nowIso();
     return note;
 }
@@ -122,6 +138,19 @@ export function addReviewReply(session, noteId, body) {
     note.updatedAt = nowIso();
     return note;
 }
+export function cycleReviewAssignee(current, noteAgent) {
+    const sequence = [
+        noteAgent,
+        noteAgent === "codex" ? "claude" : "codex",
+        "operator",
+        null
+    ];
+    const index = sequence.findIndex((item)=>item === current);
+    if (index === -1) {
+        return noteAgent;
+    }
+    return sequence[(index + 1) % sequence.length] ?? null;
+}
 export function autoResolveReviewNotesForCompletedTask(session, taskId) {
     const resolved = [];
     for (const note of session.reviewNotes){

package/dist/router.js

@@ -1,3 +1,4 @@
+import path from "node:path";
 import { CodexAppServerClient } from "./codex-app-server.js";
 import { findClaimConflicts } from "./decision-ledger.js";
 import { nowIso } from "./paths.js";
@@ -43,6 +44,55 @@ function normalizeClaimedPaths(paths) {
         ...new Set(paths.map((item)=>item.trim()).filter(Boolean))
     ].sort();
 }
+function normalizePathPattern(value) {
+    const trimmed = value.trim().replaceAll("\\", "/");
+    const withoutPrefix = trimmed.startsWith("./") ? trimmed.slice(2) : trimmed;
+    const normalized = path.posix.normalize(withoutPrefix);
+    return normalized === "." ? "" : normalized.replace(/^\/+/, "");
+}
+function globToRegex(pattern) {
+    const normalized = normalizePathPattern(pattern);
+    const escaped = normalized.replace(/[|\\{}()[\]^$+?.]/g, "\\$&");
+    const regexSource = escaped.replaceAll("**", "::double-star::").replaceAll("*", "[^/]*").replaceAll("::double-star::", ".*");
+    return new RegExp(`^${regexSource}$`);
+}
+function matchesPattern(filePath, pattern) {
+    const normalizedPath = normalizePathPattern(filePath);
+    const normalizedPattern = normalizePathPattern(pattern);
+    if (!normalizedPath || !normalizedPattern) {
+        return false;
+    }
+    return globToRegex(normalizedPattern).test(normalizedPath);
+}
+function countPathMatches(filePaths, patterns) {
+    if (filePaths.length === 0 || patterns.length === 0) {
+        return 0;
+    }
+    return filePaths.filter((filePath)=>patterns.some((pattern)=>matchesPattern(filePath, pattern))).length;
+}
+function buildPathOwnershipDecision(prompt, config) {
+    const claimedPaths = extractPromptPathHints(prompt);
+    if (claimedPaths.length === 0) {
+        return null;
+    }
+    const codexMatches = countPathMatches(claimedPaths, config.routing.codexPaths);
+    const claudeMatches = countPathMatches(claimedPaths, config.routing.claudePaths);
+    if (codexMatches === 0 && claudeMatches === 0) {
+        return null;
+    }
+    if (codexMatches === claudeMatches) {
+        return null;
+    }
+    const owner = codexMatches > claudeMatches ? "codex" : "claude";
+    const matchedPatterns = owner === "codex" ? config.routing.codexPaths : config.routing.claudePaths;
+    return {
+        owner,
+        strategy: "manual",
+        confidence: 0.97,
+        reason: `Matched explicit ${owner} path ownership rules for: ${claimedPaths.join(", ")}.`,
+        claimedPaths
+    };
+}
 export function extractPromptPathHints(prompt) {
     const candidates = [];
     const quotedMatches = prompt.matchAll(/[`'"]([^`'"\n]+)[`'"]/g);
@@ -59,6 +109,10 @@ export function extractPromptPathHints(prompt) {
     return normalizeClaimedPaths(candidates);
 }
 export function routePrompt(prompt, config) {
+    const pathDecision = buildPathOwnershipDecision(prompt, config);
+    if (pathDecision) {
+        return pathDecision.owner;
+    }
     if (containsKeyword(prompt, config.routing.frontendKeywords)) {
         return "claude";
     }
@@ -92,6 +146,11 @@ function buildKeywordDecision(prompt, config) {
     return null;
 }
 function buildRouterPrompt(prompt, session) {
+    const ownershipRules = [
+        ...session.config.routing.codexPaths.map((pattern)=>`- codex: ${pattern}`),
+        ...session.config.routing.claudePaths.map((pattern)=>`- claude: ${pattern}`)
+    ].join("\n");
+    const promptHints = extractPromptPathHints(prompt);
     const activeClaims = session.pathClaims.filter((claim)=>claim.status === "active").map((claim)=>`- ${claim.agent}: ${claim.paths.join(", ")}`).join("\n");
     return [
         "Route this task between Codex and Claude.",
@@ -103,6 +162,12 @@ function buildRouterPrompt(prompt, session) {
         "Task prompt:",
         prompt,
         "",
+        "Explicit path ownership rules:",
+        ownershipRules || "- none",
+        "",
+        "Path hints extracted from the prompt:",
+        promptHints.length > 0 ? promptHints.map((item)=>`- ${item}`).join("\n") : "- none",
+        "",
         "Active path claims:",
         activeClaims || "- none"
     ].join("\n");
@@ -174,6 +239,10 @@ function applyClaimRouting(session, decision) {
     };
 }
 export async function routeTask(prompt, session, _paths) {
+    const pathDecision = buildPathOwnershipDecision(prompt, session.config);
+    if (pathDecision) {
+        return applyClaimRouting(session, pathDecision);
+    }
     const heuristic = buildKeywordDecision(prompt, session.config);
     if (heuristic) {
         return applyClaimRouting(session, heuristic);

package/dist/rpc.js

@@ -130,13 +130,22 @@ export async function rpcAddReviewNote(paths, params) {
         hunkIndex: params.hunkIndex,
         hunkHeader: params.hunkHeader,
         disposition: params.disposition,
+        assignee: params.assignee ?? null,
         body: params.body
     });
 }
 export async function rpcUpdateReviewNote(paths, params) {
     await sendRpcRequest(paths, "updateReviewNote", {
         noteId: params.noteId,
-        body: params.body
+        ...typeof params.body === "string" ? {
+            body: params.body
+        } : {},
+        ...params.disposition ? {
+            disposition: params.disposition
+        } : {},
+        ...params.assignee === undefined ? {} : {
+            assignee: params.assignee
+        }
     });
 }
 export async function rpcAddReviewReply(paths, params) {

package/dist/session.js

@@ -59,9 +59,11 @@ export async function loadSessionRecord(paths) {
     record.reviewNotes = Array.isArray(record.reviewNotes) ? record.reviewNotes.map((note)=>({
             ...note,
             body: typeof note.body === "string" ? note.body : "",
+            assignee: note.assignee === "codex" || note.assignee === "claude" || note.assignee === "operator" ? note.assignee : null,
             taskId: typeof note.taskId === "string" ? note.taskId : null,
             hunkIndex: typeof note.hunkIndex === "number" ? note.hunkIndex : null,
             hunkHeader: typeof note.hunkHeader === "string" ? note.hunkHeader : null,
+            disposition: note.disposition === "approve" || note.disposition === "concern" || note.disposition === "question" || note.disposition === "accepted_risk" || note.disposition === "wont_fix" ? note.disposition : "note",
             status: note.status === "resolved" ? "resolved" : "open",
             comments: Array.isArray(note.comments) ? note.comments.map((comment)=>({
                     id: String(comment.id),

package/dist/task-artifacts.js

@@ -12,9 +12,11 @@ function normalizeArtifact(artifact) {
         decisionReplay: Array.isArray(artifact.decisionReplay) ? artifact.decisionReplay.map((item)=>String(item)) : [],
         reviewNotes: Array.isArray(artifact.reviewNotes) ? artifact.reviewNotes.map((note)=>({
                 ...note,
+                assignee: note.assignee === "codex" || note.assignee === "claude" || note.assignee === "operator" ? note.assignee : null,
                 taskId: typeof note.taskId === "string" ? note.taskId : null,
                 hunkIndex: typeof note.hunkIndex === "number" ? note.hunkIndex : null,
                 hunkHeader: typeof note.hunkHeader === "string" ? note.hunkHeader : null,
+                disposition: note.disposition === "approve" || note.disposition === "concern" || note.disposition === "question" || note.disposition === "accepted_risk" || note.disposition === "wont_fix" ? note.disposition : "note",
                 status: note.status === "resolved" ? "resolved" : "open",
                 summary: typeof note.summary === "string" ? note.summary : "",
                 body: typeof note.body === "string" ? note.body : "",

package/dist/tui.js

@@ -1,6 +1,7 @@
 import path from "node:path";
 import readline from "node:readline";
 import process from "node:process";
+import { cycleReviewAssignee } from "./reviews.js";
 import { extractPromptPathHints, routeTask } from "./router.js";
 import { pingRpc, rpcAddReviewNote, rpcAddReviewReply, rpcEnqueueReviewFollowUp, readSnapshot, rpcEnqueueTask, rpcResolveApproval, rpcSetReviewNoteStatus, rpcShutdown, rpcTaskArtifact, rpcUpdateReviewNote, rpcWorktreeDiff, subscribeSnapshotRpc } from "./rpc.js";
 const RESET = "\u001b[0m";
@@ -530,7 +531,10 @@ function reviewDispositionTone(disposition) {
         case "concern":
             return "bad";
         case "question":
+        case "accepted_risk":
             return "warn";
+        case "wont_fix":
+            return "muted";
         default:
             return "muted";
     }
@@ -543,12 +547,28 @@ function reviewDispositionLabel(disposition) {
             return "Concern";
         case "question":
             return "Question";
+        case "accepted_risk":
+            return "Accepted Risk";
+        case "wont_fix":
+            return "Won't Fix";
         case "note":
             return "Note";
         default:
             return disposition;
     }
 }
+function reviewAssigneeLabel(assignee) {
+    switch(assignee){
+        case "codex":
+            return "codex";
+        case "claude":
+            return "claude";
+        case "operator":
+            return "operator";
+        default:
+            return "unassigned";
+    }
+}
 function activeReviewContext(snapshot, ui) {
     const agent = reviewAgentForUi(snapshot, ui);
     if (!agent) {
@@ -605,9 +625,10 @@ function renderReviewNotesSection(notes, selectedNoteId, width) {
         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 assignee = ` | assignee=${reviewAssigneeLabel(note.assignee)}`;
         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));
+        return wrapText(`${prefix} ${detail}${assignee}${followUps}${replies}${landed}`, width).map((line)=>toneLine(line, note.status === "resolved" ? "muted" : reviewDispositionTone(note.disposition), note.id === selectedNoteId));
     });
 }
 function renderSelectedReviewNoteSection(note, width) {
@@ -618,6 +639,7 @@ function renderSelectedReviewNoteSection(note, width) {
     }
     return [
         ...wrapText(`Status: ${note.status} | Disposition: ${reviewDispositionLabel(note.disposition)} | Updated: ${shortTime(note.updatedAt)}`, width),
+        ...wrapText(`Assignee: ${reviewAssigneeLabel(note.assignee)}`, 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))
@@ -1156,7 +1178,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 | 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),
+        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 | a cycle assignee | w won't fix | x accepted risk | 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)
     ];
@@ -1374,6 +1396,34 @@ async function enqueueSelectedReviewFollowUp(paths, snapshot, ui, mode) {
     });
     setToast(ui, "info", `Queued ${mode === "fix" ? "fix" : "handoff"} follow-up for review note ${note.id} to ${owner}.`);
 }
+async function cycleSelectedReviewNoteAssignee(paths, snapshot, ui) {
+    const note = selectedReviewNote(snapshot, ui);
+    if (!note) {
+        throw new Error("No review note is selected.");
+    }
+    const assignee = cycleReviewAssignee(note.assignee, note.agent);
+    await rpcUpdateReviewNote(paths, {
+        noteId: note.id,
+        assignee
+    });
+    setToast(ui, "info", `Assigned review note ${note.id} to ${reviewAssigneeLabel(assignee)}.`);
+}
+async function resolveSelectedReviewNoteWithDisposition(paths, snapshot, ui, disposition) {
+    const note = selectedReviewNote(snapshot, ui);
+    if (!note) {
+        throw new Error("No review note is selected.");
+    }
+    await rpcUpdateReviewNote(paths, {
+        noteId: note.id,
+        disposition,
+        assignee: "operator"
+    });
+    await rpcSetReviewNoteStatus(paths, {
+        noteId: note.id,
+        status: "resolved"
+    });
+    setToast(ui, "info", `Marked review note ${note.id} as ${reviewDispositionLabel(disposition).toLowerCase()}.`);
+}
 function cycleSelectedReviewNote(snapshot, ui, delta) {
     const notes = reviewNotesForContext(snapshot, activeReviewContext(snapshot, ui));
     if (notes.length === 0) {
@@ -1927,6 +1977,20 @@ export async function attachTui(paths) {
             });
             return;
         }
+        if (input === "a") {
+            runAction(async ()=>{
+                await cycleSelectedReviewNoteAssignee(paths, view.snapshot, ui);
+                await refresh();
+            });
+            return;
+        }
+        if (input === "w" || input === "x") {
+            runAction(async ()=>{
+                await resolveSelectedReviewNoteWithDisposition(paths, view.snapshot, ui, input === "w" ? "wont_fix" : "accepted_risk");
+                await refresh();
+            });
+            return;
+        }
         if (input === "F" || input === "H") {
             runAction(async ()=>{
                 await enqueueSelectedReviewFollowUp(paths, view.snapshot, ui, input === "F" ? "fix" : "handoff");

package/package.json

@@ -1,9 +1,10 @@
 {
   "name": "@mandipadk7/kavi",
-  "version": "0.1.3",
+  "version": "0.1.4",
   "description": "Managed Codex + Claude collaboration TUI",
   "type": "module",
   "preferGlobal": true,
+  "homepage": "https://www.npmjs.com/package/@mandipadk7/kavi",
   "publishConfig": {
     "access": "public"
   },