Skip to content

fix: support Bitkit local wallet tests - #11

Draft
ben-kaufman wants to merge 1 commit into
feat/paykit-payment-lock-demofrom
codex/bitkit-local-e2e-fixes
Draft

fix: support Bitkit local wallet tests#11
ben-kaufman wants to merge 1 commit into
feat/paykit-payment-lock-demofrom
codex/bitkit-local-e2e-fixes

Conversation

@ben-kaufman

Copy link
Copy Markdown

Summary

  • make the Paykit Compose demo work with a Bitkit-authenticated creator instead of importing or generating the creator wallet identity in the demo
  • add a loopback-only Homegate bridge and expose Fulcrum so iOS and Android local test builds can sign up and use the regtest wallet backend
  • build the Locks WASM package into the demo images so a host-generated package is no longer required
  • render guarded text, JSON, images, and binary downloads safely without persisting private response bytes in browser storage

This is intentionally stacked on #10 because it hardens and completes that demo's app-wallet test path.

Contract and risk impact

  • Public API or SDK contract
  • Persisted data or migration
  • Authentication, authorization, identity, or secret handling
  • Payment or entitlement behavior
  • Runtime, deployment, or observability
  • No contract/risk impact

Verification

  • npm --prefix examples/js-sdk run check
  • npm --prefix examples/js-sdk run validate:paykit-compose
  • node examples/js-sdk/scripts/smoke-paykit-compose.mjs
  • clean docker compose -f compose.paykit-local-demo.yaml build creator-demo
  • Linux image checks: Paykit reader worker, Paykit Compose smoke, and JS SDK examples smoke
  • clean Compose startup with the creator host port remapped from occupied 8080 to 18080
  • verified every long-running service stayed healthy, reader state was waiting, Homegate returned a valid signup response, creator preflight passed, and Fulcrum answered an Electrum version request

Documentation

Updated examples/js-sdk/README.md for the external-wallet flow, local service endpoints, and runtime reset.

Checklist

  • The change is focused and self-reviewed.
  • Regression tests were added where practical.
  • No credentials, identities, private content, payment material, or generated local state are included.
  • Formatting and relevant tests/lints pass.

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.

1 participant