feat(publish): add final package helper#79
Open
RayJiang4S wants to merge 4 commits into
Open
Conversation
Add cover policy and cover direction metadata to scripts, introduce a publish_packager tool for final deliverable packaging, and record cover provenance and checksum details in a final package manifest.
Add a local final package review page with copied deliverables, reference pages, path-copy controls, generated package summaries, and documentation for package handoff workflows.
Add an optional require_timing_qa package gate that verifies Timing QA references are attached before a final package can pass. Record Timing QA status in manifests, package summaries, and review pages.
Author
|
Update from the v4 production retrospective:
Validation:
Note: a broader |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
cover_policyandcover_directionmetadata to script artifacts so cover/poster needs are decided during planning, not bolted on at the endpublish_packagertool for packaging final deliverables, copying sidecars, recording checksums, and optionally replacing the first video frame with the selected cover without prepending extra timefinal_package_manifestto record final package files, cover provenance, duration verification, and cover policy/direction audit metadataWhy
OpenMontage already has a publish stage and
publish_log, but there is no small local helper for assembling a final deliverable package or preserving the reasoning behind cover/first-frame choices. This keeps the change scoped to publish prep rather than adding a parallel "finalizer" stage.The cover metadata is intentionally provider-neutral: covers can come from rendered frames, generated images, generated video frames, source footage, or manual design.
Tests
.venv/bin/python -m pytest tests/tools/test_publish_packager.py -q