fix(sdk): correct asset retrieval in getTopupPayload

Author: Quazia

packages/sdk/src/Incentives/CGDAIncentive.ts

@@ -492,7 +492,7 @@ export class CGDAIncentive extends DeployableTarget<
    * @param {bigint} netAmount The additional tokens to add to `totalBudget`.
    * @returns {Hex} The ABI-encoded, updated CGDAIncentive payload.
    */
-  public getTopupPayload(netAmount: bigint): Hex {
+  public async getTopupPayload(netAmount: bigint): Promise<Hex> {
     return encodeAbiParameters(
       [
         { type: 'address', name: 'asset' },
@@ -503,7 +503,7 @@ export class CGDAIncentive extends DeployableTarget<
         { type: 'address', name: 'manager' },
       ],
       [
-        this.payload?.asset ?? zeroHash,
+        (await this.asset()) ?? zeroHash,
         this.payload?.initialReward ?? 0n,
         this.payload?.rewardDecay ?? 0n,
         this.payload?.rewardBoost ?? 0n,

packages/sdk/src/Incentives/ERC20Incentive.ts

@@ -512,15 +512,16 @@ export class ERC20Incentive extends DeployableTarget<
    * @param {?WriteParams} [params] (optional) if you need them
    * @returns {Hex} the ABI-encoded data for top-up
    */
-  public getTopupPayload(netAmount: bigint): Hex {
+  public async getTopupPayload(netAmount: bigint): Promise<Hex> {
     // 1. Build a typed object matching your `ERC20IncentivePayload`.
     //    For example, you might want to increment `limit` by `netAmount`,
     //    or set `reward = netAmount`. That’s up to your logic:
+    const asset = await this.asset();
     const typed: ERC20IncentivePayload = {
-      asset: this.payload?.asset || zeroAddress,
-      strategy: this.payload?.strategy || StrategyType.POOL, // e.g. StrategyType.POOL
-      reward: netAmount, // store net top-up as the "reward"
-      limit: 1n, // or maybe add netAmount to existing limit
+      asset: asset || zeroAddress,
+      strategy: StrategyType.POOL, // e.g. StrategyType.POOL
+      reward: 1n, // store net top-up as the "reward"
+      limit: netAmount, // or maybe add netAmount to existing limit
       manager: this.payload?.manager || zeroAddress,
     };
 

packages/sdk/src/Incentives/ERC20PeggedIncentive.ts

@@ -487,9 +487,9 @@ export class ERC20PeggedIncentive extends DeployableTarget<
    * @param {bigint} netAmount - The additional limit to add to this incentive.
    * @returns {Hex} The ABI-encoded payload with the updated `limit`.
    */
-  public getTopupPayload(netAmount: bigint): Hex {
+  public async getTopupPayload(netAmount: bigint): Promise<Hex> {
     return prepareERC20PeggedIncentivePayload({
-      asset: this.payload?.asset ?? zeroAddress,
+      asset: (await this.asset()) ?? zeroAddress,
       peg: this.payload?.peg ?? zeroAddress,
       reward: this.payload?.reward ?? 0n,
       limit: netAmount,

packages/sdk/src/Incentives/ERC20PeggedVariableCriteriaIncentive.ts

@@ -622,9 +622,9 @@ export class ERC20PeggedVariableCriteriaIncentive extends DeployableTarget<
    * @param {bigint} netAmount - The additional limit to add to this incentive.
    * @returns {Hex} The ABI-encoded payload with the updated `limit`.
    */
-  public getTopupPayload(netAmount: bigint): Hex {
+  public async getTopupPayload(netAmount: bigint): Promise<Hex> {
     return prepareERC20PeggedVariableCriteriaIncentivePayload({
-      asset: this.payload?.asset ?? zeroAddress,
+      asset: (await this.asset()) ?? zeroAddress,
       peg: this.payload?.peg ?? zeroAddress,
       reward: this.payload?.reward ?? 0n,
       limit: netAmount,

packages/sdk/src/Incentives/ERC20VariableIncentive.ts

@@ -404,12 +404,12 @@ export class ERC20VariableIncentive<
    * @param {bigint} netAmount - The additional limit to add to this incentive.
    * @returns {Hex} The ABI-encoded payload with the updated `limit`.
    */
-  public getTopupPayload(netAmount: bigint): Hex {
+  public async getTopupPayload(netAmount: bigint): Promise<Hex> {
     return prepareERC20VariableIncentivePayload({
-      asset: zeroAddress,
-      reward: 0n,
+      asset: (await this.asset()) as Address,
+      reward: netAmount,
       manager: zeroAddress,
-      limit: netAmount,
+      limit: 1n,
     });
   }