@sovovs/bycli 1.8.0 → 2.0.0

package/clis/twitter/unlike.test.js

@@ -1,75 +0,0 @@
-import { describe, expect, it } from 'vitest';
-import { ArgumentError, CommandExecutionError } from '@sovovs/bycli/errors';
-import { getRegistry } from '@sovovs/bycli/registry';
-import './unlike.js';
-import { createPageMock } from '../test-utils.js';
-
-describe('twitter unlike command', () => {
-    it('navigates to the tweet URL and reports success when the unlike script confirms', async () => {
-        const cmd = getRegistry().get('twitter/unlike');
-        expect(cmd?.func).toBeTypeOf('function');
-        const page = createPageMock([
-            { ok: true, message: 'Tweet successfully unliked.' },
-        ]);
-        const result = await cmd.func(page, {
-            url: 'https://x.com/alice/status/2040254679301718161',
-        });
-        expect(page.goto).toHaveBeenCalledWith('https://x.com/alice/status/2040254679301718161');
-        expect(page.wait).toHaveBeenNthCalledWith(1, { selector: '[data-testid="primaryColumn"]' });
-        // After ok:true the adapter waits an extra 2s for the network round-trip.
-        expect(page.wait).toHaveBeenNthCalledWith(2, 2);
-        const script = page.evaluate.mock.calls[0][0];
-        // Idempotency check: looks for the like button (already-not-liked path) before clicking unlike.
-        expect(script).toContain("targetArticle?.querySelector('[data-testid=\"like\"]')");
-        expect(script).toContain("targetArticle?.querySelector('[data-testid=\"unlike\"]')");
-        expect(script).toContain('unlikeBtn.click()');
-        // Article scoping comes from the shared helper (buildTwitterArticleScopeSource):
-        // emits __twHasLinkToTarget + __twGetStatusIdFromHref + the anchored
-        // tweet-path regex. JSDOM-level coverage lives in shared.test.js.
-        expect(script).toContain('__twHasLinkToTarget');
-        expect(script).toContain('__twGetStatusIdFromHref');
-        expect(script).toContain("document.querySelectorAll('article')");
-        expect(result).toEqual([
-            { status: 'success', message: 'Tweet successfully unliked.' },
-        ]);
-    });
-
-    it('returns a failed row without re-waiting when the unlike script reports a UI mismatch', async () => {
-        const cmd = getRegistry().get('twitter/unlike');
-        expect(cmd?.func).toBeTypeOf('function');
-        const page = createPageMock([
-            {
-                ok: false,
-                message: 'Could not find the Unlike button on this tweet after waiting 10 seconds. Are you logged in?',
-            },
-        ]);
-        const result = await cmd.func(page, {
-            url: 'https://x.com/alice/status/2040254679301718161',
-        });
-        expect(result).toEqual([
-            {
-                status: 'failed',
-                message: 'Could not find the Unlike button on this tweet after waiting 10 seconds. Are you logged in?',
-            },
-        ]);
-        // Only the primaryColumn wait should run when ok is false.
-        expect(page.wait).toHaveBeenCalledTimes(1);
-    });
-
-    it('throws CommandExecutionError when no page is provided', async () => {
-        const cmd = getRegistry().get('twitter/unlike');
-        await expect(cmd.func(undefined, {
-            url: 'https://x.com/alice/status/2040254679301718161',
-        })).rejects.toThrow(CommandExecutionError);
-    });
-
-    it('rejects invalid tweet URLs before navigation', async () => {
-        const cmd = getRegistry().get('twitter/unlike');
-        const page = createPageMock([]);
-        await expect(cmd.func(page, {
-            url: 'https://x.com/alice/status/2040254679301718161/photo/1',
-        })).rejects.toThrow(ArgumentError);
-        expect(page.goto).not.toHaveBeenCalled();
-        expect(page.evaluate).not.toHaveBeenCalled();
-    });
-});

package/clis/twitter/unretweet.test.js

