Skip to content

fix: .ts extension error in vite.config.ts#1562

Open
sarthakNITT wants to merge 5 commits into
EpicenterHQ:mainfrom
sarthakNITT:fix/script
Open

fix: .ts extension error in vite.config.ts#1562
sarthakNITT wants to merge 5 commits into
EpicenterHQ:mainfrom
sarthakNITT:fix/script

Conversation

@sarthakNITT

@sarthakNITT sarthakNITT commented Mar 25, 2026

Copy link
Copy Markdown
  • prior script was showing .ts extension error on running bun run dev with node version v20.19.0
  • before:
Screenshot 2026-03-25 at 11 06 44 PM
  • after:
Screenshot 2026-03-25 at 11 08 53 PM Screenshot 2026-03-25 at 11 08 39 PM

Closes #1561

Copilot AI review requested due to automatic review settings March 25, 2026 17:44

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the Whispering app’s dev script to run Vite through Bun, addressing a Node v20 ERR_UNKNOWN_FILE_EXTENSION error when vite.config.ts is loaded during local development (Closes #1561).

Changes:

  • Update apps/whispering’s dev:web script from vite dev to bun --bun vite dev to avoid the .ts config extension error.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread apps/whispering/package.json Outdated
@thisisharsh7

Copy link
Copy Markdown
Member

hey @sarthakNITT thanks for the fix. could you also apply the same changes to these apps?

"dev": "vite dev",
"build": "NODE_ENV=production vite build",
"preview": "vite preview",

"dev": "vite dev",
"build": "NODE_ENV=production vite build",
"preview": "vite preview",

@sarthakNITT

Copy link
Copy Markdown
Author

@thisisharsh7 Done.

@thisisharsh7

Copy link
Copy Markdown
Member

thanks a lot! braden will do the final review 👍

@sarthakNITT

Copy link
Copy Markdown
Author

Sure.

Comment thread apps/fuji/package.json Outdated
Comment thread apps/honeycrisp/package.json Outdated
@sarthakNITT

Copy link
Copy Markdown
Author

@thisisharsh7 Sorry, I missed that. I've fixed code as per review👍

@Leftium

Leftium commented Apr 3, 2026

Copy link
Copy Markdown
Member

Hey @sarthakNITT, thanks for the PR! The root cause here is actually a missing dependency: apps/fuji/vite.config.ts imports from @epicenter/constants/apps, but @epicenter/constants isn't listed in fuji's package.json. Same likely applies to the other apps.

Adding "@epicenter/constants": "workspace:*" to devDependencies and running bun install fixes it without needing bun --bun. The bun --bun flag works around the issue because Bun's resolver is more lenient with workspace packages, but Node's resolver (which Vite uses by default) requires the dependency to be explicitly declared and linked.

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]: extension error while running apps/whispering project locally

4 participants