@canonry/canonry 7.13.0 → 7.14.0

package/dist/{chunk-GBH7N3VZ.js → chunk-KXHPQMBP.js}

@@ -20,20 +20,20 @@ import {
   saveConfig,
   saveConfigPatch,
   sentimentInstallReadiness
-} from "./chunk-SM6ZZAPK.js";
+} from "./chunk-ZHXOR2LJ.js";
 import {
   computeFactorAverages,
   deleteSiteCrawlGraphLayout,
   persistSiteCrawlGraphLayout,
   prepareSiteCrawlGraphLayout,
   recordedSharePct
-} from "./chunk-7I64KEGM.js";
+} from "./chunk-WHVEDCHY.js";
 import {
   CanonryMcpToolNames,
   CliError,
   PACKAGE_VERSION,
   canonryMcpTools
-} from "./chunk-BTHEFJCI.js";
+} from "./chunk-4LQ4HKVK.js";
 import {
   CC_CACHE_DIR,
   DUCKDB_SPEC,
@@ -217,7 +217,7 @@ import {
   siteLivenessState,
   toAlertView,
   usageCounters
-} from "./chunk-GQMSJEMZ.js";
+} from "./chunk-EZRBFH77.js";
 import {
   AGENT_MEMORY_VALUE_MAX_BYTES,
   AGENT_NONE,
@@ -309,7 +309,6 @@ import {
   canonicalizeGtmContainerId,
   canonicalizeGtmResourceSelection,
   classifyProviderErrorMessage,
-  classifyProviderErrorMessages,
   classifyTemplateLinkEdge,
   cloudflareQueueNameSchema,
   compareSemver,
@@ -378,6 +377,7 @@ import {
   maskTelemetryAnonymousId,
   matcherMatchesText,
   missingDependency,
+  mostActionableProviderErrorCode,
   nextScheduleUpdatedAt,
   normalizeAgentSlug,
   normalizeIdTokens,
@@ -428,8 +428,9 @@ import {
   validationError,
   visibilityReportRequestSchema,
   winnabilityClassLabel,
-  withRetry
-} from "./chunk-AU7OUDWR.js";
+  withRetry,
+  withSkippedProviders
+} from "./chunk-A25FO3Z5.js";
 
 // src/runtime-context.ts
 function cliRuntimeContext(env = process.env, stdio = process) {
@@ -8041,7 +8042,7 @@ function nonBlank(value) {
   return trimmed ? trimmed : void 0;
 }
 function resolveBuildCommit(env = process.env) {
-  const embedded = true ? "c2de5a23906b2d9fff8c2543a55d52dd9cbcb889" : void 0;
+  const embedded = true ? "8eb9720cf39e4787ffa530b4083372db8c39ae8f" : void 0;
   return nonBlank(embedded) ?? nonBlank(env.CANONRY_COMMIT);
 }
 function resolveInstanceIdentity(env = process.env) {
@@ -8123,12 +8124,17 @@ function buildSiteAuditCompletedProps(input) {
   return props;
 }
 var MAX_NESTED_KEYS = 12;
-function buildProviderOutcomeProps(providers, providerErrors) {
-  if (providerErrors.size === 0) return {};
-  const names = [.../* @__PURE__ */ new Set([...providers, ...providerErrors.keys()])].slice(0, MAX_NESTED_KEYS);
+function buildProviderOutcomeProps(providers, providerErrors, skippedProviders = []) {
+  if (providerErrors.size === 0 && skippedProviders.length === 0) return {};
+  const skipped = new Set(skippedProviders);
+  const names = [.../* @__PURE__ */ new Set([...providers, ...providerErrors.keys(), ...skipped])].slice(0, MAX_NESTED_KEYS);
   const providerOutcomes = {};
   const providerHttpStatus = {};
   for (const name of names) {
+    if (skipped.has(name)) {
+      providerOutcomes[name] = "skipped";
+      continue;
+    }
     const message = providerErrors.get(name);
     if (message === void 0) {
       providerOutcomes[name] = "ok";
@@ -8140,6 +8146,20 @@ function buildProviderOutcomeProps(providers, providerErrors) {
   }
   return Object.keys(providerHttpStatus).length > 0 ? { providerOutcomes, providerHttpStatus } : { providerOutcomes };
 }
+function buildRunRefusedProps(input) {
+  const outcomes = Object.entries(input.providers).slice(0, MAX_NESTED_KEYS);
+  const props = {
+    reason: "providers_failing",
+    providerCount: Object.keys(input.providers).length,
+    providers: Object.keys(input.providers),
+    providerOutcomes: Object.fromEntries(outcomes),
+    trigger: input.trigger
+  };
+  const domainHash = hashDomain(input.canonicalDomain ?? null);
+  if (domainHash) props.domainHash = domainHash;
+  if (input.location) props.location = input.location;
+  return props;
+}
 function runFailureSite(providerCallStart, providerCallEnd) {
   if (providerCallStart === void 0) return "setup";
   return providerCallEnd === void 0 ? "provider_call" : "finalize";
@@ -8455,12 +8475,16 @@ function chunkBatchLines(lines, maxRequests, maxBytes) {
   if (current.length > 0) chunks.push(current);
   return { chunks, oversized };
 }
-function runOutcome(inserted, providerErrors, planShortfall) {
-  const someFailed = providerErrors.size > 0 || planShortfall > 0;
+function calledProviderErrors(providerErrors, skipped) {
+  return new Map([...providerErrors].filter(([provider]) => !(provider in skipped)));
+}
+function runOutcome(inserted, providerErrors, planShortfall, skipped) {
+  const errors = calledProviderErrors(providerErrors, skipped);
+  const someFailed = errors.size > 0 || planShortfall > 0 || Object.keys(skipped).length > 0;
   const allFailed = inserted === 0 && someFailed;
   return {
     status: allFailed ? RunStatuses.failed : someFailed ? RunStatuses.partial : RunStatuses.completed,
-    error: someFailed ? serializeRunError(buildProviderRunError(providerErrors)) : null
+    error: someFailed ? serializeRunError(withSkippedProviders(buildProviderRunError(errors), skipped)) : null
   };
 }
 function runRecordingContext(db, input) {
@@ -8518,8 +8542,12 @@ function classifyRunAbortReason(message) {
   return void 0;
 }
 var FAILURE_STREAK_LOOKBACK = 50;
-function classifyProviderErrors(errors) {
-  return classifyProviderErrorMessages(errors.values());
+function classifyProviderErrors(errors, skipped) {
+  const codes = [
+    ...[...errors.values()].map(classifyProviderErrorMessage),
+    ...Object.values(skipped).map((streak) => streak.code)
+  ];
+  return codes.length > 0 ? mostActionableProviderErrorCode(codes) : void 0;
 }
 var JobRunner = class {
   /**
@@ -8739,7 +8767,14 @@ var JobRunner = class {
       if (activeProviders.length === 0) {
         throw new Error("No providers configured. Add at least one provider API key.");
       }
-      log2.info("run.dispatch", { runId, providerCount: activeProviders.length, providers: activeProviders.map((p) => p.adapter.name) });
+      const skipped = existingRun.skippedProviders ?? {};
+      const calledProviders = activeProviders.filter((provider) => !(provider.adapter.name in skipped));
+      log2.info("run.dispatch", {
+        runId,
+        providerCount: calledProviders.length,
+        providers: calledProviders.map((p) => p.adapter.name),
+        ...Object.keys(skipped).length > 0 ? { skippedProviders: Object.keys(skipped) } : {}
+      });
       const projectCompetitors = this.db.select().from(competitors).where(eq3(competitors.projectId, projectId)).orderBy(competitors.domain).all();
       const competitorDomains = projectCompetitors.map((c) => c.domain);
       recording = runRecordingContext(this.db, {
@@ -8761,7 +8796,7 @@ var JobRunner = class {
       };
       const queriesPerProvider = planExecution?.maxUnitsPerProvider ?? projectQueries.length;
       const todayPeriod = getCurrentUsageDay();
-      for (const p of activeProviders) {
+      for (const p of calledProviders) {
         const providerScope = `${projectId}:${p.adapter.name}`;
         const limit = p.config.quotaPolicy.maxRequestsPerDay;
         const quota = reserveDailyQueryQuota(this.db, { scope: providerScope, period: todayPeriod, count: queriesPerProvider, limit });
@@ -8773,7 +8808,7 @@ var JobRunner = class {
         providerReservations.set(p.adapter.name, { scope: providerScope, period: todayPeriod, reserved: queriesPerProvider });
       }
       const executionGates = /* @__PURE__ */ new Map();
-      for (const provider of activeProviders) {
+      for (const provider of calledProviders) {
         executionGates.set(
           provider.adapter.name,
           getSharedProviderExecutionGate(
@@ -8785,8 +8820,8 @@ var JobRunner = class {
       }
       const providerErrors = /* @__PURE__ */ new Map();
       let totalSnapshotsInserted = 0;
-      const apiProviders = activeProviders.filter((p) => !isBrowserProvider(p.adapter.name));
-      const browserProviders = activeProviders.filter((p) => isBrowserProvider(p.adapter.name));
+      const apiProviders = calledProviders.filter((p) => !isBrowserProvider(p.adapter.name));
+      const browserProviders = calledProviders.filter((p) => isBrowserProvider(p.adapter.name));
       const processQueryForProvider = async (registeredProvider, q) => {
         const { adapter, config } = registeredProvider;
         const providerName = adapter.name;
@@ -9052,7 +9087,7 @@ var JobRunner = class {
         for (const [provider, units] of planExecution.unitsByProvider) {
           const providerName = provider;
           const dispatched = providerDispatchCounts.get(providerName) ?? 0;
-          if (dispatched >= units.length || providerErrors.has(providerName)) continue;
+          if (dispatched >= units.length || providerErrors.has(providerName) || providerName in skipped) continue;
           providerErrors.set(
             providerName,
             `No ${provider} provider was available to this worker, so ${units.length - dispatched} expected measurement(s) did not run.`
@@ -9077,6 +9112,7 @@ var JobRunner = class {
         inserted: totalSnapshotsInserted,
         providerErrors,
         planShortfall,
+        skippedProviders: skipped,
         executionContext,
         startTime,
         phases: buildPhases({ startTime, providerCallStart, providerCallEnd }),
@@ -9181,7 +9217,7 @@ var JobRunner = class {
    */
   finalizeRun(input) {
     const { runId, providerErrors } = input;
-    const outcome = runOutcome(input.inserted, providerErrors, input.planShortfall);
+    const outcome = runOutcome(input.inserted, providerErrors, input.planShortfall, input.skippedProviders ?? {});
     const finishedAt = (/* @__PURE__ */ new Date()).toISOString();
     const won = this.db.transaction((tx) => {
       const changed = tx.update(runs).set({
@@ -9207,8 +9243,10 @@ var JobRunner = class {
    * post-run pipeline.
    */
   reportFinalizedRun(input, finalStatus) {
-    const { runId, projectId, providerErrors, executionContext } = input;
-    const failureCode = providerErrors.size > 0 ? classifyProviderErrors(providerErrors) : void 0;
+    const { runId, projectId, executionContext } = input;
+    const skippedProviders = input.skippedProviders ?? {};
+    const providerErrors = calledProviderErrors(input.providerErrors, skippedProviders);
+    const failureCode = classifyProviderErrors(providerErrors, skippedProviders);
     const sampling = failureStreakSampling(finalStatus, this.priorFailureStreak(projectId, runId), runId);
     if (sampling.report) {
       trackEvent(
@@ -9225,7 +9263,7 @@ var JobRunner = class {
             phases: input.phases,
             location: executionContext.location
           }),
-          ...buildProviderOutcomeProps(executionContext.providers, providerErrors),
+          ...buildProviderOutcomeProps(executionContext.providers, providerErrors, Object.keys(skippedProviders)),
           ...sampling.props
         },
         failureCode ? { errorCode: failureCode } : void 0
@@ -9295,15 +9333,21 @@ var JobRunner = class {
     const finishedAt = (/* @__PURE__ */ new Date()).toISOString();
     const decided = this.db.transaction((tx) => {
       const txDb = tx;
-      const run = txDb.select({ kind: runs.kind, status: runs.status, pendingProviderErrors: runs.pendingProviderErrors }).from(runs).where(and3(eq3(runs.id, runId), eq3(runs.projectId, projectId))).get();
+      const run = txDb.select({
+        kind: runs.kind,
+        status: runs.status,
+        pendingProviderErrors: runs.pendingProviderErrors,
+        skippedProviders: runs.skippedProviders
+      }).from(runs).where(and3(eq3(runs.id, runId), eq3(runs.projectId, projectId))).get();
       if (!run || run.status !== RunStatuses.running || run.pendingProviderErrors === null) return null;
       const holding = txDb.select({ id: providerBatches.id }).from(providerBatches).where(and3(eq3(providerBatches.runId, runId), inArray(providerBatches.status, RUN_HOLDING_BATCH_STATUSES))).limit(1).get();
       if (holding) return null;
       const state = measurementRunSlotState(txDb, runId);
       const inserted = txDb.select({ value: count() }).from(querySnapshots).where(eq3(querySnapshots.runId, runId)).get()?.value ?? 0;
       const providerErrors = this.batchRunProviderErrors(txDb, runId, run.pendingProviderErrors, state.missing);
+      const skippedProviders = run.skippedProviders ?? {};
       const shortfall = state.missing.length + (state.readable ? 0 : 1);
-      const outcome = runOutcome(inserted, providerErrors, shortfall);
+      const outcome = runOutcome(inserted, providerErrors, shortfall, skippedProviders);
       const won = txDb.update(runs).set({
         status: outcome.status,
         finishedAt,
@@ -9312,7 +9356,7 @@ var JobRunner = class {
       }).where(and3(eq3(runs.id, runId), eq3(runs.status, RunStatuses.running), isNotNull(runs.pendingProviderErrors))).run().changes === 1;
       if (!won) return null;
       recordSentimentCompletion(tx, { projectId, runId, completionKey: "initial", completedAt: finishedAt });
-      return { kind: run.kind, inserted, providerErrors, status: outcome.status };
+      return { kind: run.kind, inserted, providerErrors, skippedProviders, status: outcome.status };
     });
     if (!decided) return false;
     const telemetry = this.batchRunTelemetry(runId, projectId);
@@ -9323,6 +9367,7 @@ var JobRunner = class {
       kind: decided.kind,
       inserted: decided.inserted,
       providerErrors: decided.providerErrors,
+      skippedProviders: decided.skippedProviders,
       executionContext: telemetry.executionContext,
       startTime: telemetry.startTime,
       // The sweep's phase split does not survive the wait (or a restart);
@@ -10373,7 +10418,8 @@ var JobRunner = class {
       measurementPlanVersionId: runs.measurementPlanVersionId,
       measurementManifest: runs.measurementManifest,
       providerDispatchModes: runs.providerDispatchModes,
-      pendingProviderErrors: runs.pendingProviderErrors
+      pendingProviderErrors: runs.pendingProviderErrors,
+      skippedProviders: runs.skippedProviders
     }).from(runs).where(eq3(runs.id, runId)).get();
   }
   isRunCancelled(runId) {
@@ -15120,6 +15166,9 @@ var Scheduler = class {
         runnableProviders: this.callbacks.getRunnableProviderNames?.(),
         providerModels: this.callbacks.getEffectiveProviderModels?.(),
         batchEligibleProviders: this.callbacks.getBatchEligibleProviderNames?.() ?? null,
+        // A skipped slot is otherwise invisible: no run row, nothing in the
+        // dashboard. One audit row per refusal keeps it in the history.
+        auditRefusal: { actor: "scheduler", entityType: "schedule", entityId: currentSchedule.id },
         ...claimedOccurrence && recurrence && currentSchedule.nextRunAt === claimedOccurrence ? {
           scheduleClaim: {
             scheduleId: currentSchedule.id,
@@ -15149,6 +15198,14 @@ var Scheduler = class {
           scheduleId: currentSchedule.id,
           ...queueResult.refused
         });
+        if (queueResult.refusalRecorded) {
+          trackEvent("run.aborted", buildRunRefusedProps({
+            providers: queueResult.refused.providers,
+            trigger: RunTriggers.scheduled,
+            canonicalDomain: project.canonicalDomain,
+            location: locationLabel
+          }), { errorCode: "PROVIDERS_FAILING" });
+        }
         if (!claimedOccurrence) this.updateScheduleTiming(currentSchedule.id, { nextRunAt });
         return;
       }
@@ -15169,7 +15226,12 @@ var Scheduler = class {
         lastRunAt: now,
         nextRunAt
       });
-      log16.info("run.triggered", { runId, projectName: project.name, providers: providers ?? "all" });
+      log16.info("run.triggered", {
+        runId,
+        projectName: project.name,
+        providers: providers ?? "all",
+        ...Object.keys(queueResult.skippedProviders).length > 0 ? { skippedProviders: queueResult.skippedProviders } : {}
+      });
       this.callbacks.onRunCreated(runId, projectId, providers, resolvedLocation);
     } catch (err) {
       log16.error("trigger.error", { scheduleId, projectId, kind, error: describeError(err) });

package/dist/{chunk-EPSHIH63.js → chunk-LUQ5G5OO.js}

@@ -4,14 +4,14 @@ import {
   installSkills,
   loadConfigRaw,
   saveConfigPatch
-} from "./chunk-SM6ZZAPK.js";
+} from "./chunk-ZHXOR2LJ.js";
 import {
   PACKAGE_VERSION
-} from "./chunk-BTHEFJCI.js";
+} from "./chunk-4LQ4HKVK.js";
 import {
   SKILL_MANIFEST_FILENAME,
   SkillsClients
-} from "./chunk-AU7OUDWR.js";
+} from "./chunk-A25FO3Z5.js";
 
 // src/skills-autosync.ts
 import fs from "fs";

package/dist/{chunk-7I64KEGM.js → chunk-WHVEDCHY.js}

@@ -5,7 +5,7 @@ import {
   siteCrawlGraphNodes,
   siteCrawlPages,
   siteCrawlSnapshots
-} from "./chunk-GQMSJEMZ.js";
+} from "./chunk-EZRBFH77.js";
 import {
   SITE_CRAWL_GRAPH_MAX_EDGES,
   SITE_CRAWL_GRAPH_MAX_NODES,
@@ -14,7 +14,7 @@ import {
   factorStatusFromDistribution,
   factorStatusFromScore,
   roundPreservingTotal
-} from "./chunk-AU7OUDWR.js";
+} from "./chunk-A25FO3Z5.js";
 
 // src/site-audit-factors.ts
 function recordedSharePct(factor) {

package/dist/{chunk-SM6ZZAPK.js → chunk-ZHXOR2LJ.js}

@@ -5,7 +5,7 @@ import {
   PACKAGE_VERSION,
   canonryMcpTools,
   isMachineFormat
-} from "./chunk-BTHEFJCI.js";
+} from "./chunk-4LQ4HKVK.js";
 import {
   AGENT_PROVIDER_IDS,
   CANONRY_NPM_PACKAGE_URL,
@@ -50,7 +50,7 @@ import {
   skillsClientSchema,
   upgradeCaveatFor,
   upgradeCommandFor
-} from "./chunk-AU7OUDWR.js";
+} from "./chunk-A25FO3Z5.js";
 
 // src/config.ts
 import fs from "fs";