Skip to content

Replace --parallel-launch flag with --sequential-launch flag#482

Merged
lpcox merged 3 commits intomainfrom
copilot/update-launch-flag-behavior
Jan 27, 2026
Merged

Replace --parallel-launch flag with --sequential-launch flag#482
lpcox merged 3 commits intomainfrom
copilot/update-launch-flag-behavior

Conversation

Copy link
Contributor

Copilot AI commented Jan 26, 2026

The --parallel-launch flag was enabled by default, making the flag name confusing since users had to explicitly set it to opt into the default behavior. Flags should represent opt-in to non-default behavior.

Changes

  • Flag renamed: --parallel-launch--sequential-launch (default: false)
  • Logic inverted: Parallel launch is now the default; flag enables sequential mode
  • Config field renamed: ParallelLaunchSequentialLaunch in TOML/JSON configs
  • Variable renaming: parallelLaunchsequentialLaunch throughout codebase
  • Log messages updated: "Parallel server launching enabled (default)" vs "Sequential server launching enabled"

Behavior

Before:

./awmg --config config.toml                    # parallel (default)
./awmg --config config.toml --parallel-launch  # parallel (explicit)

After:

./awmg --config config.toml                      # parallel (default)
./awmg --config config.toml --sequential-launch  # sequential (opt-in)

Default behavior unchanged - servers launch in parallel. Only the flag naming and logic direction changed.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • nonexistent.local
    • Triggering command: /tmp/go-build2056086634/b269/launcher.test /tmp/go-build2056086634/b269/launcher.test -test.testlogfile=/tmp/go-build2056086634/b269/testlog.txt -test.paniconexit0 -test.timeout=10m0s -test.v=true 64/src/runtime/cgo zuLZZ0Z6i /snap/bin/git credential.helpe/opt/hostedtoolcache/go/1.25.6/x64/pkg/tool/linux_amd64/vet (dns block)
  • this-host-does-not-exist-12345.com
    • Triggering command: /tmp/go-build2056086634/b278/mcp.test /tmp/go-build2056086634/b278/mcp.test -test.testlogfile=/tmp/go-build2056086634/b278/testlog.txt -test.paniconexit0 -test.timeout=10m0s -test.v=true 64/src/runtime/cgo OUOwgIYji .12/x64/as credential.usernbash (dns block)

If you need me to access, download, or install something from one of these locations, you can either:


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI and others added 2 commits January 26, 2026 18:20
- Renamed flag from --parallel-launch to --sequential-launch
- Inverted logic: parallel launch is now the default behavior
- Updated all references in code (cmd, config, server, tests)
- Updated test names and assertions to reflect new flag
- Flag description clarifies parallel is default

Co-authored-by: lpcox <15877973+lpcox@users.noreply.github.com>
Co-authored-by: lpcox <15877973+lpcox@users.noreply.github.com>
Copilot AI changed the title [WIP] Update launch flag to sequential default Replace --parallel-launch flag with --sequential-launch flag Jan 26, 2026
Copilot AI requested a review from lpcox January 26, 2026 18:25
@lpcox lpcox marked this pull request as ready for review January 27, 2026 01:08
@lpcox lpcox merged commit 87d194b into main Jan 27, 2026
3 checks passed
@lpcox lpcox deleted the copilot/update-launch-flag-behavior branch January 27, 2026 01:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants