Refactor DockerCLI for console 0.2 commands#66
Conversation
Greptile SummaryThis PR migrates
Confidence Score: 5/5The change is safe to merge; all previously identified issues have been addressed and the migration is internally consistent. All methods in DockerCLI were updated uniformly to the structured Command API with no shell escaping left behind. The three previously raised concerns are each resolved and covered by tests or confirmed removed. No new logic paths were introduced that could silently mishandle arguments. No files require special attention. Important Files Changed
Reviews (4): Last reviewed commit: "Update .gitignore" | Re-trigger Greptile |
|
@greptile-apps please review the latest commit on this PR. |
|
@greptile-apps please review the latest commit on this PR (49a86d9). The screen argv regression is fixed, label quote stripping is removed, and a regression test was added for labels containing single quotes. |
Summary
utopia-php/consoleto0.2.*and refresh the lockfileDockerCLIfrom shell-string execution to structuredUtopia\CommandusageValidation
./vendor/bin/pint --test./vendor/bin/phpstan analyse --level 6 src tests./vendor/bin/phpunit --configuration phpunit.xml tests/Orchestration/Adapter/DockerCLITest.php(fails locally because the test setup expects/usr/src/code/tests/Orchestration/Resources, which is not present in this workspace; opening the PR to run CI in the intended environment)