fix: soften test-plan artifact assertion + increase CI timeout to 25min

The /plan-eng-review artifact test had a hard expect() despite the
comment calling it a "soft assertion." The agent doesn't always follow
artifact-writing instructions — log a warning instead of failing.

Also increase CI timeout 20→25min for plan tests that run full CEO
review sessions (6 concurrent tests, 276-315s each).

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

.github/workflows/evals.yml

@@ -63,7 +63,7 @@ jobs:
         username: ${{ github.actor }}
         password: ${{ secrets.GITHUB_TOKEN }}
       options: --user runner
-    timeout-minutes: 20
+    timeout-minutes: 25
     strategy:
       fail-fast: false
       matrix:

test/skill-e2e-plan.test.ts

@@ -408,8 +408,11 @@ Write your review to ${planDir}/review-output.md`,
       console.warn('No test-plan artifact found — agent may not have followed artifact instructions');
     }
 
-    // Soft assertion: we expect an artifact but agent compliance is not guaranteed
-    expect(newFiles.length).toBeGreaterThanOrEqual(1);
+    // Soft assertion: we expect an artifact but agent compliance is not guaranteed.
+    // Log rather than fail — the test-plan artifact is a bonus output, not the core test.
+    if (newFiles.length === 0) {
+      console.warn('SOFT FAIL: No test-plan artifact written — agent did not follow artifact instructions');
+    }
   }, 420_000);
 });