Merge branch 'feat/agent-skills-rebuild' into fix/async-blocking-cleanup

Author: fangge518

agent-skills/run-quality-gate/SKILL.md

@@ -22,10 +22,13 @@ stack already running (`docker-compose up -d db`; `pre-commit`, `frontend`,
 
 Run the cheap gates (`pre-commit` ~6s, `bun run ts` 2–3s) freely; run the
 expensive gates (`build` ~78s, `pytest` ~96s) once per PR, right before push —
-not on every edit. One shortcut: `test-backend.yml` reruns pytest in CI on
-every push touching `backend/**`, so a local pytest re-run right before such a
-push just waits twice for the same answer; no equivalent CI job covers
-`bun run build` outside previews/deploys.
+not on every edit. The script automates one shortcut itself: when HEAD is
+exactly the pushed tip (clean `backend/` worktree) and CI's `test-backend.yml`
+already completed for that SHA, it reuses CI's verdict instead of running
+pytest locally — the summary line says so when it happens. A CI *failure* for
+the pushed SHA still runs pytest locally (fresh log) and prints a pointer to
+the CI record. No equivalent CI job covers `bun run build` outside
+previews/deploys, so the build gate always runs locally.
 
 ## ESLint is deliberately not a gate
 

agent-skills/run-quality-gate/scripts/run-gates.sh

@@ -84,10 +84,38 @@ if [ "${run_frontend}" = true ]; then
   build_pid=$!
 fi
 
+# CI-reuse: when HEAD is exactly what's pushed (clean backend/ worktree, local
+# tip == remote tip) and the test-backend workflow already completed for that
+# SHA, reuse CI's verdict instead of a duplicate multi-minute local run. One
+# gh call inside this script — no extra agent round-trips. Any doubt (dirty
+# tree, unpushed commits, gh missing, run still in progress) falls through to
+# a normal local run.
+ci_verdict=""
+if [ "${run_backend}" = true ] && command -v gh >/dev/null 2>&1 \
+   && [ -z "$(git status --porcelain -- backend/ 2>/dev/null)" ]; then
+  head_sha="$(git rev-parse HEAD)"
+  branch_name="$(git rev-parse --abbrev-ref HEAD)"
+  remote_sha="$(git ls-remote origin "refs/heads/${branch_name}" 2>/dev/null | cut -f1)"
+  if [ "${remote_sha}" = "${head_sha}" ]; then
+    ci_verdict="$(gh run list --commit "${head_sha}" --workflow test-backend.yml \
+      --json status,conclusion --jq \
+      'first(.[] | select(.status == "completed")) | .conclusion // empty' \
+      2>/dev/null || true)"
+  fi
+fi
+
 pytest_status=0
 pytest_pid=""
 pytest_ran=false
-if [ "${run_backend}" = true ]; then
+if [ "${run_backend}" = true ] && [ "${ci_verdict}" = "success" ]; then
+  pytest_ran=true
+  echo "backend pytest: PASS (reusing CI test-backend run for pushed HEAD $(git rev-parse --short HEAD); local run skipped)"
+  echo "CI verdict reused; see: gh run list --commit $(git rev-parse HEAD)" >"${pytest_log}"
+elif [ "${run_backend}" = true ]; then
+  if [ "${ci_verdict}" = "failure" ]; then
+    echo "note: CI test-backend FAILED for this exact commit — check the CI record" \
+      "(gh run list --commit $(git rev-parse HEAD)); running locally anyway for a fresh log."
+  fi
   pytest_ran=true
   echo "Starting backend pytest in background..."
   docker-compose exec -T backend pytest >"${pytest_log}" 2>&1 &