Skip to content

Conversation

@assafvayner
Copy link
Collaborator

@assafvayner assafvayner commented Sep 9, 2025

Updates the wasm used by the xet upload portion of the library to include latest fixes to compute_file_hash function to process large files correctly.

Also update the build-xet-wasm.sh script to (either can be reverted just comment below)

  • work on macos (base64 command requires -i flag on macos)
  • adds to the comment at the top of the chunker_wasm_bg.wasm.base64.ts to include some information about when the file was created and with what parameters.

WASM binary is 99KB

Copy link
Member

@coyotte508 coyotte508 left a comment

Choose a reason for hiding this comment

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

LGTM, feel free to merge whenever!

Comment on lines 1 to 7
// Generated by build-xet-wasm.sh
// Repository: $REPO_URL
// Branch: $BRANCH
// Package: $PACKAGE
// JS Target: $JS_TARGET
// Build Date: $(date -u +"%Y-%m-%d %H:%M:%S UTC")

Copy link
Member

Choose a reason for hiding this comment

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

Missing some env vars?

@coyotte508 coyotte508 merged commit cacf243 into main Sep 9, 2025
3 of 5 checks passed
@coyotte508 coyotte508 deleted the assaf/update-xet-file_hash-functionality branch September 9, 2025 16:18
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.

3 participants