Skip to content

Conversation

wayneg123
Copy link
Contributor

@wayneg123 wayneg123 commented Sep 18, 2025

PR Checklist

Please check if your PR fulfills the following requirements:

Bugs / Features

What is the current behavior?

What is the new behavior?

fixes #586

Notes for reviewers

Copy link

changeset-bot bot commented Sep 18, 2025

🦋 Changeset detected

Latest commit: d30ce96

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

This PR includes changesets to release 1 package
Name Type
create-voltagent-app 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

@omeraplak
Copy link
Member

Hey, thanks a lot for the PR. I’m wondering if there’s a way to handle this without using “imports”? Maybe we could solve it in a different way. What do you think?

Copy link
Member

@omeraplak omeraplak left a comment

Choose a reason for hiding this comment

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

Wow looks cool! Thank you

@wayneg123
Copy link
Contributor Author

Haha, you're too fast! I didn't even get a chance to comment on my own PR 😄

You're right to question the imports approach — I had the same thought after sleeping on it. That's why I swapped the generator over to tsdown.

Now, during npm run build, the scaffolded app bundles src/index.ts into a single, clean, Node-friendly ESM file.

Thanks for the quick review and have a nice day!

@omeraplak
Copy link
Member

Haha, you're too fast! I didn't even get a chance to comment on my own PR 😄

You're right to question the imports approach — I had the same thought after sleeping on it. That's why I swapped the generator over to tsdown.

Now, during npm run build, the scaffolded app bundles src/index.ts into a single, clean, Node-friendly ESM file.

Thanks for the quick review and have a nice day!

Great work! Would you also like to update this doc: https://voltagent.dev/docs/quick-start/#manual-setup?
Here’s the source: https://github.com/VoltAgent/voltagent/blob/main/website/docs/getting-started/quick-start.md

@wayneg123
Copy link
Contributor Author

Haha, you're too fast! I didn't even get a chance to comment on my own PR 😄
You're right to question the imports approach — I had the same thought after sleeping on it. That's why I swapped the generator over to tsdown.
Now, during npm run build, the scaffolded app bundles src/index.ts into a single, clean, Node-friendly ESM file.
Thanks for the quick review and have a nice day!

Great work! Would you also like to update this doc: https://voltagent.dev/docs/quick-start/#manual-setup? Here’s the source: https://github.com/VoltAgent/voltagent/blob/main/website/docs/getting-started/quick-start.md

Will do!

Copy link
Member

@omeraplak omeraplak left a comment

Choose a reason for hiding this comment

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

🎖️ Thank you so much!!

@omeraplak omeraplak merged commit 28d4268 into VoltAgent:main Sep 19, 2025
21 checks passed
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.

[BUG] Directory import error prevents npm start after following quick-start guide
2 participants