Migrate executor to console 0.2 commands#228
Conversation
Greptile SummaryThis PR migrates
Confidence Score: 5/5Safe to merge; the Command API migration is correct and the archive helpers are a clean consolidation of previously scattered inline shell strings. The two changed source paths (Docker.php and ExecutorTest.php) are straightforward mechanical refactors. Argument safety in the new Command API is verified: single-Command calls go through proc_open argv array (no shell interpretation), and composed commands use escapeshellarg() internally. The only findings are minor temp-file cleanup gaps in test helpers that don't affect production code. No files require special attention. Important Files Changed
Reviews (4): Last reviewed commit: "Fix zip archive temp path in e2e tests" | Re-trigger Greptile |
Summary
utopia-php/consoleto0.2.*and pointutopia-php/orchestrationatutopia-php/orchestration#66until the compatible release is availableConsole::execute()usage to structuredUtopia\Commandobjects and pass the newstderrargument required by console 0.2Validation
composer analyzecomposer test:unit(fails locally: missingext-swoole/Swoole\Tablein this environment)