Add todo

Author: MRayermannMSFT

dotnet/test/ToolsTests.cs

@@ -195,6 +195,8 @@ static string SafeLookup([Description("Lookup ID")] string id)
                 new Dictionary<string, object?> { ["skip_permission"] = true })
         });
 
+        // TODO: Once the CLI respects skip_permission, use a tracking permission handler
+        // and assert it was NOT called for this tool.
         var session = await CreateSessionAsync(new SessionConfig
         {
             Tools = [tool],

go/internal/e2e/tools_test.go

@@ -277,6 +277,8 @@ func TestTools(t *testing.T) {
 			})
 		safeLookupTool.SkipPermission = true
 
+		// TODO: Once the CLI respects SkipPermission, use a tracking permission handler
+		// and assert it was NOT called for this tool.
 		session, err := client.CreateSession(t.Context(), &copilot.SessionConfig{
 			OnPermissionRequest: copilot.PermissionHandler.ApproveAll,
 			Tools: []copilot.Tool{

nodejs/test/e2e/tools.test.ts

@@ -160,6 +160,8 @@ describe("Custom tools", async () => {
     });
 
     it("skipPermission sent in tool definition", async () => {
+        // TODO: Once the CLI respects skipPermission, use a tracking permission handler
+        // and assert it was NOT called for this tool.
         const session = await client.createSession({
             onPermissionRequest: approveAll,
             tools: [

python/e2e/test_tools.py

@@ -150,6 +150,8 @@ class LookupParams(BaseModel):
         def safe_lookup(params: LookupParams, invocation: ToolInvocation) -> str:
             return f"RESULT: {params.id}"
 
+        # TODO: Once the CLI respects skip_permission, use a tracking permission handler
+        # and assert it was NOT called for this tool.
         session = await ctx.client.create_session(
             {"tools": [safe_lookup], "on_permission_request": PermissionHandler.approve_all}
         )