Skip to content

fix: improve parseAs handling and cleanup lint issues#3859

Open
aartisonigra wants to merge 7986 commits into
hey-api:mainfrom
aartisonigra:fix/parseas-type-safe
Open

fix: improve parseAs handling and cleanup lint issues#3859
aartisonigra wants to merge 7986 commits into
hey-api:mainfrom
aartisonigra:fix/parseas-type-safe

Conversation

@aartisonigra

@aartisonigra aartisonigra commented May 9, 2026

Copy link
Copy Markdown

Summary

This PR improves getParseAs handling and fixes related type safety issues.

Changes

  • improved getParseAs() response parsing logic
  • cleaned up unused variable warnings
  • fixed empty catch block lint issue
  • migrated example scripts from require() to ES module imports
  • improved request/response handling consistency
  • fixed lint and formatting issues

Testing

  • pnpm lint:fix
  • pnpm build
  • pnpm test

mrlubos and others added 30 commits April 10, 2026 01:09
@mrlubos
fix: add WithSpecExtensions type
93311f1
@mrlubos
Merge pull request hey-api#3734 from hey-api/fix/spec-types-2
b877338 
fix: add WithSpecExtensions type
@mrlubos
docs: add readme sync
6710a42
@mrlubos
Merge pull request hey-api#3735 from hey-api/refactor/readme-sync
d54995e 
docs: add readme sync
@mrlubos
docs: new changelog
6dde61b
@mrlubos
Fix return value for primitive array entries in transformers
43ff064
@mrlubos
Merge pull request hey-api#3733 from hey-api/copilot/fix-bigint-trans…
44b05bd 
…former-output
@mrlubos
chore: clean up
e244868
@pullfrog
ci: set GH_TOKEN for release notes step
25e6e30
@renovate
chore(deps): update dependency @nuxt/test-utils to v4.0.2
e05f0ef
@renovate
chore(deps): update dependency @orpc/contract to v1.13.14
beddd86
@renovate
chore(deps): update dependency @typescript/native-preview to v7.0.0-d…
535a7c9 
…ev.20260409.1
@mrlubos
chore: clean up
39ad97a
@mrlubos
Merge pull request hey-api#3710 from hey-api/docs/changelog
3b37da1 
docs: new changelog
@mrlubos
ci: update GitHub release action
cb8b6e4
@mrlubos
Merge pull request hey-api#3744 from hey-api/ci/fix-gh-release-action
4cfc177 
ci: update GitHub release action
@mrlubos
ci: gate release notes on changesets publish
c228824
@mrlubos
Merge pull request hey-api#3742 from hey-api/renovate/typescript-nati…
2d92235 
…ve-preview-7.x

chore(deps): update dependency @typescript/native-preview to v7.0.0-dev.20260409.1
@mrlubos
Merge pull request hey-api#3746 from hey-api/ci/fix-changesets-publis…
a1eab87 
…h-notes

ci: gate release notes on changesets publish
@mrlubos
Merge pull request hey-api#3740 from hey-api/renovate/orpc-monorepo
c999ab2 
chore(deps): update dependency @orpc/contract to v1.13.14
@mrlubos
Merge pull request hey-api#3739 from hey-api/renovate/nuxt-test-utils…
36cf2c6 
…-4.x

chore(deps): update dependency @nuxt/test-utils to v4.0.2
@github-actions
ci: release
7743f70
@renovate
chore(deps): update dependency @types/node to v24.12.2
b583dc4
@mrlubos
Merge pull request hey-api#3745 from hey-api/changeset-release/main
6e20279 
ci: release
@mrlubos
Merge pull request hey-api#3741 from hey-api/renovate/node-24.x
663e270 
chore(deps): update dependency @types/node to v24.12.2
@mrlubos
ci: remove app contributors
0028041
@mrlubos
Merge pull request hey-api#3748 from hey-api/ci/fix-release-notes-con…
b31b1d7 
…tributors

