Skip to content

refactor(codegen): better Swagger compression #1909

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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

gaiaz-iusipov
Copy link
Contributor

No description provided.

@gaiaz-iusipov gaiaz-iusipov requested a review from a team as a code owner February 21, 2025 19:13
@gaiaz-iusipov gaiaz-iusipov marked this pull request as draft February 21, 2025 19:19
@gaiaz-iusipov gaiaz-iusipov marked this pull request as ready for review February 21, 2025 19:38
@jamietanna
Copy link
Member

Thanks for this - is there an issue this solves? Or was this something you noticed and think it's worthwhile doing?

Copy link

kusari-inspector bot commented Jun 15, 2025

Kusari Analysis Results

Analysis for commit: 152681f, performed at: 2025-07-28T17:48:33Z

@kusari-inspector rerun - Trigger a re-analysis of this PR

@kusari-inspector feedback [your message] - Send feedback to our AI and team


Recommendation

✅ PROCEED with this Pull Request

Summary

No Flagged Issues Detected

All values appear to be within acceptable risk parameters.

No pinned version dependency changes, code issues or exposed secrets detected!

Found this helpful? Give it a 👍 or 👎 reaction!

@gaiaz-iusipov
Copy link
Contributor Author

Thanks for this - is there an issue this solves? Or was this something you noticed and think it's worthwhile doing?

Hi! There’s no specific issue, these changes just provide a small performance boost:

  • reduce GC load (depending on the array size)
  • slightly reduce the size of the data in swaggerSpec
  • slightly speed up the decodeSpec() function

Copy link

Kusari PR Analysis rerun based on - 7ceda90 performed at: 2025-06-15T13:56:55Z - link to updated analysis

Copy link

Kusari PR Analysis rerun based on - 549d69f performed at: 2025-07-15T19:09:45Z - link to updated analysis

Copy link

Kusari PR Analysis rerun based on - 65f9a84 performed at: 2025-07-17T09:44:18Z - link to updated analysis

Copy link

Kusari PR Analysis rerun based on - 152681f performed at: 2025-07-28T17:48:33Z - link to updated analysis

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