@@ -1,73 +0,0 @@
-import { describe, expect, it } from 'vitest';
-import { ArgumentError, CommandExecutionError } from '@sovovs/bycli/errors';
-import { getRegistry } from '@sovovs/bycli/registry';
-import './unretweet.js';
-import { createPageMock } from '../test-utils.js';
-
-describe('twitter unretweet command', () => {
-    it('clicks the unretweet button then the confirm menu item and reports success', async () => {
-        const cmd = getRegistry().get('twitter/unretweet');
-        expect(cmd?.func).toBeTypeOf('function');
-        const page = createPageMock([
-            { ok: true, message: 'Tweet successfully unretweeted.' },
-        ]);
-        const result = await cmd.func(page, {
-            url: 'https://x.com/alice/status/2040254679301718161',
-        });
-        expect(page.goto).toHaveBeenCalledWith('https://x.com/alice/status/2040254679301718161');
-        expect(page.wait).toHaveBeenNthCalledWith(1, { selector: '[data-testid="primaryColumn"]' });
-        expect(page.wait).toHaveBeenNthCalledWith(2, 2);
-        const script = page.evaluate.mock.calls[0][0];
-        // Two-step UI flow must be present:
-        //   1) click the unretweet button
-        //   2) wait for and click the confirm menu item (data-testid="unretweetConfirm")
-        expect(script).toContain('unretweetBtn.click()');
-        expect(script).toContain("document.querySelector('[data-testid=\"unretweetConfirm\"]')");
-        expect(script).toContain('confirmBtn.click()');
-        // Article scoping comes from the shared helper (buildTwitterArticleScopeSource):
-        // emits __twHasLinkToTarget + __twGetStatusIdFromHref + the anchored
-        // tweet-path regex. JSDOM-level coverage lives in shared.test.js.
-        expect(script).toContain('__twHasLinkToTarget');
-        expect(script).toContain('__twGetStatusIdFromHref');
-        expect(script).toContain("document.querySelectorAll('article')");
-        expect(script).toContain("targetArticle?.querySelector('[data-testid=\"unretweet\"]')");
-        // Idempotency probe: when already not retweeted ([data-testid="retweet"] present),
-        // the script returns ok:true with an "already removed" message.
-        expect(script).toContain("targetArticle?.querySelector('[data-testid=\"retweet\"]')");
-        expect(result).toEqual([
-            { status: 'success', message: 'Tweet successfully unretweeted.' },
-        ]);
-    });
-
-    it('returns a failed row when the confirm menu item never appears', async () => {
-        const cmd = getRegistry().get('twitter/unretweet');
-        expect(cmd?.func).toBeTypeOf('function');
-        const page = createPageMock([
-            { ok: false, message: 'Unretweet menu opened but the confirm option did not appear.' },
-        ]);
-        const result = await cmd.func(page, {
-            url: 'https://x.com/alice/status/2040254679301718161',
-        });
-        expect(result).toEqual([
-            { status: 'failed', message: 'Unretweet menu opened but the confirm option did not appear.' },
-        ]);
-        expect(page.wait).toHaveBeenCalledTimes(1);
-    });
-
-    it('throws CommandExecutionError when no page is provided', async () => {
-        const cmd = getRegistry().get('twitter/unretweet');
-        await expect(cmd.func(undefined, {
-            url: 'https://x.com/alice/status/2040254679301718161',
-        })).rejects.toThrow(CommandExecutionError);
-    });
-
-    it('rejects invalid tweet URLs before navigation', async () => {
-        const cmd = getRegistry().get('twitter/unretweet');
-        const page = createPageMock([]);
-        await expect(cmd.func(page, {
-            url: 'http://x.com/alice/status/2040254679301718161',
-        })).rejects.toThrow(ArgumentError);
-        expect(page.goto).not.toHaveBeenCalled();
-        expect(page.evaluate).not.toHaveBeenCalled();
-    });
-});

package/clis/twitter/utils.test.js

