@jackwener/opencli 1.7.16 → 1.7.18

package/clis/notebooklm/get.js

@@ -18,7 +18,7 @@ cli({
         await requireNotebooklmSession(page);
         const state = await getNotebooklmPageState(page);
         if (state.kind !== 'notebook') {
-            throw new EmptyResultError('opencli notebooklm get', 'No NotebookLM notebook is open in the automation workspace. Run `opencli notebooklm open <notebook>` first.');
+            throw new EmptyResultError('opencli notebooklm get', 'No NotebookLM notebook is open in the adapter session. Run `opencli notebooklm open <notebook>` first.');
         }
         const rpcRow = await getNotebooklmDetailViaRpc(page).catch(() => null);
         if (rpcRow)

package/clis/notebooklm/history.js

@@ -17,7 +17,7 @@ cli({
         await requireNotebooklmSession(page);
         const state = await getNotebooklmPageState(page);
         if (state.kind !== 'notebook') {
-            throw new EmptyResultError('opencli notebooklm history', 'No NotebookLM notebook is open in the automation workspace. Run `opencli notebooklm open <notebook>` first.');
+            throw new EmptyResultError('opencli notebooklm history', 'No NotebookLM notebook is open in the adapter session. Run `opencli notebooklm open <notebook>` first.');
         }
         const rows = await listNotebooklmHistoryViaRpc(page);
         return rows;

package/clis/notebooklm/note-list.js

@@ -18,7 +18,7 @@ cli({
         await requireNotebooklmSession(page);
         const state = await getNotebooklmPageState(page);
         if (state.kind !== 'notebook') {
-            throw new EmptyResultError('opencli notebooklm note-list', 'No NotebookLM notebook is open in the automation workspace. Run `opencli notebooklm open <notebook>` first.');
+            throw new EmptyResultError('opencli notebooklm note-list', 'No NotebookLM notebook is open in the adapter session. Run `opencli notebooklm open <notebook>` first.');
         }
         const rows = await listNotebooklmNotesFromPage(page);
         if (rows.length > 0)

package/clis/notebooklm/notes-get.js

@@ -31,7 +31,7 @@ cli({
         await requireNotebooklmSession(page);
         const state = await getNotebooklmPageState(page);
         if (state.kind !== 'notebook') {
-            throw new EmptyResultError('opencli notebooklm notes-get', 'No NotebookLM notebook is open in the automation workspace. Run `opencli notebooklm open <notebook>` first.');
+            throw new EmptyResultError('opencli notebooklm notes-get', 'No NotebookLM notebook is open in the adapter session. Run `opencli notebooklm open <notebook>` first.');
         }
         const query = typeof kwargs.note === 'string' ? kwargs.note : String(kwargs.note ?? '');
         const visible = await readNotebooklmVisibleNoteFromPage(page);

package/clis/notebooklm/open.js

@@ -7,7 +7,7 @@ cli({
     name: 'open',
     access: 'read',
     aliases: ['select'],
-    description: 'Open one NotebookLM notebook in the automation workspace by id or URL',
+    description: 'Open one NotebookLM notebook in the adapter session by id or URL',
     domain: NOTEBOOKLM_DOMAIN,
     strategy: Strategy.COOKIE,
     browser: true,
@@ -28,7 +28,7 @@ cli({
         await requireNotebooklmSession(page);
         const state = await getNotebooklmPageState(page);
         if (state.kind !== 'notebook') {
-            throw new CliError('NOTEBOOKLM_OPEN_FAILED', `NotebookLM notebook "${notebookId}" did not open in the automation workspace`, 'Run `opencli notebooklm list -f json` first and pass a valid notebook id.');
+            throw new CliError('NOTEBOOKLM_OPEN_FAILED', `NotebookLM notebook "${notebookId}" did not open in the adapter session`, 'Run `opencli notebooklm list -f json` first and pass a valid notebook id.');
         }
         if (state.notebookId !== notebookId) {
             console.warn(`[notebooklm open] expected notebook "${notebookId}" but page reports "${state.notebookId}"; continuing`);

package/clis/notebooklm/open.test.js

@@ -38,7 +38,7 @@ describe('notebooklm open', () => {
             source: 'current-page',
         });
     });
-    it('opens a notebook by id in the automation workspace', async () => {
+    it('opens a notebook by id in the adapter session', async () => {
         const page = {
             goto: vi.fn(async () => { }),
             wait: vi.fn(async () => { }),

package/clis/notebooklm/source-fulltext.js

@@ -24,7 +24,7 @@ cli({
         await requireNotebooklmSession(page);
         const state = await getNotebooklmPageState(page);
         if (state.kind !== 'notebook') {
-            throw new EmptyResultError('opencli notebooklm source-fulltext', 'No NotebookLM notebook is open in the automation workspace. Run `opencli notebooklm open <notebook>` first.');
+            throw new EmptyResultError('opencli notebooklm source-fulltext', 'No NotebookLM notebook is open in the adapter session. Run `opencli notebooklm open <notebook>` first.');
         }
         const rpcRows = await listNotebooklmSourcesViaRpc(page).catch(() => []);
         const rows = rpcRows.length > 0 ? rpcRows : await listNotebooklmSourcesFromPage(page);

package/clis/notebooklm/source-get.js

@@ -24,7 +24,7 @@ cli({
         await requireNotebooklmSession(page);
         const state = await getNotebooklmPageState(page);
         if (state.kind !== 'notebook') {
-            throw new EmptyResultError('opencli notebooklm source-get', 'No NotebookLM notebook is open in the automation workspace. Run `opencli notebooklm open <notebook>` first.');
+            throw new EmptyResultError('opencli notebooklm source-get', 'No NotebookLM notebook is open in the adapter session. Run `opencli notebooklm open <notebook>` first.');
         }
         const rpcRows = await listNotebooklmSourcesViaRpc(page).catch(() => []);
         const rows = rpcRows.length > 0 ? rpcRows : await listNotebooklmSourcesFromPage(page);

package/clis/notebooklm/source-guide.js

@@ -24,7 +24,7 @@ cli({
         await requireNotebooklmSession(page);
         const state = await getNotebooklmPageState(page);
         if (state.kind !== 'notebook') {
-            throw new EmptyResultError('opencli notebooklm source-guide', 'No NotebookLM notebook is open in the automation workspace. Run `opencli notebooklm open <notebook>` first.');
+            throw new EmptyResultError('opencli notebooklm source-guide', 'No NotebookLM notebook is open in the adapter session. Run `opencli notebooklm open <notebook>` first.');
         }
         const rpcRows = await listNotebooklmSourcesViaRpc(page).catch(() => []);
         const rows = rpcRows.length > 0 ? rpcRows : await listNotebooklmSourcesFromPage(page);

package/clis/notebooklm/source-list.js

@@ -17,7 +17,7 @@ cli({
         await requireNotebooklmSession(page);
         const state = await getNotebooklmPageState(page);
         if (state.kind !== 'notebook') {
-            throw new EmptyResultError('opencli notebooklm source-list', 'No NotebookLM notebook is open in the automation workspace. Run `opencli notebooklm open <notebook>` first.');
+            throw new EmptyResultError('opencli notebooklm source-list', 'No NotebookLM notebook is open in the adapter session. Run `opencli notebooklm open <notebook>` first.');
         }
         const rpcRows = await listNotebooklmSourcesViaRpc(page).catch(() => []);
         if (rpcRows.length > 0)

package/clis/notebooklm/summary.js

@@ -17,7 +17,7 @@ cli({
         await requireNotebooklmSession(page);
         const state = await getNotebooklmPageState(page);
         if (state.kind !== 'notebook') {
-            throw new EmptyResultError('opencli notebooklm summary', 'No NotebookLM notebook is open in the automation workspace. Run `opencli notebooklm open <notebook>` first.');
+            throw new EmptyResultError('opencli notebooklm summary', 'No NotebookLM notebook is open in the adapter session. Run `opencli notebooklm open <notebook>` first.');
         }
         const domSummary = await readNotebooklmSummaryFromPage(page);
         if (domSummary)

package/clis/qwen/ask.js

@@ -24,7 +24,7 @@ cli({
     domain: QIANWEN_DOMAIN,
     strategy: Strategy.COOKIE,
     browser: true,
-    browserSession: { reuse: 'site' },
+    siteSession: 'persistent',
     navigateBefore: false,
     defaultFormat: 'plain',
     args: [

package/clis/qwen/detail.js

@@ -18,7 +18,7 @@ cli({
     strategy: Strategy.COOKIE,
     browser: true,
     navigateBefore: false,
-    browserSession: { reuse: 'site' },
+    siteSession: 'persistent',
     args: [
         { name: 'id', positional: true, required: true, help: 'Session ID (32-char hex) or full https://www.qianwen.com/chat/<id> URL' },
         { name: 'markdown', type: 'boolean', default: false, help: 'Emit assistant replies as markdown' },

package/clis/qwen/history.js

@@ -24,7 +24,7 @@ cli({
     domain: QIANWEN_DOMAIN,
     strategy: Strategy.COOKIE,
     browser: true,
-    browserSession: { reuse: 'site' },
+    siteSession: 'persistent',
     navigateBefore: false,
     args: [
         { name: 'limit', type: 'int', default: 20, help: 'Max conversations to show (default 20, max 100)' },

package/clis/qwen/image.js

@@ -99,7 +99,7 @@ cli({
     domain: QIANWEN_DOMAIN,
     strategy: Strategy.COOKIE,
     browser: true,
-    browserSession: { reuse: 'site' },
+    siteSession: 'persistent',
     navigateBefore: false,
     defaultFormat: 'plain',
     args: [

package/clis/qwen/new.js

@@ -9,7 +9,7 @@ cli({
     domain: QIANWEN_DOMAIN,
     strategy: Strategy.COOKIE,
     browser: true,
-    browserSession: { reuse: 'site' },
+    siteSession: 'persistent',
     navigateBefore: false,
     args: [],
     columns: ['Status'],

package/clis/qwen/read.js

@@ -16,7 +16,7 @@ cli({
     domain: QIANWEN_DOMAIN,
     strategy: Strategy.COOKIE,
     browser: true,
-    browserSession: { reuse: 'site' },
+    siteSession: 'persistent',
     navigateBefore: false,
     args: [
         { name: 'markdown', type: 'boolean', default: false, help: 'Emit assistant replies as markdown' },

package/clis/qwen/send.js

@@ -20,7 +20,7 @@ cli({
     domain: QIANWEN_DOMAIN,
     strategy: Strategy.COOKIE,
     browser: true,
-    browserSession: { reuse: 'site' },
+    siteSession: 'persistent',
     navigateBefore: false,
     args: [
         { name: 'prompt', required: true, positional: true, help: 'Prompt to send to Qianwen' },

package/clis/qwen/status.js

@@ -9,7 +9,7 @@ cli({
     domain: QIANWEN_DOMAIN,
     strategy: Strategy.COOKIE,
     browser: true,
-    browserSession: { reuse: 'site' },
+    siteSession: 'persistent',
     navigateBefore: false,
     args: [],
     columns: ['Status', 'Login', 'Model', 'SessionId', 'Url'],

package/clis/reddit/comment.js

@@ -8,7 +8,7 @@ cli({
     domain: 'reddit.com',
     strategy: Strategy.COOKIE,
     browser: true,
-    browserSession: { reuse: 'site' },
+    siteSession: 'persistent',
     args: [
         { name: 'post-id', type: 'string', required: true, positional: true, help: 'Post ID (e.g. 1abc123) or fullname (t3_xxx)' },
         { name: 'text', type: 'string', required: true, positional: true, help: 'Comment text' },

package/clis/reddit/frontpage.js

@@ -7,7 +7,7 @@ cli({
     domain: 'reddit.com',
     strategy: Strategy.COOKIE,
     browser: true,
-    browserSession: { reuse: 'site' },
+    siteSession: 'persistent',
     args: [
         { name: 'limit', type: 'int', default: 15 },
     ],

package/clis/reddit/popular.js

@@ -7,7 +7,7 @@ cli({
     domain: 'reddit.com',
     strategy: Strategy.COOKIE,
     browser: true,
-    browserSession: { reuse: 'site' },
+    siteSession: 'persistent',
     args: [
         { name: 'limit', type: 'int', default: 20 },
     ],

package/clis/reddit/read.js

@@ -16,7 +16,7 @@ cli({
     domain: 'reddit.com',
     strategy: Strategy.COOKIE,
     browser: true,
-    browserSession: { reuse: 'site' },
+    siteSession: 'persistent',
     args: [
         { name: 'post-id', required: true, positional: true, help: 'Post ID (e.g. 1abc123) or full URL' },
         { name: 'sort', default: 'best', help: 'Comment sort: best, top, new, controversial, old, qa' },

package/clis/reddit/read.test.js

@@ -3,9 +3,9 @@ import { getRegistry } from '@jackwener/opencli/registry';
 import './read.js';
 describe('reddit read adapter', () => {
     const command = getRegistry().get('reddit/read');
-    it('opts into the Reddit site browser session lease', () => {
+    it('opts into the Reddit persistent site session', () => {
         expect(command?.browser).toBe(true);
-        expect(command?.browserSession).toEqual({ reuse: 'site' });
+        expect(command?.siteSession).toBe('persistent');
     });
     it('returns threaded rows from the browser-evaluated payload', async () => {
         const page = {

package/clis/reddit/reply.js

@@ -0,0 +1,182 @@
+import { ArgumentError, AuthRequiredError, CommandExecutionError } from '@jackwener/opencli/errors';
+import { cli, Strategy } from '@jackwener/opencli/registry';
+
+const REDDIT_COMMENT_ID_RE = /^[a-z0-9]+$/i;
+
+function normalizeBareCommentId(value) {
+    const commentId = String(value || '').trim();
+    if (!REDDIT_COMMENT_ID_RE.test(commentId)) {
+        throw new ArgumentError(
+            'Comment ID must be a Reddit comment id, t1_ fullname, or reddit.com comment URL.',
+            'Use a bare comment id like okf3s7u, a fullname like t1_okf3s7u, or a full Reddit comment URL.',
+        );
+    }
+    return commentId.toLowerCase();
+}
+
+export function normalizeRedditCommentFullname(value) {
+    const raw = String(value || '').trim();
+    if (!raw) {
+        throw new ArgumentError(
+            'Comment ID is required.',
+            'Use a bare comment id like okf3s7u, a fullname like t1_okf3s7u, or a full Reddit comment URL.',
+        );
+    }
+
+    const fullname = raw.match(/^t1_([a-z0-9]+)$/i);
+    if (fullname) return `t1_${normalizeBareCommentId(fullname[1])}`;
+
+    if (/^https?:\/\//i.test(raw)) {
+        let parsed;
+        try {
+            parsed = new URL(raw);
+        } catch {
+            throw new ArgumentError(`Invalid Reddit comment URL: ${raw}`);
+        }
+        const host = parsed.hostname.toLowerCase();
+        if (parsed.protocol !== 'https:' || (host !== 'reddit.com' && !host.endsWith('.reddit.com'))) {
+            throw new ArgumentError(
+                'Comment URL must be an https reddit.com URL.',
+                'Use a URL like https://www.reddit.com/r/sub/comments/post/title/okf3s7u/',
+            );
+        }
+        const parts = parsed.pathname.split('/').filter(Boolean);
+        const commentsIndex = parts.indexOf('comments');
+        const commentIndex = commentsIndex + 3;
+        if (commentsIndex < 0 || parts.length <= commentIndex) {
+            throw new ArgumentError(
+                'Comment URL must include the target comment id.',
+                'Use a URL like https://www.reddit.com/r/sub/comments/post/title/okf3s7u/',
+            );
+        }
+        if (parts.length !== commentIndex + 1) {
+            throw new ArgumentError(
+                'Comment URL must end at the target comment id.',
+                'Remove extra path segments after the comment id.',
+            );
+        }
+        return `t1_${normalizeBareCommentId(parts[commentIndex])}`;
+    }
+
+    if (raw.includes('/') || raw.startsWith('t3_')) {
+        throw new ArgumentError(
+            'Comment ID must be a Reddit comment id, t1_ fullname, or reddit.com comment URL.',
+            'Use a bare comment id like okf3s7u, a fullname like t1_okf3s7u, or a full Reddit comment URL.',
+        );
+    }
+
+    return `t1_${normalizeBareCommentId(raw)}`;
+}
+
+export function requireReplyText(value) {
+    const text = String(value || '');
+    if (!text.trim()) {
+        throw new ArgumentError('Reply text is required.', 'Pass non-empty text to post as the Reddit reply.');
+    }
+    return text;
+}
+
+cli({
+    site: 'reddit',
+    name: 'reply',
+    access: 'write',
+    description: 'Reply to a Reddit comment',
+    domain: 'reddit.com',
+    strategy: Strategy.COOKIE,
+    browser: true,
+    args: [
+        { name: 'comment-id', type: 'string', required: true, positional: true, help: 'Comment ID (e.g. okf3s7u) or fullname (t1_xxx)' },
+        { name: 'text', type: 'string', required: true, positional: true, help: 'Reply text' },
+    ],
+    columns: ['status', 'message'],
+    func: async (page, kwargs) => {
+        const fullname = normalizeRedditCommentFullname(kwargs['comment-id']);
+        const text = requireReplyText(kwargs.text);
+        await page.goto('https://www.reddit.com');
+        // Inside page.evaluate we can't throw typed errors (they don't survive
+        // the worker boundary), so we surface a structured `kind` discriminator
+        // and re-throw the matching typed error on the Node side. Each kind
+        // maps 1:1 to a typed-error class — no silent-sentinel rows on failure.
+        //
+        // Intermediate object keys deliberately avoid `status` / `message` to
+        // sidestep the silent-column-drop audit (columns are ['status',
+        // 'message']) — see PR #1329 sediment "中间解析对象 key 不能跟 columns
+        // 任一项重叠".
+        const result = await page.evaluate(`(async () => {
+      try {
+        const fullname = ${JSON.stringify(fullname)};
+        const text = ${JSON.stringify(text)};
+
+        // Probe identity + modhash. /api/me.json returns data.name only when
+        // logged in — empty modhash alone is not a strong enough auth signal
+        // because Reddit sometimes returns 200 with empty modhash for stale
+        // anonymous sessions.
+        const meRes = await fetch('/api/me.json', { credentials: 'include' });
+        if (meRes.status === 401 || meRes.status === 403) {
+          return { kind: 'auth', detail: 'Reddit /api/me.json returned HTTP ' + meRes.status };
+        }
+        if (!meRes.ok) {
+          return { kind: 'http', httpStatus: meRes.status, where: '/api/me.json' };
+        }
+        const me = await meRes.json();
+        if (!me?.data?.name) {
+          return { kind: 'auth', detail: 'Not logged in to reddit.com (no identity in /api/me.json)' };
+        }
+        const modhash = me.data.modhash || '';
+
+        const res = await fetch('/api/comment', {
+          method: 'POST',
+          credentials: 'include',
+          headers: { 'Content-Type': 'application/x-www-form-urlencoded' },
+          body: 'parent=' + encodeURIComponent(fullname)
+            + '&text=' + encodeURIComponent(text)
+            + '&api_type=json'
+            + (modhash ? '&uh=' + encodeURIComponent(modhash) : ''),
+        });
+
+        if (res.status === 401 || res.status === 403) {
+          return { kind: 'auth', detail: 'Reddit /api/comment returned HTTP ' + res.status };
+        }
+        if (!res.ok) {
+          return { kind: 'http', httpStatus: res.status, where: '/api/comment' };
+        }
+        const data = await res.json();
+        const errors = data?.json?.errors;
+        if (errors && errors.length > 0) {
+          return { kind: 'reddit-error', detail: errors.map(e => e.join(': ')).join('; ') };
+        }
+        const things = data?.json?.data?.things;
+        const created = Array.isArray(things)
+          ? things.find((thing) => thing?.kind === 't1' || String(thing?.data?.name || '').startsWith('t1_'))
+          : null;
+        const createdName = created?.data?.name || (created?.data?.id ? 't1_' + created.data.id : '');
+        if (!createdName) {
+          return { kind: 'postcondition', detail: 'Reddit comment response did not include a created reply id' };
+        }
+        return { kind: 'ok', detail: 'Reply posted on ' + fullname + ' as ' + createdName };
+      } catch (e) {
+        return { kind: 'exception', detail: String(e && e.message || e) };
+      }
+    })()`);
+
+        if (result?.kind === 'auth') {
+            throw new AuthRequiredError('reddit.com', result.detail);
+        }
+        if (result?.kind === 'http') {
+            throw new CommandExecutionError(`HTTP ${result.httpStatus} from ${result.where}`);
+        }
+        if (result?.kind === 'reddit-error') {
+            throw new CommandExecutionError(`Reddit rejected reply: ${result.detail}`);
+        }
+        if (result?.kind === 'postcondition') {
+            throw new CommandExecutionError(result.detail);
+        }
+        if (result?.kind === 'exception') {
+            throw new CommandExecutionError(`Reply failed: ${result.detail}`);
+        }
+        if (result?.kind !== 'ok') {
+            throw new CommandExecutionError(`Unexpected result from reddit reply: ${JSON.stringify(result)}`);
+        }
+        return [{ status: 'success', message: result.detail }];
+    },
+});

package/clis/reddit/reply.test.js

@@ -0,0 +1,89 @@
+import { describe, expect, it, vi } from 'vitest';
+import { getRegistry } from '@jackwener/opencli/registry';
+import { ArgumentError, AuthRequiredError, CommandExecutionError } from '@jackwener/opencli/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/opencli/comments/1abc23/title_slug/okf3s7u/?context=3')).toBe('t1_okf3s7u');
+        expect(normalizeRedditCommentFullname('https://old.reddit.com/r/opencli/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/opencli/comments/1abc23/title_slug/okf3s7u/',
+            'http://www.reddit.com/r/opencli/comments/1abc23/title_slug/okf3s7u/',
+            'https://www.reddit.com/r/opencli/comments/1abc23/title_slug/',
+            'https://www.reddit.com/r/opencli/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/opencli/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/save.js

@@ -8,7 +8,7 @@ cli({
     domain: 'reddit.com',
     strategy: Strategy.COOKIE,
     browser: true,
-    browserSession: { reuse: 'site' },
+    siteSession: 'persistent',
     args: [
         { name: 'post-id', type: 'string', required: true, positional: true, help: 'Post ID (e.g. 1abc123) or fullname (t3_xxx)' },
         { name: 'undo', type: 'boolean', default: false, help: 'Unsave instead of save' },

package/clis/reddit/saved.js

@@ -8,7 +8,7 @@ cli({
     domain: 'reddit.com',
     strategy: Strategy.COOKIE,
     browser: true,
-    browserSession: { reuse: 'site' },
+    siteSession: 'persistent',
     args: [
         { name: 'limit', type: 'int', default: 15 },
     ],

package/clis/reddit/search.js

@@ -7,7 +7,7 @@ cli({
     domain: 'reddit.com',
     strategy: Strategy.COOKIE,
     browser: true,
-    browserSession: { reuse: 'site' },
+    siteSession: 'persistent',
     args: [
         { name: 'query', type: 'string', required: true, positional: true, help: 'Reddit search query' },
         {

package/clis/reddit/subreddit.js

@@ -7,7 +7,7 @@ cli({
     domain: 'reddit.com',
     strategy: Strategy.COOKIE,
     browser: true,
-    browserSession: { reuse: 'site' },
+    siteSession: 'persistent',
     args: [
         { name: 'name', type: 'string', required: true, positional: true, help: 'Subreddit name (no `r/` prefix; e.g. `python`)' },
         {

package/clis/reddit/subscribe.js

@@ -8,7 +8,7 @@ cli({
     domain: 'reddit.com',
     strategy: Strategy.COOKIE,
     browser: true,
-    browserSession: { reuse: 'site' },
+    siteSession: 'persistent',
     args: [
         { name: 'subreddit', type: 'string', required: true, positional: true, help: 'Subreddit name (e.g. python)' },
         { name: 'undo', type: 'boolean', default: false, help: 'Unsubscribe instead of subscribe' },

package/clis/reddit/upvote.js

@@ -8,7 +8,7 @@ cli({
     domain: 'reddit.com',
     strategy: Strategy.COOKIE,
     browser: true,
-    browserSession: { reuse: 'site' },
+    siteSession: 'persistent',
     args: [
         { name: 'post-id', type: 'string', required: true, positional: true, help: 'Post ID (e.g. 1abc123) or fullname (t3_xxx)' },
         { name: 'direction', type: 'string', default: 'up', help: 'Vote direction: up, down, none' },

package/clis/reddit/upvoted.js

@@ -8,7 +8,7 @@ cli({
     domain: 'reddit.com',
     strategy: Strategy.COOKIE,
     browser: true,
-    browserSession: { reuse: 'site' },
+    siteSession: 'persistent',
     args: [
         { name: 'limit', type: 'int', default: 15 },
     ],

package/clis/reddit/user-comments.js

@@ -7,7 +7,7 @@ cli({
     domain: 'reddit.com',
     strategy: Strategy.COOKIE,
     browser: true,
-    browserSession: { reuse: 'site' },
+    siteSession: 'persistent',
     args: [
         { name: 'username', type: 'string', required: true, positional: true, help: 'Reddit username (no `u/` prefix needed)' },
         { name: 'limit', type: 'int', default: 15 },

package/clis/reddit/user-posts.js

@@ -7,7 +7,7 @@ cli({
     domain: 'reddit.com',
     strategy: Strategy.COOKIE,
     browser: true,
-    browserSession: { reuse: 'site' },
+    siteSession: 'persistent',
     args: [
         { name: 'username', type: 'string', required: true, positional: true, help: 'Reddit username (no `u/` prefix needed)' },
         { name: 'limit', type: 'int', default: 15 },

package/clis/reddit/user.js

@@ -7,7 +7,7 @@ cli({
     domain: 'reddit.com',
     strategy: Strategy.COOKIE,
     browser: true,
-    browserSession: { reuse: 'site' },
+    siteSession: 'persistent',
     args: [
         { name: 'username', type: 'string', required: true, positional: true, help: 'Reddit username (no `u/` prefix needed)' },
     ],