@sovovs/bycli 1.8.0 → 2.0.0

package/clis/reddit/read.test.js

@@ -1,335 +0,0 @@
-import { describe, expect, it, vi } from 'vitest';
-import { getRegistry } from '@sovovs/bycli/registry';
-import { ArgumentError, AuthRequiredError, CommandExecutionError, EmptyResultError } from '@sovovs/bycli/errors';
-import { normalizeRedditPostId, parseExpandRounds } from './read.js';
-import './read.js';
-
-function makePage(result) {
-    return {
-        goto: vi.fn().mockResolvedValue(undefined),
-        evaluate: vi.fn().mockResolvedValue(result),
-    };
-}
-
-function redditPostEnvelope(children) {
-    return [
-        {
-            data: {
-                children: [{
-                    data: {
-                        title: 'Post title',
-                        selftext: '',
-                        author: 'op',
-                        score: 10,
-                        is_self: true,
-                    },
-                }],
-            },
-        },
-        { data: { children } },
-    ];
-}
-
-function commentThing(id, body, parent = 't3_abc123', score = 1) {
-    return {
-        kind: 't1',
-        data: {
-            id,
-            name: `t1_${id}`,
-            parent_id: parent,
-            author: id,
-            score,
-            body,
-            replies: '',
-        },
-    };
-}
-
-function moreThing(id, children, parent = 't3_abc123', count = children.length) {
-    return {
-        kind: 'more',
-        data: { id, parent_id: parent, children, count },
-    };
-}
-
-function jsonResponse(payload, status = 200) {
-    return {
-        ok: status >= 200 && status < 300,
-        status,
-        json: vi.fn().mockResolvedValue(payload),
-    };
-}
-
-function makeRuntimePage(fetchImpl) {
-    return {
-        goto: vi.fn().mockResolvedValue(undefined),
-        evaluate: vi.fn(async (script) => {
-            const previousFetch = globalThis.fetch;
-            globalThis.fetch = fetchImpl;
-            try {
-                return await eval(script);
-            } finally {
-                globalThis.fetch = previousFetch;
-            }
-        }),
-    };
-}
-
-describe('reddit read adapter', () => {
-    const command = getRegistry().get('reddit/read');
-
-    it('uses an ephemeral Reddit site tab by default', () => {
-        expect(command?.browser).toBe(true);
-        expect(command?.siteSession).toBeUndefined();
-        expect(command?.columns).toEqual(['type', 'author', 'score', 'text']);
-    });
-
-    it('exposes the new --expand-more / --expand-rounds args', () => {
-        const argNames = command.args.map((a) => a.name);
-        expect(argNames).toContain('expand-more');
-        expect(argNames).toContain('expand-rounds');
-        const expandMore = command.args.find((a) => a.name === 'expand-more');
-        expect(expandMore.type).toBe('bool');
-        expect(expandMore.default).toBe(false);
-        const rounds = command.args.find((a) => a.name === 'expand-rounds');
-        expect(rounds.type).toBe('int');
-        expect(rounds.default).toBe(2);
-    });
-
-    describe('normalizeRedditPostId', () => {
-        it('accepts bare ids, t3 fullnames, and exact reddit post URLs', () => {
-            expect(normalizeRedditPostId('1AbC23')).toBe('1abc23');
-            expect(normalizeRedditPostId('t3_1AbC23')).toBe('1abc23');
-            expect(normalizeRedditPostId('https://www.reddit.com/r/bycli/comments/1abc23/title_slug/?sort=top')).toBe('1abc23');
-            expect(normalizeRedditPostId('https://www.reddit.com/r/bycli/comments/1abc23/title_slug/okf3s7u/?context=3')).toBe('1abc23');
-            expect(normalizeRedditPostId('https://old.reddit.com/comments/1abc23/title_slug/')).toBe('1abc23');
-        });
-
-        it('rejects invalid or structurally loose post identities before navigation', () => {
-            for (const bad of [
-                '',
-                't1_okf3s7u',
-                'https://reddit.com.evil.com/r/bycli/comments/1abc23/title_slug/',
-                'http://www.reddit.com/r/bycli/comments/1abc23/title_slug/',
-                'https://www.reddit.com/r/bycli/comments/',
-                'https://www.reddit.com/r/bycli/comments/1abc23/title_slug/okf3s7u/evil',
-                'not/a/post',
-            ]) {
-                expect(() => normalizeRedditPostId(bad)).toThrow(ArgumentError);
-            }
-        });
-    });
-
-    describe('parseExpandRounds', () => {
-        it('returns the default for absent input but throws on out-of-range / non-integer', () => {
-            expect(parseExpandRounds(undefined)).toBe(2);
-            expect(parseExpandRounds(null)).toBe(2);
-            expect(parseExpandRounds('')).toBe(2);
-            expect(parseExpandRounds(1)).toBe(1);
-            expect(parseExpandRounds(5)).toBe(5);
-            for (const bad of [0, -1, 6, 1.5, NaN, 'abc']) {
-                expect(() => parseExpandRounds(bad)).toThrow(ArgumentError);
-            }
-        });
-    });
-
-    it('rejects a bad --expand-rounds BEFORE navigating', async () => {
-        const page = makePage({ kind: 'ok', rows: [] });
-        await expect(command.func(page, { 'post-id': 'abc123', 'expand-rounds': 99 }))
-            .rejects.toBeInstanceOf(ArgumentError);
-        expect(page.goto).not.toHaveBeenCalled();
-        expect(page.evaluate).not.toHaveBeenCalled();
-    });
-
-    it('rejects a bad post identity BEFORE navigating', async () => {
-        const page = makePage({ kind: 'ok', rows: [] });
-        await expect(command.func(page, { 'post-id': 'https://evil.test/r/x/comments/abc/title/' }))
-            .rejects.toBeInstanceOf(ArgumentError);
-        expect(page.goto).not.toHaveBeenCalled();
-        expect(page.evaluate).not.toHaveBeenCalled();
-    });
-
-    it('returns rows when the evaluate script reports kind=ok', async () => {
-        const page = makePage({
-            kind: 'ok',
-            rows: [
-                { type: 'POST', author: 'alice', score: 10, text: 'Title' },
-                { type: 'L0', author: 'bob', score: 5, text: 'Comment' },
-            ],
-            expandMeta: { rounds: 0, fetched: 0, capped: false, errors: [] },
-        });
-        const result = await command.func(page, { 'post-id': 'abc123', limit: 5 });
-        expect(page.goto).toHaveBeenCalledWith('https://www.reddit.com');
-        expect(result).toEqual([
-            { type: 'POST', author: 'alice', score: 10, text: 'Title' },
-            { type: 'L0', author: 'bob', score: 5, text: 'Comment' },
-        ]);
-    });
-
-    it('maps the five failure kinds to the right typed errors', async () => {
-        await expect(command.func(makePage({ kind: 'inaccessible', detail: 'post 403' }), { 'post-id': 'abc123' }))
-            .rejects.toBeInstanceOf(EmptyResultError);
-
-        await expect(command.func(makePage({ kind: 'auth', detail: 'morechildren 401' }), { 'post-id': 'abc123' }))
-            .rejects.toBeInstanceOf(AuthRequiredError);
-
-        await expect(command.func(makePage({ kind: 'http', httpStatus: 503, where: '/comments/abc.json' }), { 'post-id': 'abc123' }))
-            .rejects.toBeInstanceOf(CommandExecutionError);
-
-        await expect(command.func(makePage({ kind: 'malformed', detail: 'no comment listing' }), { 'post-id': 'abc123' }))
-            .rejects.toBeInstanceOf(CommandExecutionError);
-
-        await expect(command.func(makePage({ kind: 'parser-drift', detail: 'walker drift' }), { 'post-id': 'abc123' }))
-            .rejects.toBeInstanceOf(CommandExecutionError);
-
-        await expect(command.func(makePage({ kind: 'expand-failed', detail: 'morechildren errors' }), { 'post-id': 'abc123' }))
-            .rejects.toBeInstanceOf(CommandExecutionError);
-    });
-
-    it('throws CommandExecutionError on an unknown envelope shape (no kind)', async () => {
-        await expect(command.func(makePage({ random: 'stuff' }), { 'post-id': 'abc123' }))
-            .rejects.toBeInstanceOf(CommandExecutionError);
-        await expect(command.func(makePage(null), { 'post-id': 'abc123' }))
-            .rejects.toBeInstanceOf(CommandExecutionError);
-    });
-
-    it('embeds expandMore=false by default and inlines flags into the evaluate script', async () => {
-        const page = makePage({ kind: 'ok', rows: [], expandMeta: { rounds: 0, fetched: 0, capped: false, errors: [] } });
-        await command.func(page, { 'post-id': 'xyz', sort: 'top', limit: 3 });
-        const script = page.evaluate.mock.calls[0][0];
-        expect(script).toContain('var expandMore = false');
-        expect(script).toContain('var expandRounds = 2');
-        expect(script).toContain('var sort = "top"');
-        expect(script).toContain('var limit = 3');
-        expect(script).toContain('var postId = "xyz"');
-    });
-
-    it('embeds expandMore=true and the requested expandRounds when --expand-more is on', async () => {
-        const page = makePage({ kind: 'ok', rows: [], expandMeta: { rounds: 3, fetched: 12, capped: true, errors: [] } });
-        await command.func(page, { 'post-id': 'xyz', 'expand-more': true, 'expand-rounds': 3 });
-        const script = page.evaluate.mock.calls[0][0];
-        expect(script).toContain('var expandMore = true');
-        expect(script).toContain('var expandRounds = 3');
-        // The /api/morechildren request body construction must be present in
-        // the evaluate script (round-trips the link_id + children CSV).
-        expect(script).toContain("'/api/morechildren'");
-        expect(script).toContain("'api_type=json'");
-        expect(script).toContain("encodeURIComponent(linkFullname)");
-        expect(script).toContain("encodeURIComponent(batch.join(','))");
-    });
-
-    it('normalizes a full reddit URL before building the browser script', async () => {
-        const page = makePage({ kind: 'ok', rows: [], expandMeta: { rounds: 0, fetched: 0, capped: false, errors: [] } });
-        await command.func(page, { 'post-id': 'https://www.reddit.com/r/python/comments/1abc23/title_slug/' });
-        const script = page.evaluate.mock.calls[0][0];
-        expect(script).toContain('var postId = "1abc23"');
-        expect(script).not.toContain('postIdRaw.match');
-    });
-
-    it('expands morechildren in the original tree position instead of appending to the parent', async () => {
-        const fetchMock = vi.fn(async (url) => {
-            if (String(url).startsWith('/comments/')) {
-                return jsonResponse(redditPostEnvelope([
-                    commentThing('a', 'A'),
-                    moreThing('more_top', ['b', 'c']),
-                    commentThing('d', 'D'),
-                ]));
-            }
-            if (String(url) === '/api/morechildren') {
-                return jsonResponse({
-                    json: {
-                        errors: [],
-                        data: { things: [commentThing('b', 'B'), commentThing('c', 'C')] },
-                    },
-                });
-            }
-            throw new Error(`unexpected URL ${url}`);
-        });
-        const page = makeRuntimePage(fetchMock);
-
-        const result = await command.func(page, {
-            'post-id': 'abc123',
-            'expand-more': true,
-            limit: 10,
-            replies: 10,
-        });
-
-        expect(result.map((row) => row.author)).toEqual(['op', 'a', 'b', 'c', 'd']);
-        expect(fetchMock).toHaveBeenCalledWith(
-            '/api/morechildren',
-            expect.objectContaining({
-                method: 'POST',
-                body: expect.stringContaining('link_id=t3_abc123'),
-            }),
-        );
-    });
-
-    it('fails expand-more when Reddit returns a child that cannot be placed in the requested tree', async () => {
-        const fetchMock = vi.fn(async (url) => {
-            if (String(url).startsWith('/comments/')) {
-                return jsonResponse(redditPostEnvelope([moreThing('more_top', ['b'])]));
-            }
-            if (String(url) === '/api/morechildren') {
-                return jsonResponse({
-                    json: {
-                        errors: [],
-                        data: { things: [commentThing('b', 'B', 't3_other')] },
-                    },
-                });
-            }
-            throw new Error(`unexpected URL ${url}`);
-        });
-        const page = makeRuntimePage(fetchMock);
-
-        await expect(command.func(page, {
-            'post-id': 'abc123',
-            'expand-more': true,
-        })).rejects.toBeInstanceOf(CommandExecutionError);
-    });
-
-    it('fails expand-more when Reddit omits a requested child instead of silently dropping the stub', async () => {
-        const fetchMock = vi.fn(async (url) => {
-            if (String(url).startsWith('/comments/')) {
-                return jsonResponse(redditPostEnvelope([moreThing('more_top', ['b', 'c'])]));
-            }
-            if (String(url) === '/api/morechildren') {
-                return jsonResponse({
-                    json: {
-                        errors: [],
-                        data: { things: [commentThing('b', 'B')] },
-                    },
-                });
-            }
-            throw new Error(`unexpected URL ${url}`);
-        });
-        const page = makeRuntimePage(fetchMock);
-
-        await expect(command.func(page, {
-            'post-id': 'abc123',
-            'expand-more': true,
-        })).rejects.toBeInstanceOf(CommandExecutionError);
-    });
-
-    it('uses 5-kind discriminated union keys that DO NOT collide with declared columns', () => {
-        // Read the evaluate template once to assert the intermediate keys we
-        // return on the browser side never name any of `type` / `author` /
-        // `score` / `text` (the declared columns) — that pattern would
-        // trigger the silent-column-drop audit.
-        const page = makePage({ kind: 'ok', rows: [], expandMeta: { rounds: 0, fetched: 0, capped: false, errors: [] } });
-        return command.func(page, { 'post-id': 'xyz' }).then(() => {
-            const script = page.evaluate.mock.calls[0][0];
-            // Each return shape uses kind / detail / httpStatus / where /
-            // rows / expandMeta. None overlap with the four declared
-            // columns. The walker IS allowed to push column-shaped row
-            // objects into `rows` — that's the final shape, not an
-            // intermediate one.
-            expect(script).toContain("kind: 'inaccessible'");
-            expect(script).toContain("kind: 'auth'");
-            expect(script).toContain("kind: 'http'");
-            expect(script).toContain("kind: 'malformed'");
-            expect(script).toContain("kind: 'parser-drift'");
-            expect(script).toContain("kind: 'expand-failed'");
-            expect(script).toContain("kind: 'ok'");
-        });
-    });
-});