ci: remove app contributors
@inas-sirhan
feat: warn when duplicate plugins are specified
e78ce75 
Emit a console warning when the same plugin name appears multiple times
in the plugins array. The last configuration still wins, but users now
get a diagnostic pointing to the root cause of unexpected behavior.

Closes hey-api#3600
@inas-sirhan
refactor: extract duplicate plugin warning into helper and improve wo…
4bc269f 
…rding
@inas-sirhan
test: add test for duplicate plugin warning
80f67f2
pullfrog Bot and others added 15 commits April 30, 2026 03:31
@pullfrog
fix(transformers): use symbol for additionalProperties key variable
e744e29
@mrlubos
Create purple-jars-design.md
d880caa
@mrlubos
Merge pull request hey-api#3841 from hey-api/pullfrog/3838-fix-duplic…
b3c2755 
…ate-key-in-transformers
@pullfrog
chore: add changeset for @hey-api/json-schema-ref-parser
c1b6ac1
@mrlubos
Update bump-json-schema-ref-parser.md
32a298b
@mrlubos
Merge pull request hey-api#3843 from hey-api/pullfrog/3836-bump-json-…
290ef7c 
…schema-ref-parser
@github-actions
ci: release
9e12ff3
@mrlubos
Merge pull request hey-api#3842 from hey-api/changeset-release/main
ad5ff61
@renovate
chore(deps): update dependency @typescript/native-preview to v7.0.0-d…
c375c58 
…ev.20260430.1
@mrlubos
Merge pull request hey-api#3849 from hey-api/renovate/typescript-nati…
1721aa6 
…ve-preview-7.x
@aartisonigra
fix(clients): defer URL construction and thread finalError through in…
50be732 
…terceptors (hey-api#3803)
@aartisonigra
chore: add changeset for interceptor fix
30050dc
@aartisonigra
fix(client-next): defer URL construction and address review feedback
3740600
@aartisonigra
fix: correct getParseAs return type
aa188d3
@aartisonigra
fix: improve parseAs handling and lint cleanup
6ba8843
@bolt-new-by-stackblitz

bolt-new-by-stackblitz Bot commented May 9, 2026

Copy link
Copy Markdown

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

@changeset-bot

changeset-bot Bot commented May 9, 2026 •
edited
Loading

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: b684dac

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

This PR includes changesets to release 6 packages
Name Type
@hey-api/openapi-ts Patch
@hey-api/custom-client Patch
@test/openapi-ts Patch
@hey-api/shared Patch
@hey-api/vite-plugin Patch
@hey-api/openapi-python Patch

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

@vercel

vercel Bot commented May 9, 2026

Copy link
Copy Markdown

@aartisonigra is attempting to deploy a commit to the Hey API Team on Vercel.

A member of the Team first needs to authorize it.

@dosubot dosubot Bot added the size:XL This PR changes 500-999 lines, ignoring generated files. label May 9, 2026
@aartisonigra

aartisonigra commented May 10, 2026

Copy link
Copy Markdown
Author

Can you please review my PR?

@mrlubos

mrlubos commented May 24, 2026

Copy link
Copy Markdown
Member

@aartisonigra please make sure CI checks pass!

mrlubos
mrlubos requested changes May 24, 2026
View reviewed changes

@mrlubos mrlubos left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

please make sure CI checks pass!

@aartisonigra

aartisonigra commented May 24, 2026

Copy link
Copy Markdown
Author

@mrlubos ,The build output is successfully generated and all dist artifacts are present. However, tsdown fails in a post-build “pack” step with a TypeError (reading 'filename'), which appears to be a tooling issue rather than a change introduced in this PR.

@dosubot dosubot Bot added size:XXL This PR changes 1000+ lines, ignoring generated files. and removed size:XL This PR changes 500-999 lines, ignoring generated files. labels May 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mrlubos mrlubos mrlubos requested changes

Assignees

No one assigned

Labels

size:XXL This PR changes 1000+ lines, ignoring generated files.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@aartisonigra @mrlubos @inas-sirhan @sbs44 @SukkaW