@ihazz/bitrix24 1.1.0 → 1.1.2

package/tests/index.test.ts

@@ -1,94 +0,0 @@
-import { describe, it, expect, vi, afterEach } from 'vitest';
-import bitrix24Module, { collectBitrix24CallbackPaths } from '../index.ts';
-
-const runtime = {
-  config: { loadConfig: () => ({}) },
-  channel: {
-    routing: { resolveAgentRoute: vi.fn() },
-    reply: {
-      finalizeInboundContext: vi.fn(),
-      dispatchReplyWithBufferedBlockDispatcher: vi.fn(),
-    },
-    session: { recordInboundSession: vi.fn() },
-    pairing: {
-      readAllowFromStore: vi.fn(),
-      upsertPairingRequest: vi.fn(),
-      buildPairingReply: vi.fn(),
-    },
-  },
-  logging: { getChildLogger: vi.fn() },
-};
-
-describe('collectBitrix24CallbackPaths', () => {
-  afterEach(() => {
-    vi.restoreAllMocks();
-  });
-
-  it('collects unique callback paths from base config and accounts', () => {
-    const paths = collectBitrix24CallbackPaths({
-      channels: {
-        bitrix24: {
-          callbackUrl: 'https://example.com/hooks/bitrix24',
-          accounts: {
-            a: { callbackUrl: 'https://example.com/hooks/bitrix24-a' },
-            b: { callbackUrl: 'https://other.example.com/hooks/bitrix24-a' },
-          },
-        },
-      },
-    });
-
-    expect(paths).toEqual(['/hooks/bitrix24', '/hooks/bitrix24-a']);
-  });
-
-  it('skips invalid callback urls without dropping valid ones', () => {
-    const warnSpy = vi.spyOn(console, 'warn').mockImplementation(() => {});
-
-    const paths = collectBitrix24CallbackPaths({
-      channels: {
-        bitrix24: {
-          callbackUrl: 'not-a-url',
-          accounts: {
-            a: { callbackUrl: 'https://example.com/hooks/bitrix24-a' },
-          },
-        },
-      },
-    });
-
-    expect(paths).toEqual(['/hooks/bitrix24-a']);
-    expect(warnSpy).toHaveBeenCalledOnce();
-  });
-});
-
-describe('bitrix24 index module', () => {
-  it('registers HTTP routes for every configured callback path', () => {
-    const registerChannel = vi.fn();
-    const registerHttpRoute = vi.fn();
-
-    bitrix24Module.register({
-      config: {
-        channels: {
-          bitrix24: {
-            callbackUrl: 'https://example.com/hooks/bitrix24',
-            accounts: {
-              a: { callbackUrl: 'https://example.com/hooks/bitrix24-a' },
-            },
-          },
-        },
-      },
-      runtime: runtime as never,
-      registerChannel,
-      registerHttpRoute,
-    });
-
-    expect(registerChannel).toHaveBeenCalledOnce();
-    expect(registerHttpRoute).toHaveBeenCalledTimes(2);
-    expect(registerHttpRoute).toHaveBeenNthCalledWith(1, expect.objectContaining({
-      path: '/hooks/bitrix24',
-      auth: 'plugin',
-    }));
-    expect(registerHttpRoute).toHaveBeenNthCalledWith(2, expect.objectContaining({
-      path: '/hooks/bitrix24-a',
-      auth: 'plugin',
-    }));
-  });
-});

package/tests/media-service.test.ts

