From 9eacb3bd7c2666811959d6e5cddcca68c4b01bb0 Mon Sep 17 00:00:00 2001 From: Sev7eNup <79143581+Sev7eNup@users.noreply.github.com> Date: Thu, 5 Nov 2026 22:17:39 +0100 Subject: [PATCH] Hold msw 3 until the vitest family moves msw 3 needs vitest 5 (the @vitest/mocker peer range of vitest 4 stops at msw 2), and it matches requests to handlers asynchronously. With vitest 5, WorkflowEditorPage passes on msw 2 and fails two or three tests per run on msw 3, so this is a migration and not a bump. Co-Authored-By: Claude Opus 5.5 --- .github/dependabot.yml | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/.github/dependabot.yml b/.github/dependabot.yml index c5aab1a7..948c4081 100644 --- a/.github/dependabot.yml +++ b/.github/dependabot.yml @@ -74,6 +74,12 @@ updates: # on its own, but in the full src/nodepilot-ui run it exposes cross-file pollution that # fails ExecutionsPage, which passes in isolation and on vitest 4. That is a migration, # not a bump. Lift both entries together when someone takes it on. + # + # msw 3 needs vitest 5, because the @vitest/mocker peer range of vitest 4 stops at msw 2. + # It also matches requests to handlers asynchronously, so tests that act on the first + # render or end with a request in flight see state from the wrong moment. With vitest 5, + # WorkflowEditorPage passes on msw 2 and fails two or three tests per run on msw 3. + # Move it after the vitest family. ignore: - dependency-name: "typescript" versions: [">=7.0.0"] @@ -83,6 +89,8 @@ updates: versions: [">=5.0.0"] - dependency-name: "@vitest/*" versions: [">=5.0.0"] + - dependency-name: "msw" + versions: [">=3.0.0"] groups: npm-minor-patch: patterns: