Skip to content

chore(agent/agentcontainers): disable project autostart by default#19114

Merged
DanielleMaywood merged 2 commits into
mainfrom
danielle/devcontainer-project-discovery-settings
Aug 4, 2025
Merged

chore(agent/agentcontainers): disable project autostart by default#19114
DanielleMaywood merged 2 commits into
mainfrom
danielle/devcontainer-project-discovery-settings

Conversation

@DanielleMaywood

Copy link
Copy Markdown
Contributor

We disable the logic that allows autostarting discovered devcontainers by default. We want this behavior to be opt-in rather than opt-out.

We disable the logic that allows autostarting discovered devcontainers
by default. We want this behavior to be opt-in rather than opt-out.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull Request Overview

This PR disables autostarting of discovered devcontainer projects by default, making this behavior opt-in rather than opt-out for better control over automatic project initialization.

  • Adds a new CLI flag --devcontainers-discovery-autostart-enable (defaults to false)
  • Wraps autostart logic with a conditional check based on the new setting
  • Updates tests to verify the new disabled-by-default behavior

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File Description
cli/testdata/coder_agent_--help.golden Updates help text to include the new autostart flag documentation
cli/agent.go Adds CLI flag and option configuration for discovery autostart control
agent/agentcontainers/api_test.go Adds test case to verify autostart is disabled by default
agent/agentcontainers/api.go Implements the autostart conditional logic and option setter

Comment thread agent/agentcontainers/api.go Outdated
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@DanielleMaywood
DanielleMaywood marked this pull request as ready for review August 4, 2025 08:31

@mtojek mtojek left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

👍

@DanielleMaywood
DanielleMaywood merged commit b8e2344 into main Aug 4, 2025
48 of 49 checks passed
@DanielleMaywood
DanielleMaywood deleted the danielle/devcontainer-project-discovery-settings branch August 4, 2025 15:21
@github-actions github-actions Bot locked and limited conversation to collaborators Aug 4, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants