From bb5558713b5bbf8cef478a49bcc2ccbe5f9a9784 Mon Sep 17 00:00:00 2001
From: Ted Ian Osias <osiastedian@gmail.com>
Date: Thu, 3 Oct 2024 21:52:40 +0800
Subject: [PATCH] ci: multiplee docker builds

---
 .github/workflows/ghcr.yaml                   | 40 +++++++++++++++----
 components/Bridge/NEVMStepWrapepr.tsx         |  5 +--
 .../ConnectValidate/StartTransferButton.tsx   |  3 +-
 components/Bridge/UTXOStepWrapper.tsx         |  4 +-
 components/Bridge/hooks/useFeatureFlags.ts    |  2 -
 constants.ts                                  |  5 +--
 contexts/PaliWallet/V2Provider.tsx            |  9 ++---
 contexts/Transfer/constants.ts                |  8 ++--
 8 files changed, 45 insertions(+), 31 deletions(-)

diff --git a/.github/workflows/ghcr.yaml b/.github/workflows/ghcr.yaml
index 46584c7..41376d0 100644
--- a/.github/workflows/ghcr.yaml
+++ b/.github/workflows/ghcr.yaml
@@ -3,30 +3,49 @@ name: Github Image Build and Push
 on:
   push:
     branches:
-      - 'main'
-      - 'testnet-support'
+      - "main"
+      - "testnet-support"
 
 jobs:
   docker:
-    # i f: github.event_name != 'pull_request' && github.ref == 'refs/heads/main'
     permissions:
       contents: read
       packages: write
     runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        network:
+          - mainnet
+          - testnet
     steps:
       - name: Set up QEMU
         uses: docker/setup-qemu-action@v2
 
       - name: Set up Docker Buildx
         uses: docker/setup-buildx-action@v2
+
+      - name: Set environment variables
+        run: |
+          if [ "${{ matrix.network }}" == "mainnet" ]; then
+            echo "NEXT_PUBLIC_NEVM_RPC_URL=https://rpc.syscoin.org" >> $GITHUB_ENV
+            echo "NEXT_PUBLIC_IS_TESTNET=false" >> $GITHUB_ENV
+            echo "NEXT_PUBLIC_NEVM_EXPLORER_URL=https://explorer.syscoin.org" >> $GITHUB_ENV
+            echo "NEXT_PUBLIC_CHAIN_ID=57" >> $GITHUB_ENV
+            echo "NEXT_PUBLIC_BLOCKBOOK_API_URL=https://blockbook.syscoin.org" >> $GITHUB_ENV
+          elif [ "${{ matrix.network }}" == "testnet" ]; then
+            echo "NEXT_PUBLIC_NEVM_RPC_URL=https://rpc.tanenbaum.io" >> $GITHUB_ENV
+            echo "NEXT_PUBLIC_IS_TESTNET=true" >> $GITHUB_ENV
+            echo "NEXT_PUBLIC_NEVM_EXPLORER_URL=https://tanenbaum.io" >> $GITHUB_ENV
+            echo "NEXT_PUBLIC_CHAIN_ID=5700" >> $GITHUB_ENV
+            echo "NEXT_PUBLIC_BLOCKBOOK_API_URL=https://blockbook-dev.syscoin.org" >> $GITHUB_ENV
+          fi
+
       - name: Docker meta
         id: meta
         uses: docker/metadata-action@v4
         with:
-          # list of Docker images to use as base name for tags
           images: |
-            ghcr.io/syscoin/syscoin-bridge
-          # generate Docker tags based on the following events/attributes
+            ghcr.io/syscoin/syscoin-bridge-${{ matrix.network }}
           tags: |
             type=schedule
             type=ref,event=branch
@@ -35,6 +54,7 @@ jobs:
             type=semver,pattern={{major}}.{{minor}}
             type=semver,pattern={{major}}
             type=sha
+
       - name: Login to GitHub Container Registry
         uses: docker/login-action@v2
         with:
@@ -45,9 +65,16 @@ jobs:
       - name: Build and push
         uses: docker/build-push-action@v4
         with:
+          context: .
           push: ${{ github.event_name != 'pull_request' }}
           tags: ${{ steps.meta.outputs.tags }}
           labels: ${{ steps.meta.outputs.labels }}
+          build-args: |
+            NEXT_PUBLIC_NEVM_RPC_URL=$NEXT_PUBLIC_NEVM_RPC_URL
+            NEXT_PUBLIC_IS_TESTNET=$NEXT_PUBLIC_IS_TESTNET
+            NEXT_PUBLIC_NEVM_EXPLORER_URL=$NEXT_PUBLIC_NEVM_EXPLORER_URL
+            NEXT_PUBLIC_CHAIN_ID=$NEXT_PUBLIC_CHAIN_ID
+            NEXT_PUBLIC_BLOCKBOOK_API_URL=$NEXT_PUBLIC_BLOCKBOOK_API_URL
 
       - name: Write metadata
         run: "cp $DOCKER_METADATA_OUTPUT_BAKE_FILE metadata.json"