@@ -1,169 +0,0 @@
-import { describe, expect, it } from 'vitest';
-import { __test__ } from './utils.js';
-
-const { computeEngagementScore, applyTopByEngagement, ENGAGEMENT_WEIGHTS } = __test__;
-
-describe('computeEngagementScore', () => {
-    it('returns 0 for empty / nullish rows', () => {
-        expect(computeEngagementScore(null)).toBe(0);
-        expect(computeEngagementScore(undefined)).toBe(0);
-        expect(computeEngagementScore({})).toBe(0);
-    });
-
-    it('weights likes ×1', () => {
-        expect(computeEngagementScore({ likes: 10 })).toBe(10);
-    });
-
-    it('weights retweets ×3', () => {
-        expect(computeEngagementScore({ retweets: 5 })).toBe(15);
-    });
-
-    it('weights replies ×2', () => {
-        expect(computeEngagementScore({ replies: 4 })).toBe(8);
-    });
-
-    it('weights bookmarks ×5', () => {
-        expect(computeEngagementScore({ bookmarks: 6 })).toBe(30);
-    });
-
-    it('log-dampens views (log10(v+1) × 0.5)', () => {
-        // log10(99+1) * 0.5 = 1.0
-        expect(computeEngagementScore({ views: 99 })).toBeCloseTo(1.0, 2);
-        // log10(0+1) * 0.5 = 0
-        expect(computeEngagementScore({ views: 0 })).toBe(0);
-    });
-
-    it('coerces string-typed views (search/timeline returns views as a string)', () => {
-        // log10(9999+1) * 0.5 = 2.0
-        expect(computeEngagementScore({ views: '9999' })).toBeCloseTo(2.0, 2);
-    });
-
-    it('treats non-numeric strings as 0 instead of NaN-poisoning the score', () => {
-        expect(computeEngagementScore({ likes: 'abc', retweets: 5 })).toBe(15);
-    });
-
-    it('clamps negative values at 0 (defensive against bogus payloads)', () => {
-        expect(computeEngagementScore({ likes: -100, retweets: 2 })).toBe(6);
-    });
-
-    it('combines all signals additively', () => {
-        // likes×1 + retweets×3 + replies×2 + bookmarks×5 + log10(views+1)×0.5
-        // 10 + 30 + 8 + 25 + log10(1000)*0.5 = 73 + 1.5 = 74.5
-        const row = { likes: 10, retweets: 10, replies: 4, bookmarks: 5, views: 999 };
-        expect(computeEngagementScore(row)).toBeCloseTo(74.5, 2);
-    });
-
-    it('rounds to 2 decimal places for stable test fixtures', () => {
-        // log10(1+1) * 0.5 = 0.5 * log10(2) ≈ 0.150515
-        const score = computeEngagementScore({ views: 1 });
-        expect(score).toBe(0.15);
-    });
-
-    it('handles real search-row shape (no replies/bookmarks columns)', () => {
-        const searchRow = {
-            id: '123',
-            author: 'alice',
-            text: 'hi',
-            likes: 100,
-            views: '9999',
-        };
-        // 100 + log10(10000)*0.5 = 100 + 2.0 = 102.0
-        expect(computeEngagementScore(searchRow)).toBeCloseTo(102.0, 2);
-    });
-
-    it('handles real bookmarks-row shape (no views/replies columns)', () => {
-        const bookmarkRow = {
-            id: '123',
-            author: 'alice',
-            text: 'hi',
-            likes: 50,
-            retweets: 10,
-            bookmarks: 3,
-        };
-        // 50 + 30 + 15 = 95
-        expect(computeEngagementScore(bookmarkRow)).toBe(95);
-    });
-
-    it('exposes the documented weight table', () => {
-        expect(ENGAGEMENT_WEIGHTS).toEqual({
-            likes: 1,
-            retweets: 3,
-            replies: 2,
-            bookmarks: 5,
-            viewsLog: 0.5,
-        });
-    });
-});
-
-describe('applyTopByEngagement', () => {
-    const rows = [
-        { id: 'a', likes: 10 },
-        { id: 'b', likes: 50 },
-        { id: 'c', likes: 30 },
-        { id: 'd', likes: 100 },
-        { id: 'e', likes: 5 },
-    ];
-
-    it('returns rows unchanged when topN is 0 (default)', () => {
-        expect(applyTopByEngagement(rows, 0)).toBe(rows);
-    });
-
-    it('returns rows unchanged when topN is negative', () => {
-        expect(applyTopByEngagement(rows, -3)).toBe(rows);
-    });
-
-    it('returns rows unchanged when topN is non-numeric', () => {
-        expect(applyTopByEngagement(rows, 'foo')).toBe(rows);
-        expect(applyTopByEngagement(rows, null)).toBe(rows);
-        expect(applyTopByEngagement(rows, undefined)).toBe(rows);
-    });
-
-    it('sorts descending by score and trims to top N when topN > 0', () => {
-        const result = applyTopByEngagement(rows, 3);
-        expect(result.map(r => r.id)).toEqual(['d', 'b', 'c']);
-    });
-
-    it('returns all rows when topN exceeds row count', () => {
-        const result = applyTopByEngagement(rows, 99);
-        expect(result.map(r => r.id)).toEqual(['d', 'b', 'c', 'a', 'e']);
-    });
-
-    it('floors fractional topN', () => {
-        const result = applyTopByEngagement(rows, 2.9);
-        expect(result.map(r => r.id)).toEqual(['d', 'b']);
-    });
-
-    it('is stable for ties (preserves original order)', () => {
-        const tieRows = [
-            { id: 'first', likes: 10 },
-            { id: 'second', likes: 10 },
-            { id: 'third', likes: 10 },
-            { id: 'fourth', likes: 100 },
-        ];
-        const result = applyTopByEngagement(tieRows, 4);
-        // 'fourth' first, then ties retain original order
-        expect(result.map(r => r.id)).toEqual(['fourth', 'first', 'second', 'third']);
-    });
-
-    it('handles empty / non-array input gracefully', () => {
-        expect(applyTopByEngagement([], 5)).toEqual([]);
-        expect(applyTopByEngagement(null, 5)).toBeNull();
-        expect(applyTopByEngagement(undefined, 5)).toBeUndefined();
-    });
-
-    it('does not mutate the input array', () => {
-        const before = [...rows];
-        applyTopByEngagement(rows, 2);
-        expect(rows).toEqual(before);
-    });
-
-    it('mixes signals correctly when ranking', () => {
-        // bookmark-heavy row should beat like-heavy row even if likes are higher
-        const mixed = [
-            { id: 'likes-only', likes: 100 },           // score = 100
-            { id: 'bookmark-heavy', likes: 30, bookmarks: 20 }, // score = 30 + 100 = 130
-        ];
-        const result = applyTopByEngagement(mixed, 2);
-        expect(result.map(r => r.id)).toEqual(['bookmark-heavy', 'likes-only']);
-    });
-});

