fix: resolve ESLint errors and improve CI/CD workflows

  ESLint fixes:
  - Replace `as any` with proper union type in channels route
  - Replace `{} as any` with typed assertion in messaging-metrics
  - Replace require('zlib') with top-level gunzipSync import in database-session-store and session-archiver

  CI/CD improvements:
  - Upgrade setup-bun@v1 to @v2 (v1 deprecated)
  - Add bun dependency caching via actions/cache@v4
  - Add concurrency groups to prevent duplicate CI runs and overlapping deploys
  - Fix `bun test` to `bun test --run` to prevent watch mode hang
  - Upgrade appleboy/ssh-action@v1.0.0 to @v1
  - Upgrade getsentry/action-release@v1 to @V3 (v1 deprecated)
  - Add guard to skip Sentry step when auth token is missing
  - Add retry logic (3 attempts) to health check
  - Remove redundant deployment notification step

Author: HMAHD

.github/workflows/ci.yml

@@ -6,6 +6,10 @@ on:
   push:
     branches: [main, develop]
 
+concurrency:
+  group: ci-${{ github.ref }}
+  cancel-in-progress: true
+
 jobs:
   lint-and-type-check:
     name: Lint and Type Check
@@ -16,20 +20,25 @@ jobs:
         uses: actions/checkout@v4
 
       - name: Setup Bun
-        uses: oven-sh/setup-bun@v1
+        uses: oven-sh/setup-bun@v2
         with:
           bun-version: latest
 
+      - name: Cache bun dependencies
+        uses: actions/cache@v4
+        with:
+          path: ~/.bun/install/cache
+          key: bun-${{ runner.os }}-${{ hashFiles('bun.lock') }}
+          restore-keys: bun-${{ runner.os }}-
+
       - name: Install dependencies
         run: bun install --frozen-lockfile
 
       - name: Run ESLint
         run: bun run lint
-        continue-on-error: false
 
       - name: Run TypeScript type check
         run: bun run tsc --noEmit
-        continue-on-error: false
 
   test:
     name: Run Tests
@@ -41,10 +50,17 @@ jobs:
         uses: actions/checkout@v4
 
       - name: Setup Bun
-        uses: oven-sh/setup-bun@v1
+        uses: oven-sh/setup-bun@v2
         with:
           bun-version: latest
 
+      - name: Cache bun dependencies
+        uses: actions/cache@v4
+        with:
+          path: ~/.bun/install/cache
+          key: bun-${{ runner.os }}-${{ hashFiles('bun.lock') }}
+          restore-keys: bun-${{ runner.os }}-
+
       - name: Install dependencies
         run: bun install --frozen-lockfile
 

.github/workflows/deploy-production.yml

@@ -5,6 +5,10 @@ on:
     branches: [main]
   workflow_dispatch:
 
+concurrency:
+  group: deploy-production
+  cancel-in-progress: false
+
 jobs:
   test:
     name: Run Tests and Linting
@@ -19,14 +23,21 @@ jobs:
         with:
           bun-version: latest
 
+      - name: Cache bun dependencies
+        uses: actions/cache@v4
+        with:
+          path: ~/.bun/install/cache
+          key: bun-${{ runner.os }}-${{ hashFiles('bun.lock') }}
+          restore-keys: bun-${{ runner.os }}-
+
       - name: Install dependencies
         run: bun install --frozen-lockfile
 
       - name: Run linting
         run: bun run lint
 
       - name: Run tests
-        run: bun test
+        run: bun test --run
 
   deploy:
     name: Deploy to Production
@@ -43,7 +54,7 @@ jobs:
           fetch-depth: 0
 
       - name: Deploy to VPS via SSH
-        uses: appleboy/ssh-action@v1.0.0
+        uses: appleboy/ssh-action@v1
         with:
           host: ${{ secrets.PRODUCTION_VPS_HOST }}
           username: ${{ secrets.PRODUCTION_VPS_USER }}
@@ -143,26 +154,24 @@ jobs:
 
       - name: Health check
         run: |
-          sleep 15
-          curl -f https://luvora.love/api/health || exit 1
+          for i in 1 2 3; do
+            sleep 10
+            if curl -sf --max-time 10 https://luvora.love/api/health; then
+              echo "Health check passed"
+              exit 0
+            fi
+            echo "Attempt $i failed, retrying..."
+          done
+          echo "Health check failed after 3 attempts"
+          exit 1
 
       - name: Notify Sentry of deployment
-        if: success()
-        uses: getsentry/action-release@v1
+        if: success() && env.SENTRY_AUTH_TOKEN != ''
+        uses: getsentry/action-release@v3
         env:
           SENTRY_AUTH_TOKEN: ${{ secrets.SENTRY_AUTH_TOKEN }}
           SENTRY_ORG: akash-hasendra
           SENTRY_PROJECT: luvora
         with:
           environment: production
           version: ${{ github.sha }}
-
-      - name: Deployment notification
-        if: always()
-        run: |
-          if [ "${{ job.status }}" == "success" ]; then
-            echo "Production deployment successful!"
-          else
-            echo "Production deployment failed!"
-            exit 1
-          fi

src/app/api/messaging/channels/route.ts

@@ -45,7 +45,7 @@ export async function GET(request: NextRequest) {
         const channels = (['telegram', 'whatsapp', 'discord'] as const).map(platform => {
             const config = channelConfigs.find(c => c.platform === platform);
             const isRunning = messagingService.isChannelRunning(userId, platform);
-            const channel = messagingService.getChannel(userId, platform as any);
+            const channel = messagingService.getChannel(userId, platform as 'telegram' | 'whatsapp' | 'discord');
 
             return {
                 platform,

src/lib/messaging/database-session-store.ts

@@ -17,7 +17,7 @@
  */
 
 import PocketBase from 'pocketbase';
-import { brotliCompressSync, brotliDecompressSync, constants } from 'zlib';
+import { brotliCompressSync, brotliDecompressSync, gunzipSync, constants } from 'zlib';
 import fs from 'fs';
 import path from 'path';
 
@@ -271,7 +271,6 @@ export class DatabaseSessionStore {
             // Fall back to gzip for legacy sessions
             // This allows gradual migration without breaking existing sessions
             console.warn('[DatabaseSessionStore] Brotli decompression failed, trying gzip fallback');
-            const { gunzipSync } = require('zlib');
             return gunzipSync(data).toString('utf8');
         }
     }

src/lib/messaging/messaging-metrics.ts

@@ -133,7 +133,7 @@ export class MessagingMetrics {
         platforms: Record<MessagingPlatform, MetricsSummary>;
     } {
         const platforms: MessagingPlatform[] = ['telegram', 'whatsapp', 'discord'];
-        const platformMetrics: Record<MessagingPlatform, MetricsSummary> = {} as any;
+        const platformMetrics = {} as Record<MessagingPlatform, MetricsSummary>;
 
         let totalSent = 0;
         let totalFailed = 0;

src/lib/messaging/session-archiver.ts

@@ -13,7 +13,7 @@
 
 import fs from 'fs';
 import path from 'path';
-import { brotliCompressSync, brotliDecompressSync, constants } from 'zlib';
+import { brotliCompressSync, brotliDecompressSync, gunzipSync, constants } from 'zlib';
 import { execSync } from 'child_process';
 
 export interface SessionArchive {
@@ -120,7 +120,6 @@ export class SessionArchiver {
             } catch (error) {
                 // Fall back to gzip for legacy archives
                 console.warn('[SessionArchiver] Brotli decompression failed, trying gzip fallback');
-                const { gunzipSync } = require('zlib');
                 tarball = gunzipSync(compressed);
             }