@nuvin/nuvin-core 1.8.0 → 1.9.0

package/dist/VERSION

@@ -1,4 +1,4 @@
 {
-  "version": "1.8.0",
-  "commit": "da28f44"
+  "version": "1.9.0",
+  "commit": "66edafd"
 }

package/dist/index.d.ts

@@ -393,6 +393,7 @@ type TodoWriteMetadata = {
 type AssignTaskMetadata = DelegationMetadata;
 type ToolErrorMetadata = {
     errorReason?: ErrorReason;
+    editInstruction?: string;
     path?: string;
     agentId?: string;
     code?: string | number;
@@ -418,6 +419,7 @@ type ToolCall = {
         name: string;
         arguments: string;
     };
+    editInstruction?: string;
 };
 type TextContentPart = {
     type: 'text';
@@ -586,46 +588,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 +854,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 +881,7 @@ type AgentEvent = {
     approvalId: string;
     decision: ToolApprovalDecision;
     approvedCalls?: ToolCall[];
+    editInstruction?: string;
 } | {
     type: typeof AgentEventTypes.ToolResult;
     conversationId: string;
@@ -1034,7 +1048,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;
 }
 

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 {
@@ -4223,6 +4236,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 +6905,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.0",
   "description": "",
   "private": false,
   "main": "dist/index.js",