weftens 0.1.3 → 0.1.5

package/agents/writer.md

@@ -9,10 +9,10 @@ words built from found language, aimed at a reader who owes you nothing.
 ## The method spine (both output shapes)
 The writing is not the job — research is. Before drafting, mine the real language:
 - For content pieces: the research packet (Researcher's facts, sources, claims-to-verify) and the
-  format brief; the audience's own words from comments/UGC where available.
+ format brief; the audience's own words from comments/UGC where available.
 - For conversion copy: the corpus (reviews of the product AND competitors, threads, transcripts,
-  support logs) → a voice-of-customer map: pains, desired outcomes, objections, the switch moment,
-  competitor complaints — verbatim, with frequency and emotional weight per phrase.
+ support logs) → a voice-of-customer map: pains, desired outcomes, objections, the switch moment,
+ competitor complaints — verbatim, with frequency and emotional weight per phrase.
 Copy assembled from found language beats invented copy; the customer's words are more specific,
 more emotional, and more believable than anything written at a desk.
 
@@ -34,11 +34,11 @@ the highest frequency × weight pain, sequence proof against objections in desce
 
 ## Hard rules (both shapes)
 - Banned: marketer vocabulary ("streamline," "unlock," "empower," "seamless," "elevate"), wordplay
-  that costs clarity, invented benefits, superlatives without a source.
+ that costs clarity, invented benefits, superlatives without a source.
 - Every claim traces to the corpus, the research packet, or the offer — a caught overclaim kills
-  the asset.
+ the asset.
 - Voice is locked per surface (Showrunner's guardrails for own channels; the client voice doc for
-  client work). Drift is a defect even when the copy is good.
+ client work). Drift is a defect even when the copy is good.
 - Nothing publishes from this seat — Script Editor / Disclosure-QC / human approval sit downstream.
 
 Good output: a line the audience could have said themselves, in the locked voice. Bad output:

package/bin/weftens.js

@@ -15,7 +15,7 @@
 import { readFileSync } from "node:fs";
 import readline from "node:readline";
 import { orchestrate } from "../src/weftens.js";
-import { isChangeSheet, formatChangeSheet } from "../src/format.js";
+import { isChangeSheet, formatChangeSheet, looksUnparsed, unparsedNotice } from "../src/format.js";
 import { askWeftens } from "../src/agent.js";
 import { buildSpecialistClients } from "../src/specialists.js";
 import { route } from "../src/agent-router.js";
