@amityco/social-plus-vise 1.0.1 → 1.1.0

package/dist/tools/docs.js

@@ -1,4 +1,6 @@
-import { readdir, readFile } from "node:fs/promises";
+import { createHash } from "node:crypto";
+import { mkdir, readdir, readFile, rename, stat, writeFile } from "node:fs/promises";
+import os from "node:os";
 import path from "node:path";
 import { fileURLToPath } from "node:url";
 import { objectInput, optionalNumberField, stringField, textResult } from "../types.js";
@@ -11,18 +13,6 @@ let hostedCachePending;
 function localDocsRoot() {
     return process.env.SOCIAL_PLUS_DOCS_ROOT ? path.resolve(process.env.SOCIAL_PLUS_DOCS_ROOT) : undefined;
 }
-// Hermetic ("offline") docs mode. When VISE_DOCS_OFFLINE is set truthy, docs
-// lookups are served from a committed, bundled corpus instead of the hosted
-// learn.social.plus endpoint — so CI (and offline installs) never depend on the
-// docs site being reachable. This kills the learn.social.plus flake class that
-// once hit test:cli. The live-fetch path stays the default when neither
-// VISE_DOCS_OFFLINE nor SOCIAL_PLUS_DOCS_ROOT is set, so real-fetch coverage is
-// preserved (see vise/package.json test:docs-live).
-//
-// Resolution precedence in loadDocPages():
-//   1. SOCIAL_PLUS_DOCS_ROOT (explicit local override) — highest, unchanged.
-//   2. VISE_DOCS_OFFLINE      — serve the bundled docs-cache/ corpus.
-//   3. neither                — live fetch from the hosted docs (default).
 function docsOfflineMode() {
     const raw = process.env.VISE_DOCS_OFFLINE;
     if (raw === undefined) {
@@ -31,10 +21,6 @@ function docsOfflineMode() {
     const normalized = raw.trim().toLowerCase();
     return normalized !== "" && normalized !== "0" && normalized !== "false" && normalized !== "off" && normalized !== "no";
 }
-// The bundled hermetic corpus ships in the package `files` allowlist as
-// `docs-cache/`. At runtime this module is dist/tools/docs.js, so the corpus is
-// two directories up. An explicit SOCIAL_PLUS_DOCS_OFFLINE_ROOT override is
-// honoured for tests that want to point offline mode at a fixture corpus.
 function bundledDocsCacheRoot() {
     const override = process.env.SOCIAL_PLUS_DOCS_OFFLINE_ROOT;
     if (override) {
@@ -62,6 +48,40 @@ function cacheTtlMs() {
     const value = Number(raw);
     return Number.isFinite(value) && value >= 0 ? value : DEFAULT_CACHE_TTL_MS;
 }
+function docsDiskCacheFile() {
+    const dir = process.env.VISE_DOCS_CACHE_DIR ?? path.join(os.tmpdir(), "vise-docs-cache");
+    const key = createHash("sha256").update(docsBaseUrl()).digest("hex").slice(0, 16);
+    return path.join(dir, `llms-full.${key}.txt`);
+}
+async function readDiskCorpus() {
+    const ttl = cacheTtlMs();
+    if (ttl === 0)
+        return undefined;
+    const file = docsDiskCacheFile();
+    try {
+        const info = await stat(file);
+        if (Date.now() - info.mtimeMs >= ttl)
+            return undefined;
+        const pages = parseLlmsFull(await readFile(file, "utf8"), docsBaseUrl());
+        return pages.length >= MIN_EXPECTED_PAGES ? pages : undefined;
+    }
+    catch {
+        return undefined;
+    }
+}
+async function writeDiskCorpus(content) {
+    if (cacheTtlMs() === 0)
+        return;
+    const file = docsDiskCacheFile();
+    try {
+        await mkdir(path.dirname(file), { recursive: true });
+        const tmp = `${file}.tmp.${process.pid}`;
+        await writeFile(tmp, content, "utf8");
+        await rename(tmp, file);
+    }
+    catch {
+    }
+}
 export const searchDocsTool = {
     name: "search_docs",
     description: "Search social.plus documentation. For implementation requests, call plan_integration first so required inputs and stop conditions are known before docs lookup.",
@@ -143,7 +163,6 @@ async function loadDocPages() {
     if (root) {
         return loadLocalDocPages(root);
     }
-    // Hermetic mode: serve the committed bundled corpus, never the network.
     if (docsOfflineMode()) {
         const cacheRoot = bundledDocsCacheRoot();
         const pages = await loadLocalDocPages(cacheRoot);
@@ -157,7 +176,7 @@ async function loadDocPages() {
     if (hostedCacheEntry && now - hostedCacheEntry.fetchedAt < cacheTtlMs()) {
         return hostedCacheEntry.pages;
     }
-    hostedCachePending ??= loadHostedDocPages()
+    hostedCachePending ??= resolveHostedDocPages()
         .then((pages) => {
         hostedCacheEntry = { pages, fetchedAt: Date.now() };
         return pages;
@@ -167,6 +186,20 @@ async function loadDocPages() {
     });
     return hostedCachePending;
 }
+async function resolveHostedDocPages() {
+    const cached = await readDiskCorpus();
+    if (cached)
+        return cached;
+    try {
+        return await loadHostedDocPages();
+    }
+    catch (error) {
+        const bundled = await loadLocalDocPages(bundledDocsCacheRoot()).catch(() => []);
+        if (bundled.length > 0)
+            return bundled;
+        throw error;
+    }
+}
 async function loadHostedDocPages() {
     const baseUrl = docsBaseUrl();
     const fullUrl = `${baseUrl}/llms-full.txt`;
@@ -174,6 +207,7 @@ async function loadHostedDocPages() {
     const failures = [];
     try {
         const content = await fetchText(fullUrl);
+        await writeDiskCorpus(content);
         return parseLlmsFull(content, baseUrl);
     }
     catch (error) {
@@ -204,9 +238,6 @@ async function loadLocalDocPages(root) {
 async function fetchText(url) {
     return fetchTextWithTimeout(url, "text/plain, text/markdown, */*");
 }
-// Shared network primitive — same AbortController + env-configurable timeout used for
-// docs fetches. Exported so other plan-layer lookups (e.g. the npm registry SDK-version
-// check) reuse one network path rather than adding a parallel one.
 export async function fetchTextWithTimeout(url, accept = "*/*", timeoutMs = fetchTimeoutMs()) {
     const controller = new AbortController();
     const timeout = setTimeout(() => controller.abort(), timeoutMs);
@@ -314,8 +345,6 @@ async function findDocFiles(root) {
             if (entry.name === "node_modules" || entry.name === ".git" || entry.name === ".next") {
                 continue;
             }
-            // A README is corpus metadata (e.g. the docs-cache/ provenance note), not
-            // an SDK doc page — don't ingest it as a searchable page.
             if (/^readme\.(md|mdx)$/i.test(entry.name)) {
                 continue;
             }

package/dist/tools/experienceCompiler.js

@@ -29,7 +29,7 @@ export const compileExperienceTool = {
             },
             registryPath: {
                 type: "string",
-                description: "Optional local Block Factory registry path used to bridge selected objects to block install plans.",
+                description: "Optional local block registry path used to bridge selected objects to block install plans.",
             },
             write: {
                 type: "boolean",
@@ -65,7 +65,7 @@ export async function compileExperience(options) {
     ]);
     const surfaceHints = creativeSurfaceHints(selection);
     const surfacePlans = await Promise.all(surfaceHints.map((hint) => compileSurfacePlan(repoRoot, request, options.surfacePath, hint, uxHarness)));
-    const blockFactoryBridge = await blockFactoryBridgeFor({
+    const blocksBridge = await blocksBridgeFor({
         repoRoot,
         registryPath: options.registryPath,
         surfacePath: options.surfacePath,
@@ -77,7 +77,7 @@ export async function compileExperience(options) {
             : surface.intake.remainingBlocking > 0
                 ? [`Surface ${surface.surfaceId} has ${surface.intake.remainingBlocking} blocking intake question(s).`]
                 : []),
-        ...(blockFactoryBridge.status === "needs-review" ? ["One or more Block Factory bridge candidates need review before block installation."] : []),
+        ...(blocksBridge.status === "needs-review" ? ["One or more block bridge candidates need review before block installation."] : []),
     ];
     const compiler = {
         schema_version: EXPERIENCE_COMPILER_SCHEMA_VERSION,
@@ -109,7 +109,7 @@ export async function compileExperience(options) {
         surfaceSequence: surfaceSequenceFor(surfaceHints, selection, uxHarness),
         surfacePlans,
         designAdaptation: designAdaptationFor(designContract),
-        blockFactoryBridge,
+        blocksBridge,
         validationPlan: validationPlanFor(request, surfacePlans),
         sensors: sensors.map((sensor) => ({ name: sensor.name, command: sensor.command, source: sensor.source })),
         stopConditions,
@@ -204,10 +204,7 @@ function questionSummary(raw) {
         blocksImplementationWhenMissing: question.blocksImplementationWhenMissing === true,
     };
 }
-async function blockFactoryBridgeFor(args) {
-    // Gate on the creative feasibility's computed social.plus-vs-custom classification rather than
-    // guessing Block Factory's catalog: skip custom objects (no social.plus block exists), keep every
-    // social.plus-backed object as a candidate, and let the registry confirm or deny each one.
+async function blocksBridgeFor(args) {
     const customObjectIds = new Set(args.selection.selectedVariant.feasibility?.customObjectIds ?? []);
     const candidates = blockCandidatesFor(args.selection.selectedVariant.experienceObjectIds, customObjectIds);
     const customAppLayerObjects = customAppLayerObjectsFor(args.selection.selectedVariant.experienceObjectIds, customObjectIds);
@@ -219,7 +216,7 @@ async function blockFactoryBridgeFor(args) {
                 ...candidate,
                 status: "no-registry",
                 command: `vise blocks plan . --block ${candidate.blockId} --registry <path>${surfaceOption} --format json`,
-                reason: "No Block Factory registry was provided; this is a suggested bridge candidate only.",
+                reason: "No block registry was provided; this is a suggested bridge candidate only.",
             })),
             customAppLayerObjects,
         };
@@ -296,7 +293,7 @@ function installPlanFor(platforms, surfacePlans) {
         guidance: [
             `Use the ${platform} social.plus SDK installation path surfaced by focused plans.`,
             "Pin SDK package versions; do not rely on floating latest ranges.",
-            "If Block Factory registry candidates are available, review dry-run block plans before applying any package/source changes.",
+            "If block registry candidates are available, review dry-run block plans before applying any package/source changes.",
         ],
         sdkVersion,
         packageManagers: packageManagersFor(platforms),

package/dist/tools/experienceSensors.js

@@ -93,7 +93,7 @@ export async function buildExperienceSensors(options) {
         stopConditions,
         nextStep: stopConditions.length > 0
             ? "Resolve needs-setup and needs-review sensor items, then regenerate `vise experience sensors .` and `vise experience-report .`."
-            : "Use this sensor framework as the review checklist after implementation. Keep `score` null until benchmark and dogfood evidence calibrate scoring.",
+            : "Use this sensor framework as the review checklist after implementation. Keep `score` null until benchmark evidence calibrates scoring.",
     };
     if (options.write !== false) {
         framework.artifacts = await writeExperienceSensors(repoRoot, framework);

package/dist/tools/harness.js

@@ -164,19 +164,12 @@ export async function detectCommandSensors(repoPath, platforms) {
     }
     return sensors;
 }
-// xcodebuild failure text that means "this ENVIRONMENT cannot build", not "this project is
-// broken". Matched case-insensitively against combined stdout+stderr only when the exit code is
-// non-zero. Deliberately narrow: a real compile/link error ("error: cannot find 'X' in scope",
-// BUILD FAILED with diagnostics) matches none of these and stays a hard failure.
 const XCODEBUILD_ENVIRONMENT_SKIPS = [
     {
-        // xcode-select points at the Command Line Tools: xcodebuild exists on PATH but cannot build.
         pattern: "requires Xcode|command line tools instance",
         reason: "xcodebuild is present but the active developer directory is the Command Line Tools, not a full Xcode install. Run `sudo xcode-select -s /Applications/Xcode.app` (or install Xcode) and re-run; this is an environment precondition, not a project failure.",
     },
     {
-        // "Agreeing to the Xcode/iOS license requires admin privileges, please run 'sudo xcodebuild -license'…"
-        // Anchored to the agreement message — a compile error citing a LicenseView.swift must stay a failure.
         pattern: "agreeing to the .{0,20}license|xcodebuild -license|license agreement",
         reason: "The Xcode license has not been accepted in this environment. Run `sudo xcodebuild -license accept` and re-run; this is an environment precondition, not a project failure.",
     },
@@ -185,8 +178,6 @@ const XCODEBUILD_ENVIRONMENT_SKIPS = [
         reason: "The build stopped on code-signing requirements (certificates/provisioning not available in this environment). Vise already disables signing for this probe; a signing demand that persists is an environment/config precondition, not a project failure.",
     },
     {
-        // 'SDK "iphonesimulator" cannot be located' / destination errors. Anchored so a generic
-        // "cannot be located" inside project diagnostics does not classify as environmental.
         pattern: "unable to find a destination|no destinations|sdk \\S{0,30}\\s*cannot be located|unable to locate a simulator|CoreSimulator",
         reason: "No usable build destination (simulator runtime/SDK) is installed in this environment. Install an iOS Simulator runtime via Xcode and re-run; this is an environment precondition, not a project failure.",
     },
@@ -196,11 +187,8 @@ const XCODEBUILD_ENVIRONMENT_SKIPS = [
     },
 ];
 async function iosBuildSensors(root) {
-    // .xcodeproj/.xcworkspace are bundle DIRECTORIES at the app root.
     const entries = await readdir(root, { withFileTypes: true }).catch(() => []);
     const dirNames = entries.filter((entry) => entry.isDirectory()).map((entry) => entry.name);
-    // Prefer the workspace (CocoaPods projects must build via the workspace); ignore the
-    // .xcodeproj-embedded project.xcworkspace which readdir at root never returns anyway.
     const workspace = dirNames.find((name) => /\.xcworkspace$/i.test(name));
     const project = dirNames.find((name) => /\.xcodeproj$/i.test(name));
     if (!workspace && !project) {
@@ -208,9 +196,6 @@ async function iosBuildSensors(root) {
     }
     const container = workspace ?? project;
     const source = container;
-    // Visible precondition (pf-003): the Xcode project enables the sensor, but the sensor only
-    // RUNS when xcodebuild exists on PATH. Absent toolchain → skipped-with-reason, never a
-    // silent non-detection and never a spawn-ENOENT failure.
     if (!(await commandOnPath("xcodebuild"))) {
         return [
             {
@@ -225,10 +210,9 @@ async function iosBuildSensors(root) {
     }
     const containerFlag = workspace ? "-workspace" : "-project";
     const buildCommand = workspace
-        ? // A workspace build REQUIRES a scheme; the conventional shared scheme matches the
-            // workspace name. A wrong guess is classified as an environment/config skip below.
+        ?
             ["xcodebuild", containerFlag, container, "-scheme", container.replace(/\.xcworkspace$/i, ""), "build"]
-        : // A bare project build (no -scheme) builds the first target with the default config.
+        :
             ["xcodebuild", containerFlag, container, "build"];
     return [
         {
@@ -243,7 +227,6 @@ async function iosBuildSensors(root) {
             name: "iOS build (xcodebuild)",
             command: [
                 ...buildCommand,
-                // Guards: never demand signing assets for a compile probe, and keep the build local.
                 "CODE_SIGNING_ALLOWED=NO",
                 "CODE_SIGNING_REQUIRED=NO",
                 "CODE_SIGN_IDENTITY=",
@@ -256,8 +239,6 @@ async function iosBuildSensors(root) {
         },
     ];
 }
-// True when an executable with this name exists on the current PATH. Used for sensor
-// preconditions, so the answer must be computed WITHOUT spawning anything.
 async function commandOnPath(command) {
     const pathValue = process.env.PATH ?? "";
     for (const dir of pathValue.split(path.delimiter)) {
@@ -268,7 +249,6 @@ async function commandOnPath(command) {
             return true;
         }
         catch {
-            // keep scanning
         }
     }
     return false;
@@ -318,11 +298,6 @@ function hasPackageDependency(packageJson, dependencyName) {
         packageJson.peerDependencies?.[dependencyName] ??
         packageJson.optionalDependencies?.[dependencyName]);
 }
-// A package.json script that runs a watcher/dev server never exits; running it as a sensor just
-// burns the timeout and reports a misleading "timed-out". Detect the unambiguous cases and skip with
-// guidance. Deliberately conservative so a real one-shot sensor is never skipped: an explicitly
-// disabled flag (--watchAll=false) is NOT a watcher, and ambiguous short flags (e.g. `prettier -w`
-// = --write) are not treated as watch.
 function watchScriptSkipReason(scriptName, body) {
     const command = String(body);
     const watchFlag = command.match(/(?:^|\s)--watch(?:all)?(=\S+)?/i);

package/dist/tools/integration.js

@@ -2,7 +2,7 @@ import { access, readFile } from "node:fs/promises";
 import path from "node:path";
 import { BROAD_SOCIAL_REGEX, DESIGN_REGEX, getOutcomeDefinition, hasAnswer, planContextFor, resolveOutcome, } from "../outcomes.js";
 import { objectInput, optionalStringField, stringField, textResult } from "../types.js";
-import { availableOptionalCapabilityIds, capabilityChecklist, optionalCapabilityChecklist, platformCapabilityAvailability, selectedOptionalCapabilityIds, } from "../capabilities.js";
+import { availableOptionalCapabilityIds, capabilityChecklist, optionalCapabilityChecklist, platformCapabilityAvailability, selectedOptionalCapabilityIds, slimCapabilityAvailability, slimCapabilityItem, } from "../capabilities.js";
 import { applicableCompliancePlanRuleSummaries } from "./compliance.js";
 import { DESIGN_CONTRACT_CONFIRMATION_ANSWER_ID, buildDesignBrief, designContractConfirmationFromAnswers, designPreviewPath, readDesignContract, } from "./design.js";
 import { sdkVersionGuidance } from "./sdkVersion.js";
@@ -84,12 +84,6 @@ async function buildIntegrationPlan(repoPath, request, surfacePath, answers = {}
     });
     const definition = getOutcomeDefinition(outcome);
     const capabilityAvailability = await platformCapabilityAvailability(outcome, platform);
-    // The design brief is computed INTERNALLY only, to drive the missing-primary-action
-    // intake fallback. It is deliberately NOT emitted in plan output: a pre-registered
-    // ablation (benchmarks/brief-ablation/RESULTS.md, two runs, n=6/arm) showed that
-    // enumerative plan-time design guidance NARROWS what capable agents build and style —
-    // the contract + design-check loop alone scored higher on by-name token conformance.
-    // Retracted in 0.14.1; the generator stays in design.ts for future redesign.
     const designContract = await readDesignContract(repoRoot);
     const designReview = designReviewGuidance(repoRoot, designContract, answers);
     const acceptedDesignContract = designReview.status === "accepted" ? designContract : null;
@@ -111,18 +105,8 @@ async function buildIntegrationPlan(repoPath, request, surfacePath, answers = {}
         ? definition.intakeQuestions(ctx).filter((question) => question.id !== "feature_surface")
         : definition.intakeQuestions(ctx);
     const intake = intakeFor(ctx, outcomeQuestions, outcome, designBrief, capabilityAvailability, designReview);
-    // Advisory SDK-version currency guidance (npm registry for TS/RN; version-agnostic
-    // for native). Best-effort — degrades to greenfield "install latest + pin" if the
-    // registry is unreachable. Never gates.
     const packageJsonText = await readFile(path.join(root, "package.json"), "utf8").catch(() => undefined);
     const sdkVersion = await sdkVersionGuidance(platform, packageJsonText);
-    // Scope decisions, hoisted to the top of the output so they survive a `head`
-    // truncation. Blocking questions must be resolved before building; clarifying
-    // questions (e.g. which engagement surfaces are in scope) must be decided and
-    // the decision stated — so an omitted feature like comments is an explicit,
-    // reviewable choice rather than a silent drop. Mirrors the detailed `intake`
-    // block below; placed here because the intake block lands past common `head`
-    // cut points.
     const decisionsRequired = [
         ...(creativeContext
             ? [
@@ -138,11 +122,6 @@ async function buildIntegrationPlan(repoPath, request, surfacePath, answers = {}
             ? `[resolve before building] ${q.question}`
             : `[decide & state in your summary] ${q.question}`),
     ];
-    // Steps-first ordering: implementationSteps + validation lead the object so
-    // that an agent piping `vise plan` through `head` still captures the
-    // actionable guidance. The verbose scaffolding (intake, docs, surfaces, rule
-    // dumps) follows. Do not move implementationSteps back down — feature steps
-    // landing past a `head -120` cut is a documented real-world failure mode.
     return {
         outcome,
         platform,
@@ -151,7 +130,7 @@ async function buildIntegrationPlan(repoPath, request, surfacePath, answers = {}
         creativeContext,
         uxHarness: uxHarnessContext,
         socialWorkplan,
-        capabilityAvailability,
+        capabilityAvailability: slimCapabilityAvailability(capabilityAvailability),
         designReview,
         decisionsRequired,
         implementationSteps: [
@@ -182,10 +161,6 @@ async function buildIntegrationPlan(repoPath, request, surfacePath, answers = {}
         sdkVersion,
     };
 }
-// Vise-authored capability checklist for the outcome (feed-forward). The agent
-// builds each or opts out with a recorded reason (`// vise: scope-omit <id>
-// <reason>`), which `vise check` reads and reports. Advisory — Vise proposes the
-// set so completeness doesn't depend on the agent remembering it.
 function completenessChecklistFor(outcome) {
     const items = capabilityChecklist(outcome);
     if (items.length === 0) {
@@ -202,8 +177,8 @@ function optionalCapabilitiesFor(outcome, answers, request, availability) {
     if (choices.length === 0) {
         return undefined;
     }
-    const unavailable = availability?.unavailable.filter((item) => item.kind === "optional-scope") ?? [];
-    const unknown = availability?.unknown.filter((item) => item.kind === "optional-scope") ?? [];
+    const unavailable = (availability?.unavailable.filter((item) => item.kind === "optional-scope") ?? []).map(slimCapabilityItem);
+    const unknown = (availability?.unknown.filter((item) => item.kind === "optional-scope") ?? []).map(slimCapabilityItem);
     return {
         answerId: "feed_optional_capabilities",
         note: "Optional feed capabilities are feed-forward choices, not baseline compliance. Vise filters choices through the platform SDK surface before showing them. Before `vise init`, make an explicit answer: pass selected ids through `--answer feed_optional_capabilities=<id[,id]>` on `vise plan` and `vise init`, or pass `--answer feed_optional_capabilities=none`. `vise check` runs source sensors only for selected supported capabilities; explicitly unselected or unavailable capabilities remain non-mandatory.",
@@ -405,7 +380,7 @@ async function socialWorkplanFor(args) {
                 questions: intake.questions,
                 remainingBlocking: intake.remainingBlocking,
             },
-            capabilityAvailability,
+            capabilityAvailability: slimCapabilityAvailability(capabilityAvailability),
             validation: ["validate_setup", "run_sensors", ...definition.validation(args.platform)],
             docs: definition.docs(args.platform).filter((doc) => doc.path !== "unknown"),
             optionalCapabilities: optionalCapabilitiesFor(surface.outcome, surfaceAnswers, args.request, capabilityAvailability),
@@ -450,6 +425,7 @@ function designReviewGuidance(repoRoot, contract, answers) {
         digest: contract.digest,
         previewPath,
         previewCommand: "vise design preview .",
+        referenceCommand: "vise design reference .",
         requiredForDesignConformance: true,
     };
     if (confirmation === "accepted") {
@@ -472,11 +448,6 @@ function designReviewGuidance(repoRoot, contract, answers) {
         nextStep: `Open ${previewPath} and show it to the user. Re-run plan/init with --answer ${DESIGN_CONTRACT_CONFIRMATION_ANSWER_ID}=yes only if the user confirms it matches the desired design; use =no to reject and request a new design source.`,
     };
 }
-// Build advisory UI-generation guidance from an extracted design contract.
-// Declared tokens are surfaced with their custom-property name (so the agent
-// references `var(--x)` / maps it per platform); inferred tokens carry their
-// raw value plus a usage count and an explicit "inferred" marker so they are
-// never mistaken for authoritative brand values.
 function designContractGuidance(contract) {
     const byCategory = (category) => contract.tokens
         .filter((token) => token.category === category)
@@ -562,9 +533,6 @@ function intakeFor(ctx, outcomeQuestions, outcome, brief, availability, designRe
             options: ["yes", "use another source"],
         });
     }
-    // Graceful-degradation fallback: when a design contract exists for a feed or chat
-    // outcome but no primary-action token was confidently inferred, ask the developer
-    // to name the correct token. Non-blocking so it doesn't stall implementation.
     if (brief &&
         designReview?.status === "accepted" &&
         (outcome === "add-feed" || outcome === "add-chat") &&

package/dist/tools/learning.js

@@ -414,7 +414,7 @@ function summarizeLearning(events) {
         recommendationOptimization: {
             status: "not-active",
             reason: "Local learning events are evidence inputs only. Vise does not alter variant ranking from them yet.",
-            nextStep: "Use repeated dogfood/benchmark review to define a calibrated ranking policy before enabling recommendation optimization.",
+            nextStep: "Use repeated benchmark review to define a calibrated ranking policy before enabling recommendation optimization.",
         },
         privacy: privacyBlock(),
         nextStep: "Review these local events and recurring sensor gaps with Product/SE. Aggregate learning or automatic ranking changes require explicit opt-in and privacy review.",