Skip to content

Conversation

@mdrafi28
Copy link
Contributor

@mdrafi28 mdrafi28 commented Oct 23, 2025

I did understand the schema structure and accordingly i have written the index.d.ts which corresponds to the schema: https://spec.openapis.org/oas/v3.2.0.html

i have checked and rewritten schema manually and these changes i observed from 3.1 and revised it in index.d.ts to correspond properly:

  1. unknown (any) -> json
  2. schema -> OasSchema32
  3. the structure of SecurityScheme

Thank you for the patiently reviewing.

Copy link
Collaborator

@jdesrosiers jdesrosiers left a comment

Choose a reason for hiding this comment

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

This is much better. Thank you! I found a few more issues when doing a detailed review and went ahead and pushed the changed for what I found.

@jdesrosiers jdesrosiers merged commit c4117ea into hyperjump-io:main Oct 24, 2025
2 checks passed
@mdrafi28 mdrafi28 deleted the PR-82 branch October 31, 2025 13:21
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