Skip to content

Add option for structured JSON output - #16

Merged
pawelbaran merged 2 commits into
developfrom
Revit_MechanicalPlumbing_Tool-#210-AddLLMAutoMatch
Sep 10, 2026
Merged

Add option for structured JSON output#16
pawelbaran merged 2 commits into
developfrom
Revit_MechanicalPlumbing_Tool-#210-AddLLMAutoMatch

Conversation

@vietle-bh

Copy link
Copy Markdown
Contributor

Issues addressed by this PR

Closes #15

This pull request adds support for structured output using JSON Schema to the prompt execution flow. The main changes introduce new configuration options for specifying a response format and update the request logic to include a JSON Schema when provided.

Structured output support:

Request logic updates:

  • OpenAI_Adapter/AdapterActions/Execute.cs: Updated the prompt request logic to include a response_format parameter with the specified JSON schema when ResponseFormatJsonSchema is set; otherwise, the request is sent as before.

Default method parameter values are included to ensure existing functionality is maintained.

Test files

Changelog

Additional comments

@vietle-bh vietle-bh self-assigned this Sep 7, 2026
@vietle-bh vietle-bh added the type:feature New capability or enhancement label Sep 7, 2026
@bhombot-ci

bhombot-ci Bot commented Sep 8, 2026

Copy link
Copy Markdown

@vietle-bh just to let you know, I have provided a check-versioning result to this Pull Request as it was detected to be linked to other Pull Requests in a series. The comment which triggered this check came from @vietle-bh on Revit_MechanicalPlumbing_Tool

1 similar comment
@bhombot-ci

bhombot-ci Bot commented Sep 8, 2026

Copy link
Copy Markdown

@vietle-bh just to let you know, I have provided a check-versioning result to this Pull Request as it was detected to be linked to other Pull Requests in a series. The comment which triggered this check came from @vietle-bh on Revit_MechanicalPlumbing_Tool

@vietle-bh
vietle-bh requested a review from pawelbaran September 8, 2026 09:31
Comment thread OpenAI_oM/Config/PromptExecutionConfig.cs Outdated
@vietle-bh
vietle-bh requested a review from pawelbaran September 9, 2026 11:32

@pawelbaran pawelbaran left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Reviewed and tested, happy to approve 👍

@pawelbaran

Copy link
Copy Markdown
Member

@BHoMBot check required

@bhombot-ci

bhombot-ci Bot commented Sep 9, 2026

Copy link
Copy Markdown

@pawelbaran to confirm, the following actions are now queued:

  • check code-compliance
  • check documentation-compliance
  • check project-compliance
  • check core
  • check null-handling
  • check serialisation
  • check versioning
  • check installer

There are 61 requests in the queue ahead of you.

@bhombot-ci

bhombot-ci Bot commented Sep 9, 2026

Copy link
Copy Markdown

@vietle-bh just to let you know, I have provided a check-versioning result to this Pull Request as it was detected to be linked to other Pull Requests in a series. The comment which triggered this check came from @pawelbaran on Revit_MechanicalPlumbing_Tool

1 similar comment
@bhombot-ci

bhombot-ci Bot commented Sep 9, 2026

Copy link
Copy Markdown

@vietle-bh just to let you know, I have provided a check-versioning result to this Pull Request as it was detected to be linked to other Pull Requests in a series. The comment which triggered this check came from @pawelbaran on Revit_MechanicalPlumbing_Tool

@bhombot-ci

bhombot-ci Bot commented Sep 9, 2026

Copy link
Copy Markdown

The check versioning has already been run previously and recorded as a successful check. This check has not been run again at this time.

@pawelbaran
pawelbaran merged commit 3b6ead7 into develop Sep 10, 2026
19 checks passed
@pawelbaran
pawelbaran deleted the Revit_MechanicalPlumbing_Tool-#210-AddLLMAutoMatch branch September 10, 2026 07:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type:feature New capability or enhancement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Support Structured Output via JSON Schema

2 participants