filesystem: bounded-walk 100000-entry test times out on slow Windows runners (vitest's 5 s default)
Les mainteneurs répondent en général sous 1 jour
Évaluation
- Difficulté
- 1/5
- Temps estimé
- Moins d'une heure
- Accessibilité débutants
- 88/100
- Type d'issue
- Bug
- Clarté
- Clairement spécifiée
- Activité
- Active
- Stack technique
- typescript
- Domaine
- testing
Piste de recherche
Lisez src/filesystem/__tests__/bounded-walk.test.ts et trouvez le test search_files nommé « stops with an error after visiting 100000 entries ». Donnez à ce test un délai d’attente explicite et ajoutez un commentaire expliquant le parcours intentionnel de 100,001 entrées. Exécutez le test ciblé et confirmez qu’il réussit avec le délai d’attente plus long.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
src/filesystem/__tests__/bounded-walk.test.ts › search_files › "stops with an error after visiting 100000 entries" (added by #5326 for #4162) runs on vitest's default 5000 ms test timeout, but it walks MAX_WALK_ENTRIES + 1 mocked entries through the real search_files path, so its cost scales with the runner's speed.
Observed on the Windows runner (Test filesystem (Windows)):
| Run | Duration |
|---|---|
| v2/main 858cfff5 | 2638 ms (pass) |
| PR #5317 | 2845 ms (pass) |
| PR #5322 head 7b9cac11 (job) | 5047 ms → Test timed out in 5000ms |
The failure has nothing to do with the PR it failed on. The test passes about 2× under the limit on a normal run, so a slow runner pushes it over.
Expected: the test passes whatever the runner load, since its cost comes from the bound it checks and not from a bug.
Fix: give that one test an explicit timeout (for example it(name, fn, 30_000)) with a comment saying why: it walks 100,001 entries on purpose. Don't add a retry, which AGENTS.md forbids, and don't lower MAX_WALK_ENTRIES for the test, which would stop it testing the real bound. This is the same pattern as #5314, which covers the HFS+ disk-image test.
- Langage dominant
- TypeScript
- Étoiles
- 91.6k
- Forks
- 11.9k
- Merge moyen
- 4 h 38 min
- PR mergées (30 j)
- 241
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Propose un modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de modelcontextprotocol/servers
-
enhancement server-everything v2
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
modelcontextprotocol/servers#5371 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
chore server-filesystem server-memory server-sequentialthinking v2
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
modelcontextprotocol/servers#5345 ·
Les mainteneurs répondent en général sous 1 jour
-
documentation v2
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
modelcontextprotocol/servers#5343 ·
Les mainteneurs répondent en général sous 1 jour
-
chore v2
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
modelcontextprotocol/servers#5339 ·
Les mainteneurs répondent en général sous 1 jour
-
chore v2
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
modelcontextprotocol/servers#5337 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
Toutes les issues de modelcontextprotocol/servers
Issues similaires
-
Difficulté 2/5 1-3 heures Accessibilité débutants 76/100
marmelab/react-admin#11431 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 88/100
db-ux-design-system/core-web#8655 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 74/100
Les mainteneurs répondent en général sous 1 jour
-
bug
Difficulté 2/5 1-3 heures Accessibilité débutants 76/100
stripe/react-stripe-js#728 ·
-
[Upload Grade C dataset] the pop-up changes it's size while uploading the document => Jumping effectOuvertebug
Difficulté 2/5 1-3 heures Accessibilité débutants 66/100
epam/statgpt-admin-frontend#305 ·
Les mainteneurs répondent en général sous 1 jour