Fix/blockchain compatibility check (#31)

* fix: blockchain comp check

* add monad devnet

* temp: override confirm tx in action config

* fix: compatibility checks on changing action adapter

* Revert "temp: override confirm tx in action config"

This reverts commit 1069f85.

* fix: stale adapter

* add comment

* harmonize update handling

* restore setAdapter + add comment for react env

* harmonize action update

Author: tsmbl

packages/blinks-core/src/api/Action/Action.ts

@@ -174,6 +174,11 @@ export class Action {
     return this._adapter;
   }
 
+  /**
+   * Set the adapter for the action.
+   * Not recommended to use in react environments, consider using Action.update() instead that returns a new instance.
+   * @param adapter The adapter to set
+   */
   public setAdapter(adapter: ActionAdapter) {
     this._adapter = adapter;
   }
@@ -342,6 +347,22 @@ export class Action {
       this._id,
     );
   }
+
+  withUpdate(update: {
+    adapter?: ActionAdapter;
+    supportStrategy?: ActionSupportStrategy;
+  }) {
+    return new Action(
+      this._url,
+      this._data,
+      this._metadata,
+      update.supportStrategy ?? this._supportStrategy,
+      update.adapter ?? this._adapter,
+      this._chainMetadata,
+      this._id,
+      this._experimental,
+    );
+  }
 }
 
 const getActionMetadata = (response: Response): ActionMetadata => {

packages/blinks-core/src/hooks/useAction.ts

@@ -53,6 +53,7 @@ export function useAction({
   const { actionApiUrl } = useActionApiUrl(url);
   const [action, setAction] = useState<Action | null>(null);
   const [isLoading, setIsLoading] = useState(false);
+  const [hasFetched, setHasFetched] = useState(false);
 
   useEffect(() => {
     setIsLoading(true);
@@ -61,12 +62,14 @@ export function useAction({
     }
 
     let ignore = false;
+    setHasFetched(false);
     Action.fetch(actionApiUrl, undefined, supportStrategy)
       .then((action) => {
         if (ignore) {
           return;
         }
         setAction(action);
+        setHasFetched(true);
       })
       .catch((e) => {
         console.error('[@dialectlabs/blinks-core] Failed to fetch action', e);
@@ -81,11 +84,26 @@ export function useAction({
     return () => {
       ignore = true;
     };
-  }, [actionApiUrl, isRegistryLoaded, supportStrategy]);
+    // eslint-disable-next-line react-hooks/exhaustive-deps -- only update if actionApiUrl changes
+  }, [actionApiUrl, isRegistryLoaded]);
 
+  // this effect handles race conditions between fetching the action adapter change
+  // hasFetched dependency is used instead of action dependency to ensure there's no infinite loop
   useEffect(() => {
-    action?.setAdapter(adapter);
-  }, [action, adapter]);
+    if (!action || !hasFetched) {
+      return;
+    }
+    try {
+      const updated = action.withUpdate({
+        adapter,
+        supportStrategy,
+      });
+      setAction(updated);
+    } catch (e) {
+      console.error('[@dialectlabs/blinks-core] Failed to update action', e);
+    }
+    // eslint-disable-next-line react-hooks/exhaustive-deps -- only update if adapter or supportStrategy changes
+  }, [adapter, supportStrategy, hasFetched]);
 
   return { action, isLoading };
 }

packages/blinks-core/src/utils/caip-2.ts

@@ -6,12 +6,14 @@ export const BlockchainIds = {
   SOLANA_DEVNET: 'solana:EtWTRABZaYq6iMfeYKouRu166VU2xqa1',
   SOLANA_TESTNET: 'solana:4uhcVJyU9pJkvQyS88uRDiswHXSCkY3',
   ETHEREUM_MAINNET: 'eip155:1',
+  MONAD_DEVNET: 'eip155:41454',
 };
 const BlockchainNames: Record<string, string> = {
   [BlockchainIds.SOLANA_MAINNET]: 'Solana Mainnet',
   [BlockchainIds.SOLANA_DEVNET]: 'Solana Devnet',
   [BlockchainIds.SOLANA_TESTNET]: 'Solana Testnet',
   [BlockchainIds.ETHEREUM_MAINNET]: 'Ethereum Mainnet',
+  [BlockchainIds.MONAD_DEVNET]: 'Monad Devnet',
 };
 
 export function getShortBlockchainName(id: string) {