Skip to content

Filter readonly properties from input schema#63

Merged
mattt merged 2 commits intomainfrom
mattt/readonly
Oct 9, 2025
Merged

Filter readonly properties from input schema#63
mattt merged 2 commits intomainfrom
mattt/readonly

Conversation

@mattt
Copy link
Owner

@mattt mattt commented Oct 9, 2025

Related to #60

@mattt mattt requested a review from Copilot October 9, 2025 11:21
Copy link

Copilot AI 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 filters out readonly properties from input schemas to prevent them from being included in API request parameters and required fields validation.

  • Adds checks to skip readonly properties when building schema properties
  • Filters out readonly fields from required field lists to avoid validation errors
  • Prevents readonly properties from being included in request body parameters

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@mattt mattt merged commit ab8f42c into main Oct 9, 2025
1 check passed
@mattt mattt deleted the mattt/readonly branch October 9, 2025 11:34
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