@ihazz/bitrix24 1.1.1 → 1.1.2

package/tests/channel.test.ts

@@ -1,842 +0,0 @@
-import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest';
-import { Readable, PassThrough } from 'node:stream';
-import type { IncomingMessage, ServerResponse } from 'node:http';
-import {
-  __setGatewayStateForTests,
-  buildBotCodeCandidates,
-  buildConversationSessionKey,
-  canCoalesceDirectMessage,
-  convertButtonsToKeyboard,
-  extractKeyboardFromPayload,
-  handleWebhookRequest,
-  mergeBufferedDirectMessages,
-  mergeForwardedMessageContext,
-  resolveConversationRef,
-  resolveDirectMessageCoalesceDelay,
-  shouldSkipJoinChatWelcome,
-  bitrix24Plugin,
-} from '../src/channel.js';
-import type { KeyboardButton, KeyboardNewline, B24Keyboard } from '../src/types.js';
-
-// ─── Helpers ─────────────────────────────────────────────────────────────────
-
-const silentLogger = {
-  info: vi.fn(),
-  warn: vi.fn(),
-  error: vi.fn(),
-  debug: vi.fn(),
-};
-
-function isButton(item: KeyboardButton | KeyboardNewline): item is KeyboardButton {
-  return !('TYPE' in item);
-}
-
-function isNewline(item: KeyboardButton | KeyboardNewline): item is KeyboardNewline {
-  return 'TYPE' in item && item.TYPE === 'NEWLINE';
-}
-
-describe('buildBotCodeCandidates', () => {
-  it('uses explicit botCode when it is configured', () => {
-    expect(buildBotCodeCandidates({
-      webhookUrl: 'https://test.bitrix24.com/rest/42/token/',
-      botCode: 'custom_bot',
-    }, 5)).toEqual(['custom_bot']);
-  });
-
-  it('builds automatic codes from webhook user id', () => {
-    expect(buildBotCodeCandidates({
-      webhookUrl: 'https://test.bitrix24.com/rest/42/token/',
-    }, 4)).toEqual([
-      'openclaw_42',
-      'openclaw_42_2',
-      'openclaw_42_3',
-      'openclaw_42_4',
-    ]);
-  });
-});
-
-// ─── convertButtonsToKeyboard ────────────────────────────────────────────────
-
-describe('convertButtonsToKeyboard', () => {
-  it('converts a single button with slash command', () => {
-    const kb = convertButtonsToKeyboard([
-      [{ text: 'Help', callback_data: '/help' }],
-    ]);
-
-    expect(kb).toHaveLength(1);
-    const btn = kb[0];
-    expect(isButton(btn)).toBe(true);
-    if (isButton(btn)) {
-      expect(btn.TEXT).toBe('Help');
-      expect(btn.COMMAND).toBe('help');
-      expect(btn.COMMAND_PARAMS).toBeUndefined();
-      expect(btn.DISPLAY).toBe('LINE');
-    }
-  });
-
-  it('converts a registered command without leading slash', () => {
-    const kb = convertButtonsToKeyboard([
-      [{ text: 'Help', callback_data: 'help' }],
-    ]);
-
-    const btn = kb[0];
-    expect(isButton(btn)).toBe(true);
-    if (isButton(btn)) {
-      expect(btn.COMMAND).toBe('help');
-      expect(btn.COMMAND_PARAMS).toBeUndefined();
-    }
-  });
-
-  it('converts slash command with params', () => {
-    const kb = convertButtonsToKeyboard([
-      [{ text: 'Set Model', callback_data: '/model gpt-4o' }],
-    ]);
-
-    expect(kb).toHaveLength(1);
-    const btn = kb[0];
-    if (isButton(btn)) {
-      expect(btn.COMMAND).toBe('model');
-      expect(btn.COMMAND_PARAMS).toBe('gpt-4o');
-    }
-  });
-
-  it('converts a registered command with params without leading slash', () => {
-    const kb = convertButtonsToKeyboard([
-      [{ text: 'Set Model', callback_data: 'model gpt-4o' }],
-    ]);
-
-    const btn = kb[0];
-    if (isButton(btn)) {
-      expect(btn.COMMAND).toBe('model');
-      expect(btn.COMMAND_PARAMS).toBe('gpt-4o');
-    }
-  });
-
-  it('converts slash command with multiple params', () => {
-    const kb = convertButtonsToKeyboard([
-      [{ text: 'Think', callback_data: '/think high extra' }],
-    ]);
-
-    const btn = kb[0];
-    if (isButton(btn)) {
-      expect(btn.COMMAND).toBe('think');
-      expect(btn.COMMAND_PARAMS).toBe('high extra');
-    }
-  });
-
-  it('converts non-slash callback_data to ACTION SEND with text as value', () => {
-    const kb = convertButtonsToKeyboard([
-      [{ text: 'Click me', callback_data: 'some_value' }],
-    ]);
-
-    const btn = kb[0];
-    if (isButton(btn)) {
-      expect(btn.COMMAND).toBeUndefined();
-      expect(btn.ACTION).toBe('SEND');
-      expect(btn.ACTION_VALUE).toBe('Click me');
-    }
-  });
-
-  it('handles button without callback_data', () => {
-    const kb = convertButtonsToKeyboard([
-      [{ text: 'Plain' }],
-    ]);
-
-    const btn = kb[0];
-    if (isButton(btn)) {
-      expect(btn.TEXT).toBe('Plain');
-      expect(btn.COMMAND).toBeUndefined();
-      expect(btn.ACTION).toBeUndefined();
-    }
-  });
-
-  it('maps primary style to BG_COLOR_TOKEN', () => {
-    const kb = convertButtonsToKeyboard([
-      [{ text: 'Go', callback_data: '/start', style: 'primary' }],
-    ]);
-
-    const btn = kb[0];
-    if (isButton(btn)) {
-      expect(btn.BG_COLOR_TOKEN).toBe('primary');
-    }
-  });
-
-  it('maps attention style to alert BG_COLOR_TOKEN', () => {
-    const kb = convertButtonsToKeyboard([
-      [{ text: 'Stop', callback_data: '/stop', style: 'attention' }],
-    ]);
-
-    const btn = kb[0];
-    if (isButton(btn)) {
-      expect(btn.BG_COLOR_TOKEN).toBe('alert');
-    }
-  });
-
-  it('maps danger style to alert BG_COLOR_TOKEN', () => {
-    const kb = convertButtonsToKeyboard([
-      [{ text: 'Delete', callback_data: '/reset', style: 'danger' }],
-    ]);
-
-    const btn = kb[0];
-    if (isButton(btn)) {
-      expect(btn.BG_COLOR_TOKEN).toBe('alert');
-    }
-  });
-
-  it('does not set BG_COLOR_TOKEN for unknown style', () => {
-    const kb = convertButtonsToKeyboard([
-      [{ text: 'Btn', callback_data: '/ok', style: 'custom' }],
-    ]);
-
-    const btn = kb[0];
-    if (isButton(btn)) {
-      expect(btn.BG_COLOR_TOKEN).toBeUndefined();
-    }
-  });
-
-  it('adds NEWLINE between rows', () => {
-    const kb = convertButtonsToKeyboard([
-      [{ text: 'A', callback_data: '/a' }],
-      [{ text: 'B', callback_data: '/b' }],
-    ]);
-
-    // [btn, NEWLINE, btn] = 3 items
-    expect(kb).toHaveLength(3);
-    expect(isButton(kb[0])).toBe(true);
-    expect(isNewline(kb[1])).toBe(true);
-    expect(isButton(kb[2])).toBe(true);
-  });
-
-  it('handles multiple buttons per row', () => {
-    const kb = convertButtonsToKeyboard([
-      [
-        { text: 'Yes', callback_data: '/yes' },
-        { text: 'No', callback_data: '/no' },
-      ],
-    ]);
-
-    // Single row with 2 buttons, no NEWLINE
-    expect(kb).toHaveLength(2);
-    expect(isButton(kb[0])).toBe(true);
-    expect(isButton(kb[1])).toBe(true);
-    if (isButton(kb[0])) expect(kb[0].COMMAND).toBe('yes');
-    if (isButton(kb[1])) expect(kb[1].COMMAND).toBe('no');
-  });
-
-  it('handles multiple rows with multiple buttons', () => {
-    const kb = convertButtonsToKeyboard([
-      [{ text: 'A' }, { text: 'B' }],
-      [{ text: 'C' }],
-      [{ text: 'D' }, { text: 'E' }, { text: 'F' }],
-    ]);
-
-    // Row1(2) + NL + Row2(1) + NL + Row3(3) = 2+1+1+1+3 = 8
-    expect(kb).toHaveLength(8);
-    expect(isNewline(kb[2])).toBe(true);
-    expect(isNewline(kb[4])).toBe(true);
-  });
-
-  it('handles empty rows array', () => {
-    const kb = convertButtonsToKeyboard([]);
-    expect(kb).toHaveLength(0);
-  });
-
-  it('handles empty row inside rows', () => {
-    const kb = convertButtonsToKeyboard([[]]);
-    expect(kb).toHaveLength(0);
-  });
-});
-
-// ─── extractKeyboardFromPayload ──────────────────────────────────────────────
-
-describe('extractKeyboardFromPayload', () => {
-  it('returns undefined when no channelData', () => {
-    expect(extractKeyboardFromPayload({})).toBeUndefined();
-  });
-
-  it('returns undefined when channelData is empty', () => {
-    expect(extractKeyboardFromPayload({ channelData: {} })).toBeUndefined();
-  });
-
-  it('returns undefined when channelData buttons is empty array', () => {
-    const result = extractKeyboardFromPayload({
-      channelData: { telegram: { buttons: [] } },
-    });
-    expect(result).toBeUndefined();
-  });
-
-  it('extracts direct B24 keyboard from channelData.bitrix24', () => {
-    const directKeyboard: B24Keyboard = [
-      { TEXT: 'Direct', COMMAND: 'test' },
-    ];
-
-    const result = extractKeyboardFromPayload({
-      channelData: { bitrix24: { keyboard: directKeyboard } },
-    });
-
-    expect(result).toEqual(directKeyboard);
-  });
-
-  it('prefers B24 keyboard over generic channel buttons', () => {
-    const directKeyboard: B24Keyboard = [{ TEXT: 'B24' }];
-
-    const result = extractKeyboardFromPayload({
-      channelData: {
-        bitrix24: { keyboard: directKeyboard },
-        telegram: { buttons: [[{ text: 'TG', callback_data: '/tg' }]] },
-      },
-    });
-
-    expect(result).toEqual(directKeyboard);
-  });
-
-  it('converts channel buttons to B24 keyboard', () => {
-    const result = extractKeyboardFromPayload({
-      channelData: {
-        telegram: {
-          buttons: [[{ text: 'Help', callback_data: '/help' }]],
-        },
-      },
-    });
-
-    expect(result).toBeDefined();
-    expect(result).toHaveLength(1);
-    if (result && isButton(result[0])) {
-      expect(result[0].TEXT).toBe('Help');
-      expect(result[0].COMMAND).toBe('help');
-    }
-  });
-
-  it('converts multi-row channel buttons', () => {
-    const result = extractKeyboardFromPayload({
-      channelData: {
-        telegram: {
-          buttons: [
-            [{ text: 'A', callback_data: '/a' }],
-            [{ text: 'B', callback_data: '/b' }],
-          ],
-        },
-      },
-    });
-
-    // [btn, NEWLINE, btn]
-    expect(result).toHaveLength(3);
-  });
-
-  it('returns undefined when bitrix24.keyboard is empty', () => {
-    const result = extractKeyboardFromPayload({
-      channelData: { bitrix24: { keyboard: [] } },
-    });
-    expect(result).toBeUndefined();
-  });
-
-  it('ignores unrelated channelData keys', () => {
-    const result = extractKeyboardFromPayload({
-      channelData: { discord: { something: true } },
-    });
-    expect(result).toBeUndefined();
-  });
-});
-
-describe('direct message coalescing helpers', () => {
-  const baseMsgCtx = {
-    channel: 'bitrix24' as const,
-    senderId: '1',
-    senderName: 'Test User',
-    senderFirstName: 'Test',
-    chatId: '1',
-    chatInternalId: '40985',
-    messageId: '100',
-    text: 'hello',
-    isDm: true,
-    isGroup: false,
-    media: [],
-    language: 'ru',
-    raw: { type: 'ONIMBOTV2MESSAGEADD' },
-    botId: 295,
-    memberId: '',
-  };
-
-  it('coalesces only plain direct text messages', () => {
-    expect(canCoalesceDirectMessage(baseMsgCtx, { dmPolicy: 'webhookUser' })).toBe(true);
-    expect(canCoalesceDirectMessage({ ...baseMsgCtx, text: '/status' }, { dmPolicy: 'webhookUser' })).toBe(false);
-    expect(canCoalesceDirectMessage({ ...baseMsgCtx, media: [{ id: '1', name: 'x', extension: '', size: 1, type: 'file' }] }, { dmPolicy: 'webhookUser' })).toBe(false);
-    expect(canCoalesceDirectMessage({ ...baseMsgCtx, replyToMessageId: '77' }, { dmPolicy: 'webhookUser' })).toBe(false);
-    expect(canCoalesceDirectMessage({ ...baseMsgCtx, isForwarded: true }, { dmPolicy: 'webhookUser' })).toBe(false);
-    expect(canCoalesceDirectMessage({ ...baseMsgCtx, isDm: false, isGroup: true }, { dmPolicy: 'webhookUser' })).toBe(false);
-    expect(canCoalesceDirectMessage(baseMsgCtx, { dmPolicy: 'pairing' })).toBe(false);
-  });
-
-  it('merges buffered direct messages with newline separators and last message id', () => {
-    const merged = mergeBufferedDirectMessages([
-      baseMsgCtx,
-      { ...baseMsgCtx, messageId: '101', text: 'second' },
-      { ...baseMsgCtx, messageId: '102', text: 'third', language: 'en' },
-    ]);
-
-    expect(merged.text).toBe('hello\nsecond\nthird');
-    expect(merged.messageId).toBe('102');
-    expect(merged.language).toBe('en');
-  });
-
-  it('merges a forwarded message with buffered question context', () => {
-    const merged = mergeForwardedMessageContext(
-      baseMsgCtx,
-      {
-        ...baseMsgCtx,
-        messageId: '103',
-        text: 'Forwarded body',
-        replyToMessageId: '77',
-        media: [{ id: '1', name: 'photo.jpg', extension: '', size: 42, type: 'file' }],
-        isForwarded: true,
-      },
-    );
-
-    expect(merged.text).toBe('[User question about the forwarded message]\n\nhello\n\n[/User question]\n\nForwarded body');
-    expect(merged.messageId).toBe('103');
-    expect(merged.media).toHaveLength(1);
-    expect(merged.replyToMessageId).toBeUndefined();
-    expect(merged.isForwarded).toBe(false);
-  });
-
-  it('uses short debounce and extends the window until max wait', () => {
-    expect(resolveDirectMessageCoalesceDelay({
-      startedAt: 1000,
-      now: 1000,
-    })).toBe(200);
-
-    expect(resolveDirectMessageCoalesceDelay({
-      startedAt: 1000,
-      now: 1150,
-    })).toBe(200);
-
-    expect(resolveDirectMessageCoalesceDelay({
-      startedAt: 1000,
-      now: 5900,
-    })).toBe(100);
-
-    expect(resolveDirectMessageCoalesceDelay({
-      startedAt: 1000,
-      now: 6200,
-    })).toBe(0);
-  });
-});
-
-describe('conversation helpers', () => {
-  it('uses direct dialog id as the stable conversation identity', () => {
-    const conversation = resolveConversationRef({
-      accountId: 'default',
-      dialogId: '2386',
-      isDirect: true,
-    });
-
-    expect(conversation.dialogId).toBe('2386');
-    expect(conversation.address).toBe('bitrix24:2386');
-    expect(conversation.historyKey).toBe('default:2386');
-    expect(conversation.peer).toEqual({ kind: 'direct', id: '2386' });
-  });
-
-  it('uses group dialog id as the stable conversation identity', () => {
-    const conversation = resolveConversationRef({
-      accountId: 'default',
-      dialogId: 'chat520',
-      isDirect: false,
-    });
-
-    expect(conversation.dialogId).toBe('chat520');
-    expect(conversation.address).toBe('bitrix24:chat520');
-    expect(conversation.historyKey).toBe('default:chat520');
-    expect(conversation.peer).toEqual({ kind: 'group', id: 'chat520' });
-  });
-
-  it('builds session keys from the stable conversation address', () => {
-    const conversation = resolveConversationRef({
-      accountId: 'default',
-      dialogId: '2386',
-      isDirect: true,
-    });
-
-    expect(buildConversationSessionKey('route:session', conversation))
-      .toBe('route:session:bitrix24:2386');
-  });
-});
-
-describe('join chat helpers', () => {
-  it('does not skip unsupported group chats even in webhookUser mode', () => {
-    expect(shouldSkipJoinChatWelcome({
-      dialogId: 'chat123',
-      chatType: 'chat',
-      webhookUrl: 'https://test.bitrix24.com/rest/1/token/',
-      dmPolicy: 'webhookUser',
-    })).toBe(false);
-  });
-
-  it('skips welcome for non-owner direct dialogs in webhookUser mode', () => {
-    expect(shouldSkipJoinChatWelcome({
-      dialogId: '42',
-      chatType: 'user',
-      webhookUrl: 'https://test.bitrix24.com/rest/1/token/',
-      dmPolicy: 'webhookUser',
-    })).toBe(true);
-  });
-
-  it('does not skip welcome for owner direct dialogs in webhookUser mode', () => {
-    expect(shouldSkipJoinChatWelcome({
-      dialogId: '1',
-      chatType: 'user',
-      webhookUrl: 'https://test.bitrix24.com/rest/1/token/',
-      dmPolicy: 'webhookUser',
-    })).toBe(false);
-  });
-});
-
-// ─── handleWebhookRequest ────────────────────────────────────────────────────
-
-describe('handleWebhookRequest', () => {
-  afterEach(() => {
-    __setGatewayStateForTests(null);
-  });
-
-  function mockReq(method: string, body?: string): IncomingMessage {
-    const stream = new PassThrough();
-    if (body) {
-      stream.end(body);
-    } else {
-      stream.end();
-    }
-    (stream as unknown as Record<string, unknown>).method = method;
-    return stream as unknown as IncomingMessage;
-  }
-
-  function mockRes(): ServerResponse & { _statusCode: number; _body: string; _headers: Record<string, string> } {
-    const res = {
-      _statusCode: 200,
-      _body: '',
-      _headers: {} as Record<string, string>,
-      get statusCode() { return this._statusCode; },
-      set statusCode(v: number) { this._statusCode = v; },
-      setHeader(key: string, val: string) { this._headers[key] = val; },
-      end(body?: string) { this._body = body ?? ''; },
-    };
-    return res as unknown as ServerResponse & { _statusCode: number; _body: string; _headers: Record<string, string> };
-  }
-
-  it('rejects non-POST with 405', async () => {
-    const req = mockReq('GET');
-    const res = mockRes();
-
-    await handleWebhookRequest(req, res);
-
-    expect(res._statusCode).toBe(405);
-    expect(res._body).toBe('Method Not Allowed');
-  });
-
-  it('returns 503 when gateway not started', async () => {
-    const req = mockReq('POST', 'event=test');
-    const res = mockRes();
-
-    await handleWebhookRequest(req, res);
-
-    expect(res._statusCode).toBe(503);
-    expect(res._body).toBe('Channel not started');
-  });
-
-  it('returns fetch mode acknowledgement when gateway runs in fetch mode', async () => {
-    __setGatewayStateForTests({
-      eventMode: 'fetch',
-      inboundHandler: { handleWebhook: vi.fn() },
-    } as never);
-
-    const req = mockReq('POST', '{"event":"test"}');
-    const res = mockRes();
-
-    await handleWebhookRequest(req, res);
-
-    expect(res._statusCode).toBe(200);
-    expect(res._body).toBe('FETCH mode active');
-  });
-
-  it('returns 400 when webhook payload is invalid', async () => {
-    const handleWebhook = vi.fn().mockResolvedValue(false);
-    __setGatewayStateForTests({
-      eventMode: 'webhook',
-      inboundHandler: { handleWebhook },
-    } as never);
-
-    const req = mockReq('POST', 'not-json');
-    const res = mockRes();
-
-    await handleWebhookRequest(req, res);
-
-    expect(handleWebhook).toHaveBeenCalledWith('not-json');
-    expect(res._statusCode).toBe(400);
-    expect(res._body).toBe('Invalid webhook payload');
-  });
-
-  it('returns 500 when webhook processing throws', async () => {
-    const handleWebhook = vi.fn().mockRejectedValue(new Error('boom'));
-    __setGatewayStateForTests({
-      eventMode: 'webhook',
-      inboundHandler: { handleWebhook },
-    } as never);
-
-    const req = mockReq('POST', '{"event":"test"}');
-    const res = mockRes();
-
-    await handleWebhookRequest(req, res);
-
-    expect(res._statusCode).toBe(500);
-    expect(res._body).toBe('Webhook processing failed');
-  });
-
-  it('rejects oversized webhook bodies with 413', async () => {
-    const handleWebhook = vi.fn();
-    __setGatewayStateForTests({
-      eventMode: 'webhook',
-      inboundHandler: { handleWebhook },
-    } as never);
-
-    const req = mockReq('POST', 'x'.repeat((1024 * 1024) + 1));
-    const res = mockRes();
-
-    await handleWebhookRequest(req, res);
-
-    expect(handleWebhook).not.toHaveBeenCalled();
-    expect(res._statusCode).toBe(413);
-    expect(res._body).toBe('Payload Too Large');
-  });
-});
-
-// ─── bitrix24Plugin structure ────────────────────────────────────────────────
-
-describe('bitrix24Plugin', () => {
-  describe('metadata', () => {
-    it('has correct id', () => {
-      expect(bitrix24Plugin.id).toBe('bitrix24');
-    });
-
-    it('has correct meta', () => {
-      expect(bitrix24Plugin.meta.id).toBe('bitrix24');
-      expect(bitrix24Plugin.meta.label).toBe('Bitrix24');
-      expect(bitrix24Plugin.meta.aliases).toEqual(['b24', 'bx24']);
-    });
-
-    it('declares capabilities', () => {
-      expect(bitrix24Plugin.capabilities.chatTypes).toEqual(['direct', 'group']);
-      expect(bitrix24Plugin.capabilities.media).toBe(true);
-      expect(bitrix24Plugin.capabilities.reactions).toBe(true);
-      expect(bitrix24Plugin.capabilities.threads).toBe(false);
-      expect(bitrix24Plugin.capabilities.nativeCommands).toBe(true);
-    });
-  });
-
-  describe('security', () => {
-    it('resolveDmPolicy returns webhookUser policy when configured', () => {
-      const result = bitrix24Plugin.security.resolveDmPolicy({
-        account: { config: { dmPolicy: 'webhookUser', allowFrom: ['bitrix24:42', '42', 'b24:7'] } },
-      });
-      expect(result.policy).toBe('webhookUser');
-      expect(result.allowFrom).toEqual(['42', '7']);
-      expect(result.policyPath).toBe('channels.bitrix24.dmPolicy');
-      expect(result.allowFromPath).toBe('channels.bitrix24.allowFrom');
-      expect(result.approveHint).toContain('openclaw pairing approve bitrix24');
-    });
-
-    it('resolveDmPolicy returns allowlist policy when configured', () => {
-      const result = bitrix24Plugin.security.resolveDmPolicy({
-        account: { config: { dmPolicy: 'allowlist', allowFrom: ['bitrix24:42'] } },
-      });
-
-      expect(result.policy).toBe('allowlist');
-      expect(result.allowFrom).toEqual(['42']);
-    });
-
-    it('resolveDmPolicy returns open policy when configured', () => {
-      const result = bitrix24Plugin.security.resolveDmPolicy({
-        account: { config: { dmPolicy: 'open' } },
-      });
-
-      expect(result.policy).toBe('open');
-      expect(result.allowFrom).toEqual([]);
-    });
-
-    it('resolveDmPolicy defaults to webhookUser', () => {
-      const result = bitrix24Plugin.security.resolveDmPolicy({});
-      expect(result.policy).toBe('webhookUser');
-      expect(result.allowFrom).toEqual([]);
-    });
-
-    it('resolveDmPolicy reads allowFrom from cfg when account is omitted', () => {
-      const result = bitrix24Plugin.security.resolveDmPolicy({
-        cfg: {
-          channels: {
-            bitrix24: {
-              allowFrom: ['bx24:100', '100', 'bitrix24:42'],
-            },
-          },
-        },
-      });
-
-      expect(result.allowFrom).toEqual(['100', '42']);
-    });
-
-    it('resolveDmPolicy normalizeEntry strips prefixes', () => {
-      const result = bitrix24Plugin.security.resolveDmPolicy({});
-      expect(result.normalizeEntry('bitrix24:42')).toBe('42');
-      expect(result.normalizeEntry('b24:42')).toBe('42');
-      expect(result.normalizeEntry('BX24:7')).toBe('7');
-      expect(result.normalizeEntry('42')).toBe('42');
-    });
-
-    it('normalizeAllowFrom strips bitrix24: prefix', () => {
-      expect(bitrix24Plugin.security.normalizeAllowFrom('bitrix24:42')).toBe('42');
-    });
-
-    it('normalizeAllowFrom strips b24: prefix', () => {
-      expect(bitrix24Plugin.security.normalizeAllowFrom('b24:42')).toBe('42');
-    });
-
-    it('normalizeAllowFrom strips bx24: prefix', () => {
-      expect(bitrix24Plugin.security.normalizeAllowFrom('bx24:42')).toBe('42');
-    });
-
-    it('normalizeAllowFrom keeps plain id', () => {
-      expect(bitrix24Plugin.security.normalizeAllowFrom('42')).toBe('42');
-    });
-  });
-
-  describe('pairing adapter', () => {
-    it('has correct idLabel', () => {
-      expect(bitrix24Plugin.pairing.idLabel).toBe('bitrix24UserId');
-    });
-
-    it('normalizeAllowEntry strips prefixes', () => {
-      expect(bitrix24Plugin.pairing.normalizeAllowEntry!('bitrix24:42')).toBe('42');
-      expect(bitrix24Plugin.pairing.normalizeAllowEntry!('b24:42')).toBe('42');
-      expect(bitrix24Plugin.pairing.normalizeAllowEntry!('BX24:7')).toBe('7');
-      expect(bitrix24Plugin.pairing.normalizeAllowEntry!('42')).toBe('42');
-    });
-
-    it('has notifyApproval function', () => {
-      expect(typeof bitrix24Plugin.pairing.notifyApproval).toBe('function');
-    });
-  });
-
-  describe('messaging', () => {
-    it('normalizeTarget strips bitrix24: prefix', () => {
-      expect(bitrix24Plugin.messaging.normalizeTarget('bitrix24:1')).toBe('1');
-      expect(bitrix24Plugin.messaging.normalizeTarget('b24:42')).toBe('42');
-      expect(bitrix24Plugin.messaging.normalizeTarget('bx24:100')).toBe('100');
-      expect(bitrix24Plugin.messaging.normalizeTarget('123')).toBe('123');
-      expect(bitrix24Plugin.messaging.normalizeTarget('  bitrix24:5  ')).toBe('5');
-    });
-
-    it('looksLikeId recognizes numeric IDs', () => {
-      expect(bitrix24Plugin.messaging.targetResolver.looksLikeId('bitrix24:1', '1')).toBe(true);
-      expect(bitrix24Plugin.messaging.targetResolver.looksLikeId('42', '42')).toBe(true);
-      expect(bitrix24Plugin.messaging.targetResolver.looksLikeId('abc', 'abc')).toBe(false);
-    });
-  });
-
-  describe('outbound', () => {
-    it('has direct delivery mode', () => {
-      expect(bitrix24Plugin.outbound.deliveryMode).toBe('direct');
-    });
-
-    it('has textChunkLimit of 4000', () => {
-      expect(bitrix24Plugin.outbound.textChunkLimit).toBe(4000);
-    });
-
-    it('sendText throws when gateway not started', async () => {
-      await expect(
-        bitrix24Plugin.outbound.sendText({
-          cfg: { channels: { bitrix24: { webhookUrl: 'https://test.bitrix24.com/rest/1/token/' } } },
-          to: '1',
-          accountId: 'default',
-          text: 'Hello',
-        }),
-      ).rejects.toThrow('Bitrix24 gateway not started');
-    });
-
-    it('sendMedia throws when gateway not started', async () => {
-      await expect(
-        bitrix24Plugin.outbound.sendMedia({
-          cfg: { channels: { bitrix24: { webhookUrl: 'https://test.bitrix24.com/rest/1/token/' } } },
-          to: '1',
-          accountId: 'default',
-          text: 'Hello',
-          mediaUrl: '/tmp/test.jpg',
-        }),
-      ).rejects.toThrow('Bitrix24 gateway not started');
-    });
-
-    it('sendPayload throws when gateway not started', async () => {
-      await expect(
-        bitrix24Plugin.outbound.sendPayload({
-          cfg: { channels: { bitrix24: { webhookUrl: 'https://test.bitrix24.com/rest/1/token/' } } },
-          to: '1',
-          accountId: 'default',
-          text: 'Hello',
-          payload: { channelData: { telegram: { buttons: [[{ text: 'OK' }]] } } },
-        }),
-      ).rejects.toThrow('Bitrix24 gateway not started');
-    });
-  });
-
-  describe('config', () => {
-    it('listAccountIds returns empty for empty config', () => {
-      expect(bitrix24Plugin.config.listAccountIds({})).toEqual([]);
-    });
-
-    it('listAccountIds returns default when webhookUrl present', () => {
-      const cfg = {
-        channels: {
-          bitrix24: {
-            webhookUrl: 'https://test.bitrix24.com/rest/1/token/',
-          },
-        },
-      };
-      expect(bitrix24Plugin.config.listAccountIds(cfg)).toEqual(['default']);
-    });
-
-    it('resolveAccount returns config', () => {
-      const cfg = {
-        channels: {
-          bitrix24: {
-            webhookUrl: 'https://test.bitrix24.com/rest/1/token/',
-            botName: 'TestBot',
-          },
-        },
-      };
-      const account = bitrix24Plugin.config.resolveAccount(cfg);
-      expect(account.accountId).toBe('default');
-      expect(account.config.webhookUrl).toBe('https://test.bitrix24.com/rest/1/token/');
-      expect(account.config.botName).toBe('TestBot');
-      expect(account.configured).toBe(true);
-      expect(account.enabled).toBe(true);
-    });
-  });
-});
-
-// ─── gateway.startAccount ────────────────────────────────────────────────────
-
-describe('gateway.startAccount', () => {
-  it('skips when webhookUrl not configured', async () => {
-    const logger = { ...silentLogger };
-
-    const result = await bitrix24Plugin.gateway.startAccount({
-      cfg: { channels: { bitrix24: {} } },
-      accountId: 'test',
-      account: { config: {} },
-      runtime: {},
-      abortSignal: new AbortController().signal,
-      log: logger,
-    });
-
-    // Should return undefined (early return, no promise)
-    expect(result).toBeUndefined();
-    expect(logger.warn).toHaveBeenCalledWith(
-      expect.stringContaining('no webhookUrl configured'),
-    );
-  });
-});