Hacktoberfest 2026: los issues que los mantenedores marcaron para octubre, abiertos y aptos para principiantes. Explorar issues de Hacktoberfest

fix(mcp): test passes timeout_seconds and check_ignore to runners that reject them

Abierto
  • ¿Algo no está bien?
#1,779 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Los mantenedores suelen responder en 1 día

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
3/5
Tiempo estimado
1-2 días
Aptitud para principiantes
67/100
Tipo de issue
Error
Claridad
Bien especificado
Estado de actividad
Activo
Stack tecnológico
dart, flutter
Área
cli, testing

Línea de trabajo

Start with _parseTest in lib/src/mcp/mcp_server.dart, then read the runner option definitions in lib/src/commands/test/test.dart and lib/src/commands/dart/commands/dart_test_command.dart. Reproduce both argument combinations from the issue and check the MCP schema descriptions. Done when each combination either works on both runners or returns a clear validation failure naming the unsupported argument and runner.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

bug
Description

The MCP test tool builds a command line from its arguments without checking which runner will receive it. Two arguments are accepted by the tool schema but rejected by one of the two runners, so the call fails with exit 64 before any test runs.

Tool arguments Command built Result
dart: true, timeout_seconds: 180 very_good dart test --timeout 180 -- … Could not find an option named "--timeout".
check_ignore: true on a Flutter package very_good test --coverage --min-coverage 100 --check-ignore … Could not find an option named "--check-ignore".

The cause is visible on main (8c772f3):

  • _parseTest in lib/src/mcp/mcp_server.dart appends --check-ignore and --timeout unconditionally, whichever runner dart selects.
  • --timeout is registered only on very_good test (lib/src/commands/test/test.dart).
  • --check-ignore is registered only on very_good dart test (lib/src/commands/dart/commands/dart_test_command.dart).
  • Neither the timeout_seconds nor the check_ignore schema description says it is runner-specific.

This bites agents twice: an agent told to "always set timeout_seconds" cannot test a pure Dart package, and an agent told to pass check_ignore cannot test a Flutter one. The plugin's green-gate skill currently instructs both (VeryGoodOpenSource/vgv-ai-flutter-plugin#178).

Steps To Reproduce
  1. Call the MCP test tool on a pure Dart package with {"dart": true, "timeout_seconds": 60}.
  2. Observe partial_failure / validation: Could not find an option named "--timeout".
  3. Call the MCP test tool on a Flutter package with {"check_ignore": true}.
  4. Observe partial_failure / validation: Could not find an option named "--check-ignore".
Expected Behavior

One of the following, in order of preference:

  1. Each argument works on both runners: very_good dart test gains --timeout (ideally the wall-clock kill from #1765 / #1766) and very_good test gains --check-ignore.
  2. Otherwise, the handler validates combinations before running and returns a validation failure that names the argument and the runner it applies to, and the schema descriptions say which runner each argument supports.
Additional Context

  • Seen in field use in 3 of 310 calls (sessions 2007f494 on 2026-10-07 and 493ade61 on 2026-10-14). Transcript excerpts are available on request.

  • very_good_cli 1.5.0, Dart 3.13.5, Flutter 3.47.6, macOS 26.5 arm64, Claude Code 2.1.302.

  • Related: #1765 (timeout does not kill a hung run), #1705 (MCP surface narrower than the CLI).

  • Tracking issue: VeryGoodOpenSource/vgv-ai-flutter-plugin#180

Lenguaje dominante
Dart
Estrellas
2.4k
Forks
244
Merge medio
1 d 18 h
PR fusionados (30 d)
27

Preparar el entorno

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de VeryGoodOpenSource/very_good_cli

Todos los issues de VeryGoodOpenSource/very_good_cli

Issues similares

Más issues de Dart

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.