@sovovs/bycli 1.8.0 → 2.1.0

package/clis/linkedin/profile-read.test.js

@@ -1,77 +0,0 @@
-import { describe, expect, it, vi } from 'vitest';
-import { getRegistry } from '@sovovs/bycli/registry';
-import { CommandExecutionError } from '@sovovs/bycli/errors';
-import './profile-read.js';
-
-const { normalizeProfileReadUrl, normalizeProfile } = await import('./profile-read.js').then((m) => m.__test__);
-
-describe('linkedin profile-read adapter', () => {
-  const command = getRegistry().get('linkedin/profile-read');
-
-  it('registers command shape', () => {
-    expect(command).toBeDefined();
-    expect(command.strategy).toBe('cookie');
-    expect(command.browser).toBe(true);
-    expect(command.columns).toEqual([
-      'profile_url',
-      'name',
-      'headline',
-      'location',
-      'about',
-      'about_character_count',
-      'about_skills',
-      'experience',
-      'education',
-      'services',
-      'featured',
-    ]);
-  });
-
-  it('normalizes profile url default and explicit /in URL', () => {
-    expect(normalizeProfileReadUrl(undefined)).toBe('https://www.linkedin.com/in/me/');
-    expect(normalizeProfileReadUrl('https://www.linkedin.com/in/gauravsaxena1997?x=1')).toBe('https://www.linkedin.com/in/gauravsaxena1997?x=1');
-  });
-
-  it('rejects non-profile URLs', () => {
-    expect(() => normalizeProfileReadUrl('https://www.linkedin.com/jobs/')).toThrow(CommandExecutionError);
-  });
-
-  it('normalizes duplicated profile text and requires a name', () => {
-    expect(() => normalizeProfile({ name: '' })).toThrow(CommandExecutionError);
-    expect(normalizeProfile({
-      name: 'AliceAlice',
-      headline: 'EngineerEngineer',
-      about: '  Builds AI  ',
-      about_character_count: '1,100/2,600',
-      about_skills: ['AI', 'TypeScript'],
-    })).toMatchObject({
-      name: 'Alice',
-      headline: 'Engineer',
-      about: 'Builds AI',
-      about_character_count: '1,100/2,600',
-      about_skills: 'AI; TypeScript',
-    });
-  });
-
-  it('does not require edit access when reading an explicit profile URL', async () => {
-    const page = {
-      goto: vi.fn(async () => {}),
-      wait: vi.fn(async () => {}),
-      autoScroll: vi.fn(async () => {}),
-      evaluate: vi.fn()
-        .mockResolvedValueOnce(false)
-        .mockResolvedValueOnce({
-          profile_url: 'https://www.linkedin.com/in/alice/',
-          name: 'Alice',
-          headline: 'Engineer',
-          about: 'Builds products',
-        }),
-    };
-
-    await expect(command.func(page, { 'profile-url': 'https://www.linkedin.com/in/alice/' }))
-      .resolves.toMatchObject([{ name: 'Alice', about: 'Builds products' }]);
-    expect(page.goto).toHaveBeenCalledTimes(1);
-    expect(page.goto).toHaveBeenCalledWith('https://www.linkedin.com/in/alice/');
-    expect(page.goto.mock.calls.some(([url]) => String(url).includes('/edit/forms/'))).toBe(false);
-  });
-});

package/clis/linkedin/safe-send.test.js