@@ -57,4 +84,3 @@ jobs:
         with:
           name: metadata
           path: metadata.json
-
diff --git a/components/Bridge/NEVMStepWrapepr.tsx b/components/Bridge/NEVMStepWrapepr.tsx
index 4ac02be..fbe6423 100644
--- a/components/Bridge/NEVMStepWrapepr.tsx
+++ b/components/Bridge/NEVMStepWrapepr.tsx
@@ -2,7 +2,7 @@ import { useNEVM } from "@contexts/ConnectedWallet/NEVMProvider";
 import { usePaliWalletV2 } from "@contexts/PaliWallet/usePaliWallet";
 import { Button } from "@mui/material";
 import { isValidEthereumAddress } from "@pollum-io/sysweb3-utils";
-import { useFeatureFlags } from "./hooks/useFeatureFlags";
+import { CHAIN_ID } from "@constants";
 
 type Props = {
   children: React.ReactNode;
@@ -11,7 +11,6 @@ type Props = {
 const NEVMStepWrapper: React.FC<Props> = ({ children }) => {
   const { version, isBitcoinBased, switchTo, isEVMInjected, connectWallet } =
     usePaliWalletV2();
-  const { chainId: expectedChainId } = useFeatureFlags();
 
   const { connect, account, chainId, switchToMainnet } = useNEVM();
 
@@ -35,7 +34,7 @@ const NEVMStepWrapper: React.FC<Props> = ({ children }) => {
     );
   }
 
-  if (chainId !== `0x${Number(expectedChainId).toString(16)}`) {
+  if (chainId !== `0x${Number(CHAIN_ID).toString(16)}`) {
     return (
       <Button variant="contained" onClick={switchToMainnet}>
         Switch to NEVM Network
diff --git a/components/Bridge/Steps/ConnectValidate/StartTransferButton.tsx b/components/Bridge/Steps/ConnectValidate/StartTransferButton.tsx
index 590e695..c2b0e31 100644
--- a/components/Bridge/Steps/ConnectValidate/StartTransferButton.tsx
+++ b/components/Bridge/Steps/ConnectValidate/StartTransferButton.tsx
@@ -1,4 +1,4 @@
-import { MIN_AMOUNT } from "@constants";
+import { CHAIN_ID, MIN_AMOUNT } from "@constants";
 import { SYSX_ASSET_GUID } from "@contexts/Transfer/constants";
 import { ITransfer } from "@contexts/Transfer/types";
 import { CheckCircleOutline } from "@mui/icons-material";
@@ -23,7 +23,6 @@ export const ConnectValidateStartTransferButton: React.FC<{
   transfer: ITransfer;
   isSaving: boolean;
 }> = ({ isSaving, transfer }) => {
-  const { chainId: CHAIN_ID } = useFeatureFlags();
   const {
     watch,
     formState: { errors, isValid },
diff --git a/components/Bridge/UTXOStepWrapper.tsx b/components/Bridge/UTXOStepWrapper.tsx
index c35f09b..577e2c9 100644
--- a/components/Bridge/UTXOStepWrapper.tsx
+++ b/components/Bridge/UTXOStepWrapper.tsx
@@ -1,7 +1,7 @@
+import { CHAIN_ID } from "@constants";
 import { usePaliWalletV2 } from "@contexts/PaliWallet/usePaliWallet";
 import { Button } from "@mui/material";
 import { isValidSYSAddress } from "@pollum-io/sysweb3-utils";
-import { useFeatureFlags } from "./hooks/useFeatureFlags";
 
 type UTXOStepWrapperProps = {
   children: React.ReactNode;
@@ -17,8 +17,6 @@ const UTXOStepWrapper: React.FC<UTXOStepWrapperProps> = ({ children }) => {
     changeAccount,
   } = usePaliWalletV2();
 
-  const { chainId: CHAIN_ID } = useFeatureFlags();
-
   if (version === "v2" && !isBitcoinBased) {
     return (
       <Button variant="contained" onClick={() => switchTo("bitcoin")}>
diff --git a/components/Bridge/hooks/useFeatureFlags.ts b/components/Bridge/hooks/useFeatureFlags.ts
index 978de5c..8dff031 100644
--- a/components/Bridge/hooks/useFeatureFlags.ts
+++ b/components/Bridge/hooks/useFeatureFlags.ts
@@ -3,7 +3,6 @@ import { useQuery } from "react-query";
 type FeatureFlags = {
   foundationFundingAvailable: boolean;
   adminEnabled: boolean;
-  chainId: string;
 };
 
 export const useFeatureFlags = () => {
@@ -14,6 +13,5 @@ export const useFeatureFlags = () => {
   return {
     isEnabled: (flag: keyof FeatureFlags) =>
       (flags.isFetched && flags.data?.[flag]) ?? false,
-    chainId: flags.data?.chainId ? parseInt(flags.data.chainId) : 57,
   };
 };
diff --git a/constants.ts b/constants.ts
index 72212b9..3dc60e5 100644
--- a/constants.ts
+++ b/constants.ts
@@ -14,7 +14,4 @@ export const NEVM_TX_BLOCKCHAIN_URL = `${process.env.NEXT_PUBLIC_NEVM_EXPLORER}/
 
 export const ADMIN_LOGIN_MESSAGE = "Login to Syscoin Bridge Admin";
 
-export const CHAIN_ID =
-  `${process.env.NEXT_PUBLIC_IS_TESTNET}` === "true" ? 5700 : 57;
-
-console.log("next is testnet", process.env.NEXT_PUBLIC_IS_TESTNET, CHAIN_ID);
+export const CHAIN_ID = parseInt(process.env.NEXT_PUBLIC_CHAIN_ID || "57");
diff --git a/contexts/PaliWallet/V2Provider.tsx b/contexts/PaliWallet/V2Provider.tsx
index c13c5aa..fbe9fa3 100644
--- a/contexts/PaliWallet/V2Provider.tsx
+++ b/contexts/PaliWallet/V2Provider.tsx
@@ -8,7 +8,7 @@ import { utils as syscoinUtils } from "syscoinjs-lib";
 import { PaliWallet } from "./types";
 import MetamaskProvider from "@contexts/Metamask/Provider";
 import { isValidSYSAddress } from "@pollum-io/sysweb3-utils";
-import { useFeatureFlags } from "components/Bridge/hooks/useFeatureFlags";
+import { CHAIN_ID } from "@constants";
 
 export interface ProviderState {
   xpub: string;
@@ -76,7 +76,6 @@ export const PaliWalletV2Provider: React.FC<{
   children: React.ReactElement;
 }> = ({ children }) => {
   const queryClient = useQueryClient();
-  const { chainId } = useFeatureFlags();
   const installed = useQuery(["pali", "is-installed"], {
     queryFn: () => {
       return Boolean(window.pali) && window.pali.wallet === "pali-v2";
@@ -153,7 +152,7 @@ export const PaliWalletV2Provider: React.FC<{
     () =>
       connectedAccount.isSuccess &&
       connectedAccount.data &&
-      isValidSYSAddress(connectedAccount.data.address, chainId)
+      isValidSYSAddress(connectedAccount.data.address, CHAIN_ID)
         ? connectedAccount.data.address
         : undefined,
     [connectedAccount.data, connectedAccount.isSuccess]
@@ -221,7 +220,7 @@ export const PaliWalletV2Provider: React.FC<{
             method: "sys_changeUTXOEVM",
             params: [
               {
-                chainId,
+                chainId: CHAIN_ID,
               },
             ],
           })
@@ -234,7 +233,7 @@ export const PaliWalletV2Provider: React.FC<{
             method: "eth_changeUTXOEVM",
             params: [
               {
-                chainId,
+                chainId: CHAIN_ID,
               },
             ],
           })
diff --git a/contexts/Transfer/constants.ts b/contexts/Transfer/constants.ts
index 55d7f8f..d090966 100644
--- a/contexts/Transfer/constants.ts
+++ b/contexts/Transfer/constants.ts
@@ -1,9 +1,7 @@
 import { utils as syscoinUtils } from "syscoinjs-lib";
 export const SYSX_ASSET_GUID = "123456";
 const isTestnet = process.env.NEXT_PUBLIC_IS_TESTNET === "true";
-export const BlockbookAPIURL = isTestnet
-  ? "https://blockbook-dev.syscoin.org"
-  : "https://blockbook.syscoin.org";
+export const BlockbookAPIURL = process.env.NEXT_PUBLIC_BLOCKBOOK_API_URL;
 
 export const UTXO_NETWORK = isTestnet
   ? syscoinUtils.syscoinNetworks.testnet
@@ -17,6 +15,6 @@ export const NEVMNetwork = {
     symbol: isTestnet ? "TSYS" : "SYS",
     decimals: 18,
   },
-  rpcUrls: [process.env.NEVM_RPC_URL!],
-  blockExplorerUrls: [process.env.NEXT_PUBLIC_NEVM_EXPLORER],
+  rpcUrls: [process.env.NEXT_PUBLIC_NEVM_RPC_URL!],
+  blockExplorerUrls: [process.env.NEXT_PUBLIC_NEVM_EXPLORER_URL],
 };