Skip to content

Conversation

@adklempner
Copy link
Member

@adklempner adklempner commented Nov 20, 2025

Problem / Description

The browser-tests package was using an older version of @waku/sdk that wasn't using LightPush v3. This broke expected behavior in testing environment.

Solution

Update browser-tests to use latest versions of other js-waku packages

Also adds nwaku-style log upon successfully sending light push message

Notes


Checklist

  • Code changes are covered by unit tests.
  • Code changes are covered by e2e tests, if applicable.
  • Dogfooding has been performed, if feasible.
  • A test version has been published, if required.
  • All CI checks pass successfully.

@github-actions
Copy link

github-actions bot commented Nov 20, 2025

size-limit report 📦

Path Size Loading time (3g) Running time (snapdragon) Total time
Waku node 96.24 KB (0%) 2 s (0%) 566 ms (+23.11% 🔺) 2.5 s
Waku Simple Light Node 147.6 KB (0%) 3 s (0%) 548 ms (+55.49% 🔺) 3.5 s
ECIES encryption 22.62 KB (0%) 453 ms (0%) 261 ms (+208.93% 🔺) 713 ms
Symmetric encryption 22 KB (0%) 440 ms (0%) 228 ms (+22.55% 🔺) 667 ms
DNS discovery 52.17 KB (0%) 1.1 s (0%) 375 ms (+52.53% 🔺) 1.5 s
Peer Exchange discovery 52.91 KB (0%) 1.1 s (0%) 443 ms (+81.4% 🔺) 1.6 s
Peer Cache Discovery 46.64 KB (0%) 933 ms (0%) 315 ms (+89.66% 🔺) 1.3 s
Privacy preserving protocols 77.31 KB (0%) 1.6 s (0%) 390 ms (+30.73% 🔺) 2 s
Waku Filter 79.82 KB (0%) 1.6 s (0%) 542 ms (+75.12% 🔺) 2.2 s
Waku LightPush 77.97 KB (0%) 1.6 s (0%) 343 ms (+18.19% 🔺) 2 s
History retrieval protocols 83.74 KB (0%) 1.7 s (0%) 289 ms (-32.93% 🔽) 2 s
Deterministic Message Hashing 28.98 KB (0%) 580 ms (0%) 179 ms (-3.08% 🔽) 759 ms

@adklempner adklempner force-pushed the feat/bump-deps-browser-image branch from 8b0c1e0 to 89f4a6d Compare December 2, 2025 02:32
@adklempner adklempner changed the title feat: update waku dependencies in browser tests feat(browser-tests): update waku dependencies, add nwaku-style log for light push Dec 2, 2025
@adklempner adklempner marked this pull request as ready for review December 2, 2025 02:33
@adklempner adklempner requested a review from weboko December 2, 2025 03:41
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.

2 participants