[Python] Declare various modules as private (#884)

* Declare various modules private
- `telemetry`
- `jsonrpc`
- `sdk_protocol_version`

* Add back in trailing whitespace

---------

Co-authored-by: Steve Sanderson <SteveSandersonMS@users.noreply.github.com>

Author: brettcannon

nodejs/scripts/update-protocol-version.ts

@@ -8,7 +8,7 @@
  * Reads from sdk-protocol-version.json and generates:
  * - nodejs/src/sdkProtocolVersion.ts
  * - go/sdk_protocol_version.go
- * - python/copilot/sdk_protocol_version.py
+ * - python/copilot/_sdk_protocol_version.py
  * - dotnet/src/SdkProtocolVersion.cs
  *
  * Run this script whenever the protocol version changes.
@@ -89,8 +89,8 @@ def get_sdk_protocol_version() -> int:
     """
     return SDK_PROTOCOL_VERSION
 `;
-fs.writeFileSync(path.join(rootDir, "python", "copilot", "sdk_protocol_version.py"), pythonCode);
-console.log("  ✓ python/copilot/sdk_protocol_version.py");
+fs.writeFileSync(path.join(rootDir, "python", "copilot", "_sdk_protocol_version.py"), pythonCode);
+console.log("  ✓ python/copilot/_sdk_protocol_version.py");
 
 // Generate C#
 const csharpCode = `// Code generated by update-protocol-version.ts. DO NOT EDIT.

python/copilot/jsonrpc.py python/copilot/_jsonrpc.pypython/copilot/jsonrpc.py renamed to python/copilot/_jsonrpc.py

@@ -25,12 +25,12 @@
 from pathlib import Path
 from typing import Any, cast, overload
 
+from ._jsonrpc import JsonRpcClient, ProcessExitedError
+from ._sdk_protocol_version import get_sdk_protocol_version
+from ._telemetry import get_trace_context, trace_context
 from .generated.rpc import ServerRpc
 from .generated.session_events import PermissionRequest, session_event_from_dict
-from .jsonrpc import JsonRpcClient, ProcessExitedError
-from .sdk_protocol_version import get_sdk_protocol_version
 from .session import CopilotSession
-from .telemetry import get_trace_context, trace_context
 from .types import (
     ConnectionState,
     CustomAgentConfig,

python/copilot/sdk_protocol_version.py python/copilot/_sdk_protocol_version.pypython/copilot/sdk_protocol_version.py renamed to python/copilot/_sdk_protocol_version.py

@@ -6,7 +6,7 @@
 from typing import TYPE_CHECKING
 
 if TYPE_CHECKING:
-    from ..jsonrpc import JsonRpcClient
+    from .._jsonrpc import JsonRpcClient
 
 
 from dataclasses import dataclass

python/copilot/telemetry.py python/copilot/_telemetry.pypython/copilot/telemetry.py renamed to python/copilot/_telemetry.py

@@ -11,6 +11,8 @@
 from collections.abc import Callable
 from typing import Any, Literal, cast
 
+from ._jsonrpc import JsonRpcError, ProcessExitedError
+from ._telemetry import get_trace_context, trace_context
 from .generated.rpc import (
     Kind,
     Level,
@@ -23,8 +25,6 @@
     SessionToolsHandlePendingToolCallParams,
 )
 from .generated.session_events import SessionEvent, SessionEventType, session_event_from_dict
-from .jsonrpc import JsonRpcError, ProcessExitedError
-from .telemetry import get_trace_context, trace_context
 from .types import (
     Attachment,
     PermissionRequest,

python/copilot/client.py

@@ -13,7 +13,7 @@
 
 import pytest
 
-from copilot.jsonrpc import JsonRpcClient
+from copilot._jsonrpc import JsonRpcClient
 
 
 class MockProcess:

python/copilot/generated/rpc.py

@@ -4,7 +4,7 @@
 
 from unittest.mock import patch
 
-from copilot.telemetry import get_trace_context, trace_context
+from copilot._telemetry import get_trace_context, trace_context
 from copilot.types import SubprocessConfig, TelemetryConfig
 
 

python/copilot/session.py

@@ -10,8 +10,9 @@ import fs from "fs/promises";
 import type { JSONSchema7 } from "json-schema";
 import { FetchingJSONSchemaStore, InputData, JSONSchemaInput, quicktype } from "quicktype-core";
 import {
-    getSessionEventsSchemaPath,
     getApiSchemaPath,
+    getSessionEventsSchemaPath,
+    isRpcMethod,
     postProcessSchema,
     writeGeneratedFile,
     isRpcMethod,
@@ -242,7 +243,7 @@ Generated from: api.schema.json
 from typing import TYPE_CHECKING
 
 if TYPE_CHECKING:
-    from ..jsonrpc import JsonRpcClient
+    from .._jsonrpc import JsonRpcClient
 
 `);
     lines.push(typesCode);