Refactor Compiler for Strict RecipeManifest Generation (v0.9.0)#21
Refactor Compiler for Strict RecipeManifest Generation (v0.9.0)#21gowthamrao wants to merge 5 commits into
Conversation
- Bump `coreason-manifest` dependency to `^0.9.0`. - Implement `Compiler` bridge in `coreason_maco_builder.core.compiler.manifest` with strict `RecipeManifest` output and integrity hashing. - Refactor `Weaver.assemble` to use the new Compiler. - Update internal schemas (e.g., `CouncilConfig`, `RecipeManifest`) to match Shared Kernel v0.9.0. - Update tests to verify strict manifest generation and new schema structures. - Bump package version to `0.2.0`. Co-authored-by: gowthamrao <13936600+gowthamrao@users.noreply.github.com>
|
👋 Jules, reporting for duty! I'm here to lend a hand with this pull request. When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down. I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job! For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with New to Jules? Learn more at jules.google/docs. For security, I will only act on instructions from the user who triggered this task. |
- Add `__all__` to `src/coreason_maco_builder/schemas/domain.py` to fix mypy import errors. - Add type annotation for `edges` in `tests/core/compiler/test_compiler.py`. - Fix unused ignore in `tests/schemas/test_domain_complex.py`. - Apply ruff formatting and whitespace fixes. Co-authored-by: gowthamrao <13936600+gowthamrao@users.noreply.github.com>
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
- Bump `coreason-manifest` dependency to `^0.9.0`. - Implement `Compiler` bridge in `coreason_maco_builder.core.compiler.manifest` with strict `RecipeManifest` output and integrity hashing. - Refactor `Weaver.assemble` to use the new Compiler. - Update internal schemas (e.g., `CouncilConfig`, `RecipeManifest`) to match Shared Kernel v0.9.0. - Update tests to verify strict manifest generation and new schema structures. - Add edge case and complex tests for compiler. - Add documentation for kernel integration. - Fix CI issues (mypy exports, formatting). - Bump package version to `0.2.0`. Co-authored-by: gowthamrao <13936600+gowthamrao@users.noreply.github.com>
- Create `docs/coreason_manifest_v0.10.0_proposal.md` with detailed requests for schema enhancements (Ad-Hoc Agents, Integrity Hash, Builder Metadata). Co-authored-by: gowthamrao <13936600+gowthamrao@users.noreply.github.com>
- Create `docs/coreason_manifest_v0.10.0_requirements.md` specifying necessary changes to `AgentNode` and `RecipeManifest` to support ad-hoc agents and round-trip engineering. Co-authored-by: gowthamrao <13936600+gowthamrao@users.noreply.github.com>
1 participant
This PR refactors the
coreason-maco-builderto strictly produceRecipeManifestartifacts defined in thecoreason-manifestShared Kernel (v0.9.0). It introduces a dedicated Compiler module that converts the mutableRecipeDraftinto an immutable, hashedRecipeManifest, ensuring alignment with the Runtime's expectations.Key changes:
coreason-manifestto^0.9.0.compile_to_manifestandcompute_integrity_hashinsrc/coreason_maco_builder/core/compiler/manifest.py.Weaver.assembleto delegate artifact creation to the Compiler.domain.pyand tests to useCouncilConfig(strategy/voters) andRecipeManifest(topology/interface/state) as defined in v0.9.0.PR created automatically by Jules for task 16535791145825279386 started by @gowthamrao