Add reason and confirm fields to memory deletion tool

Enhances the flag_archival_memory_for_deletion tool with safety guards:
- Added 'reason' field (required, first argument) to force reasoning
- Added 'confirm' boolean field (required, last argument) for explicit confirmation
- Tool only processes deletions when confirm=true
- Deletion handler extracts and logs the reason with each deletion

This pattern ensures the agent must:
1. Explain why it's deleting (reason)
2. Specify what it's deleting (memory_text)
3. Explicitly confirm the deletion (confirm)

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <[e-mail]>

bsky.py

···
                 elif message.tool_call.name == 'flag_archival_memory_for_deletion':
                     try:
                         args = json.loads(message.tool_call.arguments)
+                        reason = args.get('reason', '')
                         memory_text = args.get('memory_text', '')
-                        if memory_text:
-                            flagged_memories.append(memory_text)
-                            logger.debug(f"Found memory flagged for deletion: {memory_text[:50]}...")
+                        confirm = args.get('confirm', False)
+
+                        # Only flag for deletion if confirmed
+                        if confirm and memory_text:
+                            flagged_memories.append({
+                                'reason': reason,
+                                'memory_text': memory_text
+                            })
+                            logger.debug(f"Found memory flagged for deletion (reason: {reason}): {memory_text[:50]}...")
+                        elif not confirm:
+                            logger.debug(f"Memory deletion not confirmed, skipping: {memory_text[:50]}...")
                     except json.JSONDecodeError as e:
                         logger.error(f"Failed to parse flag_archival_memory_for_deletion arguments: {e}")
                 
···
         # Handle archival memory deletion if any were flagged (only if no halt was received)
         if flagged_memories:
             logger.info(f"Processing {len(flagged_memories)} flagged memories for deletion")
-            for memory_text in flagged_memories:
+            for flagged_memory in flagged_memories:
+                reason = flagged_memory['reason']
+                memory_text = flagged_memory['memory_text']
+
                 try:
                     # Search for passages with this exact text
                     logger.debug(f"Searching for passages matching: {memory_text[:100]}...")
···
                                 logger.error(f"Failed to delete passage {passage.id}: {delete_error}")
 
                     if deleted_count > 0:
-                        logger.info(f"🗑️ Deleted {deleted_count} archival memory passage(s): {memory_text[:50]}...")
+                        logger.info(f"🗑️ Deleted {deleted_count} archival memory passage(s) (reason: {reason}): {memory_text[:50]}...")
                     else:
                         logger.warning(f"No exact matches found for deletion: {memory_text[:50]}...")
 

tools/flag_memory_deletion.py

···
 
 
 class FlagArchivalMemoryForDeletionArgs(BaseModel):
+    reason: str = Field(
+        ...,
+        description="The reason why this memory should be deleted"
+    )
     memory_text: str = Field(
         ...,
         description="The exact text content of the archival memory to delete"
     )
+    confirm: bool = Field(
+        ...,
+        description="Confirmation that you want to delete this memory (must be true to proceed)"
+    )
 
 
-def flag_archival_memory_for_deletion(memory_text: str) -> str:
+def flag_archival_memory_for_deletion(reason: str, memory_text: str, confirm: bool) -> str:
     """
     Flag an archival memory for deletion based on its exact text content.
 
···
     The system will search for all archival memories with this exact text and delete them.
 
     Args:
+        reason: The reason why this memory should be deleted
         memory_text: The exact text content of the archival memory to delete
+        confirm: Confirmation that you want to delete this memory (must be true)
 
     Returns:
         Confirmation message
     """
     # This is a dummy tool - it just returns a confirmation
     # The actual deletion will be handled by the bot loop after the agent's turn completes
-    return f"Memory flagged for deletion. Will be removed at the end of this turn if no halt is received."
+    if not confirm:
+        return "Deletion cancelled - confirm must be set to true to delete the memory."
+
+    return f"Memory flagged for deletion (reason: {reason}). Will be removed at the end of this turn if no halt is received."