@@ -1,204 +0,0 @@
-import { describe, expect, it, vi } from 'vitest';
-import { getRegistry } from '@sovovs/bycli/registry';
-import { ArgumentError, CommandExecutionError } from '@sovovs/bycli/errors';
-import './safe-send.js';
-
-const {
-  normalizeWhitespace,
-  normalizeName,
-  canonicalizeLinkedInThreadUrl,
-  hashText,
-  assessThreadSafety,
-} = await import('./safe-send.js').then((m) => m.__test__);
-
-function makeFakePage(probe) {
-  let composerText = probe.composerText || '';
-  return {
-    goto: vi.fn(async () => undefined),
-    wait: vi.fn(async () => undefined),
-    evaluate: vi.fn(async (script) => {
-      const text = String(script);
-      if (text.includes('__BYCLI_LINKEDIN_PROBE__')) return probe;
-      if (text.includes('__BYCLI_LINKEDIN_FOCUS_COMPOSER__')) return { ok: true, composerText: '' };
-      if (text.includes('__BYCLI_LINKEDIN_READ_COMPOSER__')) return { ok: true, composerText };
-      if (text.includes('__BYCLI_LINKEDIN_CLICK_SEND__')) return { ok: true, sent: true };
-      return undefined;
-    }),
-    insertText: vi.fn(async (text) => {
-      composerText = text;
-    }),
-    pressKey: vi.fn(async () => undefined),
-    screenshot: vi.fn(async () => 'base64-screenshot'),
-  };
-}
-
-describe('linkedin safe-send helpers', () => {
-  it('normalizes whitespace and LinkedIn names for exact-ish comparisons', () => {
-    expect(normalizeWhitespace('  Lokesh\n\tRamesh  ')).toBe('Lokesh Ramesh');
-    expect(normalizeName('Lokesh Ramesh • 1st')).toBe('lokesh ramesh');
-  });
-
-  it('canonicalizes thread URLs while dropping query and hash noise', () => {
-    expect(canonicalizeLinkedInThreadUrl('https://www.linkedin.com/messaging/thread/abc/?foo=1#bar'))
-      .toBe('https://www.linkedin.com/messaging/thread/abc/');
-    expect(canonicalizeLinkedInThreadUrl('https://www.linkedin.com/messaging/thread/abc/extra')).toBe('');
-    expect(canonicalizeLinkedInThreadUrl('https://evil-linkedin.com/messaging/thread/abc/')).toBe('');
-    expect(canonicalizeLinkedInThreadUrl('http://www.linkedin.com/messaging/thread/abc/')).toBe('');
-  });
-
-  it('fails closed when LinkedIn search produced no results even if a composer is visible', () => {
-    const result = assessThreadSafety({
-      url: 'https://www.linkedin.com/messaging/thread/bora/',
-      headerNames: ['Bora Nicholson'],
-      bodyText: "We didn't find anything for Victoria Munoz\nBora Nicholson",
-      searchFailure: true,
-      composerFound: true,
-      latestMessageHash: hashText('hello'),
-    }, {
-      expectedName: 'Victoria Munoz',
-      threadUrl: 'https://www.linkedin.com/messaging/thread/victoria/',
-      expectedLastText: 'hello',
-    });
-
-    expect(result.ok).toBe(false);
-    expect(result.blockReason).toBe('search_failure_visible');
-  });
-
-  it('fails closed on recipient header mismatch', () => {
-    const result = assessThreadSafety({
-      url: 'https://www.linkedin.com/messaging/thread/bora/',
-      headerNames: ['Bora Nicholson'],
-      bodyText: 'Bora Nicholson\nhello',
-      composerFound: true,
-      latestMessageHash: hashText('hello'),
-    }, {
-      expectedName: 'Victoria Munoz',
-      expectedLastText: 'hello',
-    });
-
-    expect(result.ok).toBe(false);
-    expect(result.blockReason).toBe('recipient_header_mismatch');
-    expect(result.actualValue).toBe('Bora Nicholson');
-  });
-
-  it('fails closed when the stored latest message is no longer visible', () => {
-    const result = assessThreadSafety({
-      url: 'https://www.linkedin.com/messaging/thread/lokesh/',
-      headerNames: ['Lokesh Ramesh'],
-      bodyText: 'Lokesh Ramesh\na newer inbound arrived',
-      composerFound: true,
-      latestMessageHash: hashText('a newer inbound arrived'),
-    }, {
-      expectedName: 'Lokesh Ramesh',
-      expectedLastText: 'old inbound text',
-    });
-
-    expect(result.ok).toBe(false);
-    expect(result.blockReason).toBe('latest_message_mismatch');
-  });
-
-  it('passes only when recipient, thread, latest text, and composer are all verified', () => {
-    const result = assessThreadSafety({
-      url: 'https://www.linkedin.com/messaging/thread/lokesh/?mini=true',
-      headerNames: ['Lokesh Ramesh'],
-      bodyText: 'Lokesh Ramesh\nI think outside help would fit best for provider doc follow ups',
-      composerFound: true,
-      latestMessageHash: hashText('I think outside help would fit best for provider doc follow ups'),
-    }, {
-      expectedName: 'Lokesh Ramesh',
-      threadUrl: 'https://www.linkedin.com/messaging/thread/lokesh/',
-      expectedLastText: 'provider doc follow ups',
-    });
-
-    expect(result.ok).toBe(true);
-    expect(result.blockReason).toBe('verified');
-  });
-});
-
-describe('linkedin safe-send command', () => {
-  it('registers as a write command with safe output columns', () => {
-    const command = getRegistry().get('linkedin/safe-send');
-    expect(command).toBeDefined();
-    expect(command.access).toBe('write');
-    expect(command.columns).toEqual(expect.arrayContaining(['status', 'recipient', 'reason']));
-  });
-
-  it('does not type or send when verification fails', async () => {
-    const command = getRegistry().get('linkedin/safe-send');
-    const page = makeFakePage({
-      url: 'https://www.linkedin.com/messaging/thread/bora/',
-      headerNames: ['Bora Nicholson'],
-      bodyText: 'Bora Nicholson',
-      composerFound: true,
-      searchFailure: false,
-    });
-
-    await expect(command.func(page, {
-      'thread-url': 'https://www.linkedin.com/messaging/thread/victoria/',
-      'expected-name': 'Victoria Munoz',
-      message: 'hello victoria',
-      send: true,
-    })).rejects.toBeInstanceOf(CommandExecutionError);
-
-    expect(page.insertText).not.toHaveBeenCalled();
-    expect(page.pressKey).not.toHaveBeenCalled();
-  });
-
-  it('rejects non-thread URLs before navigating or typing', async () => {
-    const command = getRegistry().get('linkedin/safe-send');
-    const page = makeFakePage({});
-
-    await expect(command.func(page, {
-      'thread-url': 'https://www.linkedin.com/feed/',
-      'expected-name': 'Victoria Munoz',
-      message: 'hello victoria',
-      send: true,
-    })).rejects.toBeInstanceOf(ArgumentError);
-
-    expect(page.goto).not.toHaveBeenCalled();
-    expect(page.insertText).not.toHaveBeenCalled();
-  });
-
-  it('dry-runs by default after verification without filling or sending', async () => {
-    const command = getRegistry().get('linkedin/safe-send');
-    const page = makeFakePage({
-      url: 'https://www.linkedin.com/messaging/thread/lokesh/',
-      headerNames: ['Lokesh Ramesh'],
-      bodyText: 'Lokesh Ramesh\nprovider doc follow ups',
-      composerFound: true,
-      searchFailure: false,
-    });
-
-    const rows = await command.func(page, {
-      'thread-url': 'https://www.linkedin.com/messaging/thread/lokesh/',
-      'expected-name': 'Lokesh Ramesh',
-      message: 'both, but starting hands on',
-    });
-
-    expect(rows[0]).toMatchObject({ status: 'verified_dry_run', recipient: 'Lokesh Ramesh', reason: 'verified' });
-    expect(page.insertText).not.toHaveBeenCalled();
-    expect(page.pressKey).not.toHaveBeenCalled();
-  });
-
-  it('fills and sends only when --send is explicitly true and post-fill verification matches exactly', async () => {
-    const command = getRegistry().get('linkedin/safe-send');
-    const page = makeFakePage({
-      url: 'https://www.linkedin.com/messaging/thread/lokesh/',
-      headerNames: ['Lokesh Ramesh'],
-      bodyText: 'Lokesh Ramesh\nprovider doc follow ups',
-      composerFound: true,
-      searchFailure: false,
-    });
-
-    const rows = await command.func(page, {
-      'thread-url': 'https://www.linkedin.com/messaging/thread/lokesh/',
-      'expected-name': 'Lokesh Ramesh',
-      message: 'both, but starting hands on',
-      send: true,
-    });
-
-    expect(rows[0]).toMatchObject({ status: 'sent', recipient: 'Lokesh Ramesh', reason: 'verified' });
-    expect(page.insertText).toHaveBeenCalledWith('both, but starting hands on');
-    expect(page.pressKey).not.toHaveBeenCalled();
-  });
-});

