@boostxyz/sdk 1.1.0-alpha.24 → 2.0.0-alpha.26

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@boostxyz/sdk",
-  "version": "1.1.0-alpha.24",
+  "version": "2.0.0-alpha.26",
   "license": "GPL-3.0-or-later",
   "type": "module",
   "files": [
@@ -169,6 +169,18 @@
       "node": "./dist/Incentives/ERC20Incentive.js",
       "types": "./dist/Incentives/ERC20Incentive.d.ts"
     },
+    "./Incentives/ERC20VariableIncentive": {
+      "require": "./dist/Incentives/ERC20VariableIncentive.cjs",
+      "import": "./dist/Incentives/ERC20VariableIncentive.js",
+      "node": "./dist/Incentives/ERC20VariableIncentive.js",
+      "types": "./dist/Incentives/ERC20VariableIncentive.d.ts"
+    },
+    "./Incentives/ERC20VariableCriteriaIncentive": {
+      "require": "./dist/Incentives/ERC20VariableCriteriaIncentive.cjs",
+      "import": "./dist/Incentives/ERC20VariableCriteriaIncentive.js",
+      "node": "./dist/Incentives/ERC20VariableCriteriaIncentive.js",
+      "types": "./dist/Incentives/ERC20VariableCriteriaIncentive.d.ts"
+    },
     "./Incentives/Incentive": {
       "require": "./dist/Incentives/Incentive.cjs",
       "import": "./dist/Incentives/Incentive.js",
@@ -192,15 +204,19 @@
       "import": "./dist/Validators/SignerValidator.js",
       "node": "./dist/Validators/SignerValidator.js",
       "types": "./dist/Validators/SignerValidator.d.ts"
+    },
+    "./Validators/LimitedSignerValidator": {
+      "require": "./dist/Validators/LimitedSignerValidator.cjs",
+      "import": "./dist/Validators/LimitedSignerValidator.js",
+      "node": "./dist/Validators/LimitedSignerValidator.js",
+      "types": "./dist/Validators/LimitedSignerValidator.d.ts"
     }
   },
   "dependencies": {
-    "ts-pattern": "5.4.0",
-    "@boostxyz/signatures": "1.1.0-alpha.23"
+    "ts-pattern": "5.4.0"
   },
   "optionalDependencies": {
-    "@boostxyz/evm": "1.1.0-alpha.23",
-    "@boostxyz/signatures": "1.1.0-alpha.23"
+    "@boostxyz/evm": "2.0.0-alpha.24"
   },
   "peerDependencies": {
     "@wagmi/core": "2.x",

package/src/Actions/EventAction.ts

@@ -47,6 +47,7 @@ import {
   ValidationAbiMissingError,
 } from '../errors';
 import {
+  CheatCodes,
   type Overwrite,
   type ReadParams,
   RegistryType,
@@ -222,12 +223,13 @@ export interface ActionStep {
  * @property {AbiEvent | AbiFunction} [abiItem] - Optional ABI item definition.
  * @property {EventLogs} [logs] - Event logs to validate against. Required if 'hash' is not provided.
  * @property {Hex} [hash] - Transaction hash to validate against. Required if 'logs' is not provided.
- * @property {number} [chainId] - Chain ID for the transaction. Required if 'hash' is provided.
+ * @property {number} [chainId] - Chain ID for the transaction.
  */
 export type ValidateActionStepParams = {
   knownSignatures: Record<Hex, AbiEvent | AbiFunction>;
   abiItem?: AbiEvent | AbiFunction;
-} & ({ logs: EventLogs } | { hash: Hex; chainId: number });
+  chainId: number;
+} & ({ logs: EventLogs } | { hash: Hex });
 
 /**
  * You can either supply a simplified version of the payload, or one that explicitly declares action steps.
@@ -483,7 +485,7 @@ export class EventAction extends DeployableTarget<
   /**
    * Derives the action claimant address from a transaction based on the provided ActionClaimant configuration.
    * This method supports both event-based and function-based claimant derivation.
-   * **Important**: The claimant is considered to be `transaction.from` when `claimant.fieldIndex` is 255.
+   * **Important**: The claimant is considered to be `transaction.from` when `claimant.fieldIndex` is 255 using CheatCodes enum.
    * This may have unintended side effects for bridged transactions and SCW transactions, so these are considered unsupported use cases for the time being.
    *
    ** @example
@@ -520,10 +522,11 @@ export class EventAction extends DeployableTarget<
   ): Promise<Address | undefined> {
     // find message sender and return it
     // WARNING: this is error prone in bridged transactions and SCW transactions, as this will return exit node
-    if (claimant.fieldIndex === 255) {
+    if (claimant.fieldIndex === CheatCodes.TX_SENDER_CLAIMANT) {
       if ('hash' in params) {
         const transaction = await getTransaction(this._config, {
           hash: params.hash,
+          chainId: params.chainId,
         });
         return transaction.from;
       }
@@ -532,6 +535,7 @@ export class EventAction extends DeployableTarget<
           if (log.transactionHash) {
             const transaction = await getTransaction(this._config, {
               hash: log.transactionHash,
+              chainId: claimant.chainid,
             });
             return transaction.from;
           }
@@ -582,13 +586,10 @@ export class EventAction extends DeployableTarget<
       }
       return address;
     }
-    if (
-      claimant.signatureType === SignatureType.FUNC &&
-      'hash' in params &&
-      'chainId' in params
-    ) {
+    if (claimant.signatureType === SignatureType.FUNC && 'hash' in params) {
       const transaction = await getTransaction(this._config, {
         hash: params.hash,
+        chainId: params.chainId,
       });
       if (!isAddressEqual(transaction.to!, claimant.targetContract)) return;
       let func: AbiFunction;
@@ -698,11 +699,9 @@ export class EventAction extends DeployableTarget<
         return this.isActionEventValid(actionStep, params.logs);
       }
 
-      const client = this._config.getClient({
-        chainId: params.chainId,
-      }) as PublicClient;
-      const receipt = await client.getTransactionReceipt({
+      const receipt = await getTransactionReceipt(this._config, {
         hash: params.hash,
+        chainId: params.chainId,
       });
       const decodedLogs = receipt.logs.map((log) => {
         const { eventName, args } = decodeEventLog({
@@ -718,11 +717,9 @@ export class EventAction extends DeployableTarget<
     }
     if (actionStep.signatureType === SignatureType.FUNC) {
       if ('hash' in params && 'chainId' in params) {
-        const client = this._config.getClient({
-          chainId: params.chainId,
-        }) as PublicClient;
-        const transaction = await client.getTransaction({
+        const transaction = await getTransaction(this._config, {
           hash: params.hash,
+          chainId: params.chainId,
         });
         return this.isActionFunctionValid(actionStep, transaction, params);
       }
@@ -828,7 +825,7 @@ export class EventAction extends DeployableTarget<
     if (
       criteria.filterType === FilterType.EQUAL &&
       criteria.fieldType === PrimitiveType.BYTES &&
-      criteria.fieldIndex === 255
+      criteria.fieldIndex === CheatCodes.ANY_ACTION_PARAM
     ) {
       return true;
     }
@@ -932,7 +929,7 @@ export class EventAction extends DeployableTarget<
 
   /**
    * Validates a {@link Log} against a given criteria.
-   * If the criteria's fieldIndex is 255, it is reserved for anyValidation
+   * If the criteria's fieldIndex is 255 (using CheatCodes enum), it is reserved for anyValidation
    *
    * @param {Criteria} criteria - The criteria to validate against.
    * @param {Log} log - The Viem event log.
@@ -944,7 +941,8 @@ export class EventAction extends DeployableTarget<
   ): boolean {
     if (
       !Array.isArray(log.args) ||
-      (log.args.length <= criteria.fieldIndex && criteria.fieldIndex !== 255)
+      (log.args.length <= criteria.fieldIndex &&
+        criteria.fieldIndex !== CheatCodes.ANY_ACTION_PARAM)
     ) {
       throw new DecodedArgsMalformedError({
         log,
@@ -954,7 +952,9 @@ export class EventAction extends DeployableTarget<
     }
 
     const fieldValue =
-      criteria.fieldIndex === 255 ? zeroHash : log.args.at(criteria.fieldIndex);
+      criteria.fieldIndex === CheatCodes.ANY_ACTION_PARAM
+        ? zeroHash
+        : log.args.at(criteria.fieldIndex);
 
     if (fieldValue === undefined) {
       throw new FieldValueUndefinedError({ log, criteria, fieldValue });
@@ -964,7 +964,7 @@ export class EventAction extends DeployableTarget<
 
   /**
    * Validates a function's decoded arguments against a given criteria.
-   * If the criteria's fieldIndex is 255, it is reserved for anyValidation
+   * If the criteria's fieldIndex is 255 (using CheatCodes enum), it is reserved for anyValidation
    *
    * @param {Criteria} criteria - The criteria to validate against.
    * @param {unknown[]} decodedArgs - The decoded arguments of the function call.
@@ -975,7 +975,9 @@ export class EventAction extends DeployableTarget<
     decodedArgs: readonly (string | bigint)[],
   ): boolean {
     const fieldValue =
-      criteria.fieldIndex === 255 ? zeroHash : decodedArgs[criteria.fieldIndex];
+      criteria.fieldIndex === CheatCodes.ANY_ACTION_PARAM
+        ? zeroHash
+        : decodedArgs[criteria.fieldIndex];
     if (fieldValue === undefined) {
       throw new FieldValueUndefinedError({
         criteria,
@@ -1247,7 +1249,7 @@ export function prepareEventActionPayload({
  * This function returns a Criteria object with the following properties:
  * - filterType: Set to EQUAL for exact matching
  * - fieldType: Set to BYTES to handle any data type
- * - fieldIndex: Set to 255, which is typically used to indicate "any" or "all" in this context
+ * - fieldIndex: Set to 255, which is typically used to indicate "any" or "all" in this context using CheatCodes enum
  * - filterData: Set to zeroHash (0x0000...0000)
  *
  * @returns {Criteria} A Criteria object that can be used to match any action parameter
@@ -1264,7 +1266,7 @@ export function anyActionParameter(): Criteria {
   return {
     filterType: FilterType.EQUAL,
     fieldType: PrimitiveType.BYTES,
-    fieldIndex: 255,
+    fieldIndex: CheatCodes.ANY_ACTION_PARAM,
     filterData: zeroHash,
   };
 }
@@ -1277,7 +1279,7 @@ export function anyActionParameter(): Criteria {
  * The returned ActionClaimant has the following properties:
  * - signatureType: Set to SignatureType.EVENT (though it doesn't matter for this case)
  * - signature: Set to zeroHash (0x0000...0000)
- * - fieldIndex: Set to 255, indicating "any" field
+ * - fieldIndex: Set to 255, indicating "any" field using CheatCodes enum
  * - targetContract: Set to zeroAddress (0x0000...0000)
  * - chainid: Set to 0, indicating it's valid for any chain
  *
@@ -1297,7 +1299,7 @@ export function transactionSenderClaimant(): ActionClaimant {
   return {
     signatureType: SignatureType.EVENT,
     signature: zeroHash,
-    fieldIndex: 255,
+    fieldIndex: CheatCodes.TX_SENDER_CLAIMANT,
     targetContract: zeroAddress,
     chainid: 0,
   };

package/src/BoostCore.test.ts

@@ -17,6 +17,7 @@ import type { ERC20Incentive } from "./Incentives/ERC20Incentive";
 import { StrategyType } from "./claiming";
 import { BoostNotFoundError, IncentiveNotCloneableError } from "./errors";
 import { bytes4 } from "./utils";
+import { BoostValidatorEOA } from "./Validators/Validator";
 
 let fixtures: Fixtures, budgets: BudgetFixtures;
 
@@ -843,7 +844,7 @@ describe("BoostCore", () => {
     expect(validator.payload?.validatorCaller).toBe(core.assertValidAddress());
 
     // expect current account to be a signer
-    const signer = await validator.signers(defaultOptions.account.address);
+    const signer = await validator.signers(BoostValidatorEOA.TESTNET);
     expect(signer).toBeDefined();
     expect(signer).toBe(true);
   });
@@ -867,12 +868,11 @@ describe("BoostCore", () => {
 
     const claimant = trustedSigner.account;
     const incentiveData = pad("0xdef456232173821931823712381232131391321934");
-    const incentiveQuantity = 1;
     const claimDataPayload = await boost.validator.encodeClaimData({
       signer: trustedSigner,
       incentiveData,
       chainId: defaultOptions.config.chains[0].id,
-      incentiveQuantity,
+      incentiveQuantity: boost.incentives.length,
       claimant,
       boostId: boost.id,
     });

package/src/BoostCore.ts

@@ -21,6 +21,7 @@ import { bytecode } from '@boostxyz/evm/artifacts/contracts/BoostCore.sol/BoostC
 import {
   type GetTransactionReceiptParameters,
   getAccount,
+  getChains,
   getTransactionReceipt,
   waitForTransactionReceipt,
 } from '@wagmi/core';
@@ -93,17 +94,26 @@ import {
   PointsIncentive,
   type PointsIncentivePayload,
 } from './Incentives/PointsIncentive';
+import {
+  LimitedSignerValidator,
+  type LimitedSignerValidatorPayload,
+} from './Validators/LimitedSignerValidator';
 import {
   SignerValidator,
   type SignerValidatorPayload,
 } from './Validators/SignerValidator';
-import { type Validator, validatorFromAddress } from './Validators/Validator';
+import {
+  BoostValidatorEOA,
+  type Validator,
+  validatorFromAddress,
+} from './Validators/Validator';
 import {
   BoostCoreNoIdentifierEmitted,
   BoostNotFoundError,
   BudgetMustAuthorizeBoostCore,
   DeployableUnknownOwnerProvidedError,
   IncentiveNotCloneableError,
+  InvalidProtocolChainIdError,
   MustInitializeBudgetError,
 } from './errors';
 import {
@@ -347,8 +357,6 @@ export class BoostCore extends Deployable<
       options,
     );
 
-    console.log(onChainPayload);
-
     const boostHash = await boostFactory(options.config, {
       ...this.optionallyAttachAccount(options.account),
       // biome-ignore lint/suspicious/noExplicitAny: Accept any shape of valid wagmi/viem parameters, wagmi does the same thing internally
@@ -441,9 +449,24 @@ export class BoostCore extends Deployable<
       }
     }
 
+    // If not providing a custom validator, use either Boost's mainnet or testnet EOA, depending on provided chain id and given chain configurations
     if (!payload.validator) {
+      const chains = getChains(options.config).filter(
+        (chain) => !!this.addresses[chain.id] && chain.id === chainId,
+      );
+      const chain = chains.at(0);
+      if (!chain)
+        throw new InvalidProtocolChainIdError(
+          chainId,
+          Object.keys(this.addresses).map(Number),
+        );
+      const testnet = chain.testnet || chain.id === 31337;
       payload.validator = this.SignerValidator({
-        signers: [payload.owner],
+        signers: [
+          (testnet
+            ? BoostValidatorEOA.TESTNET
+            : BoostValidatorEOA.MAINNET) as unknown as Address,
+        ],
         validatorCaller: coreAddress,
       });
     }
@@ -800,7 +823,7 @@ export class BoostCore extends Deployable<
       budget,
       validator,
       allowList,
-      incentives,
+      incentives: incentives as Incentive[],
       protocolFee,
       maxParticipants,
       owner,
@@ -1372,6 +1395,29 @@ export class BoostCore extends Deployable<
     );
   }
 
+  /**
+   * Bound {@link LimitedSignerValidator} constructor that reuses the same configuration as the Boost Core instance.
+   *
+   * @example
+   * ```ts
+   * const validator = core.LimitedSignerValidator({ ... }) // is roughly equivalent to
+   * const validator = new LimitedSignerValidator({ config: core._config, account: core._account }, { ... })
+   * ```
+   * @param {DeployablePayloadOrAddress<LimitedSignerValidatorPayload>} options
+   * @param {?boolean} [isBase]
+   * @returns {LimitedSignerValidator}
+   */
+  LimitedSignerValidator(
+    options: DeployablePayloadOrAddress<LimitedSignerValidatorPayload>,
+    isBase?: boolean,
+  ) {
+    return new LimitedSignerValidator(
+      { config: this._config, account: this._account },
+      options,
+      isBase,
+    );
+  }
+
   /**
    * Bound {@link ERC20VariableCriteriaIncentive} constructor that reuses the same configuration as the Boost Core instance.
    *

package/src/Incentives/AllowListIncentive.test.ts

@@ -1,15 +1,15 @@
-import { loadFixture } from '@nomicfoundation/hardhat-network-helpers';
-import { isAddress, pad, parseEther, zeroAddress } from 'viem';
-import { beforeAll, describe, expect, test } from 'vitest';
-import { accounts } from '@boostxyz/test/accounts';
+import { loadFixture } from "@nomicfoundation/hardhat-network-helpers";
+import { isAddress, pad, parseEther, zeroAddress } from "viem";
+import { beforeAll, describe, expect, test } from "vitest";
+import { accounts } from "@boostxyz/test/accounts";
 import {
   type Fixtures,
   defaultOptions,
   deployFixtures,
   freshBoost,
-} from '@boostxyz/test/helpers';
-import { PointsIncentive } from './PointsIncentive';
-import { Roles } from '../Deployable/DeployableTargetWithRBAC';
+} from "@boostxyz/test/helpers";
+import { PointsIncentive } from "./PointsIncentive";
+import { Roles } from "../Deployable/DeployableTargetWithRBAC";
 
 let fixtures: Fixtures;
 
@@ -25,15 +25,15 @@ function freshAllowList(fixtures: Fixtures) {
   };
 }
 
-describe('AllowListIncentive', () => {
+describe("AllowListIncentive", () => {
   beforeAll(async () => {
     fixtures = await loadFixture(deployFixtures(defaultOptions));
   });
 
-  test('can successfully be deployed', async () => {
+  test("can successfully be deployed", async () => {
     const action = new PointsIncentive(defaultOptions, {
       venue: zeroAddress,
-      selector: '0xdeadb33f',
+      selector: "0xdeadb33f",
       reward: 1n,
       limit: 1n,
     });
@@ -41,7 +41,7 @@ describe('AllowListIncentive', () => {
     expect(isAddress(action.assertValidAddress())).toBe(true);
   });
 
-  test('can claim', async () => {
+  test("can claim", async () => {
     // biome-ignore lint/style/noNonNullAssertion: we know this is defined
     const referrer = accounts.at(1)?.account!;
     // biome-ignore lint/style/noNonNullAssertion: we know this is defined
@@ -63,15 +63,14 @@ describe('AllowListIncentive', () => {
     );
 
     const claimant = trustedSigner.account;
-    const incentiveData = pad('0xdef456232173821931823712381232131391321934');
+    const incentiveData = pad("0xdef456232173821931823712381232131391321934");
     console.log(claimant);
 
-    const incentiveQuantity = 1;
     const claimDataPayload = await boost.validator.encodeClaimData({
       signer: trustedSigner,
       incentiveData,
       chainId: defaultOptions.config.chains[0].id,
-      incentiveQuantity,
+      incentiveQuantity: boost.incentives.length,
       claimant,
       boostId: boost.id,
     });
@@ -82,12 +81,12 @@ describe('AllowListIncentive', () => {
       0n,
       referrer,
       claimDataPayload,
-      { value: parseEther('0.000075'), account: trustedSigner.privateKey },
+      { value: parseEther("0.000075"), account: trustedSigner.privateKey },
     );
     expect(await allowList.isAllowed(trustedSigner.account)).toBe(true);
   });
 
-  test('cannot claim twice', async () => {
+  test("cannot claim twice", async () => {
     // biome-ignore lint/style/noNonNullAssertion: we know this is defined
     const referrer = accounts.at(1)?.account!;
     // biome-ignore lint/style/noNonNullAssertion: we know this is defined
@@ -107,16 +106,15 @@ describe('AllowListIncentive', () => {
       [allowListIncentive.assertValidAddress()],
       [Roles.MANAGER],
     );
-    const incentiveQuantity = 1;
     const claimant = trustedSigner.account;
-    const incentiveData = pad('0xdef456232173821931823712381232131391321934');
+    const incentiveData = pad("0xdef456232173821931823712381232131391321934");
     console.log(claimant);
 
     const claimDataPayload = await boost.validator.encodeClaimData({
       signer: trustedSigner,
       incentiveData,
       chainId: defaultOptions.config.chains[0].id,
-      incentiveQuantity,
+      incentiveQuantity: boost.incentives.length,
       claimant,
       boostId: boost.id,
     });
@@ -126,7 +124,7 @@ describe('AllowListIncentive', () => {
       0n,
       referrer,
       claimDataPayload,
-      { value: parseEther('0.000075'), account: trustedSigner.privateKey },
+      { value: parseEther("0.000075"), account: trustedSigner.privateKey },
     );
     try {
       await fixtures.core.claimIncentive(
@@ -134,7 +132,7 @@ describe('AllowListIncentive', () => {
         0n,
         referrer,
         claimDataPayload,
-        { value: parseEther('0.000075'), account: trustedSigner.privateKey },
+        { value: parseEther("0.000075"), account: trustedSigner.privateKey },
       );
     } catch (e) {
       expect(e).toBeInstanceOf(Error);

package/src/Incentives/CGDAIncentive.test.ts

@@ -1,8 +1,8 @@
-import { readMockErc20BalanceOf } from '@boostxyz/evm';
-import { loadFixture } from '@nomicfoundation/hardhat-network-helpers';
-import { isAddress, pad, parseEther, zeroAddress } from 'viem';
-import { beforeAll, beforeEach, describe, expect, test } from 'vitest';
-import { accounts } from '@boostxyz/test/accounts';
+import { readMockErc20BalanceOf } from "@boostxyz/evm";
+import { loadFixture } from "@nomicfoundation/hardhat-network-helpers";
+import { isAddress, pad, parseEther, zeroAddress } from "viem";
+import { beforeAll, beforeEach, describe, expect, test } from "vitest";
+import { accounts } from "@boostxyz/test/accounts";
 import {
   type BudgetFixtures,
   type Fixtures,
@@ -10,12 +10,12 @@ import {
   deployFixtures,
   freshBoost,
   fundBudget,
-} from '@boostxyz/test/helpers';
-import { CGDAIncentive } from './CGDAIncentive';
+} from "@boostxyz/test/helpers";
+import { CGDAIncentive } from "./CGDAIncentive";
 
 let fixtures: Fixtures, budgets: BudgetFixtures;
 
-describe('CGDAIncentive', () => {
+describe("CGDAIncentive", () => {
   beforeAll(async () => {
     fixtures = await loadFixture(deployFixtures(defaultOptions));
   });
@@ -24,20 +24,20 @@ describe('CGDAIncentive', () => {
     budgets = await loadFixture(fundBudget(defaultOptions, fixtures));
   });
 
-  test('can successfully be deployed', async () => {
+  test("can successfully be deployed", async () => {
     const action = new CGDAIncentive(defaultOptions, {
       asset: budgets.erc20.assertValidAddress(),
       initialReward: 1n,
       totalBudget: 10n,
       rewardBoost: 1n,
       rewardDecay: 1n,
-      manager: budgets.budget.address || zeroAddress 
+      manager: budgets.budget.address || zeroAddress,
     });
     await action.deploy();
     expect(isAddress(action.assertValidAddress())).toBe(true);
   });
 
-  test('can claim', async () => {
+  test("can claim", async () => {
     // biome-ignore lint/style/noNonNullAssertion: we know this is defined
     const referrer = accounts.at(1)!.account!;
     // biome-ignore lint/style/noNonNullAssertion: we know this is defined
@@ -56,14 +56,13 @@ describe('CGDAIncentive', () => {
     });
 
     const claimant = trustedSigner.account;
-    const incentiveData = pad('0xdef456232173821931823712381232131391321934');
+    const incentiveData = pad("0xdef456232173821931823712381232131391321934");
 
-    const incentiveQuantity = 1;
     const claimDataPayload = await boost.validator.encodeClaimData({
       signer: trustedSigner,
       incentiveData,
       chainId: defaultOptions.config.chains[0].id,
-      incentiveQuantity,
+      incentiveQuantity: boost.incentives.length,
       claimant,
       boostId: boost.id,
     });
@@ -73,7 +72,7 @@ describe('CGDAIncentive', () => {
       0n,
       referrer,
       claimDataPayload,
-      { value: parseEther('0.000075') },
+      { value: parseEther("0.000075") },
     );
     expect(
       await readMockErc20BalanceOf(defaultOptions.config, {
@@ -83,7 +82,7 @@ describe('CGDAIncentive', () => {
     ).toBe(1n);
   });
 
-  test('cannot claim twice', async () => {
+  test("cannot claim twice", async () => {
     // biome-ignore lint/style/noNonNullAssertion: we know this is defined
     const referrer = accounts.at(1)!.account!;
     // biome-ignore lint/style/noNonNullAssertion: we know this is defined
@@ -94,7 +93,7 @@ describe('CGDAIncentive', () => {
       totalBudget: 10n,
       rewardBoost: 1n,
       rewardDecay: 1n,
-      manager: budgets.budget.address || zeroAddress 
+      manager: budgets.budget.address || zeroAddress,
     });
     const boost = await freshBoost(fixtures, {
       budget: budgets.budget,
@@ -102,13 +101,12 @@ describe('CGDAIncentive', () => {
     });
 
     const claimant = trustedSigner.account;
-    const incentiveData = pad('0xdef456232173821931823712381232131391321934');
-    const incentiveQuantity = 1;
+    const incentiveData = pad("0xdef456232173821931823712381232131391321934");
     const claimDataPayload = await boost.validator.encodeClaimData({
       signer: trustedSigner,
       incentiveData,
       chainId: defaultOptions.config.chains[0].id,
-      incentiveQuantity,
+      incentiveQuantity: boost.incentives.length,
       claimant,
       boostId: boost.id,
     });
@@ -118,7 +116,7 @@ describe('CGDAIncentive', () => {
       0n,
       referrer,
       claimDataPayload,
-      { value: parseEther('0.000075') },
+      { value: parseEther("0.000075") },
     );
     try {
       await fixtures.core.claimIncentive(
@@ -126,7 +124,7 @@ describe('CGDAIncentive', () => {
         0n,
         referrer,
         claimDataPayload,
-        { value: parseEther('0.000075') },
+        { value: parseEther("0.000075") },
       );
     } catch (e) {
       expect(e).toBeInstanceOf(Error);