mirror of
https://github.com/jackwener/OpenCLI.git
synced 2026-09-14 18:25:42 +08:00
12d88e4b23
* refactor(runtime): unify command timeout into a single --timeout arg Drop the cli-level `timeoutSeconds` build-time ceiling field. A command now opts into runtime-enforced timeouts purely by declaring an arg named `timeout`; the user-facing `--timeout` value (its default or override) is the single authoritative knob, used both by the adapter polling loop and by the runtime ceiling (with a 30s padding for return + closeWindow + trace export). Behavior: - Browser commands without a `--timeout` arg fall back to OPENCLI_BROWSER_COMMAND_TIMEOUT (default 60s, unchanged). - Non-browser commands without a `--timeout` arg now run unbounded rather than against the previously implicit `timeoutSeconds` cap. Affected commands keep their old caps via newly added `--timeout` args. - LLM adapters (gemini/claude/deepseek/doubao/qwen/yuanbao ask) keep their current `--timeout` defaults; the runtime ceiling is now strictly more generous (userTimeout + 30s vs. the previous 180s cap), so `--timeout 600` actually buys 600s of polling rather than dying at 180s. Closes the design discussion that started from PR #1227, which proposed a per-site `OPENCLI_GEMINI_ASK_TIMEOUT` env var to work around the same underlying mismatch. * fix(timeout): wire --timeout arg into chatgpt/gemini image adapter polling codex-coder review on PR #1364 caught that the new --timeout arg I added to chatgpt/image and gemini/image only drove the runtime ceiling — the adapter still hardcoded `const timeout = 120`, so users passing --timeout 240/600 saw runtime allow 270s/630s but the adapter stop polling at 120s. That recreated the same single-knob mismatch this PR was meant to delete. Also add the browser-path runWithTimeout assertion codex-coder flagged as missing: a browser command with --timeout default=5 must call runWithTimeout with timeout: 35; a browser command without --timeout arg must fall back to DEFAULT_BROWSER_COMMAND_TIMEOUT. Image adapters now read kwargs.timeout and reject non-positive-integer values with ArgumentError (no silent fallback). chatgpt/image.test.js updated to pass an explicit timeout when calling .func directly (the test bypasses arg coercion). * fix(runtime): reject invalid timeout ceilings * fix(timeout): normalize timeout args to integer values * fix(timeout): preserve remaining command ceilings * fix(runtime): validate timeout before browser setup
79 lines
3.4 KiB
JavaScript
79 lines
3.4 KiB
JavaScript
import { cli, Strategy } from '@jackwener/opencli/registry';
|
|
import { ArgumentError, selectorError } from '@jackwener/opencli/errors';
|
|
import { conversationSelectionArgs, openCodexConversation } from './sidebar.js';
|
|
export const askCommand = cli({
|
|
site: 'codex',
|
|
name: 'ask',
|
|
access: 'write',
|
|
description: 'Send a prompt to the current or selected Codex conversation and wait for the AI response',
|
|
domain: 'localhost',
|
|
strategy: Strategy.UI,
|
|
browser: true,
|
|
args: [
|
|
{ name: 'text', required: true, positional: true, help: 'Prompt to send' },
|
|
{ name: 'timeout', type: 'int', required: false, help: 'Max seconds to wait for response (default: 60)', default: 60 },
|
|
...conversationSelectionArgs,
|
|
],
|
|
columns: ['Role', 'Project', 'Conversation', 'Text'],
|
|
func: async (page, kwargs) => {
|
|
const text = kwargs.text;
|
|
const timeout = kwargs.timeout;
|
|
if (!Number.isInteger(timeout) || timeout < 1) {
|
|
throw new ArgumentError('--timeout must be a positive integer (seconds)');
|
|
}
|
|
const selected = await openCodexConversation(page, kwargs);
|
|
// Snapshot the current content length before sending
|
|
const beforeLen = await page.evaluate(`
|
|
(function() {
|
|
const turns = document.querySelectorAll('[data-content-search-turn-key]');
|
|
return turns.length;
|
|
})()
|
|
`);
|
|
// Inject and send
|
|
const injected = await page.evaluate(`
|
|
(function(text) {
|
|
const editables = Array.from(document.querySelectorAll('[contenteditable="true"]'));
|
|
const composer = editables.length > 0 ? editables[editables.length - 1] : document.querySelector('textarea');
|
|
if (!composer) return false;
|
|
composer.focus();
|
|
document.execCommand('insertText', false, text);
|
|
return true;
|
|
})(${JSON.stringify(text)})
|
|
`);
|
|
if (!injected)
|
|
throw selectorError('Codex input element');
|
|
await page.wait(0.5);
|
|
await page.pressKey('Enter');
|
|
// Poll for new content
|
|
const pollInterval = 3;
|
|
const maxPolls = Math.ceil(timeout / pollInterval);
|
|
let response = '';
|
|
for (let i = 0; i < maxPolls; i++) {
|
|
await page.wait(pollInterval);
|
|
const result = await page.evaluate(`
|
|
(function(prevLen) {
|
|
const turns = document.querySelectorAll('[data-content-search-turn-key]');
|
|
if (turns.length <= prevLen) return null;
|
|
const lastTurn = turns[turns.length - 1];
|
|
const text = lastTurn.innerText || lastTurn.textContent;
|
|
return text ? text.trim() : null;
|
|
})(${beforeLen})
|
|
`);
|
|
if (result) {
|
|
response = result;
|
|
break;
|
|
}
|
|
}
|
|
if (!response) {
|
|
return [
|
|
{ Role: 'User', Project: selected?.project || '', Conversation: selected?.conversation || '', Text: text },
|
|
{ Role: 'System', Project: selected?.project || '', Conversation: selected?.conversation || '', Text: `No response within ${timeout}s. The agent may still be working.` },
|
|
];
|
|
}
|
|
return [
|
|
{ Role: 'User', Project: selected?.project || '', Conversation: selected?.conversation || '', Text: text },
|
|
{ Role: 'Assistant', Project: selected?.project || '', Conversation: selected?.conversation || '', Text: response },
|
|
];
|
|
},
|
|
});
|