Single tool with critical definition gaps. Tool name 'dynamic_makefile_targets' is generic and does not follow verb_noun pattern (should be 'run_makefile_target' or similar). Description is vague: 'Dynamically generated tools from Makefile targets...' reads as implementation detail, not user-facing intent. Input schema present but parameters lack descriptions: 'variables' and 'timeout' have no guidance on format, constraints, or when to use them. No output schema documented. No error handling guidance. Tool appears to be a catch-all wrapper rather than discrete, composable operations. The @internal/@skip filtering mechanism is undocumented in the schema.

• Rename tool to 'run_makefile_target' and add a required 'target_name' parameter to the schema. Currently the tool name suggests it lists targets, but it actually executes them.
• Rewrite description: 'Execute a Makefile target with optional environment variables and execution timeout. Returns stdout, stderr, and exit code. Useful for running build, test, deploy, or cleanup commands defined in the project Makefile.'
• Add parameter descriptions: 'variables' → 'Environment variables to pass to make (e.g., {"DEBUG": "1", "ENV": "prod"}). Merged with system environment.'; 'timeout' → 'Maximum execution time in seconds. Default 300. Recommended max 3600. Raises TimeoutError if exceeded.'
• Document output schema in tool description or as a separate schema field: 'Returns {success: boolean, stdout: string, stderr: string, exit_code: integer, duration_seconds: number, target: string}.'
• Add error handling guidance: 'If exit_code != 0, the target failed. Check stderr for details. Timeouts are retryable; other failures may require user intervention. Targets marked @internal or @skip are not exposed as tools.'

Inferred effective spec: <=2025-11-25.

• Relies on Logging (deprecated) - log to stderr or use OpenTelemetry