stacklok / toolhive-studio / 22092248031

57%

Build Type

push

github

Committed by web-flow

Commit Message

fix: filter deleted MCP servers before restarting on app startup (#1655)

* fix: scope optimistic update to only requested server names

The onMutate handler was injecting all shutdown servers from the store
into the query cache, including ones not being restarted. Filter
groupShutdownServers to only include servers in the mutation variables.

Closes #886

* refactor: move restart-on-startup logic to root route

Extract useRestartShutdownServers hook to common/hooks and call it from
the root route instead of the group route. This ensures the logic runs
once on app startup regardless of which group loads first.

Also validates shutdown servers against the API before restarting, so
servers deleted via CLI while the app was stopped are filtered out.

* test: move and update restart-on-startup tests

Move useMutationRestartServerAtStartup tests to match new hook location.
Add tests for deleted server filtering and shutdown history cleanup.

* feat: just be sure that tha name is valorized

Run Details

2568 of 4787 branches covered (53.65%)

Branch coverage included in aggregate %.

64 of 67 new or added lines in 2 files covered. (95.52%)

2 existing lines in 2 files now uncovered.

4033 of 6976 relevant lines covered (57.81%)

120.48 hits per line