@whisperr/wizard 0.1.10 → 0.1.12

package/dist/index.js

@@ -6,8 +6,8 @@ import * as p from "@clack/prompts";
 
 // src/core/config.ts
 var DEFAULT_API_BASE = "https://api.whisperr.net";
-var DEFAULT_MODEL = "claude-opus-4-8";
-var DEFAULT_EFFORT = "low";
+var DEFAULT_MODEL = "claude-sonnet-4-6";
+var DEFAULT_EFFORT = "high";
 var EFFORT_LEVELS = ["low", "medium", "high", "xhigh", "max"];
 function resolveEffort() {
   const raw = (process.env.WHISPERR_WIZARD_EFFORT ?? "").toLowerCase();
@@ -25,7 +25,13 @@ function resolveConfig(flags = {}) {
     llmBaseUrl,
     model: flags.model ?? process.env.WHISPERR_WIZARD_MODEL ?? DEFAULT_MODEL,
     effort: resolveEffort(),
-    maxTurns: Number(process.env.WHISPERR_WIZARD_MAX_TURNS ?? 55),
+    // Cost is the real limiter (budgetUsd below). maxTurns is only a high safety
+    // backstop against a stuck loop — it should NOT bind on a normal run. (It
+    // used to be 55, which guillotined high-effort multi-phase runs mid-work.)
+    maxTurns: Number(process.env.WHISPERR_WIZARD_MAX_TURNS) || 200,
+    // Hard ceiling on TOTAL spend across all phases. If hit, the run stops
+    // cleanly and keeps whatever already landed. Real runs finish well under it.
+    budgetUsd: Number(process.env.WHISPERR_WIZARD_BUDGET_USD) || 10,
     directAnthropicKey,
     offline
   };
@@ -1022,6 +1028,31 @@ describe the SAME person, so they belong on the SAME surface. If you wired
 account_created into one controller and are about to put identify() in a
 different one, stop \u2014 that's the tell you've split the subject. Co-locate them.
 
+WHAT THE EVENT MEANS \u2014 place by SEMANTICS, not by keyword. This is where most
+mistakes happen. An event names a SPECIFIC moment, not "something happened
+nearby." For each event, before you place it:
+- Model the exact transition. \`subscription_started\` is the FIRST subscription,
+  NOT every payment \u2014 renewals, retries, and reactivations are DIFFERENT events.
+  Firing one event on a path that also handles the other cases is a real bug.
+- Read the control flow and find the branch that separates look-alike cases.
+  Payment/callback handlers routinely route initial vs recurring/renewal, success
+  vs retry, trial vs paid through ONE function. Key the event to the right branch
+  (e.g. a \`RENEW_CARD\` / \`is_recurring\` / \`type\` check), or branch yourself and
+  emit the correct event per case. Do not drop one event on the shared success
+  line and move on \u2014 open the function and see which cases it actually covers.
+- Read the codebase's own flags literally. A field like \`type: promo|standard\`
+  is pricing tier, not lifecycle \u2014 confirm what a flag means before keying on it.
+- Capture the discriminators as properties. The fields that tell cases apart \u2014
+  charge_type, is_recurring, payment_source/gateway, amount, plan, currency \u2014 are
+  exactly what makes the event usable for churn (a failed RENEWAL is a churn
+  signal; a failed first charge is not). If the code sets such a field right
+  there (e.g. \`type => 'initial'\`), pass it. A billing event without them is
+  half-blind.
+- Cover the whole lifecycle. Two gateways each with an initial and a recurring
+  path is FOUR call sites \u2014 instrument all of them. A recurring/secondary
+  callback with no events is a churn blind spot. If you deliberately skip a path,
+  say so in the summary.
+
 Every step costs time and the user is watching. Work FAST and DECISIVELY.
 
 EFFICIENCY \u2014 non-negotiable:
@@ -1183,14 +1214,21 @@ async function runIntegrationAgent(opts) {
     repoPath,
     model: config.model,
     effort: config.effort,
-    maxTurns: 35,
+    maxTurns: 50,
+    budgetUsd: config.budgetUsd - costUsd,
     progress
   });
   costUsd += core.costUsd;
   if (core.summary) summaries.push(`Core setup:
 ${core.summary}`);
+  const BUDGET_FLOOR = 0.25;
   let eventsComplete = true;
-  if (manifest.events.length > 0) {
+  if (manifest.events.length > 0 && config.budgetUsd - costUsd <= BUDGET_FLOOR) {
+    eventsComplete = false;
+    summaries.push(
+      "Events: skipped \u2014 the spend limit was reached during core setup. Re-run with a higher WHISPERR_WIZARD_BUDGET_USD to instrument events."
+    );
+  } else if (manifest.events.length > 0) {
     progress?.onPhase?.("Instrumenting your events");
     const eventsPrompt = [
       `The Whisperr ${playbook.target.displayName} SDK is already installed and`,
@@ -1216,12 +1254,52 @@ ${core.summary}`);
       model: config.model,
       effort: config.effort,
       maxTurns: config.maxTurns,
+      budgetUsd: config.budgetUsd - costUsd,
       progress
     });
     costUsd += events.costUsd;
     eventsComplete = !events.maxedOut;
     if (events.summary) summaries.push(`Events:
 ${events.summary}`);
