@sovovs/bycli 1.8.0 → 2.1.0

package/clis/deepseek/ask.test.js

@@ -1,312 +0,0 @@
-import { beforeEach, describe, expect, it, vi } from 'vitest';
-import { CliError, CommandExecutionError, EXIT_CODES } from '@sovovs/bycli/errors';
-
-const {
-  mockEnsureOnDeepSeek,
-  mockSelectModel,
-  mockSetFeature,
-  mockSendMessage,
-  mockSendWithFile,
-  mockGetBubbleCount,
-  mockWaitForResponse,
-  mockParseBoolFlag,
-  mockWithRetry,
-  mockPickResumeUrl,
-} = vi.hoisted(() => ({
-  mockEnsureOnDeepSeek: vi.fn(),
-  mockSelectModel: vi.fn(),
-  mockSetFeature: vi.fn(),
-  mockSendMessage: vi.fn(),
-  mockSendWithFile: vi.fn(),
-  mockGetBubbleCount: vi.fn(),
-  mockWaitForResponse: vi.fn(),
-  mockParseBoolFlag: vi.fn((v) => v === true || v === 'true'),
-  mockWithRetry: vi.fn(async (fn) => fn()),
-  mockPickResumeUrl: vi.fn(),
-}));
-
-vi.mock('./utils.js', () => ({
-  DEEPSEEK_DOMAIN: 'chat.deepseek.com',
-  DEEPSEEK_URL: 'https://chat.deepseek.com/',
-  ensureOnDeepSeek: mockEnsureOnDeepSeek,
-  selectModel: mockSelectModel,
-  setFeature: mockSetFeature,
-  sendMessage: mockSendMessage,
-  sendWithFile: mockSendWithFile,
-  getBubbleCount: mockGetBubbleCount,
-  waitForResponse: mockWaitForResponse,
-  parseBoolFlag: mockParseBoolFlag,
-  withRetry: mockWithRetry,
-  pickResumeUrl: mockPickResumeUrl,
-}));
-
-import { askCommand } from './ask.js';
-
-describe('deepseek ask --file', () => {
-  const page = {
-    wait: vi.fn().mockResolvedValue(undefined),
-    goto: vi.fn().mockResolvedValue(undefined),
-    evaluate: vi.fn().mockResolvedValue('https://chat.deepseek.com/'),
-  };
-
-  beforeEach(() => {
-    vi.clearAllMocks();
-    page.evaluate.mockResolvedValue('https://chat.deepseek.com/');
-    mockEnsureOnDeepSeek.mockResolvedValue(false);
-    mockSelectModel.mockResolvedValue({ ok: true, toggled: false });
-    mockSetFeature.mockResolvedValue({ ok: true, toggled: false });
-    mockSendWithFile.mockResolvedValue({ ok: true });
-    mockGetBubbleCount.mockResolvedValue(7);
-    mockWaitForResponse.mockResolvedValue('new reply');
-  });
-
-  it('captures the existing baseline before sending a file prompt', async () => {
-    const rows = await askCommand.func(page, {
-      prompt: 'summarize this',
-      timeout: 120,
-      file: './report.pdf',
-      new: false,
-      model: 'instant',
-      think: false,
-      search: false,
-    });
-
-    expect(rows).toEqual([{ response: 'new reply' }]);
-    expect(mockGetBubbleCount).toHaveBeenCalledTimes(1);
-    expect(mockSendWithFile).toHaveBeenCalledWith(page, './report.pdf', 'summarize this');
-    expect(mockWaitForResponse).toHaveBeenCalledWith(page, 7, 'summarize this', 120000, false);
-  });
-
-  it('still fails when explicit instant model selection cannot be verified', async () => {
-    mockSelectModel.mockResolvedValue({ ok: false });
-
-    await expect(askCommand.func(page, {
-      prompt: 'summarize this',
-      timeout: 120,
-      new: false,
-      model: 'instant',
-      think: false,
-      search: false,
-    })).rejects.toThrow(new CommandExecutionError('Could not switch to instant model'));
-  });
-});
-
-describe('deepseek ask --think', () => {
-  const page = {
-    wait: vi.fn().mockResolvedValue(undefined),
-    goto: vi.fn().mockResolvedValue(undefined),
-    evaluate: vi.fn().mockResolvedValue('https://chat.deepseek.com/'),
-  };
-
-  beforeEach(() => {
-    vi.clearAllMocks();
-    page.evaluate.mockResolvedValue('https://chat.deepseek.com/');
-    mockEnsureOnDeepSeek.mockResolvedValue(false);
-    mockSelectModel.mockResolvedValue({ ok: true, toggled: false });
-    mockSetFeature.mockResolvedValue({ ok: true, toggled: false });
-    mockSendMessage.mockResolvedValue({ ok: true });
-    mockGetBubbleCount.mockResolvedValue(5);
-  });
-
-  it('returns separate thinking and response fields when --think is enabled', async () => {
-    mockWaitForResponse.mockResolvedValue({
-      response: 'The answer is 42.',
-      thinking: 'Let me analyze this...',
-      thinking_time: '2.5',
-    });
-
-    const rows = await askCommand.func(page, {
-      prompt: 'what is the answer?',
-      timeout: 120,
-      new: false,
-      model: 'instant',
-      think: true,
-      search: false,
-    });
-
-    expect(rows).toEqual([{
-      response: 'The answer is 42.',
-      thinking: 'Let me analyze this...',
-      thinking_time: '2.5',
-    }]);
-    expect(mockWaitForResponse).toHaveBeenCalledWith(page, 5, 'what is the answer?', 120000, true);
-  });
-
-  it('returns plain response when --think is disabled', async () => {
-    mockWaitForResponse.mockResolvedValue('The answer is 42.');
-
-    const rows = await askCommand.func(page, {
-      prompt: 'what is the answer?',
-      timeout: 120,
-      new: false,
-      model: 'instant',
-      think: false,
-      search: false,
-    });
-
-    expect(rows).toEqual([{ response: 'The answer is 42.' }]);
-    expect(mockWaitForResponse).toHaveBeenCalledWith(page, 5, 'what is the answer?', 120000, false);
-  });
-
-  it('does not declare static columns (derived from row keys)', () => {
-    // columns should be undefined so the renderer infers from row keys,
-    // avoiding empty trailing columns on non-think output.
-    expect(askCommand.columns).toBeUndefined();
-  });
-
-  it('non-think rows only contain response key', async () => {
-    mockWaitForResponse.mockResolvedValue('Plain answer.');
-
-    const rows = await askCommand.func(page, {
-      prompt: 'hello',
-      timeout: 120,
-      new: false,
-      model: 'instant',
-      think: false,
-      search: false,
-    });
-
-    // Row keys drive rendered columns; no thinking/thinking_time present.
-    expect(Object.keys(rows[0])).toEqual(['response']);
-  });
-});
-
-describe('deepseek ask conversation resume', () => {
-  const page = {
-    wait: vi.fn().mockResolvedValue(undefined),
-    goto: vi.fn().mockResolvedValue(undefined),
-    evaluate: vi.fn(),
-  };
-
-  beforeEach(() => {
-    vi.clearAllMocks();
-    mockSetFeature.mockResolvedValue({ ok: true, toggled: false });
-    mockSendMessage.mockResolvedValue({ ok: true });
-    mockGetBubbleCount.mockResolvedValue(2);
-    mockWaitForResponse.mockResolvedValue('follow-up reply');
-  });
-
-  it('resumes the most recent conversation and skips model selection', async () => {
-    mockEnsureOnDeepSeek.mockResolvedValue(true);
-    mockPickResumeUrl.mockResolvedValue('https://chat.deepseek.com/a/chat/s/abc-123');
-    // URL check after resume navigation: now inside a conversation.
-    page.evaluate.mockResolvedValueOnce('https://chat.deepseek.com/a/chat/s/abc-123');
-
-    const rows = await askCommand.func(page, {
-      prompt: 'follow up',
-      timeout: 120,
-      new: false,
-      model: 'instant',
-      think: false,
-      search: false,
-    });
-
-    expect(rows).toEqual([{ response: 'follow-up reply' }]);
-    expect(page.goto).toHaveBeenCalledWith('https://chat.deepseek.com/a/chat/s/abc-123');
-    expect(mockSelectModel).not.toHaveBeenCalled();
-    expect(mockSendMessage).toHaveBeenCalled();
-  });
-
-  it('skips model selection when already inside an existing conversation', async () => {
-    mockEnsureOnDeepSeek.mockResolvedValue(false);
-    page.evaluate.mockResolvedValue('https://chat.deepseek.com/a/chat/s/abc-123');
-
-    const rows = await askCommand.func(page, {
-      prompt: 'continue',
-      timeout: 120,
-      new: false,
-      model: 'expert',
-      think: false,
-      search: false,
-    });
-
-    expect(rows).toEqual([{ response: 'follow-up reply' }]);
-    expect(mockSelectModel).not.toHaveBeenCalled();
-  });
-
-  it('fails fast when --model is explicitly requested inside an existing conversation', async () => {
-    mockEnsureOnDeepSeek.mockResolvedValue(false);
-    page.evaluate.mockResolvedValue('https://chat.deepseek.com/a/chat/s/abc-123');
-
-    await expect(askCommand.func(page, {
-      prompt: 'continue',
-      timeout: 120,
-      new: false,
-      model: 'expert',
-      think: false,
-      search: false,
-      __bycliOptionSources: { model: 'cli' },
-    })).rejects.toMatchObject(new CliError(
-      'ARGUMENT',
-      'Cannot switch to expert model inside an existing conversation.',
-      'Re-run with --new to start a fresh chat before selecting a model.',
-      EXIT_CODES.USAGE_ERROR,
-    ));
-
-    expect(mockSelectModel).not.toHaveBeenCalled();
-  });
-
-  it('fails fast when the workspace was recycled but no conversation surfaces in time', async () => {
-    mockEnsureOnDeepSeek.mockResolvedValue(true);
-    mockPickResumeUrl.mockResolvedValue(null);
-
-    await expect(askCommand.func(page, {
-      prompt: 'hello',
-      timeout: 120,
-      new: false,
-      model: 'instant',
-      think: false,
-      search: false,
-    })).rejects.toBeInstanceOf(CommandExecutionError);
-
-    expect(page.goto).not.toHaveBeenCalled();
-    expect(mockSelectModel).not.toHaveBeenCalled();
-    expect(mockSendMessage).not.toHaveBeenCalled();
-  });
-
-  it('skips search toggle in vision mode when search is not requested', async () => {
-    mockEnsureOnDeepSeek.mockResolvedValue(false);
-    mockSelectModel.mockResolvedValue({ ok: true, toggled: false });
-    mockSetFeature.mockResolvedValue({ ok: true, toggled: false });
-    mockSendMessage.mockResolvedValue({ ok: true });
-    mockGetBubbleCount.mockResolvedValue(0);
-    mockWaitForResponse.mockResolvedValue('vision reply');
-    page.evaluate.mockResolvedValue('https://chat.deepseek.com/');
-
-    const rows = await askCommand.func(page, {
-      prompt: 'describe',
-      timeout: 120,
-      new: false,
-      model: 'vision',
-      think: false,
-      search: false,
-    });
-
-    expect(rows).toEqual([{ response: 'vision reply' }]);
-    expect(mockSetFeature).toHaveBeenCalledTimes(1);
-    expect(mockSetFeature).toHaveBeenCalledWith(expect.anything(), 'DeepThink', false);
-  });
-
-  it('fails fast instead of silently ignoring --search in vision mode', async () => {
-    mockEnsureOnDeepSeek.mockResolvedValue(false);
-    mockSelectModel.mockResolvedValue({ ok: true, toggled: false });
-    page.evaluate.mockResolvedValue('https://chat.deepseek.com/');
-
-    await expect(askCommand.func(page, {
-      prompt: 'describe',
-      timeout: 120,
-      new: false,
-      model: 'vision',
-      think: false,
-      search: true,
-    })).rejects.toMatchObject(new CliError(
-      'ARGUMENT',
-      'DeepSeek vision mode does not support --search.',
-      'Run without --search, or use --model instant/expert for web search.',
-      EXIT_CODES.USAGE_ERROR,
-    ));
-
-    expect(mockSendMessage).not.toHaveBeenCalled();
-    expect(mockSendWithFile).not.toHaveBeenCalled();
-  });
-});

