@ihazz/bitrix24 1.1.1 → 1.1.2

package/tests/polling-service.test.ts

@@ -1,115 +0,0 @@
-import { mkdtempSync, readFileSync, rmSync } from 'node:fs';
-import { join } from 'node:path';
-import { tmpdir } from 'node:os';
-import { afterEach, describe, expect, it, vi } from 'vitest';
-import { PollingService } from '../src/polling-service.js';
-import type { Bitrix24Api } from '../src/api.js';
-
-const silentLogger = {
-  info: vi.fn(),
-  warn: vi.fn(),
-  error: vi.fn(),
-  debug: vi.fn(),
-};
-
-describe('PollingService', () => {
-  let tempHome = '';
-  const originalHome = process.env.HOME;
-
-  afterEach(() => {
-    if (tempHome) {
-      rmSync(tempHome, { recursive: true, force: true });
-      tempHome = '';
-    }
-
-    if (originalHome === undefined) {
-      delete process.env.HOME;
-    } else {
-      process.env.HOME = originalHome;
-    }
-
-    vi.clearAllMocks();
-  });
-
-  it('persists offset only up to the last successfully processed event', async () => {
-    tempHome = mkdtempSync(join(tmpdir(), 'b24-polling-'));
-    process.env.HOME = tempHome;
-
-    const abortController = new AbortController();
-    const fetchEvents = vi.fn().mockResolvedValue({
-      events: [
-        { eventId: 10, type: 'ONE', date: '', data: {} },
-        { eventId: 11, type: 'TWO', date: '', data: {} },
-      ],
-      lastEventId: 11,
-      hasMore: false,
-    });
-
-    const api = { fetchEvents } as unknown as Bitrix24Api;
-    const onEvent = vi.fn(async (event: { eventId: number }) => {
-      if (event.eventId === 11) {
-        abortController.abort();
-        throw new Error('boom');
-      }
-    });
-
-    const service = new PollingService({
-      api,
-      webhookUrl: 'https://test.bitrix24.com/rest/1/token/',
-      bot: { botId: 7, botToken: 'bot_token' },
-      accountId: 'default',
-      pollingIntervalMs: 10,
-      pollingFastIntervalMs: 10,
-      onEvent: onEvent as never,
-      abortSignal: abortController.signal,
-      logger: silentLogger,
-    });
-
-    await service.start();
-
-    const statePath = join(tempHome, '.openclaw', 'state', 'bitrix24', 'poll-offset-default.json');
-    const state = JSON.parse(readFileSync(statePath, 'utf-8')) as { offset: number };
-
-    expect(fetchEvents).toHaveBeenCalledOnce();
-    expect(onEvent).toHaveBeenCalledTimes(2);
-    expect(state.offset).toBe(11);
-  });
-
-  it('passes withUserEvents through to fetch polling', async () => {
-    tempHome = mkdtempSync(join(tmpdir(), 'b24-polling-agent-'));
-    process.env.HOME = tempHome;
-
-    const abortController = new AbortController();
-    const fetchEvents = vi.fn().mockImplementation(async () => {
-      abortController.abort();
-      return {
-        events: [],
-        lastEventId: 0,
-        hasMore: false,
-      };
-    });
-
-    const api = { fetchEvents } as unknown as Bitrix24Api;
-    const service = new PollingService({
-      api,
-      webhookUrl: 'https://test.bitrix24.com/rest/1/token/',
-      bot: { botId: 7, botToken: 'bot_token' },
-      accountId: 'default',
-      pollingIntervalMs: 1,
-      pollingFastIntervalMs: 1,
-      withUserEvents: true,
-      onEvent: vi.fn(),
-      abortSignal: abortController.signal,
-      logger: silentLogger,
-    });
-
-    await service.start();
-
-    expect(fetchEvents).toHaveBeenCalledOnce();
-    expect(fetchEvents).toHaveBeenCalledWith(
-      'https://test.bitrix24.com/rest/1/token/',
-      { botId: 7, botToken: 'bot_token' },
-      { offset: 0, limit: 100, withUserEvents: true },
-    );
-  });
-});

package/tests/rate-limiter.test.ts

