Skip to content

[python] Change CopilotClient.__init__ to take config objects #793

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
SteveSandersonMS merged 5 commits into from
Mar 13, 2026
Merged

[python] Change CopilotClient.__init__ to take config objects #793

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
a175956
Change `CopilotClient.__init__` to take config objects
brettcannon Mar 11, 2026
3472714
Improve CLI path verification in CopilotClient to use `shutil.which`
brettcannon Mar 11, 2026
162a551
Fix changes made in `main`
brettcannon Mar 12, 2026
2e424ac
Fix shutil.which error message showing None instead of original path
SteveSandersonMS Mar 13, 2026
d9273c7
Remove dead auto_restart field from SubprocessConfig
SteveSandersonMS Mar 13, 2026
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
Prev Previous commit
Remove dead auto_restart field from SubprocessConfig 
This field was defined but never read anywhere in client.py.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
  • Loading branch information
@SteveSandersonMS
SteveSandersonMS and Copilot committed Mar 13, 2026
commit d9273c7d80c2a098aab96ae06f86bb9efcd7083e
1 change: 0 additions & 1 deletion python/README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -125,7 +125,6 @@ CopilotClient(
- `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_restart` (bool): Auto-restart on crash (default: True)
- `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).
Expand Down
4 changes: 0 additions & 4 deletions python/copilot/types.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -72,7 +72,6 @@ class SelectionAttachment(TypedDict):
# Configuration for CopilotClient connection modes



@dataclass
class SubprocessConfig:
"""Config for spawning a local Copilot CLI subprocess.
Expand Down Expand Up @@ -109,9 +108,6 @@ class SubprocessConfig:
log_level: LogLevel = "info"
"""Log level for the CLI process."""

auto_restart: bool = True
"""Auto-restart the CLI server if it crashes."""

env: dict[str, str] | None = None
"""Environment variables for the CLI process. ``None`` inherits the current env."""

Expand Down
Loading