package/clis/deepseek/detail.test.js

@@ -1,81 +0,0 @@
-import { beforeEach, describe, expect, it, vi } from 'vitest';
-import { ArgumentError, EmptyResultError } from '@sovovs/bycli/errors';
-import { getRegistry } from '@sovovs/bycli/registry';
-
-const {
-  mockEnsureOnDeepSeek,
-  mockGetVisibleMessages,
-} = vi.hoisted(() => ({
-  mockEnsureOnDeepSeek: vi.fn(),
-  mockGetVisibleMessages: vi.fn(),
-}));
-
-vi.mock('./utils.js', async () => {
-  const actual = await vi.importActual('./utils.js');
-  return {
-    ...actual,
-    ensureOnDeepSeek: mockEnsureOnDeepSeek,
-    getVisibleMessages: mockGetVisibleMessages,
-  };
-});
-
-import './detail.js';
-
-describe('deepseek detail', () => {
-  const command = getRegistry().get('deepseek/detail');
-  const id = '749e6bbd-6a45-4440-beaa-ae5238bf06d8';
-
-  beforeEach(() => {
-    vi.clearAllMocks();
-    mockEnsureOnDeepSeek.mockResolvedValue(false);
-  });
-
-  it('registers as a cookie-browser read command', () => {
-    expect(command).toBeDefined();
-    expect(command.browser).toBe(true);
-    expect(command.strategy).toBe('cookie');
-    expect(command.access).toBe('read');
-    expect(command.columns).toEqual(['Role', 'Text']);
-  });
-
-  it('navigates to the conversation URL and returns visible messages', async () => {
-    mockGetVisibleMessages.mockResolvedValue([
-      { Role: 'user', Text: 'hello' },
-      { Role: 'assistant', Text: 'hi' },
-    ]);
-    const page = { wait: vi.fn().mockResolvedValue(undefined), goto: vi.fn().mockResolvedValue(undefined) };
-
-    const rows = await command.func(page, { id });
-
-    expect(rows).toEqual([
-      { Role: 'user', Text: 'hello' },
-      { Role: 'assistant', Text: 'hi' },
-    ]);
-    expect(page.goto).toHaveBeenCalledWith(`https://chat.deepseek.com/a/chat/s/${id}`);
-    expect(mockGetVisibleMessages).toHaveBeenCalledWith(page);
-  });
-
-  it('accepts a full chat URL and normalises it before navigation', async () => {
-    mockGetVisibleMessages.mockResolvedValue([{ Role: 'user', Text: 'hi' }]);
-    const page = { wait: vi.fn().mockResolvedValue(undefined), goto: vi.fn().mockResolvedValue(undefined) };
-
-    await command.func(page, { id: `https://chat.deepseek.com/a/chat/s/${id.toUpperCase()}?ref=foo` });
-
-    expect(page.goto).toHaveBeenCalledWith(`https://chat.deepseek.com/a/chat/s/${id}`);
-  });
-
-  it('rejects malformed IDs before browser navigation', async () => {
-    const page = { wait: vi.fn(), goto: vi.fn() };
-
-    await expect(command.func(page, { id: 'not-a-uuid' })).rejects.toThrow(ArgumentError);
-    expect(page.goto).not.toHaveBeenCalled();
-    expect(mockEnsureOnDeepSeek).not.toHaveBeenCalled();
-  });
-
-  it('throws EmptyResultError when the conversation has no visible messages', async () => {
-    mockGetVisibleMessages.mockResolvedValue([]);
-    const page = { wait: vi.fn().mockResolvedValue(undefined), goto: vi.fn().mockResolvedValue(undefined) };
-
-    await expect(command.func(page, { id })).rejects.toThrow(EmptyResultError);
-  });
-});

