skip postToolUse hook tests broken by runtime (issue #972)

Author: SteveSandersonMS

dotnet/test/HooksTests.cs

@@ -46,7 +46,8 @@ await session.SendAsync(new MessageOptions
         Assert.Contains(preToolUseInputs, i => !string.IsNullOrEmpty(i.ToolName));
     }
 
-    [Fact]
+    // TODO: Re-enable once runtime postToolUse hooks are fixed (https://github.com/github/copilot-sdk/issues/972)
+    [Fact(Skip = "Runtime postToolUse hooks broken")]
     public async Task Should_Invoke_PostToolUse_Hook_After_Model_Runs_A_Tool()
     {
         var postToolUseInputs = new List<PostToolUseHookInput>();
@@ -83,7 +84,8 @@ await session.SendAsync(new MessageOptions
         Assert.Contains(postToolUseInputs, i => i.ToolResult != null);
     }
 
-    [Fact]
+    // TODO: Re-enable once runtime postToolUse hooks are fixed (https://github.com/github/copilot-sdk/issues/972)
+    [Fact(Skip = "Runtime postToolUse hooks broken")]
     public async Task Should_Invoke_Both_PreToolUse_And_PostToolUse_Hooks_For_Single_Tool_Call()
     {
         var preToolUseInputs = new List<PreToolUseHookInput>();

go/internal/e2e/hooks_test.go

@@ -74,7 +74,9 @@ func TestHooks(t *testing.T) {
 		}
 	})
 
+	// TODO: Re-enable once runtime postToolUse hooks are fixed (https://github.com/github/copilot-sdk/issues/972)
 	t.Run("should invoke postToolUse hook after model runs a tool", func(t *testing.T) {
+		t.Skip("Runtime postToolUse hooks broken")
 		ctx.ConfigureForTest(t)
 
 		var postToolUseInputs []copilot.PostToolUseHookInput
@@ -139,7 +141,9 @@ func TestHooks(t *testing.T) {
 		}
 	})
 
+	// TODO: Re-enable once runtime postToolUse hooks are fixed (https://github.com/github/copilot-sdk/issues/972)
 	t.Run("should invoke both preToolUse and postToolUse hooks for a single tool call", func(t *testing.T) {
+		t.Skip("Runtime postToolUse hooks broken")
 		ctx.ConfigureForTest(t)
 
 		var preToolUseInputs []copilot.PreToolUseHookInput

nodejs/test/e2e/hooks.test.ts

@@ -48,7 +48,8 @@ describe("Session hooks", async () => {
         await session.disconnect();
     });
 
-    it("should invoke postToolUse hook after model runs a tool", async () => {
+    // TODO: Re-enable once runtime postToolUse hooks are fixed (https://github.com/github/copilot-sdk/issues/972)
+    it.skip("should invoke postToolUse hook after model runs a tool", async () => {
         const postToolUseInputs: PostToolUseHookInput[] = [];
 
         const session = await client.createSession({
@@ -79,7 +80,8 @@ describe("Session hooks", async () => {
         await session.disconnect();
     });
 
-    it("should invoke both preToolUse and postToolUse hooks for a single tool call", async () => {
+    // TODO: Re-enable once runtime postToolUse hooks are fixed (https://github.com/github/copilot-sdk/issues/972)
+    it.skip("should invoke both preToolUse and postToolUse hooks for a single tool call", async () => {
         const preToolUseInputs: PreToolUseHookInput[] = [];
         const postToolUseInputs: PostToolUseHookInput[] = [];
 

python/e2e/test_hooks.py

@@ -41,6 +41,8 @@ async def on_pre_tool_use(input_data, invocation):
 
         await session.disconnect()
 
+    # TODO: Re-enable once runtime postToolUse hooks are fixed (https://github.com/github/copilot-sdk/issues/972)
+    @pytest.mark.skip(reason="Runtime postToolUse hooks broken")
     async def test_should_invoke_posttooluse_hook_after_model_runs_a_tool(
         self, ctx: E2ETestContext
     ):
@@ -71,6 +73,8 @@ async def on_post_tool_use(input_data, invocation):
 
         await session.disconnect()
 
+    # TODO: Re-enable once runtime postToolUse hooks are fixed (https://github.com/github/copilot-sdk/issues/972)
+    @pytest.mark.skip(reason="Runtime postToolUse hooks broken")
     async def test_should_invoke_both_pretooluse_and_posttooluse_hooks_for_a_single_tool_call(
         self, ctx: E2ETestContext
     ):