v0.4.32 Urgently fix the issue of TODO tool response loss after rejection in non-YOLO mode, as well as the issue of markdown table not being fully displayed.

Author: MayDay-wpf

.github/workflows/publish.yml

@@ -58,21 +58,7 @@ jobs:
             
             ### What's New
 
-            - feat(pdf): Add support for PDF.js Worker files
-
-            - feat(cli): Implement sensitive command detection and confirmation mechanism
-
-            - feat(api): Support sub-agent configuration and custom system prompts
-
-            - feat(task): Implement background task management function
-
-            - feat(codebase): Enhance error handling for file listeners and code formatting
-
-            - feat(chat): Add functionality to restore the last session
-
-            - fix(MarkdownRenderer): Prevent rendering issues caused by empty lines
-
-            - docs(readme): Change SNOW.md to AGENTS.md to match the new project documentation standards
+            - Urgently fix the issue of TODO tool response loss after rejection in non-YOLO mode, as well as the issue of markdown table not being fully displayed.
             
             ### Installation
             ```bash

CHANGELOG.md

@@ -1,5 +1,9 @@
 # Changelog
 
+## v0.4.32
+
+- Urgently fix the issue of TODO tool response loss after rejection in non-YOLO mode, as well as the issue of markdown table not being fully displayed.
+
 ## v0.4.31
 
 - feat(pdf): Add support for PDF.js Worker files

package-lock.json

@@ -1,6 +1,6 @@
 {
 	"name": "snow-ai",
-	"version": "0.4.31",
+	"version": "0.4.32",
 	"description": "Intelligent Command Line Assistant powered by AI",
 	"license": "MIT",
 	"bin": {

package.json

@@ -665,6 +665,7 @@ export async function handleConversationWithTools(
 							// Create UI messages for rejected tools
 							const rejectedToolUIMessages: Message[] = [];
 
+							// Handle sensitive tools that needed confirmation
 							for (const toolCall of sensitiveTools) {
 								const rejectionMessage = {
 									role: 'tool' as const,
@@ -697,6 +698,27 @@ export async function handleConversationWithTools(
 								});
 							}
 
+							// Also handle auto-approved tools (including TODO tools) - they weren't executed either
+							for (const toolCall of [
+								...autoApprovedTools,
+								...nonSensitiveTools,
+							]) {
+								const rejectionMessage = {
+									role: 'tool' as const,
+									tool_call_id: toolCall.id,
+									content: rejectMessage,
+								};
+								conversationMessages.push(rejectionMessage);
+								saveMessage(rejectionMessage).catch(error => {
+									console.error(
+										'Failed to save auto-approved tool rejection message:',
+										error,
+									);
+								});
+
+								// No UI message for auto-approved tools since they were meant to be silent
+							}
+
 							// Add rejected tool messages to UI
 							if (rejectedToolUIMessages.length > 0) {
 								setMessages(prev => [...prev, ...rejectedToolUIMessages]);
@@ -759,6 +781,7 @@ export async function handleConversationWithTools(
 						// Create UI messages for rejected tools
 						const rejectedToolUIMessages: Message[] = [];
 
+						// Handle tools that needed confirmation
 						for (const toolCall of toolsNeedingConfirmation) {
 							const rejectionMessage = {
 								role: 'tool' as const,
@@ -788,6 +811,24 @@ export async function handleConversationWithTools(
 							});
 						}
 
+						// Also handle auto-approved tools (including TODO tools) - they weren't executed either
+						for (const toolCall of autoApprovedTools) {
+							const rejectionMessage = {
+								role: 'tool' as const,
+								tool_call_id: toolCall.id,
+								content: rejectMessage,
+							};
+							conversationMessages.push(rejectionMessage);
+							saveMessage(rejectionMessage).catch(error => {
+								console.error(
+									'Failed to save auto-approved tool rejection message:',
+									error,
+								);
+							});
+
+							// No UI message for auto-approved tools since they were meant to be silent
+						}
+
 						// Add rejected tool messages to UI
 						if (rejectedToolUIMessages.length > 0) {
 							setMessages(prev => [...prev, ...rejectedToolUIMessages]);