@vellumai/assistant 0.3.5 → 0.3.6

package/src/__tests__/channel-approval.test.ts

@@ -195,6 +195,43 @@ describe('parseApprovalDecision', () => {
   ])('returns null for non-matching text: "%s"', (input) => {
     expect(parseApprovalDecision(input)).toBeNull();
   });
+
+  // ── Run-reference tag extraction ────────────────────────────────
+
+  test('extracts runId from [ref:...] tag with approve decision', () => {
+    const result = parseApprovalDecision('yes [ref:run-abc-123]');
+    expect(result).not.toBeNull();
+    expect(result!.action).toBe('approve_once');
+    expect(result!.source).toBe('plain_text');
+    expect(result!.runId).toBe('run-abc-123');
+  });
+
+  test('extracts runId from [ref:...] tag with reject decision', () => {
+    const result = parseApprovalDecision('no [ref:run-xyz-456]');
+    expect(result).not.toBeNull();
+    expect(result!.action).toBe('reject');
+    expect(result!.runId).toBe('run-xyz-456');
+  });
+
+  test('extracts runId from [ref:...] tag with always decision', () => {
+    const result = parseApprovalDecision('always [ref:run-789]');
+    expect(result).not.toBeNull();
+    expect(result!.action).toBe('approve_always');
+    expect(result!.runId).toBe('run-789');
+  });
+
+  test('handles ref tag on separate line', () => {
+    const result = parseApprovalDecision('yes\n[ref:run-abc-123]');
+    expect(result).not.toBeNull();
+    expect(result!.action).toBe('approve_once');
+    expect(result!.runId).toBe('run-abc-123');
+  });
+
+  test('decision without ref tag has no runId', () => {
+    const result = parseApprovalDecision('yes');
+    expect(result).not.toBeNull();
+    expect(result!.runId).toBeUndefined();
+  });
 });
 
 // ═══════════════════════════════════════════════════════════════════════════

package/src/__tests__/channel-approvals.test.ts

@@ -40,7 +40,6 @@ import {
   getChannelApprovalPrompt,
   buildApprovalUIMetadata,
   handleChannelDecision,
-  buildReminderPrompt,
   buildGuardianApprovalPrompt,
   channelSupportsRichApprovalUI,
 } from '../runtime/channel-approvals.js';
@@ -488,67 +487,7 @@ describe('handleChannelDecision', () => {
 });
 
 // ═══════════════════════════════════════════════════════════════════════════
-// 4. buildReminderPrompt
-// ═══════════════════════════════════════════════════════════════════════════
-
-describe('buildReminderPrompt', () => {
-  test('prefixes promptText with a reminder', () => {
-    const original: ChannelApprovalPrompt = {
-      promptText: 'Allow shell?',
-      actions: [
-        { id: 'approve_once', label: 'Approve once' },
-        { id: 'reject', label: 'Reject' },
-      ],
-      plainTextFallback: 'Reply yes or no.',
-    };
-
-    const reminder = buildReminderPrompt(original);
-    expect(reminder.promptText).toContain("I'm still waiting");
-    expect(reminder.promptText).toContain('Allow shell?');
-  });
-
-  test('preserves the original actions', () => {
-    const original: ChannelApprovalPrompt = {
-      promptText: 'Approve file_edit?',
-      actions: [
-        { id: 'approve_once', label: 'Approve once' },
-        { id: 'approve_always', label: 'Approve always' },
-        { id: 'reject', label: 'Reject' },
-      ],
-      plainTextFallback: 'Reply yes, always, or no.',
-    };
-
-    const reminder = buildReminderPrompt(original);
-    expect(reminder.actions).toEqual(original.actions);
-  });
-
-  test('prefixes plainTextFallback with a reminder', () => {
-    const original: ChannelApprovalPrompt = {
-      promptText: 'Allow bash?',
-      actions: [],
-      plainTextFallback: 'Reply yes or no.',
-    };
-
-    const reminder = buildReminderPrompt(original);
-    expect(reminder.plainTextFallback).toContain("I'm still waiting");
-    expect(reminder.plainTextFallback).toContain('Reply yes or no.');
-  });
-
-  test('does not mutate the original prompt', () => {
-    const original: ChannelApprovalPrompt = {
-      promptText: 'Allow grep?',
-      actions: [{ id: 'approve_once', label: 'Approve once' }],
-      plainTextFallback: 'Reply yes.',
-    };
-
-    const originalText = original.promptText;
-    buildReminderPrompt(original);
-    expect(original.promptText).toBe(originalText);
-  });
-});
-
-// ═══════════════════════════════════════════════════════════════════════════
-// 5. buildGuardianApprovalPrompt
+// 4. buildGuardianApprovalPrompt
 // ═══════════════════════════════════════════════════════════════════════════
 
 describe('buildGuardianApprovalPrompt', () => {
@@ -594,7 +533,7 @@ describe('buildGuardianApprovalPrompt', () => {
 });
 
 // ═══════════════════════════════════════════════════════════════════════════
-// 6. channelSupportsRichApprovalUI
+// 5. channelSupportsRichApprovalUI
 // ═══════════════════════════════════════════════════════════════════════════
 
 describe('channelSupportsRichApprovalUI', () => {
@@ -602,8 +541,8 @@ describe('channelSupportsRichApprovalUI', () => {
     expect(channelSupportsRichApprovalUI('telegram')).toBe(true);
   });
 
-  test('returns false for http-api', () => {
-    expect(channelSupportsRichApprovalUI('http-api')).toBe(false);
+  test('returns false for macos', () => {
+    expect(channelSupportsRichApprovalUI('macos')).toBe(false);
   });
 
   test('returns false for sms', () => {