fix(orca-core): remove read-only QC skip that broke pappy evaluation

Author: junkyard22

packages/orca-core/src/runtime.ts

@@ -266,17 +266,9 @@ export function createOrcaRuntime(deps: OrcaRuntimeDeps): OrcaRuntime {
         result: maestroResult,
       });
 
-      // Skip QC on pure read-only results — no files were written and no tools
-      // failed, so there is nothing for Pappy to verify.  Saves one round-trip
-      // and the repair loop on tasks like "read this file and explain it".
-      const isReadOnlyResult =
-        qcEnabled &&
-        (!maestroResult.filesChanged || maestroResult.filesChanged.length === 0) &&
-        (!maestroResult.toolEvents || maestroResult.toolEvents.every((e) => e.ok !== false));
-
-      if (!qcEnabled || isReadOnlyResult) {
+      if (!qcEnabled) {
         recordTrace("qc.skipped", {
-          reason: !qcEnabled ? "pappy_not_configured" : "read_only_result",
+          reason: "pappy_not_configured",
         });
         result = {
           status: "SUCCESS",