@@ -1,52 +0,0 @@
-import { describe, it, expect, afterEach } from 'vitest';
-import { RateLimiter } from '../src/rate-limiter.js';
-
-describe('RateLimiter', () => {
-  let limiter: RateLimiter;
-
-  afterEach(() => {
-    limiter?.destroy();
-  });
-
-  it('allows immediate requests within budget', async () => {
-    limiter = new RateLimiter({ maxPerSecond: 2 });
-    // First two should be immediate
-    const start = Date.now();
-    await limiter.acquire();
-    await limiter.acquire();
-    const elapsed = Date.now() - start;
-    expect(elapsed).toBeLessThan(50);
-  });
-
-  it('queues requests when budget exhausted', async () => {
-    limiter = new RateLimiter({ maxPerSecond: 2 });
-    // Exhaust budget
-    await limiter.acquire();
-    await limiter.acquire();
-    // Third should be delayed
-    const start = Date.now();
-    await limiter.acquire();
-    const elapsed = Date.now() - start;
-    expect(elapsed).toBeGreaterThanOrEqual(400); // ~500ms for 1 token at 2/sec
-  });
-
-  it('tracks pending count', async () => {
-    limiter = new RateLimiter({ maxPerSecond: 1 });
-    await limiter.acquire();
-    expect(limiter.pending).toBe(0);
-
-    // Start a pending acquire
-    const p = limiter.acquire();
-    expect(limiter.pending).toBe(1);
-    await p;
-    expect(limiter.pending).toBe(0);
-  });
-
-  it('rejects pending on destroy', async () => {
-    limiter = new RateLimiter({ maxPerSecond: 1 });
-    await limiter.acquire();
-    const p = limiter.acquire();
-    limiter.destroy();
-    await expect(p).rejects.toThrow('RateLimiter destroyed');
-  });
-});

package/tests/send-service.test.ts

