Skip to content

fix(llm): enforce reliable structured output - #237

Merged
laipz8200 merged 4 commits into
mainfrom
laipz8200/fix-reliable-structured-output
Aug 7, 2026
Merged

fix(llm): enforce reliable structured output#237
laipz8200 merged 4 commits into
mainfrom
laipz8200/fix-reliable-structured-output

Conversation

@laipz8200

@laipz8200 laipz8200 commented Aug 3, 2026

Copy link
Copy Markdown
Member

Important

  1. Make sure you have read our contribution guidelines
  2. Search existing issues and pull requests to confirm this change is not a duplicate
  3. Open or identify the issue this pull request resolves or advances
  4. Use a Conventional Commits title for this pull request, and mark breaking changes with !
  5. Remember that the pull request title will become the squash merge commit message
  6. If CLA Assistant prompts you, sign CLA.md in the pull request conversation

Related Issue

Closes #236

Summary

  • normalize legacy structured-output switches to the canonical structured_output_switch_on field
  • validate Draft 7 schema configuration and final blocking, streaming, and polling results with stage and JSON path context
  • preserve supported, unsupported, and unknown provider capability states while forwarding merged Slim parameters

Checklist

  • This pull request links the issue it resolves or advances
  • This pull request title follows Conventional Commits, and any breaking change is marked with !
  • If CLA Assistant prompted me, I signed CLA.md in the pull request conversation

CLA Assistant has not prompted me.

Normalize the legacy structured-output switch to the canonical field, validate Draft 7 schemas and final results, and preserve provider capability tri-state behavior.
@laipz8200
laipz8200 marked this pull request as ready for review August 3, 2026 02:09
@dosubot dosubot Bot added the size:L This PR changes 100-499 lines, ignoring generated files. label Aug 3, 2026
@laipz8200
laipz8200 enabled auto-merge (squash) August 3, 2026 02:09
Resolve the LLM node conflict against centralized prompt construction while preserving structured-output capability and schema validation.
Comment thread src/graphon/nodes/llm/node.py Outdated
Fail schema validation before provider invocation so user-controlled references cannot trigger retrieval.
Use an empty referencing registry so external schema references fail without network access while local references continue to work.
@laipz8200
laipz8200 merged commit 3d14fc9 into main Aug 7, 2026
8 checks passed
@laipz8200
laipz8200 deleted the laipz8200/fix-reliable-structured-output branch August 7, 2026 03:36
@dosubot dosubot Bot added the lgtm This PR has been approved by a maintainer label Aug 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm This PR has been approved by a maintainer size:L This PR changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Make structured output schema enforcement reliable

2 participants