@nuvin/nuvin-core 1.8.0 → 1.9.1

package/dist/VERSION

@@ -1,4 +1,4 @@
 {
-  "version": "1.8.0",
-  "commit": "da28f44"
+  "version": "1.9.1",
+  "commit": "a486158"
 }

package/dist/index.d.ts

@@ -352,47 +352,71 @@ type FileReadMetadata = FileMetadata & {
     bomStripped?: boolean;
 };
 type FileEditMetadata = FileMetadata & {
-    changesMade?: {
-        linesAdded: number;
-        linesRemoved: number;
-        bytesChanged: number;
-    };
-    backup?: {
-        path: string;
-        created: string;
+    eol: 'lf' | 'crlf';
+    oldTextLength: number;
+    newTextLength: number;
+    bytesWritten: number;
+    beforeSha: string;
+    afterSha: string;
+    dryRun: boolean;
+    lineNumbers: {
+        oldStartLine: number;
+        oldEndLine: number;
+        newStartLine: number;
+        newEndLine: number;
+        oldLineCount: number;
+        newLineCount: number;
     };
+    noChange?: boolean;
 };
-type FileNewMetadata = FileMetadata & {
+type FileNewMetadata = {
+    file_path: string;
+    bytes: number;
+    lines: number;
+    created: string;
     overwritten?: boolean;
 };
 type DirLsMetadata = {
-    path: string;
-    totalEntries?: number;
-    entriesReturned?: number;
-    truncated?: boolean;
+    limit: number;
+    includeHidden: boolean;
 };
 type WebSearchMetadata = {
-    query: string;
-    totalResults?: number;
-    resultsReturned: number;
-    searchTime?: number;
-    provider?: string;
+    offset: number;
+    totalRequested: number;
+    hydrated: boolean;
 };
 type WebFetchMetadata = {
     url: string;
-    statusCode?: number;
-    contentType?: string;
-    contentLength?: number;
-    fetchTime?: number;
-    markdown?: boolean;
+    contentType: string;
+    statusCode: number;
+    format: 'markdown' | 'json' | 'text';
+    size: number;
+    fetchedAt: string;
 };
 type TodoWriteMetadata = {
     todosWritten: number;
-    conversationId?: string;
+    conversationId: string;
+    recentChanges: boolean;
+    progress: string;
+    allCompleted: boolean;
+    items: Array<{
+        id: string;
+        content: string;
+        status: 'pending' | 'in_progress' | 'completed';
+        priority: 'high' | 'medium' | 'low';
+        createdAt: string;
+    }>;
+    stats: {
+        total: number;
+        pending: number;
+        inProgress: number;
+        completed: number;
+    };
 };
 type AssignTaskMetadata = DelegationMetadata;
 type ToolErrorMetadata = {
     errorReason?: ErrorReason;
+    editInstruction?: string;
     path?: string;
     agentId?: string;
     code?: string | number;
@@ -418,6 +442,7 @@ type ToolCall = {
         name: string;
         arguments: string;
     };
+    editInstruction?: string;
 };
 type TextContentPart = {
     type: 'text';
@@ -586,46 +611,57 @@ type ToolInvocation = {
     id: string;
     name: 'bash_tool';
     parameters: BashToolArgs;
+    editInstruction?: string;
 } | {
     id: string;
     name: 'file_read';
     parameters: FileReadArgs;
+    editInstruction?: string;
 } | {
     id: string;
     name: 'file_edit';
     parameters: FileEditArgs;
+    editInstruction?: string;
 } | {
     id: string;
     name: 'file_new';
     parameters: FileNewArgs;
+    editInstruction?: string;
 } | {
     id: string;
     name: 'dir_ls';
     parameters: DirLsArgs;
+    editInstruction?: string;
 } | {
     id: string;
     name: 'web_search';
     parameters: WebSearchArgs;
+    editInstruction?: string;
 } | {
     id: string;
     name: 'web_fetch';
     parameters: WebFetchArgs;
+    editInstruction?: string;
 } | {
     id: string;
     name: 'todo_write';
     parameters: TodoWriteArgs;
+    editInstruction?: string;
 } | {
     id: string;
     name: 'assign_task';
     parameters: AssignTaskArgs;
+    editInstruction?: string;
 } | {
     id: string;
     name: string;
     parameters: Record<string, unknown>;
+    editInstruction?: string;
 };
 declare enum ErrorReason {
     Aborted = "aborted",
     Denied = "denied",
+    Edited = "edited",
     Timeout = "timeout",
     NotFound = "not_found",
     PermissionDenied = "permission_denied",
@@ -841,7 +877,7 @@ declare const AgentEventTypes: {
     readonly SubAgentCompleted: "sub_agent_completed";
     readonly SubAgentMetrics: "sub_agent_metrics";
 };
-type ToolApprovalDecision = 'approve' | 'deny' | 'approve_all';
+type ToolApprovalDecision = 'approve' | 'deny' | 'approve_all' | 'edit';
 type AgentEvent = {
     type: typeof AgentEventTypes.MessageStarted;
     conversationId: string;
@@ -868,6 +904,7 @@ type AgentEvent = {
     approvalId: string;
     decision: ToolApprovalDecision;
     approvedCalls?: ToolCall[];
+    editInstruction?: string;
 } | {
     type: typeof AgentEventTypes.ToolResult;
     conversationId: string;
@@ -1034,7 +1071,7 @@ declare class AgentOrchestrator {
     private processToolApproval;
     send(content: UserMessagePayload, opts?: SendMessageOptions): Promise<MessageResponse>;
     private waitForToolApproval;
-    handleToolApproval(approvalId: string, decision: ToolApprovalDecision, approvedCalls?: ToolCall[]): void;
+    handleToolApproval(approvalId: string, decision: ToolApprovalDecision, approvedCalls?: ToolCall[], editInstruction?: string): void;
     private toInvocations;
 }
 
@@ -1532,14 +1569,10 @@ type BashSuccessResult$1 = {
     status: 'success';
     type: 'text';
     result: string;
-    metadata?: CommandMetadata & {
-        stdout?: string;
-        stderr?: string;
-        stripped?: boolean;
-    };
+    metadata?: BashToolMetadata;
 };
 type BashErrorResult = ExecResultError & {
-    metadata?: CommandMetadata & {
+    metadata?: BashToolMetadata & {
         errorReason?: ErrorReason;
     };
 };
@@ -1643,11 +1676,7 @@ type FileReadSuccessResult$1 = {
     status: 'success';
     type: 'text';
     result: string;
-    metadata?: FileMetadata & {
-        lineRange?: LineRangeMetadata;
-        encoding?: string;
-        bomStripped?: boolean;
-    };
+    metadata?: FileReadMetadata;
 };
 type FileReadErrorResult = ExecResultError & {
     metadata?: {
@@ -1661,24 +1690,7 @@ type FileEditSuccessResult$1 = {
     status: 'success';
     type: 'text';
     result: string;
-    metadata: FileMetadata & {
-        eol: 'lf' | 'crlf';
-        oldTextLength: number;
-        newTextLength: number;
-        bytesWritten: number;
-        beforeSha: string;
-        afterSha: string;
-        dryRun: boolean;
-        lineNumbers: {
-            oldStartLine: number;
-            oldEndLine: number;
-            newStartLine: number;
-            newEndLine: number;
-            oldLineCount: number;
-            newLineCount: number;
-        };
-        noChange?: boolean;
-    };
+    metadata: FileEditMetadata;
 };
 type FileEditResult = FileEditSuccessResult$1 | ExecResultError;
 
@@ -1690,13 +1702,7 @@ type FileNewSuccessResult$1 = {
     status: 'success';
     type: 'text';
     result: string;
-    metadata: {
-        file_path: string;
-        bytes: number;
-        lines: number;
-        created: string;
-        overwritten?: boolean;
-    };
+    metadata: FileNewMetadata;
 };
 type FileNewResult = FileNewSuccessResult$1 | ExecResultError;
 
@@ -1721,10 +1727,7 @@ type DirLsSuccessResult$1 = {
         truncated: boolean;
         total: number;
     };
-    metadata?: {
-        limit: number;
-        includeHidden: boolean;
-    };
+    metadata?: DirLsMetadata;
 };
 type DirLsResult = DirLsSuccessResult$1 | ExecResultError;
 
@@ -1775,11 +1778,7 @@ type WebSearchSuccessResult$1 = {
             region?: string;
         };
     };
-    metadata?: {
-        offset: number;
-        totalRequested: number;
-        hydrated: boolean;
-    };
+    metadata?: WebSearchMetadata;
 };
 type WebSearchToolResult = WebSearchSuccessResult$1 | ExecResultError;
 
@@ -1790,14 +1789,7 @@ type WebFetchSuccessResult$1 = {
     status: 'success';
     type: 'text';
     result: string;
-    metadata: {
-        url: string;
-        contentType: string;
-        statusCode: number;
-        format: 'markdown' | 'json' | 'text';
-        size: number;
-        fetchedAt: string;
-    };
+    metadata: WebFetchMetadata;
 };
 type WebFetchResult = WebFetchSuccessResult$1 | ExecResultError;
 
@@ -1805,18 +1797,7 @@ type TodoWriteSuccessResult$1 = {
     status: 'success';
     type: 'text';
     result: string;
-    metadata: {
-        recentChanges: boolean;
-        stats: {
-            total: number;
-            pending: number;
-            inProgress: number;
-            completed: number;
-        };
-        progress: string;
-        allCompleted: boolean;
-        conversationId: string;
-    };
+    metadata: TodoWriteMetadata;
 };
 type TodoWriteResult = TodoWriteSuccessResult$1 | ExecResultError;
 
@@ -1824,7 +1805,7 @@ type AssignSuccessResult$1 = {
     status: 'success';
     type: 'text';
     result: string;
-    metadata: DelegationMetadata;
+    metadata: AssignTaskMetadata;
 };
 type AssignErrorResult = ExecResultError & {
     metadata?: {

package/dist/index.js

@@ -2,6 +2,7 @@
 var ErrorReason = /* @__PURE__ */ ((ErrorReason2) => {
   ErrorReason2["Aborted"] = "aborted";
   ErrorReason2["Denied"] = "denied";
+  ErrorReason2["Edited"] = "edited";
   ErrorReason2["Timeout"] = "timeout";
   ErrorReason2["NotFound"] = "not_found";
   ErrorReason2["PermissionDenied"] = "permission_denied";
@@ -734,7 +735,8 @@ function convertToolCall(toolCall, options = {}) {
     invocation: {
       id: toolCall.id,
       name: toolCall.function.name,
-      parameters
+      parameters,
+      editInstruction: toolCall.editInstruction
     }
   };
 }
@@ -752,7 +754,8 @@ function convertToolCalls(toolCalls, options = {}) {
       invocations.push({
         id: result.callId,
         name: result.toolName,
-        parameters: {}
+        parameters: {},
+        editInstruction: tc.editInstruction
       });
     } else {
       invocations.push(result.invocation);
@@ -998,8 +1001,16 @@ var AgentOrchestrator = class {
       return { approvedCalls: callsToAutoApprove, wasDenied: false };
     }
     try {
-      const manuallyApproved = await this.waitForToolApproval(callsNeedingApproval, conversationId, messageId);
-      return { approvedCalls: [...manuallyApproved, ...callsToAutoApprove], wasDenied: false };
+      const result = await this.waitForToolApproval(callsNeedingApproval, conversationId, messageId);
+      if ("editInstruction" in result) {
+        const editInstruction = result.editInstruction;
+        const editedCalls = toolCalls.map((call) => ({
+          ...call,
+          editInstruction
+        }));
+        return { approvedCalls: editedCalls, wasDenied: false };
+      }
+      return { approvedCalls: [...result, ...callsToAutoApprove], wasDenied: false };
     } catch (err2) {
       const errorMsg = err2 instanceof Error ? err2.message : "Tool approval failed";
       const denialMessage = `Tool execution was not approved: ${errorMsg}`;
@@ -1372,7 +1383,7 @@ var AgentOrchestrator = class {
       });
     });
   }
-  handleToolApproval(approvalId, decision, approvedCalls) {
+  handleToolApproval(approvalId, decision, approvedCalls, editInstruction) {
     const approval = this.pendingApprovals.get(approvalId);
     if (!approval) {
       console.warn(`[Orchestrator] Received approval for unknown or already processed ID: ${approvalId}`);
@@ -1381,6 +1392,8 @@ var AgentOrchestrator = class {
     this.pendingApprovals.delete(approvalId);
     if (decision === "deny") {
       approval.reject(new Error("Tool execution denied by user"));
+    } else if (decision === "edit" && editInstruction) {
+      approval.resolve({ editInstruction });
     } else if (decision === "approve_all" || decision === "approve") {
       approval.resolve(approvedCalls || []);
     } else {
@@ -2116,7 +2129,15 @@ Your todo list has changed. DO NOT mention this explicitly to the user. Latest c
       stats,
       progress: `${progress}%`,
       allCompleted,
-      conversationId
+      conversationId,
+      items: items.map((item) => ({
+        id: item.id,
+        content: item.content,
+        status: item.status,
+        priority: item.priority,
+        createdAt: item.createdAt || nowIso
+      })),
+      todosWritten: items.length
     });
   }
 };
@@ -4223,6 +4244,22 @@ var ToolRegistry = class {
               durationMs: 0
             };
           }
+          if (c.editInstruction) {
+            const editResult = `${c.editInstruction}
+<system-reminder>
+This is not a result from the tool call. The user wants something else. Please follow the user's instruction.
+DO NOT mention this explicitly to the user.
+</system-reminder>`;
+            return {
+              id: c.id,
+              name: c.name,
+              status: "error",
+              type: "text",
+              result: editResult,
+              metadata: { errorReason: "edited" /* Edited */, editInstruction: c.editInstruction },
+              durationMs: 0
+            };
+          }
           const startTime = performance.now();
           const impl = this.tools.get(c.name);
           if (!impl) {
@@ -6876,6 +6913,22 @@ var MCPToolPort = class {
               result: "Tool execution aborted by user"
             };
           }
+          if (c.editInstruction) {
+            const editResult = `${c.editInstruction}
+<system-reminder>
+This is not a result from the tool call. The user wants something else. Please follow the user's instruction.
+DO NOT mention this explicitly to the user.
+</system-reminder>`;
+            return {
+              id: c.id,
+              name: c.name,
+              status: "error",
+              type: "text",
+              result: editResult,
+              metadata: { errorReason: "edited" /* Edited */, editInstruction: c.editInstruction },
+              durationMs: 0
+            };
+          }
           const original = this.map.get(c.name);
           if (!original) {
             return { id: c.id, name: c.name, status: "error", type: "text", result: `Unknown MCP tool: ${c.name}` };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nuvin/nuvin-core",
-  "version": "1.8.0",
+  "version": "1.9.1",
   "description": "",
   "private": false,
   "main": "dist/index.js",