fix: make generated Python package private

Author: omidsaffari

.gitattributes

@@ -6,7 +6,7 @@ java/**/*.java text eol=lf
 # Generated files — keep LF line endings so codegen output is deterministic across platforms.
 nodejs/src/generated/* eol=lf linguist-generated=true
 dotnet/src/Generated/* eol=lf linguist-generated=true
-python/copilot/generated/* eol=lf linguist-generated=true
+python/copilot/_generated/* eol=lf linguist-generated=true
 go/zsession_events.go eol=lf linguist-generated=true
 go/zsession_encoding.go eol=lf linguist-generated=true
 go/rpc/zrpc.go eol=lf linguist-generated=true

.github/workflows/codegen-check.yml

@@ -10,7 +10,7 @@ on:
       - 'scripts/codegen/**'
       - 'nodejs/src/generated/**'
       - 'dotnet/src/Generated/**'
-      - 'python/copilot/generated/**'
+      - 'python/copilot/_generated/**'
       - 'go/generated_*.go'
       - 'go/rpc/**'
       - 'rust/src/generated/**'

python/copilot/__init__.py

@@ -11,6 +11,19 @@
 
 # Register the public ``copilot.session_events`` namespace.
 from . import session_events as session_events  # noqa: F401
+from ._generated.rpc import (
+    CurrentToolMetadata,
+    GitHubTelemetryClientInfo,
+    GitHubTelemetryEvent,
+    GitHubTelemetryNotification,
+    ModelBillingTokenPrices,
+    ModelBillingTokenPricesLongContext,
+)
+from ._generated.session_events import (
+    PermissionRequest,
+    SessionEvent,
+    SessionEventType,
+)
 from ._mode import (
     BUILTIN_TOOLS_ISOLATED,
     CopilotClientMode,
@@ -78,19 +91,6 @@
     CopilotWebSocketHandler,
     LlmInferenceHeaders,
 )
-from .generated.rpc import (
-    CurrentToolMetadata,
-    GitHubTelemetryClientInfo,
-    GitHubTelemetryEvent,
-    GitHubTelemetryNotification,
-    ModelBillingTokenPrices,
-    ModelBillingTokenPricesLongContext,
-)
-from .generated.session_events import (
-    PermissionRequest,
-    SessionEvent,
-    SessionEventType,
-)
 from .session import (
     AgentStopHandler,
     AgentStopHookInput,

python/copilot/generated/__init__.py python/copilot/_generated/__init__.pypython/copilot/generated/__init__.py renamed to python/copilot/_generated/__init__.py

@@ -19,7 +19,7 @@
 from dataclasses import dataclass
 from typing import Any
 
-from .generated.rpc import (
+from ._generated.rpc import (
     CanvasAction,
     CanvasHostContext,
     CanvasHostContextCapabilities,

python/copilot/generated/rpc.py python/copilot/_generated/rpc.pypython/copilot/generated/rpc.py renamed to python/copilot/_generated/rpc.py

@@ -33,6 +33,26 @@
 
 from ._diagnostics import log_timing
 from ._ffi_runtime_host import FfiRuntimeHost
+from ._generated.rpc import (
+    ClientGlobalApiHandlers,
+    ClientSessionApiHandlers,
+    GitHubTelemetryNotification,
+    ModelBillingTokenPrices,
+    ModelBillingTokenPricesLongContext,  # noqa: F401
+    OpenCanvasInstance,
+    RemoteSessionMode,
+    ServerRpc,
+    _ConnectResult,
+    _HookInvokeRequest,
+    _HookInvokeResponse,
+    from_datetime,
+    register_client_global_api_handlers,
+    register_client_session_api_handlers,
+)
+from ._generated.session_events import (
+    SessionEvent,
+    session_event_from_dict,
+)
 from ._jsonrpc import JsonRpcClient, JsonRpcError, ProcessExitedError
 from ._mode import (
     CopilotClientMode,
@@ -64,26 +84,6 @@
     ExtensionInfo,
 )
 from .copilot_request_handler import CopilotRequestHandler, create_copilot_request_adapter
-from .generated.rpc import (
-    ClientGlobalApiHandlers,
-    ClientSessionApiHandlers,
-    GitHubTelemetryNotification,
-    ModelBillingTokenPrices,
-    ModelBillingTokenPricesLongContext,  # noqa: F401
-    OpenCanvasInstance,
-    RemoteSessionMode,
-    ServerRpc,
-    _ConnectResult,
-    _HookInvokeRequest,
-    _HookInvokeResponse,
-    from_datetime,
-    register_client_global_api_handlers,
-    register_client_session_api_handlers,
-)
-from .generated.session_events import (
-    SessionEvent,
-    session_event_from_dict,
-)
 from .session import (
     AutoModeSwitchHandler,
     BearerTokenProvider,
@@ -4356,7 +4356,7 @@ async def _apply_post_create_options_patch(
         If the patch is rejected, tear the session down so empty-mode callers
         never end up with a permissive session.
         """
-        from .generated.rpc import SessionInstalledPlugin, SessionUpdateOptionsParams
+        from ._generated.rpc import SessionInstalledPlugin, SessionUpdateOptionsParams
 
         patch = _post_create_options_patch(
             mode,

…thon/copilot/generated/session_events.py …hon/copilot/_generated/session_events.pypython/copilot/generated/session_events.py renamed to python/copilot/_generated/session_events.py python/copilot/generated/session_events.py renamed to python/copilot/_generated/session_events.py

@@ -15,7 +15,7 @@
   transparent forwarding connection via the ``websockets`` library.
 
 :func:`create_copilot_request_adapter` converts a handler into the generated
-:class:`~copilot.generated.rpc.LlmInferenceHandler` shape so the RPC dispatcher
+:class:`~copilot.rpc.LlmInferenceHandler` shape so the RPC dispatcher
 can route inbound ``httpRequestStart`` / ``httpRequestChunk`` frames through it.
 """
 
@@ -27,7 +27,7 @@
 from dataclasses import dataclass, field
 from typing import TYPE_CHECKING, Any
 
-from .generated.rpc import (
+from ._generated.rpc import (
     LlmInferenceHTTPRequestChunkRequest,
     LlmInferenceHTTPRequestChunkResult,
     LlmInferenceHTTPRequestStartRequest,

python/copilot/canvas.py

@@ -3,11 +3,11 @@
 These types are auto-generated from the Copilot CLI protocol schemas. This
 module is the stable public access point so callers can write
 ``copilot.rpc.SessionUpdateOptionsParams`` without depending on the internal
-``copilot.generated`` package layout.
+``copilot._generated`` package layout.
 """
 
-from .generated.rpc import *  # noqa: F401, F403
-from .generated.rpc import (
+from ._generated.rpc import *  # noqa: F401, F403
+from ._generated.rpc import (
     SessionFsReaddirWithTypesEntryType as SessionFSReaddirWithTypesEntryType,  # noqa: F401
 )
-from .generated.rpc import __all__  # noqa: F401
+from ._generated.rpc import __all__  # noqa: F401