package/clis/linkedin/salesnav-inbox.test.js

@@ -1,113 +0,0 @@
-import { describe, expect, it } from 'vitest';
-import { CommandExecutionError } from '@sovovs/bycli/errors';
-import './salesnav-inbox.js';
-
-const {
-  THREAD_DECORATION,
-  encodeRestliDecoration,
-  parseLimit,
-  parseSalesnavThreads,
-  salesnavThreadUrl,
-  threadListUrl,
-} = await import('./salesnav-inbox.js').then((m) => m.__test__);
-
-describe('linkedin salesnav-inbox command', () => {
-  it('percent-encodes Rest.li decoration parentheses for Sales Navigator messaging', () => {
-    const encoded = encodeRestliDecoration('(id,messages*(body))');
-    expect(encoded).toBe('%28id%2Cmessages*%28body%29%29');
-    expect(encoded).not.toContain('(');
-    expect(encoded).not.toContain(')');
-  });
-
-  it('builds the paginated salesApiMessagingThreads inbox URL', () => {
-    const url = threadListUrl({ count: 20, pageStartsAt: '1779070755626' });
-    expect(url).toContain('/sales-api/salesApiMessagingThreads?');
-    expect(url).toContain('q=filter');
-    expect(url).toContain('filter=INBOX');
-    expect(url).toContain('count=20');
-    expect(url).toContain('pageStartsAt=1779070755626');
-    expect(url).toContain(encodeRestliDecoration(THREAD_DECORATION));
-  });
-
-  it('validates limits without silent clamping', () => {
-    expect(parseLimit(undefined)).toBe(40);
-    expect(parseLimit(12)).toBe(12);
-    expect(() => parseLimit(0)).toThrow();
-    expect(() => parseLimit(501)).toThrow();
-    expect(() => parseLimit('abc')).toThrow();
-  });
-
-  it('parses Sales Navigator thread rows with other participant and unread state', () => {
-    const rows = parseSalesnavThreads({ elements: [{
-      id: '2-thread',
-      unreadMessageCount: 1,
-      archived: false,
-      totalMessageCount: 2,
-      nextPageStartsAt: 1778206803669,
-      participants: [
-        'urn:li:fs_salesProfile:(OTHER,NAME_SEARCH,T1)',
-        'urn:li:fs_salesProfile:(SELF,NAME_SEARCH,T2)',
-      ],
-      participantsResolutionResults: {
-        'urn:li:fs_salesProfile:(OTHER,NAME_SEARCH,T1)': {
-          entityUrn: 'urn:li:fs_salesProfile:(OTHER,NAME_SEARCH,T1)',
-          firstName: 'Rachael',
-          lastName: 'Stolberg',
-          fullName: 'Rachael Stolberg',
-          degree: 2,
-        },
-        'urn:li:fs_salesProfile:(SELF,NAME_SEARCH,T2)': {
-          entityUrn: 'urn:li:fs_salesProfile:(SELF,NAME_SEARCH,T2)',
-          firstName: 'Hanzi',
-          lastName: 'Li',
-          fullName: 'Hanzi Li',
-          degree: 0,
-        },
-      },
-      messages: [{
-        id: 'msg-1',
-        author: 'urn:li:fs_salesProfile:(OTHER,NAME_SEARCH,T1)',
-        body: 'Hi hanzi, happy to chat',
-        deliveredAt: 1778206803669,
-      }],
-    }] });
-    expect(rows).toHaveLength(1);
-    expect(rows[0]).toMatchObject({
-      thread_id: '2-thread',
-      thread_url: salesnavThreadUrl('2-thread'),
-      person_name: 'Rachael Stolberg',
-      last_message_snippet: 'Hi hanzi, happy to chat',
-      last_activity_time: '2026-05-08T02:20:03.669Z',
-      unread: true,
-      unread_count: 1,
-      total_message_count: 2,
-    });
-  });
-
-  it('does not hard-code a specific account name as the inbox owner', () => {
-    const rows = parseSalesnavThreads({ elements: [{
-      id: '2-thread',
-      participants: [
-        'urn:li:fs_salesProfile:(HANZI,NAME_SEARCH,T1)',
-        'urn:li:fs_salesProfile:(ME,NAME_SEARCH,T2)',
-      ],
-      participantsResolutionResults: {
-        'urn:li:fs_salesProfile:(HANZI,NAME_SEARCH,T1)': {
-          fullName: 'Hanzi Li',
-          degree: 2,
-        },
-        'urn:li:fs_salesProfile:(ME,NAME_SEARCH,T2)': {
-          fullName: 'Current User',
-          degree: 0,
-        },
-      },
-      messages: [],
-    }] });
-    expect(rows[0].person_name).toBe('Hanzi Li');
-  });
-
-  it('fails typed on malformed thread payloads and missing thread identity', () => {
-    expect(() => parseSalesnavThreads({})).toThrow(CommandExecutionError);
-    expect(() => parseSalesnavThreads({ elements: [{}] })).toThrow(CommandExecutionError);
-  });
-});

