Fixes and tests for assigning protocol-specific context mint actions

src/contextualizers/protocol/highlight/highlight.spec.ts

@@ -90,6 +90,21 @@ describe('Highlight', () => {
     expect(containsBigInt(highlightMultipleMint1.context)).toBe(false);
   });
 
+  it('Should have protocol-specific context action', () => {
+    const highlight1 = detect(
+      highlightErc20_0x1f0c411b as unknown as Transaction,
+    );
+    expect(highlight1).toBe(true);
+
+    const contextualized = generate(
+      highlightErc20_0x1f0c411b as unknown as Transaction,
+    );
+    if (!contextualized.context) {
+      throw new Error('Context is undefined');
+    }
+    expect(contextualized['context']['actions']).toContain('HIGHLIGHT.MINTED');
+  });
+
   it('Should not detect as highlight', () => {
     const highlightMintWithRewards1 = detect(
       catchall0xc35c01ac as unknown as Transaction,

src/contextualizers/protocol/highlight/highlight.ts

@@ -44,6 +44,12 @@ export const generate = (transaction: Transaction): Transaction => {
   transaction.context.summaries.category = 'PROTOCOL_1';
   transaction.context.summaries.en.title = 'Highlight';
 
+  // update context action list
+  transaction.context.actions = [
+    `${Protocols.HIGHLIGHT}.${HighlightContextActionEnum.MINTED}`,
+    ...(transaction.context.actions || []),
+  ];
+
   // check if mint with rewards
   const logs = transaction.logs ?? [];
   let decodedLog;
@@ -62,11 +68,6 @@ export const generate = (transaction: Transaction): Transaction => {
     const mintReferralAmount = decodedLog.args['amount'].toString();
     const mintReferralCurrency = decodedLog.args['currency'].toLowerCase();
 
-    transaction.context.actions = [
-      `${Protocols.HIGHLIGHT}.${HighlightContextActionEnum.MINTED}`,
-      ...(transaction.context.actions || []),
-    ];
-
     transaction.context.variables = {
       ...transaction.context.variables,
       vectorId: {

src/contextualizers/protocol/rodeo/rodeo.spec.ts

@@ -27,4 +27,15 @@ describe('Rodeo Mint', () => {
     );
     expect(zoraMintWithRewards1).toBe(false);
   });
+
+  it('Should have protocol-specific context action', () => {
+    const rodeo1 = detect(rodeo0x3c346a6d as unknown as Transaction);
+    expect(rodeo1).toBe(true);
+
+    const contextualized = generate(rodeo0x3c346a6d as unknown as Transaction);
+    if (!contextualized.context) {
+      throw new Error('Context is undefined');
+    }
+    expect(contextualized['context']['actions']).toContain('RODEO.MINTED');
+  });
 });

src/contextualizers/protocol/rodeo/rodeo.ts

@@ -47,6 +47,12 @@ export const generate = (transaction: Transaction): Transaction => {
   transaction.context.summaries.category = 'PROTOCOL_1';
   transaction.context.summaries.en.title = 'Rodeo';
 
+  // update context action list
+  transaction.context.actions = [
+    `${Protocols.RODEO}.${RodeoContextActionEnum.MINTED}`,
+    ...(transaction.context.actions || []),
+  ];
+
   const logs =
     transaction.logs && transaction.logs.length > 0 ? transaction.logs : [];
   const mintFromFixedPriceSaleLog = findMintFromFixedPriceSaleLog(logs);
@@ -68,10 +74,6 @@ export const generate = (transaction: Transaction): Transaction => {
   )
     return transaction;
 
-  transaction.context.actions = [
-    `${Protocols.RODEO}.${RodeoContextActionEnum.MINTED}`,
-    ...(transaction.context.actions || []),
-  ];
   transaction.context.variables = {
     ...transaction.context.variables,
     numOfEth: {

src/contextualizers/protocol/zoraCreator/zoraCreator.spec.ts

@@ -3,6 +3,7 @@ import { detect, generate } from './zoraCreator';
 import { containsBigInt, contextSummary } from '../../../helpers/utils';
 import mintWithRewards0x6ccb3140 from '../../test/transactions/mintWithRewards-0x6ccb3140.json';
 import zoraMintWithRewards0x837a9a69 from '../../test/transactions/zoraMintWithRewards-0x837a9a69.json';
+import zoraProtocolSpecific0xf811acb9 from '../../test/transactions/zoraProtocolSpecific-0xf811acb9.json';
 import catchall0xc35c01ac from '../../test/transactions/catchall-0xc35c01ac.json';
 
 describe('Zora Mint', () => {
@@ -52,4 +53,19 @@ describe('Zora Mint', () => {
     );
     expect(zoraMintWithRewards1).toBe(false);
   });
+
+  it('Should have protocol-specific context action even with early return', () => {
+    const zoraProtocolSpecific1 = detect(
+      zoraProtocolSpecific0xf811acb9 as unknown as Transaction,
+    );
+    expect(zoraProtocolSpecific1).toBe(true);
+
+    const contextualized = generate(
+      zoraProtocolSpecific0xf811acb9 as unknown as Transaction,
+    );
+    if (!contextualized.context) {
+      throw new Error('Context is undefined');
+    }
+    expect(contextualized['context']['actions']).toContain('ZORA.MINTED');
+  });
 });

src/contextualizers/protocol/zoraCreator/zoraCreator.ts

@@ -50,6 +50,12 @@ export const generate = (transaction: Transaction): Transaction => {
   transaction.context.summaries.category = 'PROTOCOL_1';
   transaction.context.summaries.en.title = 'Zora';
 
+  // update context action list
+  transaction.context.actions = [
+    `${Protocols.ZORA}.${ZoraContextActionEnum.MINTED}`,
+    ...(transaction.context.actions || []),
+  ];
+
   const logs =
     transaction.logs && transaction.logs.length > 0 ? transaction.logs : [];
   const rewardsDepositLog = logs.find(
@@ -73,10 +79,6 @@ export const generate = (transaction: Transaction): Transaction => {
   )
     return transaction;
 
-  transaction.context.actions = [
-    `${Protocols.ZORA}.${ZoraContextActionEnum.MINTED}`,
-    ...(transaction.context.actions || []),
-  ];
   transaction.context.variables = {
     ...transaction.context.variables,
     numOfEth: {