fix: exclude test-origin queries from the gate (bump core to 0.13.0)

package.json

@@ -21,7 +21,7 @@
     "@libpg-query/parser": "^17.6.3",
     "@opentelemetry/api": "^1.9.0",
     "@pgsql/types": "^17.6.2",
-    "@query-doctor/core": "^0.12.0",
+    "@query-doctor/core": "^0.13.0",
     "async-sema": "^3.1.1",
     "capnweb": "^0.7.0",
     "dedent": "^1.7.1",

src/remote/api-client.ts

@@ -1,6 +1,6 @@
 import { newWebSocketRpcSession, RpcTarget } from "capnweb";
 import type { RpcStub } from "capnweb";
-import type { ConnectionMode, UnauthenticatedServerApi, ClientApi, IndexDefinition, ServerApi } from "@query-doctor/core";
+import type { ConnectionMode, UnauthenticatedServerApi, ClientApi, IndexDefinition, LiveQueryOptimization, ServerApi } from "@query-doctor/core";
 import type { ExportedStats } from "@query-doctor/core";
 import { PgIdentifier, Statistics } from "@query-doctor/core";
 import { log } from "../log.ts";
@@ -183,7 +183,10 @@ export class ApiClient extends RpcTarget implements ClientApi {
     );
   }
 
-  async runQuery(_query: string): Promise<void> {
-    log.warn("runQuery is not implemented", ApiClient.name);
+  async runQuery(_query: string): Promise<LiveQueryOptimization> {
+    // Part of the ClientApi RPC surface (core 0.13.0) but never invoked on the
+    // CI client — throw rather than return a fabricated optimization, so a stray
+    // call surfaces loudly instead of silently yielding a bogus result.
+    throw new Error("runQuery is not implemented on the CI ApiClient");
   }
 }

src/reporters/site-api.test.ts

@@ -245,6 +245,30 @@ describe("compareRuns", () => {
       expect(result.regressed).toHaveLength(1);
       expect(result.testOriginExcluded).toHaveLength(0);
     });
+
+    // Real SQLCommenter `file` tags carry a `:line:col` suffix
+    // (`…repository.spec.ts:509:10`). Before core stripped that suffix, the
+    // `$`-anchored `.spec.ts` pattern never matched, so a test read-back leaked
+    // into newQueries and phantom-blocked unrelated PRs run after run — the
+    // server dropped it as test-origin while the analyzer did not (Site #3606).
+    test("excludes a .spec.ts query whose file tag carries a :line:col suffix", async () => {
+      const withSuffix: CiQueryPayload = {
+        ...makeQuery("hash-new", 200),
+        tags: [
+          {
+            key: "file",
+            value:
+              "/home/runner/work/Site/Site/apps/api/src/projects/project-queries.repository.spec.ts:509:10",
+          },
+        ],
+      };
+      const previousRun = makePreviousRun([]);
+
+      const result = await compareRuns([withSuffix], previousRun, 10);
+
+      expect(result.newQueries).toHaveLength(0);
+      expect(result.testOriginExcluded.map((q) => q.hash)).toEqual(["hash-new"]);
+    });
   });
 
   describe("changed-query detection via shape (#3367)", () => {