package/clis/deepseek/send.test.js

@@ -1,107 +0,0 @@
-import { beforeEach, describe, expect, it, vi } from 'vitest';
-import { ArgumentError, CommandExecutionError } from '@sovovs/bycli/errors';
-import { getRegistry } from '@sovovs/bycli/registry';
-
-import './send.js';
-
-describe('deepseek send', () => {
-  const command = getRegistry().get('deepseek/send');
-  const id = '749e6bbd-6a45-4440-beaa-ae5238bf06d8';
-
-  function createPage(overrides = {}) {
-    return {
-      goto: vi.fn().mockResolvedValue(undefined),
-      wait: vi.fn().mockResolvedValue(undefined),
-      evaluate: vi.fn(),
-      nativeType: vi.fn().mockResolvedValue(undefined),
-      screenshot: vi.fn().mockResolvedValue(''),
-      ...overrides,
-    };
-  }
-
-  beforeEach(() => {
-    vi.clearAllMocks();
-  });
-
-  it('registers as a cookie-browser write command with id + prompt positional args', () => {
-    expect(command).toBeDefined();
-    expect(command.browser).toBe(true);
-    expect(command.strategy).toBe('cookie');
-    expect(command.access).toBe('write');
-    expect(command.columns).toEqual(['Status', 'InjectedText']);
-    expect(command.args.map((a) => a.name)).toEqual(['id', 'prompt', 'timeout']);
-    expect(command.args.find((a) => a.name === 'timeout')).toMatchObject({ type: 'int', default: 60 });
-  });
-
-  it('rejects malformed conversation IDs before any browser navigation', async () => {
-    const page = createPage();
-    await expect(command.func(page, { id: 'not-a-uuid', prompt: 'hi' })).rejects.toThrow(ArgumentError);
-    expect(page.goto).not.toHaveBeenCalled();
-    expect(page.nativeType).not.toHaveBeenCalled();
-  });
-
-  it('navigates to the conversation, drives input through nativeType, and verifies the bubble', async () => {
-    const page = createPage();
-    page.evaluate
-      .mockResolvedValueOnce(true)             // waitForTextareaReady probe
-      .mockResolvedValueOnce(true)             // focus check
-      .mockResolvedValueOnce({ ok: true });    // submit + verify IIFE
-
-    const rows = await command.func(page, { id, prompt: 'hello' });
-
-    expect(rows).toEqual([{ Status: 'Success', InjectedText: 'hello' }]);
-    expect(page.goto).toHaveBeenCalledWith(`https://chat.deepseek.com/a/chat/s/${id}`);
-    expect(page.nativeType).toHaveBeenCalledWith('hello');
-  });
-
-  it('throws when the textarea never mounts within the readiness timeout', async () => {
-    const page = createPage();
-    page.evaluate.mockResolvedValue(false);    // probe always fails
-
-    await expect(command.func(page, { id, prompt: 'hi' })).rejects.toThrow(CommandExecutionError);
-    expect(page.nativeType).not.toHaveBeenCalled();
-  });
-
-  it('throws when nativeType is not exposed on the page object', async () => {
-    const page = createPage({ nativeType: undefined });
-    page.evaluate
-      .mockResolvedValueOnce(true)             // waitForTextareaReady
-      .mockResolvedValueOnce(true);            // focus
-
-    await expect(command.func(page, { id, prompt: 'hi' })).rejects.toThrow(CommandExecutionError);
-  });
-
-  it('throws when the focus probe reports the textarea did not take focus', async () => {
-    const page = createPage();
-    page.evaluate
-      .mockResolvedValueOnce(true)             // waitForTextareaReady
-      .mockResolvedValueOnce(false);           // focus failed
-
-    await expect(command.func(page, { id, prompt: 'hi' })).rejects.toThrow(CommandExecutionError);
-    expect(page.nativeType).not.toHaveBeenCalled();
-  });
-
-  it('translates the IIFE failure reason into a CommandExecutionError', async () => {
-    const page = createPage();
-    page.evaluate
-      .mockResolvedValueOnce(true)
-      .mockResolvedValueOnce(true)
-      .mockResolvedValueOnce({ ok: false, reason: 'send button stayed disabled after native input' });
-
-    await expect(command.func(page, { id, prompt: 'hi' })).rejects.toThrow(
-      new CommandExecutionError('send button stayed disabled after native input'),
-    );
-  });
-
-  it('treats "Promise was collected" from the IIFE as a successful submit', async () => {
-    const page = createPage();
-    page.evaluate
-      .mockResolvedValueOnce(true)
-      .mockResolvedValueOnce(true)
-      .mockRejectedValueOnce(new Error('{"code":-32000,"message":"Promise was collected"}'));
-
-    const rows = await command.func(page, { id, prompt: 'hi' });
-
-    expect(rows).toEqual([{ Status: 'Success', InjectedText: 'hi' }]);
-  });
-});