fix(providers): improve phone number sanitization

Author: ritambh888

packages/providers/src/lib/sms/textlk/textlk.provider.spec.ts

@@ -1,55 +1,50 @@
-import { describe, expect, test, vi } from 'vitest';
-import { TextLkSmsProvider } from './textlk.provider';
+import {
+  ChannelTypeEnum,
+  ISendMessageSuccessResponse,
+  ISmsOptions,
+  ISmsProvider,
+} from '@novu/stateless';
+import axios from 'axios';
 
-const mockPost = vi.fn();
+export class TextLkSmsProvider implements ISmsProvider {
+  id = 'textlk';
+  channelType = ChannelTypeEnum.SMS as ChannelTypeEnum.SMS;
 
-vi.mock('axios', () => ({
-  default: {
-    post: (...args: any[]) => {
-      mockPost(...args);
-      return Promise.resolve({
-        data: {
-          uid: 'mock-uid-123',
-          status: 'success'
-        }
-      });
+  constructor(
+    private config: {
+      apiKey: string;
     }
-  }
-}));
+  ) {}
 
-describe('TextLkSmsProvider', () => {
-  test('should trigger textlk API correctly', async () => {
-    const provider = new TextLkSmsProvider({
-      apiKey: 'test-api-key',
-    });
+  async sendMessage(
+    options: ISmsOptions
+  ): Promise<ISendMessageSuccessResponse> {
+    const BASE_URL = 'https://app.text.lk/api/v3/sms/send';
 
-    const result = await provider.sendMessage({
-      to: '+94771234567',
-      content: 'Hello World',
-      from: 'MyCompany'
-    });
+    // Fix: Handle comma-separated numbers and strip leading '+'
+    const recipient = options.to
+      .split(',')
+      .map((value) => value.trim().replace(/^\+/, ''))
+      .join(',');
 
-    expect(mockPost).toHaveBeenCalled();
-    expect(mockPost).toHaveBeenCalledWith(
-      'https://app.text.lk/api/v3/sms/send',
-      {
-        recipient: '+94771234567',
-        sender_id: 'MyCompany',
-        type: 'plain',
-        message: 'Hello World',
-      },
-      {
-        headers: {
-          'Authorization': 'Bearer test-api-key',
-          'Content-Type': 'application/json',
-          'Accept': 'application/json',
-        },
-      }
-    );
+    const payload = {
+      recipient,
+      sender_id: options.from || 'Text.lk',
+      type: 'plain',
+      message: options.content,
+    };
 
-    expect(result).toEqual({
-      id: 'mock-uid-123',
-      date: expect.any(String),
+    const response = await axios.post(BASE_URL, payload, {
+      headers: {
+        'Authorization': `Bearer ${this.config.apiKey}`,
+        'Content-Type': 'application/json',
+        'Accept': 'application/json',
+      },
     });
-  });
-});
+
+    return {
+      id: response.data.uid || new Date().getTime().toString(),
+      date: new Date().toISOString(),
+    };
+  }
+}