Skip to content

fix: recognize dynamic site deployment runs - #56

Merged
lyc-aon merged 1 commit into
mainfrom
fix/maintainer-site-runname-adoption
Jul 18, 2026
Merged

fix: recognize dynamic site deployment runs#56
lyc-aon merged 1 commit into
mainfrom
fix/maintainer-site-runname-adoption

Conversation

@lyc-aon

@lyc-aon lyc-aon commented Jul 18, 2026

Copy link
Copy Markdown
Contributor

Problem

The maintainer matched the site workflow by both its stable path and the literal name Deploy project site. The workflow now defines a dynamic run-name, so a successful exact-commit v0.1.23 site deployment was misclassified as terminal/stale. That sent the compatibility flow back to Sol instead of adopting the public release.

Fix

  • identify site deployment runs by stable workflow path, exact commit, and workflow_dispatch event
  • retain status/conclusion checks and exact CI/release workflow checks
  • make maintainer fixtures use the real dynamic site run-name
  • assert the contract never relies on the dynamic site name

Verification

  • bash -n ops/t4-maintainer/run.sh
  • git diff --check
  • node --test scripts/t4-maintainer-contract.test.mjs scripts/t4-maintainer-integration.test.mjs (195/195)

This is a production blocker for adopting and deterministically deploying the already-published immutable T4 Code v0.1.23 release.

@lyc-aon
lyc-aon merged commit 719b7b6 into main Jul 18, 2026
4 checks passed
@lyc-aon
lyc-aon deleted the fix/maintainer-site-runname-adoption branch July 18, 2026 20:30
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