Fix default import placeholder replacement in TypeScript renderer

[Copilot]

The TypeScript renderer was not properly replacing symbol placeholders in default imports, causing generated code to contain internal placeholder names instead of the resolved symbol names.

Problem:
When generating default imports, the renderer used value.defaultBinding directly without processing it through renderIds to replace symbol placeholders. This resulted in imports like:

import _heyapi_95_ from 'foo';

Instead of the expected:

import foo from 'foo';

Solution:
Applied the same renderIds processing to default bindings that was already being used for namespace bindings. This ensures that symbol placeholders are properly resolved to their final names during import generation.

Changes:

• Modified getImportLines method in packages/openapi-ts/src/generate/renderer.ts to process default binding placeholders
• Added focused test coverage to verify the fix and prevent regressions

Testing:
The fix is verified by a new test that creates a symbol with a placeholder and confirms it gets replaced with the resolved name in the generated import statement.

Fixes #2673.

---

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

[vercel]

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

Project	Deployment	Preview	Comments	Updated (UTC)
hey-api-docs	Ready	Preview	Comment	Sep 22, 2025 1:00pm

[bolt-new-by-stackblitz]

Run & review this pull request in StackBlitz Codeflow.

[changeset-bot]

🦋 Changeset detected

Latest commit: 762b20e

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

This PR includes changesets to release 1 package

Name	Type
@hey-api/openapi-ts	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

[mrlubos]

@copilot I think the problem is that the renderer doesn't handle replacing symbols for default imports yet

[Copilot]

@copilot I think the problem is that the renderer doesn't handle replacing symbols for default imports yet

Exactly right! I've fixed this by adding renderIds processing to default bindings in the getImportLines method, making it consistent with how namespace bindings are handled. The fix is in commit 8098b5f and includes test coverage to verify that default import placeholders are now properly replaced with resolved symbol names.

[pkg-pr-new]

Open in StackBlitz

npm i https://pkg.pr.new/hey-api/openapi-ts/@hey-api/codegen-core@2674

npm i https://pkg.pr.new/hey-api/openapi-ts/@hey-api/nuxt@2674

npm i https://pkg.pr.new/hey-api/openapi-ts/@hey-api/openapi-ts@2674

npm i https://pkg.pr.new/hey-api/openapi-ts/@hey-api/vite-plugin@2674

commit: 762b20e