package/clis/uisdc/news.test.js

@@ -1,66 +0,0 @@
-import { JSDOM } from 'jsdom';
-import { describe, expect, it, vi } from 'vitest';
-import { ArgumentError, CommandExecutionError, EmptyResultError } from '@sovovs/bycli/errors';
-import { uisdcNewsCommand, __test__ } from './news.js';
-
-function runBrowserScript(html, script, url = 'https://www.uisdc.com/news') {
-    const dom = new JSDOM(html, { url, runScripts: 'outside-only' });
-    return dom.window.eval(script);
-}
-
-function makePage(evaluateResult) {
-    return {
-        goto: vi.fn().mockResolvedValue(undefined),
-        evaluate: vi.fn().mockResolvedValue(evaluateResult),
-    };
-}
-
-describe('uisdc/news', () => {
-    it('registers stable URL in columns', () => {
-        expect(uisdcNewsCommand.access).toBe('read');
-        expect(uisdcNewsCommand.columns).toEqual(['rank', 'title', 'summary', 'url']);
-    });
-
-    it('validates limit before browser navigation', async () => {
-        const page = makePage({ ok: true, rows: [] });
-        await expect(uisdcNewsCommand.func(page, { limit: 0 })).rejects.toBeInstanceOf(ArgumentError);
-        await expect(uisdcNewsCommand.func(page, { limit: 51 })).rejects.toBeInstanceOf(ArgumentError);
-        expect(page.goto).not.toHaveBeenCalled();
-    });
-
-    it('extracts rows from the UISDC news DOM', async () => {
-        const html = `
-          <div class="news-list">
-            <div class="news-item">
-              <div class="item-content">
-                <div class="dubao-items">
-                  <div class="dubao-item">
-                    <a href="/article-1"><span class="dubao-title"> AI design news </span></a>
-                    <div class="dubao-content"> summary text </div>
-                  </div>
-                </div>
-              </div>
-            </div>
-          </div>
-        `;
-        const payload = runBrowserScript(html, __test__.buildExtractUisdcNewsJs());
-        const page = makePage(payload);
-
-        const rows = await uisdcNewsCommand.func(page, { limit: 10 });
-
-        expect(page.goto).toHaveBeenCalledWith('https://www.uisdc.com/news', { waitUntil: 'load', settleMs: 3000 });
-        expect(rows).toEqual([{
-            rank: 1,
-            title: 'AI design news',
-            summary: 'summary text',
-            url: 'https://www.uisdc.com/article-1',
-        }]);
-    });
-
-    it('maps selector drift and empty rows to typed errors', async () => {
-        await expect(uisdcNewsCommand.func(makePage({ ok: false, reason: 'selector-missing' }), { limit: 1 }))
-            .rejects.toBeInstanceOf(CommandExecutionError);
-        await expect(uisdcNewsCommand.func(makePage({ ok: true, rows: [{ title: '', url: '' }] }), { limit: 1 }))
-            .rejects.toBeInstanceOf(EmptyResultError);
-    });
-});

