Shouldn't this be something that occurs on release and not part of CI?

Should this be taken care of in tlsn-js instead?

Shouldn't this be something that occurs on release and not part of CI?

Should this be taken care of in tlsn-js instead?

Doing it here has the advantage that a build is available for every code change.

This would also allow us to easily add it as an artifact to releases. The tlsn-js job can then simply fetch it.

The automation could look something like this:

name: Create Draft Release

on:
  push:
    tags:
      - 'v*'  # Triggers on version tags like v1.0.0

jobs:
  release:
    runs-on: ubuntu-latest

    steps:
    - name: Download build artifacts
      uses: actions/download-artifact@v4
      with:
        name: tlsn-wasm-pkg

    - uses: ncipollo/release-action@v1
      with:
        artifacts: "tlsn-wasm-pkg.zip"
        name: ${{ github.ref }}
        draft: true
        prerelease: true
        generateReleaseNotes: true

@0xtsukino thoughts on this?

@0xtsukino thoughts on this?

@heeckhau and I discussed after his comment and we agreed at at least for the near future it would be easier for me to simply run the build locally, test them via npm link, and then publish to npm as tlsn-wasm on my machine.

tlsn-js will then just have a dependency on tlsn-wasm. tlsn-wasm will see little to no changes after each release, and tlsn-js might see more integration related bug fixes, and is purposely built for direct integration into tlsn-extension-esque architecture.

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 55.43%. Comparing base (d974fb7) to head (17515eb).
Report is 4 commits behind head on dev.

@@           Coverage Diff           @@
##              dev     #553   +/-   ##
=======================================
  Coverage   55.43%   55.43%           
=======================================
  Files         193      193           
  Lines       20853    20853           
=======================================
  Hits        11560    11560           
  Misses       9293     9293           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.