fix: action (#260)

Author: FinleyGe

.github/workflows/build-image-preview.yml

@@ -24,69 +24,72 @@ jobs:
           REPO=${{ github.repository }}
           TOKEN=${{ secrets.GITHUB_TOKEN }}
 
-          # 获取 PR 修改的文件列表
           RESPONSE=$(curl -s -H "Authorization: token $TOKEN" \
             "https://api.github.com/repos/$REPO/pulls/$PR_NUMBER/files")
 
-          # 提取文件路径
           FILES=$(echo "$RESPONSE" | jq -r '.[].filename')
 
           echo "All changed files:"
           echo "$FILES"
 
-          # 检查是否有相关文件需要构建
           HAS_RELEVANT_CHANGES="false"
 
-          # 检查是否有根目录文件或配置文件变更
+          # Check for relevant changes
           for file in $FILES; do
-            case "$file" in
-              "package.json"|"bun.lockb"|".github/workflows/"*"|"bun.lock"|yarn.lock|"pnpm-lock.yaml")
-                echo "Found relevant root file: $file"
-                HAS_RELEVANT_CHANGES="true"
-                ;;
-              "modules/tool/packages/"*)
-                echo "Found package file: $file"
-                HAS_RELEVANT_CHANGES="true"
-                ;;
-            esac
+            if [ "$file" = "package.json" ]; then
+              echo "Found package.json"
+              HAS_RELEVANT_CHANGES="true"
+            elif [ "$file" = "bun.lock" ]; then
+              echo "Found bun.lock"
+              HAS_RELEVANT_CHANGES="true"
+            elif [ "$file" = "bun.lockb" ]; then
+              echo "Found bun.lockb"
+              HAS_RELEVANT_CHANGES="true"
+            elif [[ "$file" == .github/workflows/* ]]; then
+              echo "Found workflow file: $file"
+              HAS_RELEVANT_CHANGES="true"
+            elif [[ "$file" == modules/tool/packages/* ]]; then
+              echo "Found package file: $file"
+              HAS_RELEVANT_CHANGES="true"
+            fi
           done
 
-          # 提取在 /modules/tool/packages 下变更的包目录
+          # Extract package names from changed files
           PACKAGE_CHANGES=""
           for file in $FILES; do
             if [[ "$file" == modules/tool/packages/* ]]; then
-              # 提取包名（modules/tool/packages/package-name/...）
-              PACKAGE_NAME=$(echo "$file" | sed 's|^modules/tool/packages/[^/]*/||' | cut -d'/' -f1)
-              if [[ -n "$PACKAGE_NAME" ]]; then
-                # 检查是否已经添加过这个包
-                if ! echo "$PACKAGE_CHANGES" | grep -q "$PACKAGE_NAME"; then
-                  if [ -z "$PACKAGE_CHANGES" ]; then
-                    PACKAGE_CHANGES="$PACKAGE_NAME"
-                  else
-                    PACKAGE_CHANGES="$PACKAGE_CHANGES $PACKAGE_NAME"
-                  fi
+              # Extract package directory name (first segment after packages/)
+              PACKAGE_NAME=$(echo "$file" | cut -d'/' -f4)
+              echo "Found package directory: $PACKAGE_NAME"
+
+              # Check if we already have this package
+              if echo "$PACKAGE_CHANGES" | grep -vqw "$PACKAGE_NAME"; then
+                if [ -z "$PACKAGE_CHANGES" ]; then
+                  PACKAGE_CHANGES="$PACKAGE_NAME"
+                else
+                  PACKAGE_CHANGES="$PACKAGE_CHANGES $PACKAGE_NAME"
                 fi
               fi
             fi
           done
 
-          # 构建稀疏检出配置
+          # Build sparse checkout configuration
           SPARSE_CONFIG="/*"
           SPARSE_CONFIG="$SPARSE_CONFIG"$'\n'"!/modules/tool/packages"
 
-          # 为每个变更的包添加包含规则
+          # Add changed packages to sparse checkout
           for package in $PACKAGE_CHANGES; do
             if [ -n "$package" ]; then
               SPARSE_CONFIG="$SPARSE_CONFIG"$'\n'"/modules/tool/packages/$package"
-              echo "Adding package to sparse checkout: modules/tool/packages/$package"
+              echo "Adding to sparse checkout: modules/tool/packages/$package"
             fi
           done
 
-          # 总是需要包含根目录的关键文件
+          # Always include essential files
           SPARSE_CONFIG="$SPARSE_CONFIG"$'\n'"package.json"
           SPARSE_CONFIG="$SPARSE_CONFIG"$'\n'"bun.lockb"
 
-          # 保存到输出
+          # Output results for GitHub Actions
           {
             echo "files<<EOF"
             echo "$FILES"
@@ -169,17 +172,61 @@ jobs:
           cache-from: type=local,src=/tmp/.buildx-cache
           cache-to: type=local,dest=/tmp/.buildx-cache
 
-      - name: '@finleyge/github-tools'
-        uses: FinleyGe/[email protected]
-        id: print-image-label
-        if: success()
+      - name: Add PR comment on success
+        if: steps.pr-files.outputs.has_relevant_changes == 'true' && success()
+        uses: actions/github-script@v6
         with:
-          token: ${{ secrets.GITHUB_TOKEN }}
-          tool: issue-comment
-          title: Preview Image Label
-          body: |
-            Preview Images for this PR:
-
-            ```
+          script: |
+            const { data: comments } = await github.rest.issues.listComments({
+              owner: context.repo.owner,
+              repo: context.repo.repo,
+              issue_number: context.issue.number,
+            });
+
+            const existingComment = comments.find(comment =>
+              comment.body.includes('Preview Images for this PR') &&
+              comment.user.type === 'Bot'
+            );
+
+            const commentBody = `
+            ✅ **Build Successful** - Preview Images for this PR:
+
+            \`\`\`
             ${{ secrets.ALI_IMAGE_NAME }}/fastgpt-plugin-pr:${{ github.event.pull_request.head.sha }}
-            ```
+            \`\`\`
+
+            **Changed packages:**
+            ${{ steps.pr-files.outputs.package_changes || 'None' }}
+            `;
+
+            if (existingComment) {
+              await github.rest.issues.updateComment({
+                owner: context.repo.owner,
+                repo: context.repo.repo,
+                comment_id: existingComment.id,
+                body: commentBody
+              });
+            } else {
+              await github.rest.issues.createComment({
+                owner: context.repo.owner,
+                repo: context.repo.repo,
+                issue_number: context.issue.number,
+                body: commentBody
+              });
+            }
+
+      - name: Add PR comment on skip
+        if: steps.pr-files.outputs.has_relevant_changes == 'false'
+        uses: actions/github-script@v6
+        with:
+          script: |
+            await github.rest.issues.createComment({
+              owner: context.repo.owner,
+              repo: context.repo.repo,
+              issue_number: context.issue.number,
+              body: `
+              ℹ️ **Build Skipped** - No relevant changes found in this PR
+
+              This PR doesn't contain changes that require rebuilding (no package.json, bun.lockb, or plugin package changes).
+              `
+            });