Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Include a trivial items keyword if missing.#6669

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged

Conversation

@eiriktsarpalis
Copy link
Member

@eiriktsarpaliseiriktsarpalis commentedJul 31, 2025
edited by dotnet-policy-servicebot
Loading

Fix#6656.

Microsoft Reviewers:Open in CodeFlow

CopilotAI review requested due to automatic review settingsJuly 31, 2025 13:05
@eiriktsarpaliseiriktsarpalis requested a review froma team as acode ownerJuly 31, 2025 13:05
@github-actionsgithub-actionsbot added the area-aiMicrosoft.Extensions.AI libraries labelJul 31, 2025
Copy link
Contributor

CopilotAI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds handling for trivial array types in JSON schema generation to ensure they include an "items" property. The change addresses issue#6656 by modifying the schema transformation logic to automatically add an empty "items" object when processing enumerable types that don't already have this property defined.

  • Adds logic to include a trivial "items" keyword for enumerable types missing this property
  • Includes a new test case to verify the behavior forobject[] arrays
  • Updates the test JSON serialization context to support the new test type

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

FileDescription
AIJsonUtilities.Schema.Create.csAdds logic to include empty "items" object for enumerable types without existing items property
AIJsonUtilitiesTests.csAdds test case and JSON serialization support forobject[] to verify the fix

Copy link
Member

@stephentoubstephentoub left a comment

Choose a reason for hiding this comment

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

Thanks

@eiriktsarpaliseiriktsarpalis merged commitc9584a4 intodotnet:mainJul 31, 2025
7 checks passed
This was referencedAug 14, 2025
This was referencedAug 27, 2025
@github-actionsgithub-actionsbot locked and limited conversation to collaboratorsAug 31, 2025
Sign up for freeto subscribe to this conversation on GitHub. Already have an account?Sign in.

Reviewers

Copilot code reviewCopilotCopilot left review comments

@stephentoubstephentoubstephentoub approved these changes

Assignees

@eiriktsarpaliseiriktsarpalis

Labels

area-aiMicrosoft.Extensions.AI libraries

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

response_format error after updating to => 9.5.0

2 participants

@eiriktsarpalis@stephentoub

[8]ページ先頭

©2009-2025 Movatter.jp