.gitea/workflows/security.yml

@@ -64,17 +64,15 @@ jobs:
           REPO_FULL: ${{ gitea.repository }}
           PR_NUMBER: ${{ gitea.event.pull_request.number }}
           COMMIT_SHA: ${{ gitea.event.pull_request.head.sha }}
-          HEAD_REF: ${{ gitea.head_ref }}
         run: |
           python3 <<'PYEOF'
-          import base64, json, os, sys, time, urllib.error, urllib.request
+          import json, os, sys, time, urllib.error, urllib.request
 
           SERVER = os.environ["GITEA_SERVER"].rstrip("/")
           REPO = os.environ["REPO_FULL"]
           TOKEN = os.environ["SECURITY_BOT_TOKEN"]
           PR_NUMBER = os.environ.get("PR_NUMBER", "")
           COMMIT_SHA = os.environ["COMMIT_SHA"]
-          HEAD_REF = os.environ["HEAD_REF"]
 
           def gitea(method, path, payload=None):
               req = urllib.request.Request(
@@ -170,17 +168,17 @@ jobs:
               print(f"не удалось прочитать прежние статусы, продолжаю без sticky-fail: {e}")
           sticky = prior_failed and not blocked
 
-          outdir = f"security-analysis/findings/pr-{PR_NUMBER or 'manual'}"
+          # Раньше находки ещё и коммитились в ветку PR. Плохая
-          os.makedirs(outdir, exist_ok=True)
+          # идея с точки зрения git-гигиены: чужая ветка растёт
-          outpath = f"{outdir}/{int(time.time())}.json"
+          # коммитами от бота, каждый такой коммит сам порождает
-          json.dump(findings, open(outpath, "w"), ensure_ascii=False, indent=2)
+          # pull_request:synchronize (нужна отдельная защита от
-
+          # петли), а когда коммит бота становится головой PR —
-          gitea("POST", f"/repos/{REPO}/contents/{outpath}", {
+          # обязательные security/scan и security/review больше
-              "content": base64.b64encode(open(outpath, "rb").read()).decode(),
+          # никогда не выставляются на этой голове, потому что
-              "message": f"security: scan results for PR #{PR_NUMBER}",
+          # job их специально пропускает для бота же — PR
-              "branch": HEAD_REF,
+          # перестаёт мержиться насовсем (поймано живьём).
-          })
+          # Комментарий в PR (ниже) и так несёт всё нужное
-
+          # человеку; отдельного коммита с JSON не требуется.
           counts = {}
           for f in findings:
               counts[f["severity"]] = counts.get(f["severity"], 0) + 1
@@ -192,7 +190,6 @@ jobs:
           body += "\n\n"
           for f in top:
               body += f"**{f['severity'].upper()}** `{f['file']}:{f['line_start']}` — {f['title']} ({f['source']})\n\n"
-          body += f"Полный список: `{outpath}`\n\n"
           if sticky:
               body += "🔴 **Merge заблокирован** — этот же коммит уже был помечен как блокирующий в более раннем прогоне; новый прогон нашёл меньше, но старую блокировку это не снимает (нужен новый коммит).\n"
           else:
@@ -204,7 +201,6 @@ jobs:
               "state": "failure" if (blocked or sticky) else "success",
               "context": "security/scan",
               "description": (f"{len(findings)} findings, {len(blocking)} blocking" + (" (sticky: prior run flagged this commit)" if sticky else "")),
-              "target_url": f"{SERVER}/{REPO}/src/branch/{HEAD_REF}/{outpath}",
           })
 
           print(f"findings: {len(findings)}, blocking: {len(blocking)}, sticky: {sticky}")
@@ -363,17 +359,15 @@ jobs:
           REPO_FULL: ${{ gitea.repository }}
           PR_NUMBER: ${{ gitea.event.pull_request.number }}
           COMMIT_SHA: ${{ gitea.event.pull_request.head.sha }}
-          HEAD_REF: ${{ gitea.head_ref }}
         run: |
           python3 <<'PYEOF2'
-          import base64, json, os, sys, time, urllib.error, urllib.request
+          import json, os, sys, time, urllib.error, urllib.request
 
           SERVER = os.environ["GITEA_SERVER"].rstrip("/")
           REPO = os.environ["REPO_FULL"]
           TOKEN = os.environ["SECURITY_BOT_TOKEN"]
           PR_NUMBER = os.environ.get("PR_NUMBER", "")
           COMMIT_SHA = os.environ["COMMIT_SHA"]
-          HEAD_REF = os.environ["HEAD_REF"]
 
           def gitea(method, path, payload=None):
               req = urllib.request.Request(
@@ -473,16 +467,9 @@ jobs:
               print(f"не удалось прочитать прежние статусы, продолжаю без sticky-fail: {e}")
           sticky = prior_failed and not blocked
 
-          outdir = f"security-analysis/findings/pr-{PR_NUMBER or 'manual'}"
+          # Коммит с findings-json в ветку PR убран — та же причина,
-          outpath = f"{outdir}/{int(time.time())}-review.json"
+          # что и в шаге scan: он ломает мержимость PR насовсем,
-
+          # как только становится головой (см. комментарий там).
-          if findings:
-              gitea("POST", f"/repos/{REPO}/contents/{outpath}", {
-                  "content": base64.b64encode(json.dumps(findings, ensure_ascii=False, indent=2).encode()).decode(),
-                  "message": f"security: AI review results for PR #{PR_NUMBER}",
-                  "branch": HEAD_REF,
-              })
-
           counts = {}
           for f in findings:
               counts[f["severity"]] = counts.get(f["severity"], 0) + 1
@@ -499,7 +486,6 @@ jobs:
                   body += f"**{f['severity'].upper()}** (`{f['confidence']:.2f}`) `{f['file']}:{f['line_start']}` — {f['title']}\n\n"
                   if f.get("exploit_scenario"):
                       body += f"  сценарий: {f['exploit_scenario']}\n\n"
-              body += f"Полный список: `{outpath}`\n\n"
           if sticky:
               body += "🔴 **Merge заблокирован** — этот же коммит уже был помечен как блокирующий в более раннем прогоне ревью; новый прогон ничего не нашёл, но старую блокировку это не снимает (нужен новый коммит).\n"
           else: