Skip to content

fix(examples): configure Vite to handle bb.js dependencies in browser example#10401

Closed
0xethll wants to merge 2 commits intonoir-lang:masterfrom
0xethll:fix/browser-example
Closed

fix(examples): configure Vite to handle bb.js dependencies in browser example#10401
0xethll wants to merge 2 commits intonoir-lang:masterfrom
0xethll:fix/browser-example

Conversation

@0xethll
Copy link

@0xethll 0xethll commented Nov 6, 2025

Description

Problem*

Resolves #10399

When running yarn dlx vite, the application would hang at "Generating proof..." with no errors. The root cause was that @aztec/bb.js is not excluded from Vite's dependency pre-bundling.

Summary*

Fixes the browser example to work correctly with Vite development server (yarn dlx vite). Previously, the example only worked after building (yarn build) and serving static files (yarn test).

Additional Context

Documentation*

Check one:

  • [ ✅ ] No documentation needed.
  • Documentation included in this PR.
  • [For Experimental Features] Documentation to be submitted in a separate PR.

PR Checklist*

  • [ ✅ ] I have tested the changes locally.
  • I have formatted the changes with Prettier and/or cargo fmt on default settings.

@github-actions
Copy link
Contributor

github-actions bot commented Nov 6, 2025

Thank you for your contribution to the Noir language.

Please do not force push to this branch after the Noir team have started review of this PR. Doing so will only delay us merging your PR as we will need to start the review process from scratch.

Thanks for your understanding.

@0xethll
Copy link
Author

0xethll commented Nov 8, 2025

@TomAFrench Would you have a chance to review?

@TomAFrench
Copy link
Member

BHey @0xethll, thanks for this PR however it seems to have been handled in #10400 so I'm going to close this PR.

@TomAFrench TomAFrench closed this Nov 10, 2025
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.

The browser example is not working

2 participants