CLI 'wrap' command needs to build smaller files. · ballercat/walt#164

(1 comment) (0 reactions) (0 assignees)JavaScript (155 forks)batch import

good first issuehelp wanted

Repository metrics

Stars: (4,637 stars)
PR merge metrics: (No merged PRs in 30d)

Description

Problem

walt-cli package when linking multiple .walt files together can "wrap" the modules in a stand-alone JS module. The resulting module is too large because it serialized the dependency tree with the AST information encoded directly into the output, resulting in a massive amount of js.

Encoding the dependencies into the file is done to ensure the module can be used stand-alone in browser or node. Only the opcodes objects should be necessary for each module.

Goal

Patch the cli wrap function to result in a stripped down version of the modules (without AST information) which can still build but only include the bare minimum buffers in the resulting JS module.

Notes

https://github.com/ballercat/walt/blob/master/packages/walt-cli/src/wrap.js - wrap logic
https://github.com/ballercat/walt/blob/master/packages/walt-buildtools/src/index.js#L157 - the "build" method which consumes the stringified tree structure to create a run-able Wasm module.

Contributor guide

Research direction: Inspect wrap.js and index.js to understand AST serialization, then modify wrap to only include opcodes.
Tech stack: javascriptnodejs
Domain: clibuild system
Issue type: Performance
Difficulty: 3
Estimated time: Half day
Activity status: Active
Clarity: Clear
Prerequisites: GitJavaScriptNode.js
Newbie friendliness: 45