package/clis/reddit/reply.test.js

@@ -1,89 +0,0 @@
-import { describe, expect, it, vi } from 'vitest';
-import { getRegistry } from '@sovovs/bycli/registry';
-import { ArgumentError, AuthRequiredError, CommandExecutionError } from '@sovovs/bycli/errors';
-import { normalizeRedditCommentFullname, requireReplyText } from './reply.js';
-import './reply.js';
-
-function makePage(result = { kind: 'ok', detail: 'Reply posted on t1_okf3s7u as t1_reply123' }) {
-    return {
-        goto: vi.fn().mockResolvedValue(undefined),
-        evaluate: vi.fn().mockResolvedValue(result),
-    };
-}
-
-describe('reddit reply command', () => {
-    const command = getRegistry().get('reddit/reply');
-
-    it('normalizes bare ids, fullnames, and exact reddit comment URLs', () => {
-        expect(normalizeRedditCommentFullname('okf3s7u')).toBe('t1_okf3s7u');
-        expect(normalizeRedditCommentFullname('T1_OKF3S7U')).toBe('t1_okf3s7u');
-        expect(normalizeRedditCommentFullname('https://www.reddit.com/r/bycli/comments/1abc23/title_slug/okf3s7u/?context=3')).toBe('t1_okf3s7u');
-        expect(normalizeRedditCommentFullname('https://old.reddit.com/r/bycli/comments/1abc23/title_slug/okf3s7u/')).toBe('t1_okf3s7u');
-    });
-
-    it('rejects invalid or ambiguous comment identities before navigation', async () => {
-        const page = makePage();
-
-        for (const value of [
-            '',
-            't3_1abc23',
-            'abc/def',
-            'https://reddit.com.evil.com/r/bycli/comments/1abc23/title_slug/okf3s7u/',
-            'http://www.reddit.com/r/bycli/comments/1abc23/title_slug/okf3s7u/',
-            'https://www.reddit.com/r/bycli/comments/1abc23/title_slug/',
-            'https://www.reddit.com/r/bycli/comments/1abc23/title_slug/okf3s7u/evil',
-        ]) {
-            await expect(command.func(page, { 'comment-id': value, text: 'hello' })).rejects.toBeInstanceOf(ArgumentError);
-        }
-
-        expect(page.goto).not.toHaveBeenCalled();
-        expect(page.evaluate).not.toHaveBeenCalled();
-    });
-
-    it('rejects blank reply text before navigation', async () => {
-        const page = makePage();
-
-        await expect(command.func(page, { 'comment-id': 'okf3s7u', text: '   ' })).rejects.toBeInstanceOf(ArgumentError);
-
-        expect(page.goto).not.toHaveBeenCalled();
-        expect(page.evaluate).not.toHaveBeenCalled();
-        expect(() => requireReplyText('hello')).not.toThrow();
-    });
-
-    it('posts to the normalized t1 fullname and returns success only on ok result', async () => {
-        const page = makePage();
-
-        const rows = await command.func(page, {
-            'comment-id': 'https://www.reddit.com/r/bycli/comments/1abc23/title_slug/okf3s7u/',
-            text: 'hello',
-        });
-
-        expect(page.goto).toHaveBeenCalledWith('https://www.reddit.com');
-        const script = page.evaluate.mock.calls[0][0];
-        expect(script).toContain('const fullname = "t1_okf3s7u"');
-        expect(script).toContain('const text = "hello"');
-        expect(rows).toEqual([{ status: 'success', message: 'Reply posted on t1_okf3s7u as t1_reply123' }]);
-    });
-
-    it('maps auth, http, reddit, exception, and postcondition failures to typed errors', async () => {
-        await expect(command.func(makePage({ kind: 'auth', detail: 'login required' }), { 'comment-id': 'okf3s7u', text: 'hello' }))
-            .rejects.toBeInstanceOf(AuthRequiredError);
-        await expect(command.func(makePage({ kind: 'http', httpStatus: 500, where: '/api/comment' }), { 'comment-id': 'okf3s7u', text: 'hello' }))
-            .rejects.toBeInstanceOf(CommandExecutionError);
-        await expect(command.func(makePage({ kind: 'reddit-error', detail: 'RATELIMIT: try later' }), { 'comment-id': 'okf3s7u', text: 'hello' }))
-            .rejects.toBeInstanceOf(CommandExecutionError);
-        await expect(command.func(makePage({ kind: 'exception', detail: 'bad json' }), { 'comment-id': 'okf3s7u', text: 'hello' }))
-            .rejects.toBeInstanceOf(CommandExecutionError);
-        await expect(command.func(makePage({ kind: 'postcondition', detail: 'Reddit comment response did not include a created reply id' }), { 'comment-id': 'okf3s7u', text: 'hello' }))
-            .rejects.toBeInstanceOf(CommandExecutionError);
-    });
-
-    it('requires the Reddit response to include a created reply id', async () => {
-        const page = makePage();
-
-        await command.func(page, { 'comment-id': 'okf3s7u', text: 'hello' });
-
-        expect(page.evaluate.mock.calls[0][0]).toContain('Reddit comment response did not include a created reply id');
-        expect(page.evaluate.mock.calls[0][0]).toContain("String(thing?.data?.name || '').startsWith('t1_')");
-    });
-});

