fix(test): ensure fake timers cleanup in manifest timeout test

Wrap the initToolTypesManifest timeout test in try/finally so
vi.useRealTimers() runs even when assertions fail. Also ruff-format
test_tool_dispatch_sync.py manifest validation helper (CI gate).

Author: clean6378-max-it

static/js/render/tool_types_manifest.test.js

@@ -49,26 +49,29 @@ describe('initToolTypesManifest', () => {
 
     it('warns when fetch times out', async () => {
         vi.useFakeTimers();
-        const warn = vi.spyOn(console, 'warn').mockImplementation(() => {});
-        vi.stubGlobal(
-            'fetch',
-            vi.fn((_url, init) =>
-                new Promise((_resolve, reject) => {
-                    init?.signal?.addEventListener('abort', () => {
-                        reject(Object.assign(new Error('aborted'), { name: 'AbortError' }));
-                    });
-                }),
-            ),
-        );
+        try {
+            const warn = vi.spyOn(console, 'warn').mockImplementation(() => {});
+            vi.stubGlobal(
+                'fetch',
+                vi.fn((_url, init) =>
+                    new Promise((_resolve, reject) => {
+                        init?.signal?.addEventListener('abort', () => {
+                            reject(Object.assign(new Error('aborted'), { name: 'AbortError' }));
+                        });
+                    }),
+                ),
+            );
 
-        const promise = initToolTypesManifest();
-        await vi.advanceTimersByTimeAsync(5000);
-        await promise;
+            const promise = initToolTypesManifest();
+            await vi.advanceTimersByTimeAsync(5000);
+            await promise;
 
-        expect(getManifestToolTypes()).toBeNull();
-        expect(warn).toHaveBeenCalledWith(
-            '[tool registry] Could not load /static/tool_types.json: timed out after 5000ms',
-        );
-        vi.useRealTimers();
+            expect(getManifestToolTypes()).toBeNull();
+            expect(warn).toHaveBeenCalledWith(
+                '[tool registry] Could not load /static/tool_types.json: timed out after 5000ms',
+            );
+        } finally {
+            vi.useRealTimers();
+        }
     });
 });

tests/test_tool_dispatch_sync.py

@@ -96,9 +96,7 @@ def _load_manifest_tool_types(path: Path) -> frozenset[str]:
         raise ValueError(msg)
     for i, item in enumerate(raw):
         if not isinstance(item, str):
-            msg = (
-                f"Invalid tool_types[{i}] in {path}: expected string, got {type(item).__name__}"
-            )
+            msg = f"Invalid tool_types[{i}] in {path}: expected string, got {type(item).__name__}"
             raise ValueError(msg)
     return frozenset(raw)