[unicoderbot]

Summary

Add validation to the MCP test tool handler to reject check_ignore: true when not using the Dart runner. The --check-ignore flag is only supported by very_good dart test, not very_good test (Flutter).

Problem

The MCP test tool was building command lines without validating runner-specific arguments:

• check_ignore is only registered on very_good dart test
• Passing it to very_good test (Flutter) caused exit 64 before any test runs
• Agents instructed to always set check_ignore could not test Flutter packages

Solution

Validate the check_ignore and dart argument combination in _handleTest before parsing arguments. Return a clear validation error if check_ignore: true is set without dart: true.

Changes

• Add pre-parsing validation in _handleTest to check check_ignore requirements
• Return descriptive StructuredToolError with validation failure type
• Add comprehensive test coverage for all check_ignore combinations

Closes #1779

[github-actions]

📊 Cognitive Complexity Analysis

Net Delta: +2 | Added: 0 | Increased: 1 | Improved: 0 | Removed: 0 | Violations: 0

Status	Declaration	Location	Delta	Score
🟡	VeryGoodMCPServer._handleTest	lib/src/mcp/mcp_server.dart:L486-513	+2	1 -> 3