chore: update to latest algosdk

Author: neilcampbell

package-lock.json

@@ -60,7 +60,7 @@
     "buffer": "^6.0.3"
   },
   "peerDependencies": {
-    "algosdk": "^3.5.0"
+    "algosdk": "^3.5.1"
   },
   "devDependencies": {
     "@algorandfoundation/tealscript": "^0.106.3",
@@ -75,7 +75,7 @@
     "@types/uuid": "^10.0.0",
     "@typescript-eslint/eslint-plugin": "^8.8.1",
     "@vitest/coverage-v8": "^2.1.2",
-    "algosdk": "^3.5.0",
+    "algosdk": "^3.5.1",
     "better-npm-audit": "^3.11.0",
     "conventional-changelog-conventionalcommits": "8.0.0",
     "cpy-cli": "^5.0.0",

package.json

@@ -1,4 +1,4 @@
-import algosdk, { ABIAddressType, ABIMethod, ABIType, Account, Address } from 'algosdk'
+import algosdk, { ABIMethod, ABIType, Account, Address } from 'algosdk'
 import invariant from 'tiny-invariant'
 import { afterAll, beforeAll, beforeEach, describe, expect, test } from 'vitest'
 import { APP_SPEC as nestedContractAppSpec } from '../../tests/example-contracts/client/TestContractClient'
@@ -1161,7 +1161,6 @@ describe('abi return', () => {
   })
 })
 
