-
Notifications
You must be signed in to change notification settings - Fork 0
refactor: consolidate the production workflow runtime #40
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Changes from all commits
Commits
Show all changes
43 commits
Select commit
Hold shift + click to select a range
6a4f67d
test: define single-workflow architecture boundary
tim8es 3e8662f
docs: define consolidated workflow architecture
tim8es a6a55d4
refactor: remove legacy migration surface
tim8es b2bb005
refactor: drop migration API exports
tim8es 764c5b6
refactor: remove migration admission path
tim8es f90e683
refactor: make status current-workflow only
tim8es bd07add
refactor: require current source identity schema
tim8es 2cfcd93
refactor: remove migration CLI registration
tim8es ca14fde
refactor: enforce current workspace at preflight boundary
tim8es 5bf406f
refactor: reject unsealed legacy workspaces
tim8es f91cadc
refactor: disable legacy document normalization
tim8es 38401ae
docs: collapse orchestration to one current workflow
tim8es 4b4ffac
refactor: remove migration gate from review promotion
tim8es c675951
refactor: require current workspace metadata contract
tim8es edc65fa
fix: enforce current workspace after extraction
tim8es 7ea8d7e
refactor: fail closed on incomplete workflow workspaces
tim8es b5c7533
test: replace legacy book CLI expectations
tim8es 6940ef1
test: require versioned repository state
tim8es 5390b70
test: reject unsealed workflow workspaces
tim8es c9333ee
test: require review evidence marker
tim8es 1c0a520
test: use current workflow fixtures on GitHub backend
tim8es a62a1e0
test: use current parallel workflow fixtures
tim8es 3d314c9
docs: name translation acceptance evidence
tim8es 072e38a
test: assert current build admission failure
tim8es abd8cb1
test: align invalid status reporting exit semantics
tim8es adf5585
test: keep corpus restore fixture on current workflow
tim8es 8463829
test: map missing GitHub artifacts to domain semantics
tim8es 30afe19
fix: close current workflow build admission gaps
tim8es a7f3156
fix: require translator acceptance before review
tim8es 3267fc9
test: keep review fixtures behind translator acceptance
tim8es 9d2510b
fix: normalize storage not-found semantics
tim8es 11cd536
test: exercise native GitHub not-found semantics
tim8es 1bf0e96
test: cover current build admission gates
tim8es 5d4cd0b
test: accept translations in EPUB fixtures
tim8es 47f9a50
test: preserve accepted translation across corpus restore
tim8es 256c87a
test: accept translations in finalize fixtures
tim8es 72de21d
test: accept translations in finalize recovery fixtures
tim8es d0a90ee
test: accept translations in EPUB reliability fixtures
tim8es fb19b20
test: validate reviews on accepted translations
tim8es f406b2e
test: drive reliability fixtures through translator acceptance
tim8es 2a426b4
test: classify reviewed translation tamper as invalid acceptance
tim8es f02266d
test: give corpus restore current workflow provenance
tim8es d78c35d
fix: allow source corpus recovery before acceptance recheck
tim8es File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,29 @@ | ||
| # Architecture | ||
|
|
||
| Book Translator has one supported workflow runtime. | ||
|
|
||
| The current workflow is repository-authoritative and uses one durable state contract: schema-versioned metadata/progress, explicit source identity and corpus manifest, durable claims, machine review evidence, coordination, finalize/build state, and backend-neutral compare-and-swap storage. | ||
|
|
||
| ## Supported runtime | ||
|
|
||
| - `scripts/book.py` is the primary CLI entrypoint. | ||
| - `scripts/corpus.py` owns source-corpus integrity and restore operations. | ||
| - `scripts/workflow_v2/` is the internal workflow package. The `v2` suffix is an implementation-era module name, not a second supported workflow version. | ||
| - `docs/ORCHESTRATION.md` and `docs/TRANSLATION.md` are the canonical execution contracts. | ||
| - Files under `books/<slug>/` are authoritative durable book state. | ||
|
|
||
| ## State contract | ||
|
|
||
| Every supported book workspace uses the current schema and must contain current workflow provenance, explicit source identity, a sealed source manifest, and machine review evidence. Missing schema versions, legacy lifecycle-only review state, unsealed legacy source state, and workflow migration journals are not supported runtime modes. | ||
|
tim8es marked this conversation as resolved.
|
||
|
|
||
| An older workspace must be converted outside the production runtime before it is used. The current runtime does not auto-normalize legacy schemas and does not perform workflow-version migrations. | ||
|
|
||
| ## Runtime boundaries | ||
|
|
||
| The orchestration core owns state transitions. Translator and Reviewer roles produce artifacts and decisions but do not race shared mutable state. Storage backends provide create-if-absent and compare-and-swap semantics; filesystem and GitHub API storage implement the same contract. | ||
|
|
||
| Source integrity is independent from literary review. A valid source corpus does not imply a reviewed translation, and a Reviewer PASS does not replace structural or source-integrity validation. | ||
|
|
||
| ## Completion | ||
|
|
||
| A book is complete only when all intended units have current machine-verifiable PASS evidence, durable lifecycle state is reviewed, structural and corpus checks pass, and requested output artifacts are built and verified. | ||
Large diffs are not rendered by default.
Oops, something went wrong.
This file was deleted.
Oops, something went wrong.
Oops, something went wrong.
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.
Uh oh!
There was an error while loading. Please reload this page.