@tellescope/sdk 1.256.0 → 1.256.1

package/src/tests/api_tests/start_ai_conversation.test.ts

@@ -0,0 +1,256 @@
+require('source-map-support').install();
+
+import { Session } from "../../sdk"
+import { log_header, wait, async_test, assert } from "@tellescope/testing"
+import { setup_tests } from "../setup"
+
+const host = process.env.API_URL || "http://localhost:8080"
+
+const TEST_MARKER = '__TELLESCOPE_TEST_INCOMING_SMS_AUTOMATION__'
+const TEST_PHONE = '+15555555555' // send_sms short-circuits real sends for this number
+
+// SMS AI conversations (startAIConversation journey action).
+// Runtime flows (start/supersede/takeover/inbound absorption) require the organization to have an
+// SMS number configured — when it doesn't (typical dev org), those sections assert the
+// configuration error path and are otherwise skipped; config validation runs everywhere.
+export const start_ai_conversation_tests = async ({ sdk, sdkNonAdmin } : { sdk: Session, sdkNonAdmin: Session }) => {
+  log_header("Start AI Conversation (SMS Agent) Tests")
+
+  const users = await sdk.api.users.getSome({})
+  const senderId = users[0].id
+
+  const journey = await sdk.api.journeys.createOne({ title: 'SMS Agent Test Journey' })
+  const createdEnduserIds: string[] = []
+
+  try {
+    /*************** Config validation round-trip ***************/
+    const validInfo = {
+      channel: 'SMS' as const,
+      senderId,
+      initialMessage: 'Hello {{enduser.fname}}! Quick question for you.',
+      prompt: 'You are texting on behalf of Test Clinic. Answer briefly.',
+      outcomes: [{ value: 'resolved', description: 'The question was answered' }],
+    }
+
+    const step = await sdk.api.automation_steps.createOne({
+      journeyId: journey.id,
+      events: [{ type: 'onJourneyStart', info: {} }],
+      action: { type: 'startAIConversation', info: validInfo },
+    })
+    assert(
+      step.action.type === 'startAIConversation' && step.action.info.initialMessage === validInfo.initialMessage,
+      'startAIConversation step round-trip failed',
+      'startAIConversation step round-trips',
+    )
+
+    const outcomeChild = await sdk.api.automation_steps.createOne({
+      journeyId: journey.id,
+      events: [{ type: 'onAIConversationOutcome', info: { automationStepId: step.id, outcome: 'resolved' } }],
+      action: { type: 'addEnduserTags', info: { tags: ['sms-agent-resolved'] } },
+    })
+    assert(
+      outcomeChild.events[0].type === 'onAIConversationOutcome',
+      'onAIConversationOutcome event round-trip failed',
+      'onAIConversationOutcome event round-trips',
+    )
+
+    await async_test('startAIConversation without prompt is rejected',
+      () => sdk.api.automation_steps.createOne({
+        journeyId: journey.id,
+        events: [{ type: 'onJourneyStart', info: {} }],
+        action: { type: 'startAIConversation', info: { ...validInfo, prompt: undefined } as any },
+      }),
+      { shouldError: true, onError: () => true },
+    )
+    await async_test('startAIConversation without senderId is rejected',
+      () => sdk.api.automation_steps.createOne({
+        journeyId: journey.id,
+        events: [{ type: 'onJourneyStart', info: {} }],
+        action: { type: 'startAIConversation', info: { ...validInfo, senderId: undefined } as any },
+      }),
+      { shouldError: true, onError: () => true },
+    )
+    await async_test('startAIConversation without initialMessage is rejected',
+      () => sdk.api.automation_steps.createOne({
+        journeyId: journey.id,
+        events: [{ type: 'onJourneyStart', info: {} }],
+        action: { type: 'startAIConversation', info: { ...validInfo, initialMessage: undefined } as any },
+      }),
+      { shouldError: true, onError: () => true },
+    )
+    await async_test('startAIConversation with invalid tool is rejected',
+      () => sdk.api.automation_steps.createOne({
+        journeyId: journey.id,
+        events: [{ type: 'onJourneyStart', info: {} }],
+        action: { type: 'startAIConversation', info: { ...validInfo, tools: [{ type: 'Submit Form', info: { formId: 'not-an-id' } }] } as any },
+      }),
+      { shouldError: true, onError: () => true },
+    )
+
+    /*************** Runtime: the process endpoint ***************/
+    const processFor = async (enduserId: string) => {
+      const automatedAction = await sdk.api.automated_actions.createOne({
+        enduserId,
+        journeyId: journey.id,
+        automationStepId: step.id,
+        event: { type: 'onJourneyStart', info: {} },
+        action: { type: 'startAIConversation', info: validInfo },
+        status: 'active',
+        processAfter: Date.now(),
+      })
+      return sdk.api.automated_actions.process({ automatedActionId: automatedAction.id })
+    }
+
+    // no-phone error path works regardless of org SMS configuration
+    const phonelessEnduser = await sdk.api.endusers.createOne({ fname: 'sms-agent-nophone' })
+    createdEnduserIds.push(phonelessEnduser.id)
+    const noPhoneResult = await processFor(phonelessEnduser.id)
+    assert(
+      !noPhoneResult.success && !!noPhoneResult.error,
+      'phoneless start should fail with an error',
+      'start action errors for a contact without a phone',
+    )
+
+    // resolve the same organization the action does (last organizationId, else businessId) —
+    // getSome({})[0] is an arbitrary sub-organization and can be a different record entirely
+    const session = sdk.userInfo as { businessId?: string, organizationIds?: string[] }
+    const resolvedOrgId = [...(session.organizationIds ?? [])].pop() || session.businessId
+    const organization = (await sdk.api.organizations.getSome({})).find(o => o.id === resolvedOrgId)
+    const orgHasNumber = !!(organization as any)?.twilioNumber || !!((organization as any)?.twilioNumbers?.length)
+
+    if (!orgHasNumber) {
+      // still assert the wiring: the action processes and reports a configuration error rather than
+      // crashing. AI enablement is checked before the SMS number and short-circuits first, and
+      // bedrockAIAllowed/creditCount are super-admin-only (unreadable here), so accept either.
+      const enduser = await sdk.api.endusers.createOne({ fname: 'sms-agent-nonumber', phone: TEST_PHONE })
+      createdEnduserIds.push(enduser.id)
+      const result = await processFor(enduser.id)
+      assert(
+        !result.success && /no sms number configured|ai is not enabled/i.test(result.error || ''),
+        `expected a configuration error, got: ${result.error}`,
+        'start action reports a configuration error (org has no SMS number — runtime flows skipped)',
+      )
+      return
+    }
+
+    /*************** Runtime flows (org has an SMS number) ***************/
+    const enduser = await sdk.api.endusers.createOne({ fname: 'sms-agent-runtime', phone: TEST_PHONE })
+    createdEnduserIds.push(enduser.id)
+
+    const startResult = await processFor(enduser.id)
+    assert(startResult.success === true, `start failed: ${startResult.error}`, 'start action succeeds')
+    await wait(undefined, 1000)
+
+    let refreshed = await sdk.api.endusers.getOne(enduser.id)
+    const entry = (refreshed.aiConversations ?? []).find(c => c.active)
+    assert(!!entry, 'no active aiConversations entry on enduser', 'enduser has an active conversation entry')
+    assert(entry?.destination === TEST_PHONE, 'wrong destination on entry', 'conversation entry pairs the contact number')
+
+    const conversation = await sdk.api.ai_conversations.getOne(entry!.aiConversationId)
+    assert(conversation.type === 'sms_agent', 'wrong ai_conversations type', 'ai_conversations doc has type sms_agent')
+
+    const initialMessages = await sdk.api.sms_messages.getSome({ filter: { enduserId: enduser.id } })
+    const outbound = initialMessages.find(m => !m.inbound && m.automationStepId === step.id)
+    // exact equality on a merge-field-containing opener guards two regressions: templating must
+    // happen in send_sms (raw text on the record), and the agent-reply {{ escaping must NOT be
+    // applied to the org-authored initial message (preserveMergeFields)
+    assert(
+      !!outbound && outbound.message === validInfo.initialMessage,
+      'initial SMS record missing or not raw text',
+      'initial SMS logged with raw (untemplated) text + automationStepId',
+    )
+
+    // supersede: second start on the same pairing replaces the first
+    const secondStart = await processFor(enduser.id)
+    assert(secondStart.success === true, `second start failed: ${secondStart.error}`, 'duplicate start succeeds (replace)')
+    await wait(undefined, 1000)
+    refreshed = await sdk.api.endusers.getOne(enduser.id)
+    const activeEntries = (refreshed.aiConversations ?? []).filter(c => c.active)
+    assert(activeEntries.length === 1, 'expected exactly one active conversation', 'supersede leaves exactly one active conversation')
+    assert(
+      activeEntries[0].aiConversationId !== entry!.aiConversationId,
+      'active conversation was not replaced',
+      'supersede started a fresh conversation',
+    )
+
+    // inbound absorption via the testing hook: STOP ends the conversation
+    await sdk.api.sms_messages.createOne({
+      enduserId: enduser.id,
+      message: `stop${TEST_MARKER}`, // marker is stripped before the AI branch sees the text
+      inbound: true,
+      logOnly: true,
+      phoneNumber: activeEntries[0].source,
+      enduserPhoneNumber: TEST_PHONE,
+    } as any)
+    await wait(undefined, 2000)
+    refreshed = await sdk.api.endusers.getOne(enduser.id)
+    const stopped = (refreshed.aiConversations ?? []).find(c => c.aiConversationId === activeEntries[0].aiConversationId)
+    assert(
+      stopped?.active === false && stopped?.endedReason === 'unsubscribed',
+      `STOP did not end the conversation (${JSON.stringify(stopped)})`,
+      'STOP reply ends the conversation',
+    )
+
+    // staff takeover: start again, then a manual (user-session) SMS on the thread ends it
+    const takeoverStart = await processFor(enduser.id)
+    assert(takeoverStart.success === true, `takeover-start failed: ${takeoverStart.error}`, 'start (for takeover test) succeeds')
+    await wait(undefined, 1000)
+    await sdk.api.sms_messages.createOne({
+      enduserId: enduser.id,
+      message: 'Hi, this is a real person taking over.',
+      inbound: false,
+      phoneNumber: (await sdk.api.endusers.getOne(enduser.id)).aiConversations?.find(c => c.active)?.source,
+      enduserPhoneNumber: TEST_PHONE,
+    } as any)
+    await wait(undefined, 2000)
+    refreshed = await sdk.api.endusers.getOne(enduser.id)
+    const takenOver = (refreshed.aiConversations ?? []).find(c => c.endedReason === 'staff-takeover')
+    assert(!!takenOver, 'no staff-takeover ended entry found', 'manual staff SMS ends the conversation (takeover)')
+
+    // no-conversation baseline: marker inbound for a fresh enduser produces zero AI artifacts
+    const baselineEnduser = await sdk.api.endusers.createOne({ fname: 'sms-agent-baseline', phone: '+15555555554' })
+    createdEnduserIds.push(baselineEnduser.id)
+    await sdk.api.sms_messages.createOne({
+      enduserId: baselineEnduser.id,
+      message: `hello there${TEST_MARKER}`,
+      inbound: true,
+      logOnly: true,
+      phoneNumber: activeEntries[0].source,
+      enduserPhoneNumber: '+15555555554',
+    } as any)
+    await wait(undefined, 2000)
+    const baselineRefreshed = await sdk.api.endusers.getOne(baselineEnduser.id)
+    const baselineMessages = await sdk.api.sms_messages.getSome({ filter: { enduserId: baselineEnduser.id } })
+    assert(
+      !(baselineRefreshed.aiConversations ?? []).length && !baselineMessages.find(m => !m.inbound),
+      'AI artifacts appeared for an enduser with no conversation',
+      'no-conversation inbound produces zero AI artifacts (regression baseline)',
+    )
+  } finally {
+    await sdk.api.journeys.deleteOne(journey.id).catch(() => {})
+    for (const id of createdEnduserIds) {
+      await sdk.api.endusers.deleteOne(id).catch(() => {})
+    }
+  }
+}
+
+// Allow running this test file independently
+if (require.main === module) {
+  const sdk = new Session({ host })
+  const sdkNonAdmin = new Session({ host })
+
+  const runTests = async () => {
+    await setup_tests(sdk, sdkNonAdmin)
+    await start_ai_conversation_tests({ sdk, sdkNonAdmin })
+  }
+
+  runTests()
+    .then(() => {
+      console.log("✅ Start AI Conversation test suite completed successfully")
+      process.exit(0)
+    })
+    .catch((error) => {
+      console.error("❌ Start AI Conversation test suite failed:", error)
+      process.exit(1)
+    })
+}

