@skillrecordings/cli 0.20.0 → 0.21.0

package/README.md

@@ -7,8 +7,10 @@ Agent-friendly CLI for the Skill Recordings support platform.
 You need: a GitHub account in the `skillrecordings` org.
 
 ```bash
-# 1. Install
-curl -fsSL https://raw.githubusercontent.com/skillrecordings/support/main/packages/cli/install.sh | bash
+# 1. Install (private repo, so use gh)
+gh auth login
+gh api -H "Accept: application/vnd.github.raw" \
+  repos/skillrecordings/support/contents/packages/cli/install.sh?ref=main | bash
 
 # 2. Add to PATH (if not already)
 echo 'export PATH="$HOME/.local/bin:$PATH"' >> ~/.zshrc && source ~/.zshrc
@@ -25,6 +27,23 @@ That's it. No 1Password CLI, no age keys, no manual secret management.
 The CLI authenticates via GitHub device flow → the broker verifies your org
 membership → secrets are delivered encrypted and held in memory only.
 
+## Install troubleshooting
+
+| Symptom | Fix |
+|---|---|
+| `gh: command not found` | Install GitHub CLI: https://cli.github.com/ |
+| `gh` asks you to log in | Run `gh auth login`, then `gh auth status` |
+| `HTTP 404` / repo access failure on bootstrap | Confirm you can access the private repo: `gh release list --repo skillrecordings/support --limit 5` |
+| `skill: command not found` after install | Add `~/.local/bin` to PATH |
+
+Quick verification:
+
+```bash
+gh auth status
+which skill
+skill --version
+```
+
 ## Commands
 
 ```bash

package/dist/{chunk-F3WI3BN5.js → chunk-4VWYLYQ6.js}

@@ -14,7 +14,7 @@ import {
 } from "./chunk-HKRLO2GE.js";
 import {
   queryVectors
-} from "./chunk-H3D6VCME.js";
+} from "./chunk-ZZGGSV6N.js";
 import {
   Index2
 } from "./chunk-MG37YDAK.js";
