Skip to content

Conversation

@marc-romu
Copy link
Member

Description

Fixes small json schema request issue with gpt-5 models in scritp_new tool.

Breaking Changes

None.

Testing Done

RH8.21 on windows.

Checklist

  • This PR is focused on a single feature or bug fix
  • Version in Solution.props was updated, if necessary, and follows semantic versioning
  • CHANGELOG.md has been updated
  • PR title follows Conventional Commits format
  • PR description follows Pull Request Description Template

@marc-romu marc-romu added this to the 0.5.3-alpha milestone Aug 20, 2025
Copilot AI review requested due to automatic review settings August 20, 2025 09:07
@github-actions
Copy link
Contributor

🏷️ This PR has been automatically assigned to milestone 0.5.3-alpha based on the version in Solution.props.

Copy link
Contributor

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

Fixes a JSON schema validation issue in the script_new AI tool where required fields were incomplete, causing problems with GPT-5 model requests.

  • Updated JSON schema to include all required fields (description and access) for proper validation
  • Incremented version from 0.5.2-alpha to 0.5.3-alpha
  • Added new AI tool entry in documentation

Reviewed Changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
src/SmartHopper.Core.Grasshopper/AITools/script_new.cs Fixed JSON schema by adding missing required fields for input and output parameters
Solution.props Bumped version to 0.5.3-alpha
README.md Updated version badge and added new gh_connect tool entry
CHANGELOG.md Added changelog entry for the schema fix
.github/labels.yml Added label for new AI GhConnect component

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

@marc-romu marc-romu changed the title Hotfix/0.5.3 alpha fix(script_new): incompatibility with gpt-5 json schema requirements Aug 20, 2025
@marc-romu marc-romu merged commit 27fe074 into main Aug 20, 2025
12 of 13 checks passed
@github-actions github-actions bot deleted the hotfix/0.5.3-alpha branch August 20, 2025 09:09
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.

2 participants