@mandipadk7/kavi 0.1.2 → 0.1.4

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

@@ -122,6 +122,51 @@ export async function rpcRecentEvents(paths, limit) {
         limit
     });
 }
+export async function rpcAddReviewNote(paths, params) {
+    await sendRpcRequest(paths, "addReviewNote", {
+        agent: params.agent,
+        taskId: params.taskId,
+        filePath: params.filePath,
+        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,
+        ...typeof params.body === "string" ? {
+            body: params.body
+        } : {},
+        ...params.disposition ? {
+            disposition: params.disposition
+        } : {},
+        ...params.assignee === undefined ? {} : {
+            assignee: params.assignee
+        }
+    });
+}
+export async function rpcAddReviewReply(paths, params) {
+    await sendRpcRequest(paths, "addReviewReply", {
+        noteId: params.noteId,
+        body: params.body
+    });
+}
+export async function rpcSetReviewNoteStatus(paths, params) {
+    await sendRpcRequest(paths, "setReviewNoteStatus", {
+        noteId: params.noteId,
+        status: params.status
+    });
+}
+export async function rpcEnqueueReviewFollowUp(paths, params) {
+    await sendRpcRequest(paths, "enqueueReviewFollowUp", {
+        noteId: params.noteId,
+        owner: params.owner,
+        mode: params.mode
+    });
+}
 export async function rpcWorktreeDiff(paths, agent, filePath) {
     return await sendRpcRequest(paths, "worktreeDiff", {
         agent,

package/dist/session.js

@@ -35,6 +35,7 @@ export async function createSessionRecord(paths, config, runtime, sessionId, bas
         peerMessages: [],
         decisions: [],
         pathClaims: [],
+        reviewNotes: [],
         agentStatus: {
             codex: initialAgentStatus("codex", "codex-app-server"),
             claude: initialAgentStatus("claude", "claude-print")
@@ -55,6 +56,32 @@ export async function loadSessionRecord(paths) {
         })) : [];
     record.decisions = Array.isArray(record.decisions) ? record.decisions : [];
     record.pathClaims = Array.isArray(record.pathClaims) ? record.pathClaims : [];
+    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),
+                    body: typeof comment.body === "string" ? comment.body : "",
+                    createdAt: String(comment.createdAt),
+                    updatedAt: String(comment.updatedAt)
+                })) : typeof note.body === "string" && note.body ? [
+                {
+                    id: `${note.id}-root`,
+                    body: note.body,
+                    createdAt: typeof note.createdAt === "string" ? note.createdAt : nowIso(),
+                    updatedAt: typeof note.updatedAt === "string" ? note.updatedAt : nowIso()
+                }
+            ] : [],
+            resolvedAt: typeof note.resolvedAt === "string" ? note.resolvedAt : null,
+            landedAt: typeof note.landedAt === "string" ? note.landedAt : null,
+            followUpTaskIds: Array.isArray(note.followUpTaskIds) ? note.followUpTaskIds.map((item)=>String(item)) : []
+        })) : [];
     return record;
 }
 export async function saveSessionRecord(paths, record) {

package/dist/task-artifacts.js

@@ -9,7 +9,34 @@ function normalizeArtifact(artifact) {
         ...artifact,
         routeReason: typeof artifact.routeReason === "string" ? artifact.routeReason : null,
         claimedPaths: Array.isArray(artifact.claimedPaths) ? artifact.claimedPaths.map((item)=>String(item)) : [],
-        decisionReplay: Array.isArray(artifact.decisionReplay) ? artifact.decisionReplay.map((item)=>String(item)) : []
+        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 : "",
+                comments: Array.isArray(note.comments) ? note.comments.map((comment)=>({
+                        id: String(comment.id),
+                        body: typeof comment.body === "string" ? comment.body : "",
+                        createdAt: String(comment.createdAt),
+                        updatedAt: String(comment.updatedAt)
+                    })) : typeof note.body === "string" && note.body ? [
+                    {
+                        id: `${note.id}-root`,
+                        body: note.body,
+                        createdAt: typeof note.createdAt === "string" ? note.createdAt : artifact.startedAt,
+                        updatedAt: typeof note.updatedAt === "string" ? note.updatedAt : artifact.finishedAt
+                    }
+                ] : [],
+                resolvedAt: typeof note.resolvedAt === "string" ? note.resolvedAt : null,
+                landedAt: typeof note.landedAt === "string" ? note.landedAt : null,
+                followUpTaskIds: Array.isArray(note.followUpTaskIds) ? note.followUpTaskIds.map((item)=>String(item)) : []
+            })) : []
     };
 }
 export async function saveTaskArtifact(paths, artifact) {