package/src/tests/tests.ts

@@ -120,6 +120,8 @@ import { load_inbox_redaction_tests } from "./api_tests/security/F-0151-load-inb
 import { webhook_timeout_tests } from "./api_tests/security/F-0155-webhook-timeout.test";
 import { enduser_ai_summary_trust_gate_tests } from "./api_tests/security/enduser-ai-summary-trust-gate.test";
 import { allowed_paths_confinement_tests } from "./api_tests/security/F-0166-allowed-paths-wildcard-bypass.test";
+import { proxy_image_ssrf_tests } from "./api_tests/security/F-0025-proxy-image-unauth-ssrf.test";
+import { webhook_outbound_ssrf_tests } from "./api_tests/security/F-0015-webhook-outbound-ssrf.test";
 import { public_endpoint_auth_field_lock_tests } from "./api_tests/security/public_endpoint_auth_field_lock.test";
 import { formsort_webhook_auth_field_lock_tests } from "./api_tests/security/formsort_webhook_auth_field_lock.test";
 import { bulk_assignment_tests } from "./api_tests/bulk_assignment.test";
@@ -133,6 +135,7 @@ import { load_team_chat_tests } from "./api_tests/load_team_chat.test";
 import { form_started_trigger_tests } from "./api_tests/form_started_trigger.test";
 import { formsort_header_auth_tests } from "./api_tests/formsort_header_auth.test";
 import { form_submitted_trigger_tests } from "./api_tests/form_submitted_trigger.test";
