Skip to content

ci(action): Exercise cancellation finalization - #523

Closed
jamieQ wants to merge 6 commits into
feat/graceful-action-cancellationfrom
ci/exercise-action-cancellation
Closed

ci(action): Exercise cancellation finalization#523
jamieQ wants to merge 6 commits into
feat/graceful-action-cancellationfrom
ci/exercise-action-cancellation

Conversation

@jamieQ

@jamieQ jamieQ commented Aug 27, 2026

Copy link
Copy Markdown
Member

This stacks on #521 and temporarily turns Warden's own PR workflow into an
integration harness for graceful cancellation. Superseding commits cancel an
in-progress Analyze step, verify that it finalized a findings file with a
cancelled outcome and completion marker, and retain the artifact for
inspection.

The cancelled job then continues through report mode when credentials are
available. This lets us confirm that partial findings and usage accumulated
before cancellation are published by the existing analyze/report ownership
boundary. The workflow-only commit is intended to be discarded after the
experiment.

jamieQ and others added 2 commits August 27, 2026 10:54
Abort in-flight analysis, preserve partial findings and usage, and finalize findings artifacts with a cancelled outcome. Keep analyze/report publication ownership intact and conclude active checks consistently.

Co-Authored-By: GPT-5 <noreply@anthropic.com>
Cancel superseded PR analysis runs, verify their finalized findings artifacts, and continue into report mode so cancelled usage publication can be observed.

Co-Authored-By: GPT-5 <noreply@anthropic.com>
@vercel

vercel Bot commented Aug 27, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
warden-prod Skipped Skipped Aug 27, 2026 5:01pm

Request Review

Supersede the in-progress Analyze run to exercise graceful cancellation.

Co-Authored-By: GPT-5 <noreply@anthropic.com>
Comment thread .github/workflows/warden.yml
jamieQ and others added 2 commits August 27, 2026 11:59
Replace the composite action shell with the Node process so runner cancellation reaches Warden's signal handler. Log the first accepted signal before finalizing partial results.

Co-Authored-By: GPT-5 <noreply@anthropic.com>
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.

1 participant