@canonry/canonry 4.177.2 → 4.177.3

package/dist/{chunk-GJE334DA.js → chunk-5EH666EO.js}

@@ -633,7 +633,7 @@ import {
 } from "./chunk-PQT3ECM3.js";
 
 // src/intelligence-service.ts
-import { eq as eq61, desc as desc27, asc as asc11, and as and50, ne as ne8, or as or14, inArray as inArray21, gte as gte14, lte as lte12 } from "drizzle-orm";
+import { eq as eq61, desc as desc27, asc as asc11, and as and50, ne as ne8, or as or14, inArray as inArray21, gte as gte14, lte as lte12, isNull as isNull8, sql as sql25, exists } from "drizzle-orm";
 
 // ../db/src/client.ts
 import { mkdirSync } from "fs";
@@ -7208,6 +7208,13 @@ function detectRegressions(currentRun, previousRun) {
   return regressions;
 }
 
+// ../intelligence/src/observation-coverage.ts
+function observedKeys(run, keyOf) {
+  const observed2 = /* @__PURE__ */ new Set();
+  for (const snap of run.snapshots) observed2.add(keyOf(snap));
+  return observed2;
+}
+
 // ../intelligence/src/gains.ts
 function snapshotKey2(snap) {
   const loc = snap.location ?? "__none__";
@@ -7218,6 +7225,7 @@ function detectGains(currentRun, previousRun) {
     return [];
   }
   const gains = [];
+  const previousObserved = observedKeys(previousRun, snapshotKey2);
   const previousCited = /* @__PURE__ */ new Set();
   for (const snap of previousRun.snapshots) {
     if (snap.cited) {
@@ -7226,7 +7234,7 @@ function detectGains(currentRun, previousRun) {
   }
   for (const snap of currentRun.snapshots) {
     const key = snapshotKey2(snap);
-    if (snap.cited && !previousCited.has(key)) {
+    if (snap.cited && previousObserved.has(key) && !previousCited.has(key)) {
       gains.push({
         query: snap.query,
         provider: snap.provider,
@@ -7452,6 +7460,7 @@ function generateInsights(input) {
 
 // ../intelligence/src/first-citations.ts
 function detectFirstCitations(currentRun, previousRun) {
+  const previousObservedQueries = observedKeys(previousRun, (snap) => snap.query);
   const previousCitedQueries = /* @__PURE__ */ new Set();
   for (const snap of previousRun.snapshots) {
     if (snap.cited) previousCitedQueries.add(snap.query);
@@ -7460,6 +7469,7 @@ function detectFirstCitations(currentRun, previousRun) {
   const seen = /* @__PURE__ */ new Set();
   for (const snap of currentRun.snapshots) {
     if (!snap.cited) continue;
+    if (!previousObservedQueries.has(snap.query)) continue;
     if (previousCitedQueries.has(snap.query)) continue;
     const key = `${snap.query}:${snap.provider}`;
     if (seen.has(key)) continue;
@@ -7477,6 +7487,7 @@ function detectFirstCitations(currentRun, previousRun) {
 
 // ../intelligence/src/provider-pickups.ts
 function detectProviderPickups(currentRun, previousRun) {
+  const previousObservedPairs = observedKeys(previousRun, (snap) => `${snap.query}:${snap.provider}`);
   const previousCitedQueries = /* @__PURE__ */ new Set();
   const previousCitedPairs = /* @__PURE__ */ new Set();
   for (const snap of previousRun.snapshots) {
@@ -7490,6 +7501,7 @@ function detectProviderPickups(currentRun, previousRun) {
     if (!snap.cited) continue;
     if (!previousCitedQueries.has(snap.query)) continue;
     const key = `${snap.query}:${snap.provider}`;
+    if (!previousObservedPairs.has(key)) continue;
     if (previousCitedPairs.has(key)) continue;
     if (seen.has(key)) continue;
     seen.add(key);
@@ -7551,11 +7563,13 @@ function detectCompetitorGains(currentRun, previousRun, opts) {
   if (tracked.size === 0) return [];
   const currentMap = buildCompetitorQueryMap(currentRun, tracked);
   const previousMap = buildCompetitorQueryMap(previousRun, tracked);
+  const previousObservedQueries = observedKeys(previousRun, (snap) => snap.query);
   const result = [];
   for (const competitorDomain of tracked) {
     const currentQs = currentMap.get(competitorDomain) ?? /* @__PURE__ */ new Set();
     const previousQs = previousMap.get(competitorDomain) ?? /* @__PURE__ */ new Set();
     for (const query of currentQs) {
+      if (!previousObservedQueries.has(query)) continue;
       if (previousQs.has(query)) continue;
       result.push({ query, competitorDomain });
     }
@@ -7567,11 +7581,13 @@ function detectCompetitorLosses(currentRun, previousRun, opts) {
   if (tracked.size === 0) return [];
   const currentMap = buildCompetitorQueryMap(currentRun, tracked);
   const previousMap = buildCompetitorQueryMap(previousRun, tracked);
+  const currentObservedQueries = observedKeys(currentRun, (snap) => snap.query);
   const result = [];
   for (const competitorDomain of tracked) {
     const currentQs = currentMap.get(competitorDomain) ?? /* @__PURE__ */ new Set();
     const previousQs = previousMap.get(competitorDomain) ?? /* @__PURE__ */ new Set();
     for (const query of previousQs) {
+      if (!currentObservedQueries.has(query)) continue;
       if (currentQs.has(query)) continue;
       result.push({ query, competitorDomain });
     }
@@ -51119,7 +51135,7 @@ async function queryBacklinks(opts) {
   const reversed = opts.targets.map(reverseDomain);
   const targetList = reversed.map(quote).join(", ");
   const limitClause = opts.limitPerTarget ? `QUALIFY row_number() OVER (PARTITION BY t.target_rev_domain ORDER BY v.num_hosts DESC) <= ${Math.floor(opts.limitPerTarget)}` : "";
-  const sql25 = `
+  const sql26 = `
     WITH vertices AS (
       SELECT * FROM read_csv(
         ${quote(opts.vertexPath)},
@@ -51155,7 +51171,7 @@ async function queryBacklinks(opts) {
   const conn = await instance.connect();
   let rows;
   try {
-    const reader = await conn.runAndReadAll(sql25);
+    const reader = await conn.runAndReadAll(sql26);
     rows = reader.getRowObjects();
   } finally {
     conn.disconnectSync?.();
@@ -77833,6 +77849,19 @@ function matchesBrandAlias(candidate, aliases) {
 var RECURRENCE_LOOKBACK_RUNS = 5;
 var HISTORY_WINDOW_RUNS = Math.max(PERSISTENT_GAP_THRESHOLD, 5);
 var log = createLogger("IntelligenceService");
+var runChronologyKey = sql25`coalesce(${runs.finishedAt}, ${runs.createdAt})`;
+function analyzableRunPredicates(projectId) {
+  return [
+    eq61(runs.projectId, projectId),
+    // Only this kind writes query_snapshots.
+    eq61(runs.kind, RunKinds["answer-visibility"]),
+    or14(eq61(runs.status, RunStatuses.completed), eq61(runs.status, RunStatuses.partial)),
+    // Defensive: RunCoordinator already skips probes before analyzeAndPersist
+    // is called, but a future call site invoking it directly for a probe must
+    // still not pollute the intelligence window.
+    ne8(runs.trigger, RunTriggers.probe)
+  ];
+}
 function readStoredGroundingSources(rawResponse) {
   const parsed = parseJsonColumn(rawResponse, {});
   const sources = parsed.groundingSources;
@@ -77865,26 +77894,27 @@ var IntelligenceService = class {
    * Returns the analysis result for the coordinator to inspect (e.g. for webhook dispatch).
    */
   analyzeAndPersist(runId, projectId) {
+    const currentRunRecord = this.db.select().from(runs).where(and50(eq61(runs.id, runId), ...analyzableRunPredicates(projectId))).get();
+    if (!currentRunRecord) {
+      log.info("intelligence.skip", { runId, reason: "run not eligible for analysis" });
+      return null;
+    }
+    const currentLocation = currentRunRecord.location ?? null;
     const recentRuns = this.db.select().from(runs).where(
       and50(
-        eq61(runs.projectId, projectId),
-        or14(eq61(runs.status, "completed"), eq61(runs.status, "partial")),
-        // Defensive: RunCoordinator already skips probes before this is
-        // called, but if a future call site invokes analyzeAndPersist
-        // directly for a probe, probes still must not pollute the
-        // intelligence window.
-        ne8(runs.trigger, RunTriggers.probe)
+        ...analyzableRunPredicates(projectId),
+        currentLocation === null ? isNull8(runs.location) : eq61(runs.location, currentLocation),
+        // Never look forward. The window is this run's own past, so
+        // re-analyzing a historical run compares it against its true
+        // predecessor rather than against sweeps that came after it.
+        lte12(runChronologyKey, currentRunRecord.finishedAt ?? currentRunRecord.createdAt),
+        // MEASUREMENT: only runs that actually wrote snapshots — see above.
+        // Index seek on idx_snapshots_run, short-circuited on the first row.
+        exists(
+          this.db.select({ one: sql25`1` }).from(querySnapshots).where(eq61(querySnapshots.runId, runs.id))
+        )
       )
     ).orderBy(desc27(runs.finishedAt), desc27(runs.createdAt)).limit(HISTORY_WINDOW_RUNS).all();
-    if (recentRuns.length === 0) {
-      log.info("intelligence.skip", { runId, reason: "no completed runs" });
-      return null;
-    }
-    const currentRunRecord = recentRuns.find((r) => r.id === runId);
-    if (!currentRunRecord) {
-      log.info("intelligence.skip", { runId, reason: "run not in recent completed list" });
-      return null;
-    }
     const trackedCompetitors = this.loadTrackedCompetitors(projectId);
     const currentRun = this.buildRunData(
       runId,
@@ -77898,10 +77928,8 @@ var IntelligenceService = class {
       return null;
     }
     const orderedRecent = [...recentRuns].reverse();
-    const currentLocation = currentRunRecord.location ?? null;
-    const sameLocationOrdered = orderedRecent.filter((r) => (r.location ?? null) === currentLocation);
-    const currentLocIdx = sameLocationOrdered.findIndex((r) => r.id === runId);
-    const previousRunRecord = currentLocIdx > 0 ? sameLocationOrdered[currentLocIdx - 1] : null;
+    const currentIdx = orderedRecent.findIndex((r) => r.id === runId);
+    const previousRunRecord = currentIdx > 0 ? orderedRecent[currentIdx - 1] : null;
     const previousRun = previousRunRecord ? this.buildRunData(
       previousRunRecord.id,
       projectId,
@@ -77909,7 +77937,7 @@ var IntelligenceService = class {
       previousRunRecord.location ?? null,
       trackedCompetitors
     ) : null;
-    const history = sameLocationOrdered.slice(0, currentLocIdx + 1).map((r) => r.id === runId ? currentRun : this.buildRunData(
+    const history = orderedRecent.slice(0, currentIdx + 1).map((r) => r.id === runId ? currentRun : this.buildRunData(
       r.id,
       projectId,
       r.finishedAt ?? r.createdAt,
@@ -78184,14 +78212,9 @@ var IntelligenceService = class {
       }
       sinceTimestamp = parsed;
     }
-    const allRuns = this.db.select().from(runs).where(
-      and50(
-        eq61(runs.projectId, project.id),
-        or14(eq61(runs.status, "completed"), eq61(runs.status, "partial")),
-        // Backfill must not replay probe runs as if they were real sweeps.
-        ne8(runs.trigger, RunTriggers.probe)
-      )
-    ).orderBy(asc11(runs.finishedAt)).all();
+    const allRuns = this.db.select().from(runs).where(and50(...analyzableRunPredicates(project.id))).orderBy(asc11(runs.finishedAt)).all();
+    const measuredRunIds = this.runIdsWithSnapshots(allRuns.map((r) => r.id));
+    const measuredRuns = allRuns.filter((r) => measuredRunIds.has(r.id));
     let startIdx = 0;
     let endIdx = allRuns.length;
     if (opts?.fromRunId) {
@@ -78229,7 +78252,7 @@ var IntelligenceService = class {
     for (let i = 0; i < targetRuns.length; i++) {
       const run = targetRuns[i];
       const runLocation = run.location ?? null;
-      const sameLocationRuns = allRuns.filter((r) => (r.location ?? null) === runLocation);
+      const sameLocationRuns = measuredRuns.filter((r) => (r.location ?? null) === runLocation);
       const sameLocIdx = sameLocationRuns.indexOf(run);
       const previousRun = sameLocIdx > 0 ? sameLocationRuns[sameLocIdx - 1] : null;
       const historyStart = Math.max(0, sameLocIdx - (HISTORY_WINDOW_RUNS - 1));
@@ -78268,6 +78291,16 @@ var IntelligenceService = class {
   loadTrackedCompetitors(projectId) {
     return this.db.select({ domain: competitors.domain }).from(competitors).where(eq61(competitors.projectId, projectId)).all().map((r) => r.domain);
   }
+  /**
+   * Which of `runIds` actually wrote query snapshots. One grouped read, so
+   * callers can drop the runs that measured nothing before paying to build
+   * RunData for them. An empty input returns an empty set without querying.
+   */
+  runIdsWithSnapshots(runIds) {
+    if (runIds.length === 0) return /* @__PURE__ */ new Set();
+    const rows = this.db.select({ runId: querySnapshots.runId }).from(querySnapshots).where(inArray21(querySnapshots.runId, [...runIds])).groupBy(querySnapshots.runId).all();
+    return new Set(rows.map((r) => r.runId));
+  }
   /**
    * Wipe transition signals from an analysis result while keeping health.
    * Used when there's no baseline (first run) to avoid emitting false transitions.

package/dist/{chunk-XFAPF2YX.js → chunk-7ABFIVKO.js}

@@ -145,7 +145,7 @@ import {
   siteCrawlSnapshots,
   toAlertView,
   usageCounters
-} from "./chunk-GJE334DA.js";
+} from "./chunk-5EH666EO.js";
 import {
   AGENT_MEMORY_VALUE_MAX_BYTES,
   AGENT_PROVIDER_IDS,
@@ -11839,7 +11839,7 @@ function readStoredGroundingSources(rawResponse) {
   return result;
 }
 async function backfillInsightsCommand(project, opts) {
-  const { IntelligenceService: IntelligenceService2 } = await import("./intelligence-service-C57RIB37.js");
+  const { IntelligenceService: IntelligenceService2 } = await import("./intelligence-service-CHZGHFGS.js");
   const config = loadConfig();
   const db = createClient(config.database);
   migrate(db);

package/dist/cli.js

@@ -26,7 +26,7 @@ import {
   showFirstRunNotice,
   trackCliCommandFinished,
   trackEvent
-} from "./chunk-XFAPF2YX.js";
+} from "./chunk-7ABFIVKO.js";
 import {
   autoSyncSkills,
   formatAutoSyncNotice
@@ -69,7 +69,7 @@ import {
   projects,
   queries,
   renderReportHtml
-} from "./chunk-GJE334DA.js";
+} from "./chunk-5EH666EO.js";
 import {
   AdsDeliverySnapshotStatuses,
   AdsHistoricalCampaignRollupStatuses,

package/dist/index.js

@@ -3,11 +3,11 @@ import {
   createGoogleMarketingCredentialStore,
   createGoogleMarketingRuntime,
   createServer
-} from "./chunk-XFAPF2YX.js";
+} from "./chunk-7ABFIVKO.js";
 import {
   loadConfig
 } from "./chunk-6XUWDSLN.js";
-import "./chunk-GJE334DA.js";
+import "./chunk-5EH666EO.js";
 import "./chunk-PQT3ECM3.js";
 export {
   GoogleMarketingRuntimeError,

package/dist/{intelligence-service-C57RIB37.js → intelligence-service-CHZGHFGS.js}

@@ -1,6 +1,6 @@
 import {
   IntelligenceService
-} from "./chunk-GJE334DA.js";
+} from "./chunk-5EH666EO.js";
 import "./chunk-PQT3ECM3.js";
 export {
   IntelligenceService

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@canonry/canonry",
-  "version": "4.177.2",
+  "version": "4.177.3",
   "type": "module",
   "description": "Self-hosted AI visibility (AEO) platform: track how ChatGPT, Claude, Gemini, and Perplexity cite your domain, join it with Search Console, GA4, server-side traffic, and paid media, and fix what you find through agent tools (CLI, REST, MCP). Local SQLite.",
   "license": "FSL-1.1-ALv2",
@@ -71,30 +71,30 @@
     "tsup": "^8.5.1",
     "tsx": "^4.19.0",
     "@ainyc/canonry-api-client": "0.0.0",
-    "@ainyc/canonry-contracts": "0.0.0",
     "@ainyc/canonry-api-routes": "0.0.0",
     "@ainyc/canonry-config": "0.0.0",
     "@ainyc/canonry-db": "0.0.0",
+    "@ainyc/canonry-contracts": "0.0.0",
     "@ainyc/canonry-integration-bing": "0.0.0",
-    "@ainyc/canonry-integration-cloudflare-worker": "0.0.0",
-    "@ainyc/canonry-integration-cloudflare-queue": "0.0.0",
     "@ainyc/canonry-integration-cloud-run": "0.0.0",
-    "@ainyc/canonry-integration-google": "0.0.0",
+    "@ainyc/canonry-integration-cloudflare-queue": "0.0.0",
+    "@ainyc/canonry-integration-cloudflare-worker": "0.0.0",
     "@ainyc/canonry-integration-commoncrawl": "0.0.0",
     "@ainyc/canonry-integration-google-ads": "0.0.0",
     "@ainyc/canonry-integration-google-business-profile": "0.0.0",
+    "@ainyc/canonry-integration-google": "0.0.0",
     "@ainyc/canonry-integration-google-places": "0.0.0",
     "@ainyc/canonry-integration-google-tag-manager": "0.0.0",
+    "@ainyc/canonry-integration-traffic": "0.0.0",
     "@ainyc/canonry-integration-openai-ads": "0.0.0",
     "@ainyc/canonry-integration-wordpress": "0.0.0",
-    "@ainyc/canonry-intelligence": "0.0.0",
-    "@ainyc/canonry-provider-claude": "0.0.0",
+    "@ainyc/canonry-provider-gemini": "0.0.0",
     "@ainyc/canonry-provider-cdp": "0.0.0",
+    "@ainyc/canonry-intelligence": "0.0.0",
     "@ainyc/canonry-provider-local": "0.0.0",
     "@ainyc/canonry-provider-openai": "0.0.0",
-    "@ainyc/canonry-provider-gemini": "0.0.0",
-    "@ainyc/canonry-provider-perplexity": "0.0.0",
-    "@ainyc/canonry-integration-traffic": "0.0.0"
+    "@ainyc/canonry-provider-claude": "0.0.0",
+    "@ainyc/canonry-provider-perplexity": "0.0.0"
   },
   "scripts": {
     "build": "tsx scripts/copy-agent-assets.ts && tsup && tsx build-web.ts",