Skip to content
Merged
Show file tree
Hide file tree
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Next Next commit
Change CopilotClient.__init__ to take config objects
  • Loading branch information
brettcannon authored and SteveSandersonMS committed Mar 13, 2026
commit a175956be0cab64632edc81e62216a45a3e6215f
49 changes: 35 additions & 14 deletions python/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -79,12 +79,10 @@ async with await client.create_session({"model": "gpt-5"}) as session:
### CopilotClient

```python
client = CopilotClient({
"cli_path": "copilot", # Optional: path to CLI executable
"cli_url": None, # Optional: URL of existing server (e.g., "localhost:8080")
"log_level": "info", # Optional: log level (default: "info")
"auto_start": True, # Optional: auto-start server (default: True)
})
from copilot import CopilotClient, SubprocessConfig

# Spawn a local CLI process (default)
client = CopilotClient() # uses bundled CLI, stdio transport
await client.start()

session = await client.create_session({"model": "gpt-5"})
Expand All @@ -101,17 +99,40 @@ await session.disconnect()
await client.stop()
```

**CopilotClient Options:**
```python
from copilot import CopilotClient, ExternalServerConfig

- `cli_path` (str): Path to CLI executable (default: "copilot" or `COPILOT_CLI_PATH` env var)
- `cli_url` (str): URL of existing CLI server (e.g., `"localhost:8080"`, `"http://127.0.0.1:9000"`, or just `"8080"`). When provided, the client will not spawn a CLI process.
- `cwd` (str): Working directory for CLI process
- `port` (int): Server port for TCP mode (default: 0 for random)
# Connect to an existing CLI server
client = CopilotClient(ExternalServerConfig(url="localhost:3000"))
```

**CopilotClient Constructor:**

```python
CopilotClient(
config=None, # SubprocessConfig | ExternalServerConfig | None
*,
auto_start=True, # auto-start server on first use
on_list_models=None, # custom handler for list_models()
)
```

**SubprocessConfig** — spawn a local CLI process:

- `cli_path` (str | None): Path to CLI executable (default: bundled binary)
- `cli_args` (list[str]): Extra arguments for the CLI executable
- `cwd` (str | None): Working directory for CLI process (default: current dir)
- `use_stdio` (bool): Use stdio transport instead of TCP (default: True)
- `port` (int): Server port for TCP mode (default: 0 for random)
- `log_level` (str): Log level (default: "info")
- `auto_start` (bool): Auto-start server on first use (default: True)
- `github_token` (str): GitHub token for authentication. When provided, takes priority over other auth methods.
- `use_logged_in_user` (bool): Whether to use logged-in user for authentication (default: True, but False when `github_token` is provided). Cannot be used with `cli_url`.
- `auto_restart` (bool): Auto-restart on crash (default: True)
Copy link

Copilot AI Mar 11, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Docs list SubprocessConfig.auto_restart as supported, but the current CopilotClient implementation does not use this setting (no restart behavior). Either document it as not implemented or remove it until it's supported, otherwise readers will assume crash-restart works.

Suggested change
- `auto_restart` (bool): Auto-restart on crash (default: True)
- `auto_restart` (bool): Reserved for future use — not yet implemented; currently has no effect (default: True)

Copilot uses AI. Check for mistakes.
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See #789

- `env` (dict | None): Environment variables for the CLI process
- `github_token` (str | None): GitHub token for authentication. When provided, takes priority over other auth methods.
- `use_logged_in_user` (bool | None): Whether to use logged-in user for authentication (default: True, but False when `github_token` is provided).

**ExternalServerConfig** — connect to an existing CLI server:

- `url` (str): Server URL (e.g., `"localhost:8080"`, `"http://127.0.0.1:9000"`, or just `"8080"`).

**SessionConfig Options (for `create_session`):**

Expand Down
4 changes: 4 additions & 0 deletions python/copilot/__init__.py
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,7 @@
AzureProviderOptions,
ConnectionState,
CustomAgentConfig,
ExternalServerConfig,
GetAuthStatusResponse,
GetStatusResponse,
MCPLocalServerConfig,
Expand All @@ -33,6 +34,7 @@
SessionListFilter,
SessionMetadata,
StopError,
SubprocessConfig,
Tool,
ToolHandler,
ToolInvocation,
Expand All @@ -47,6 +49,7 @@
"CopilotSession",
"ConnectionState",
"CustomAgentConfig",
"ExternalServerConfig",
"GetAuthStatusResponse",
"GetStatusResponse",
"MCPLocalServerConfig",
Expand All @@ -69,6 +72,7 @@
"SessionListFilter",
"SessionMetadata",
"StopError",
"SubprocessConfig",
"Tool",
"ToolHandler",
"ToolInvocation",
Expand Down
Loading