main

Aaflyingtofix: address round-1 P1 — move dry-run before auth + use AuthClientFromFactory + -y shorthand

f1ce431bAdd on Jul 10History

Files	Last commit	Last update
delete.go	fix: address round-1 P1 — move dry-run before auth + use AuthClientFromFactory + -y shorthand - Move dry-run block before auth (mirrors label delete; --dry-run now works without a token since it makes zero API calls). - Use cmdutil.AuthenticatedClientFromFactory instead of manual two-step (consistency with sibling list/view/download commands). - Add -y shorthand for --yes (consistency with label delete). Round-1 P1+P2 from PR #389 review (approved). Refs #433 Co-Authored-By: opencode (glm-5.2) <noreply@anthropic.com>	3 months ago
delete_test.go	feat(actions): add gc actions artifact delete for artifact deletion Add support for the GitCode Actions v8 endpoint DELETE /api/v8/repos/{owner}/{repo}/actions/artifacts/{artifact_id}. - api: new DeleteActionsArtifact via RawREST (DELETE, v8 path). Endpoint returns 204 No Content on success. - cmd: new gc actions artifact delete <artifact-id> command (ExactArgs(1)), registered under the artifact parent. Destructive: default confirmation gate (type artifact id to confirm in TTY); non-TTY requires --yes. --dry-run previews without deleting. --json outputs {artifact_id, owner, repo, action}. - tests: dry-run (no API call), dry-run JSON, --yes delete (DELETE method + path verified), --yes JSON, non-TTY without --yes refused, 404 error exit-code-through-%w, param validation. - docs: COMMANDS.md gains an actions artifact delete section; README entry. Risk: high (deletion). Real-data: infra-test dry-run + --yes bad artifact_id → HTTP 400 exit 2 (API format validation). Closes #433 Co-Authored-By: opencode (glm-5.2) <noreply@anthropic.com>	3 months ago