test(mcp): the 2 s protocol-call deadline in the MCP test harness is flaky on slow Windows runners
I maintainer di solito rispondono entro 1 giorno
Una pull request collegata è già stata integrata.
- #405 di @cristim — integrata
Valutazione
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Idoneità per principianti
- 66/100
Direzione di ricerca
Read src/mcp_tests/mod.rs around the deadline arm and inspect how the MCP harness starts protocol calls. Decide how the configured deadline is read once and how the short local default is preserved, then add the requested configuration test. Done means a configured value is honored and genuinely stalled calls still fail within the selected deadline.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
The MCP test harness gives each protocol call a fixed 2 s deadline (src/mcp_tests/mod.rs, the sleep_until(deadline) => panic!("protocol call did not complete") arm). On a slow or loaded runner a correct call can miss it, and the test panics with no assertion failure.
Observed: Check (windows-2025), run 42235641613, on PR #242 head 0840f9d. Two tests in official_client/files.rs panicked at src/mcp_tests/mod.rs:172 with "protocol call did not complete":
open_scene_failure_withholds_directory_and_file_namesnative_sdk_saved_source_changed_is_an_acknowledged_write_and_blocks_discard
The commit under test touched neither file. The same message has also appeared locally under heavy machine load.
Proposed fix: make the deadline configurable, for example through an environment variable read once with a documented default, or longer on CI. Keep a short deadline locally, so a call that really hangs still fails fast. Add a test that the configured value is honoured.
- Lingua principale
- Rust
- Stelle
- 4
- Fork
- 2
- Merge medio
- 1g 1h
- PR unite (30g)
- 225
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Ha un modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di cristim/AztecCraft
-
fix(operations): the NoSelection message says 'before framing' for Delete and pivot resetForse già presa @calmasacow l’ha presa oggi. Apertaarea/ui-parity effort/xs impact/few priority/p3 severity/low triaged type/bug urgency/eventually
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
cristim/AztecCraft#431 · 2 commenti ·
I maintainer di solito rispondono entro 1 giorno
-
area/ui-parity effort/s impact/few priority/p3 severity/low triaged type/bug urgency/eventually
Difficoltà 3/5 1-2 giorni Idoneità per principianti 35/100
cristim/AztecCraft#430 ·
I maintainer di solito rispondono entro 1 giorno
-
refactor(mcp): make the Keys error-code match exhaustive instead of a scene_error wildcardForse già presa @cristim l’ha presa 1 giorno fa. Apertaarea/mcp effort/xs impact/few priority/p3 severity/low triaged type/chore urgency/eventually
Difficoltà 3/5 Mezza giornata Idoneità per principianti 25/100
cristim/AztecCraft#420 ·
I maintainer di solito rispondono entro 1 giorno
-
fix(mcp): range-check NDC arguments on the wire value before narrowing to f32Forse già presa @cristim l’ha presa 1 giorno fa. Apertaarea/mcp effort/s impact/few priority/p3 severity/low triaged type/bug urgency/eventually
Difficoltà 5/5 Più di una settimana Idoneità per principianti 25/100
cristim/AztecCraft#419 ·
I maintainer di solito rispondono entro 1 giorno
-
fix(viewport): use a vertex-relative signed area in polygon_overlaps_rectForse già presa @cristim l’ha presa 1 giorno fa. Apertaarea/selection effort/xs impact/few priority/p3 severity/low triaged type/bug urgency/eventually
Difficoltà 3/5 Mezza giornata Idoneità per principianti 22/100
cristim/AztecCraft#418 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di cristim/AztecCraft
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 88/100
I maintainer di solito rispondono entro 1 giorno
-
ci javascript tests
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
jonathanong/no-mistakes#3258 ·
I maintainer di solito rispondono entro 1 giorno
-
Add to bmi docsAperta
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 72/100
eWaterCycle/bmi-rust#2 ·
-
area: cpu-engines area: fuzzer good first issue
Difficoltà 2/5 1-3 ore Idoneità per principianti 91/100
hffgzktbcc-blip/ounce-core#402 ·
I maintainer di solito rispondono entro 1 giorno
-
compiler
Difficoltà 2/5 1-3 ore Idoneità per principianti 74/100
HerringtonDarkholme/hd-lang#1699 ·
I maintainer di solito rispondono entro 1 giorno