@vellumai/assistant 0.4.11 → 0.4.13

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@vellumai/assistant",
-  "version": "0.4.11",
+  "version": "0.4.13",
   "type": "module",
   "bin": {
     "vellum": "./src/index.ts"

package/src/__tests__/guardian-verify-setup-skill-regression.test.ts

@@ -5,113 +5,128 @@
  * so the user does not have to manually ask whether verification succeeded.
  */
 
-import { readFileSync } from 'node:fs';
-import { resolve } from 'node:path';
+import { readFileSync } from "node:fs";
+import { resolve } from "node:path";
 
-import { describe, expect, test } from 'bun:test';
+import { describe, expect, test } from "bun:test";
 
 // ---------------------------------------------------------------------------
 // Locate the skill SKILL.md
 // ---------------------------------------------------------------------------
 
-const ASSISTANT_DIR = resolve(import.meta.dirname ?? __dirname, '..', '..');
+const ASSISTANT_DIR = resolve(import.meta.dirname ?? __dirname, "..", "..");
 const SKILL_PATH = resolve(
   ASSISTANT_DIR,
-  'src',
-  'config',
-  'vellum-skills',
-  'guardian-verify-setup',
-  'SKILL.md',
+  "src",
+  "config",
+  "bundled-skills",
+  "guardian-verify-setup",
+  "SKILL.md",
 );
 
-const skillContent = readFileSync(SKILL_PATH, 'utf-8');
+const skillContent = readFileSync(SKILL_PATH, "utf-8");
 
 // ---------------------------------------------------------------------------
 // Tests
 // ---------------------------------------------------------------------------
 
-describe('guardian-verify-setup skill — voice auto-followup', () => {
-  test('voice path in Step 3 references the auto-check polling loop', () => {
+describe("guardian-verify-setup skill — voice auto-followup", () => {
+  test("voice path in Step 3 references the auto-check polling loop", () => {
     // The voice success instruction in Step 3 must direct the assistant to
     // begin the polling loop rather than waiting for the user to report back.
     expect(skillContent).toContain(
-      'immediately begin the voice auto-check polling loop',
+      "immediately begin the voice auto-check polling loop",
     );
   });
 
-  test('voice path in Step 4 (resend) references the auto-check polling loop', () => {
+  test("voice path in Step 4 (resend) references the auto-check polling loop", () => {
     // After a voice resend, the same auto-check behavior must kick in.
-    const resendSection = skillContent.split('## Step 4')[1]?.split('## Step 5')[0] ?? '';
-    expect(resendSection).toContain(
-      'voice auto-check polling loop',
-    );
+    const resendSection =
+      skillContent.split("## Step 4")[1]?.split("## Step 5")[0] ?? "";
+    expect(resendSection).toContain("voice auto-check polling loop");
   });
 
-  test('contains a Voice Auto-Check Polling section', () => {
-    expect(skillContent).toContain('## Voice Auto-Check Polling');
+  test("contains a Voice Auto-Check Polling section", () => {
+    expect(skillContent).toContain("## Voice Auto-Check Polling");
   });
 
-  test('polling section specifies the correct status endpoint for voice', () => {
+  test("polling section specifies the correct status endpoint for voice", () => {
     const pollingSection =
-      skillContent.split('## Voice Auto-Check Polling')[1]?.split('## Step 6')[0] ?? '';
+      skillContent
+        .split("## Voice Auto-Check Polling")[1]
+        ?.split("## Step 6")[0] ?? "";
     expect(pollingSection).toContain(
-      '/v1/integrations/guardian/status?channel=voice',
+      "/v1/integrations/guardian/status?channel=voice",
     );
   });
 
-  test('polling section includes ~15 second interval', () => {
+  test("polling section includes ~15 second interval", () => {
     const pollingSection =
-      skillContent.split('## Voice Auto-Check Polling')[1]?.split('## Step 6')[0] ?? '';
-    expect(pollingSection).toContain('~15 seconds');
+      skillContent
+        .split("## Voice Auto-Check Polling")[1]
+        ?.split("## Step 6")[0] ?? "";
+    expect(pollingSection).toContain("~15 seconds");
   });
 
-  test('polling section includes 2-minute timeout', () => {
+  test("polling section includes 2-minute timeout", () => {
     const pollingSection =
-      skillContent.split('## Voice Auto-Check Polling')[1]?.split('## Step 6')[0] ?? '';
-    expect(pollingSection).toContain('2 minutes');
+      skillContent
+        .split("## Voice Auto-Check Polling")[1]
+        ?.split("## Step 6")[0] ?? "";
+    expect(pollingSection).toContain("2 minutes");
   });
 
-  test('polling section checks for bound: true', () => {
+  test("polling section checks for bound: true", () => {
     const pollingSection =
-      skillContent.split('## Voice Auto-Check Polling')[1]?.split('## Step 6')[0] ?? '';
-    expect(pollingSection).toContain('bound: true');
+      skillContent
+        .split("## Voice Auto-Check Polling")[1]
+        ?.split("## Step 6")[0] ?? "";
+    expect(pollingSection).toContain("bound: true");
   });
 
-  test('polling section includes proactive success confirmation', () => {
+  test("polling section includes proactive success confirmation", () => {
     const pollingSection =
-      skillContent.split('## Voice Auto-Check Polling')[1]?.split('## Step 6')[0] ?? '';
-    expect(pollingSection).toContain('proactive success message');
+      skillContent
+        .split("## Voice Auto-Check Polling")[1]
+        ?.split("## Step 6")[0] ?? "";
+    expect(pollingSection).toContain("proactive success message");
   });
 
-  test('polling section includes timeout fallback with resend/restart offer', () => {
+  test("polling section includes timeout fallback with resend/restart offer", () => {
     const pollingSection =
-      skillContent.split('## Voice Auto-Check Polling')[1]?.split('## Step 6')[0] ?? '';
-    expect(pollingSection).toContain('timeout');
-    expect(pollingSection).toContain('resend');
+      skillContent
+        .split("## Voice Auto-Check Polling")[1]
+        ?.split("## Step 6")[0] ?? "";
+    expect(pollingSection).toContain("timeout");
+    expect(pollingSection).toContain("resend");
   });
 
-  test('polling section includes rebind guard against false-success from pre-existing binding', () => {
+  test("polling section includes rebind guard against false-success from pre-existing binding", () => {
     const pollingSection =
-      skillContent.split('## Voice Auto-Check Polling')[1]?.split('## Step 6')[0] ?? '';
+      skillContent
+        .split("## Voice Auto-Check Polling")[1]
+        ?.split("## Step 6")[0] ?? "";
     // Must mention rebind guard concept
-    expect(pollingSection).toContain('Rebind guard');
+    expect(pollingSection).toContain("Rebind guard");
     // Must instruct not to trust the first bound: true in a rebind flow
     expect(pollingSection).toContain(
-      'do NOT treat the first `bound: true` poll result as success',
+      "do NOT treat the first `bound: true` poll result as success",
     );
     // Must reference bound_at timestamp comparison as the primary mechanism
-    expect(pollingSection).toContain('bound_at');
+    expect(pollingSection).toContain("bound_at");
     // Must have a fallback for when bound_at is unavailable
-    expect(pollingSection).toContain('second poll onward');
+    expect(pollingSection).toContain("second poll onward");
     // Must clarify non-rebind flows are unaffected
-    expect(pollingSection).toContain('Non-rebind flows');
+    expect(pollingSection).toContain("Non-rebind flows");
   });
 
-  test('polling is voice-only — does not apply to SMS or Telegram', () => {
+  test("polling is voice-only — does not apply to SMS or Telegram", () => {
     const pollingSection =
-      skillContent.split('## Voice Auto-Check Polling')[1]?.split('## Step 6')[0] ?? '';
-    expect(pollingSection).toContain('voice-only');
-    expect(pollingSection).toContain('Do NOT poll for SMS or Telegram');
+      skillContent
+        .split("## Voice Auto-Check Polling")[1]
+        ?.split("## Step 6")[0] ?? "";
+    expect(pollingSection).toContain("voice-only");
+    expect(pollingSection).toContain("Do NOT poll for SMS or Telegram");
   });
 
   test('no instruction requires waiting for user to ask "did it work?"', () => {
@@ -119,23 +134,22 @@ describe('guardian-verify-setup skill — voice auto-followup', () => {
     // confirm that voice verification worked. The auto-check polling loop
     // makes this unnecessary.
     const voiceAutoCheckSection =
-      skillContent.split('## Voice Auto-Check Polling')[1]?.split('## Step 6')[0] ?? '';
-    expect(voiceAutoCheckSection).toContain(
-      'Do NOT require the user to ask',
-    );
+      skillContent
+        .split("## Voice Auto-Check Polling")[1]
+        ?.split("## Step 6")[0] ?? "";
+    expect(voiceAutoCheckSection).toContain("Do NOT require the user to ask");
     // The voice bullet in Step 3 should not instruct the assistant to wait
     // for the user to confirm or ask if it worked. Narrow to just the voice
     // bullet line to avoid false positives from Telegram's "wait for the
     // user to confirm they clicked the link" which is unrelated to voice.
-    const step3Section = skillContent
-      .split('## Step 3')[1]
-      ?.split('## Step 4')[0] ?? '';
+    const step3Section =
+      skillContent.split("## Step 3")[1]?.split("## Step 4")[0] ?? "";
     const voiceBullet = step3Section
-      .split('\n')
+      .split("\n")
       .filter((line) => /^\s*-\s+\*\*Voice\*\*/.test(line))
-      .join('\n');
+      .join("\n");
     expect(voiceBullet).not.toHaveLength(0);
-    expect(voiceBullet).not.toContain('wait for the user to confirm');
-    expect(voiceBullet).not.toContain('ask the user if it worked');
+    expect(voiceBullet).not.toContain("wait for the user to confirm");
+    expect(voiceBullet).not.toContain("ask the user if it worked");
   });
 });