Removed root $ref from GenerateSchemaForType

[eiixy]

#1030

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 87.41%. Comparing base (1bf77f6) to head (fef366a).
Report is 1 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #1033      +/-   ##
==========================================
+ Coverage   87.26%   87.41%   +0.14%     
==========================================
  Files          43       43              
  Lines        2387     2415      +28     
==========================================
+ Hits         2083     2111      +28     
  Misses        284      284              
  Partials       20       20              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[Copilot]

Pull Request Overview

This PR removes the top‐level $ref from generated schemas and flattens definitions, adjusting tests to validate the new behavior.

• Insert logic in GenerateSchemaForType to detect and remove a root $ref, merging its definitions into the root schema.
• Introduce a containsRef helper to detect circular references when flattening.
• Extend TestStructToSchema with new types (Person, MyStructuredResponse, User, Order) and corresponding expected JSON schemas.

Reviewed Changes

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

File	Description
jsonschema/json.go	Implement root $ref removal and add containsRef helper.
jsonschema/json_test.go	Add tests for Person, MyStructuredResponse, User, and Order schema output.

[sashabaranov]

@eiixy please lmk if copilot suggestions are legit, otherwise lgtm!

[eiixy]

@eiixy please lmk if copilot suggestions are legit, otherwise lgtm!

The suggestion about field order isn’t an issue — we convert the structToMap before comparing via reflect.DeepEqual, so the order doesn’t affect the result.