Skip to content

fix(ci): handle failed builders before contract validation - #53

Merged
tiankaima-copilot merged 1 commit into
masterfrom
audit/build-status-before-contracts
Aug 27, 2026
Merged

fix(ci): handle failed builders before contract validation#53
tiankaima-copilot merged 1 commit into
masterfrom
audit/build-status-before-contracts

Conversation

@tiankaima-copilot

Copy link
Copy Markdown
Collaborator

Summary

  • move build-status handling ahead of schema/report validation
  • skip upstream contract diagnostics when the curriculum builder fails, allowing cached outputs to remain usable
  • add a regression test for failed authenticated builders with no prior .artifacts/upstream-contracts

The scheduled build at https://github.com/Life-USTC/static/actions/runs/33041890460 failed while running find .artifacts/upstream-contracts after authenticated upstream endpoints were unavailable. This change preserves the existing builder rollback/cache behavior and reports the actual builder status first.

Validation

  • uv run ruff check .
  • uv run ruff format --check .
  • uv run coverage run -m unittest discover -s tests -p 'test_*.py'\n- uv run coverage report

@tiankaima-copilot
tiankaima-copilot force-pushed the audit/build-status-before-contracts branch from 89d06d8 to 4e7b074 Compare August 27, 2026 10:03
@tiankaima-copilot
tiankaima-copilot merged commit 0066e42 into master Aug 27, 2026
4 checks passed
@tiankaima-copilot
tiankaima-copilot deleted the audit/build-status-before-contracts branch August 27, 2026 10:04
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