Page MenuHomePhabricator

[Web] Package opaque-ke-wasm files
ClosedPublic

Authored by jon on Mar 27 2023, 7:34 AM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Apr 8, 11:07 PM
Unknown Object (File)
Mon, Apr 8, 11:07 PM
Unknown Object (File)
Mon, Apr 8, 11:07 PM
Unknown Object (File)
Mon, Apr 8, 11:07 PM
Unknown Object (File)
Mon, Apr 8, 11:04 PM
Unknown Object (File)
Sat, Apr 6, 4:53 AM
Unknown Object (File)
Thu, Apr 4, 2:27 AM
Unknown Object (File)
Mar 8 2024, 6:57 PM
Subscribers

Details

Summary

Include generated wasm files, and add script to create them.

https://linear.app/comm/issue/ENG-3387

Test Plan

Not sure about packaging this with npm, but the script can be tested:

cd web/opaque-ke-wasm/
yarn build
ls pkg/

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

web/opaque-ke-wasm/package.json
8–10 ↗(On Diff #24206)

not sure if this is supposed to be a flat list, or reflect the structure when publishing.

jon retitled this revision from [Web] Package wasm files to [Web] Package opaque-ke-wasm files.

After running yarn arcpatch D7197, I noticed that the commit before this one is D7155. Does this work depend on D7155? If so, can you specify the stack on Phabricator?

Fixes after in work in the subsequent diff

This revision is now accepted and ready to land.Mar 27 2023, 1:18 PM

After running yarn arcpatch D7197, I noticed that the commit before this one is D7155. Does this work depend on D7155? If so, can you specify the stack on Phabricator?

I eventually did this, but thanks for the reminder.

web/opaque-ke-wasm/package.json
7 ↗(On Diff #24232)

should this be removed? we switched main to something else.

web/opaque-ke-wasm/package.json
7 ↗(On Diff #24232)

Good call!! It should be removed from both this list and from Git entirely

jon marked an inline comment as done.

Remove index.js stub

This revision was automatically updated to reflect the committed changes.