publishport-opencli 1.8.5-pp.2 → 1.8.5-pp.21

package/clis/twitter/profile.js

@@ -1,97 +1,12 @@
-import { ArgumentError, AuthRequiredError, CommandExecutionError, EmptyResultError } from '@jackwener/opencli/errors';
-import { cli, Strategy } from '@jackwener/opencli/registry';
-import { describeTwitterApiError, normalizeTwitterScreenName, resolveTwitterQueryId, unwrapBrowserResult } from './shared.js';
-import { TWITTER_BEARER_TOKEN } from './utils.js';
-const USER_BY_SCREEN_NAME_QUERY_ID = 'IGgvgiOx4QZndDHuD3x9TQ';
-
-function isPlainObject(value) {
-    return value != null && typeof value === 'object' && !Array.isArray(value);
-}
-
-function stringField(value) {
-    return typeof value === 'string' ? value : '';
-}
-
-export function mapTwitterProfileResult(result, screenName) {
-    if (!isPlainObject(result)) {
-        throw new CommandExecutionError(`Twitter profile response for @${screenName} is malformed`);
-    }
-    const hasLegacy = isPlainObject(result.legacy);
-    const hasCore = isPlainObject(result.core);
-    if (!hasLegacy && !hasCore) {
-        throw new CommandExecutionError(`Twitter profile response for @${screenName} is missing profile fields`);
-    }
-    const legacy = hasLegacy ? result.legacy : {};
-    const core = hasCore ? result.core : {};
-    if (!stringField(core.screen_name) && !stringField(legacy.screen_name) && !stringField(core.name) && !stringField(legacy.name) && !stringField(core.created_at) && !stringField(legacy.created_at)) {
-        throw new CommandExecutionError(`Twitter profile response for @${screenName} is missing profile identity fields`);
-    }
-    const location = isPlainObject(result.location) ? result.location : {};
-    const expandedUrl = legacy.entities?.url?.urls?.[0]?.expanded_url || '';
-    return [{
-        screen_name: stringField(core.screen_name) || stringField(legacy.screen_name) || screenName,
-        name: stringField(core.name) || stringField(legacy.name),
-        bio: stringField(legacy.description),
-        location: stringField(location.location) || stringField(legacy.location),
-        url: stringField(expandedUrl),
-        followers: legacy.followers_count || 0,
-        following: legacy.friends_count || 0,
-        tweets: legacy.statuses_count || 0,
-        likes: legacy.favourites_count || 0,
-        verified: Boolean(result.is_blue_verified || legacy.verified),
-        created_at: stringField(core.created_at) || stringField(legacy.created_at),
-    }];
-}
-
-cli({
-    site: 'twitter',
-    name: 'profile',
-    access: 'read',
-    description: 'Fetch a Twitter user profile — bio, stats, etc. (defaults to the logged-in user when no username is given)',
-    domain: 'x.com',
-    strategy: Strategy.COOKIE,
-    browser: true,
-    args: [
-        { name: 'username', type: 'string', positional: true, help: 'Twitter screen name (with or without @). Defaults to the logged-in user when omitted.' },
-    ],
-    columns: ['screen_name', 'name', 'bio', 'location', 'url', 'followers', 'following', 'tweets', 'likes', 'verified', 'created_at'],
-    func: async (page, kwargs) => {
-        const rawUsername = String(kwargs.username ?? '').trim();
-        let username = normalizeTwitterScreenName(rawUsername);
-        if (rawUsername && !username) {
-            throw new ArgumentError('twitter profile username must be a valid Twitter/X handle', 'Example: opencli twitter profile @jack');
-        }
-        // If no username, 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) {
-            await page.goto('https://x.com/home');
-            await page.wait({ selector: '[data-testid="primaryColumn"]' });
-            const href = unwrapBrowserResult(await page.evaluate(`() => {
+import{ArgumentError as k,AuthRequiredError as z,CommandExecutionError as M,EmptyResultError as B}from"@jackwener/opencli/errors";import{cli as I,Strategy as q}from"@jackwener/opencli/registry";import{describeTwitterApiError as U,normalizeTwitterScreenName as Q,resolveTwitterQueryId as b,unwrapBrowserResult as Y}from"./shared.js";import{TWITTER_BEARER_TOKEN as x}from"./utils.js";const S="IGgvgiOx4QZndDHuD3x9TQ";function W(G){return G!=null&&typeof G==="object"&&!Array.isArray(G)}function H(G){return typeof G==="string"?G:""}export function mapTwitterProfileResult(G,J){if(!W(G))throw new M(`Twitter profile response for @${J} is malformed`);const K=W(G.legacy),Z=W(G.core);if(!K&&!Z)throw new M(`Twitter profile response for @${J} is missing profile fields`);const V=K?G.legacy:{},$=Z?G.core:{};if(!H($.screen_name)&&!H(V.screen_name)&&!H($.name)&&!H(V.name)&&!H($.created_at)&&!H(V.created_at))throw new M(`Twitter profile response for @${J} is missing profile identity fields`);const L=W(G.location)?G.location:{},X=V.entities?.url?.urls?.[0]?.expanded_url||"";return[{screen_name:H($.screen_name)||H(V.screen_name)||J,name:H($.name)||H(V.name),bio:H(V.description),location:H(L.location)||H(V.location),url:H(X),followers:V.followers_count||0,following:V.friends_count||0,tweets:V.statuses_count||0,likes:V.favourites_count||0,verified:Boolean(G.is_blue_verified||V.verified),created_at:H($.created_at)||H(V.created_at)}]}I({site:"twitter",name:"profile",access:"read",description:"Fetch a Twitter user profile — bio, stats, etc. (defaults to the logged-in user when no username is given)",domain:"x.com",strategy:q.COOKIE,browser:!0,args:[{name:"username",type:"string",positional:!0,help:"Twitter screen name (with or without @). Defaults to the logged-in user when omitted."}],columns:["screen_name","name","bio","location","url","followers","following","tweets","likes","verified","created_at"],func:async(G,J)=>{const K=String(J.username??"").trim();let Z=Q(K);if(K&&!Z)throw new k("twitter profile username must be a valid Twitter/X handle","Example: opencli twitter profile @jack");if(!Z){await G.goto("https://x.com/home");await G.wait({selector:'[data-testid="primaryColumn"]'});const D=Y(await G.evaluate(`() => {
         const link = document.querySelector('a[data-testid="AppTabBar_Profile_Link"]');
         return link ? link.getAttribute('href') : null;
-      }`));
-            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?');
-        }
-        // Navigate directly to the user's profile page (gives us cookie context)
-        await page.goto(`https://x.com/${username}`);
-        await page.wait(3);
-        // Read CSRF token directly from the cookie store via CDP — zero page.evaluate round-trip
-        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)');
-        const queryId = await resolveTwitterQueryId(page, 'UserByScreenName', USER_BY_SCREEN_NAME_QUERY_ID);
-        const rawResult = unwrapBrowserResult(await page.evaluate(`
+      }`));if(!D||typeof D!=="string")throw new z("x.com","Could not detect logged-in user. Are you logged in?");Z=Q(D);if(!Z)throw new z("x.com","Could not detect logged-in user. Are you logged in?")}await G.goto(`https://x.com/${Z}`);await G.wait(3);const $=(await G.getCookies({url:"https://x.com"})).find((D)=>D.name==="ct0")?.value||null;if(!$)throw new z("x.com","Not logged into x.com (no ct0 cookie)");const L=await b(G,"UserByScreenName",S),X=Y(await G.evaluate(`
       async () => {
-        const screenName = "${username}";
-        const ct0 = ${JSON.stringify(ct0)};
+        const screenName = "${Z}";
+        const ct0 = ${JSON.stringify($)};
 
-        const bearer = ${JSON.stringify(TWITTER_BEARER_TOKEN)};
+        const bearer = ${JSON.stringify(x)};
         const headers = {
           'Authorization': 'Bearer ' + decodeURIComponent(bearer),
           'X-Csrf-Token': ct0,
@@ -118,7 +33,7 @@ cli({
           responsive_web_graphql_timeline_navigation_enabled: true,
         });
 
-        const url = '/i/api/graphql/' + ${JSON.stringify(queryId)} + '/UserByScreenName?variables='
+        const url = '/i/api/graphql/' + ${JSON.stringify(L)} + '/UserByScreenName?variables='
           + encodeURIComponent(variables)
           + '&features=' + encodeURIComponent(features);
 
@@ -148,27 +63,4 @@ cli({
         if (!result) return {ok: false, notFound: true, error: 'User @' + screenName + ' not found'};
         return {ok: true, result};
       }
-    `));
-        if (!isPlainObject(rawResult)) {
-            throw new CommandExecutionError('Twitter profile response payload is malformed');
-        }
-        if (!rawResult.ok) {
-            // For HTTP errors, use fork's rich code mapping (429/401/403/404/5xx differentiation
-            // from describeTwitterApiError); fall back to the plain message for non-HTTP failures
-            // (fetch threw, JSON parse failed, payload malformed).
-            const message = typeof rawResult.httpStatus === 'number'
-                ? describeTwitterApiError('UserByScreenName', rawResult.httpStatus, rawResult.hint)
-                : rawResult.error + (rawResult.hint ? ` (${rawResult.hint})` : '');
-            if (rawResult.auth) {
-                throw new AuthRequiredError('x.com', message);
-            }
-            if (rawResult.notFound) {
-                throw new EmptyResultError('twitter profile', message);
-            }
-            throw new CommandExecutionError(message);
-        }
-        return mapTwitterProfileResult(rawResult.result, username);
-    }
-});
-
-export const __test__ = { mapTwitterProfileResult };
+    `));if(!W(X))throw new M("Twitter profile response payload is malformed");if(!X.ok){const D=typeof X.httpStatus==="number"?U("UserByScreenName",X.httpStatus,X.hint):X.error+(X.hint?` (${X.hint})`:"");if(X.auth)throw new z("x.com",D);if(X.notFound)throw new B("twitter profile",D);throw new M(D)}return mapTwitterProfileResult(X.result,Z)}});export const __test__={mapTwitterProfileResult};

package/clis/twitter/quote.js

@@ -1,28 +1,6 @@
-import * as fs from 'node:fs';
-import { CommandExecutionError } from '@jackwener/opencli/errors';
-import { cli, Strategy } from '@jackwener/opencli/registry';
-import { parseTweetUrl, buildTwitterArticleScopeSource } from './shared.js';
-import {
-    COMPOSER_FILE_INPUT_SELECTOR,
-    attachComposerImage,
-    downloadRemoteImage,
-    resolveImagePath,
-} from './utils.js';
-
-function buildQuoteComposerUrl(url) {
-    // Twitter/X quote-tweet compose URL: the `url` param attaches the source
-    // tweet as a quoted card. Validating tweet-id shape early surfaces obvious
-    // typos before any browser interaction.
-    const parsed = parseTweetUrl(url);
-    return `https://x.com/compose/post?url=${encodeURIComponent(parsed.url)}`;
-}
-
-async function openQuoteComposerFromRetweetMenu(page, target) {
-    await page.goto(target.url, { waitUntil: 'load', settleMs: 2500 });
-    await page.wait({ selector: '[data-testid="primaryColumn"]', timeout: 15 });
-    const opened = await page.evaluate(`(async () => {
+import*as Z from"node:fs";import{CommandExecutionError as J}from"@jackwener/opencli/errors";import{cli as $,Strategy as f}from"@jackwener/opencli/registry";import{parseTweetUrl as K,buildTwitterArticleScopeSource as V}from"./shared.js";import{COMPOSER_FILE_INPUT_SELECTOR as W,attachComposerImage as X,downloadRemoteImage as y,resolveImagePath as A}from"./utils.js";function N(q){const j=K(q);return`https://x.com/compose/post?url=${encodeURIComponent(j.url)}`}async function D(q,j){await q.goto(j.url,{waitUntil:"load",settleMs:2500});await q.wait({selector:'[data-testid="primaryColumn"]',timeout:15});const v=await q.evaluate(`(async () => {
         try {
-            ${buildTwitterArticleScopeSource(target.id)}
+            ${V(j.id)}
             const visible = (el) => !!el && (el.offsetParent !== null || el.getClientRects().length > 0);
             let retweetBtn = null;
             let targetArticle = null;
@@ -52,16 +30,9 @@ async function openQuoteComposerFromRetweetMenu(page, target) {
         } catch (e) {
             return { ok: false, message: e.toString() };
         }
-    })()`);
-    if (!opened?.ok) return opened;
-    await page.wait({ selector: '[data-testid="tweetTextarea_0"]', timeout: 15 });
-    return { ok: true };
-}
-
-async function submitQuote(page, text, target, { allowMenuFallback = true } = {}) {
-    return page.evaluate(`(async () => {
+    })()`);if(!v?.ok)return v;await q.wait({selector:'[data-testid="tweetTextarea_0"]',timeout:15});return{ok:!0}}async function Y(q,j,v,{allowMenuFallback:z=!0}={}){return q.evaluate(`(async () => {
         try {
-            ${buildTwitterArticleScopeSource(target.id)}
+            ${V(v.id)}
             const visible = (el) => !!el && (el.offsetParent !== null || el.getClientRects().length > 0);
             const normalize = s => String(s || '').replace(/\\u00a0/g, ' ').replace(/\\s+/g, ' ').trim();
             const insertTextIntoBox = async (box, value) => {
@@ -86,7 +57,7 @@ async function submitQuote(page, text, target, { allowMenuFallback = true } = {}
                 return { ok: false, message: 'Could not find the quote composer text area. Are you logged in?' };
             }
 
-            const textToInsert = ${JSON.stringify(text)};
+            const textToInsert = ${JSON.stringify(j)};
             if (!(await insertTextIntoBox(box, textToInsert))) {
                 return { ok: false, message: 'Could not verify quote text in the composer after typing.' };
             }
@@ -107,7 +78,7 @@ async function submitQuote(page, text, target, { allowMenuFallback = true } = {}
                 cardAttempts++;
             }
             if (!hasQuoteCard) {
-                const allowMenuFallback = ${JSON.stringify(allowMenuFallback)};
+                const allowMenuFallback = ${JSON.stringify(z)};
                 if (allowMenuFallback) {
                     return { ok: false, retryWithMenuFallback: true, message: 'Quote target did not render in the dedicated composer; retrying through the repost menu.' };
                 }
@@ -149,86 +120,4 @@ async function submitQuote(page, text, target, { allowMenuFallback = true } = {}
         } catch (e) {
             return { ok: false, message: e.toString() };
         }
-    })()`);
-}
-
-cli({
-    site: 'twitter',
-    name: 'quote',
-    access: 'write',
-    description: 'Quote-tweet a specific tweet with your own text, optionally with a local or remote image',
-    domain: 'x.com',
-    strategy: Strategy.UI,
-    browser: true,
-    args: [
-        { name: 'url', type: 'string', required: true, positional: true, help: 'The URL of the tweet to quote' },
-        { name: 'text', type: 'string', required: true, positional: true, help: 'The text content of your quote' },
-        { name: 'image', help: 'Optional local image path to attach to the quote tweet' },
-        { name: 'image-url', help: 'Optional remote image URL to download and attach to the quote tweet' },
-    ],
-    columns: ['status', 'message', 'text'],
-    func: async (page, kwargs) => {
-        if (!page)
-            throw new CommandExecutionError('Browser session required for twitter quote');
-        if (kwargs.image && kwargs['image-url']) {
-            throw new CommandExecutionError('Use either --image or --image-url, not both.');
-        }
-
-        // Validate URL (typed ArgumentError on malformed/off-domain inputs)
-        // before any browser interaction or remote image download.
-        const target = parseTweetUrl(kwargs.url);
-
-        let localImagePath;
-        let cleanupDir;
-        try {
-            if (kwargs.image) {
-                localImagePath = resolveImagePath(kwargs.image);
-            } else if (kwargs['image-url']) {
-                const downloaded = await downloadRemoteImage(kwargs['image-url']);
-                localImagePath = downloaded.absPath;
-                cleanupDir = downloaded.cleanupDir;
-            }
-
-            // Dedicated composer is more reliable than the inline quote-tweet button.
-            await page.goto(`https://x.com/compose/post?url=${encodeURIComponent(target.url)}`, { waitUntil: 'load', settleMs: 2500 });
-            await page.wait({ selector: '[data-testid="tweetTextarea_0"]', timeout: 15 });
-
-            if (localImagePath) {
-                await page.wait({ selector: COMPOSER_FILE_INPUT_SELECTOR, timeout: 20 });
-                await attachComposerImage(page, localImagePath);
-            }
-
-            let result = await submitQuote(page, kwargs.text, target);
-            if (result.retryWithMenuFallback) {
-                const opened = await openQuoteComposerFromRetweetMenu(page, target);
-                if (!opened?.ok) {
-                    return [{ status: 'failed', message: opened?.message ?? 'Could not open quote composer from repost menu.', text: kwargs.text }];
-                }
-                if (localImagePath) {
-                    await page.wait({ selector: COMPOSER_FILE_INPUT_SELECTOR, timeout: 20 });
-                    await attachComposerImage(page, localImagePath);
-                }
-                result = await submitQuote(page, kwargs.text, target, { allowMenuFallback: false });
-            }
-            if (result.ok) {
-                // Wait for network submission to complete
-                await page.wait(3);
-            }
-            return [{
-                    status: result.ok ? 'success' : 'failed',
-                    message: result.message,
-                    text: kwargs.text,
-                    ...(kwargs.image ? { image: kwargs.image } : {}),
-                    ...(kwargs['image-url'] ? { 'image-url': kwargs['image-url'] } : {}),
-                }];
-        } finally {
-            if (cleanupDir) {
-                fs.rmSync(cleanupDir, { recursive: true, force: true });
-            }
-        }
-    }
-});
-
-export const __test__ = {
-    buildQuoteComposerUrl,
-};
+    })()`)}$({site:"twitter",name:"quote",access:"write",description:"Quote-tweet a specific tweet with your own text, optionally with a local or remote image",domain:"x.com",strategy:f.UI,browser:!0,args:[{name:"url",type:"string",required:!0,positional:!0,help:"The URL of the tweet to quote"},{name:"text",type:"string",required:!0,positional:!0,help:"The text content of your quote"},{name:"image",help:"Optional local image path to attach to the quote tweet"},{name:"image-url",help:"Optional remote image URL to download and attach to the quote tweet"}],columns:["status","message","text"],func:async(q,j)=>{if(!q)throw new J("Browser session required for twitter quote");if(j.image&&j["image-url"])throw new J("Use either --image or --image-url, not both.");const v=K(j.url);let z,H;try{if(j.image)z=A(j.image);else if(j["image-url"]){const G=await y(j["image-url"]);z=G.absPath;H=G.cleanupDir}await q.goto(`https://x.com/compose/post?url=${encodeURIComponent(v.url)}`,{waitUntil:"load",settleMs:2500});await q.wait({selector:'[data-testid="tweetTextarea_0"]',timeout:15});if(z){await q.wait({selector:W,timeout:20});await X(q,z)}let B=await Y(q,j.text,v);if(B.retryWithMenuFallback){const G=await D(q,v);if(!G?.ok)return[{status:"failed",message:G?.message??"Could not open quote composer from repost menu.",text:j.text}];if(z){await q.wait({selector:W,timeout:20});await X(q,z)}B=await Y(q,j.text,v,{allowMenuFallback:!1})}if(B.ok)await q.wait(3);return[{status:B.ok?"success":"failed",message:B.message,text:j.text,...j.image?{image:j.image}:{},...j["image-url"]?{"image-url":j["image-url"]}:{}}]}finally{if(H)Z.rmSync(H,{recursive:!0,force:!0})}}});export const __test__={buildQuoteComposerUrl:N};