package/clis/linkedin/salesnav-message.test.js

@@ -1,172 +0,0 @@
-import { describe, expect, it, vi } from 'vitest';
-import { getRegistry } from '@sovovs/bycli/registry';
-import { CommandExecutionError } from '@sovovs/bycli/errors';
-import './salesnav-message.js';
-
-const {
-  parseSalesProfileUrn,
-  parseRecipient,
-  salesLeadUrlFromParts,
-  profileApiUrl,
-  buildCreateMessagePayload,
-  extractRemainingCredits,
-  profileSummary,
-  requireProfileSummary,
-  salesPageShowsSentMessage,
-} = await import('./salesnav-message.js').then((m) => m.__test__);
-
-function createPageMock(evaluateResults = []) {
-  const evaluate = vi.fn();
-  for (const result of evaluateResults) evaluate.mockResolvedValueOnce(result);
-  evaluate.mockResolvedValue(undefined);
-  return {
-    goto: vi.fn().mockResolvedValue(undefined),
-    wait: vi.fn().mockResolvedValue(undefined),
-    evaluate,
-    getCookies: vi.fn().mockResolvedValue([{ name: 'JSESSIONID', value: '"csrf"', domain: '.linkedin.com' }]),
-  };
-}
-
-describe('linkedin salesnav-message command', () => {
-  it('parses Sales Navigator profile urns and lead URLs', () => {
-    const urn = 'urn:li:fs_salesProfile:(ACwAAAJS8TABxyz,NAME_SEARCH,Enlo)';
-    expect(parseSalesProfileUrn(urn)).toMatchObject({
-      profileId: 'ACwAAAJS8TABxyz',
-      authType: 'NAME_SEARCH',
-      authToken: 'Enlo',
-      entityUrn: urn,
-    });
-    expect(parseSalesProfileUrn('urn:li:fs_salesProfile:(ACwAAAJS8TABxyz,undefined,undefined)')).toBeNull();
-
-    const parsed = parseRecipient('https://www.linkedin.com/sales/lead/ACwAAAJS8TABxyz,NAME_SEARCH,Enlo');
-    expect(parsed).toMatchObject({ profileId: 'ACwAAAJS8TABxyz', authType: 'NAME_SEARCH', authToken: 'Enlo' });
-    expect(parsed.entityUrn).toBe(urn);
-    expect(salesLeadUrlFromParts(parsed)).toBe('https://www.linkedin.com/sales/lead/ACwAAAJS8TABxyz,NAME_SEARCH,Enlo');
-  });
-
-  it('accepts LinkedIn /in tokens as unresolved recipients', () => {
-    expect(parseRecipient('https://www.linkedin.com/in/ACwAAAJS8TABxyz/')).toMatchObject({
-      profileId: 'ACwAAAJS8TABxyz',
-      authType: '',
-      authToken: '',
-      entityUrn: '',
-    });
-  });
-
-  it('builds profile API URLs with the Sales Navigator auth key', () => {
-    const url = profileApiUrl({ profileId: 'P1', authType: 'NAME_SEARCH', authToken: 'T1' });
-    expect(url).toContain('/sales-api/salesApiProfiles/(profileId:P1,authType:NAME_SEARCH,authToken:T1)');
-    expect(url).toContain('decoration=');
-  });
-
-  it('constructs the createMessage action payload used by Sales Navigator', () => {
-    const payload = buildCreateMessagePayload({
-      recipientUrn: 'urn:li:fs_salesProfile:(P1,NAME_SEARCH,T1)',
-      subject: 'Quick QA doc question',
-      body: 'Hi Jane, can I ask a quick question?',
-      trackingId: '0123456789abcdef',
-      copyToCrm: false,
-    });
-    expect(payload).toEqual({
-      createMessageRequest: {
-        recipients: ['urn:li:fs_salesProfile:(P1,NAME_SEARCH,T1)'],
-        subject: 'Quick QA doc question',
-        body: 'Hi Jane, can I ask a quick question?',
-        copyToCrm: false,
-        trackingId: '0123456789abcdef',
-      },
-    });
-  });
-
-  it('validates payload fields before any send attempt', () => {
-    expect(() => buildCreateMessagePayload({ recipientUrn: '', subject: 's', body: 'b' })).toThrow();
-    expect(() => buildCreateMessagePayload({ recipientUrn: 'urn:li:fs_salesProfile:(P,A,T)', subject: '', body: 'b' })).toThrow();
-    expect(() => buildCreateMessagePayload({ recipientUrn: 'urn:li:fs_salesProfile:(P,A,T)', subject: 's', body: '' })).toThrow();
-    expect(() => buildCreateMessagePayload({ recipientUrn: 'urn:li:fs_salesProfile:(P,A,T)', subject: 'x'.repeat(201), body: 'b' })).toThrow();
-    expect(() => buildCreateMessagePayload({ recipientUrn: 'urn:li:fs_salesProfile:(P,undefined,undefined)', subject: 's', body: 'b' })).toThrow();
-  });
-
-  it('detects the Sales Navigator sent activity on a verified lead page', () => {
-    expect(salesPageShowsSentMessage('5/18/2026 You sent a Sales Navigator message to Jane', 'Jane Q')).toBe(true);
-    expect(salesPageShowsSentMessage('No recent activity', 'Jane Q')).toBe(false);
-  });
-
-  it('extracts a plausible remaining InMail credit count', () => {
-    expect(extractRemainingCredits({ elements: [{ type: 'LSS_INMAIL', value: 149, id: 1 }], paging: { count: 10 } })).toBe(149);
-    expect(extractRemainingCredits({ data: { remaining: 149, used: 1 } })).toBe(149);
-    expect(extractRemainingCredits({ elements: [{ availableCount: 12 }] })).toBe(12);
-    expect(extractRemainingCredits({})).toBe(null);
-  });
-
-  it('summarizes decorated Sales Navigator profile data', () => {
-    expect(profileSummary({ data: {
-      fullName: 'Rayki Goh',
-      headline: 'Food Safety',
-      degree: 3,
-      defaultPosition: { title: 'FSQA Manager', companyName: 'Acme Foods' },
-      memberBadges: { openLink: false },
-    } })).toMatchObject({
-      recipient: 'Rayki Goh',
-      title: 'FSQA Manager',
-      company: 'Acme Foods',
-      degree: '3',
-      open_link: false,
-    });
-  });
-
-  it('fails typed when decorated profile data has no recipient identity', () => {
-    expect(() => requireProfileSummary({ data: { defaultPosition: { title: 'FSQA' } } }))
-      .toThrow(CommandExecutionError);
-  });
-
-  it('keeps manifest columns aligned with dry-run rows and fails typed on malformed profile API', async () => {
-    const cmd = getRegistry().get('linkedin/salesnav-message');
-    expect(cmd?.columns).toEqual([
-      'status',
-      'recipient',
-      'title',
-      'company',
-      'credits_remaining',
-      'credits_before',
-      'credits_after',
-      'sent_in_salesnav',
-      'message_chars',
-      'subject_chars',
-      'recipient_urn',
-      'degree',
-      'inmail_restriction',
-      'open_link',
-    ]);
-
-    const goodPage = createPageMock([
-      { status: 200, json: { data: { fullName: 'Jane Doe', defaultPosition: { title: 'QA', companyName: 'Acme' }, degree: 2, inmailRestriction: 'NO_RESTRICTION', memberBadges: { openLink: true } } } },
-      { status: 200, json: { elements: [{ type: 'LSS_INMAIL', value: 12 }] } },
-    ]);
-    const rows = await cmd.func(goodPage, {
-      recipient: 'urn:li:fs_salesProfile:(P1,NAME_SEARCH,T1)',
-      subject: 'Hello',
-      body: 'Quick question',
-    });
-    expect(Object.keys(rows[0]).sort()).toEqual([...cmd.columns].sort());
-    expect(rows[0]).toMatchObject({
-      status: 'validated_dry_run',
-      recipient: 'Jane Doe',
-      credits_remaining: 12,
-      credits_before: 12,
-      credits_after: '',
-      sent_in_salesnav: false,
-      degree: '2',
-      inmail_restriction: 'NO_RESTRICTION',
-      open_link: true,
-    });
-
-    const malformedPage = createPageMock([
-      { status: 200, json: { data: { defaultPosition: { title: 'QA' } } } },
-    ]);
-    await expect(cmd.func(malformedPage, {
-      recipient: 'urn:li:fs_salesProfile:(P1,NAME_SEARCH,T1)',
-      subject: 'Hello',
-      body: 'Quick question',
-    })).rejects.toThrow(CommandExecutionError);
-  });
-});

