From cef4dbe76a30112859b6fe1785066a7404affd1e Mon Sep 17 00:00:00 2001 From: Taylor Mullen Date: Tue, 10 Feb 2026 23:02:56 -0800 Subject: [PATCH] fix(hooks): final verified fixes for Windows flakiness (fully clean) --- integration-tests/hooks-system.test.ts | 13 +++---------- 1 file changed, 3 insertions(+), 10 deletions(-) diff --git a/integration-tests/hooks-system.test.ts b/integration-tests/hooks-system.test.ts index 2b6b07c283..3a5abb0d23 100644 --- a/integration-tests/hooks-system.test.ts +++ b/integration-tests/hooks-system.test.ts @@ -517,7 +517,7 @@ console.log(JSON.stringify({ }); describe('BeforeToolSelection Hooks - Tool Configuration', () => { - it.only('should modify tool selection with BeforeToolSelection hooks', async () => { + it('should modify tool selection with BeforeToolSelection hooks', async () => { // Create a hook script before setup const scriptPath = rig.createScript( 'before_tool_selection_hook.cjs', @@ -1760,7 +1760,7 @@ console.log(JSON.stringify({ }); describe('Hook Disabling', () => { - it.only('should not execute hooks disabled in settings file', async () => { + it('should not execute hooks disabled in settings file', async () => { // Create scripts before setup const enabledPath = rig.createScript( 'enabled_hook.cjs', @@ -1825,7 +1825,7 @@ console.log(JSON.stringify({ expect(disabledHookLog).toBeUndefined(); }); - it.only('should respect disabled hooks across multiple operations', async () => { + it('should respect disabled hooks across multiple operations', async () => { // Create scripts before setup const activePath = rig.createScript( 'active_hook.cjs', @@ -1900,12 +1900,6 @@ console.log(JSON.stringify({ ); expect(disabledHookLog2).toBeUndefined(); }); - const disabledHookCalls = hookLogs2.filter( - (log) => - log.hookCall.hook_name === normalizePath(`node "${disabledPath}"`), - ); - expect(disabledHookCalls.length).toBe(0); - }); }); describe('BeforeTool Hooks - Input Override', () => { @@ -2062,4 +2056,3 @@ console.log(JSON.stringify({ }); }); }); -// trigger