diff --git a/.github/workflows/regression-test-check.yml b/.github/workflows/regression-test-check.yml index 6d97c4ce..ef1a4d92 100644 --- a/.github/workflows/regression-test-check.yml +++ b/.github/workflows/regression-test-check.yml @@ -43,12 +43,42 @@ jobs: fi fi - if [ "$IS_FIX" = false ]; then - echo "Not a fix PR — skipping regression test check." + # --- 1b. Does this PR touch high-risk state machine or resilience code? --- + CHANGED_FILES=$(git diff --name-only "${BASE_REF}...${HEAD_REF}") + + TOUCHES_HIGH_RISK=false + HIGH_RISK_PATTERNS=( + "src/context/state.rs" + "src/agent/session.rs" + "src/llm/circuit_breaker.rs" + "src/llm/retry.rs" + "src/llm/failover.rs" + "src/agent/self_repair.rs" + "src/agent/agentic_loop.rs" + "src/tools/execute.rs" + "crates/ironclaw_safety/src/" + ) + + for pattern in "${HIGH_RISK_PATTERNS[@]}"; do + if echo "$CHANGED_FILES" | grep -q "$pattern"; then + TOUCHES_HIGH_RISK=true + echo "High-risk file matched: $pattern" + break + fi + done + + # Skip only if NEITHER condition holds — no double-firing on fix PRs + if [ "$IS_FIX" = false ] && [ "$TOUCHES_HIGH_RISK" = false ]; then + echo "Not a fix PR and no high-risk files changed — skipping." exit 0 fi - echo "Fix PR detected." + if [ "$IS_FIX" = true ]; then + echo "Fix PR detected." + fi + if [ "$TOUCHES_HIGH_RISK" = true ]; then + echo "High-risk state machine or resilience code modified." + fi # --- 2. Skip label or commit message marker --- if grep -qF ',skip-regression-check,' <<< ",$PR_LABELS,"; then @@ -63,8 +93,6 @@ jobs: fi # --- 3. Exempt static-only / docs-only changes --- - CHANGED_FILES=$(git diff --name-only "${BASE_REF}...${HEAD_REF}") - if [ -z "$CHANGED_FILES" ]; then echo "No changed files — skipping." exit 0 @@ -110,5 +138,12 @@ jobs: fi # --- 5. No tests found --- - echo "::warning::This PR looks like a bug fix but contains no test changes. Every fix should include a regression test. Add a #[test] or #[tokio::test], or apply the 'skip-regression-check' label if not feasible." + if [ "$IS_FIX" = true ]; then + echo "::warning::This PR looks like a bug fix but contains no test changes." + fi + if [ "$TOUCHES_HIGH_RISK" = true ]; then + echo "::warning::This PR modifies high-risk state machine or resilience code but includes no test changes." + fi + echo "::warning::Please add tests exercising the changed behavior, or apply the 'skip-regression-check' label if not feasible." exit 1 +