Fix CI: prettier formatting, ruff lint, unused Go function

- Run prettier on Node.js files (client.ts, session.ts, test)
- Add 'from __future__ import annotations' to Python session.py and
  remove quoted forward references (ruff UP037)
- Remove unused trackShellProcess function in Go (golangci-lint)

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Author: 2 people

go/session.go

@@ -619,16 +619,6 @@ func (s *Session) dispatchShellExit(notification ShellExitNotification) {
 	}
 }
 
-// trackShellProcess tracks a shell process ID so notifications are routed to this session.
-func (s *Session) trackShellProcess(processID string) {
-	s.trackedProcessMux.Lock()
-	s.trackedProcessIDs[processID] = struct{}{}
-	s.trackedProcessMux.Unlock()
-	if s.registerShellProc != nil {
-		s.registerShellProc(processID, s)
-	}
-}
-
 // untrackShellProcess stops tracking a shell process ID.
 func (s *Session) untrackShellProcess(processID string) {
 	s.trackedProcessMux.Lock()

nodejs/src/client.ts

@@ -562,7 +562,7 @@ export class CopilotClient {
         const session = new CopilotSession(sessionId, this.connection!);
         session._setShellProcessCallbacks(
             (processId, session) => this.shellProcessMap.set(processId, session),
-            (processId) => this.shellProcessMap.delete(processId),
+            (processId) => this.shellProcessMap.delete(processId)
         );
         session.registerTools(config.tools);
         session.registerPermissionHandler(config.onPermissionRequest);
@@ -666,7 +666,7 @@ export class CopilotClient {
         const session = new CopilotSession(sessionId, this.connection!);
         session._setShellProcessCallbacks(
             (processId, session) => this.shellProcessMap.set(processId, session),
-            (processId) => this.shellProcessMap.delete(processId),
+            (processId) => this.shellProcessMap.delete(processId)
         );
         session.registerTools(config.tools);
         session.registerPermissionHandler(config.onPermissionRequest);

nodejs/src/session.ts

@@ -421,7 +421,7 @@ export class CopilotSession {
      */
     _setShellProcessCallbacks(
         register: (processId: string, session: CopilotSession) => void,
-        unregister: (processId: string) => void,
+        unregister: (processId: string) => void
     ): void {
         this._registerShellProcess = register;
         this._unregisterShellProcess = unregister;

nodejs/test/session-shell.test.ts

@@ -136,7 +136,7 @@ describe("CopilotSession shell notifications", () => {
 
             session._setShellProcessCallbacks(
                 (processId, s) => registered.set(processId, s),
-                (processId) => registered.delete(processId),
+                (processId) => registered.delete(processId)
             );
 
             session._trackShellProcess("proc-1");

python/copilot/session.py

@@ -5,6 +5,8 @@
 conversation sessions with the Copilot CLI.
 """
 
+from __future__ import annotations
+
 import asyncio
 import inspect
 import threading
@@ -106,7 +108,7 @@ def __init__(self, session_id: str, client: Any, workspace_path: str | None = No
         self._shell_exit_handlers_lock = threading.Lock()
         self._tracked_process_ids: set[str] = set()
         self._tracked_process_ids_lock = threading.Lock()
-        self._register_shell_process: Callable[[str, "CopilotSession"], None] | None = None
+        self._register_shell_process: Callable[[str, CopilotSession], None] | None = None
         self._unregister_shell_process_fn: Callable[[str], None] | None = None
         self._rpc: SessionRpc | None = None
 
@@ -348,7 +350,7 @@ def _untrack_shell_process(self, process_id: str) -> None:
 
     def _set_shell_process_callbacks(
         self,
-        register: Callable[[str, "CopilotSession"], None],
+        register: Callable[[str, CopilotSession], None],
         unregister: Callable[[str], None],
     ) -> None:
         """Set the registration callbacks for shell process tracking.
@@ -810,7 +812,7 @@ async def destroy(self) -> None:
         )
         await self.disconnect()
 
-    async def __aenter__(self) -> "CopilotSession":
+    async def __aenter__(self) -> CopilotSession:
         """Enable use as an async context manager."""
         return self