@@ -1,162 +0,0 @@
-import { describe, it, expect, vi, beforeEach } from 'vitest';
-import { SendService } from '../src/send-service.js';
-import type { Bitrix24Api } from '../src/api.js';
-
-const silentLogger = {
-  info: vi.fn(),
-  warn: vi.fn(),
-  error: vi.fn(),
-  debug: vi.fn(),
-};
-
-function mockApi(): Bitrix24Api {
-  return {
-    answerCommand: vi.fn().mockResolvedValue(true),
-    notifyInputAction: vi.fn().mockResolvedValue(true),
-    sendTyping: vi.fn().mockResolvedValue(true),
-    readMessage: vi.fn().mockResolvedValue({
-      chatId: 1,
-      lastId: 123,
-      counter: 0,
-      viewedMessages: [123],
-    }),
-    sendMessage: vi.fn(),
-    updateMessage: vi.fn(),
-  } as unknown as Bitrix24Api;
-}
-
-describe('SendService', () => {
-  let api: Bitrix24Api;
-  let service: SendService;
-
-  const ctx = {
-    webhookUrl: 'https://test.bitrix24.com/rest/1/token/',
-    bot: { botId: 7, botToken: 'bot_token' },
-    dialogId: '42',
-  };
-  const commandCtx = {
-    ...ctx,
-    commandId: 55,
-    messageId: 123,
-  };
-
-  beforeEach(() => {
-    api = mockApi();
-    service = new SendService(api, silentLogger);
-    vi.clearAllMocks();
-  });
-
-  it('sends explicit input action status', async () => {
-    await service.sendStatus(ctx, 'IMBOT_AGENT_ACTION_COMPOSING', 45);
-
-    expect(api.notifyInputAction).toHaveBeenCalledWith(
-      ctx.webhookUrl,
-      ctx.bot,
-      ctx.dialogId,
-      {
-        statusMessageCode: 'IMBOT_AGENT_ACTION_COMPOSING',
-        duration: 45,
-      },
-    );
-  });
-
-  it('uses notifyInputAction for sendTyping', async () => {
-    await service.sendTyping(ctx);
-
-    expect(api.notifyInputAction).toHaveBeenCalledWith(
-      ctx.webhookUrl,
-      ctx.bot,
-      ctx.dialogId,
-    );
-  });
-
-  it('marks a specific message as read', async () => {
-    await service.markRead(ctx, 123);
-
-    expect(api.readMessage).toHaveBeenCalledWith(
-      ctx.webhookUrl,
-      ctx.bot,
-      ctx.dialogId,
-      123,
-    );
-  });
-
-  it('swallows read failures', async () => {
-    (api.readMessage as ReturnType<typeof vi.fn>).mockRejectedValueOnce(new Error('read failed'));
-
-    await expect(service.markRead(ctx, 123)).resolves.toBeUndefined();
-    expect(silentLogger.debug).toHaveBeenCalled();
-  });
-
-  it('answers native commands via imbot.v2.Command.answer', async () => {
-    await service.answerCommandText(commandCtx, 'Status ok');
-
-    expect(api.answerCommand).toHaveBeenCalledWith(
-      ctx.webhookUrl,
-      ctx.bot,
-      55,
-      123,
-      ctx.dialogId,
-      'Status ok',
-      undefined,
-    );
-    expect(api.sendMessage).not.toHaveBeenCalled();
-  });
-
-  it('sends command follow-up chunks as regular chat messages', async () => {
-    (api.sendMessage as ReturnType<typeof vi.fn>).mockResolvedValueOnce(777);
-    const longText = `${'A'.repeat(20000)}${'B'.repeat(50)}`;
-
-    const result = await service.answerCommandText(commandCtx, longText);
-
-    expect(result.ok).toBe(true);
-    expect(api.answerCommand).toHaveBeenCalledOnce();
-    expect(api.sendMessage).toHaveBeenCalledOnce();
-    expect(api.sendMessage).toHaveBeenCalledWith(
-      ctx.webhookUrl,
-      ctx.bot,
-      ctx.dialogId,
-      'B'.repeat(50),
-      undefined,
-    );
-  });
-
-  it('throws when sendText cannot deliver a chunk', async () => {
-    (api.sendMessage as ReturnType<typeof vi.fn>).mockRejectedValueOnce(new Error('send failed'));
-
-    await expect(service.sendText(ctx, 'Hello')).rejects.toThrow('send failed');
-    expect(silentLogger.error).toHaveBeenCalled();
-  });
-
-  it('throws when command answer fails on the first chunk', async () => {
-    (api.answerCommand as ReturnType<typeof vi.fn>).mockRejectedValueOnce(new Error('answer failed'));
-
-    await expect(service.answerCommandText(commandCtx, 'Status ok')).rejects.toThrow('answer failed');
-    expect(silentLogger.error).toHaveBeenCalled();
-  });
-
-  it('throws when command follow-up chunk cannot be sent', async () => {
-    (api.sendMessage as ReturnType<typeof vi.fn>).mockRejectedValueOnce(new Error('follow-up failed'));
-    const longText = `${'A'.repeat(20000)}${'B'.repeat(50)}`;
-
-    await expect(service.answerCommandText(commandCtx, longText)).rejects.toThrow('follow-up failed');
-    expect(silentLogger.error).toHaveBeenCalled();
-  });
-
-  it('passes forwardMessages to the first outbound chunk', async () => {
-    (api.sendMessage as ReturnType<typeof vi.fn>).mockResolvedValueOnce(777);
-
-    await service.sendText(ctx, 'Forward this', {
-      convertMarkdown: false,
-      forwardMessages: [101, 102],
-    });
-
-    expect(api.sendMessage).toHaveBeenCalledWith(
-      ctx.webhookUrl,
-      ctx.bot,
-      ctx.dialogId,
-      'Forward this',
-      { forwardMessages: [101, 102] },
-    );
-  });
-});

package/tsconfig.json

@@ -1,22 +0,0 @@
-{
-  "compilerOptions": {
-    "target": "ES2022",
-    "module": "ESNext",
-    "moduleResolution": "bundler",
-    "lib": ["ES2022"],
-    "types": ["node"],
-    "outDir": "./dist",
-    "rootDir": ".",
-    "declaration": true,
-    "declarationMap": true,
-    "sourceMap": true,
-    "strict": true,
-    "esModuleInterop": true,
-    "skipLibCheck": true,
-    "forceConsistentCasingInFileNames": true,
-    "resolveJsonModule": true,
-    "isolatedModules": true
-  },
-  "include": ["index.ts", "src/**/*.ts"],
-  "exclude": ["node_modules", "dist", "tests"]
-}

package/vitest.config.ts

@@ -1,9 +0,0 @@
-import { defineConfig } from 'vitest/config';
-
-export default defineConfig({
-  test: {
-    globals: true,
-    environment: 'node',
-    include: ['tests/**/*.test.ts'],
-  },
-});