@jackwener/opencli 1.7.15 → 1.7.16

package/clis/chatgpt/utils.js

@@ -12,15 +12,23 @@ export const CHATGPT_URL = 'https://chatgpt.com';
 // Selectors
 const COMPOSER_SELECTORS = [
     '[aria-label="Chat with ChatGPT"]',
+    '[aria-label="与 ChatGPT 聊天"]',
     '[placeholder="Ask anything"]',
+    '[placeholder="有问题，尽管问"]',
     '#prompt-textarea',
     '[data-testid="prompt-textarea"]',
     '[contenteditable="true"][role="textbox"]',
 ];
+const SEND_BUTTON_SELECTOR = 'button[data-testid="send-button"]:not([disabled])';
 const SEND_BUTTON_LABELS = [
     'Send prompt',
     'Send message',
     'Send',
+    '发送提示',
+];
+const CLOSE_SIDEBAR_LABELS = [
+    'Close sidebar',
+    '关闭边栏',
 ];
 
 function isSameChatGPTConversation(currentUrl, expectedUrl) {
@@ -119,16 +127,34 @@ export async function isOnChatGPT(page) {
     }
 }
 
+// Comma-joined CSS selector list passed to page.wait({ selector }) so the
+// wait succeeds as soon as any composer flavour mounts (querySelectorAll
+// matches all of them). Tracks the most stable subset of COMPOSER_SELECTORS;
+// we only need to know "the composer is ready", not which variant rendered.
+const COMPOSER_WAIT_SELECTOR = '#prompt-textarea, [data-testid="prompt-textarea"]';
+const CONVERSATION_LINK_SELECTOR = 'a[href*="/c/"]';
+// Selector used by detail.js to wait for at least one rendered message bubble
+// after navigating to /c/<id>; mirrors the markup queried by getVisibleMessages.
+export const CONVERSATION_MESSAGE_SELECTOR = '[data-message-author-role], article[data-testid*="conversation-turn"]';
+
 export async function ensureOnChatGPT(page) {
     if (await isOnChatGPT(page)) return false;
     await page.goto(CHATGPT_URL, { settleMs: 2000 });
-    await page.wait(2);
+    try {
+        await page.wait({ selector: COMPOSER_WAIT_SELECTOR, timeout: 8 });
+    } catch {
+        // Composer didn't mount; downstream ensureChatGPTLogin / ensureChatGPTComposer surfaces a typed error.
+    }
     return true;
 }
 
 export async function startNewChat(page) {
     await page.goto(`${CHATGPT_URL}/new`, { settleMs: 2000 });
-    await page.wait(2);
+    try {
+        await page.wait({ selector: COMPOSER_WAIT_SELECTOR, timeout: 8 });
+    } catch {
+        // Composer didn't mount; downstream ensureChatGPTComposer surfaces a typed error.
+    }
 }
 
 export async function getPageState(page) {
@@ -185,15 +211,16 @@ export async function sendChatGPTMessage(page, text) {
     // Close sidebar if open (it can cover the chat composer)
     await page.evaluate(`
         (() => {
-            const closeBtn = Array.from(document.querySelectorAll('button')).find(b => b.getAttribute('aria-label') === 'Close sidebar');
+            const labels = ${JSON.stringify(CLOSE_SIDEBAR_LABELS)};
+            const closeBtn = Array.from(document.querySelectorAll('button')).find(b => labels.includes(b.getAttribute('aria-label') || ''));
             if (closeBtn) closeBtn.click();
         })()
     `);
-    await page.wait(0.5);
+    // The previous 0.5 s + 1.5 s pre-composer settles are dropped: the next
+    // page.evaluate roundtrip flushes the close-sidebar React update and
+    // findComposer() retries inside a single CDP call, so no fixed sleep is
+    // needed before reading the composer.
 
-    // Wait for composer to be ready and use Playwright's type()
-    await page.wait(1.5);
-    
     const typeResult = await page.evaluate(`
         (() => {
             ${buildComposerLocatorScript()}
@@ -234,9 +261,10 @@ export async function sendChatGPTMessage(page, text) {
     // Click send button
     const sent = await page.evaluate(`
         (() => {
+            const primary = document.querySelector(${JSON.stringify(SEND_BUTTON_SELECTOR)});
             const btns = Array.from(document.querySelectorAll('button'));
             const labels = ${JSON.stringify(SEND_BUTTON_LABELS)};
-            const sendBtn = btns.find(b => labels.includes(b.getAttribute('aria-label') || '') && !b.disabled);
+            const sendBtn = primary || btns.find(b => labels.includes(b.getAttribute('aria-label') || '') && !b.disabled);
             return { sendBtnFound: !!sendBtn };
         })()
     `);
@@ -247,8 +275,9 @@ export async function sendChatGPTMessage(page, text) {
     
     await page.evaluate(`
         (() => {
+            const primary = document.querySelector(${JSON.stringify(SEND_BUTTON_SELECTOR)});
             const labels = ${JSON.stringify(SEND_BUTTON_LABELS)};
-            const sendBtn = Array.from(document.querySelectorAll('button')).find(b => labels.includes(b.getAttribute('aria-label') || '') && !b.disabled);
+            const sendBtn = primary || Array.from(document.querySelectorAll('button')).find(b => labels.includes(b.getAttribute('aria-label') || '') && !b.disabled);
             if (sendBtn) sendBtn.click();
         })()
     `);
@@ -361,8 +390,9 @@ export async function waitForChatGPTResponse(page, baselineCount, prompt, timeou
 }
 
 export async function getConversationList(page) {
+    // ensureOnChatGPT already waits for the composer selector after navigation,
+    // so the previous standalone 2 s settle is redundant.
     await ensureOnChatGPT(page);
-    await page.wait(2);
 
     const openSidebar = await page.evaluate(`(() => {
         const button = Array.from(document.querySelectorAll('button'))
@@ -373,12 +403,22 @@ export async function getConversationList(page) {
         }
         return false;
     })()`);
-    if (openSidebar) await page.wait(1);
+    if (openSidebar) {
+        try {
+            await page.wait({ selector: CONVERSATION_LINK_SELECTOR, timeout: 3 });
+        } catch {
+            // Sidebar slide-in didn't surface conversation links; extractConversationLinks below tolerates empty and falls back to home goto.
+        }
+    }
 
     let items = await extractConversationLinks(page);
     if (!items.length) {
         await page.goto(CHATGPT_URL, { settleMs: 2000 });
-        await page.wait(2);
+        try {
+            await page.wait({ selector: CONVERSATION_LINK_SELECTOR, timeout: 8 });
+        } catch {
+            // No conversation links visible after fallback goto; extractConversationLinks returns empty.
+        }
         items = await extractConversationLinks(page);
     }
 
@@ -532,7 +572,9 @@ export async function waitForChatGPTImages(page, beforeUrls, timeoutSeconds, con
 
 export const __test__ = {
     COMPOSER_SELECTORS,
+    SEND_BUTTON_SELECTOR,
     SEND_BUTTON_LABELS,
+    CLOSE_SIDEBAR_LABELS,
     isSameChatGPTConversation,
     parseChatGPTConversationId,
 };

package/clis/chatgpt/utils.test.js

@@ -1,5 +1,5 @@
 import { describe, expect, it, vi } from 'vitest';
-import { __test__, waitForChatGPTImages } from './utils.js';
+import { __test__, sendChatGPTMessage, waitForChatGPTImages } from './utils.js';
 
 function createPageMock({ location = '', generating = [], imageUrls = [] } = {}) {
     let generatingIndex = 0;
@@ -74,3 +74,38 @@ describe('chatgpt conversation id parsing', () => {
         expect(() => __test__.parseChatGPTConversationId('https://chatgpt.com/')).toThrow(/conversation id/);
     });
 });
+
+describe('chatgpt send selectors', () => {
+    it('keeps locale-independent send-button selector before aria-label fallbacks', async () => {
+        const page = {
+            wait: vi.fn().mockResolvedValue(undefined),
+            nativeType: vi.fn().mockResolvedValue(undefined),
+            evaluate: vi.fn((script) => {
+                if (script.includes('findComposer')) return Promise.resolve(true);
+                if (script.includes('sendBtnFound')) {
+                    expect(script).toContain('data-testid=\\\"send-button\\\"');
+                    return Promise.resolve({ sendBtnFound: true });
+                }
+                if (script.includes('if (sendBtn) sendBtn.click')) {
+                    expect(script).toContain('data-testid=\\\"send-button\\\"');
+                }
+                return Promise.resolve(undefined);
+            }),
+        };
+
+        await expect(sendChatGPTMessage(page, 'hello')).resolves.toBe(true);
+    });
+
+    it('keeps zh-CN aria and placeholder fallbacks without replacing English selectors', () => {
+        expect(__test__.COMPOSER_SELECTORS).toEqual(expect.arrayContaining([
+            '[aria-label="Chat with ChatGPT"]',
+            '[aria-label="与 ChatGPT 聊天"]',
+            '[placeholder="Ask anything"]',
+            '[placeholder="有问题，尽管问"]',
+            '[data-testid="prompt-textarea"]',
+        ]));
+        expect(__test__.SEND_BUTTON_SELECTOR).toBe('button[data-testid="send-button"]:not([disabled])');
+        expect(__test__.SEND_BUTTON_LABELS).toEqual(expect.arrayContaining(['Send prompt', 'Send message', 'Send', '发送提示']));
+        expect(__test__.CLOSE_SIDEBAR_LABELS).toEqual(expect.arrayContaining(['Close sidebar', '关闭边栏']));
+    });
+});

package/clis/claude/ask.js

@@ -1,7 +1,8 @@
 import { cli, Strategy } from '@jackwener/opencli/registry';
 import { ArgumentError, CommandExecutionError, EmptyResultError } from '@jackwener/opencli/errors';
 import {
-    CLAUDE_DOMAIN, CLAUDE_URL, ensureOnClaude, selectModel, setAdaptiveThinking,
+    CLAUDE_DOMAIN, CLAUDE_URL, COMPOSER_SELECTOR, MESSAGE_SELECTOR,
+    ensureOnClaude, selectModel, setAdaptiveThinking,
     sendMessage, sendWithFile, getBubbleCount, waitForResponse, parseBoolFlag, withRetry,
     ensureClaudeComposer, requireNonEmptyPrompt, requirePositiveInt,
 } from './utils.js';
@@ -38,7 +39,11 @@ export const askCommand = cli({
 
         if (parseBoolFlag(kwargs.new)) {
             await page.goto(CLAUDE_URL);
-            await page.wait(3);
+            try {
+                await page.wait({ selector: COMPOSER_SELECTOR, timeout: 8 });
+            } catch {
+                // Composer didn't mount; ensureClaudeComposer below surfaces a typed error.
+            }
         } else {
             const navigated = await ensureOnClaude(page);
             if (navigated) {
@@ -48,11 +53,18 @@ export const askCommand = cli({
                     var link = document.querySelector('a[href*="/chat/"]');
                     if (link) link.click();
                 })()`);
-                await page.wait(2);
+                // Wait for the resumed conversation to render messages, or
+                // fall through if the link click had no effect (no recents).
+                try {
+                    await page.wait({ selector: MESSAGE_SELECTOR, timeout: 5 });
+                } catch {
+                    // No prior conversation; ensureClaudeComposer still requires composer below.
+                }
             }
         }
 