@@ -1,319 +0,0 @@
-import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest';
-import { existsSync, unlinkSync } from 'node:fs';
-import { join } from 'node:path';
-import { MediaService, resolveManagedMediaDir } from '../src/media-service.js';
-import { Bitrix24Api } from '../src/api.js';
-
-const silentLogger = {
-  info: vi.fn(),
-  warn: vi.fn(),
-  error: vi.fn(),
-  debug: vi.fn(),
-};
-
-// Mock the API
-function mockApi(): Bitrix24Api {
-  return {
-    getFileDownloadUrl: vi.fn().mockResolvedValue('https://example.com/download/test.txt?token=abc'),
-    uploadFile: vi.fn().mockResolvedValue({
-      file: { id: 100, name: 'upload-test.txt' },
-      messageId: 100,
-      chatId: 42,
-      dialogId: 'chat40985',
-    }),
-  } as unknown as Bitrix24Api;
-}
-
-describe('MediaService', () => {
-  let service: MediaService;
-  let api: Bitrix24Api;
-  const savedPaths: string[] = [];
-  const tempStateDir = '/tmp/openclaw-test-state';
-
-  beforeEach(() => {
-    process.env.OPENCLAW_STATE_DIR = tempStateDir;
-    api = mockApi();
-    service = new MediaService(api, silentLogger);
-    vi.clearAllMocks();
-  });
-
-  afterEach(() => {
-    // Clean up any files created during tests
-    for (const p of savedPaths) {
-      try { unlinkSync(p); } catch { /* ignore */ }
-    }
-    savedPaths.length = 0;
-    delete process.env.OPENCLAW_STATE_DIR;
-    delete process.env.CLAWDBOT_STATE_DIR;
-  });
-
-  describe('resolveManagedMediaDir', () => {
-    it('uses OpenClaw state media directory by default', () => {
-      const mediaDir = resolveManagedMediaDir({
-        HOME: '/tmp/test-home',
-      } as NodeJS.ProcessEnv);
-
-      expect(mediaDir).toBe('/tmp/test-home/.openclaw/media/bitrix24');
-    });
-
-    it('prefers OPENCLAW_STATE_DIR when it is configured', () => {
-      const mediaDir = resolveManagedMediaDir({
-        OPENCLAW_STATE_DIR: '/srv/openclaw-state',
-        HOME: '/tmp/test-home',
-      } as NodeJS.ProcessEnv);
-
-      expect(mediaDir).toBe('/srv/openclaw-state/media/bitrix24');
-    });
-  });
-
-  describe('downloadMedia', () => {
-    it('downloads and saves a file', async () => {
-      const mockContent = Buffer.from('hello world');
-      vi.spyOn(globalThis, 'fetch').mockResolvedValueOnce(
-        new Response(mockContent, { status: 200 }),
-      );
-
-      const result = await service.downloadMedia({
-        fileId: '123',
-        fileName: 'test.txt',
-        extension: 'txt',
-        webhookUrl: 'https://test.bitrix24.com/rest/1/token/',
-        bot: { botId: 7, botToken: 'bot_token' },
-        dialogId: 'chat40985',
-      });
-
-      expect(result).not.toBeNull();
-      expect(result!.contentType).toBe('text/plain');
-      expect(result!.name).toBe('test.txt');
-      expect(result!.path).toContain('test.txt');
-      expect(result!.path.startsWith(join(tempStateDir, 'media', 'bitrix24'))).toBe(true);
-      expect(existsSync(result!.path)).toBe(true);
-      savedPaths.push(result!.path);
-
-      expect(api.getFileDownloadUrl).toHaveBeenCalledWith(
-        'https://test.bitrix24.com/rest/1/token/',
-        { botId: 7, botToken: 'bot_token' },
-        'chat40985',
-        123,
-      );
-    });
-
-    it('returns correct MIME for images', async () => {
-      vi.spyOn(globalThis, 'fetch').mockResolvedValueOnce(
-        new Response(Buffer.from([0x89, 0x50, 0x4e, 0x47]), { status: 200 }),
-      );
-
-      const result = await service.downloadMedia({
-        fileId: '456',
-        fileName: 'photo.jpg',
-        extension: 'jpg',
-        webhookUrl: 'https://test.bitrix24.com/rest/1/token/',
-        bot: { botId: 7, botToken: 'bot_token' },
-        dialogId: 'chat40985',
-      });
-
-      expect(result).not.toBeNull();
-      expect(result!.contentType).toBe('image/jpeg');
-      if (result) savedPaths.push(result.path);
-    });
-
-    it('cleans up managed downloaded temp files', async () => {
-      vi.spyOn(globalThis, 'fetch').mockResolvedValueOnce(
-        new Response(Buffer.from('cleanup me'), { status: 200 }),
-      );
-
-      const result = await service.downloadMedia({
-        fileId: '654',
-        fileName: 'cleanup.txt',
-        extension: 'txt',
-        webhookUrl: 'https://test.bitrix24.com/rest/1/token/',
-        bot: { botId: 7, botToken: 'bot_token' },
-        dialogId: 'chat40985',
-      });
-
-      expect(result).not.toBeNull();
-      expect(existsSync(result!.path)).toBe(true);
-
-      await service.cleanupDownloadedMedia([result!.path]);
-
-      expect(existsSync(result!.path)).toBe(false);
-    });
-
-    it('rewrites private Bitrix download URLs to webhook origin', async () => {
-      (api.getFileDownloadUrl as ReturnType<typeof vi.fn>).mockResolvedValueOnce(
-        'https://192.168.50.50/rest/1/token/download/?token=abc',
-      );
-      vi.spyOn(globalThis, 'fetch').mockResolvedValueOnce(
-        new Response(Buffer.from('hello world'), { status: 200 }),
-      );
-
-      const result = await service.downloadMedia({
-        fileId: '777',
-        fileName: 'private.txt',
-        extension: 'txt',
-        webhookUrl: 'https://test.bitrix24.com/rest/1/token/',
-        bot: { botId: 7, botToken: 'bot_token' },
-        dialogId: '1',
-      });
-
-      expect(result).not.toBeNull();
-      expect(globalThis.fetch).toHaveBeenCalledWith(
-        'https://test.bitrix24.com/rest/1/token/download/?token=abc',
-        expect.objectContaining({ signal: expect.any(AbortSignal) }),
-      );
-      if (result) savedPaths.push(result.path);
-    });
-
-    it('returns null when API fails', async () => {
-      (api.getFileDownloadUrl as ReturnType<typeof vi.fn>).mockRejectedValueOnce(
-        new Error('API error'),
-      );
-
-      const result = await service.downloadMedia({
-        fileId: '789',
-        fileName: 'fail.txt',
-        extension: 'txt',
-        webhookUrl: 'https://test.bitrix24.com/rest/1/token/',
-        bot: { botId: 7, botToken: 'bot_token' },
-        dialogId: 'chat40985',
-      });
-
-      expect(result).toBeNull();
-      expect(silentLogger.error).toHaveBeenCalled();
-    });
-
-    it('returns null when no DOWNLOAD_URL', async () => {
-      (api.getFileDownloadUrl as ReturnType<typeof vi.fn>).mockResolvedValueOnce('');
-
-      const result = await service.downloadMedia({
-        fileId: '789',
-        fileName: 'nourl.txt',
-        extension: 'txt',
-        webhookUrl: 'https://test.bitrix24.com/rest/1/token/',
-        bot: { botId: 7, botToken: 'bot_token' },
-        dialogId: 'chat40985',
-      });
-
-      expect(result).toBeNull();
-    });
-
-    it('returns null when download fails', async () => {
-      vi.spyOn(globalThis, 'fetch').mockResolvedValueOnce(
-        new Response(null, { status: 404 }),
-      );
-
-      const result = await service.downloadMedia({
-        fileId: '789',
-        fileName: 'missing.txt',
-        extension: 'txt',
-        webhookUrl: 'https://test.bitrix24.com/rest/1/token/',
-        bot: { botId: 7, botToken: 'bot_token' },
-        dialogId: 'chat40985',
-      });
-
-      expect(result).toBeNull();
-    });
-
-    it('handles unknown extensions with octet-stream', async () => {
-      vi.spyOn(globalThis, 'fetch').mockResolvedValueOnce(
-        new Response(Buffer.from('data'), { status: 200 }),
-      );
-
-      const result = await service.downloadMedia({
-        fileId: '100',
-        fileName: 'file.xyz',
-        extension: 'xyz',
-        webhookUrl: 'https://test.bitrix24.com/rest/1/token/',
-        bot: { botId: 7, botToken: 'bot_token' },
-        dialogId: 'chat40985',
-      });
-
-      expect(result).not.toBeNull();
-      expect(result!.contentType).toBe('application/octet-stream');
-      if (result) savedPaths.push(result.path);
-    });
-
-    it('prefers response content type when extension is unknown', async () => {
-      vi.spyOn(globalThis, 'fetch').mockResolvedValueOnce(
-        new Response(Buffer.from([0xff, 0xd8, 0xff]), {
-          status: 200,
-          headers: { 'content-type': 'image/jpeg; charset=binary' },
-        }),
-      );
-
-      const result = await service.downloadMedia({
-        fileId: '101',
-        fileName: 'forwarded-file',
-        extension: '',
-        webhookUrl: 'https://test.bitrix24.com/rest/1/token/',
-        bot: { botId: 7, botToken: 'bot_token' },
-        dialogId: 'chat40985',
-      });
-
-      expect(result).not.toBeNull();
-      expect(result!.contentType).toBe('image/jpeg');
-      if (result) savedPaths.push(result.path);
-    });
-  });
-
-  describe('uploadMediaToChat', () => {
-    it('uploads file through imbot.v2.File.upload', async () => {
-      // Create a temp file to upload
-      const { writeFileSync, mkdirSync } = await import('node:fs');
-      const { join } = await import('node:path');
-      const { tmpdir } = await import('node:os');
-      const testDir = join(tmpdir(), 'openclaw-b24-test');
-      mkdirSync(testDir, { recursive: true });
-      const testPath = join(testDir, 'upload-test.txt');
-      writeFileSync(testPath, 'upload content');
-      savedPaths.push(testPath);
-
-      const result = await service.uploadMediaToChat({
-        localPath: testPath,
-        fileName: 'upload-test.txt',
-        webhookUrl: 'https://test.bitrix24.com/rest/1/token/',
-        bot: { botId: 7, botToken: 'bot_token' },
-        dialogId: 'chat40985',
-        message: 'hello',
-      });
-
-      expect(result).toEqual({ ok: true, messageId: 100 });
-      expect(api.uploadFile).toHaveBeenCalledWith(
-        'https://test.bitrix24.com/rest/1/token/',
-        { botId: 7, botToken: 'bot_token' },
-        'chat40985',
-        {
-          name: 'upload-test.txt',
-          content: Buffer.from('upload content').toString('base64'),
-          message: 'hello',
-        },
-      );
-    });
-
-    it('returns { ok: false } when upload fails', async () => {
-      (api.uploadFile as ReturnType<typeof vi.fn>).mockRejectedValueOnce(
-        new Error('upload error'),
-      );
-
-      const { writeFileSync, mkdirSync } = await import('node:fs');
-      const { join } = await import('node:path');
-      const { tmpdir } = await import('node:os');
-      const testDir = join(tmpdir(), 'openclaw-b24-test');
-      mkdirSync(testDir, { recursive: true });
-      const testPath = join(testDir, 'fail-upload.txt');
-      writeFileSync(testPath, 'content');
-      savedPaths.push(testPath);
-
-      const result = await service.uploadMediaToChat({
-        localPath: testPath,
-        fileName: 'fail-upload.txt',
-        webhookUrl: 'https://test.bitrix24.com/rest/1/token/',
-        bot: { botId: 7, botToken: 'bot_token' },
-        dialogId: 'chat40985',
-      });
-
-      expect(result).toEqual({ ok: false });
-      expect(silentLogger.error).toHaveBeenCalled();
-    });
-  });
-});

