[SBA-03] extract WASM functions from LNC into WasmManager class

[jamaljsr]

Summary

This PR moves almost all WASM-related lifecycle, connection, and RPC logic out of the main LNC class into a dedicated WasmManager, while keeping the public LNC API the same. The goal is to keep LNC small and focused as it grows with upcoming session-based auth and passkey features, so new behavior can be added without turning LNC into an unmaintainable “god class.” These changes build on the groundwork from #130 and #131 as part of the Session-Based Authentication & Passkeys work.

Technical Notes

• Keep LNC focused and maintainable: As we add more auth/session logic, the main LNC class would otherwise accumulate a lot of low-level WASM, connection, and RPC details; pushing those concerns into WasmManager keeps LNC closer to a thin façade over the APIs.
• Support future auth features with clearer seams: With a dedicated manager that already handles connection state and credential providers, upcoming session-based auth and passkey features have a clear extension point instead of threading new logic through an already-large core class.

Steps to Test

1. Run the unit tests (including the new WasmManager tests and updated LNC tests):

   yarn test

2. Build the library to ensure the production bundle still compiles correctly:

   yarn build

Related Issues & Pull Requests

• Session-Based Authentication & Passkeys #128

Depends on:

• [SBA-01] chore: normalize indentation to 2 spaces #130
• [SBA-02] update typescript and add typecheck script #131
• [SBA-02b] migrate from deprecated tslint to typescript-eslint #133

[Copilot]

Pull request overview

This PR extracts WASM-related lifecycle, connection, and RPC logic from the LNC class into a new WasmManager class. The refactoring maintains the existing public API while improving maintainability and preparing for future session-based authentication features.

Key Changes:

• Created WasmManager class to encapsulate all WASM operations, connection management, and RPC communication
• Updated LNC class to delegate WASM operations to WasmManager while preserving its public interface
• Moved lncGlobal reference from lnc.ts to wasmManager.ts for better encapsulation

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated no comments.

Show a summary per file

File	Description
lib/wasmManager.ts	New class implementing WASM lifecycle, connection logic, and RPC communication
lib/wasmManager.test.ts	Comprehensive test suite for the new WasmManager class
lib/lnc.ts	Refactored to delegate WASM operations to WasmManager while maintaining public API
lib/lnc.test.ts	Updated tests to reflect delegation pattern and removed implementation detail assertions
test/utils/test-helpers.ts	Updated import path for lncGlobal reference
lib/index.ts	Exported WasmManager for external use
lib/index.test.ts	Added test coverage for WebAssembly polyfill functionality

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review for a chance to win a $100 gift card. Take the survey.

[jbrill]

Great work! I've left quite a few comments (some of which may or may not be helpful). The code reads much cleaner 👏

[jbrill]

Some minor issues:

1. No immediate check — It always waits 500ms before the first check, even if isReady is already true.
2. Magic numbers — 500 and 20 are hardcoded.
3. If the callback takes time to execute, this might not actually measure the 10 seconds.
4. No cancellation possible

Since waitForConnection has the same pattern, we could maybe abstract some of this:

private async pollUntil(
  condition: () => boolean,
  errorMessage: string,
  timeout = 10_000
): Promise<void> {
  const start = Date.now();
  while (!condition()) {
    if (Date.now() - start > timeout) {
      throw new Error(errorMessage);
    }
    await new Promise(r => setTimeout(r, 500));
  }
}

async waitTilReady(): Promise<void> {
  await this.pollUntil(() => this.isReady, 'Failed to load the WASM client');
  log.info('The WASM client is ready');
}

[jbrill]

Should we remove the listener we added in connect?
https://github.com/lightninglabs/lnc-web/pull/132/files#diff-d35515cfe8cd5410830da9527756c4638e092d394fd6c9f5ca5399578db9e37fR212

[jbrill]

I wonder if it makes sense to store our connections to ensure there are no race conditions when this is called twice:

private connectionPromise?: Promise<void>;

async connect(credentialProvider?: CredentialProvider): Promise<void> {
  if (this.isConnected) return;
  
  // Return existing connection attempt if in progress
  if (this.connectionPromise) {
    return this.connectionPromise;
  }
  
  this.connectionPromise = this.doConnect(credentialProvider);
  try {
    await this.connectionPromise;
  } finally {
    this.connectionPromise = undefined;
  }
}

[jbrill]

Would the check for downloaded be if (!this.result) {?

[jbrill]

Should we check if this.result already exists?

[jbrill]

Would it make sense to have the debug level configurable?

[jbrill]

Logging the private keys seems like something we might not want to do 😨

[jbrill]

Might be worth logging if the credential provider doesn't exist here.

[jbrill]

Are we doing anything on auth data? Is this just for debugging?