@hashgraph/hedera-wallet-connect 1.3.8-canary.9350581.0 → 1.3.8-canary.a321e2e.0

package/dist/lib/dapp/DAppSigner.d.ts

@@ -1,5 +1,6 @@
 import { Signer, AccountBalance, AccountId, AccountInfo, Executable, Key, LedgerId, SignerSignature, Transaction, TransactionRecord } from '@hashgraph/sdk';
 import type { CoreTypes, ISignClient } from '@walletconnect/types';
+import { LogLevel } from '../shared/logger';
 export declare class DAppSigner implements Signer {
     private readonly accountId;
     private readonly signClient;
@@ -7,12 +8,12 @@ export declare class DAppSigner implements Signer {
     private readonly ledgerId;
     readonly extensionId?: string | undefined;
     private logger;
-    constructor(accountId: AccountId, signClient: ISignClient, topic: string, ledgerId?: LedgerId, extensionId?: string | undefined, logLevel?: 'error' | 'warn' | 'info' | 'debug');
+    constructor(accountId: AccountId, signClient: ISignClient, topic: string, ledgerId?: LedgerId, extensionId?: string | undefined, logLevel?: LogLevel);
     /**
      * Sets the logging level for the DAppSigner
      * @param level - The logging level to set
      */
-    setLogLevel(level: 'error' | 'warn' | 'info' | 'debug'): void;
+    setLogLevel(level: LogLevel): void;
     private _getHederaClient;
     private get _signerAccountId();
     private _getRandomNodes;

package/dist/lib/dapp/DAppSigner.js

@@ -21,6 +21,7 @@ import { AccountBalance, AccountId, AccountInfo, LedgerId, SignerSignature, Tran
 import { proto } from '@hashgraph/proto';
 import { HederaJsonRpcMethod, base64StringToSignatureMap, base64StringToUint8Array, ledgerIdToCAIPChainId, queryToBase64String, transactionBodyToBase64String, transactionToBase64String, transactionToTransactionBody, extensionOpen, Uint8ArrayToBase64String, Uint8ArrayToString, } from '../shared';
 import { DefaultLogger } from '../shared/logger';
+import { SessionNotFoundError } from './SessionNotFoundError';
 const clients = {};
 export class DAppSigner {
     constructor(accountId, signClient, topic, ledgerId = LedgerId.MAINNET, extensionId, logLevel = 'debug') {
@@ -60,6 +61,12 @@ export class DAppSigner {
         return allNodes.slice(0, numberOfNodes);
     }
     request(request) {
+        var _a, _b;
+        // Avoid a wallet call if the session is no longer valid
+        if (!((_b = (_a = this === null || this === void 0 ? void 0 : this.signClient) === null || _a === void 0 ? void 0 : _a.session) === null || _b === void 0 ? void 0 : _b.get(this.topic))) {
+            this.logger.error('Session no longer exists. Please reconnect to the wallet.');
+            throw new SessionNotFoundError('Session no longer exists. Please reconnect to the wallet.');
+        }
         if (this.extensionId)
             extensionOpen(this.extensionId);
         return this.signClient.request({
@@ -164,6 +171,10 @@ export class DAppSigner {
     }
     async _tryExecuteTransactionRequest(request) {
         try {
+            // Verify session is still valid before proceeding
+            if (!this.signClient.session.get(this.topic)) {
+                throw new Error('Session no longer exists. Please reconnect to the wallet.');
+            }
             const requestToBytes = request.toBytes();
             this.logger.debug('Creating transaction from bytes', requestToBytes, request);
             const transaction = Transaction.fromBytes(requestToBytes);
@@ -180,6 +191,18 @@ export class DAppSigner {
         }
         catch (error) {
             this.logger.error('Error executing transaction request:', error);
+            if (error instanceof SessionNotFoundError) {
+                this.logger.error('Session was deleted, removing signer');
+                // Notify DAppConnector to remove this signer
+                this.signClient.emit({
+                    topic: this.topic,
+                    event: {
+                        name: 'session_delete',
+                        data: { topic: this.topic },
+                    },
+                    chainId: ledgerIdToCAIPChainId(this.ledgerId),
+                });
+            }
             return { error };
         }
     }
@@ -255,6 +278,7 @@ export class DAppSigner {
             return { result: this._parseQueryResponse(query, result.response) };
         }
         catch (error) {
+            this.logger.error('Error executing query request:', error);
             return { error };
         }
     }

package/dist/lib/dapp/SessionNotFoundError.d.ts

@@ -0,0 +1,3 @@
+export declare class SessionNotFoundError extends Error {
+    constructor(message: string);
+}

package/dist/lib/dapp/SessionNotFoundError.js

@@ -0,0 +1,6 @@
+export class SessionNotFoundError extends Error {
+    constructor(message) {
+        super(message);
+        this.name = 'SessionNotFoundError';
+    }
+}

package/dist/lib/dapp/index.d.ts

@@ -2,6 +2,7 @@ import { AccountId, LedgerId } from '@hashgraph/sdk';
 import { SessionTypes, SignClientTypes } from '@walletconnect/types';
 import { WalletConnectModal } from '@walletconnect/modal';
 import SignClient from '@walletconnect/sign-client';
+import { LogLevel } from '../shared/logger';
 import { GetNodeAddressesResult, ExecuteTransactionParams, ExecuteTransactionResult, SignMessageParams, SignMessageResult, SignAndExecuteQueryResult, SignAndExecuteQueryParams, SignAndExecuteTransactionParams, SignAndExecuteTransactionResult, SignTransactionParams, SignTransactionResult, ExtensionData } from '../shared';
 import { DAppSigner } from './DAppSigner';
 export * from './DAppSigner';
@@ -30,12 +31,12 @@ export declare class DAppConnector {
      * @param chains - Array of supported chains for the DApp (optional).
      * @param logLevel - Logging level for the DAppConnector (optional).
      */
-    constructor(metadata: SignClientTypes.Metadata, network: LedgerId, projectId: string, methods?: string[], events?: string[], chains?: string[], logLevel?: 'error' | 'warn' | 'info' | 'debug');
+    constructor(metadata: SignClientTypes.Metadata, network: LedgerId, projectId: string, methods?: string[], events?: string[], chains?: string[], logLevel?: LogLevel);
     /**
      * Sets the logging level for the DAppConnector
      * @param level - The logging level to set
      */
-    setLogLevel(level: 'error' | 'warn' | 'info' | 'debug'): void;
+    setLogLevel(level: LogLevel): void;
     /**
      * Initializes the DAppConnector instance.
      * @param logger - `BaseLogger` for logging purposes (optional).
@@ -79,9 +80,10 @@ export declare class DAppConnector {
      */
     connectExtension(extensionId: string, pairingTopic?: string): Promise<SessionTypes.Struct>;
     /**
-     * Validates the session by checking if the session exists.
+     * Validates the session by checking if the session exists and is valid.
+     * Also ensures the signer exists for the session.
      * @param topic - The topic of the session to validate.
-     * @returns {boolean} - True if the session exists, false otherwise.
+     * @returns {boolean} - True if the session exists and has a valid signer, false otherwise.
      */
     private validateSession;
     /**

package/dist/lib/dapp/index.js

@@ -113,6 +113,8 @@ export class DAppConnector {
             this.walletConnectClient.on('session_event', this.handleSessionEvent.bind(this));
             this.walletConnectClient.on('session_update', this.handleSessionUpdate.bind(this));
             this.walletConnectClient.on('session_delete', this.handleSessionDelete.bind(this));
+            // Listen for custom session_delete events from DAppSigner
+            this.walletConnectClient.core.events.on('session_delete', this.handleSessionDelete.bind(this));
             this.walletConnectClient.core.pairing.events.on('pairing_delete', this.handlePairingDelete.bind(this));
         }
         catch (e) {
@@ -217,9 +219,10 @@ export class DAppConnector {
         }, pairingTopic, extension.availableInIframe ? undefined : extensionId);
     }
     /**
-     * Validates the session by checking if the session exists.
+     * Validates the session by checking if the session exists and is valid.
+     * Also ensures the signer exists for the session.
      * @param topic - The topic of the session to validate.
-     * @returns {boolean} - True if the session exists, false otherwise.
+     * @returns {boolean} - True if the session exists and has a valid signer, false otherwise.
      */
     validateSession(topic) {
         try {
@@ -227,12 +230,23 @@ export class DAppConnector {
                 return false;
             }
             const session = this.walletConnectClient.session.get(topic);
+            const hasSigner = this.signers.some((signer) => signer.topic === topic);
             if (!session) {
+                // If session doesn't exist but we have a signer for it, clean up
+                if (hasSigner) {
+                    this.logger.warn(`Signer exists but no session found for topic: ${topic}`);
+                    this.handleSessionDelete({ topic });
+                }
+                return false;
+            }
+            if (!hasSigner) {
+                this.logger.warn(`Session exists but no signer found for topic: ${topic}`);
                 return false;
             }
             return true;
         }
-        catch (_a) {
+        catch (e) {
+            this.logger.error('Error validating session:', e);
             return false;
         }
     }
@@ -521,14 +535,24 @@ export class DAppConnector {
     }
     handleSessionDelete(event) {
         this.logger.info('Session deleted:', event);
-        this.signers = this.signers.filter((signer) => signer.topic !== event.topic);
-        try {
-            this.disconnect(event.topic);
-        }
-        catch (e) {
-            this.logger.error('Error disconnecting session:', e);
+        let deletedSigner = false;
+        this.signers = this.signers.filter((signer) => {
+            if (signer.topic !== event.topic) {
+                return true;
+            }
+            deletedSigner = true;
+            return false;
+        });
+        // prevent emitting disconnected event if signers is untouched.
+        if (deletedSigner) {
+            try {
+                this.disconnect(event.topic);
+            }
+            catch (e) {
+                this.logger.error('Error disconnecting session:', e);
+            }
+            this.logger.info('Session deleted and signer removed');
         }
-        this.logger.info('Session deleted by wallet');
     }
     handlePairingDelete(event) {
         this.logger.info('Pairing deleted:', event);

package/dist/lib/shared/logger.d.ts

@@ -4,11 +4,12 @@ export interface ILogger {
     info(message: string, ...args: any[]): void;
     debug(message: string, ...args: any[]): void;
 }
+export type LogLevel = 'error' | 'warn' | 'info' | 'debug' | 'off';
 export declare class DefaultLogger implements ILogger {
     private logLevel;
-    constructor(logLevel?: 'error' | 'warn' | 'info' | 'debug');
-    setLogLevel(level: 'error' | 'warn' | 'info' | 'debug'): void;
-    getLogLevel(): 'error' | 'warn' | 'info' | 'debug';
+    constructor(logLevel?: LogLevel);
+    setLogLevel(level: LogLevel): void;
+    getLogLevel(): LogLevel;
     error(message: string, ...args: any[]): void;
     warn(message: string, ...args: any[]): void;
     info(message: string, ...args: any[]): void;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@hashgraph/hedera-wallet-connect",
-  "version": "1.3.8-canary.9350581.0",
+  "version": "1.3.8-canary.a321e2e.0",
   "description": "A library to facilitate integrating Hedera with WalletConnect",
   "repository": {
     "type": "git",
@@ -28,7 +28,7 @@
     "concurrently": "^9.0.1",
     "esbuild": "^0.24.0",
     "esbuild-plugin-copy": "^2.1.1",
-    "eslint-plugin-tsdoc": "^0.3.0",
+    "eslint-plugin-tsdoc": "^0.4.0",
     "husky": "^9.0.6",
     "jest": "^29.7.0",
     "lint-staged": "^15.1.0",