ci: add lint/typecheck/build checks and fix lint errors (#22)

Author: Cordtus

.github/workflows/build.yml

@@ -2,16 +2,15 @@ name: build
 
 on:
   push:
-    branches:
-      - main
+    branches: [main]
     paths:
       - 'src/**'
       - 'public/**'
       - 'package.json'
       - 'bun.lock'
       - 'tsconfig.json'
       - 'panda.config.ts'
-      - 'eslint.config.js'
+      - 'biome.json'
       - 'Dockerfile'
       - 'fly.toml'
       - 'nginx.conf'
@@ -33,9 +32,6 @@ jobs:
       - name: Install dependencies
         run: bun install --frozen-lockfile
 
-      - name: Type check
-        run: bun run typecheck
-
       - name: Build
         run: bun run build
         env:

.github/workflows/ci.yml

@@ -0,0 +1,67 @@
+name: CI
+
+on:
+  pull_request:
+    branches: [main]
+  push:
+    branches: [main]
+
+concurrency:
+  group: ci-${{ github.ref }}
+  cancel-in-progress: true
+
+jobs:
+  lint:
+    name: Lint
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+
+      - name: Setup Bun
+        uses: oven-sh/setup-bun@v2
+        with:
+          bun-version: latest
+
+      - name: Install dependencies
+        run: bun install --frozen-lockfile
+
+      - name: Run Biome lint
+        run: bun run lint
+
+  typecheck:
+    name: Type Check
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+
+      - name: Setup Bun
+        uses: oven-sh/setup-bun@v2
+        with:
+          bun-version: latest
+
+      - name: Install dependencies
+        run: bun install --frozen-lockfile
+
+      - name: Type check
+        run: bun run typecheck
+
+  build:
+    name: Build
+    runs-on: ubuntu-latest
+    needs: [typecheck]
+    steps:
+      - uses: actions/checkout@v4
+
+      - name: Setup Bun
+        uses: oven-sh/setup-bun@v2
+        with:
+          bun-version: latest
+
+      - name: Install dependencies
+        run: bun install --frozen-lockfile
+
+      - name: Build
+        run: bun run build
+        env:
+          VITE_POSTGREST_URL: https://yaci-explorer-apis.fly.dev
+          VITE_CHAIN_REST_ENDPOINT: https://api.republicai.io

.github/workflows/preview.yml

@@ -1,3 +1,6 @@
+# Disabled - requires FLY_API_TOKEN secret to be configured
+# To re-enable: remove 'if: false' from the preview job
+
 name: Preview Deployment
 
 on:
@@ -23,6 +26,7 @@ env:
 
 jobs:
   preview:
+    if: false  # Disabled until FLY_API_TOKEN is configured
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v4

src/components/JsonViewer.tsx

@@ -71,15 +71,18 @@ function JsonNode({ data, name, level = 0, isLast = true }: JsonNodeProps) {
 
   return (
     <div>
-      <div
-        role="button"
-        tabIndex={0}
+      <button
+        type="button"
         className={css({
           display: 'flex',
           gap: '2',
           py: '0.5',
           cursor: 'pointer',
           rounded: 'md',
+          bg: 'transparent',
+          border: 'none',
+          w: 'full',
+          textAlign: 'left',
           _hover: { bg: 'bg.muted/50' }
         })}
         style={{ paddingLeft: `${indent}px` }}
@@ -96,7 +99,7 @@ function JsonNode({ data, name, level = 0, isLast = true }: JsonNodeProps) {
           {isArray ? '[' : '{'}
           {!isExpanded && <span className={css({ fontSize: 'xs', color: 'fg.muted', ml: '1' })}>{preview}</span>}
         </span>
-      </div>
+      </button>
 
       {isExpanded && (
         <>

src/components/MessageDetails.tsx

@@ -197,8 +197,8 @@ export function MessageDetails({ type, metadata, events }: MessageDetailsProps)
     const inputs = metadata.inputs || []
     const outputs = metadata.outputs || []
 
-    // Calculate total amounts
-    const totalInputAmount = inputs.reduce((sum, input) => {
+    // Calculate total amounts (currently unused, kept for potential display)
+    const _totalInputAmount = inputs.reduce((sum, input) => {
       return sum + input.coins.reduce((coinSum, coin) => coinSum + BigInt(coin.amount), BigInt(0))
     }, BigInt(0))
 
@@ -522,7 +522,7 @@ export function MessageDetails({ type, metadata, events }: MessageDetailsProps)
   // IBC Transfer
   if (type === '/ibc.applications.transfer.v1.MsgTransfer') {
     // Try to get actual sent amount from send_packet or transfer event
-    const sendPacketAmount = getEventAttribute(events || [], 'send_packet', 'packet_data_hex')
+    const _sendPacketAmount = getEventAttribute(events || [], 'send_packet', 'packet_data_hex')
     const transferAmountStr = getEventAttribute(events || [], 'transfer', 'amount')
     const transferAmounts = transferAmountStr ? parseMultiDenomAmount(transferAmountStr) : []
 
@@ -570,7 +570,7 @@ export function MessageDetails({ type, metadata, events }: MessageDetailsProps)
       try {
         const decoded = atob(metadata.msg)
         decodedMsg = JSON.parse(decoded)
-      } catch (e) {
+      } catch {
         // Ignore decode errors
       }
     }