@jackwener/opencli 1.7.18 → 1.7.19

package/clis/twitter/likes.js

@@ -1,9 +1,10 @@
 import { cli, Strategy } from '@jackwener/opencli/registry';
-import { AuthRequiredError, CommandExecutionError } from '@jackwener/opencli/errors';
-import { resolveTwitterQueryId, sanitizeQueryId, extractMedia } from './shared.js';
+import { ArgumentError, AuthRequiredError, CommandExecutionError } from '@jackwener/opencli/errors';
+import { normalizeTwitterScreenName, resolveTwitterQueryId, sanitizeQueryId, extractMedia, unwrapBrowserResult } from './shared.js';
 import { TWITTER_BEARER_TOKEN, applyTopByEngagement } from './utils.js';
 const LIKES_QUERY_ID = 'RozQdCp4CilQzrcuU0NY5w';
 const USER_BY_SCREEN_NAME_QUERY_ID = 'qRednkZG-rn1P6b48NINmQ';
+const MAX_PAGINATION_PAGES = 100;
 const FEATURES = {
     rweb_video_screen_enabled: false,
     profile_label_improvements_pcf_label_in_post_enabled: true,
@@ -151,20 +152,33 @@ cli({
     columns: ['id', 'author', 'name', 'text', 'likes', 'retweets', 'created_at', 'url', 'has_media', 'media_urls'],
     func: async (page, kwargs) => {
         const limit = kwargs.limit || 20;
-        let username = (kwargs.username || '').replace(/^@/, '');
+        const rawUsername = String(kwargs.username ?? '').trim();
+        let username = normalizeTwitterScreenName(rawUsername);
+        if (rawUsername && !username) {
+            throw new ArgumentError('twitter likes username must be a valid Twitter/X handle', 'Example: opencli twitter likes @jack --limit 20');
+        }
         const cookies = await page.getCookies({ url: 'https://x.com' });
         const ct0 = cookies.find((c) => c.name === 'ct0')?.value || null;
         if (!ct0)
             throw new AuthRequiredError('x.com', 'Not logged into x.com (no ct0 cookie)');
-        // If no username provided, detect the logged-in user
+        // If no username provided, detect the logged-in user.
+        // Bridge wraps primitive page.evaluate returns as { session, data:<value> };
+        // unwrap so the href string is usable downstream.
         if (!username) {
-            const href = await page.evaluate(`() => {
+            // Force a navigation to the home surface so the AppTabBar sidebar
+            // is rendered; the framework pre-nav lands on bare x.com which
+            // does not always expose AppTabBar_Profile_Link.
+            await page.goto('https://x.com/home');
+            await page.wait({ selector: '[data-testid="primaryColumn"]' });
+            const href = unwrapBrowserResult(await page.evaluate(`() => {
         const link = document.querySelector('a[data-testid="AppTabBar_Profile_Link"]');
         return link ? link.getAttribute('href') : null;
-      }`);
-            if (!href)
+      }`));
+            if (!href || typeof href !== 'string')
+                throw new AuthRequiredError('x.com', 'Could not detect logged-in user. Are you logged in?');
+            username = normalizeTwitterScreenName(href);
+            if (!username)
                 throw new AuthRequiredError('x.com', 'Could not detect logged-in user. Are you logged in?');
-            username = href.replace('/', '');
         }
         const likesQueryId = await resolveTwitterQueryId(page, 'Likes', LIKES_QUERY_ID);
         const userByScreenNameQueryId = await resolveTwitterQueryId(page, 'UserByScreenName', USER_BY_SCREEN_NAME_QUERY_ID);
@@ -175,27 +189,28 @@ cli({
             'X-Twitter-Active-User': 'yes',
         });
         // Get userId from screen_name
-        const userId = await page.evaluate(`async () => {
+        const userId = unwrapBrowserResult(await page.evaluate(`async () => {
       const screenName = ${JSON.stringify(username)};
       const url = ${JSON.stringify(buildUserByScreenNameUrl(userByScreenNameQueryId, username))};
       const resp = await fetch(url, { headers: ${headers}, credentials: 'include' });
       if (!resp.ok) return null;
       const d = await resp.json();
       return d.data?.user?.result?.rest_id || null;
-    }`);
+    }`));
         if (!userId) {
             throw new CommandExecutionError(`Could not find user @${username}`);
         }
         const allTweets = [];
         const seen = new Set();
         let cursor = null;
-        for (let i = 0; i < 5 && allTweets.length < limit; i++) {
+        // Runaway guard only; --limit and cursor exhaustion control normal pagination.
+        for (let i = 0; i < MAX_PAGINATION_PAGES && allTweets.length < limit; i++) {
             const fetchCount = Math.min(100, limit - allTweets.length + 10);
             const apiUrl = buildLikesUrl(likesQueryId, userId, fetchCount, cursor);
-            const data = await page.evaluate(`async () => {
+            const data = unwrapBrowserResult(await page.evaluate(`async () => {
         const r = await fetch("${apiUrl}", { headers: ${headers}, credentials: 'include' });
         return r.ok ? await r.json() : { error: r.status };
-      }`);
+      }`));
             if (data?.error) {
                 if (allTweets.length === 0)
                     throw new CommandExecutionError(`HTTP ${data.error}: Failed to fetch likes. queryId may have expired.`);

package/clis/twitter/likes.test.js

@@ -1,5 +1,50 @@
-import { describe, expect, it } from 'vitest';
+import { describe, expect, it, vi } from 'vitest';
+import { getRegistry } from '@jackwener/opencli/registry';
+import { ArgumentError, AuthRequiredError } from '@jackwener/opencli/errors';
 import { __test__ } from './likes.js';
+
+function likesPayload() {
+    return {
+        data: {
+            user: {
+                result: {
+                    timeline_v2: {
+                        timeline: {
+                            instructions: [{
+                                entries: [{
+                                    entryId: 'tweet-1',
+                                    content: {
+                                        itemContent: {
+                                            tweet_results: {
+                                                result: {
+                                                    rest_id: '1',
+                                                    legacy: {
+                                                        full_text: 'liked post',
+                                                        favorite_count: 7,
+                                                        retweet_count: 2,
+                                                        created_at: 'now',
+                                                    },
+                                                    core: {
+                                                        user_results: {
+                                                            result: {
+                                                                legacy: { screen_name: 'alice', name: 'Alice' },
+                                                            },
+                                                        },
+                                                    },
+                                                },
+                                            },
+                                        },
+                                    },
+                                }],
+                            }],
+                        },
+                    },
+                },
+            },
+        },
+    };
+}
+
 describe('twitter likes helpers', () => {
     it('falls back when queryId contains unsafe characters', () => {
         expect(__test__.sanitizeQueryId('safe_Query-123', 'fallback')).toBe('safe_Query-123');
@@ -83,3 +128,68 @@ describe('twitter likes helpers', () => {
         });
     });
 });
+
+describe('twitter likes command', () => {
+    it('rejects invalid explicit username before cookies or navigation', async () => {
+        const command = getRegistry().get('twitter/likes');
+        const page = {
+            goto: vi.fn(),
+            wait: vi.fn(),
+            getCookies: vi.fn(),
+            evaluate: vi.fn(),
+        };
+
+        await expect(command.func(page, { username: 'viewer/extra', limit: 10 })).rejects.toBeInstanceOf(ArgumentError);
+        expect(page.getCookies).not.toHaveBeenCalled();
+        expect(page.goto).not.toHaveBeenCalled();
+        expect(page.evaluate).not.toHaveBeenCalled();
+    });
+
+    it('rejects route-like AppTabBar hrefs as AuthRequiredError', async () => {
+        const command = getRegistry().get('twitter/likes');
+        const page = {
+            goto: vi.fn().mockResolvedValue(undefined),
+            wait: vi.fn().mockResolvedValue(undefined),
+            getCookies: vi.fn(async () => [{ name: 'ct0', value: 'token' }]),
+            evaluate: vi.fn(async (script) => {
+                if (String(script).includes('AppTabBar_Profile_Link')) return '/home';
+                throw new Error(`Unexpected evaluate: ${String(script).slice(0, 80)}`);
+            }),
+        };
+
+        await expect(command.func(page, { limit: 10 })).rejects.toBeInstanceOf(AuthRequiredError);
+        expect(page.goto).toHaveBeenCalledWith('https://x.com/home');
+        expect(page.evaluate).toHaveBeenCalledTimes(1);
+    });
+
+    it('unwraps Browser Bridge envelopes for default-self user lookup and likes payload', async () => {
+        const command = getRegistry().get('twitter/likes');
+        const page = {
+            goto: vi.fn().mockResolvedValue(undefined),
+            wait: vi.fn().mockResolvedValue(undefined),
+            getCookies: vi.fn(async () => [{ name: 'ct0', value: 'token' }]),
+            evaluate: vi.fn(async (script) => {
+                const text = String(script);
+                if (text.includes('AppTabBar_Profile_Link')) {
+                    return { session: 'site:twitter', data: '/viewer' };
+                }
+                if (text.includes('operationName')) return null;
+                if (text.includes('/UserByScreenName')) {
+                    return { session: 'site:twitter', data: '42' };
+                }
+                if (text.includes('/Likes')) {
+                    return { session: 'site:twitter', data: likesPayload() };
+                }
+                throw new Error(`Unexpected evaluate: ${text.slice(0, 80)}`);
+            }),
+        };
+
+        const rows = await command.func(page, { limit: 1 });
+
+        expect(rows).toHaveLength(1);
+        expect(rows[0]).toMatchObject({ id: '1', author: 'alice', text: 'liked post' });
+        const likesCall = page.evaluate.mock.calls.find(([script]) => String(script).includes('/Likes')) || [];
+        expect(decodeURIComponent(String(likesCall[0]))).toContain('"userId":"42"');
+        expect(decodeURIComponent(String(likesCall[0]))).not.toContain('[object Object]');
+    });
+});

package/clis/twitter/list-add.js

@@ -1,11 +1,14 @@
 import { cli, Strategy } from '@jackwener/opencli/registry';
-import { AuthRequiredError, CommandExecutionError } from '@jackwener/opencli/errors';
+import { ArgumentError, AuthRequiredError, CommandExecutionError } from '@jackwener/opencli/errors';
 import { resolveTwitterQueryId } from './shared.js';
 import { parseListsManagement } from './lists.js';
 import { TWITTER_BEARER_TOKEN } from './utils.js';
 
 const USER_BY_SCREEN_NAME_QUERY_ID = 'qRednkZG-rn1P6b48NINmQ';
 const LISTS_MANAGEMENT_QUERY_ID = '78UbkyXwXBD98IgUWXOy9g';
+// 2026-05 fallback — X rotates queryIds; resolveTwitterQueryId() does live lookup,
+// this constant is just the default if live lookup fails.
+const LIST_ADD_MEMBER_QUERY_ID = 'vWPi0CTMoPFsjsL6W4IynQ';
 
 const LISTS_MANAGEMENT_FEATURES = {
     rweb_video_screen_enabled: false,
@@ -62,6 +65,65 @@ function buildUserByScreenNameUrl(queryId, screenName) {
         + `&features=${encodeURIComponent(feats)}`;
 }
 
+function fatalGraphqlErrors(errors) {
+    const list = Array.isArray(errors) ? errors : [];
+    return list.filter((e) =>
+        !(e?.path || []).join('.').includes('default_banner_media_results')
+        && !/decode/i.test(e?.message || '')
+    );
+}
+
+export function buildListAddMemberRow({ addResult, memberCountBefore, listId, username, userId }) {
+    if (!addResult?.httpOk) {
+        throw new CommandExecutionError(
+            `Failed to add @${username} to list ${listId}: HTTP ${addResult?.status ?? 0}${addResult?.fetchError ? ' (' + addResult.fetchError + ')' : ''}${addResult?.raw ? ' — ' + addResult.raw : ''}`
+        );
+    }
+
+    // X often returns a partial GraphQL error on `default_banner_media_results`
+    // even on successful mutations. Treat only missing main data or non-decode
+    // GraphQL errors as command failures.
+    const hasMemberCount = addResult.mc !== null && addResult.mc !== undefined;
+    const fatalErrors = fatalGraphqlErrors(addResult.errors);
+    if (!hasMemberCount && fatalErrors.length) {
+        const msg = fatalErrors.map((e) => e.message || JSON.stringify(e)).join('; ');
+        throw new CommandExecutionError(`Failed to add @${username} to list ${listId}: ${msg.slice(0, 300)}`);
+    }
+    if (!hasMemberCount) {
+        throw new CommandExecutionError(`Failed to add @${username} to list ${listId}: no member_count in response`);
+    }
+
+    const memberCountAfter = Number(addResult.mc);
+    if (!Number.isFinite(memberCountAfter)) {
+        throw new CommandExecutionError(`Failed to add @${username} to list ${listId}: invalid member_count in response`);
+    }
+
+    if (memberCountAfter < memberCountBefore) {
+        throw new CommandExecutionError(
+            `Failed to add @${username} to list ${listId}: member_count decreased unexpectedly (${memberCountBefore} → ${memberCountAfter})`
+        );
+    }
+
+    const countIncreased = memberCountAfter > memberCountBefore;
+    if (!countIncreased && addResult.isMember !== true) {
+        throw new CommandExecutionError(
+            `Failed to add @${username} to list ${listId}: member_count unchanged (${memberCountBefore} → ${memberCountAfter}) and response did not confirm membership`
+        );
+    }
+
+    const noop = !countIncreased;
+    const verifiedBy = `member_count ${memberCountBefore} → ${memberCountAfter}`;
+    return {
+        listId,
+        username,
+        userId: String(userId),
+        status: noop ? 'noop' : 'success',
+        message: noop
+            ? `@${username} is already a member of list ${listId}`
+            : `Added @${username} to list ${listId} (verified via ${verifiedBy})`,
+    };
+}
+
 cli({
     site: 'twitter',
     name: 'list-add',
@@ -79,10 +141,10 @@ cli({
         const listId = String(kwargs.listId || '').trim();
         const username = String(kwargs.username || '').replace(/^@/, '').trim();
         if (!listId || !/^\d+$/.test(listId)) {
-            throw new CommandExecutionError(`Invalid listId: ${JSON.stringify(kwargs.listId)}. Expected numeric ID (see \`opencli twitter lists\`).`);
+            throw new ArgumentError(`Invalid listId: ${JSON.stringify(kwargs.listId)}. Expected numeric ID.`, 'Example: opencli twitter list-add 123456789 alice');
         }
         if (!username) {
-            throw new CommandExecutionError('Username is required');
+            throw new ArgumentError('twitter list-add username is required', 'Example: opencli twitter list-add 123456789 alice');
         }
         // Strategy.UI does not get a domain URL pre-nav from the framework.
         // This page context is load-bearing for pre-target GraphQL calls below.
@@ -101,25 +163,33 @@ cli({
             'X-Twitter-Active-User': 'yes',
         });
 
+        // opencli >=1.7.x wraps page.evaluate return values as { session, data }.
+        // Unwrap before use so JSON.stringify of nested values doesn't become "[object Object]".
+        const unwrap = (v) => (v && typeof v === 'object' && 'session' in v && 'data' in v ? v.data : v);
+
         const userLookupUrl = buildUserByScreenNameUrl(userByScreenNameQueryId, username);
-        const userId = await page.evaluate(`async () => {
+        const userIdRaw = await page.evaluate(`async () => {
             const resp = await fetch(${JSON.stringify(userLookupUrl)}, { headers: ${headers}, credentials: 'include' });
             if (!resp.ok) return null;
             const d = await resp.json();
             return d.data?.user?.result?.rest_id || null;
         }`);
+        const userId = unwrap(userIdRaw);
         if (!userId) {
             throw new CommandExecutionError(`Could not resolve user @${username}`);
         }
 
-        // ListsManagementPageTimeline — used both for id→name resolution and post-op verification.
+        // ListsManagementPageTimeline — used for list existence check + before/after member_count.
         const listsQueryId = await resolveTwitterQueryId(page, 'ListsManagementPageTimeline', LISTS_MANAGEMENT_QUERY_ID);
         const listsUrl = `/i/api/graphql/${listsQueryId}/ListsManagementPageTimeline?features=${encodeURIComponent(JSON.stringify(LISTS_MANAGEMENT_FEATURES))}`;
-        const listsData = await page.evaluate(`async () => {
+        const listsDataRaw = await page.evaluate(`async () => {
             const r = await fetch(${JSON.stringify(listsUrl)}, { headers: ${headers}, credentials: 'include' });
             if (!r.ok) return { __error: 'HTTP ' + r.status };
             return await r.json();
         }`);
+        // Don't unwrap listsData: opencli spreads GraphQL response to top-level + adds session;
+        // parseListsManagement reads `.data.viewer.*` from this shape directly.
+        const listsData = listsDataRaw;
         const parsedLists = listsData && !listsData.__error
             ? parseListsManagement(listsData, new Set())
             : [];
@@ -131,209 +201,63 @@ cli({
             throw new CommandExecutionError(`List ${listId} not found among your lists (${parsedLists.length} lists fetched).`);
         }
 
-        // Use UI strategy — programmatically open "Add/Remove from Lists" dialog and toggle the target list.
-        await page.goto(`https://x.com/${username}`);
-        await page.wait({ selector: '[data-testid="primaryColumn"]' });
-        const targetName = targetList.name;
-        const uiResult = await page.evaluate(`(async () => {
-            const sleep = (ms) => new Promise(r => setTimeout(r, ms));
-            const findOne = (sel, root = document) => root.querySelector(sel);
-            const waitFor = async (fn, { timeoutMs = 8000, intervalMs = 200 } = {}) => {
-                const t0 = Date.now();
-                while (Date.now() - t0 < timeoutMs) {
-                    const v = fn();
-                    if (v) return v;
-                    await sleep(intervalMs);
-                }
-                return null;
-            };
+        // Direct GraphQL ListAddMember mutation.
+        //
+        // Previously this command opened the X profile, clicked "…" → "Add/remove from Lists",
+        // navigated the dialog and used nativeClick on the Save button. In 2026-05 X replaced
+        // the dialog with a full-page route (/i/lists/add_member), breaking that UI flow.
+        //
+        // The mutation is the same one the UI fires under the hood; calling it directly is
+        // both more reliable and ~10x faster (no goto-profile + scroll-dialog roundtrip).
+        const memberCountBefore = Number(targetList.members) || 0;
+        const listAddMemberQueryId = await resolveTwitterQueryId(page, 'ListAddMember', LIST_ADD_MEMBER_QUERY_ID);
+        const addUrl = `/i/api/graphql/${listAddMemberQueryId}/ListAddMember`;
+        const addBody = JSON.stringify({
+            variables: { listId, userId: String(userId) },
+            queryId: listAddMemberQueryId,
+        });
+        const addResultJsonRaw = await page.evaluate(`async () => {
             try {
-                // Install fetch + XHR interceptors to observe list-membership mutations.
-                const MUTATION_RE = /ListAddMember|ListRemoveMember|lists\\/members\\/(create|destroy)|ListManagement.*Add|ListManagement.*Remove|\\/add_member|\\/remove_member|ListAddMembers|ListRemoveMembers|list.*member.*create|list.*member.*destroy/i;
-                if (!window.__opencliListMutations) {
-                    window.__opencliListMutations = [];
-                    window.__opencliAllRequests = [];
-                    const origFetch = window.fetch.bind(window);
-                    window.fetch = async function(...args) {
-                        const url = typeof args[0] === 'string' ? args[0] : (args[0] && args[0].url) || '';
-                        const method = (args[1] && args[1].method) || 'GET';
-                        let resp;
-                        try { resp = await origFetch(...args); }
-                        catch (err) {
-                            if (MUTATION_RE.test(url)) window.__opencliListMutations.push({ url, method, status: 0, error: String(err), ts: Date.now(), via: 'fetch' });
-                            throw err;
-                        }
-                        if (method !== 'GET' && method !== 'HEAD') {
-                            window.__opencliAllRequests.push({ url, method, status: resp.status, ts: Date.now(), via: 'fetch' });
-                        }
-                        if (MUTATION_RE.test(url)) {
-                            window.__opencliListMutations.push({ url, method, status: resp.status, ts: Date.now(), via: 'fetch' });
-                        }
-                        return resp;
-                    };
-                    // Also hook XMLHttpRequest
-                    const OrigXhrOpen = XMLHttpRequest.prototype.open;
-                    const OrigXhrSend = XMLHttpRequest.prototype.send;
-                    XMLHttpRequest.prototype.open = function(method, url, ...rest) {
-                        this.__opencliMethod = method;
-                        this.__opencliUrl = url;
-                        return OrigXhrOpen.call(this, method, url, ...rest);
-                    };
-                    XMLHttpRequest.prototype.send = function(...args) {
-                        const xhr = this;
-                        xhr.addEventListener('loadend', () => {
-                            const url = xhr.__opencliUrl || '';
-                            const method = xhr.__opencliMethod || 'GET';
-                            if (method !== 'GET' && method !== 'HEAD') {
-                                window.__opencliAllRequests.push({ url, method, status: xhr.status, ts: Date.now(), via: 'xhr' });
-                            }
-                            if (MUTATION_RE.test(url)) {
-                                window.__opencliListMutations.push({ url, method, status: xhr.status, ts: Date.now(), via: 'xhr' });
-                            }
-                        });
-                        return OrigXhrSend.apply(this, args);
-                    };
-                }
-                window.__opencliListMutations.length = 0;
-                window.__opencliAllRequests.length = 0;
-
-                const caret = await waitFor(() => findOne('[data-testid="userActions"]'));
-                if (!caret) return { ok: false, message: 'Could not find user actions (…) button. Are you logged in?' };
-                caret.click();
-                await sleep(600);
-                const menuItems = Array.from(document.querySelectorAll('[role="menuitem"]'));
-                const addToListItem = menuItems.find(el => /add\\/remove|从列表|列表|add to list|add or remove/i.test(el.innerText));
-                if (!addToListItem) {
-                    return { ok: false, message: 'Could not find "Add/remove from Lists" menu item' };
-                }
-                addToListItem.click();
-                await sleep(1200);
-                const dialog = await waitFor(() => findOne('[role="dialog"]'));
-                if (!dialog) return { ok: false, message: 'List selection dialog did not open' };
-
-                const targetName = ${JSON.stringify(targetName)};
-                // Find the real scroll container (virtualized list). Try a few candidates.
-                const scrollCandidates = [
-                    dialog.querySelector('[data-viewportview="true"]'),
-                    dialog.querySelector('[aria-label]')?.parentElement,
-                    ...Array.from(dialog.querySelectorAll('div')).filter(d => d.scrollHeight > d.clientHeight + 10),
-                ].filter(Boolean);
-                let row = null;
-                let scrollEl = scrollCandidates[0] || dialog;
-                for (const se of scrollCandidates) {
-                    if (se.scrollHeight > se.clientHeight + 10) { scrollEl = se; break; }
-                }
-                let lastScrollTop = -1;
-                for (let i = 0; i < 12; i++) {
-                    const cells = Array.from(dialog.querySelectorAll('[data-testid="cellInnerDiv"]'));
-                    row = cells.find(c => (c.innerText || '').split('\\n')[0].trim() === targetName);
-                    if (row) break;
-                    // Incremental scroll within the container
-                    const prev = scrollEl.scrollTop;
-                    scrollEl.scrollTop = prev + Math.max(200, scrollEl.clientHeight - 100);
-                    if (scrollEl.scrollTop === prev) {
-                        // Couldn't scroll further. Give up.
-                        if (scrollEl.scrollTop === lastScrollTop) break;
-                    }
-                    lastScrollTop = scrollEl.scrollTop;
-                    await sleep(500);
-                }
-                if (!row) {
-                    const names = Array.from(dialog.querySelectorAll('[data-testid="cellInnerDiv"]'))
-                        .map(c => (c.innerText || '').split('\\n')[0].trim()).filter(Boolean);
-                    const dialogText = (dialog.innerText || '').slice(0, 500);
-                    return { ok: false, message: 'List "' + targetName + '" not found. Cells: [' + names.join(' | ') + ']. DialogText: ' + dialogText };
-                }
-                const listCell = row.querySelector('[data-testid="listCell"]') || row.querySelector('[role="checkbox"]') || row;
-                const readChecked = () => {
-                    const v = listCell.getAttribute('aria-checked');
-                    return v === 'true' || v === 'false' ? v : null;
-                };
-                await sleep(600);
-                let ariaChecked = readChecked();
-                for (let i = 0; i < 8; i++) {
-                    await sleep(500);
-                    const next = readChecked();
-                    if (next && next === ariaChecked) break;
-                    ariaChecked = next || ariaChecked;
-                }
-                const isMember = ariaChecked === 'true';
-                if (isMember) {
-                    const closeBtn = findOne('[data-testid="app-bar-close"]') || findOne('[aria-label="Close"]');
-                    if (closeBtn) closeBtn.click();
-                    return { ok: true, noop: true };
-                }
-                try { listCell.scrollIntoView({ block: 'center' }); } catch {}
-                await sleep(400);
-                const mutationsBefore = window.__opencliListMutations.length;
-                const rowRect = listCell.getBoundingClientRect();
-                // Find the Save button (top-right of dialog). Match by text "Save" / "Done" / CJK equivalents.
-                const saveButton = Array.from(dialog.querySelectorAll('[role="button"], button')).find(b => {
-                    const txt = (b.innerText || '').trim();
-                    return /^(Save|Done|保存|完成|儲存)$/i.test(txt);
+                const r = await fetch(${JSON.stringify(addUrl)}, {
+                    method: 'POST',
+                    headers: Object.assign({}, ${headers}, { 'Content-Type': 'application/json' }),
+                    credentials: 'include',
+                    body: ${JSON.stringify(addBody)},
                 });
-                const saveRect = saveButton ? saveButton.getBoundingClientRect() : null;
-                return {
-                    ok: true,
-                    needsNativeInteraction: true,
-                    rowClickX: Math.round(rowRect.left + rowRect.width / 2),
-                    rowClickY: Math.round(rowRect.top + rowRect.height / 2),
-                    saveClickX: saveRect ? Math.round(saveRect.left + saveRect.width / 2) : null,
-                    saveClickY: saveRect ? Math.round(saveRect.top + saveRect.height / 2) : null,
-                    saveText: saveButton ? (saveButton.innerText || '').trim() : null,
-                    mutationsBefore,
-                    ariaBefore: ariaChecked,
-                };
+                const text = await r.text();
+                let body;
+                let raw = null;
+                try { body = JSON.parse(text); } catch { body = null; raw = text.slice(0, 300); }
+                const list = body && body.data && body.data.list ? body.data.list : null;
+                return JSON.stringify([
+                    r.ok,
+                    r.status,
+                    list ? list.member_count : null,
+                    list ? list.is_member : null,
+                    body && body.errors ? body.errors : null,
+                    raw,
+                    null,
+                ]);
             } catch (e) {
-                return { ok: false, message: 'UI error: ' + (e?.message || String(e)) };
-            }
-        })()`);
-
-        if (!uiResult.ok) {
-            throw new CommandExecutionError(`Failed to add @${username} to list ${listId}: ${uiResult.message}`);
-        }
-
-        let verifiedBy = null;
-        if (uiResult.needsNativeInteraction) {
-            if (typeof page.nativeClick !== 'function' || typeof page.nativeKeyPress !== 'function') {
-                throw new CommandExecutionError('Requires up-to-date Chrome extension (nativeClick + nativeKeyPress).');
-            }
-            if (!uiResult.saveClickX) {
-                throw new CommandExecutionError(`Save button not found in dialog (X expected text Save/Done). Dialog structure may have changed.`);
-            }
-            const memberCountBefore = Number(targetList.members) || 0;
-            // 1. Trusted click on row → aria flips false→true (optimistic UI)
-            await page.nativeClick(uiResult.rowClickX, uiResult.rowClickY);
-            await new Promise((r) => setTimeout(r, 800));
-            // 2. Trusted click on Save button → X commits to server
-            await page.nativeClick(uiResult.saveClickX, uiResult.saveClickY);
-            await new Promise((r) => setTimeout(r, 3500));
-            // Ground truth: re-fetch ListsManagementPageTimeline and compare member_count
-            const listsAfter = await page.evaluate(`async () => {
-                const r = await fetch(${JSON.stringify(listsUrl)}, { headers: ${headers}, credentials: 'include' });
-                if (!r.ok) return { __error: 'HTTP ' + r.status };
-                return await r.json();
-            }`);
-            const parsedAfter = listsAfter && !listsAfter.__error
-                ? parseListsManagement(listsAfter, new Set())
-                : [];
-            const afterList = parsedAfter.find((l) => l.id === listId);
-            const memberCountAfter = afterList ? Number(afterList.members) || 0 : -1;
-            if (memberCountAfter > memberCountBefore) {
-                verifiedBy = `member_count ${memberCountBefore} → ${memberCountAfter}`;
-            } else {
-                throw new CommandExecutionError(`Failed to add @${username} to list ${listId}: member_count unchanged (${memberCountBefore} → ${memberCountAfter}). X's UI flipped but did not commit — try reloading page/extension.`);
+                return JSON.stringify([false, 0, null, null, null, null, String(e)]);
             }
+        }`);
+        const addResultJson = unwrap(addResultJsonRaw);
+        let addResultTuple;
+        try {
+            addResultTuple = JSON.parse(addResultJson);
+        } catch {
+            throw new CommandExecutionError(`Failed to add @${username} to list ${listId}: malformed mutation response envelope`);
         }
+        const addResult = Object.create(null);
+        addResult.httpOk = Boolean(addResultTuple?.[0]);
+        addResult.status = Number(addResultTuple?.[1]) || 0;
+        addResult.mc = addResultTuple?.[2];
+        addResult.isMember = addResultTuple?.[3];
+        addResult.errors = addResultTuple?.[4];
+        addResult.raw = addResultTuple?.[5];
+        addResult.fetchError = addResultTuple?.[6];
 
-        return [{
-            listId,
-            username,
-            userId: String(userId),
-            status: uiResult.noop ? 'noop' : 'success',
-            message: uiResult.noop
-                ? `@${username} is already a member of list ${listId}`
-                : `Added @${username} to list ${listId} (verified via ${verifiedBy})`,
-        }];
+        return [buildListAddMemberRow({ addResult, memberCountBefore, listId, username, userId })];
     },
 });