Python SDK example code failed with 'Callable' has no attribute 'approve_all'

[shangerxin]

Description

Test the SDK code with https://github.com/github/copilot-sdk/blob/main/docs/setup/bundled-cli.md python QuickStart.

Code

>>> async def main():           
...     client = CopilotClient()
...     await client.start()
...     await client.start()                                                                                                            
...     session = await client.create_session(on_permission_request=PermissionHandler.approve_all, model="auto")                                                         
...     response = await session.send_and_wait("What is 2 + 2?")
...     print(response.data.content)
...     await client.stop()
... 
>>> asyncio.run(main())
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "C:\Users\erxinsha\projects\frameworks.ai.benchmarkqa.ci\venv\lib\asyncio\runners.py", line 44, in run
    return loop.run_until_complete(main)
  File "C:\Users\erxinsha\projects\frameworks.ai.benchmarkqa.ci\venv\lib\asyncio\base_events.py", line 649, in run_until_complete
    return future.result()
  File "<stdin>", line 5, in main
  File "C:\Users\erxinsha\projects\frameworks.ai.benchmarkqa.ci\venv\lib\typing.py", line 983, in __getattr__
    return getattr(self.__origin__, attr)
AttributeError: type object 'Callable' has no attribute 'approve_all'
>>> 

Environment

Python 3.10.19
OS Windows
copilot 0.1.9
github-copilot-sdk 0.1.25

[github-actions]

Investigation Findings

Short answer: This is not a bug in the SDK code — it's a documentation gap. The PermissionHandler.approve_all static method exists and works correctly, but only on Python 3.11+.

What I investigated

• python/copilot/session.py — confirmed PermissionHandler class and its approve_all static method are correctly defined (lines 381–390)
• python/copilot/__init__.py — confirmed PermissionHandler is exported properly
• python/pyproject.toml — found requires-python = ">=3.11", meaning Python 3.10 is not supported
• docs/setup/bundled-cli.md — the Python QuickStart snippet uses PermissionHandler.approve_all but never mentions the Python 3.11+ requirement

Root cause

You are running Python 3.10.19, but the SDK requires Python 3.11 or newer (as declared in pyproject.toml). On Python 3.10, the from __future__ import annotations directive in session.py combined with the _PermissionHandlerFn = Callable[...] type alias causes PermissionHandler to be resolved at attribute access time as the Callable generic alias instead of the PermissionHandler class — hence the confusing AttributeError: type object 'Callable' has no attribute 'approve_all'.

Fix

Upgrade to Python 3.11 or later and the example code will work as shown in the docs.

Documentation gap

The bundled-cli.md QuickStart Python snippet should include a note that Python 3.11+ is required, matching the requires-python = ">=3.11" constraint in pyproject.toml. Without this, users on 3.10 encounter a cryptic AttributeError with no clear guidance. This issue has been relabeled as documentation to track that gap.

Generated by Bug Handler for #2293 · sonnet46 38.1 AIC · ⌖ 5.21 AIC · ⊞ 5.9K · ◷