Skip to content

fix(ci): isolate current creative storyboards - #7523

Draft
bokelley wants to merge 3 commits into
mainfrom
conductor/isolate-creative-storyboard-runner
Draft

bokelley wants to merge 3 commits into
mainfrom
conductor/isolate-creative-storyboard-runner

Conversation

@bokelley

@bokelley bokelley commented Sep 14, 2026 •

Copy link
Copy Markdown
Contributor

Run current creative storyboards through the existing isolated child runner, retaining per-child failures, sibling results, the 49-clean/209-pass grading floors, and required-clean assertions. Current profile rows keep their selected storyboard and tenant path.

The existing branch is refreshed onto main 033ce862 and retains its original head as a merge ancestor. Its narrow operational changeset exemption now fits the current classifier and reads the actual merge-base. Branch-registration and mixed-change guards still fail closed.

Validation on 8322b7a: 27 isolation regressions and the changeset classifier pass. The normal commit hook passes with the full selected root/server tests, guards and typechecks. Local tests do not claim a new live storyboard grade; authoritative service CI must qualify this exact head.

Release hold retained: Draft, auto-merge off. Genuine exact-head maintainer/workflow review and composition with #7520 remain required. Land one classifier change first and refresh the second onto actual merged main; their independent source commits do not waive that coordination.

Draft disposition — 2026-10-09

  • Existing author: @bokelley.
  • Workstream / hold: Engineering / human review; source qualified and pushed; genuine draft/human hold retained.
  • Next action: Current workflow routing, actual merge-base lookup and strict creative-isolation classifier ported; 27 focused tests pass. Full normal hooks and normal push completed on the recorded head. Draft, auto-merge off, exact-head human workflow review and coordination with fix(ci): retry the exact C2PA install fallback failure once #7520 remain; no live seller grading claim.

This draft is retained for the stated workstream. Continue on this existing branch; no parallel PR is needed. This disposition does not waive CI, review, migration, or release gates.

Copy link
Copy Markdown
Contributor Author

IPR is required by the effective main rules and is now green on exact head 9569185a50fb8779953f490740091a3ba722db81 after rerunning only the canceled verification against the existing author signature (successful job). All five required contexts are now green; build/storyboard CI passed without reruns. The earlier reported-check listing omitted the absent IPR context.

@bmilekic, please provide human maintainer review of this exact head (tree 9d2548b1e69f54416c63256899de90ab0536d4b4, direct main base 36a86d6c4ae5c42e5e2e783d95bdb97b5cb6f614). Both independent exact-head audits found no blocker. This remains Draft with auto-merge off; approval is still required.

#7520 overlaps the workflow and classifier/tests: whichever lands second must deliberately compose both exact exemptions and rerun/audit both suites. Historic rc.3 root cause remains unproven. Any recovery must use a newly generated, reviewed current candidate after mitigation lands.

@bokelley

bokelley commented Nov 8, 2026

Copy link
Copy Markdown
Contributor Author

Parked until after 3.3 GA per the maintainer's ruling. It carries no GA value, so it's reviewed together with the post-GA CI work. #7520 (C2PA retry) goes ahead now on its own.

This branch has not been deployed

No deployments
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