package/clis/uiverse/_shared.test.js

@@ -1,62 +0,0 @@
-import { describe, expect, it } from 'vitest';
-import {
-  UIVERSE_BASE_URL,
-  parseComponentInput,
-  parseHtmlRootSignature,
-  getPreviewFallbackTags,
-  inferLanguage,
-  getCodeLength,
-} from './_shared.js';
-
-describe('uiverse shared helpers', () => {
-  it('parses full URLs and author/slug identifiers', () => {
-    expect(parseComponentInput('Galahhad/strong-squid-82')).toEqual({
-      raw: 'Galahhad/strong-squid-82',
-      username: 'Galahhad',
-      slug: 'strong-squid-82',
-      url: `${UIVERSE_BASE_URL}/Galahhad/strong-squid-82`,
-    });
-
-    expect(parseComponentInput('https://uiverse.io/Galahhad/strong-squid-82')).toEqual({
-      raw: 'https://uiverse.io/Galahhad/strong-squid-82',
-      username: 'Galahhad',
-      slug: 'strong-squid-82',
-      url: `${UIVERSE_BASE_URL}/Galahhad/strong-squid-82`,
-    });
-  });
-
-  it('rejects unsupported hosts and malformed identifiers', () => {
-    expect(() => parseComponentInput('https://example.com/foo/bar')).toThrow('Unsupported non-Uiverse URL');
-    expect(() => parseComponentInput('only-author')).toThrow('Could not parse author/slug');
-    expect(() => parseComponentInput('a/b/c')).toThrow('Could not parse author/slug');
-  });
-
-  it('parses the HTML root signature', () => {
-    expect(parseHtmlRootSignature('<label id="x" class="theme-switch primary"></label>')).toEqual({
-      tag: 'label',
-      id: 'x',
-      classes: ['theme-switch', 'primary'],
-    });
-    expect(parseHtmlRootSignature('')).toEqual({ tag: null, id: null, classes: [] });
-  });
-
-  it('uses broader preview fallback tags for input roots', () => {
-    expect(getPreviewFallbackTags({ tag: 'input' })).toEqual(['input', 'label', 'button', 'a', 'div']);
-    expect(getPreviewFallbackTags({ tag: 'label' })).toEqual(['label']);
-    expect(getPreviewFallbackTags({ tag: null })).toEqual(['label', 'button', 'a', 'div']);
-  });
-
-  it('infers the language from target and metadata', () => {
-    expect(inferLanguage('react', {})).toBe('tsx');
-    expect(inferLanguage('vue', {})).toBe('vue');
-    expect(inferLanguage('html', { isTailwind: true })).toBe('html+tailwind');
-    expect(inferLanguage('css', {})).toBe('css');
-    expect(inferLanguage('unknown', {})).toBe('text');
-  });
-
-  it('returns the code length safely', () => {
-    expect(getCodeLength('abc')).toBe(3);
-    expect(getCodeLength('')).toBe(0);
-    expect(getCodeLength(null)).toBe(0);
-  });
-});

package/clis/uiverse/navigation.test.js

@@ -1,12 +0,0 @@
-import { describe, expect, it } from 'vitest';
-import { getRegistry } from '@sovovs/bycli/registry';
-import './code.js';
-import './preview.js';
-
-describe('uiverse navigateBefore hardening', () => {
-  it.each(['uiverse/code', 'uiverse/preview'])('%s starts from uiverse home instead of about:blank', (name) => {
-    const cmd = getRegistry().get(name);
-    expect(cmd).toBeDefined();
-    expect(cmd.navigateBefore).toBe('https://uiverse.io');
-  });
-});