-        await page.wait(2);
+        // ensureClaudeComposer reads composer presence directly via getPageState,
+        // so the previous standalone 2 s settle is redundant.
         await withRetry(() => ensureClaudeComposer(page, 'Claude ask requires a visible composer on the current page.'));
 
         // Model selector is only available on the new-chat page, not inside
@@ -80,14 +92,16 @@ export const askCommand = cli({
                 }
                 throw new CommandExecutionError(`Could not switch to ${wantModel} model`);
             }
-            if (modelResult?.toggled) await page.wait(0.5);
+            // Post-toggle settle dropped — the next CDP eval (setAdaptiveThinking) gives
+            // React enough time to flush aria-checked updates between rountrips.
         }
 
         const thinkResult = await withRetry(() => setAdaptiveThinking(page, wantThink));
         if (!thinkResult?.ok && wantThink) {
             throw new CommandExecutionError('Could not enable Adaptive thinking');
         }
-        if (thinkResult?.toggled) await page.wait(0.5);
+        // Post-toggle settle dropped — the next CDP eval (sendMessage / sendWithFile)
+        // gives React enough time to flush aria-checked updates.
 
         if (kwargs.file) {
             const baseline = await withRetry(() => getBubbleCount(page));
@@ -100,7 +114,8 @@ export const askCommand = cli({
                 // SPA navigates after send; "Promise was collected" means send succeeded
                 if (!String(err?.message || err).includes('Promise was collected')) throw err;
             }
-            await page.wait(3);
+            // Pre-waitForResponse settle dropped — waitForResponse's first 3 s polling
+            // tick covers the same window without an unconditional sleep.
             const result = await waitForResponse(page, baseline, prompt, timeoutMs);
             if (!result) {
                 throw new EmptyResultError(

package/clis/claude/detail.js

@@ -1,6 +1,6 @@
 import { cli, Strategy } from '@jackwener/opencli/registry';
 import { EmptyResultError } from '@jackwener/opencli/errors';
-import { CLAUDE_DOMAIN, getVisibleMessages, ensureClaudeLogin, requireConversationId } from './utils.js';
+import { CLAUDE_DOMAIN, MESSAGE_SELECTOR, getVisibleMessages, ensureClaudeLogin, requireConversationId } from './utils.js';
 
 export const detailCommand = cli({
     site: 'claude',
@@ -21,7 +21,14 @@ export const detailCommand = cli({
         const id = requireConversationId(kwargs.id);
 
         await page.goto(`https://claude.ai/chat/${id}`);
-        await page.wait(4);
+        // Wait for the first assistant message bubble to render instead of a
+        // fixed 4 s sleep. Swallow the timeout so empty conversations and
+        // login redirects fall through to ensureClaudeLogin / EmptyResultError.
+        try {
+            await page.wait({ selector: MESSAGE_SELECTOR, timeout: 10 });
+        } catch {
+            // Empty conversation, missing access, or login redirect — handled below.
+        }
         await ensureClaudeLogin(page, 'Claude detail requires a logged-in Claude session.');
 
         const messages = await getVisibleMessages(page);

package/clis/claude/new.js

@@ -1,5 +1,5 @@
 import { cli, Strategy } from '@jackwener/opencli/registry';
-import { CLAUDE_DOMAIN, CLAUDE_URL, ensureClaudeComposer } from './utils.js';
+import { CLAUDE_DOMAIN, CLAUDE_URL, COMPOSER_SELECTOR, ensureClaudeComposer } from './utils.js';
 
 export const newCommand = cli({
     site: 'claude',
@@ -16,7 +16,13 @@ export const newCommand = cli({
 
     func: async (page) => {
         await page.goto(CLAUDE_URL);
-        await page.wait(2);
+        // Wait for the composer to mount instead of a fixed 2 s sleep. If it
+        // never mounts, swallow and let ensureClaudeComposer surface a typed error.
+        try {
+            await page.wait({ selector: COMPOSER_SELECTOR, timeout: 8 });
+        } catch {
+            // Login or error page — ensureClaudeComposer below throws AuthRequiredError / CommandExecutionError.
+        }
         await ensureClaudeComposer(page, 'Claude new requires a logged-in Claude session with a visible composer.');
         return [{ Status: 'New chat started' }];
     },

package/clis/claude/read.js

@@ -16,8 +16,9 @@ export const readCommand = cli({
     columns: ['Index', 'Role', 'Text'],
 
     func: async (page) => {
+        // ensureOnClaude now waits for the composer selector; the previous post-nav
+        // 3 s settle is covered by that event-based wait.
         await ensureOnClaude(page);
-        await page.wait(3);
         await ensureClaudeLogin(page, 'Claude read requires a logged-in Claude session.');
         const messages = await getVisibleMessages(page);
         if (messages.length > 0) return messages;

package/clis/claude/send.js

@@ -1,6 +1,6 @@
 import { cli, Strategy } from '@jackwener/opencli/registry';
 import { CommandExecutionError } from '@jackwener/opencli/errors';
-import { CLAUDE_DOMAIN, CLAUDE_URL, ensureOnClaude, sendMessage, parseBoolFlag, withRetry, ensureClaudeComposer, requireNonEmptyPrompt } from './utils.js';
+import { CLAUDE_DOMAIN, CLAUDE_URL, COMPOSER_SELECTOR, ensureOnClaude, sendMessage, parseBoolFlag, withRetry, ensureClaudeComposer, requireNonEmptyPrompt } from './utils.js';
 
 export const sendCommand = cli({
     site: 'claude',
@@ -23,10 +23,15 @@ export const sendCommand = cli({
 
         if (parseBoolFlag(kwargs.new)) {
             await page.goto(CLAUDE_URL);
-            await page.wait(3);
+            try {
+                await page.wait({ selector: COMPOSER_SELECTOR, timeout: 8 });
+            } catch {
+                // Composer didn't mount; ensureClaudeComposer below surfaces a typed error.
+            }
         } else {
+            // ensureOnClaude now waits for the composer selector; the previous
+            // post-nav 2 s settle is covered by that event-based wait.
             await ensureOnClaude(page);
-            await page.wait(2);
         }
         await withRetry(() => ensureClaudeComposer(page, 'Claude send requires a visible composer on the current page.'));
 

package/clis/claude/utils.js

@@ -26,7 +26,14 @@ export async function isOnClaude(page) {
 export async function ensureOnClaude(page) {
     if (await isOnClaude(page)) return false;
     await page.goto(CLAUDE_URL);
-    await page.wait(3);
+    // Wait for the composer textarea instead of a fixed 3 s sleep. On the login
+    // page it never mounts; swallow the timeout so callers (read / detail /
+    // send) can still inspect page state and produce typed errors.
+    try {
+        await page.wait({ selector: COMPOSER_SELECTOR, timeout: 8 });
+    } catch {
+        // Login or error page — downstream ensureClaudeLogin / ensureClaudeComposer surfaces a typed error.
+    }
     return true;
 }
 
@@ -111,7 +118,13 @@ export async function getVisibleMessages(page) {
 export async function getConversationList(page) {
     if (!(await isOnClaude(page)) || !(await page.evaluate('window.location.href') || '').includes('/recents')) {
         await page.goto('https://claude.ai/recents');
-        await page.wait(3);
+        // Recents list mounts <a href="/chat/...">; an empty history is also
+        // valid (returns []), so swallow the timeout instead of raising.
+        try {
+            await page.wait({ selector: 'a[href*="/chat/"]', timeout: 8 });
+        } catch {
+            // Empty history or login page — downstream evaluate returns [].
+        }
     }
     const items = await page.evaluate(`(() => {
         var links = Array.from(document.querySelectorAll('a[href*="/chat/"]'));
@@ -147,7 +160,12 @@ export async function selectModel(page, modelName) {
     if (!opened?.ok) return opened;
     if (!opened.opened) return opened;
 
-    await page.wait(0.6);
+    // Wait for the dropdown menu items to mount instead of a fixed 0.6 s sleep.
+    try {
+        await page.wait({ selector: 'div[role="menuitemradio"]', timeout: 3 });
+    } catch {
+        // Dropdown didn't open — next evaluate finds no target and returns { ok: false }.
+    }
 
     return page.evaluate(`(() => {
         var items = Array.from(document.querySelectorAll('div[role="menuitemradio"]'));
@@ -175,7 +193,12 @@ export async function setAdaptiveThinking(page, enabled) {
     })()`);
     if (!opened?.ok) return { ok: false };
 
-    await page.wait(0.6);
+    // Wait for the dropdown menu items to mount instead of a fixed 0.6 s sleep.
+    try {
+        await page.wait({ selector: 'div[role="menuitem"]', timeout: 3 });
+    } catch {
+        // Dropdown didn't open — next evaluate finds no target and returns { ok: false }.
+    }
 
     return page.evaluate(`(() => {
         var items = Array.from(document.querySelectorAll('div[role="menuitem"]'));

package/clis/deepseek/ask.js

@@ -3,7 +3,7 @@ import { CliError, CommandExecutionError, EXIT_CODES } from '@jackwener/opencli/
 import {
     DEEPSEEK_DOMAIN, DEEPSEEK_URL, ensureOnDeepSeek, selectModel, setFeature,
     sendMessage, sendWithFile, getBubbleCount, waitForResponse, parseBoolFlag, withRetry,
-    pickResumeUrl,
+    pickResumeUrl, TEXTAREA_SELECTOR,
 } from './utils.js';
 
 export const askCommand = cli({
@@ -35,7 +35,13 @@ export const askCommand = cli({
 
         if (parseBoolFlag(kwargs.new)) {
             await page.goto(DEEPSEEK_URL);
-            await page.wait(3);
+            // Wait for the composer to mount instead of a fixed 3 s sleep.
+            try {
+                await page.wait({ selector: TEXTAREA_SELECTOR, timeout: 8 });
+            } catch {
+                // Selector still missing → downstream selectModel/sendMessage
+                // will surface the failure with a typed error.
+            }
         } else {
             const navigated = await ensureOnDeepSeek(page);
             if (navigated) {
@@ -49,12 +55,15 @@ export const askCommand = cli({
                     );
                 }
                 await page.goto(resumeUrl);
-                await page.wait(2);
+                try {
+                    await page.wait({ selector: TEXTAREA_SELECTOR, timeout: 5 });
+                } catch {
+                    // Conversation page may still be loading; subsequent steps
+                    // will retry or report.
+                }
             }
         }
 
-        await page.wait(2);
-
         // Model selector is only available on the new-chat page, not inside
         // an existing conversation. Skip it when we resumed a prior thread.
         const currentUrl = await page.evaluate('window.location.href') || '';
@@ -76,7 +85,9 @@ export const askCommand = cli({
             if (!modelResult?.ok) {
                 throw new CommandExecutionError(`Could not switch to ${wantModel} model`);
             }
-            if (modelResult?.toggled) await page.wait(0.5);
+            // The 0.5 s settle previously here was redundant: each subsequent
+            // step (setFeature, sendMessage) issues a fresh CDP eval, giving
+            // React more than enough time to flush the toggle's state update.
         }
 
         const thinkResult = await withRetry(() => setFeature(page, 'DeepThink', wantThink));
@@ -102,7 +113,8 @@ export const askCommand = cli({
             }
         }
 
-        if (thinkResult?.toggled || searchResult?.toggled) await page.wait(0.5);
+        // No settle wait after toggles: the next CDP eval below already gives
+        // React time to flush the aria-checked state.
 
         if (kwargs.file) {
             const baseline = await withRetry(() => getBubbleCount(page));
@@ -115,7 +127,8 @@ export const askCommand = cli({
                 // SPA navigates after send; "Promise was collected" means send succeeded
                 if (!String(err?.message || err).includes('Promise was collected')) throw err;
             }
-            await page.wait(3);
+            // waitForResponse polls every 3 s for new bubbles, so the previous
+            // 3 s settle here was a redundant sleep on top of the first poll.
             const result = await waitForResponse(page, baseline, prompt, timeoutMs, wantThink);
             if (!result) {
                 return [{ response: `[NO RESPONSE] No reply within ${kwargs.timeout}s.` }];

package/clis/deepseek/detail.js

@@ -2,6 +2,7 @@ import { cli, Strategy } from '@jackwener/opencli/registry';
 import { EmptyResultError } from '@jackwener/opencli/errors';
 import {
     DEEPSEEK_DOMAIN,
+    MESSAGE_SELECTOR,
     ensureOnDeepSeek,
     getVisibleMessages,
     parseDeepSeekConversationId,
@@ -25,7 +26,14 @@ export const detailCommand = cli({
         const id = parseDeepSeekConversationId(kwargs.id);
         await ensureOnDeepSeek(page);
         await page.goto(`https://chat.deepseek.com/a/chat/s/${id}`);
-        await page.wait(5);
+        // Wait for at least one rendered bubble instead of a fixed 5 s sleep.
+        // Empty / invalid conversations fall through to the EmptyResultError
+        // below.
+        try {
+            await page.wait({ selector: MESSAGE_SELECTOR, timeout: 10 });
+        } catch {
+            // No bubble mounted within 10 s; treated as empty by the check below.
+        }
         const messages = await getVisibleMessages(page);
         if (messages.length === 0) {
             throw new EmptyResultError(

package/clis/deepseek/new.js

@@ -1,5 +1,6 @@
 import { cli, Strategy } from '@jackwener/opencli/registry';
-import { DEEPSEEK_DOMAIN, DEEPSEEK_URL } from './utils.js';
+import { CommandExecutionError } from '@jackwener/opencli/errors';
+import { DEEPSEEK_DOMAIN, DEEPSEEK_URL, TEXTAREA_SELECTOR } from './utils.js';
 
 export const newCommand = cli({
     site: 'deepseek',
@@ -16,7 +17,17 @@ export const newCommand = cli({
 
     func: async (page) => {
         await page.goto(DEEPSEEK_URL);
-        await page.wait(2);
+        // Confirm the composer mounted before reporting success. The previous
+        // 2 s blind sleep would return "New chat started" even when the page
+        // was still loading or the user was logged out.
+        try {
+            await page.wait({ selector: TEXTAREA_SELECTOR, timeout: 8 });
+        } catch {
+            throw new CommandExecutionError(
+                'DeepSeek composer did not mount within 8 s',
+                'Verify you are logged into chat.deepseek.com.',
+            );
+        }
         return [{ Status: 'New chat started' }];
     },
 });

package/clis/deepseek/read.js

@@ -15,8 +15,9 @@ export const readCommand = cli({
     columns: ['Role', 'Text'],
 
     func: async (page) => {
+        // ensureOnDeepSeek already waits for the composer to mount; the
+        // follow-up 5 s sleep was redundant.
         await ensureOnDeepSeek(page);
-        await page.wait(5);
         const messages = await getVisibleMessages(page);
         if (messages.length > 0) return messages;
         return [{ Role: 'system', Text: 'No visible messages found.' }];

package/clis/deepseek/utils.js

@@ -43,7 +43,14 @@ export async function isOnDeepSeek(page) {
 export async function ensureOnDeepSeek(page) {
     if (await isOnDeepSeek(page)) return false;
     await page.goto(DEEPSEEK_URL);
-    await page.wait(3);
+    // Wait for the composer textarea instead of a fixed 3 s sleep. On the login
+    // page it never mounts; swallow the timeout so callers (status / read /
+    // history) can still inspect page state.
+    try {
+        await page.wait({ selector: TEXTAREA_SELECTOR, timeout: 8 });
+    } catch {
+        // Login or error page — downstream will see hasTextarea=false / empty results.
+    }
     return true;
 }
 

package/clis/linkedin/search.js

@@ -245,23 +245,20 @@ async function fetchJobCards(page, input) {
     const MAX_BATCH = 25;
     const allJobs = [];
     let offset = input.start;
+    // Read JSESSIONID directly from the cookie store via CDP — zero page.evaluate round-trip
+    const cookies = await page.getCookies({ url: 'https://www.linkedin.com' });
+    const jsession = cookies.find((c) => c.name === 'JSESSIONID')?.value;
+    if (!jsession) {
+        throw new AuthRequiredError(LINKEDIN_DOMAIN, 'LinkedIn JSESSIONID cookie not found. Please sign in to LinkedIn in the browser.');
+    }
+    const csrf = jsession.replace(/^"|"$/g, '');
     while (allJobs.length < input.limit) {
         const count = Math.min(MAX_BATCH, input.limit - allJobs.length);
         const apiPath = buildVoyagerUrl(input, offset, count);
         const batch = await page.evaluate(`(async () => {
-      const jsession = document.cookie.split(';').map(p => p.trim())
-        .find(p => p.startsWith('JSESSIONID='))?.slice('JSESSIONID='.length);
-      if (!jsession) {
-        return {
-          authRequired: true,
-          error: 'LinkedIn JSESSIONID cookie not found. Please sign in to LinkedIn in the browser.'
-        };
-      }
-
-      const csrf = jsession.replace(/^"|"$/g, '');
       const res = await fetch(${JSON.stringify(apiPath)}, {
         credentials: 'include',
-        headers: { 'csrf-token': csrf, 'x-restli-protocol-version': '2.0.0' },
+        headers: { 'csrf-token': ${JSON.stringify(csrf)}, 'x-restli-protocol-version': '2.0.0' },
       });
       if (res.status === 401 || res.status === 403) {
         const text = await res.text();

package/clis/maimai/search-talents.js

@@ -80,18 +80,22 @@ cli({
       },
     };
 
+    // Read csrftoken directly from the cookie store via CDP — zero page.evaluate round-trip
+    const cookies = await page.getCookies({ url: 'https://maimai.cn' });
+    const csrftokenFromCookie = cookies.find((c) => c.name === 'csrftoken')?.value || '';
+
     // Execute the search API call in browser context
-    const data = await page.evaluate(async (body) => {
-      // Get CSRF token from cookie or meta tag
-      let csrftoken = document.cookie.split('; ')
-        .find(row => row.startsWith('csrftoken='))
-        ?.split('=')[1] || '';
+    const data = await page.evaluate(`async () => {
+      // Prefer cookie-derived csrftoken (hoisted from CDP); fall back to meta tag
+      let csrftoken = ${JSON.stringify(csrftokenFromCookie)};
 
       if (!csrftoken) {
         const meta = document.querySelector('meta[name="csrf-token"]');
         if (meta) csrftoken = meta.getAttribute('content') || '';
       }
 
+      const body = ${JSON.stringify(requestBody)};
+
       const res = await fetch('https://maimai.cn/api/ent/discover/search?channel=www&data_version=3.0&version=1.0.0', {
         method: 'POST',
         headers: {
@@ -117,7 +121,7 @@ cli({
       }
 
       return result;
-    }, requestBody);
+    }`);
 
     // Extract talent list from response
     const talentList = data.data?.list || data.data?.talent_list || data.list || data.talent_list || [];