package/clis/reddit/search.test.js

@@ -1,26 +0,0 @@
-import { describe, expect, it } from 'vitest';
-import { getRegistry } from '@sovovs/bycli/registry';
-import './search.js';
-
-describe('reddit search adapter', () => {
-  const command = getRegistry().get('reddit/search');
-
-  it('exposes the full search-result shape including the 4 media columns', () => {
-    expect(command?.columns).toEqual([
-      'id', 'title', 'subreddit', 'author', 'score', 'comments', 'url',
-      'created_utc', 'selftext',
-      'post_hint', 'url_overridden_by_dest', 'preview_image_url', 'gallery_urls',
-    ]);
-  });
-
-  it('surfaces media via extractRedditMedia in evaluate + map', () => {
-    expect(command?.pipeline?.[1]?.evaluate).toContain('function extractRedditMedia');
-    expect(command?.pipeline?.[1]?.evaluate).toContain('...extractRedditMedia(c.data)');
-    expect(command?.pipeline?.[2]?.map).toMatchObject({
-      post_hint: '${{ item.post_hint }}',
-      url_overridden_by_dest: '${{ item.url_overridden_by_dest }}',
-      preview_image_url: '${{ item.preview_image_url }}',
-      gallery_urls: '${{ item.gallery_urls }}',
-    });
-  });
-});