+import { start_ai_conversation_tests } from "./api_tests/start_ai_conversation.test";
 import { dont_sync_to_elation_form_submission_tests } from "./api_tests/dont_sync_to_elation_form_submission.test";
 import { medication_added_trigger_tests } from "./api_tests/medication_added_trigger.test";
 import { conditional_logic_medication_unit_tests } from "./unit_tests/conditional_logic_medication.test";
@@ -144,6 +147,7 @@ import { openloop_webhooks_tests } from "./api_tests/openloop_webhooks.test";
 import { beluga_pharmacy_mappings_tests } from "./api_tests/beluga_pharmacy_mappings.test";
 import { mdi_case_offerings_tests } from "./api_tests/mdi_case_offerings.test";
 import { beluga_manual_sync_tests } from "./api_tests/beluga_manual_sync.test";
+import { beluga_trigger_refill_tests } from "./api_tests/beluga_trigger_refill.test";
 import { mdi_webhooks_tests } from "./api_tests/mdi_webhooks.test";
 import { account_switcher_tests } from "./api_tests/account_switcher.test";
 import { totp_mfa_tests } from "./api_tests/totp_mfa.test";
@@ -15274,6 +15278,8 @@ const ip_address_form_tests = async () => {
     await replace_form_field_template_values_tests()
     await mfa_tests()
     await setup_tests(sdk, sdkNonAdmin)
+    await proxy_image_ssrf_tests({ sdk, sdkNonAdmin })
+    await webhook_outbound_ssrf_tests({ sdk, sdkNonAdmin })
     await allowed_paths_confinement_tests({ sdk, sdkNonAdmin })
     await phone_calls_conference_hold_tests({ sdk, sdkNonAdmin })
     await tickets_bulk_assign_care_team_tests({ sdk, sdkNonAdmin })
@@ -15282,6 +15288,7 @@ const ip_address_form_tests = async () => {
     await automation_trigger_tests()
     await resource_access_tags_tests({ sdk, sdkNonAdmin })
     await beluga_manual_sync_tests({ sdk, sdkNonAdmin })
+    await beluga_trigger_refill_tests({ sdk, sdkNonAdmin })
     await beluga_pharmacy_mappings_tests({ sdk, sdkNonAdmin })
     await mdi_case_offerings_tests({ sdk, sdkNonAdmin })
     await enduser_write_restrictions_tests({ sdk, sdkNonAdmin })
@@ -15320,6 +15327,7 @@ const ip_address_form_tests = async () => {
     await chats_analytics_tests({ sdk, sdkNonAdmin })
     await field_redaction_tests({ sdk, sdkNonAdmin })
     await form_submitted_trigger_tests({ sdk, sdkNonAdmin })
+    await start_ai_conversation_tests({ sdk, sdkNonAdmin })
     await dont_sync_to_elation_form_submission_tests({ sdk, sdkNonAdmin })
     await date_string_validation_tests({ sdk, sdkNonAdmin })
     await phone_tree_enduser_condition_tests({ sdk, sdkNonAdmin })