Skip to content

Improve action reliability for failed diff checks#19

Draft
ajvalenty wants to merge 1 commit into
mainfrom
cursor/action-reliability-improvements-bf9f
Draft

Improve action reliability for failed diff checks#19
ajvalenty wants to merge 1 commit into
mainfrom
cursor/action-reliability-improvements-bf9f

Conversation

@ajvalenty

Copy link
Copy Markdown
Contributor

Summary

  • default MATCH=0 at the start of the diff-check step before any git operations
  • only set MATCH=1 after a full successful comparison confirms no change
  • move SHA write/upload after the check step and gate both on steps.check.outcome == 'success'

Why

  • if check logic fails mid-step, dismissal remains the safe default (MATCH=0)
  • failed checks no longer overwrite SHA baseline artifacts, so subsequent runs compare from the last successful baseline

Validation

  • parsed action.yml as YAML
  • ran targeted assertions to verify check/upload ordering and success gating
Open in Web Open in Cursor 

Co-authored-by: AJ Valenty <ajvalenty@users.noreply.github.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.

2 participants