package/clis/reddit/subreddit-info.test.js

@@ -1,163 +0,0 @@
-import { describe, expect, it, vi } from 'vitest';
-import { getRegistry } from '@sovovs/bycli/registry';
-import { ArgumentError, CommandExecutionError, EmptyResultError } from '@sovovs/bycli/errors';
-import { parseSubredditName } from './subreddit-info.js';
-import './subreddit-info.js';
-
-function makePage(result) {
-    return {
-        goto: vi.fn().mockResolvedValue(undefined),
-        evaluate: vi.fn().mockResolvedValue(result),
-    };
-}
-
-describe('reddit subreddit-info command', () => {
-    const command = getRegistry().get('reddit/subreddit-info');
-
-    it('registers with the expected shape', () => {
-        expect(command).toBeDefined();
-        expect(command.access).toBe('read');
-        expect(command.browser).toBe(true);
-        expect(command.columns).toEqual(['field', 'value']);
-    });
-
-    it('parseSubredditName strips prefixes and validates the name shape', () => {
-        expect(parseSubredditName('python')).toBe('python');
-        expect(parseSubredditName('r/python')).toBe('python');
-        expect(parseSubredditName('/r/python')).toBe('python');
-        expect(parseSubredditName('  AskReddit  ')).toBe('AskReddit');
-        expect(parseSubredditName('aw3some_sub')).toBe('aw3some_sub');
-    });
-
-    it('parseSubredditName rejects invalid names without silent fallback', () => {
-        for (const bad of [
-            '',
-            '  ',
-            'py',                   // too short
-            '1abc',                 // must start with letter
-            '_sub',                 // must start with letter
-            'has space',            // no spaces
-            'has-dash',             // no dashes
-            'way_too_long_subreddit_name_here', // too long (>21)
-        ]) {
-            expect(() => parseSubredditName(bad)).toThrow(ArgumentError);
-        }
-    });
-
-    it('rejects bad subreddit names BEFORE navigating', async () => {
-        const page = makePage({ kind: 'ok', info: {} });
-        await expect(command.func(page, { name: 'has space' })).rejects.toBeInstanceOf(ArgumentError);
-        expect(page.goto).not.toHaveBeenCalled();
-        expect(page.evaluate).not.toHaveBeenCalled();
-    });
-
-    it('throws EmptyResultError for missing / banned / private / quarantined subreddits', async () => {
-        for (const detail of ['not found', 'banned', 'private', 'quarantined']) {
-            await expect(command.func(makePage({ kind: 'missing', detail }), { name: 'python' }))
-                .rejects.toBeInstanceOf(EmptyResultError);
-        }
-    });
-
-    it('treats HTTP 401/403/404 about.json responses as inaccessible subreddit, not auth-required', async () => {
-        for (const status of [401, 403, 404]) {
-            const scriptResults = [];
-            const page = {
-                goto: vi.fn().mockResolvedValue(undefined),
-                evaluate: vi.fn(async (script) => {
-                    const result = await (new Function('fetch', `return (${script})`))(vi.fn(async () => ({
-                        status,
-                        ok: false,
-                    })));
-                    scriptResults.push(result);
-                    return result;
-                }),
-            };
-            await expect(command.func(page, { name: 'python' })).rejects.toBeInstanceOf(EmptyResultError);
-            expect(scriptResults[0]).toMatchObject({ kind: 'missing' });
-        }
-    });
-
-    it('throws CommandExecutionError on HTTP and exception failure modes', async () => {
-        await expect(command.func(makePage({ kind: 'http', httpStatus: 500, where: 'about.json' }), { name: 'python' }))
-            .rejects.toBeInstanceOf(CommandExecutionError);
-        await expect(command.func(makePage({ kind: 'exception', detail: 'bad' }), { name: 'python' }))
-            .rejects.toBeInstanceOf(CommandExecutionError);
-    });
-
-    it('throws CommandExecutionError for malformed 200 subreddit payloads', async () => {
-        await expect(command.func(makePage({
-            kind: 'malformed',
-            detail: 'Reddit returned malformed subreddit info for r/python (missing data.display_name).',
-        }), { name: 'python' }))
-            .rejects.toMatchObject({
-                code: 'COMMAND_EXEC',
-                message: expect.stringContaining('malformed subreddit info'),
-            });
-    });
-
-    it('maps a normal subreddit payload into typed field/value rows', async () => {
-        const info = {
-            display_name: 'python',
-            display_name_prefixed: 'r/python',
-            title: 'Python',
-            public_description: '  News about the programming language Python.  ',
-            subscribers: 1234567,
-            active_user_count: 4321,
-            over18: false,
-            subreddit_type: 'public',
-            created_utc: 1201881600, // 2008-02-01
-            url: '/r/python/',
-        };
-        const rows = await command.func(makePage({ kind: 'ok', info }), { name: 'python' });
-        const byField = Object.fromEntries(rows.map((r) => [r.field, r.value]));
-
-        expect(byField.Name).toBe('r/python');
-        expect(byField.Title).toBe('Python');
-        expect(byField.Subscribers).toBe('1234567');
-        expect(byField['Active Now']).toBe('4321');
-        expect(byField.NSFW).toBe('No');
-        expect(byField.Type).toBe('public');
-        expect(byField.Description).toBe('News about the programming language Python.');
-        expect(byField.Created).toBe('2008-02-01');
-        expect(byField.URL).toBe('https://www.reddit.com/r/python/');
-
-        for (const row of rows) {
-            expect(Object.keys(row).sort()).toEqual(['field', 'value']);
-        }
-    });
-
-    it('falls back to accounts_active when active_user_count is missing', async () => {
-        const info = {
-            display_name: 'sub',
-            display_name_prefixed: 'r/sub',
-            accounts_active: 99,
-            url: '/r/sub/',
-        };
-        const rows = await command.func(makePage({ kind: 'ok', info }), { name: 'sub' });
-        const byField = Object.fromEntries(rows.map((r) => [r.field, r.value]));
-        expect(byField['Active Now']).toBe('99');
-    });
-
-    it('emits typed null (not "-" sentinel) when subscribers / activity / created are missing', async () => {
-        const info = {
-            display_name: 'sparse',
-            display_name_prefixed: 'r/sparse',
-            url: '/r/sparse/',
-        };
-        const rows = await command.func(makePage({ kind: 'ok', info }), { name: 'sparse' });
-        const byField = Object.fromEntries(rows.map((r) => [r.field, r.value]));
-        expect(byField.Subscribers).toBeNull();
-        expect(byField['Active Now']).toBeNull();
-        expect(byField.Created).toBeNull();
-        expect(byField.Description).toBeNull();
-        expect(byField.Title).toBeNull();
-        expect(byField.Type).toBeNull();
-    });
-
-    it('encodes the subreddit name into the fetch URL embedded in evaluate', async () => {
-        const page = makePage({ kind: 'ok', info: { display_name: 'python', display_name_prefixed: 'r/python', url: '/r/python/' } });
-        await command.func(page, { name: 'r/python' });
-        const script = page.evaluate.mock.calls[0][0];
-        expect(script).toContain('const sub = "python"');
-    });
-});

