chore: smart sessions update ++ (#167)

Author: VGabriel45

CHANGELOG.md

@@ -1,5 +1,11 @@
 # @biconomy/sdk
 
+## 0.0.25
+
+### Patch Changes
+
+- Upgrade smart session and rhinestone sdk version
+
 ## 0.0.24
 
 ### Patch Changes

bun.lockb

@@ -1,6 +1,6 @@
 {
   "name": "@biconomy/sdk",
-  "version": "0.0.24",
+  "version": "0.0.25",
   "author": "Biconomy",
   "repository": "github:bcnmy/sdk",
   "main": "./dist/_cjs/index.js",
@@ -38,12 +38,12 @@
     "viem": "2.21.6",
     "vitest": "^1.3.1",
     "yargs": "^17.7.2",
-    "@rhinestone/module-sdk": "0.1.28"
+    "@rhinestone/module-sdk": ">=0.1.0 <=0.2.3"
   },
   "peerDependencies": {
     "typescript": "^5",
     "viem": "^2.20.0",
-    "@rhinestone/module-sdk": "0.1.28"
+    "@rhinestone/module-sdk": ">=0.1.0 <=0.2.3"
   },
   "exports": {
     ".": {

package.json

@@ -78,7 +78,6 @@ describe("nexus.account.addresses", async () => {
     const counterfactualAddressFromHelper = await getCounterFactualAddress(
       testClient as unknown as PublicClient,
       eoaAccount.address,
-      true,
       0n,
       [RHINESTONE_ATTESTER_ADDRESS],
       1,
@@ -96,7 +95,6 @@ describe("nexus.account.addresses", async () => {
     const counterfactualAddressFromHelper = await getCounterFactualAddress(
       testClient as unknown as PublicClient,
       eoaAccount.address,
-      true,
       0n,
       [RHINESTONE_ATTESTER_ADDRESS],
       1,
@@ -146,6 +144,6 @@ describe("nexus.account.addresses", async () => {
     const testnetAddress = await testnetClient.account.getAddress()
     const mainnetAddress = await mainnetClient.account.getAddress()
 
-    expect(testnetAddress).not.toBe(mainnetAddress)
+    expect(testnetAddress).toBe(mainnetAddress)
   })
 })

src/sdk/account/toNexusAccount.addresses.test.ts

@@ -48,7 +48,6 @@ import {
   ENTRY_POINT_ADDRESS,
   MAINNET_ADDRESS_K1_VALIDATOR_ADDRESS,
   MAINNET_ADDRESS_K1_VALIDATOR_FACTORY_ADDRESS,
-  MOCK_ATTESTER_ADDRESS,
   RHINESTONE_ATTESTER_ADDRESS
 } from "../constants"
 // Constants
@@ -216,7 +215,6 @@ export const toNexusAccount = async (
 
   // Review:
   // Todo: attesters can be added here to do one time setup upon deployment.
-  chain?.testnet && attesters_.push(MOCK_ATTESTER_ADDRESS)
   const factoryData = encodeFunctionData({
     abi: parseAbi([
       "function createAccount(address eoaOwner, uint256 index, address[] attesters, uint8 threshold) external returns (address)"
@@ -255,7 +253,6 @@ export const toNexusAccount = async (
     const addressFromFactory = await getCounterFactualAddress_(
       publicClient,
       signerAddress,
-      false,
       index,
       attesters_,
       attesterThreshold,

src/sdk/account/toNexusAccount.ts

@@ -2,7 +2,6 @@ import type { Address } from "viem"
 import type { PublicClient } from "viem"
 import {
   MAINNET_ADDRESS_K1_VALIDATOR_FACTORY_ADDRESS,
-  MOCK_ATTESTER_ADDRESS,
   RHINESTONE_ATTESTER_ADDRESS
 } from "../../constants"
 
@@ -26,16 +25,11 @@ import {
 export const getCounterFactualAddress = async (
   publicClient: PublicClient,
   signerAddress: Address,
-  isTestnet = false,
   index = 0n,
   attesters = [RHINESTONE_ATTESTER_ADDRESS],
   threshold = 1,
   factoryAddress = MAINNET_ADDRESS_K1_VALIDATOR_FACTORY_ADDRESS
 ) => {
-  if (isTestnet) {
-    attesters.push(MOCK_ATTESTER_ADDRESS)
-  }
-
   return await publicClient.readContract({
     address: factoryAddress,
     abi: [

src/sdk/account/utils/getCounterFactualAddress.ts

@@ -145,6 +145,8 @@ describe("bico.paymaster", async () => {
       bundlerTransport: http(bundlerUrl)
     })
 
+    console.log(nexusClient.account.address, "nexusClient.account.address")
+
     const initialBalance = await publicClient.getBalance({
       address: nexusAccountAddress
     })
@@ -160,6 +162,7 @@ describe("bico.paymaster", async () => {
       feeTokenAddress: baseSepoliaUSDCAddress
     })
     const receipt = await nexusClient.waitForUserOperationReceipt({ hash })
+
     expect(receipt.success).toBe("true")
 
     // Get final balance

src/sdk/clients/createBicoPaymasterClient.test.ts

@@ -1,3 +1,4 @@
+import { isSessionEnabled } from "@rhinestone/module-sdk"
 import { http, type Address, type Chain, type Hex } from "viem"
 import type { LocalAccount, PublicClient } from "viem"
 import { encodeFunctionData } from "viem"
@@ -13,11 +14,7 @@ import {
 } from "../../test/testUtils"
 import type { MasterClient, NetworkConfig } from "../../test/testUtils"
 import { SMART_SESSIONS_ADDRESS, SmartSessionMode } from "../constants"
-import {
-  isPermissionEnabled,
-  parse,
-  stringify
-} from "../modules/smartSessionsValidator/Helpers"
+import { parse, stringify } from "../modules/smartSessionsValidator/Helpers"
 import type {
   CreateSessionDataParams,
   SessionData
@@ -121,7 +118,8 @@ describe("nexus.session.client", async () => {
         actionPoliciesInfo: [
           {
             contractAddress: testAddresses.Counter, // counter address
-            functionSelector: "0x273ea3e3" as Hex // function selector for increment count
+            functionSelector: "0x273ea3e3" as Hex, // function selector for increment count,
+            sudo: true
           }
         ]
       }
@@ -157,9 +155,13 @@ describe("nexus.session.client", async () => {
 
     expect(receipt.success).toBe(true)
 
-    const isEnabled = await isPermissionEnabled({
+    const isEnabled = await isSessionEnabled({
       client: nexusClient.account.client as PublicClient,
-      accountAddress: nexusClient.account.address,
+      account: {
+        type: "nexus",
+        address: nexusClient.account.address,
+        deployedOnChains: [chain.id]
+      },
       permissionId: createSessionsResponse.permissionIds[0]
     })
     expect(isEnabled).toBe(true)
@@ -243,9 +245,13 @@ describe("nexus.session.client", async () => {
       smartSessionUseActions(usePermissionsModule)
     )
 
-    const isEnabled = await isPermissionEnabled({
+    const isEnabled = await isSessionEnabled({
       client: testClient as unknown as PublicClient,
-      accountAddress: nexusClient.account.address,
+      account: {
+        type: "nexus",
+        address: nexusClient.account.address,
+        deployedOnChains: [chain.id]
+      },
       permissionId: sessionData.moduleData.permissionIds[0]
     })
     expect(isEnabled).toBe(true)