package/clis/twitter/reply-dm.js

@@ -1,34 +1,4 @@
-import { CommandExecutionError } from '@jackwener/opencli/errors';
-import { cli, Strategy } from '@jackwener/opencli/registry';
-cli({
-    site: 'twitter',
-    name: 'reply-dm',
-    access: 'write',
-    description: 'Send a message to recent DM conversations',
-    domain: 'x.com',
-    strategy: Strategy.UI,
-    browser: true,
-    args: [
-        { name: 'text', type: 'string', required: true, positional: true, help: 'Message text to send (e.g. "我的微信 wxkabi")' },
-        { name: 'max', type: 'int', required: false, default: 20, help: 'Maximum number of conversations to reply to (default: 20)' },
-        { name: 'skip-replied', type: 'boolean', required: false, default: true, help: 'Skip conversations where you already sent the same text (default: true)' },
-        { name: 'timeout', type: 'int', required: false, default: 600, help: 'Max seconds for the overall command (default: 600 — batch op)' },
-    ],
-    columns: ['index', 'status', 'user', 'message'],
-    func: async (page, kwargs) => {
-        if (!page)
-            throw new CommandExecutionError('Browser session required for twitter reply-dm');
-        const messageText = kwargs.text;
-        const maxSend = kwargs.max ?? 20;
-        const skipReplied = kwargs['skip-replied'] !== false;
-        const results = [];
-        let sentCount = 0;
-        // Step 1: Navigate to messages to get conversation list
-        await page.goto('https://x.com/messages');
-        await page.wait({ selector: '[data-testid="primaryColumn"]' });
-        // Step 2: Collect conversations with scroll-to-load
-        const needed = maxSend + 10; // extra buffer for skips
-        const convList = await page.evaluate(`(async () => {
+import{CommandExecutionError as N}from"@jackwener/opencli/errors";import{cli as O,Strategy as P}from"@jackwener/opencli/registry";import{assertLiteralContent as Q}from"../_shared/content-guard.js";O({site:"twitter",name:"reply-dm",access:"write",description:"Send a message to recent DM conversations",domain:"x.com",strategy:P.UI,browser:!0,args:[{name:"text",type:"string",required:!0,positional:!0,help:'Message text to send (e.g. "我的微信 wxkabi")'},{name:"max",type:"int",required:!1,default:20,help:"Maximum number of conversations to reply to (default: 20)"},{name:"skip-replied",type:"boolean",required:!1,default:!0,help:"Skip conversations where you already sent the same text (default: true)"},{name:"timeout",type:"int",required:!1,default:600,help:"Max seconds for the overall command (default: 600 — batch op)"}],columns:["index","status","user","message"],func:async(b,A)=>{if(!b)throw new N("Browser session required for twitter reply-dm");const F=A.text;Q(F,{label:"私信内容"});const G=A.max??20,J=A["skip-replied"]!==!1,f=[];let B=0;await b.goto("https://x.com/messages");await b.wait({selector:'[data-testid="primaryColumn"]'});const K=G+10,D=await b.evaluate(`(async () => {
       try {
         // Wait for initial items
         let attempts = 0;
@@ -40,7 +10,7 @@ cli({
         }
 
         // Scroll to load more conversations
-        const needed = ${needed};
+        const needed = ${K};
         const seenIds = new Set();
         let noNewCount = 0;
 
@@ -93,32 +63,16 @@ cli({
       } catch(e) {
         return { ok: false, error: String(e), conversations: [], total: 0 };
       }
-    })()`);
-        if (!convList?.ok || !convList.conversations?.length) {
-            return [{ index: 1, status: 'info', user: 'System', message: 'No conversations found' }];
-        }
-        const conversations = convList.conversations;
-        // Step 3: Iterate through conversations and send message
-        for (const conv of conversations) {
-            if (sentCount >= maxSend)
-                break;
-            const convUrl = conv.convId
-                ? `https://x.com/messages/${conv.convId}`
-                : conv.href;
-            if (!convUrl)
-                continue;
-            await page.goto(convUrl);
-            await page.wait(3);
-            const sendResult = await page.evaluate(`(async () => {
+    })()`);if(!D?.ok||!D.conversations?.length)return[{index:1,status:"info",user:"System",message:"No conversations found"}];const M=D.conversations;for(const q of M){if(B>=G)break;const H=q.convId?`https://x.com/messages/${q.convId}`:q.href;if(!H)continue;await b.goto(H);await b.wait(3);const z=await b.evaluate(`(async () => {
         try {
-          const messageText = ${JSON.stringify(messageText)};
-          const skipReplied = ${skipReplied};
+          const messageText = ${JSON.stringify(F)};
+          const skipReplied = ${J};
 
           // Get username from conversation
           const dmHeader = document.querySelector('[data-testid="DmActivityContainer"] [dir="ltr"] span') ||
                            document.querySelector('[data-testid="conversation-header"]') ||
                            document.querySelector('[data-testid="DmActivityContainer"] h2');
-          const username = dmHeader ? dmHeader.innerText.trim().split('\\\\n')[0] : '${conv.user}';
+          const username = dmHeader ? dmHeader.innerText.trim().split('\\\\n')[0] : '${q.user}';
 
           // Check if we already sent this message
           if (skipReplied) {
@@ -155,29 +109,4 @@ cli({
         } catch(e) {
           return { status: 'error', user: 'system', message: String(e) };
         }
-      })()`);
-            if (sendResult?.status === 'sent') {
-                sentCount++;
-                results.push({
-                    index: sentCount,
-                    status: 'sent',
-                    user: sendResult.user || conv.user,
-                    message: sendResult.message,
-                });
-            }
-            else if (sendResult?.status === 'skipped') {
-                results.push({
-                    index: results.length + 1,
-                    status: 'skipped',
-                    user: sendResult.user || conv.user,
-                    message: sendResult.message,
-                });
-            }
-            await page.wait(1);
-        }
-        if (results.length === 0) {
-            results.push({ index: 0, status: 'info', user: 'System', message: 'No conversations processed' });
-        }
-        return results;
-    }
-});
+      })()`);if(z?.status==="sent"){B++;f.push({index:B,status:"sent",user:z.user||q.user,message:z.message})}else if(z?.status==="skipped")f.push({index:f.length+1,status:"skipped",user:z.user||q.user,message:z.message});await b.wait(1)}if(f.length===0)f.push({index:0,status:"info",user:"System",message:"No conversations processed"});return f}});

package/clis/twitter/reply.js

@@ -1,63 +1,11 @@
-import * as fs from 'node:fs';
-import * as path from 'node:path';
-import { CommandExecutionError } from '@jackwener/opencli/errors';
-import { cli, Strategy } from '@jackwener/opencli/registry';
-import { parseTweetUrl } from './shared.js';
-import {
-    COMPOSER_FILE_INPUT_SELECTOR,
-    attachComposerImage,
-    downloadRemoteImage,
-    resolveImagePath,
-} from './utils.js';
-
-const COMPOSER_SELECTOR = '[data-testid="tweetTextarea_0"]';
-const SUBMIT_POLL_MS = 500;
-const SUBMIT_TIMEOUT_MS = 15_000;
-
-function buildReplyComposerUrl(rawUrl) {
-    // Replaces the legacy local extractTweetId which used `/\/status\/(\d+)/`
-    // (silent: matched `/status/1234567` on substring `/status/123` and
-    // accepted any host). parseTweetUrl bubbles ArgumentError on
-    // malformed/off-domain inputs.
-    const target = parseTweetUrl(rawUrl);
-    return `https://x.com/compose/post?in_reply_to=${target.id}`;
-}
-
-function isPromiseCollectedError(err) {
-    const msg = err instanceof Error ? err.message : String(err);
-    return msg.includes('Promise was collected');
-}
-
-async function openReplyComposer(page, rawUrl) {
-    await page.goto(buildReplyComposerUrl(rawUrl), { waitUntil: 'load', settleMs: 2500 });
-    try {
-        await page.wait({ selector: COMPOSER_SELECTOR, timeout: 15 });
-        return { ok: true };
-    } catch {
-        // X sometimes leaves /compose/post?in_reply_to=<id> on the Home
-        // timeline behind a loading dialog. Fall back to the canonical tweet
-        // page and click the visible Reply action there.
-        await page.goto(rawUrl, { waitUntil: 'load', settleMs: 2500 });
-        const clicked = await page.evaluate(`(() => {
+import*as W from"node:fs";import*as f from"node:path";import{CommandExecutionError as K}from"@jackwener/opencli/errors";import{cli as v,Strategy as F}from"@jackwener/opencli/registry";import{parseTweetUrl as j}from"./shared.js";import{assertLiteralContent as B}from"../_shared/content-guard.js";import{COMPOSER_FILE_INPUT_SELECTOR as b,attachComposerImage as L,downloadRemoteImage as h,resolveImagePath as O}from"./utils.js";const Z='[data-testid="tweetTextarea_0"]',V=500,$=15000;function N(z){return`https://x.com/compose/post?in_reply_to=${j(z).id}`}function X(z){return(z instanceof Error?z.message:String(z)).includes("Promise was collected")}async function y(z,q){await z.goto(N(q),{waitUntil:"load",settleMs:2500});try{await z.wait({selector:Z,timeout:15});return{ok:!0}}catch{await z.goto(q,{waitUntil:"load",settleMs:2500});const A=await z.evaluate(`(() => {
             const visible = (el) => !!el && (el.offsetParent !== null || el.getClientRects().length > 0);
             const buttons = Array.from(document.querySelectorAll('[data-testid="reply"]'));
             const btn = buttons.find((el) => visible(el) && !el.disabled && el.getAttribute('aria-disabled') !== 'true');
             if (!btn) return { ok: false, message: 'Could not find the reply button on the target tweet.' };
             btn.click();
             return { ok: true };
-        })()`);
-        if (!clicked?.ok) return clicked;
-        await page.wait({ selector: COMPOSER_SELECTOR, timeout: 15 });
-        return { ok: true };
-    }
-}
-
-async function insertReplyText(page, text) {
-    // beforeunload neutralization is handled globally by stealth.ts (#14),
-    // injected on every nav (covers x.com re-registering after media staging).
-    // PR #1972's extra per-command strip here was redundant and inserted an
-    // extra page.evaluate that desynced the reply flow, so it is dropped.
-    return page.evaluate(`(async () => {
+        })()`);if(!A?.ok)return A;await z.wait({selector:Z,timeout:15});return{ok:!0}}}async function D(z,q){return z.evaluate(`(async () => {
       try {
           const visible = (el) => !!el && (el.offsetParent !== null || el.getClientRects().length > 0);
           const boxes = Array.from(document.querySelectorAll('[data-testid="tweetTextarea_0"]'));
@@ -67,7 +15,7 @@ async function insertReplyText(page, text) {
           }
 
           box.focus();
-          const textToInsert = ${JSON.stringify(text)};
+          const textToInsert = ${JSON.stringify(q)};
           // execCommand('insertText') is more reliable with Twitter's Draft.js editor
           if (!document.execCommand('insertText', false, textToInsert)) {
               // Fallback to paste event if execCommand fails
@@ -90,15 +38,10 @@ async function insertReplyText(page, text) {
       } catch (e) {
           return { ok: false, message: e.toString() };
       }
-  })()`);
-}
-
-async function clickReplyButton(page) {
-    const iterations = Math.ceil(SUBMIT_TIMEOUT_MS / SUBMIT_POLL_MS);
-    return page.evaluate(`(async () => {
+  })()`)}async function I(z){const q=Math.ceil($/V);return z.evaluate(`(async () => {
       try {
           const visible = (el) => !!el && (el.offsetParent !== null || el.getClientRects().length > 0);
-          for (let i = 0; i < ${JSON.stringify(iterations)}; i++) {
+          for (let i = 0; i < ${JSON.stringify(q)}; i++) {
               const buttons = Array.from(
                   document.querySelectorAll('[data-testid="tweetButton"], [data-testid="tweetButtonInline"]')
               );
@@ -107,17 +50,13 @@ async function clickReplyButton(page) {
                   btn.click();
                   return { ok: true };
               }
-              await new Promise(r => setTimeout(r, ${JSON.stringify(SUBMIT_POLL_MS)}));
+              await new Promise(r => setTimeout(r, ${JSON.stringify(V)}));
           }
           return { ok: false, message: 'Reply button is disabled or not found.' };
       } catch (e) {
           return { ok: false, message: e.toString() };
       }
-  })()`);
-}
-
-async function detectReplySent(page) {
-    return page.evaluate(`(() => {
+  })()`)}async function M(z){return z.evaluate(`(() => {
         const visible = (el) => !!el && (el.offsetParent !== null || el.getClientRects().length > 0);
         const toasts = Array.from(document.querySelectorAll('[role="alert"], [data-testid="toast"]'))
             .filter((el) => visible(el));
@@ -129,19 +68,13 @@ async function detectReplySent(page) {
             message: 'Reply posted successfully.',
             url: link?.href || link?.getAttribute('href') || undefined
         };
-    })()`);
-}
-
-async function waitForReplySent(page, text) {
-    const iterations = Math.ceil(SUBMIT_TIMEOUT_MS / SUBMIT_POLL_MS);
-    try {
-        return await page.evaluate(`(async () => {
-            const expected = ${JSON.stringify(text)};
+    })()`)}async function R(z,q){const A=Math.ceil($/V);try{return await z.evaluate(`(async () => {
+            const expected = ${JSON.stringify(q)};
             const normalize = s => String(s || '').replace(/\\u00a0/g, ' ').replace(/\\s+/g, ' ').trim();
             const expectedText = normalize(expected);
             const visible = (el) => !!el && (el.offsetParent !== null || el.getClientRects().length > 0);
-            for (let i = 0; i < ${JSON.stringify(iterations)}; i++) {
-                await new Promise(r => setTimeout(r, ${JSON.stringify(SUBMIT_POLL_MS)}));
+            for (let i = 0; i < ${JSON.stringify(A)}; i++) {
+                await new Promise(r => setTimeout(r, ${JSON.stringify(V)}));
                 const toasts = Array.from(document.querySelectorAll('[role="alert"], [data-testid="toast"]'))
                     .filter((el) => visible(el));
                 const successToast = toasts.find((el) => /sent|posted|your post was sent|your tweet was sent/i.test(el.textContent || ''));
@@ -161,92 +94,4 @@ async function waitForReplySent(page, text) {
                 if (!composerStillHasText) return { ok: true, message: 'Reply posted successfully.' };
             }
             return { ok: false, message: 'Reply submission did not complete before timeout.' };
-        })()`);
-    } catch (err) {
-        // X may route the SPA immediately after click, making CDP collect the
-        // polling promise even though the reply was submitted. If the page now
-        // shows the success toast, report success instead of a false negative.
-        if (!isPromiseCollectedError(err)) throw err;
-        await page.wait(2);
-        const recovered = await detectReplySent(page);
-        if (recovered?.ok) return recovered;
-        throw err;
-    }
-}
-
-async function submitReply(page, text) {
-    const typed = await insertReplyText(page, text);
-    if (!typed?.ok) return typed;
-    let clicked;
-    try {
-        clicked = await clickReplyButton(page);
-    } catch (err) {
-        if (!isPromiseCollectedError(err)) throw err;
-    }
-    if (clicked && !clicked.ok) return clicked;
-    return waitForReplySent(page, text);
-}
-
-cli({
-    site: 'twitter',
-    name: 'reply',
-    access: 'write',
-    description: 'Reply to a specific tweet, optionally with a local or remote image',
-    domain: 'x.com',
-    strategy: Strategy.UI, // Uses the UI directly to input and click post
-    browser: true,
-    args: [
-        { name: 'url', type: 'string', required: true, positional: true, help: 'The URL of the tweet to reply to' },
-        { name: 'text', type: 'string', required: true, positional: true, help: 'The text content of your reply' },
-        { name: 'image', help: 'Optional local image path to attach to the reply' },
-        { name: 'image-url', help: 'Optional remote image URL to download and attach to the reply' },
-    ],
-    columns: ['status', 'message', 'text', 'url'],
-    func: async (page, kwargs) => {
-        if (!page)
-            throw new CommandExecutionError('Browser session required for twitter reply');
-        if (kwargs.image && kwargs['image-url']) {
-            throw new CommandExecutionError('Use either --image or --image-url, not both.');
-        }
-        let localImagePath;
-        let cleanupDir;
-        try {
-            if (kwargs.image) {
-                localImagePath = resolveImagePath(kwargs.image);
-            } else if (kwargs['image-url']) {
-                const downloaded = await downloadRemoteImage(kwargs['image-url']);
-                localImagePath = downloaded.absPath;
-                cleanupDir = downloaded.cleanupDir;
-            }
-            // Dedicated composer is normally more reliable than the inline
-            // tweet page reply box, but X occasionally leaves that route on the
-            // Home timeline behind a loading dialog. openReplyComposer falls
-            // back to the target tweet's visible Reply action.
-            const composer = await openReplyComposer(page, kwargs.url);
-            if (!composer?.ok) {
-                return [{ status: 'failed', message: composer?.message ?? 'Could not open the reply composer.', text: kwargs.text }];
-            }
-            if (localImagePath) {
-                await page.wait({ selector: COMPOSER_FILE_INPUT_SELECTOR, timeout: 20 });
-                await attachComposerImage(page, localImagePath);
-            }
-            const result = await submitReply(page, kwargs.text);
-            return [{
-                    status: result.ok ? 'success' : 'failed',
-                    message: result.message,
-                    text: kwargs.text,
-                    ...(result.url ? { url: result.url } : {}),
-                    ...(kwargs.image ? { image: kwargs.image } : {}),
-                    ...(kwargs['image-url'] ? { 'image-url': kwargs['image-url'] } : {}),
-                }];
-        } finally {
-            if (cleanupDir) {
-                fs.rmSync(cleanupDir, { recursive: true, force: true });
-            }
-        }
-    }
-});
-export const __test__ = {
-    buildReplyComposerUrl,
-    isPromiseCollectedError,
-};
+        })()`)}catch(G){if(!X(G))throw G;await z.wait(2);const H=await M(z);if(H?.ok)return H;throw G}}async function U(z,q){let A=await D(z,q);if(!A?.ok&&/Could not verify/i.test(A?.message||"")){await z.wait(2);A=await D(z,q)}if(!A?.ok)return A;let G;try{G=await I(z)}catch(H){if(!X(H))throw H}if(G&&!G.ok)return G;return R(z,q)}v({site:"twitter",name:"reply",access:"write",description:"Reply to a specific tweet, optionally with a local or remote image",domain:"x.com",strategy:F.UI,browser:!0,args:[{name:"url",type:"string",required:!0,positional:!0,help:"The URL of the tweet to reply to"},{name:"text",type:"string",required:!1,positional:!0,help:"The text content of your reply (or use --file)"},{name:"file",help:"从本机文件读取回复正文（UTF-8），与位置参数 <text> 二选一"},{name:"image",help:"Optional local image path to attach to the reply"},{name:"image-url",help:"Optional remote image URL to download and attach to the reply"}],columns:["status","message","text","url"],func:async(z,q)=>{if(!z)throw new K("Browser session required for twitter reply");const A=q.file?String(q.file).trim():"";if(A){if(q.text)throw new K("回复正文位置参数和 --file 只能二选一，不能同时给");const J=f.resolve(A);if(!W.existsSync(J))throw new K(`--file 指向的文件不存在: ${J}`);q.text=W.readFileSync(J,"utf-8").trim();if(!q.text)throw new K(`--file 指向的文件内容为空: ${J}`)}else{if(!q.text)throw new K("缺少回复正文：给位置参数 <text> 或 --file 其一");B(q.text,{label:"回复内容"})}if(q.image&&q["image-url"])throw new K("Use either --image or --image-url, not both.");let G,H;try{if(q.image)G=O(q.image);else if(q["image-url"]){const Y=await h(q["image-url"]);G=Y.absPath;H=Y.cleanupDir}const J=await y(z,q.url);if(!J?.ok)return[{status:"failed",message:J?.message??"Could not open the reply composer.",text:q.text}];if(G){await z.wait({selector:b,timeout:20});await L(z,G)}const Q=await U(z,q.text);return[{status:Q.ok?"success":"failed",message:Q.message,text:q.text,...Q.url?{url:Q.url}:{},...q.image?{image:q.image}:{},...q["image-url"]?{"image-url":q["image-url"]}:{}}]}finally{if(H)W.rmSync(H,{recursive:!0,force:!0})}}});export const __test__={buildReplyComposerUrl:N,isPromiseCollectedError:X};