Skip to content

Require permission handler on session creation #331

Require permission handler on session creation

Require permission handler on session creation #331

Triggered via pull request February 24, 2026 09:53
Status Failure
Total duration 2m 1s
Artifacts

nodejs-sdk-tests.yml

on: pull_request
Matrix: Node.js SDK Tests
Fit to window
Zoom out
Zoom in

Annotations

21 errors
Node.js SDK Tests (ubuntu-latest)
Process completed with exit code 1.
test/e2e/mcp_and_agents.test.ts > MCP Servers and Custom Agents > MCP Servers > should accept MCP server configuration on session create: nodejs/src/client.ts#L513
Error: An onPermissionRequest handler is required when creating a session. For example, to allow all permissions, use { onPermissionRequest: approveAll }. ❯ CopilotClient.createSession src/client.ts:513:19 ❯ test/e2e/mcp_and_agents.test.ts:30:42
test/e2e/hooks.test.ts > Session hooks > should deny tool execution when preToolUse returns deny: nodejs/src/client.ts#L513
Error: An onPermissionRequest handler is required when creating a session. For example, to allow all permissions, use { onPermissionRequest: approveAll }. ❯ CopilotClient.createSession src/client.ts:513:19 ❯ test/e2e/hooks.test.ts:122:38
test/e2e/compaction.test.ts > Compaction > should not emit compaction events when infinite sessions disabled: nodejs/src/client.ts#L513
Error: An onPermissionRequest handler is required when creating a session. For example, to allow all permissions, use { onPermissionRequest: approveAll }. ❯ CopilotClient.createSession src/client.ts:513:19 ❯ test/e2e/compaction.test.ts:65:38
test/e2e/compaction.test.ts > Compaction > should trigger compaction with low threshold and emit events: nodejs/src/client.ts#L513
Error: An onPermissionRequest handler is required when creating a session. For example, to allow all permissions, use { onPermissionRequest: approveAll }. ❯ CopilotClient.createSession src/client.ts:513:19 ❯ test/e2e/compaction.test.ts:10:38
test/e2e/client.test.ts > Client > should report error with stderr when CLI fails to start: nodejs/test/e2e/client.test.ts#L159
AssertionError: expected 'An onPermissionRequest handler is req…' to contain 'Connection is closed' Expected: "Connection is closed" Received: "An onPermissionRequest handler is required when creating a session. For example, to allow all permissions, use { onPermissionRequest: approveAll }." ❯ test/e2e/client.test.ts:159:46
test/e2e/client.test.ts > Client > should forceStop without cleanup: nodejs/src/client.ts#L513
Error: An onPermissionRequest handler is required when creating a session. For example, to allow all permissions, use { onPermissionRequest: approveAll }. ❯ CopilotClient.createSession src/client.ts:513:19 ❯ test/e2e/client.test.ts:72:22
test/e2e/client.test.ts > Client > should return errors on failed cleanup: nodejs/src/client.ts#L513
Error: An onPermissionRequest handler is required when creating a session. For example, to allow all permissions, use { onPermissionRequest: approveAll }. ❯ CopilotClient.createSession src/client.ts:513:19 ❯ test/e2e/client.test.ts:54:22
test/e2e/ask_user.test.ts > User input (ask_user) > should handle freeform user input response: nodejs/src/client.ts#L513
Error: An onPermissionRequest handler is required when creating a session. For example, to allow all permissions, use { onPermissionRequest: approveAll }. ❯ CopilotClient.createSession src/client.ts:513:19 ❯ test/e2e/ask_user.test.ts:76:38
test/e2e/ask_user.test.ts > User input (ask_user) > should receive choices in user input request: nodejs/src/client.ts#L513
Error: An onPermissionRequest handler is required when creating a session. For example, to allow all permissions, use { onPermissionRequest: approveAll }. ❯ CopilotClient.createSession src/client.ts:513:19 ❯ test/e2e/ask_user.test.ts:45:38
test/e2e/ask_user.test.ts > User input (ask_user) > should invoke user input handler when model uses ask_user tool: nodejs/src/client.ts#L513
Error: An onPermissionRequest handler is required when creating a session. For example, to allow all permissions, use { onPermissionRequest: approveAll }. ❯ CopilotClient.createSession src/client.ts:513:19 ❯ test/e2e/ask_user.test.ts:15:38
test/e2e/mcp_and_agents.test.ts > MCP Servers and Custom Agents > MCP Servers > should accept MCP server configuration on session resume: nodejs/src/client.ts#L513
Error: An onPermissionRequest handler is required when creating a session. For example, to allow all permissions, use { onPermissionRequest: approveAll }. ❯ CopilotClient.createSession src/client.ts:513:19 ❯ test/e2e/mcp_and_agents.test.ts:47:43
test/e2e/mcp_and_agents.test.ts > MCP Servers and Custom Agents > MCP Servers > should accept MCP server configuration on session create: nodejs/src/client.ts#L513
Error: An onPermissionRequest handler is required when creating a session. For example, to allow all permissions, use { onPermissionRequest: approveAll }. ❯ CopilotClient.createSession src/client.ts:513:19 ❯ test/e2e/mcp_and_agents.test.ts:30:42
test/e2e/hooks.test.ts > Session hooks > should deny tool execution when preToolUse returns deny: nodejs/src/client.ts#L513
Error: An onPermissionRequest handler is required when creating a session. For example, to allow all permissions, use { onPermissionRequest: approveAll }. ❯ CopilotClient.createSession src/client.ts:513:19 ❯ test/e2e/hooks.test.ts:122:38
test/e2e/compaction.test.ts > Compaction > should not emit compaction events when infinite sessions disabled: nodejs/src/client.ts#L513
Error: An onPermissionRequest handler is required when creating a session. For example, to allow all permissions, use { onPermissionRequest: approveAll }. ❯ CopilotClient.createSession src/client.ts:513:19 ❯ test/e2e/compaction.test.ts:65:38
test/e2e/compaction.test.ts > Compaction > should trigger compaction with low threshold and emit events: nodejs/src/client.ts#L513
Error: An onPermissionRequest handler is required when creating a session. For example, to allow all permissions, use { onPermissionRequest: approveAll }. ❯ CopilotClient.createSession src/client.ts:513:19 ❯ test/e2e/compaction.test.ts:10:38
test/e2e/client.test.ts > Client > should report error with stderr when CLI fails to start: nodejs/test/e2e/client.test.ts#L159
AssertionError: expected 'An onPermissionRequest handler is req…' to contain 'Connection is closed' Expected: "Connection is closed" Received: "An onPermissionRequest handler is required when creating a session. For example, to allow all permissions, use { onPermissionRequest: approveAll }." ❯ test/e2e/client.test.ts:159:46
test/e2e/client.test.ts > Client > should forceStop without cleanup: nodejs/src/client.ts#L513
Error: An onPermissionRequest handler is required when creating a session. For example, to allow all permissions, use { onPermissionRequest: approveAll }. ❯ CopilotClient.createSession src/client.ts:513:19 ❯ test/e2e/client.test.ts:72:22
test/e2e/ask_user.test.ts > User input (ask_user) > should handle freeform user input response: nodejs/src/client.ts#L513
Error: An onPermissionRequest handler is required when creating a session. For example, to allow all permissions, use { onPermissionRequest: approveAll }. ❯ CopilotClient.createSession src/client.ts:513:19 ❯ test/e2e/ask_user.test.ts:76:38
test/e2e/ask_user.test.ts > User input (ask_user) > should receive choices in user input request: nodejs/src/client.ts#L513
Error: An onPermissionRequest handler is required when creating a session. For example, to allow all permissions, use { onPermissionRequest: approveAll }. ❯ CopilotClient.createSession src/client.ts:513:19 ❯ test/e2e/ask_user.test.ts:45:38
test/e2e/ask_user.test.ts > User input (ask_user) > should invoke user input handler when model uses ask_user tool: nodejs/src/client.ts#L513
Error: An onPermissionRequest handler is required when creating a session. For example, to allow all permissions, use { onPermissionRequest: approveAll }. ❯ CopilotClient.createSession src/client.ts:513:19 ❯ test/e2e/ask_user.test.ts:15:38