package/clis/reddit/subreddit.test.js

@@ -1,31 +0,0 @@
-import { describe, expect, it } from 'vitest';
-import { getRegistry } from '@sovovs/bycli/registry';
-import './subreddit.js';
-
-describe('reddit subreddit adapter', () => {
-  const command = getRegistry().get('reddit/subreddit');
-
-  it('exposes the full subreddit-list shape including the 4 media columns', () => {
-    expect(command?.columns).toEqual([
-      'id', 'title', 'subreddit', 'author', 'upvotes', 'comments', 'url',
-      'created_utc', 'selftext',
-      'post_hint', 'url_overridden_by_dest', 'preview_image_url', 'gallery_urls',
-    ]);
-  });
-
-  it('shapes children into the intermediate-object pattern with media spread in', () => {
-    expect(command?.pipeline?.[1]?.evaluate).toContain('function extractRedditMedia');
-    expect(command?.pipeline?.[1]?.evaluate).toContain('...extractRedditMedia(c.data)');
-    expect(command?.pipeline?.[2]?.map).toMatchObject({
-      title: '${{ item.title }}',
-      author: '${{ item.author }}',
-      upvotes: '${{ item.upvotes }}',
-      comments: '${{ item.comments }}',
-      url: '${{ item.url }}',
-      post_hint: '${{ item.post_hint }}',
-      url_overridden_by_dest: '${{ item.url_overridden_by_dest }}',
-      preview_image_url: '${{ item.preview_image_url }}',
-      gallery_urls: '${{ item.gallery_urls }}',
-    });
-  });
-});