fix: ensure transaction logging wrapping is disabled with boolean flag

Signed-off-by: Tomás Migone <[email protected]>

Author: tmigone

packages/hardhat-graph-protocol/CHANGELOG.md

@@ -1,5 +1,19 @@
 # hardhat-graph-protocol
 
+## 0.1.15
+
+### Patch Changes
+
+- Updated dependencies
+  - @graphprotocol/toolshed@0.4.2
+
+## 0.1.14
+
+### Patch Changes
+
+- Updated dependencies
+  - @graphprotocol/toolshed@0.4.1
+
 ## 0.1.13
 
 ### Patch Changes

packages/hardhat-graph-protocol/README.md

@@ -145,6 +145,8 @@ An example log output:
 [2025-04-10T20:32:40.946Z]    ✔ Transaction succeeded!
 ```
 
+__Note__ Transaction logging requires using js Proxy which strips down some type definitions from contract methods. This means that when transaction logging is enabled `contract.functionName.estimateGas` for example will not be available.
+
 **Transaction auto-awaiting**
 
 Any transactions made using the `contracts` object will be automatically awaited:

packages/hardhat-graph-protocol/package.json

@@ -1,6 +1,6 @@
 {
   "name": "hardhat-graph-protocol",
-  "version": "0.1.13",
+  "version": "0.1.15",
   "publishConfig": {
     "access": "public"
   },

packages/horizon/package.json

@@ -55,7 +55,7 @@
     "hardhat": "^2.22.18",
     "hardhat-contract-sizer": "^2.10.0",
     "hardhat-gas-reporter": "^1.0.8",
-    "hardhat-graph-protocol": "workspace:^0.1.13",
+    "hardhat-graph-protocol": "workspace:^0.1.15",
     "hardhat-secure-accounts": "^1.0.5",
     "lint-staged": "^15.2.2",
     "prettier": "^3.2.5",

packages/subgraph-service/package.json

@@ -56,7 +56,7 @@
     "hardhat": "^2.22.18",
     "hardhat-contract-sizer": "^2.10.0",
     "hardhat-gas-reporter": "^1.0.8",
-    "hardhat-graph-protocol": "workspace:^0.1.13",
+    "hardhat-graph-protocol": "workspace:^0.1.15",
     "hardhat-secure-accounts": "^1.0.5",
     "json5": "^2.2.3",
     "lint-staged": "^15.2.2",

packages/toolshed/CHANGELOG.md

@@ -1,5 +1,21 @@
 # @graphprotocol/toolshed
 
+## 0.4.2
+
+### Patch Changes
+
+- Ensure all contracts are not wrapped when using connect shortcuts
+  - @graphprotocol/horizon@0.3.2
+  - @graphprotocol/subgraph-service@0.3.4
+
+## 0.4.1
+
+### Patch Changes
+
+- Build before publishing
+  - @graphprotocol/horizon@0.3.2
+  - @graphprotocol/subgraph-service@0.3.4
+
 ## 0.4.0
 
 ### Minor Changes

packages/toolshed/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@graphprotocol/toolshed",
-  "version": "0.4.0",
+  "version": "0.4.2",
   "publishConfig": {
     "access": "public"
   },
@@ -29,6 +29,7 @@
   },
   "scripts": {
     "build": "tsc",
+    "prepublishOnly": "tsc",
     "lint": "eslint '**/*.{js,ts}' --fix",
     "clean": "rm -rf dist"
   },

packages/toolshed/src/deployments/horizon/address-book.ts

@@ -41,7 +41,9 @@ export class GraphHorizonAddressBook extends AddressBook<number, GraphHorizonCon
         ),
         signerOrProvider,
       )
-      contracts.HorizonStaking = wrapTransactionCalls(stakingOverride, 'HorizonStaking')
+      contracts.HorizonStaking = enableTxLogging
+        ? wrapTransactionCalls(stakingOverride, 'HorizonStaking')
+        : stakingOverride
     }
 
     this._assertGraphHorizonContracts(contracts)
@@ -52,11 +54,16 @@ export class GraphHorizonAddressBook extends AddressBook<number, GraphHorizonCon
     contracts.Curation = contracts.L2Curation
     if (contracts.HorizonStaking) {
       // add LegacyStaking alias using old IL2Staking abi
-      contracts.LegacyStaking = wrapTransactionCalls(new Contract(
+      const contract = new Contract(
         contracts.HorizonStaking.target,
         loadArtifact('IL2Staking', GraphHorizonArtifactsMap.LegacyStaking).abi,
         signerOrProvider,
-      ), 'LegacyStaking') as unknown as LegacyStaking
+      )
+      contracts.LegacyStaking = (
+        enableTxLogging
+          ? wrapTransactionCalls(contract, 'LegacyStaking')
+          : contract
+      ) as unknown as LegacyStaking
     }
 
     return contracts

packages/toolshed/src/deployments/subgraph-service/address-book.ts

@@ -7,8 +7,8 @@ import { Contract } from 'ethers'
 import { loadArtifact } from '../artifact'
 import { wrapTransactionCalls } from '../tx-logging'
 
+import type { LegacyDisputeManager, LegacyServiceRegistry } from './types'
 import type { SubgraphServiceContractName, SubgraphServiceContracts } from './contracts'
-import { LegacyDisputeManager, LegacyServiceRegistry } from './types'
 
 export class SubgraphServiceAddressBook extends AddressBook<number, SubgraphServiceContractName> {
   isContractName(name: unknown): name is SubgraphServiceContractName {
@@ -44,21 +44,31 @@ export class SubgraphServiceAddressBook extends AddressBook<number, SubgraphServ
     // Load LegacyDisputeManager manually
     if (this.entryExists('LegacyDisputeManager')) {
       const entry = this.getEntry('LegacyDisputeManager')
-      contractsWithAliases.LegacyDisputeManager = wrapTransactionCalls(new Contract(
+      const contract = new Contract(
         entry.address,
         loadArtifact('IDisputeManager', SubgraphServiceArtifactsMap.LegacyDisputeManager).abi,
         signerOrProvider,
-      ), 'LegacyDisputeManager') as unknown as LegacyDisputeManager
+      )
+      contractsWithAliases.LegacyDisputeManager = (
+        enableTxLogging
+          ? wrapTransactionCalls(contract, 'LegacyDisputeManager')
+          : contract
+      ) as unknown as LegacyDisputeManager
     }
 
     // Load ServiceRegistry manually
     if (this.entryExists('LegacyServiceRegistry')) {
       const entry = this.getEntry('LegacyServiceRegistry')
-      contractsWithAliases.LegacyServiceRegistry = wrapTransactionCalls(new Contract(
+      const contract = new Contract(
         entry.address,
         loadArtifact('IServiceRegistry', SubgraphServiceArtifactsMap.LegacyServiceRegistry).abi,
         signerOrProvider,
-      ), 'LegacyServiceRegistry') as unknown as LegacyServiceRegistry
+      )
+      contractsWithAliases.LegacyServiceRegistry = (
+        enableTxLogging
+          ? wrapTransactionCalls(contract, 'LegacyServiceRegistry')
+          : contract
+      ) as unknown as LegacyServiceRegistry
     }
 
     this._assertSubgraphServiceContracts(contractsWithAliases)