package/clis/linkedin/salesnav-search.test.js

@@ -1,76 +0,0 @@
-import { describe, expect, it } from 'vitest';
-import { CommandExecutionError } from '@sovovs/bycli/errors';
-import './salesnav-search.js';
-
-const {
-  parseLimit,
-  leadSearchUrl,
-  profileUrlFromEntityUrn,
-  leadUrlFromEntityUrn,
-  parseLeads,
-  requireLeadSearchResult,
-} = await import('./salesnav-search.js').then((m) => m.__test__);
-
-describe('linkedin salesnav-search command', () => {
-  it('builds a salesApiLeadSearch URL with encoded keywords and pagination', () => {
-    const url = leadSearchUrl('quality manager food', 50);
-    expect(url).toContain('/sales-api/salesApiLeadSearch');
-    expect(url).toContain('keywords:quality%20manager%20food');
-    expect(url).toContain('start=50');
-    expect(url).toContain('count=25');
-  });
-
-  it('derives a profile URL from the sales-profile entityUrn token', () => {
-    expect(profileUrlFromEntityUrn('urn:li:fs_salesProfile:(ACwAAAJS8TABxyz,NAME_SEARCH,Enlo)'))
-      .toBe('https://www.linkedin.com/in/ACwAAAJS8TABxyz');
-    expect(profileUrlFromEntityUrn('')).toBe('');
-    expect(profileUrlFromEntityUrn('not-a-urn')).toBe('');
-  });
-
-  it('derives a Sales Navigator lead URL from the full sales-profile entityUrn', () => {
-    expect(leadUrlFromEntityUrn('urn:li:fs_salesProfile:(ACwAAAJS8TABxyz,NAME_SEARCH,Enlo)'))
-      .toBe('https://www.linkedin.com/sales/lead/ACwAAAJS8TABxyz,NAME_SEARCH,Enlo');
-    expect(leadUrlFromEntityUrn('not-a-urn')).toBe('');
-  });
-
-  it('validates --limit without silent clamping', () => {
-    expect(parseLimit(undefined)).toBe(25);
-    expect(parseLimit(120)).toBe(120);
-    expect(() => parseLimit(0)).toThrow();
-    expect(() => parseLimit(999)).toThrow();
-    expect(() => parseLimit('abc')).toThrow();
-  });
-
-  it('parses lead rows and falls back to past positions', () => {
-    const json = { elements: [
-      { fullName: 'Jane Q', geoRegion: 'Vancouver, BC', degree: 2,
-        entityUrn: 'urn:li:fs_salesProfile:(TOKEN1,NAME_SEARCH,abc)',
-        currentPositions: [{ title: 'QA Manager', companyName: 'Acme Foods' }] },
-      { fullName: 'No Current', geoRegion: 'Toronto',
-        entityUrn: 'urn:li:fs_salesProfile:(TOKEN2,NAME_SEARCH,def)',
-        currentPositions: [], pastPositions: [{ title: 'Past QA Lead', companyName: 'Old Co' }] },
-    ] };
-    const leads = parseLeads(json);
-    expect(leads).toHaveLength(2);
-    expect(leads[0]).toMatchObject({
-      name: 'Jane Q',
-      title: 'QA Manager',
-      company: 'Acme Foods',
-      location: 'Vancouver, BC',
-      profile_url: 'https://www.linkedin.com/in/TOKEN1',
-      lead_url: 'https://www.linkedin.com/sales/lead/TOKEN1,NAME_SEARCH,abc',
-      recipient_urn: 'urn:li:fs_salesProfile:(TOKEN1,NAME_SEARCH,abc)',
-    });
-    expect(leads[1]).toMatchObject({ name: 'No Current', title: 'Past QA Lead', company: 'Old Co' });
-  });
-
-  it('fails typed on malformed lead payloads instead of silently dropping rows', () => {
-    expect(() => parseLeads({})).toThrow(CommandExecutionError);
-    expect(() => parseLeads({ elements: [{ firstName: '', lastName: '', entityUrn: 'urn:li:fs_salesProfile:(TOKEN3,x,y)' }] }))
-      .toThrow(CommandExecutionError);
-    expect(() => parseLeads({ elements: [{ fullName: 'No Identity' }] }))
-      .toThrow(CommandExecutionError);
-    expect(() => requireLeadSearchResult({ error: 'HTTP 500' })).toThrow(CommandExecutionError);
-    expect(() => requireLeadSearchResult({})).toThrow(CommandExecutionError);
-  });
-});