@@ -153,6 +153,13 @@ async function cmdSend(args, { routeOnly } = {}) {
   if (outcome.result !== undefined) {
     // A change sheet renders for a human by default — the whole point of one is that a
     // person can read a line and decide. `--json` returns the raw object for piping.
+    if (!asJson && looksUnparsed(outcome.result)) {
+      process.stdout.write(`
+${unparsedNotice(outcome.result)}
+`);
+      process.exitCode = 1;
+      return;
+    }
     if (!asJson && isChangeSheet(outcome.result)) {
       process.stdout.write(`\n${formatChangeSheet(outcome.result)}\n`);
     } else {

package/package.json

@@ -1,8 +1,8 @@
 {
   "name": "weftens",
-  "version": "0.1.3",
+  "version": "0.1.5",
   "type": "module",
-  "description": "Weftens \u2014 one front door that routes a marketing/representation request to the right agent and runs it on your own AI subscription (bring-your-own-model). Usable as a CLI or an SDK.",
+  "description": "Weftens — one front door that routes a marketing/representation request to the right agent and runs it on your own AI subscription (bring-your-own-model). Usable as a CLI or an SDK.",
   "main": "index.js",
   "exports": {
     ".": "./index.js",
@@ -23,6 +23,8 @@
     "providers.json.example",
     "README.md",
     "LICENSE",
+    "CHANGELOG.md",
+    "ARCHITECTURE.md",
     "!**/.preflight",
     "!**/.preflight/**"
   ],
@@ -39,7 +41,8 @@
     "vitest": "^2.0.0"
   },
   "dependencies": {
-    "@anthropic-ai/sdk": "^0.110.0"
+    "@anthropic-ai/sdk": "^0.110.0",
+    "weftens": "^0.1.4"
   },
   "repository": {
     "type": "git",

package/src/format.js

@@ -35,6 +35,80 @@ export function isChangeSheet(result) {
   return Boolean(result && typeof result === "object" && Array.isArray(result.changes));
 }
 
+
+// A core that was handed the WRONG report does not usually error — it parses the rows, finds none of
+// the columns it wants, and returns a well-formed sheet with zero changes. Rendered normally that
+// reads as "your account is clean", which is the most expensive lie this tool could tell: a user
+// exports the wrong file and is told there is nothing to fix. Five of seven cores did exactly this.
+//
+// The signal is that rows were read but every money/volume figure came back zero or null. A real
+// account under audit always has spend. An account with genuinely zero activity is indistinguishable
+// — and for that one, "this doesn't look like the right export, or there was no activity" is also
+// the truthful answer.
+// Two different kinds of number, and conflating them is what let four cores slip past the first two
+// versions of this check. ROW fields count what was READ — a non-zero row count proves the file was
+// opened, not that it was understood. VALUE fields are what the report is FOR. The signal for "wrong
+// file" is: rows were read, and every value came back zero.
+const ROW_FIELDS = ["rows", "terms", "placements", "urls", "pages", "items", "searchTermRows", "campaignRows"];
+const VALUE_FIELDS = [
+  "totalCost", "totalSpend", "totalClicks", "totalImpressions", "totalConversions",
+  "totalResults", "spend", "cost", "clicks", "impressions", "conversions", "sales", "orders",
+  "flaggedSpend", "reclaimableSpend", "totalViews", "views",
+];
+
+export function looksUnparsed(result) {
+  if (!result || typeof result !== "object") return false;
+  if (Array.isArray(result.changes) && result.changes.length > 0) return false;
+  // Cores don't share one result shape: some report totals under `account`, others under `summary`,
+  // others at the top level. Check wherever the volume figures actually live rather than assuming a
+  // shape — assuming `account` is why four of the five cores slipped past the first version of this.
+  // Every core names its totals block differently — `account`, `summary`, `totals` — and the first
+  // two versions of this guard each missed the cores they didn't happen to name. Scan any one-level
+  // block that carries volume fields instead of guessing at keys; a new core is covered for free.
+  const blocks = [result, ...Object.values(result).filter((v) => v && typeof v === "object" && !Array.isArray(v))];
+  const zero = (v) => v === 0 || v === null || v === undefined;
+
+  // Strongest signal first: a core that reports it read ZERO rows read nothing at all. Whatever the
+  // file was, it was not this report.
+  for (const b of blocks) {
+    for (const f of ROW_FIELDS) if (b[f] === 0) return true;
+  }
+
+  for (const b of blocks) {
+    const values = VALUE_FIELDS.filter((f) => f in b && typeof b[f] !== "object");
+    if (values.length === 0) continue;
+    if (!values.every((f) => zero(b[f]))) continue;
+    // Every value is zero. If rows were read, the file parsed as text but not as this report.
+    // If no row count is reported either, an all-zero result is still not a clean bill of health.
+    return true;
+  }
+  return false;
+}
+
+/** Row count from wherever a core reports it — used only to make the refusal message concrete. */
+function rowsRead(result) {
+  if (!result || typeof result !== "object") return null;
+  const blocks = [result, ...Object.values(result).filter((v) => v && typeof v === "object" && !Array.isArray(v))];
+  for (const b of blocks) {
+    for (const f of ROW_FIELDS) if (typeof b[f] === "number" && b[f] > 0) return b[f];
+  }
+  return null;
+}
+
+
+/** The one message for "this file did not parse", shared by every core shape. */
+export function unparsedNotice(result) {
+  const rows = rowsRead(result);
+  return [
+    "Could not read this as the report this agent expects.",
+    typeof rows === "number" && rows > 0
+      ? `  ${rows} row(s) were read, but none of the expected columns (spend, clicks, results) had values.`
+      : "  No rows with the expected columns were found.",
+    "  This is NOT a finding that your account is clean — it means the file did not parse.",
+    "  Check you exported the right report for this agent, then run it again.",
+  ].join("\n");
+}
+
 /**
  * Render a change sheet as plain text. Never invents a field: anything missing is simply
  * not printed, so a core that doesn't produce risk/evidence still renders cleanly.
@@ -50,6 +124,9 @@ export function formatChangeSheet(result) {
   const changes = result.changes.filter((c) => c && typeof c === "object");
   const dropped = result.changes.length - changes.length;
 
+  // Refuse to present an unreadable file as a clean bill of health.
+  if (looksUnparsed(result)) return unparsedNotice(result);
+
   const flagged = money(result.wastedSpendRecoverable, currency);
   const header = [`${changes.length} proposed change${changes.length === 1 ? "" : "s"}`];
   // "flagged", not "recoverable": the tool measured spend against a threshold, it did not