Skip to content

Conversation

mrlubos
Copy link
Member

@mrlubos mrlubos commented Sep 2, 2025

  • remove all const file = plugin.context.file({ id
  • remove clientId
  • remove sdkId
  • remove typesId
  • remove file.getName()
  • remove file.names
  • fix non-ascii character rendering
  • remove getId()

Copy link

Review PR in StackBlitz Codeflow Run & review this pull request in StackBlitz Codeflow.

Copy link

changeset-bot bot commented Sep 2, 2025

🦋 Changeset detected

Latest commit: f6e3b24

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages
Name Type
@hey-api/openapi-ts Minor
@hey-api/codegen-core Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Copy link

vercel bot commented Sep 2, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
hey-api-docs Ready Ready Preview Comment Sep 9, 2025 7:48pm

@mrlubos mrlubos force-pushed the feat/codegen-symbol branch from 8bd3044 to 84d636d Compare September 3, 2025 00:47
@mrlubos mrlubos force-pushed the feat/codegen-symbol branch from 84d636d to 580d606 Compare September 3, 2025 03:27
@mrlubos mrlubos force-pushed the feat/codegen-symbol branch from 580d606 to 6359673 Compare September 3, 2025 08:28
@mrlubos mrlubos force-pushed the feat/codegen-symbol branch from 6359673 to 0b23c9a Compare September 3, 2025 08:34
@mrlubos mrlubos force-pushed the feat/codegen-symbol branch from 0b23c9a to 6b2d498 Compare September 3, 2025 09:37
@mrlubos mrlubos changed the title feat: symbol api 🪧 Symbol api Sep 3, 2025
@mrlubos mrlubos changed the title 🪧 Symbol api 🪧 Symbol API Sep 3, 2025
@mrlubos mrlubos force-pushed the feat/codegen-symbol branch from 6b2d498 to 9e28790 Compare September 7, 2025 10:33
@mrlubos mrlubos force-pushed the feat/codegen-symbol branch from 9e28790 to 5f04461 Compare September 7, 2025 10:49
@mrlubos mrlubos force-pushed the feat/codegen-symbol branch from 5f04461 to 2db3605 Compare September 7, 2025 14:08
@mrlubos mrlubos force-pushed the feat/codegen-symbol branch from 2db3605 to c3d1fd8 Compare September 7, 2025 14:49
@mrlubos mrlubos force-pushed the feat/codegen-symbol branch from c3d1fd8 to 7ed4347 Compare September 7, 2025 15:39
@mrlubos mrlubos force-pushed the feat/codegen-symbol branch from 7ed4347 to 418a09f Compare September 8, 2025 15:17
@mrlubos mrlubos force-pushed the feat/codegen-symbol branch from 418a09f to 64ca84b Compare September 8, 2025 15:59
@mrlubos mrlubos marked this pull request as ready for review September 9, 2025 18:52
@dosubot dosubot bot added the size:XXL This PR changes 1000+ lines, ignoring generated files. label Sep 9, 2025
@dosubot dosubot bot added the javascript Pull requests that update Javascript code label Sep 9, 2025
Copy link

pkg-pr-new bot commented Sep 9, 2025

Open in StackBlitz

npm i https://pkg.pr.new/hey-api/openapi-ts/@hey-api/codegen-core@2582
npm i https://pkg.pr.new/hey-api/openapi-ts/@hey-api/nuxt@2582
npm i https://pkg.pr.new/hey-api/openapi-ts/@hey-api/openapi-ts@2582
npm i https://pkg.pr.new/hey-api/openapi-ts/@hey-api/vite-plugin@2582

commit: f6e3b24

Copy link

codecov bot commented Sep 9, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 23.79%. Comparing base (a17130e) to head (f6e3b24).
⚠️ Report is 7 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2582      +/-   ##
==========================================
- Coverage   24.14%   23.79%   -0.35%     
==========================================
  Files         363      379      +16     
  Lines       36625    36930     +305     
  Branches     1641     1627      -14     
==========================================
- Hits         8843     8788      -55     
- Misses      27769    28129     +360     
  Partials       13       13              
Flag Coverage Δ
unittests 23.79% <ø> (-0.35%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ 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.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@mrlubos mrlubos force-pushed the feat/codegen-symbol branch from eb7b5c7 to f6e3b24 Compare September 9, 2025 19:48
@mrlubos mrlubos merged commit 9b8efe7 into main Sep 9, 2025
12 of 14 checks passed
@mrlubos mrlubos deleted the feat/codegen-symbol branch September 9, 2025 19:56
@hey-api hey-api bot mentioned this pull request Sep 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
javascript Pull requests that update Javascript code size:XXL This PR changes 1000+ lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant