feat: upgraded to puya-ts 1.0.1 & fixed a registration bug with the escrow factory

Author: kylebeee

assets/arc-0058/projects/arc-0058/package-lock.json

@@ -4,7 +4,7 @@
   "description": "Smart contract deployer",
   "main": "smart_contracts/index.ts",
   "scripts": {
-    "build": "algokit compile ts smart_contracts --output-source-map --out-dir artifacts && algokit generate client smart_contracts/artifacts --output {app_spec_dir}/{contract_name}Client.ts",
+    "build": "algokit compile ts smart_contracts --puya-path=puya --output-source-map --out-dir artifacts && algokit generate client smart_contracts/artifacts --output {app_spec_dir}/{contract_name}Client.ts",
     "deploy": "ts-node-dev --transpile-only --watch .env -r dotenv/config smart_contracts/index.ts",
     "deploy:ci": "ts-node --transpile-only -r dotenv/config smart_contracts/index.ts",
     "lint": "eslint smart_contracts",
@@ -21,7 +21,7 @@
     "npm": ">=9.0"
   },
   "dependencies": {
-    "@algorandfoundation/algorand-typescript": "1.0.0-alpha.75",
+    "@algorandfoundation/algorand-typescript": "^1.0.1",
     "@jest/globals": "^29.7.0",
     "jest": "^29.7.0",
     "ts-jest": "^29.4.0"
@@ -30,8 +30,8 @@
     "@algorandfoundation/algokit-client-generator": "^6.0.1-beta.2",
     "@algorandfoundation/algokit-utils": "^9.1.2",
     "@algorandfoundation/algokit-utils-debug": "^1.0.4",
-    "@algorandfoundation/algorand-typescript-testing": "~1.0.0-beta.30 <1.0.0",
-    "@algorandfoundation/puya-ts": "1.0.0-alpha.75",
+    "@algorandfoundation/algorand-typescript-testing": "^1.0.1",
+    "@algorandfoundation/puya-ts": "^1.0.1",
     "@eslint/js": "^9.18.0",
     "@rollup/plugin-typescript": "^12.1.2",
     "@tsconfig/node22": "^22.0.0",

assets/arc-0058/projects/arc-0058/package.json

@@ -23,8 +23,8 @@ export const DynamicOffsetAndLength: uint64 = 4
 export const DynamicOffset: uint64 = 2
 export const DynamicLength: uint64 = 2
 
-export const MinPluginMBR: uint64 = 36_100
-export const MinNamedPluginMBR: uint64 = 18_900
-export const MinEscrowsMBR: uint64 = 6_100
+export const MinPluginMBR: uint64 = 38_900
+export const MinNamedPluginMBR: uint64 = 21_700
+export const MinEscrowsMBR: uint64 = 6_500
 export const MinAllowanceMBR: uint64 = 27_700
 export const MinExecutionsMBR: uint64 = 20_500

assets/arc-0058/projects/arc-0058/smart_contracts/abstracted_account/constants.ts

@@ -91,20 +91,17 @@ export class AbstractedAccount extends Contract {
         .submit()
     }
 
-    const id = abiCall(
-      EscrowFactory.prototype.new,
-      {
-        sender: this.controlledAddress.value,
-        appId: this.escrowFactory.value,
-        args: [
-          itxn.payment({
-            sender: this.controlledAddress.value,
-            amount: NewCostForARC58 + Global.minBalance,
-            receiver: this.escrowFactory.value.address
-          }),
-        ]
-      }
-    ).returnValue
+    const id = abiCall<typeof EscrowFactory.prototype.new>({
+      sender: this.controlledAddress.value,
+      appId: this.escrowFactory.value,
+      args: [
+        itxn.payment({
+          sender: this.controlledAddress.value,
+          amount: NewCostForARC58 + Global.minBalance,
+          receiver: this.escrowFactory.value.address
+        }),
+      ]
+    }).returnValue
 
     this.escrows(escrow).value = { id, locked: false }
 
@@ -438,19 +435,16 @@ export class AbstractedAccount extends Contract {
       app = this.escrows(escrow).value.id
     }
 
-    abiCall(
-      EscrowFactory.prototype.register,
-      {
-        appId: this.escrowFactory.value,
-        args: [
-          itxn.payment({
-            receiver: this.escrowFactory.value.address,
-            amount: ARC58WalletIDsByAccountsMbr
-          }),
-          app
-        ]
-      }
-    )
+    abiCall<typeof EscrowFactory.prototype.register>({
+      appId: this.escrowFactory.value,
+      args: [
+        itxn.payment({
+          receiver: this.escrowFactory.value.address,
+          amount: ARC58WalletIDsByAccountsMbr
+        }),
+        app
+      ]
+    })
   }
 
   /**
@@ -1000,7 +994,7 @@ export class AbstractedAccount extends Contract {
 
     assert(this.plugins(key).exists, ERR_PLUGIN_DOES_NOT_EXIST)
     assert(this.escrows(escrow).exists, ERR_ESCROW_DOES_NOT_EXIST)
-    assert(this.escrows(escrow).value.locked, ERR_ESCROW_LOCKED)
+    assert(!this.escrows(escrow).value.locked, ERR_ESCROW_LOCKED)
 
     const escrowID = this.escrows(escrow).value.id