Skip to content

feat: (m-decompose) Module Prompt V3#770

Open
csbobby wants to merge 20 commits intogenerative-computing:mainfrom
csbobby:tagparse
Open

feat: (m-decompose) Module Prompt V3#770
csbobby wants to merge 20 commits intogenerative-computing:mainfrom
csbobby:tagparse

Conversation

@csbobby
Copy link
Copy Markdown
Contributor

@csbobby csbobby commented Apr 1, 2026

Module Prompt V3

Description

  • Enhance the module execution and extraction robustness.

Implementation Checklist

Protocol Compliance

  • parts() returns list of constituent parts (Components or CBlocks)
  • format_for_llm() returns TemplateRepresentation or string
  • _parse(computed: ModelOutputThunk) parses model output correctly into the specified Component return type

Testing

  • Tests added to tests/components/
  • New code has 100% coverage
  • Ensure existing tests and github automation passes (a maintainer will kick off the github automation when the rest of the PR is populated)

@csbobby csbobby requested a review from a team as a code owner April 1, 2026 03:16
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 1, 2026

The PR description has been updated. Please fill out the template for your PR to be reviewed.

@csbobby csbobby changed the title Tagparse feat (m decompose) Module Prompt V3 Apr 1, 2026
@csbobby csbobby changed the title feat (m decompose) Module Prompt V3 feat (m decompose): Module Prompt V3 Apr 1, 2026
@csbobby csbobby changed the title feat (m decompose): Module Prompt V3 feat: (m-decompose) Module Prompt V3 Apr 1, 2026
@github-actions github-actions bot added the enhancement New feature or request label Apr 1, 2026
@csbobby csbobby requested a review from a team as a code owner April 1, 2026 18:46
@csbobby csbobby removed the request for review from a team April 1, 2026 22:46
@csbobby
Copy link
Copy Markdown
Contributor Author

csbobby commented Apr 1, 2026

Updated the modular prompts to adapt the recent llms.

@csbobby csbobby enabled auto-merge April 2, 2026 06:16
@csbobby csbobby disabled auto-merge April 2, 2026 06:18
@csbobby
Copy link
Copy Markdown
Contributor Author

csbobby commented Apr 2, 2026

Hi This passes the tests and ready to be reviewed @jakelorocco.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should this v3 get added back in given the jinja2 template being added?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Are you asking if the v3 jinja2 template is been added? Yes, m_decomp_result_v3.py.jinja2 is included in the PR.

image

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The m-decompose will use the latest version of templates.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"v3" is commented out of the code here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants