feat(m-decomp): constraint upgrade#556
Open
csbobby wants to merge 19 commits intogenerative-computing:mainfrom
Open
feat(m-decomp): constraint upgrade#556csbobby wants to merge 19 commits intogenerative-computing:mainfrom
csbobby wants to merge 19 commits intogenerative-computing:mainfrom
Conversation
Signed-off-by: Tulio Coppola <tuliocoppola@ibm.com>
Contributor
|
The PR description has been updated. Please fill out the template for your PR to be reviewed. |
Merge ProtectionsYour pull request matches the following merge protections and will not be merged until they are valid. 🟢 Enforce conventional commitWonderful, this rule succeeded.Make sure that we follow https://www.conventionalcommits.org/en/v1.0.0/
|
jakelorocco
requested changes
Feb 25, 2026
Contributor
jakelorocco
left a comment
There was a problem hiding this comment.
Please make sure to run the pre-commit hooks. There's some formatting issues scattered throughout the PR.
I also worry that this is continuing the trend of the decompose module not being melleaic (mostly that it doesn't use components for templating). But I guess it doesn't worsen the problem, so I'm just flagging it now for future consideration.
Contributor
Author
|
:) The fixes are done @jakelorocco. |
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.
M-Decompose Upgrade
Constraint Extraction and Validation
Description
Added whitespace canonicalization, separator unification, and newline-aware block segmentation to eliminate layout- and order-sensitive parsing failures. Produces stable constraint IDs and consistent downstream bindings, significantly improving subtask–constraint determinism and template reproducibility.
Inserted between constraint extraction and code generation, emitting a constraint to validation-type mapping (e.g., llm / code). Validation strategy is now a first-class, inspectable, template-addressable feature, enabling different generation paths per constraint type.
The validation_code_generator binds each constraint to a generated function stub in the V2. The CLI output shifts from a descriptive decomposition result to an importable, runnable validation module, allowing downstream systems to call validations directly and removing manual reinterpretation.