fix(grader): validate dependencies before grader preflight

christso · christso · commit 207ea9318599 · 2026-07-07T10:06:05.000+02:00
diff --git a/packages/core/src/evaluation/orchestrator.ts b/packages/core/src/evaluation/orchestrator.ts
@@ -885,6 +885,7 @@ export async function runEvaluation(
     }
     return [];
   }
+  validateDependencyGraph(filteredEvalCases);
 
   const runtime = createEvaluationRuntime({
     target,
@@ -1068,8 +1069,7 @@ export async function runEvaluation(
     // fail_on_error tracking (best-effort under concurrency > 1, matching budgetExhausted semantics)
     let failOnErrorTriggered = false;
 
-    // --- Validate dependency graph and compute execution waves ---
-    validateDependencyGraph(filteredEvalCases);
+    // --- Compute dependency-aware execution waves ---
     const waves = computeWaves(filteredEvalCases);
 
     // Track completed test results for dependency injection

Original file line number	Diff line number	Diff line change
`@@ -885,6 +885,7 @@ export async function runEvaluation(`
`885`	`885`	`}`
`886`	`886`	`return [];`
`887`	`887`	`}`
	`888`	`+ validateDependencyGraph(filteredEvalCases);`
`888`	`889`
`889`	`890`	`const runtime = createEvaluationRuntime({`
`890`	`891`	`target,`
`@@ -1068,8 +1069,7 @@ export async function runEvaluation(`
`1068`	`1069`	`// fail_on_error tracking (best-effort under concurrency > 1, matching budgetExhausted semantics)`
`1069`	`1070`	`let failOnErrorTriggered = false;`
`1070`	`1071`
`1071`		`- // --- Validate dependency graph and compute execution waves ---`
`1072`		`- validateDependencyGraph(filteredEvalCases);`
	`1072`	`+ // --- Compute dependency-aware execution waves ---`
`1073`	`1073`	`const waves = computeWaves(filteredEvalCases);`
`1074`	`1074`
`1075`	`1075`	`// Track completed test results for dependency injection`