fix: expose scoped artifact cleanup apply command#616
Conversation
Homeboy Results —
|
|
CI Root cause: str_replace('\', '/', $cwd)That reproduces the CI failure shape: Upstream fix: Automattic/wp-codebox#860 Local verification with the fixed WP Codebox worktree wired into Homeboy passed: HOMEBOY_WP_CODEBOX_BIN=/Users/chubes/Developer/wp-codebox@fix-escape-phpunit-cwd-backslash/packages/cli/dist/index.js \
HOMEBOY_WP_CODEBOX_CORE_MODULE=/Users/chubes/Developer/wp-codebox@fix-escape-phpunit-cwd-backslash/packages/runtime-core/dist/index.js \
homeboy --force-hot --allow-local-hot test data-machine-code \
--path /Users/chubes/Developer/data-machine-code@fix-issue-615-cleanup-artifacts-dry-run-apply \
--changed-since origin/mainResult: |
Summary
apply_commandto artifact cleanup dry-run results so JSON callers can apply the same reviewed bounded page.--limit,--offset, or--exhaustive.Closes #615.
Verification
php tests/smoke-worktree-cleanup-artifacts.phpphp tests/smoke-worktree-cleanup-cli.phpphp -l inc/Workspace/WorkspaceArtifactCleanup.php && php -l inc/Cli/Commands/WorkspaceCommand.php && php -l tests/smoke-worktree-cleanup-artifacts.php && php -l tests/smoke-worktree-cleanup-cli.phpAI assistance