Skip to content

Fix --sandbox when running cli plugin mode#2151

Merged
dgageot merged 1 commit intodocker:mainfrom
gtardif:fix_sandbox_cli_plugin_mode
Mar 18, 2026
Merged

Fix --sandbox when running cli plugin mode#2151
dgageot merged 1 commit intodocker:mainfrom
gtardif:fix_sandbox_cli_plugin_mode

Conversation

@gtardif
Copy link
Contributor

@gtardif gtardif commented Mar 18, 2026

No description provided.

@gtardif gtardif requested a review from a team as a code owner March 18, 2026 07:40
Signed-off-by: Guillaume Tardif <guillaume.tardif@gmail.com>
@gtardif gtardif force-pushed the fix_sandbox_cli_plugin_mode branch from 975e4c4 to 25c14c8 Compare March 18, 2026 07:41
Copy link

@docker-agent docker-agent bot left a comment

Choose a reason for hiding this comment

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

Assessment: 🟢 APPROVE

No bugs detected in the changed code. The implementation correctly:

  • Adds logic to strip the "agent" argument when running as a CLI plugin
  • Uses plugin.RunningStandalone() to detect CLI plugin mode
  • Tracks state with agentStripped flag to ensure single stripping
  • Includes comprehensive test coverage for both CLI plugin scenarios

@dgageot dgageot merged commit cd4e7da into docker:main Mar 18, 2026
4 checks passed
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