@ainyc/canonry 4.61.0 → 4.61.1

package/assets/agent-workspace/skills/aero/references/regression-playbook.md

@@ -51,3 +51,5 @@ A `gbp-sync` run produces a separate family of **location-scoped** insights (`pr
 | `gbp-keyword-drop` (high/medium) | A head local search term's impressions fell month-over-month | Check whether the property still ranks for the term; refresh the profile / categories. Needs ≥2 accumulated months of `gbp_keyword_monthly` history. |
 
 These flow through the same notification + proactive wake-up path as visibility insights, so you'll see them in the post-`gbp-sync` follow-up. Dismissals are location-scoped (one location's gap can be dismissed without silencing the same gap at a sibling location).
+
+**Calibrating `gbp-listing-discrepancy` (don't over-claim).** The Places cross-reference only sees a narrow, schema-bound amenity subset; the broader rendered hotel module (wifi, pool, room service, room rates) lives in Google Hotel Center, which the Places API can't read. Run live against a real hotel, it surfaced exactly **one** amenity (`wheelchair accessibility`). So quote the named amenities as concrete proof, but frame the discrepancy as a **floor** ("the public listing advertises at least X that your profile doesn't"), not a full inventory. The converse also holds: a thin or empty `gbp places` result is NOT evidence the public listing is bare; it means Places carries little structured data for that place. Either way the recommendation is the same and unchanged by the count: the owner controls the structured GBP attributes AI engines cite, so any amenity the profile fails to assert is a gap worth closing.

package/assets/agent-workspace/skills/canonry/references/google-business-profile.md

@@ -256,6 +256,7 @@ Validated against three live businesses of different types (a computer-support s
 - **Impressions skew to Maps for physical-destination businesses.** The hotel pulled 7,402 desktop-maps impressions vs 2,257 desktop-search in 30 days — people find it on Maps.
 - **Keyword thresholding scales with volume.** A busy hotel was ~89% thresholded (its head terms like `hotels`→10,939 had exact values); both small businesses were **100% thresholded** (every keyword redacted). For the typical SMB location, expect zero exact keyword values — design the UI to lead with the `<N` floor, not exact counts.
 - **Empty lodging / place-action profiles are the norm, and the emptiness is the product.** A real operating hotel returned a lodging resource with only `{ "name": ... }` (no amenities) and zero place-action links. That gap — "AI engines have no structured amenity data or direct-booking CTA to cite" — is exactly what canonry should surface, not an error to suppress.
+- **The Places cross-reference is a thin slice for hotels, not the full rendered listing.** Run live against the Venice Beach hotel at the Atmosphere tier, Place Details surfaced exactly one structured amenity, `wheelchair accessibility`, even though the rendered Google hotel module advertises far more (wifi, pool, room service, room rates). Those richer fields come from **Hotel Center**, which the Places API does not expose. So a thin or empty `gbp places` amenity list is NOT evidence the public listing is sparse; Places only carries a narrow, schema-bound subset (breakfast, dining, parking, pet-friendly, accessibility, restroom, family-friendly, outdoor seating, reservations). Read `gbp-listing-discrepancy` as a **floor** on the public-vs-owner gap (proof the listing advertises *at least* the named amenities), never a complete inventory. The owner-control point still stands at any size: even one amenity the profile fails to assert is a structured-data gap the operator can close.
 
 ## Troubleshooting
 

package/dist/{chunk-ZTVBTGDW.js → chunk-4FLI5VSQ.js}

@@ -93,7 +93,7 @@ import {
   runs,
   schedules,
   usageCounters
-} from "./chunk-QZN3J35I.js";
+} from "./chunk-QU62IX7K.js";
 import {
   AGENT_MEMORY_VALUE_MAX_BYTES,
   AGENT_PROVIDER_IDS,
@@ -5616,7 +5616,7 @@ function readStoredGroundingSources(rawResponse) {
   return result;
 }
 async function backfillInsightsCommand(project, opts) {
-  const { IntelligenceService: IntelligenceService2 } = await import("./intelligence-service-RWVBWSB5.js");
+  const { IntelligenceService: IntelligenceService2 } = await import("./intelligence-service-AZDX2EBS.js");
   const config = loadConfig();
   const db = createClient(config.database);
   migrate(db);

package/dist/{chunk-QZN3J35I.js → chunk-QU62IX7K.js}

@@ -31222,6 +31222,17 @@ function matchesBrandKey(candidateKey, brandKeys) {
 var RECURRENCE_LOOKBACK_RUNS = 5;
 var HISTORY_WINDOW_RUNS = Math.max(PERSISTENT_GAP_THRESHOLD, 5);
 var log = createLogger("IntelligenceService");
+function gbpInsightSlot(typeOrSlot) {
+  return typeOrSlot === "gbp-lodging-gap" || typeOrSlot === "gbp-listing-discrepancy" ? "lodging" : typeOrSlot;
+}
+function gbpInsightId(projectId, locationName, type) {
+  return `${projectId}::gbp::${locationName}::${gbpInsightSlot(type)}`;
+}
+function parseGbpInsightId(id) {
+  const parts = id.split("::");
+  if (parts.length !== 4 || parts[1] !== "gbp") return null;
+  return { location: parts[2], slot: gbpInsightSlot(parts[3]) };
+}
 var IntelligenceService = class {
   constructor(db) {
     this.db = db;
@@ -31320,7 +31331,7 @@ var IntelligenceService = class {
     const runRow = this.db.select({ createdAt: runs.createdAt, startedAt: runs.startedAt, finishedAt: runs.finishedAt }).from(runs).where(eq30(runs.id, runId)).get();
     if (!runRow) {
       log.info("gbp-intelligence.skip", { runId, reason: "run not found" });
-      this.persistGbpInsights(runId, projectId, []);
+      this.persistGbpInsights(runId, projectId, [], []);
       return [];
     }
     const windowStart = runRow.startedAt ?? runRow.createdAt;
@@ -31333,7 +31344,7 @@ var IntelligenceService = class {
     )).all();
     if (selected.length === 0) {
       log.info("gbp-intelligence.skip", { runId, reason: "no locations synced during run" });
-      this.persistGbpInsights(runId, projectId, []);
+      this.persistGbpInsights(runId, projectId, [], []);
       return [];
     }
     const today = (/* @__PURE__ */ new Date()).toISOString().slice(0, 10);
@@ -31343,10 +31354,11 @@ var IntelligenceService = class {
     const drafts = analyzeGbp(signals);
     const now = (/* @__PURE__ */ new Date()).toISOString();
     const builtInsights = drafts.map((d) => ({
-      // Stable id: unique per (run, location, type). Embedding runId keeps the
-      // PK unique across runs; embedding locationName keeps two chain locations
-      // that share a displayName from colliding.
-      id: `${runId}::gbp::${d.locationName}::${d.type}`,
+      // Stable, run-independent id per (project, location, slot) so each sync
+      // REPLACES the prior insight for that slot instead of appending a per-run
+      // copy. The lodging slot is shared by gbp-lodging-gap and its
+      // gbp-listing-discrepancy upgrade, so the latter supersedes the former.
+      id: gbpInsightId(projectId, d.locationName, d.type),
       type: d.type,
       severity: d.severity,
       title: d.title,
@@ -31355,7 +31367,7 @@ var IntelligenceService = class {
       recommendation: d.recommendation,
       createdAt: now
     }));
-    this.persistGbpInsights(runId, projectId, builtInsights);
+    this.persistGbpInsights(runId, projectId, builtInsights, signals.map((s) => s.locationName));
     log.info("gbp-intelligence.analyzed", { runId, locations: selected.length, insights: builtInsights.length });
     return builtInsights;
   }
@@ -31416,20 +31428,34 @@ var IntelligenceService = class {
     return { recentMonth, priorMonth, points };
   }
   /**
-   * Persist GBP insights for a run. Mirrors `persistResult`'s idempotency
-   * (delete-then-insert for the run) and dismissal preservation, but keyed by
-   * the stable insight id (GBP insights aren't query/provider-scoped) and
-   * WITHOUT a health snapshot.
+   * Persist GBP insights as CURRENT STATE for the locations a sync evaluated.
+   * GBP insights are point-in-time profile facts, not run-to-run transitions, so
+   * each sync REPLACES the prior insights for the locations it covered — keyed by
+   * the run-independent `${projectId}::gbp::${location}::${slot}` id — instead of
+   * appending a fresh per-run copy. This collapses run-over-run duplicates,
+   * supersedes `gbp-lodging-gap` with `gbp-listing-discrepancy` (shared `lodging`
+   * slot), and clears a location's stale insight once its gap is resolved.
+   * Dismissals are preserved by (location, slot). Locations NOT covered by this
+   * run (e.g. a `--location`-scoped sync) are left untouched.
    */
-  persistGbpInsights(runId, projectId, gbpInsights) {
-    const previouslyDismissed = /* @__PURE__ */ new Set();
-    const existing = this.db.select({ id: insights.id, dismissed: insights.dismissed }).from(insights).where(eq30(insights.runId, runId)).all();
+  persistGbpInsights(runId, projectId, gbpInsights, coveredLocationNames) {
+    const covered = new Set(coveredLocationNames);
+    const existing = this.db.select({ id: insights.id, dismissed: insights.dismissed }).from(insights).where(and23(eq30(insights.projectId, projectId), eq30(insights.provider, GBP_INSIGHT_PROVIDER))).all();
+    const staleIds = [];
+    const dismissedSlots = /* @__PURE__ */ new Set();
     for (const row of existing) {
-      if (row.dismissed) previouslyDismissed.add(row.id);
+      const parsed = parseGbpInsightId(row.id);
+      if (!parsed) continue;
+      if (covered.has(parsed.location)) staleIds.push(row.id);
+      if (row.dismissed) dismissedSlots.add(`${parsed.location}::${parsed.slot}`);
     }
     this.db.transaction((tx) => {
-      tx.delete(insights).where(eq30(insights.runId, runId)).run();
+      for (const id of staleIds) {
+        tx.delete(insights).where(eq30(insights.id, id)).run();
+      }
       for (const insight of gbpInsights) {
+        const parsed = parseGbpInsightId(insight.id);
+        const dismissed = parsed ? dismissedSlots.has(`${parsed.location}::${parsed.slot}`) : false;
         tx.insert(insights).values({
           id: insight.id,
           projectId,
@@ -31441,12 +31467,12 @@ var IntelligenceService = class {
           provider: insight.provider,
           recommendation: insight.recommendation ?? null,
           cause: insight.cause ?? null,
-          dismissed: previouslyDismissed.has(insight.id),
+          dismissed,
           createdAt: insight.createdAt
         }).run();
       }
     });
-    log.info("gbp-intelligence.persisted", { runId, insights: gbpInsights.length });
+    log.info("gbp-intelligence.persisted", { runId, insights: gbpInsights.length, replaced: staleIds.length });
   }
   /**
    * Analyze a single run given an explicit previous run (or null for first run).

package/dist/cli.js

@@ -27,7 +27,7 @@ import {
   setTelemetrySource,
   showFirstRunNotice,
   trackEvent
-} from "./chunk-ZTVBTGDW.js";
+} from "./chunk-4FLI5VSQ.js";
 import {
   CliError,
   EXIT_SYSTEM_ERROR,
@@ -51,7 +51,7 @@ import {
   projects,
   queries,
   renderReportHtml
-} from "./chunk-QZN3J35I.js";
+} from "./chunk-QU62IX7K.js";
 import {
   CcReleaseSyncStatuses,
   CheckScopes,

package/dist/index.js

@@ -1,10 +1,10 @@
 import {
   createServer
-} from "./chunk-ZTVBTGDW.js";
+} from "./chunk-4FLI5VSQ.js";
 import {
   loadConfig
 } from "./chunk-PITZUUFV.js";
-import "./chunk-QZN3J35I.js";
+import "./chunk-QU62IX7K.js";
 import "./chunk-URPUUKLC.js";
 export {
   createServer,

package/dist/{intelligence-service-RWVBWSB5.js → intelligence-service-AZDX2EBS.js}

@@ -1,6 +1,6 @@
 import {
   IntelligenceService
-} from "./chunk-QZN3J35I.js";
+} from "./chunk-QU62IX7K.js";
 import "./chunk-URPUUKLC.js";
 export {
   IntelligenceService

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ainyc/canonry",
-  "version": "4.61.0",
+  "version": "4.61.1",
   "type": "module",
   "description": "Agent-first open-source AEO operating platform - track how answer engines cite your domain",
   "license": "FSL-1.1-ALv2",
@@ -61,26 +61,26 @@
     "@types/node-cron": "^3.0.11",
     "tsup": "^8.5.1",
     "tsx": "^4.19.0",
-    "@ainyc/canonry-config": "0.0.0",
-    "@ainyc/canonry-api-client": "0.0.0",
     "@ainyc/canonry-api-routes": "0.0.0",
+    "@ainyc/canonry-config": "0.0.0",
+    "@ainyc/canonry-contracts": "0.0.0",
     "@ainyc/canonry-db": "0.0.0",
     "@ainyc/canonry-integration-bing": "0.0.0",
-    "@ainyc/canonry-contracts": "0.0.0",
     "@ainyc/canonry-integration-cloud-run": "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-api-client": "0.0.0",
     "@ainyc/canonry-integration-commoncrawl": "0.0.0",
+    "@ainyc/canonry-integration-google": "0.0.0",
+    "@ainyc/canonry-integration-google-business-profile": "0.0.0",
     "@ainyc/canonry-integration-traffic": "0.0.0",
-    "@ainyc/canonry-intelligence": "0.0.0",
+    "@ainyc/canonry-integration-wordpress": "0.0.0",
     "@ainyc/canonry-provider-cdp": "0.0.0",
     "@ainyc/canonry-provider-claude": "0.0.0",
-    "@ainyc/canonry-integration-wordpress": "0.0.0",
+    "@ainyc/canonry-intelligence": "0.0.0",
+    "@ainyc/canonry-provider-local": "0.0.0",
     "@ainyc/canonry-provider-gemini": "0.0.0",
     "@ainyc/canonry-provider-openai": "0.0.0",
     "@ainyc/canonry-provider-perplexity": "0.0.0",
-    "@ainyc/canonry-provider-local": "0.0.0"
+    "@ainyc/canonry-integration-google-places": "0.0.0"
   },
   "scripts": {
     "build": "tsx scripts/copy-agent-assets.ts && tsup && tsx build-web.ts",