package/tests/message-utils.test.ts

@@ -1,184 +0,0 @@
-import { describe, it, expect } from 'vitest';
-import { markdownToBbCode, splitMessage, buildKeyboard } from '../src/message-utils.js';
-
-describe('markdownToBbCode', () => {
-  it('converts bold **text**', () => {
-    expect(markdownToBbCode('This is **bold** text')).toBe('This is [B]bold[/B] text');
-  });
-
-  it('converts bold __text__', () => {
-    expect(markdownToBbCode('This is __bold__ text')).toBe('This is [B]bold[/B] text');
-  });
-
-  it('converts italic *text*', () => {
-    expect(markdownToBbCode('This is *italic* text')).toBe('This is [I]italic[/I] text');
-  });
-
-  it('converts italic _text_', () => {
-    expect(markdownToBbCode('This is _italic_ text')).toBe('This is [I]italic[/I] text');
-  });
-
-  it('converts strikethrough ~~text~~', () => {
-    expect(markdownToBbCode('This is ~~deleted~~ text')).toBe('This is [S]deleted[/S] text');
-  });
-
-  it('converts inline code', () => {
-    expect(markdownToBbCode('Use `console.log`')).toBe('Use [CODE]console.log[/CODE]');
-  });
-
-  it('renders slash commands as bold burgundy inline accents', () => {
-    expect(markdownToBbCode('Try `/status`')).toBe('Try [B][COLOR=#7A1F3D]/status[/COLOR][/B]');
-  });
-
-  it('renders short acronyms as bold burgundy inline accents', () => {
-    expect(markdownToBbCode('Use `API` here')).toBe('Use [B][COLOR=#7A1F3D]API[/COLOR][/B] here');
-  });
-
-  it('converts code blocks', () => {
-    const md = '```js\nconst x = 1;\n```';
-    const expected = '[CODE]const x = 1;[/CODE]';
-    expect(markdownToBbCode(md)).toBe(expected);
-  });
-
-  it('converts links [text](url)', () => {
-    expect(markdownToBbCode('[Google](https://google.com)'))
-      .toBe('[URL=https://google.com]Google[/URL]');
-  });
-
-  it('converts markdown images to lowercase img tag without title text', () => {
-    expect(markdownToBbCode('![Заголовок изображения](https://via.placeholder.com/150 "Заголовок изображения")'))
-      .toBe('[img size=medium]https://via.placeholder.com/150#image.jpg [/img]');
-  });
-
-  it('keeps supported image extensions unchanged', () => {
-    expect(markdownToBbCode('![x](https://example.com/image.png)'))
-      .toBe('[img size=medium]https://example.com/image.png [/img]');
-  });
-
-  it('adds image.jpg suffix for unsupported image extensions', () => {
-    expect(markdownToBbCode('![x](https://mcgrp.ru/img/2017_05/img_1007484.avif)'))
-      .toBe('[img size=medium]https://mcgrp.ru/img/2017_05/img_1007484.avif#image.jpg [/img]');
-  });
-
-  it('converts blockquotes', () => {
-    expect(markdownToBbCode('> This is a quote')).toBe('>>This is a quote');
-  });
-
-  it('converts nested blockquotes to Bitrix-compatible quoted blocks', () => {
-    expect(markdownToBbCode([
-      '> цитата уровня 1',
-      '>> цитата уровня два',
-    ].join('\n'))).toBe([
-      '>>цитата уровня 1',
-      '>>------------------------------------------------------',
-      '>>цитата уровня два',
-      '>>------------------------------------------------------',
-    ].join('\n'));
-  });
-
-  it('skips empty spacer lines inside nested blockquotes', () => {
-    expect(markdownToBbCode([
-      '> Это первая цитата уровня 1.',
-      '>',
-      '>> Это вторая цитата уровня 2, вложенная в первую.',
-    ].join('\n'))).toBe([
-      '>>Это первая цитата уровня 1.',
-      '>>------------------------------------------------------',
-      '>>Это вторая цитата уровня 2, вложенная в первую.',
-      '>>------------------------------------------------------',
-    ].join('\n'));
-  });
-
-  it('converts GFM tables to simplified plain text tables', () => {
-    const input = [
-      '| Заголовок 1 | Заголовок 2 |',
-      '| ----------- | ----------- |',
-      '| Ячейка 1    | Ячейка 2    |',
-      '| Ячейка 3    | Ячейка 4    |',
-    ].join('\n');
-
-    expect(markdownToBbCode(input)).toBe([
-      'Заголовок 1 | Заголовок 2',
-      '_________________________',
-      'Ячейка 1    | Ячейка 2',
-      'Ячейка 3    | Ячейка 4',
-    ].join('\n'));
-  });
-
-  it('handles mixed formatting', () => {
-    const input = '**Bold** and *italic* with `code`';
-    const expected = '[B]Bold[/B] and [I]italic[/I] with [CODE]code[/CODE]';
-    expect(markdownToBbCode(input)).toBe(expected);
-  });
-
-  it('preserves plain text', () => {
-    expect(markdownToBbCode('Hello world')).toBe('Hello world');
-  });
-});
-
-describe('splitMessage', () => {
-  it('returns single chunk for short messages', () => {
-    expect(splitMessage('Hello')).toEqual(['Hello']);
-  });
-
-  it('splits long messages at newlines', () => {
-    const text = 'Line 1\nLine 2\nLine 3';
-    const chunks = splitMessage(text, 10);
-    expect(chunks.length).toBeGreaterThan(1);
-    expect(chunks.join('\n')).toBe(text);
-  });
-
-  it('splits at limit when no newline found', () => {
-    const text = 'A'.repeat(30);
-    const chunks = splitMessage(text, 10);
-    expect(chunks.length).toBe(3);
-    expect(chunks.join('')).toBe(text);
-  });
-
-  it('handles exactly maxLen text', () => {
-    const text = 'A'.repeat(100);
-    expect(splitMessage(text, 100)).toEqual([text]);
-  });
-});
-
-describe('buildKeyboard', () => {
-  it('builds a single row keyboard (flat array)', () => {
-    const kb = buildKeyboard([
-      [{ text: 'Yes', command: 'answer', commandParams: 'yes' }],
-    ]);
-    // Flat array: 1 button, no NEWLINE
-    expect(kb).toHaveLength(1);
-    expect(kb[0]).toMatchObject({ TEXT: 'Yes', COMMAND: 'answer', COMMAND_PARAMS: 'yes' });
-  });
-
-  it('builds multi-row keyboard with NEWLINE separators', () => {
-    const kb = buildKeyboard([
-      [
-        { text: 'Yes', command: 'answer', commandParams: 'yes' },
-        { text: 'No', command: 'answer', commandParams: 'no' },
-      ],
-      [{ text: 'More info', link: 'https://example.com', fullWidth: true }],
-    ]);
-    // Flat: [btn, btn, NEWLINE, btn] = 4 items
-    expect(kb).toHaveLength(4);
-    expect(kb[0]).toMatchObject({ TEXT: 'Yes' });
-    expect(kb[1]).toMatchObject({ TEXT: 'No' });
-    expect(kb[2]).toMatchObject({ TYPE: 'NEWLINE' });
-    expect(kb[3]).toMatchObject({ TEXT: 'More info', LINK: 'https://example.com', DISPLAY: 'LINE' });
-  });
-
-  it('applies default colors', () => {
-    const kb = buildKeyboard([[{ text: 'Click' }]]);
-    expect(kb[0]).toMatchObject({ BG_COLOR: '#29619b', TEXT_COLOR: '#fff' });
-  });
-
-  it('applies custom colors', () => {
-    const kb = buildKeyboard([[{ text: 'Click', bgColor: '#333', textColor: '#eee' }]]);
-    expect(kb[0]).toMatchObject({ BG_COLOR: '#333', TEXT_COLOR: '#eee' });
-  });
-
-  it('sets BLOCK=Y when disableAfterClick is true', () => {
-    const kb = buildKeyboard([[{ text: 'Once', command: 'once', disableAfterClick: true }]]);
-    expect(kb[0]).toMatchObject({ BLOCK: 'Y' });
-  });
-});