Formatting

Author: SteveSandersonMS

nodejs/src/client.ts

@@ -511,7 +511,7 @@ export class CopilotClient {
     async createSession(config: SessionConfig): Promise<CopilotSession> {
         if (!config?.onPermissionRequest) {
             throw new Error(
-                'An onPermissionRequest handler is required when creating a session. For example, to allow all permissions, use { onPermissionRequest: approveAll }.'
+                "An onPermissionRequest handler is required when creating a session. For example, to allow all permissions, use { onPermissionRequest: approveAll }."
             );
         }
 
@@ -594,13 +594,10 @@ export class CopilotClient {
      * });
      * ```
      */
-    async resumeSession(
-        sessionId: string,
-        config: ResumeSessionConfig
-    ): Promise<CopilotSession> {
+    async resumeSession(sessionId: string, config: ResumeSessionConfig): Promise<CopilotSession> {
         if (!config?.onPermissionRequest) {
             throw new Error(
-                'An onPermissionRequest handler is required when resuming a session. For example, to allow all permissions, use { onPermissionRequest: approveAll }.'
+                "An onPermissionRequest handler is required when resuming a session. For example, to allow all permissions, use { onPermissionRequest: approveAll }."
             );
         }
 

nodejs/test/e2e/rpc.test.ts

@@ -71,7 +71,10 @@ describe("Session RPC", async () => {
 
     // session.model.getCurrent is defined in schema but not yet implemented in CLI
     it.skip("should call session.rpc.model.getCurrent", async () => {
-        const session = await client.createSession({ onPermissionRequest: approveAll, model: "claude-sonnet-4.5" });
+        const session = await client.createSession({
+            onPermissionRequest: approveAll,
+            model: "claude-sonnet-4.5",
+        });
 
         const result = await session.rpc.model.getCurrent();
         expect(result.modelId).toBeDefined();
@@ -80,7 +83,10 @@ describe("Session RPC", async () => {
 
     // session.model.switchTo is defined in schema but not yet implemented in CLI
     it.skip("should call session.rpc.model.switchTo", async () => {
-        const session = await client.createSession({ onPermissionRequest: approveAll, model: "claude-sonnet-4.5" });
+        const session = await client.createSession({
+            onPermissionRequest: approveAll,
+            model: "claude-sonnet-4.5",
+        });
 
         // Get initial model
         const before = await session.rpc.model.getCurrent();

nodejs/test/e2e/session.test.ts

@@ -8,7 +8,10 @@ describe("Sessions", async () => {
     const { copilotClient: client, openAiEndpoint, homeDir, env } = await createSdkTestContext();
 
     it("should create and destroy sessions", async () => {
-        const session = await client.createSession({ onPermissionRequest: approveAll, model: "fake-test-model" });
+        const session = await client.createSession({
+            onPermissionRequest: approveAll,
+            model: "fake-test-model",
+        });
         expect(session.sessionId).toMatch(/^[a-f0-9-]+$/);
 
         expect(await session.getMessages()).toMatchObject([
@@ -187,7 +190,9 @@ describe("Sessions", async () => {
         });
 
         onTestFinished(() => newClient.forceStop());
-        const session2 = await newClient.resumeSession(sessionId, { onPermissionRequest: approveAll });
+        const session2 = await newClient.resumeSession(sessionId, {
+            onPermissionRequest: approveAll,
+        });
         expect(session2.sessionId).toBe(sessionId);
 
         // TODO: There's an inconsistency here. When resuming with a new client, we don't see
@@ -199,7 +204,9 @@ describe("Sessions", async () => {
     });
 
     it("should throw error when resuming non-existent session", async () => {
-        await expect(client.resumeSession("non-existent-session-id", { onPermissionRequest: approveAll })).rejects.toThrow();
+        await expect(
+            client.resumeSession("non-existent-session-id", { onPermissionRequest: approveAll })
+        ).rejects.toThrow();
     });
 
     it("should create session with custom tool", async () => {

python/copilot/client.py

@@ -453,7 +453,8 @@ async def create_session(self, config: SessionConfig) -> CopilotSession:
         if not cfg.get("on_permission_request"):
             raise ValueError(
                 "An on_permission_request handler is required when creating a session. "
-                'For example, to allow all permissions, use {"on_permission_request": PermissionHandler.approve_all}.'
+                "For example, to allow all permissions, use "
+                '{"on_permission_request": PermissionHandler.approve_all}.'
             )
 
         tool_defs = []
@@ -623,7 +624,8 @@ async def resume_session(self, session_id: str, config: ResumeSessionConfig) ->
         if not cfg.get("on_permission_request"):
             raise ValueError(
                 "An on_permission_request handler is required when resuming a session. "
-                'For example, to allow all permissions, use {"on_permission_request": PermissionHandler.approve_all}.'
+                "For example, to allow all permissions, use "
+                '{"on_permission_request": PermissionHandler.approve_all}.'
             )
 
         tool_defs = []