feat(ci): optimize CI workflows and i18n translation system (#204)

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Author: lyzno1

.github/workflows/auto-i18n-translation.yml

@@ -0,0 +1,77 @@
+name: Auto I18n Translation
+
+on:
+  push:
+    branches: [main]
+    paths:
+      - "messages/en-US.json"
+
+permissions:
+  contents: write
+  pull-requests: write
+
+jobs:
+  translate:
+    name: Auto Translation
+    runs-on: ubuntu-latest
+    if: ${{ github.repository == 'ifLabX/AgentifUI' }}
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v4
+        with:
+          token: ${{ secrets.GITHUB_TOKEN }}
+
+      - name: Setup pnpm
+        uses: pnpm/action-setup@v4
+        with:
+          version: "9"
+
+      - name: Setup Node.js
+        uses: actions/setup-node@v4
+        with:
+          node-version: "18"
+          cache: "pnpm"
+
+      - name: Setup Python
+        uses: actions/setup-python@v5
+        with:
+          python-version: "3.x"
+
+      - name: Install dependencies
+        run: pnpm install --frozen-lockfile
+
+      - name: Clean extra keys and run full translation
+        run: |
+          echo "🧹 Cleaning extra keys..."
+          pnpm i18n:remove-extra || true
+          
+          echo "🤖 Starting translation..."
+          pnpm i18n:translate
+          
+          echo "✅ Validating translations..."
+          pnpm i18n:check || true
+
+      - name: Create Pull Request
+        uses: peter-evans/create-pull-request@v6
+        with:
+          token: ${{ secrets.GITHUB_TOKEN }}
+          branch: auto-i18n-updates
+          delete-branch: true
+          title: "chore(i18n): auto-update translations"
+          body: |
+            ## 🤖 Automated Translation Update
+
+            Auto-generated translations based on changes to `messages/en-US.json`.
+
+            **Triggered by**: Push to main branch
+            **Languages**: All supported languages
+            **Mode**: Full translation update
+
+            Please review the translations and merge when ready.
+          labels: |
+            i18n
+            automated
+          commit-message: |
+            chore(i18n): auto-update translations
+            
+            Auto-generated by GitHub Actions

.github/workflows/ci.yml

@@ -109,6 +109,7 @@ jobs:
   format-and-lint:
     name: Format and Lint Check
     runs-on: ubuntu-latest
+    timeout-minutes: 5
     needs: [changes, install]
     if: needs.changes.outputs.source == 'true' || needs.changes.outputs.config == 'true' || needs.changes.outputs.styles == 'true'
     permissions:
@@ -150,6 +151,7 @@ jobs:
   type-check:
     name: TypeScript Type Check
     runs-on: ubuntu-latest
+    timeout-minutes: 5
     needs: [changes, install]
     if: needs.changes.outputs.source == 'true' || needs.changes.outputs.config == 'true'
     permissions:
@@ -175,47 +177,12 @@ jobs:
       - name: Run TypeScript type check
         run: pnpm run type-check
 
-  # Internationalization validation
-  i18n-validation:
-    name: I18n Validation
-    runs-on: ubuntu-latest
-    needs: [changes, install]
-    if: needs.changes.outputs.i18n == 'true' || needs.changes.outputs.source == 'true'
-    permissions:
-      contents: read
-    steps:
-      - name: Checkout repository
-        uses: actions/checkout@v4
-
-      - name: Setup Python
-        uses: actions/setup-python@v5
-        with:
-          python-version: '3.x'
-
-      - name: Setup pnpm
-        uses: pnpm/action-setup@v4
-        with:
-          version: ${{ env.PNPM_VERSION }}
-
-      - name: Setup Node.js
-        uses: actions/setup-node@v4
-        with:
-          node-version: ${{ env.NODE_VERSION }}
-          cache: 'pnpm'
-
-      - name: Install dependencies
-        run: pnpm install --frozen-lockfile
-
-      - name: Quick i18n check
-        run: pnpm run i18n:check
-
-      - name: Validate i18n consistency
-        run: pnpm run i18n:validate
 
   # Build the application
   build:
     name: Build Application
     runs-on: ubuntu-latest
+    timeout-minutes: 10
     needs: [changes, install]
     if: needs.changes.outputs.source == 'true' || needs.changes.outputs.config == 'true'
     permissions:
@@ -252,19 +219,18 @@ jobs:
           path: .next/
           retention-days: 7
 
-  # Run tests for changed test files
+  # Run tests
   test:
     name: Run Tests
     runs-on: ubuntu-latest
+    timeout-minutes: 10
     needs: [changes, install]
     if: needs.changes.outputs.tests == 'true' || needs.changes.outputs.source == 'true'
     permissions:
       contents: read
     steps:
       - name: Checkout repository
         uses: actions/checkout@v4
-        with:
-          fetch-depth: 0
 
       - name: Setup pnpm
         uses: pnpm/action-setup@v4
@@ -280,45 +246,14 @@ jobs:
       - name: Install dependencies
         run: pnpm install --frozen-lockfile
 
-      - name: Run tests for changed files
-        run: |
-          # Get list of changed test files and source files that might have related tests
-          CHANGED_TEST_FILES=$(git diff --name-only --diff-filter=ACMRT ${{ github.event.pull_request.base.sha }}..${{ github.event.pull_request.head.sha }} | grep -E '\.(test|spec)\.(js|jsx|ts|tsx)$' | tr '\n' ' ' || true)
-          CHANGED_SOURCE_FILES=$(git diff --name-only --diff-filter=ACMRT ${{ github.event.pull_request.base.sha }}..${{ github.event.pull_request.head.sha }} | grep -E '\.(js|jsx|ts|tsx)$' | grep -v -E '\.(test|spec)\.' | tr '\n' ' ' || true)
-          
-          # Find related test files for changed source files
-          RELATED_TEST_FILES=""
-          if [ -n "$CHANGED_SOURCE_FILES" ]; then
-            for file in $CHANGED_SOURCE_FILES; do
-              # Look for corresponding test files
-              base_name=$(basename "$file" | sed 's/\.[^.]*$//')
-              dir_name=$(dirname "$file")
-              
-              # Check multiple test file patterns
-              for pattern in "${dir_name}/${base_name}.test.*" "${dir_name}/${base_name}.spec.*" "__tests__/**/${base_name}.*" "**/${base_name}.test.*" "**/${base_name}.spec.*"; do
-                if ls $pattern 1> /dev/null 2>&1; then
-                  RELATED_TEST_FILES="$RELATED_TEST_FILES $(ls $pattern)"
-                fi
-              done
-            done
-          fi
-          
-          # Combine all test files to run
-          ALL_TEST_FILES="$CHANGED_TEST_FILES $RELATED_TEST_FILES"
-          ALL_TEST_FILES=$(echo $ALL_TEST_FILES | tr ' ' '\n' | sort -u | tr '\n' ' ')
-          
-          if [ -n "$ALL_TEST_FILES" ] && [ "$ALL_TEST_FILES" != " " ]; then
-            echo "Running tests for files: $ALL_TEST_FILES"
-            pnpm exec jest $ALL_TEST_FILES --passWithNoTests
-          else
-            echo "No test files found for changed source files, running all tests"
-            pnpm test
-          fi
+      - name: Run tests
+        run: pnpm test
 
   # Security checks
   security-check:
     name: Security Check
     runs-on: ubuntu-latest
+    timeout-minutes: 5
     needs: [changes, install]
     if: needs.changes.outputs.source == 'true' || needs.changes.outputs.config == 'true'
     permissions:
@@ -364,7 +299,6 @@ jobs:
       - changes
       - format-and-lint
       - type-check
-      - i18n-validation
       - build
       - test
       - security-check
@@ -379,7 +313,6 @@ jobs:
           # Check if any job failed
           if [[ "${{ needs.format-and-lint.result }}" == "failure" ]] || \
              [[ "${{ needs.type-check.result }}" == "failure" ]] || \
-             [[ "${{ needs.i18n-validation.result }}" == "failure" ]] || \
              [[ "${{ needs.build.result }}" == "failure" ]] || \
              [[ "${{ needs.test.result }}" == "failure" ]] || \
              [[ "${{ needs.security-check.result }}" == "failure" ]]; then

.gitignore

@@ -39,9 +39,9 @@ next-env.d.ts
 
 # test files
 /files
-scripts/*
-!scripts/validate-i18n-consistency.py
-!scripts/i18n-refactor-helper.py
+
+# scripts - allow all development scripts
+# scripts/
 
 # venv
 .venv

app/admin/api-config/layout.tsx

@@ -2,8 +2,10 @@
 
 import { InstanceFilterSelector } from '@components/admin/api-config/instance-filter-selector';
 import { SearchInput } from '@components/ui';
+import { ConfirmDialog } from '@components/ui/confirm-dialog';
 import { useTheme } from '@lib/hooks/use-theme';
 import { useApiConfigStore } from '@lib/stores/api-config-store';
+import { ServiceInstance } from '@lib/types/database';
 import { cn } from '@lib/utils';
 import {
   Bot,
@@ -76,6 +78,10 @@ export default function ApiConfigLayout({ children }: ApiConfigLayoutProps) {
   const [selectedInstanceId, setSelectedInstanceId] = useState<string | null>(
     null
   );
+  const [showDeleteDialog, setShowDeleteDialog] = useState(false);
+  const [instanceToDelete, setInstanceToDelete] =
+    useState<ServiceInstance | null>(null);
+  const [isDeleting, setIsDeleting] = useState(false);
 
   // get filter state from URL query params
   const [filterProviderId, setFilterProviderId] = useState<string | null>(
@@ -223,6 +229,26 @@ export default function ApiConfigLayout({ children }: ApiConfigLayoutProps) {
     [instances, t, tDebug, setDefaultInstance]
   );
 
+  const handleDeleteInstance = (instance: ServiceInstance) => {
+    setInstanceToDelete(instance);
+    setShowDeleteDialog(true);
+  };
+
+  const handleConfirmDelete = async () => {
+    if (!instanceToDelete) return;
+
+    setIsDeleting(true);
+    try {
+      await deleteInstance(instanceToDelete.id);
+      setShowDeleteDialog(false);
+      setInstanceToDelete(null);
+    } catch (error) {
+      console.error('Failed to delete instance:', error);
+    } finally {
+      setIsDeleting(false);
+    }
+  };
+
   // listen to page component's state change, fully sync page's form state
   useEffect(() => {
     const handleAddFormToggled = (event: CustomEvent) => {
@@ -533,7 +559,7 @@ export default function ApiConfigLayout({ children }: ApiConfigLayoutProps) {
                         <button
                           onClick={e => {
                             e.stopPropagation();
-                            deleteInstance(instance.instance_id);
+                            handleDeleteInstance(instance);
                           }}
                           disabled={isProcessing || instance.is_default}
                           className={cn(
@@ -582,6 +608,18 @@ export default function ApiConfigLayout({ children }: ApiConfigLayoutProps) {
       <div className="ml-80 h-full flex-1 overflow-hidden pl-px">
         {children}
       </div>
+
+      <ConfirmDialog
+        isOpen={showDeleteDialog}
+        onClose={() => !isDeleting && setShowDeleteDialog(false)}
+        onConfirm={handleConfirmDelete}
+        title={t('delete')}
+        message={t('deleteConfirm')}
+        confirmText={t('delete')}
+        variant="danger"
+        icon="delete"
+        isLoading={isDeleting}
+      />
     </div>
   );
 }