Skip to content

WIP: Expose MultichainApiClient provider with a transport adapter without needing to connect first #45

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
jiexi wants to merge 8 commits into
base: main
Choose a base branch
from
Open

WIP: Expose MultichainApiClient provider with a transport adapter without needing to connect first #45

Changes from 1 commit
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
9b39202
add provider proxy
jiexi Nov 21, 2025
80c7884
fix provider getter
jiexi Nov 21, 2025
6cb57fb
update provider proxy
jiexi Nov 21, 2025
3db0e50
use providerProxy instead of actual provider
jiexi Nov 21, 2025
1ebcfeb
add session properties
jiexi Nov 21, 2025
32aca34
add rpc-errors
jiexi Nov 21, 2025
cf8c4d8
replace provider proxy with MultichainApiClientWrapperTransport
jiexi Nov 21, 2025
b55e937
Fix transport check
jiexi Nov 21, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Fix transport check
  • Loading branch information
@jiexi
jiexi committed Nov 21, 2025
commit b55e9376d97fb85a4b67a7f407ca7ee122c24004
16 changes: 12 additions & 4 deletions packages/connect-multichain/src/multichain/transports/multichainApiClientWrapper/index.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -22,6 +22,14 @@ export class MultichainApiClientWrapperTransport implements Transport {
constructor(private multichainSDK: MultichainSDK) {
}

isTransportDefined(): boolean {
try {
return Boolean(this.multichainSDK.transport)
} catch (error) {
return false;
}
}

clearNotificationCallbacks() {
this.notificationCallbacks.clear();
}
Expand Down Expand Up @@ -74,7 +82,7 @@ export class MultichainApiClientWrapperTransport implements Transport {
}

onNotification(callback: (data: unknown) => void) {
if (!this.multichainSDK.transport) {
if (!this.isTransportDefined()) {
this.notificationCallbacks.add(callback);
return () => {
this.notificationCallbacks.delete(callback);
Expand Down Expand Up @@ -108,7 +116,7 @@ export class MultichainApiClientWrapperTransport implements Transport {
}

async #walletGetSession(request: TransportRequestWithId) {
if (!this.multichainSDK.transport) {
if (!this.isTransportDefined()) {
return {
jsonrpc: '2.0',
id: request.id,
Expand All @@ -121,7 +129,7 @@ export class MultichainApiClientWrapperTransport implements Transport {
}

async #walletRevokeSession(request: TransportRequestWithId) {
if (!this.multichainSDK.transport) {
if (!this.isTransportDefined()) {
return { jsonrpc: '2.0', id: request.id, result: true };
}

Expand All @@ -134,7 +142,7 @@ export class MultichainApiClientWrapperTransport implements Transport {
}

async #walletInvokeMethod(request: TransportRequestWithId) {
if (!this.multichainSDK.transport) {
if (!this.isTransportDefined()) {
return { error: providerErrors.unauthorized() }
}
return this.multichainSDK.invokeMethod(request.params as InvokeMethodOptions)
Expand Down