+  }
+  if (core.ok && config.budgetUsd - costUsd > BUDGET_FLOOR) {
+    progress?.onPhase?.("Reviewing & correcting placements");
+    const reviewPrompt = [
+      `You wired the Whisperr ${playbook.target.displayName} SDK into this repo.`,
+      "Now AUDIT YOUR OWN WORK and fix mistakes before finishing.",
+      `Project root: ${repoPath}`,
+      "",
+      "Run `git diff` and read the surrounding code to see exactly what you added.",
+      "For every identify() and track() call, verify \u2014 and FIX in place:",
+      "1. Subject \u2014 keys to the END USER, never an admin/staff/operator/seller.",
+      "   identify() sits on the SAME surface as account_created (same person).",
+      "2. Lifecycle \u2014 the event fires ONLY on its real moment. A 'first time'",
+      "   event (e.g. subscription_started) must NOT fire on a renewal/recurring",
+      "   path; look for RENEW_CARD / is_recurring / type branches in the SAME",
+      "   handler. If one path handles both cases, branch and emit the right",
+      "   event per case.",
+      "3. Discriminators \u2014 billing/subscription events carry the properties that",
+      "   tell cases apart when the code exposes them (charge_type / is_recurring",
+      "   / payment_source / amount / plan). Add the ones you missed.",
+      "4. Coverage \u2014 every gateway/callback path that should emit an event does;",
+      "   no recurring or secondary path left as a dead zone.",
+      "Change ONLY what is genuinely wrong or incomplete \u2014 do not churn correct",
+      "calls or re-explore the whole repo. Be surgical. End with a one-line,",
+      "plain-text note of what you corrected, or 'No corrections needed.'."
+    ].join("\n");
+    const review = await runPass({
+      prompt: reviewPrompt,
+      systemPrompt: systemPrompt9,
+      repoPath,
+      model: config.model,
+      effort: config.effort,
+      maxTurns: 40,
+      budgetUsd: config.budgetUsd - costUsd,
+      progress
+    });
+    costUsd += review.costUsd;
+    if (review.summary) summaries.push(`Review:
+${review.summary}`);
   }
   return {
     summary: summaries.join("\n\n"),
@@ -1231,8 +1309,12 @@ ${events.summary}`);
     eventsComplete
   };
 }
+function isLimitStop(err) {
+  const msg = err?.message ?? "";
+  return /max(imum)?[\s_-]*(turn|budget)|budget.*exceeded|number of turns/i.test(msg);
+}
 async function runPass(opts) {
-  const { prompt, systemPrompt: systemPrompt9, repoPath, model, effort, maxTurns, progress } = opts;
+  const { prompt, systemPrompt: systemPrompt9, repoPath, model, effort, maxTurns, budgetUsd, progress } = opts;
   let summary = "";
   let costUsd = 0;
   let ok = false;
@@ -1251,23 +1333,36 @@ async function runPass(opts) {
       allowedTools: ["Read", "Edit", "Write", "Bash", "Glob", "Grep"],
       permissionMode: "bypassPermissions",
       maxTurns,
+      // Native SDK hard cost cap. Stops this pass with an `error_max_budget_usd`
+      // result once spend exceeds the remaining budget — this, not maxTurns, is
+      // the real limiter. Omitted when there's no positive budget left.
+      ...budgetUsd && budgetUsd > 0 ? { maxBudgetUsd: budgetUsd } : {},
       settingSources: []
     }
   });
-  for await (const message of response) {
-    if (message.type === "assistant") {
-      for (const block of message.message?.content ?? []) {
-        if (isTextBlock(block) && block.text?.trim()) {
-          progress?.onActivity?.(firstLine(block.text));
-        } else if (isToolUseBlock(block)) {
-          progress?.onActivity?.(describeToolUse(block));
+  try {
+    for await (const message of response) {
+      if (message.type === "assistant") {
+        for (const block of message.message?.content ?? []) {
+          if (isTextBlock(block) && block.text?.trim()) {
+            progress?.onActivity?.(firstLine(block.text));
+          } else if (isToolUseBlock(block)) {
+            progress?.onActivity?.(describeToolUse(block));
+          }
         }
+      } else if (message.type === "result") {
+        ok = message.subtype === "success";
+        const sub = message.subtype ?? "";
+        maxedOut = sub.includes("max_turns") || sub.includes("max_budget");
+        costUsd = message.total_cost_usd ?? 0;
+        summary = message.result ?? extractLastText(message) ?? summary;
       }
-    } else if (message.type === "result") {
-      ok = message.subtype === "success";
-      maxedOut = (message.subtype ?? "").includes("max_turns");
-      costUsd = message.total_cost_usd ?? 0;
-      summary = message.result ?? extractLastText(message) ?? summary;
+    }
+  } catch (err) {
+    if (isLimitStop(err)) {
+      maxedOut = true;
+    } else {
+      throw err;
     }
   }
   return { summary, costUsd, ok, maxedOut };
@@ -1827,7 +1922,7 @@ async function main() {
     return;
   }
   if ("version" in parsed) {
-    console.log("0.1.10");
+    console.log("0.1.12");
     return;
   }
   try {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@whisperr/wizard",
-  "version": "0.1.10",
+  "version": "0.1.12",
   "description": "Whisperr Wizard — one command to integrate the Whisperr SDK into your app. Authenticates with your onboarded account and uses an AI coding agent to wire up identify() and your business events automatically.",
   "license": "UNLICENSED",
   "private": false,