Skip to content

[POC] Add Data Designer recipe discovery#613

Draft
oliverholworthy wants to merge 2 commits intoNVIDIA-NeMo:mainfrom
oliverholworthy:oholworthy/data-designer-recipe-flag
Draft

[POC] Add Data Designer recipe discovery#613
oliverholworthy wants to merge 2 commits intoNVIDIA-NeMo:mainfrom
oliverholworthy:oholworthy/data-designer-recipe-flag

Conversation

@oliverholworthy
Copy link
Copy Markdown

@oliverholworthy oliverholworthy commented May 7, 2026

Summary

POC layered on top of #612 that lets installed packages register Data Designer recipes without DataDesigner core depending on any specific recipe repository.

External packages can register a named recipe with the data_designer.recipes entry point group. Data Designer can then inspect and execute those recipes through the normal generation verbs:

data-designer recipes list --output json
data-designer recipes show retrieval-sdg --output json
data-designer recipes help retrieval-sdg
data-designer preview --recipe retrieval-sdg -- --input-dir ./docs
data-designer create --recipe retrieval-sdg -- --input-dir ./docs

Design

  • Adds the data_designer.recipes entry point group as the external registration mechanism.
  • Adds data-designer recipes list, show, and help for human and agent inspection.
  • Supports text and JSON output for recipe listing and metadata inspection.
  • Lets a recipe expose load_config_builder(params) for execution.
  • Lets a recipe module expose build_typer_app() so Data Designer can surface recipe-specific help and structured argument metadata from the same Typer command shape used for execution.
  • Keeps argparse parser inspection as a compatibility fallback for simple external recipes, but the POC consumer in DataDesignerPlugins uses Typer.
  • Keeps bespoke recipes out of DataDesigner core while making installed recipe packages discoverable.

Review Notes

This PR currently includes the #612 commit because the base branch is in a fork. The recipe-discovery work is the top commit. Once #612 lands, this can be rebased so the diff contains only the recipe registry and inspection layer.

Signed-off-by: Oliver Holworthy <1216955+oliverholworthy@users.noreply.github.com>
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 7, 2026

Linked Issue Check

This PR does not reference an issue. External contributions must link to
a triaged issue before the PR can be merged.

Add one of the following to your PR description:

  • Fixes #<issue-number>
  • Closes #<issue-number>
  • Resolves #<issue-number>

If no issue exists yet, open one
and a maintainer will triage it.

See CONTRIBUTING.md
for details.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 7, 2026

Thank you for your submission! We ask that you sign our Developer Certificate of Origin before we can accept your contribution. You can sign the DCO by adding a comment below using this text:


I have read the DCO document and I hereby sign the DCO.


You can retrigger this bot by commenting recheck in this Pull Request. Posted by the DCO Assistant Lite bot.

Signed-off-by: Oliver Holworthy <1216955+oliverholworthy@users.noreply.github.com>
@oliverholworthy oliverholworthy force-pushed the oholworthy/data-designer-recipe-flag branch from d9a6c90 to 0cf8039 Compare May 7, 2026 15:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant