[Node] Set requestPermission False If Using Default Permission Handler

Author: MRayermannMSFT

nodejs/src/client.ts

@@ -55,6 +55,7 @@ import type {
     TraceContextProvider,
     TypedSessionLifecycleHandler,
 } from "./types.js";
+import { defaultJoinSessionPermissionHandler } from "./types.js";
 
 /**
  * Minimum protocol version this SDK can communicate with.
@@ -868,7 +869,7 @@ export class CopilotClient {
                 })),
                 provider: config.provider,
                 modelCapabilities: config.modelCapabilities,
-                requestPermission: true,
+                requestPermission: config.onPermissionRequest !== defaultJoinSessionPermissionHandler,
                 requestUserInput: !!config.onUserInputRequest,
                 requestElicitation: !!config.onElicitationRequest,
                 hooks: !!(config.hooks && Object.values(config.hooks).some(Boolean)),

nodejs/src/extension.ts

@@ -4,11 +4,7 @@
 
 import { CopilotClient } from "./client.js";
 import type { CopilotSession } from "./session.js";
-import type { PermissionHandler, PermissionRequestResult, ResumeSessionConfig } from "./types.js";
-
-const defaultJoinSessionPermissionHandler: PermissionHandler = (): PermissionRequestResult => ({
-    kind: "no-result",
-});
+import { defaultJoinSessionPermissionHandler, type PermissionHandler, type PermissionRequestResult, type ResumeSessionConfig } from "./types.js";
 
 export type JoinSessionConfig = Omit<ResumeSessionConfig, "onPermissionRequest"> & {
     onPermissionRequest?: PermissionHandler;

nodejs/src/types.ts

@@ -662,6 +662,10 @@ export type PermissionHandler = (
 
 export const approveAll: PermissionHandler = () => ({ kind: "approved" });
 
+export const defaultJoinSessionPermissionHandler: PermissionHandler = (): PermissionRequestResult => ({
+    kind: "no-result",
+});
+
 // ============================================================================
 // User Input Request Types
 // ============================================================================