-// TODO: NC - Switch these tests to not use bare.send(...) once algosdk supports access references
 describe('access references', () => {
   const fixture = algorandFixture()
   beforeEach(fixture.newScope)
@@ -1231,52 +1230,38 @@ describe('access references', () => {
   }, 20_000) // Account generation and funding can be slow
 
   test('address reference enables access', async () => {
-    const alice = await fixture.context.generateAccount({ initialFunds: AlgoAmount.Algo(0.1) })
-
-    const method = appClient.getABIMethod('addressBalance').getSelector()
-
-    const addressType = new ABIAddressType()
-    const aliceAddr = addressType.encode(alice)
-
-    await appClient.send.bare.call({ args: [method, aliceAddr], populateAppCallResources: false, accessReferences: [{ address: alice }] })
+    await appClient.send.call({
+      method: 'addressBalance',
+      args: [alice],
+      populateAppCallResources: false,
+      accessReferences: [{ address: alice }],
+    })
   })
 
   test('up to 8 non access reference accounts can be used', async () => {
-    const method = appClient.getABIMethod('addressBalance').getSelector()
-
-    const addressType = new ABIAddressType()
-    const aliceAddr = addressType.encode(alice)
-
-    await appClient.send.bare.call({
-      args: [method, aliceAddr],
+    await appClient.send.call({
+      method: 'addressBalance',
+      args: [alice],
       populateAppCallResources: false,
       accountReferences: [alice, bob, charlie, dan, eve, frank, grace, heidi],
     })
   })
 
   test('throws when more than 8 non access reference accounts are supplied', async () => {
-    const method = appClient.getABIMethod('addressBalance').getSelector()
-
-    const addressType = new ABIAddressType()
-    const aliceAddr = addressType.encode(alice)
-
     await expect(
-      appClient.send.bare.call({
-        args: [method, aliceAddr],
+      appClient.send.call({
+        method: 'addressBalance',
+        args: [alice],
         populateAppCallResources: false,
         accountReferences: [alice, bob, charlie, dan, eve, frank, grace, heidi, ivan],
       }),
     ).rejects.toThrow(/max number of accounts is 8/)
   })
 
   test('up to 16 access addresses can be used', async () => {
-    const method = appClient.getABIMethod('addressBalance').getSelector()
-
-    const addressType = new ABIAddressType()
-    const aliceAddr = addressType.encode(alice)
-
-    await appClient.send.bare.call({
-      args: [method, aliceAddr],
+    await appClient.send.call({
+      method: 'addressBalance',
+      args: [alice],
       populateAppCallResources: false,
       accessReferences: [
         { address: alice },
@@ -1300,14 +1285,10 @@ describe('access references', () => {
   })
 
   test('throws when more than 16 access addresses are supplied', async () => {
-    const method = appClient.getABIMethod('addressBalance').getSelector()
-
-    const addressType = new ABIAddressType()
-    const aliceAddr = addressType.encode(alice)
-
     await expect(
-      appClient.send.bare.call({
-        args: [method, aliceAddr],
+      appClient.send.call({
+        method: 'addressBalance',
+        args: [alice],
         populateAppCallResources: false,
         accessReferences: [
           { address: alice },
@@ -1333,32 +1314,28 @@ describe('access references', () => {
   })
 
   test('app reference enables access', async () => {
-    const method = appClient.getABIMethod('externalAppCall').getSelector()
-
-    await appClient.send.bare.call({
-      args: [method],
+    await appClient.send.call({
+      method: 'externalAppCall',
       populateAppCallResources: false,
       accessReferences: [{ appId: externalClient.appId }],
       staticFee: microAlgo(2000),
     })
   })
 
   test('asset reference enables access', async () => {
-    const method = appClient.getABIMethod('assetTotal').getSelector()
     const assetId = (await appClient.getGlobalState()).asa.value as bigint
 
-    await appClient.send.bare.call({
-      args: [method],
+    await appClient.send.call({
+      method: 'assetTotal',
       populateAppCallResources: false,
       accessReferences: [{ assetId }],
     })
   })
 
   test('box reference enables access', async () => {
-    const method = appClient.getABIMethod('smallBox').getSelector()
-
-    await appClient.send.bare.call({
-      args: [method],
+    await appClient.send.call({
+      method: 'smallBox',
+      args: [],
       populateAppCallResources: false,
       accessReferences: [{ box: { appId: appClient.appId, name: new Uint8Array([115]) } }],
     })
@@ -1368,13 +1345,9 @@ describe('access references', () => {
     const alice = await fixture.context.generateAccount({ initialFunds: AlgoAmount.Algo(0.1) })
     const assetId = (await appClient.getGlobalState()).asa.value as bigint
 
-    const method = appClient.getABIMethod('hasAsset').getSelector()
-
-    const addressType = new ABIAddressType()
-    const aliceAddr = addressType.encode(alice)
-
-    await appClient.send.bare.call({
-      args: [method, aliceAddr],
+    await appClient.send.call({
+      method: 'hasAsset',
+      args: [alice],
       populateAppCallResources: false,
       accessReferences: [{ holding: { address: alice, assetId } }],
     })
@@ -1386,13 +1359,9 @@ describe('access references', () => {
     // Opt alice into the external app
     await fixture.algorand.send.appCallMethodCall(await externalClient.params.optIn({ method: 'optInToApplication', sender: alice }))
 
-    const method = appClient.getABIMethod('externalLocal').getSelector()
-
-    const addressType = new ABIAddressType()
-    const aliceAddr = addressType.encode(alice)
-
-    await appClient.send.bare.call({
-      args: [method, aliceAddr],
+    await appClient.send.call({
+      method: 'externalLocal',
+      args: [alice],
       populateAppCallResources: false,
       accessReferences: [{ locals: { address: alice, appId: externalClient.appId } }],
     })

src/transaction/transaction.spec.ts

@@ -1636,7 +1636,7 @@ export class TransactionComposer {
       appForeignApps: params.appReferences?.map((x) => Number(x)),
       appForeignAssets: params.assetReferences?.map((x) => Number(x)),
       boxes: params.boxReferences?.map(AppManager.getBoxReference),
-      access: params.accessReferences?.map(getAccessReference), // TODO: NC - This is currently not supported in algosdk
+      access: params.accessReferences?.map(getAccessReference),
       approvalProgram,
       clearProgram: clearStateProgram,
       extraPages: