Repository navigation
refactor: group built-in workflows into dedicated packages - #269
Merged
Merged
Conversation
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.
Built-in workflow helpers currently share one flat directory, obscuring which files implement Duo, Weave, and Reconstruct. Group each workflow and its private helpers into its own package so the complete implementation is easy to find.
Move each existing entry module to its package's
__init__.py, preserving its public import path, function and class module identity, registry entries, and serialization paths. Keep prompts and execution logic unchanged. Update internal test imports and both Duo guides with the source reading entry points.Validation completed includes Ruff, all four import contracts, dependency checks, documentation translation checks, an sdist-to-wheel build, and an isolated installed-wheel probe that imports every built-in module, checks public identities and class serialization, and executes the existing Weave delivery smoke. Independent review approved the exact head and passed 18 installation/discovery tests, including the OCE wrapper import and argument forwarding. GitHub distribution, macOS, browser fixtures, documentation, title, hygiene, and secret checks have passed. The Python 3.10–3.14 CI matrix is running. Focused workflow regressions completed with 426 passed and 4 optional-interpreter skips. The overlapping local full-suite run was interrupted after 1,466 passed and 2 skipped in favor of the five-version CI matrix; it is not claimed as a completed full-suite result.