@@ -81,7 +81,9 @@ function computeMessageSignals(body, subject = "") {
     isPersonalToInstructor: PERSONAL_MESSAGE_PATTERN.test(text) || INSTRUCTOR_MENTION_PATTERN.test(text) && /(?:thank|love|amazing|big fan)/i.test(text),
     // Presales signals (set by LLM classifier, defaulted here)
     isPresalesFaq: false,
-    isPresalesTeam: false
+    isPresalesTeam: false,
+    isThirdPartyReceipt: false,
+    isKnownVendorDomain: false
   };
 }
 function computeThreadSignals(input) {
@@ -165,6 +167,8 @@ function computeThreadSignals(input) {
     // Presales signals (merged from base)
     isPresalesFaq: baseSignals.isPresalesFaq || threadWideSignals.isPresalesFaq,
     isPresalesTeam: baseSignals.isPresalesTeam || threadWideSignals.isPresalesTeam,
+    isThirdPartyReceipt: false,
+    isKnownVendorDomain: false,
     // Thread structure
     threadLength: sorted.length,
     threadDurationHours,
@@ -283,6 +287,24 @@ var VENDOR_PATTERNS = [
   /\bproduction[- ]ready\s+(?:saas|app|template|boilerplate|platform|starter)/i
   // cold SaaS pitch
 ];
+var THIRD_PARTY_RECEIPT_PATTERNS = [
+  /\b(?:invoice|receipt|payment)\b.*\b(?:fly\.io|upstash|dmarc|mux|inngest|vercel|github|heroku|aws|digitalocean|cloudflare|netlify|render)\b/i,
+  /\b(?:fly\.io|upstash|dmarc|mux|inngest)\b.*\b(?:invoice|receipt|payment|billing|subscription|charge)\b/i,
+  /your\s+(?:monthly|annual|weekly)\s+(?:invoice|receipt|statement)\s+(?:from|for)\b/i
+];
+var KNOWN_VENDOR_DOMAINS = [
+  "fly.io",
+  "upstash.com",
+  "dmarcdigests.com",
+  "dmarc.postmarkapp.com",
+  "mux.com",
+  "inngest.com",
+  "gaming1.com",
+  "guestposting",
+  "meecocreative.com",
+  "getrecall.ai",
+  "seaverse.ai"
+];
 var LEGAL_THREAT_PATTERNS = [
   /\b(?:my\s+)?lawyer/i,
   /\blegal\s+action/i,
@@ -404,6 +426,9 @@ function extractSignals(input) {
     hasAngrySentiment: ANGRY_PATTERNS.some((p) => p.test(text)),
     isAutomated: AUTOMATED_PATTERNS.some((p) => p.test(text)) || (input.from ? /noreply|no-reply|mailer-daemon/i.test(input.from) : false),
     isVendorOutreach: VENDOR_PATTERNS.some((p) => p.test(text)),
+    // ADR-0003 D2: Third-party receipts and known vendor domains
+    isThirdPartyReceipt: THIRD_PARTY_RECEIPT_PATTERNS.some((p) => p.test(text)),
+    isKnownVendorDomain: input.from ? KNOWN_VENDOR_DOMAINS.some((d) => input.from.toLowerCase().includes(d)) : false,
     // Escalation signals
     hasLegalThreat: LEGAL_THREAT_PATTERNS.some((p) => p.test(fullText)),
     hasOutsidePolicyTimeframe: OUTSIDE_POLICY_PATTERNS.some(
@@ -429,6 +454,15 @@ function fastClassify(input, signals) {
       _fastPathRule: "automated_system"
     };
   }
+  if (signals.isThirdPartyReceipt || signals.isKnownVendorDomain) {
+    return {
+      category: "system",
+      confidence: 0.95,
+      signals,
+      reasoning: signals.isKnownVendorDomain ? `Sender domain matches known vendor/service (${input.from})` : "Third-party receipt/notification \u2014 not a customer support request",
+      _fastPathRule: signals.isKnownVendorDomain ? "known_vendor_domain" : "third_party_receipt"
+    };
+  }
   if (signals.isVendorOutreach && !signals.hasEmailInBody) {
     return {
       category: "spam",
@@ -1467,8 +1501,10 @@ var EditDraftSchema = z2.object({
   cc: z2.array(z2.string()).optional(),
   bcc: z2.array(z2.string()).optional(),
   subject: z2.string().optional(),
-  version: z2.string()
+  version: z2.string(),
   // Required for optimistic locking
+  channel_id: z2.string().optional()
+  // Required by Front API for PATCH
 });
 var MessageTemplateFolderSchema = z2.object({
   _links: z2.object({
@@ -2318,6 +2354,12 @@ function createInstrumentedFrontClient(config) {
 }
 
 // ../core/src/pipeline/steps/comment.ts
+var REASONING_MAX_LENGTH = 150;
+function truncateReasoning(text, maxLength = REASONING_MAX_LENGTH) {
+  if (!text) return text;
+  if (text.length <= maxLength) return text;
+  return text.slice(0, maxLength).trimEnd() + "\u2026";
+}
 function formatSupportComment(context) {
   const parts = ["\u{1F916} **Agent Research Context**\n"];
   if (context.user) {
@@ -2406,121 +2448,47 @@ function formatEscalationType(type) {
       return "Escalated";
   }
 }
-function formatPurchase(purchase, adminBaseUrl) {
-  const status = purchase.status !== "active" ? ` (${purchase.status})` : "";
-  const amount = purchase.amount ? ` - $${(purchase.amount / 100).toFixed(2)}` : "";
-  if (adminBaseUrl && purchase.productId) {
-    const adminLink = `${adminBaseUrl}/purchases/${purchase.productId}`;
-    return `- [${purchase.productName}](${adminLink})${amount}${status}`;
-  }
-  return `- ${purchase.productName}${amount}${status}`;
-}
 function formatEscalationComment(context) {
-  const parts = [];
   const emoji = getEscalationEmoji(context.type);
   const typeLabel = formatEscalationType(context.type);
-  parts.push(`${emoji} **Agent Escalation (${typeLabel})**
-`);
-  parts.push(`**Reason:** ${context.reason}`);
-  if (context.classification) {
-    parts.push(`**Category:** ${context.classification.category}`);
-    parts.push(
-      `**Confidence:** ${Math.round(context.classification.confidence * 100)}%`
-    );
-    if (context.classification.reasoning) {
-      parts.push(`**Agent reasoning:** ${context.classification.reasoning}`);
-    }
-  }
-  parts.push("\n---");
-  parts.push("**Customer Info**");
-  parts.push(`- **Email:** ${context.customer.email}`);
-  if (context.customer.name) {
-    parts.push(`- **Name:** ${context.customer.name}`);
-  }
-  if (context.customer.id) {
-    parts.push(`- **ID:** ${context.customer.id}`);
-  }
-  parts.push("\n**Purchases:**");
-  if (context.purchases && context.purchases.length > 0) {
-    const adminBaseUrl = context.links?.admin?.replace(/\/users\/.*$/, "");
-    for (const p of context.purchases) {
-      parts.push(formatPurchase(p, adminBaseUrl));
-    }
-  } else {
-    parts.push("_No purchases found for this email_");
-  }
-  if (context.agentFindings && context.agentFindings.length > 0) {
-    parts.push("\n**What Agent Found/Tried:**");
-    for (const finding of context.agentFindings) {
-      parts.push(`- ${finding}`);
-    }
-  }
-  if (context.links) {
-    const hasLinks = context.links.admin || context.links.magicLogin || context.links.frontConversation;
-    if (hasLinks) {
-      parts.push("\n---");
-      parts.push("**Quick Links**");
-      if (context.links.admin) {
-        parts.push(`- [Admin Profile](${context.links.admin})`);
-      }
-      if (context.links.magicLogin) {
-        parts.push(`- [Magic Login](${context.links.magicLogin})`);
-      }
-      if (context.links.frontConversation) {
-        parts.push(`- [Front Conversation](${context.links.frontConversation})`);
-      }
-    }
+  const parts = [];
+  parts.push(`${emoji} **${typeLabel}:** ${context.reason}`);
+  const purchaseCount = context.purchases?.length ?? 0;
+  const purchaseSummary = purchaseCount > 0 ? `${purchaseCount} purchase${purchaseCount > 1 ? "s" : ""}` : "no purchases";
+  parts.push(`${context.customer.email} (${purchaseSummary})`);
+  if (context.links?.magicLogin) {
+    parts.push(`[Magic Login](${context.links.magicLogin})`);
   }
   return parts.join("\n");
 }
 function formatApprovalComment(context) {
-  const parts = [];
-  parts.push("\u{1F50D} **Draft Pending Review**\n");
-  parts.push(`**Review Reason:** ${context.reviewReason}`);
   const confidencePercent = Math.round(context.confidence * 100);
-  const confidenceEmoji = confidencePercent >= 80 ? "\u{1F7E2}" : confidencePercent >= 60 ? "\u{1F7E1}" : "\u{1F534}";
-  parts.push(`**Confidence:** ${confidenceEmoji} ${confidencePercent}%`);
-  if (context.category) {
-    parts.push(`**Category:** ${context.category}`);
-  }
+  const parts = [];
+  const cat = context.category ? `${context.category} ` : "";
+  parts.push(
+    `\u{1F50D} **Review:** ${cat}(${confidencePercent}%) \u2014 ${context.reviewReason}`
+  );
   if (context.customerEmail) {
-    parts.push(`**Customer:** ${context.customerEmail}`);
-  }
-  parts.push("\n---");
-  parts.push("**Draft Preview:**");
-  parts.push("");
-  const draftLines = context.draft.split("\n");
-  for (const line of draftLines) {
-    parts.push(`> ${line}`);
+    parts.push(context.customerEmail);
   }
   if (context.actionLinks) {
-    parts.push("\n---");
-    parts.push("**Actions:**");
+    const links = [];
     if (context.actionLinks.approve) {
-      parts.push(`- [\u2705 Approve & Send](${context.actionLinks.approve})`);
+      links.push(`[Approve](${context.actionLinks.approve})`);
     }
     if (context.actionLinks.edit) {
-      parts.push(`- [\u270F\uFE0F Edit Draft](${context.actionLinks.edit})`);
+      links.push(`[Edit](${context.actionLinks.edit})`);
+    }
+    if (links.length > 0) {
+      parts.push(links.join(" \xB7 "));
     }
   }
   return parts.join("\n");
 }
 function formatAuditComment(context) {
-  const parts = [];
   const actionEmoji = context.action === "auto_sent" ? "\u2705" : context.action === "draft_created" ? "\u{1F4DD}" : context.action === "silenced" ? "\u{1F507}" : context.action === "escalated" ? "\u26A0\uFE0F" : "\u{1F916}";
-  parts.push(
-    `${actionEmoji} **Agent Action: ${formatActionLabel(context.action)}**`
-  );
   const confidencePercent = Math.round(context.confidence * 100);
-  parts.push(
-    `Category: ${context.category} | Confidence: ${confidencePercent}%`
-  );
-  const timestamp = context.timestamp ?? /* @__PURE__ */ new Date();
-  parts.push(`Timestamp: ${timestamp.toISOString()}`);
-  if (context.messageId) {
-    parts.push(`Message ID: ${context.messageId}`);
-  }
-  return parts.join("\n");
+  return `${actionEmoji} ${formatActionLabel(context.action)} \u2014 ${context.category} (${confidencePercent}%)`;
 }
 function formatActionLabel(action) {
   switch (action) {
@@ -2573,7 +2541,7 @@ function formatDecisionComment(context) {
   const confidencePercent = Math.round(context.confidence * 100);
   parts.push(`**Category:** ${context.category} (${confidencePercent}%)`);
   if (context.reasoning) {
-    parts.push(`**Reasoning:** ${context.reasoning}`);
+    parts.push(`**Reasoning:** ${truncateReasoning(context.reasoning)}`);
   }
   if (context.customerEmail) {
     const customerLine = context.customerName ? `${context.customerName} (${context.customerEmail})` : context.customerEmail;
@@ -2760,12 +2728,13 @@ init_esm_shims();
 var BASE_DRAFT_PROMPT = `You are a support agent. Write a helpful response to the customer.
 
 ## Style Guide
-- Be direct and concise
-- No corporate speak
-- No enthusiasm performance ("Great!", "Happy to help!")
+- Be direct and concise \u2014 1-3 sentences is ideal, not paragraphs
+- Give the answer or link first, explain only if needed
+- No corporate speak, no enthusiasm performance
 - Get to the point immediately
-- If you need info, just ask - no softening
-- 2-3 short paragraphs max
+- If you need info, just ask \u2014 no softening
+- NEVER use numbered step-by-step lists for simple questions. A direct link + 1 sentence beats a 5-step guide every time.
+- NEVER say "I've escalated this" or "someone will reach out" \u2014 if you have a URL or answer, give it directly
 
 ## NEVER Use These Phrases
 - "Great!" or exclamatory openers
@@ -2776,6 +2745,14 @@ var BASE_DRAFT_PROMPT = `You are a support agent. Write a helpful response to th
 - "I understand" or "I hear you"
 - "Thanks for reaching out"
 - Em dashes (\u2014)
+- "Someone will reach out" / "I've escalated this"
+- "Pricing varies" (give the link instead)
+
+## Prices and Numbers
+- NEVER cite specific dollar amounts, percentages, or discount numbers unless they appear in the gathered context below
+- If you don't have pricing data, link to the pricing/buy page instead
+- Wrong: "The course is $495 with a 45% discount"
+- Right: "You can see current pricing at [URL]"
 
 ## If You Don't Have Info
 Don't make things up. If knowledge base has no answer:
@@ -2812,9 +2789,10 @@ function buildBillingPrompt(context) {
   return `${BASE_DRAFT_PROMPT}
 
 ## Billing/Invoice
-- Point them to the invoices page: ${invoicesUrl}
-- Invoices are customizable - they can add company/tax info
-- PDFs are editable if they need adjustments`;
+- Give them the direct link: ${invoicesUrl}
+- That's it. Invoices are customizable (company name, tax ID, address) and PDFs are editable.
+- Don't write a numbered step-by-step guide \u2014 just the link + one sentence.
+- NEVER fabricate dollar amounts or discount percentages.`;
 }
 function buildPresalesFaqPrompt(context) {
   let promotionsSection = "";
@@ -2839,7 +2817,8 @@ Use this data to answer pricing, discount, and coupon questions. If a customer a
 ## Presales FAQ
 - Answer pricing, curriculum, requirements, and discount questions
 - Only reference information from the knowledge base or gathered context
-- Don't fabricate pricing or feature details
+- NEVER fabricate specific dollar amounts or discount percentages
+- For pricing questions: link to the buy/pricing page, explain that PPP/regional pricing adjusts automatically at checkout
 - If unsure, offer to connect them with the team${promotionsSection}`;
 }
 function buildPresalesTeamPrompt(context) {
@@ -2870,17 +2849,16 @@ Use this data to answer team seat questions. If a customer needs more seats or w
 ## Team/Enterprise Inquiries
 - Answer questions about team licensing, seat management, enterprise pricing
 - If license data is available above, reference it directly
-- For pricing questions beyond what's shown, offer to connect with the team
-- For enterprise custom deals, escalate to the team${licenseSection}`;
+- Link to the buy/pricing page for pricing questions \u2014 don't say "someone will reach out" or "pricing varies"
+- NEVER fabricate seat counts, pricing tiers, or discount numbers${licenseSection}`;
 }
 var STATIC_CATEGORY_PROMPTS = {
   support_access: `${BASE_DRAFT_PROMPT}
 
 ## Access Issues
-- First check if we found their purchase
-- If no purchase found: ask which email they used to buy
-- If purchase found: offer magic link or check their login method
-- GitHub login issues: they may have multiple GitHub accounts`,
+- If purchase found: send the magic login link directly
+- If no purchase found: ask which email they used to buy (1 sentence)
+- Don't write a multi-step troubleshooting guide unless they've already tried the basics`,
   support_transfer: `${BASE_DRAFT_PROMPT}
 
 ## Transfer Requests
@@ -3148,6 +3126,34 @@ async function gather(input, options = {}) {
       })
     );
   }
+  if (tools.searchResolvedThreads) {
+    const query = `${message.subject} ${message.body}`.slice(0, 500);
+    gatherPromises.push(
+      withTimeout(
+        (async () => {
+          try {
+            const resolvedThreads = await tools.searchResolvedThreads(
+              query,
+              appId
+            );
+            result.knowledge = [...result.knowledge, ...resolvedThreads];
+          } catch (error) {
+            result.gatherErrors.push({
+              step: "resolved_threads",
+              error: error instanceof Error ? error.message : "Unknown error"
+            });
+          }
+        })(),
+        timeout,
+        "resolved threads search"
+      ).catch((err) => {
+        result.gatherErrors.push({
+          step: "resolved_threads",
+          error: err.message
+        });
+      })
+    );
+  }
   if (tools.getHistory && message.conversationId) {
     gatherPromises.push(
       withTimeout(
@@ -3560,7 +3566,7 @@ ${message.body}
 ---
 Write your response:`;
   if (useAgentMode && appId) {
-    const { runSupportAgent } = await import("./config-HHQF5U6R.js");
+    const { runSupportAgent } = await import("./config-3GIHLVO4.js");
     await log("debug", "draft using agent mode", {
       workflow: "pipeline",
       step: "draft",
@@ -3670,6 +3676,25 @@ Write your response:`;
   });
   const toolsUsed = memories.length > 0 ? ["memory_query"] : [];
   const durationMs = Date.now() - startTime;
+  if (!result.text?.trim()) {
+    await log("warn", "draft:empty-response", {
+      workflow: "pipeline",
+      step: "draft",
+      appId,
+      conversationId,
+      category: classification.category,
+      model,
+      durationMs
+    });
+    return {
+      draft: "",
+      reasoning: "LLM returned empty response",
+      toolsUsed,
+      durationMs,
+      templateUsed: void 0,
+      memoriesCited: void 0
+    };
+  }
   await log("info", "draft:decision", {
     workflow: "pipeline",
     step: "draft",
@@ -3841,7 +3866,8 @@ var ROUTING_RULES = [
   // NOTE: Must be BEFORE unknown_escalate so isPersonalToInstructor signal is honored
   {
     name: "personal_to_instructor",
-    condition: ({ classification }) => classification.signals.isPersonalToInstructor,
+    condition: ({ classification }) => classification.signals.isPersonalToInstructor && // ADR-0003 D4: presales_faq should respond, not escalate to instructor
+    classification.category !== "presales_faq",
     action: "escalate_instructor",
     reason: "Personal message addressed to instructor"
   },
@@ -4055,7 +4081,8 @@ var THREAD_ROUTING_RULES = [
   // NOTE: Must be BEFORE unknown_escalate so isPersonalToInstructor signal is honored
   {
     name: "personal_to_instructor",
-    condition: ({ classification }) => classification.signals.isPersonalToInstructor,
+    condition: ({ classification }) => classification.signals.isPersonalToInstructor && // ADR-0003 D4: presales_faq should respond, not escalate to instructor
+    classification.category !== "presales_faq",
     action: "escalate_instructor",
     reason: "Personal message addressed to instructor"
   },
@@ -4700,6 +4727,19 @@ var FABRICATION_PATTERNS = {
   percentage: /\d+%/g,
   guarantee: /guarantee|always|never|definitely|certainly/gi
 };
+var SIGNATURE_PATTERNS = [
+  /\n\s*Best,?\s*$/im,
+  /\n\s*Thanks,?\s*$/im,
+  /\n\s*Cheers,?\s*$/im,
+  /\n\s*Sincerely,?\s*$/im,
+  /\n\s*Kind regards,?\s*$/im,
+  /\n\s*Best regards,?\s*$/im,
+  /\n\s*Warm regards,?\s*$/im,
+  /\bTeam AI Hero\b/i,
+  /\bTeam Total TypeScript\b/i,
+  /\bTeam Epic Web\b/i,
+  /\bTeam egghead\b/i
+];
 var MIN_RESPONSE_LENGTH = 10;
 var MAX_RESPONSE_LENGTH = 2e3;
 function calculateConfidenceScore(issues) {
@@ -4767,6 +4807,23 @@ function checkBannedPhrases(draft2) {
   }
   return issues;
 }
+function checkSignatures(draft2) {
+  const issues = [];
+  for (const pattern of SIGNATURE_PATTERNS) {
+    const match = draft2.match(pattern);
+    if (match) {
+      issues.push({
+        type: "banned_phrase",
+        severity: "error",
+        message: "Response contains a signature (Front adds signatures automatically)",
+        match: match[0].trim(),
+        position: match.index
+      });
+      break;
+    }
+  }
+  return issues;
+}
 function checkFabrication(draft2, context) {
   const issues = [];
   const hasKnowledge = (context.knowledge?.length ?? 0) > 0;
@@ -5059,6 +5116,7 @@ function validateSync(input) {
     ...checkInternalLeaks(draft2),
     ...checkMetaCommentary(draft2),
     ...checkBannedPhrases(draft2),
+    ...checkSignatures(draft2),
     ...checkFabrication(draft2, context),
     ...checkLength(draft2)
   ];
@@ -5096,6 +5154,7 @@ async function validate(input, options = {}) {
     ...checkInternalLeaks(draft2),
     ...checkMetaCommentary(draft2),
     ...checkBannedPhrases(draft2),
+    ...checkSignatures(draft2),
     ...checkFabrication(draft2, context),
     ...checkLength(draft2)
   ];
@@ -7096,4 +7155,4 @@ export {
   runPipeline,
   runThreadPipeline
 };
-//# sourceMappingURL=chunk-F3WI3BN5.js.map
+//# sourceMappingURL=chunk-4VWYLYQ6.js.map