Merge pull request #210 from StarRocks/review

review comment

Author: DanRoscigno

.github/workflows/ci-doc-translater.yml

@@ -33,7 +33,7 @@ jobs:
         uses: actions/github-script@v7
         with:
           script: |
-            github.rest.reactions.createForIssueComment({
+            await github.rest.reactions.createForIssueComment({
               owner: context.repo.owner,
               repo: context.repo.repo,
               comment_id: context.payload.comment.id,

.github/workflows/ci-doc-translation-check.yml

@@ -19,11 +19,6 @@ jobs:
         with:
           result-encoding: string
           script: |
-            // If triggered by the Translation Runner, we can always proceed
-            if (context.payload.workflow_run.name === 'Translation Runner') {
-              return 'true';
-            }
-            
             // Query the jobs of the CI DOC Checker run
             const { data: jobsData } = await github.rest.actions.listJobsForWorkflowRun({
               owner: context.repo.owner,
@@ -259,9 +254,24 @@ jobs:
         uses: actions/github-script@v7
         with:
           script: |
-            github.rest.issues.addLabels({
-              owner: context.repo.owner,
-              repo: context.repo.repo,
-              issue_number: ${{ steps.pr_number.outputs.pr_number }},
-              labels: ['docs-maintainer']
-            });
+            try {
+              await github.rest.issues.addLabels({
+                owner: context.repo.owner,
+                repo: context.repo.repo,
+                issue_number: ${{ steps.pr_number.outputs.pr_number }},
+                labels: ['docs-maintainer']
+              });
+            } catch (error) {
+              const status = error?.status ?? error?.response?.status;
+              const message = String(error?.message ?? '').toLowerCase();
+              const hasAlreadyExistsCode = Array.isArray(error?.response?.data?.errors)
+                && error.response.data.errors.some(item => item?.code === 'already_exists');
+
+              // Ignore only idempotent duplicate-label errors; surface all other failures.
+              if (status === 422 && (message.includes('already exists') || hasAlreadyExistsCode)) {
+                core.info('docs-maintainer label already exists. Skipping.');
+                return;
+              }
+
+              throw error;
+            }