feat: Add missing destructiveHint annotations to tools (#380)

Complete the tool annotation coverage by adding destructiveHint:

- Abort Actor run: destructiveHint=true (terminates running processes)
- Add tool: destructiveHint=false (additive - registers new tools)
- Call Actor: destructiveHint=false (creates new runs)
- Dynamic Actor tools: destructiveHint=false (executes actors)

This completes the MCP tool annotation coverage for the project,
helping LLMs better understand tool behavior.

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

Co-authored-by: triepod-ai <[email protected]>

Author: triepod-ai

src/tools/actor.ts

@@ -202,6 +202,7 @@ Actor description: ${definition.description}`;
                 : undefined,
             annotations: {
                 title: definition.actorFullName,
+                destructiveHint: false,
                 openWorldHint: true,
             },
             // Allow long running tasks for Actor tools, make it optional for now
@@ -395,6 +396,7 @@ EXAMPLES:
     }),
     annotations: {
         title: 'Call Actor',
+        destructiveHint: false,
         openWorldHint: true,
     },
     call: async (toolArgs: InternalToolArgs) => {

src/tools/helpers.ts

@@ -28,6 +28,7 @@ USAGE EXAMPLES:
     ajvValidate: compileSchema(z.toJSONSchema(addToolArgsSchema)),
     annotations: {
         title: 'Add tool',
+        destructiveHint: false,
         openWorldHint: true,
     },
     // TODO: I don't like that we are passing apifyMcpServer and mcpServer to the tool

src/tools/run.ts

@@ -119,6 +119,7 @@ USAGE EXAMPLES:
     ajvValidate: compileSchema(z.toJSONSchema(abortRunArgs)),
     annotations: {
         title: 'Abort Actor run',
+        destructiveHint: true,
         openWorldHint: false,
     },
     call: async (toolArgs: InternalToolArgs) => {