Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Publish versions of bb.js without WASM stack traces stripped under a new dev tag #5647

Open
Savio-Sou opened this issue Apr 9, 2024 · 0 comments
Labels
A-devops Area: Devops. E.g. CI / build systems C-bb.js Component: bb.js - wrapping bberg in js

Comments

@Savio-Sou
Copy link
Member

Savio-Sou commented Apr 9, 2024

Problem

bb.js is currently published with WASM stack traces stripped before releasing, which makes debugging unreachable errors challenging.

Meanwhile, removing stripping across the board would make the package bulkier in size, possibly 50MB+.

Impact

This initiative currently blocks the further debugging of noir-lang/noir#4409 and noir-lang/noir#4618, which in turn are blocking the development of a number of Noir projects.

Happy Case

Publish bb.js without WASM stack traces stripped under a new dev tag, in parallel to the usual releases with stack traces stripped.

Devs debugging problems would use the dev releases with WASM stack traces, while application production releases use the more optimized latest releases.

Alternative Approach

Publish a separate bb.js-dev package.

The tag approach seems cleaner but both works.

@Savio-Sou Savio-Sou added A-devops Area: Devops. E.g. CI / build systems C-bb.js Component: bb.js - wrapping bberg in js labels Apr 9, 2024
@github-project-automation github-project-automation bot moved this to Todo in A3 Apr 9, 2024
ludamad added a commit that referenced this issue Apr 15, 2024
Side step of
#5647.

This just keeps the debug data for threaded wasm, as the one we expect
to use.
AztecBot pushed a commit to AztecProtocol/barretenberg that referenced this issue Apr 16, 2024
Side step of
AztecProtocol/aztec-packages#5647.

This just keeps the debug data for threaded wasm, as the one we expect
to use.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-devops Area: Devops. E.g. CI / build systems C-bb.js Component: bb.js - wrapping bberg in js
Projects
Status: Todo
Development

No branches or pull requests

1 participant