Skip to content

ci: skip feedback artifacts after preflight failure#59

Merged
haasonsaas merged 1 commit into
mainfrom
codex/feedback-clean-preflight-20260510
May 10, 2026
Merged

ci: skip feedback artifacts after preflight failure#59
haasonsaas merged 1 commit into
mainfrom
codex/feedback-clean-preflight-20260510

Conversation

@haasonsaas
Copy link
Copy Markdown
Contributor

Summary

  • add step ids to the feedback sentinel/backfill sweep steps
  • skip artifact uploads when the org-token preflight prevents the sweep from running
  • keep missing-token failures focused on the actionable preflight error instead of follow-on missing artifact noise

Verification

  • actionlint .github/workflows/review-feedback-backfill.yml .github/workflows/review-feedback-sentinel.yml
  • YAML parse check for both workflows

@cursor
Copy link
Copy Markdown

cursor Bot commented May 10, 2026

PR Summary

Low Risk
Low risk CI-only change that adjusts workflow step conditions to avoid failing artifact uploads when the sweep/backfill step never ran (e.g., missing org token).

Overview
Prevents the review-feedback-sentinel and review-feedback-backfill workflows from uploading ledger/backlog artifacts when the sweep/backfill step is skipped.

This adds step ids to the sweep/backfill steps and gates the upload-artifact steps with steps.<id>.outcome != 'skipped', reducing noisy follow-on failures after the missing-token preflight error.

Reviewed by Cursor Bugbot for commit be4b60d. Bugbot is set up for automated code reviews on this repo. Configure here.

@haasonsaas haasonsaas merged commit baa3750 into main May 10, 2026
4 checks passed
@haasonsaas haasonsaas deleted the codex/feedback-clean-preflight-20260510 branch May 10, 2026 06:29
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