@ynhcj/xiaoyi-channel 0.0.94-beta → 0.0.95-beta

package/dist/src/provider.js

@@ -23,8 +23,29 @@ function isRetryableProviderError(message) {
         return true;
     return false;
 }
+/** Check if the request is triggered by a cron job by inspecting the first user message. */
+function isCronTriggered(messages) {
+    if (!messages)
+        return false;
+    const firstUser = messages.find(m => m.role === "user");
+    if (!firstUser)
+        return false;
+    let text = "";
+    if (typeof firstUser.content === "string") {
+        text = firstUser.content;
+    }
+    else if (Array.isArray(firstUser.content)) {
+        const block = firstUser.content.find(b => b.type === "text" && typeof b.text === "string");
+        if (block)
+            text = block.text;
+    }
+    return /^\[cron:/i.test(text.trim());
+}
 /** Compute retry delay in ms for the given 1-based attempt, with up to 10s jitter. */
-function getRetryDelayMs(attempt) {
+function getRetryDelayMs(attempt, isCron = false) {
+    if (isCron) {
+        return 60_000 + Math.floor(Math.random() * 10_000);
+    }
     const base = attempt <= RETRY_DELAYS_MS.length
         ? RETRY_DELAYS_MS[attempt - 1]
         : RETRY_DELAYS_MS[RETRY_DELAYS_MS.length - 1];
@@ -64,6 +85,128 @@ function buildReplayStream(result) {
         },
     };
 }
+/**
+ * Wrap the underlying stream with retry logic while preserving real-time streaming.
+ *
+ * Strategy:
+ *  1. Buffer events until the first content-bearing event is seen.
+ *  2. If the stream errors before any content, the buffer is tiny (start + error)
+ *     and we can safely retry with a fresh API call.
+ *  3. Once content events appear, flush the buffer and switch to pass-through mode
+ *     — the consumer sees every text_delta in real time.
+ */
+function createRetryingStream(createStream, cronJob) {
+    let resultResolve;
+    const resultPromise = new Promise(resolve => { resultResolve = resolve; });
+    const CONTENT_EVENT_TYPES = new Set([
+        "text_start", "text_delta", "text_end",
+        "thinking_start", "thinking_delta", "thinking_end",
+        "toolcall_start", "toolcall_delta", "toolcall_end",
+    ]);
+    async function* retryGenerator() {
+        for (let attempt = 0; attempt < MAX_RETRY_ATTEMPTS; attempt++) {
+            const stream = await createStream();
+            let hasContent = false;
+            const buffer = [];
+            let errorResult = null;
+            for await (const event of stream) {
+                const isContent = CONTENT_EVENT_TYPES.has(event.type);
+                if (!hasContent && !isContent) {
+                    // ── Buffer phase (no content yet) ──
+                    if (event.type === "done") {
+                        console.log(`[xiaoyiprovider] stream completed (no content), usage: input=${event.message?.usage?.input} output=${event.message?.usage?.output}`);
+                        for (const b of buffer)
+                            yield b;
+                        resultResolve(event.message);
+                        yield event;
+                        return;
+                    }
+                    if (event.type === "error") {
+                        errorResult = event.error;
+                    }
+                    buffer.push(event);
+                }
+                else {
+                    // ── Streaming phase ──
+                    if (!hasContent) {
+                        console.log("[xiaoyiprovider] first content event received, switching to streaming mode");
+                        hasContent = true;
+                        for (const b of buffer)
+                            yield b;
+                    }
+                    // IMPORTANT: resolve result() BEFORE yielding terminal events to avoid deadlock.
+                    // The SDK calls result() when it sees done/error — if we yield first, the generator
+                    // suspends and can never reach resolve, causing a permanent deadlock.
+                    if (event.type === "done") {
+                        console.log(`[xiaoyiprovider] stream completed, usage: input=${event.message?.usage?.input} output=${event.message?.usage?.output}`);
+                        resultResolve(event.message);
+                        yield event;
+                        return;
+                    }
+                    if (event.type === "error") {
+                        console.log(`[xiaoyiprovider] stream error after content: ${event.error?.errorMessage}`);
+                        resultResolve(event.error);
+                        yield event;
+                        return;
+                    }
+                    yield event;
+                }
+            }
+            // Stream ended during buffer phase — decide whether to retry
+            if (errorResult?.stopReason === "error" && isRetryableProviderError(errorResult.errorMessage)) {
+                if (attempt < MAX_RETRY_ATTEMPTS - 1) {
+                    const delayMs = getRetryDelayMs(attempt + 1, cronJob);
+                    console.log(`[xiaoyiprovider] retryable error (attempt ${attempt + 1}/${MAX_RETRY_ATTEMPTS}): ` +
+                        `${errorResult.errorMessage} — retrying in ${delayMs}ms`);
+                    await sleep(delayMs);
+                    continue; // discard buffer, retry with a new stream
+                }
+                console.log(`[xiaoyiprovider] all ${MAX_RETRY_ATTEMPTS} retries exhausted, surfacing last error`);
+            }
+            else if (errorResult) {
+                console.log(`[xiaoyiprovider] non-retryable error: ${errorResult.errorMessage}`);
+            }
+            // Non-retryable or retries exhausted — yield buffered events.
+            // Resolve before yielding the terminal event to avoid the same deadlock.
+            for (const b of buffer) {
+                if (b.type === "done") {
+                    resultResolve(b.message);
+                }
+                else if (b.type === "error") {
+                    resultResolve(b.error);
+                }
+                yield b;
+            }
+            if (errorResult && buffer.every(b => b.type !== "done" && b.type !== "error")) {
+                resultResolve(errorResult);
+            }
+            return;
+        }
+        // Safety: final fallback attempt
+        console.log("[xiaoyiprovider] entering final fallback attempt");
+        const lastStream = await createStream();
+        for await (const event of lastStream) {
+            if (event.type === "done") {
+                resultResolve(event.message);
+                yield event;
+                return;
+            }
+            if (event.type === "error") {
+                resultResolve(event.error);
+                yield event;
+                return;
+            }
+            yield event;
+        }
+    }
+    const gen = retryGenerator();
+    return {
+        result: () => resultPromise,
+        push: () => { },
+        end: () => { },
+        [Symbol.asyncIterator]: () => gen,
+    };
+}
 /**
  * Dynamic header keys injected via extraParams and forwarded to the HTTP request.
  * Correspond to the three fields written to .xiaoyiruntime:
@@ -195,48 +338,18 @@ export const xiaoyiProvider = {
                 const deviceSection = `\n\n## Current User Device Context\nThe current user is using the following device: ${displayDevice}\nYou need to be aware of the user's current device and provide guidance accordingly. If the response involves device-related tools or actions, you must tailor the reply based on the user's current device, using device-specific references such as "saved to the Notes/Calendar on your {deviceType}.\n"`;
                 context.systemPrompt = (context.systemPrompt ?? "") + deviceSection;
             }
-            // ── Retry loop ─────────────────────────────────────────
-            for (let attempt = 0; attempt < MAX_RETRY_ATTEMPTS; attempt++) {
-                const stream = await underlying(model, context, {
-                    ...options,
-                    headers: {
-                        ...options?.headers,
-                        ...dynamicHeaders,
-                    },
-                });
-                // Wait for the stream to settle (done or error) to inspect the result.
-                // stream.result() resolves to the final AssistantMessage (even on error).
-                const result = await stream.result();
-                // Check if this is a retryable error
-                if (result.stopReason === "error" && isRetryableProviderError(result.errorMessage)) {
-                    const delayMs = getRetryDelayMs(attempt);
-                    console.log(`[xiaoyiprovider] retryable error (attempt ${attempt + 1}/${MAX_RETRY_ATTEMPTS}): ` +
-                        `${result.errorMessage} — retrying in ${delayMs}ms`);
-                    await sleep(delayMs);
-                    continue;
-                }
-                // Success or non-retryable error — log and return
-                if (result.stopReason === "error") {
-                    console.log(`[xiaoyiprovider] non-retryable error: ${result.errorMessage}`);
-                }
-                else {
-                    console.log(`[xiaoyiprovider] stream completed, usage: input=${result.usage?.input} output=${result.usage?.output}`);
-                }
-                // The original stream has already been consumed by result().
-                // Build a replay stream that delivers the final result.
-                return buildReplayStream(result);
-            }
-            // All retries exhausted — return the last attempt's real error via a new stream
-            console.log(`[xiaoyiprovider] all ${MAX_RETRY_ATTEMPTS} retries exhausted, surfacing last error`);
-            const lastStream = await underlying(model, context, {
+            // ── Retry-capable streaming ──────────────────────────────
+            const cronJob = isCronTriggered(context.messages);
+            if (cronJob)
+                console.log("[xiaoyiprovider] detected cron-triggered request, using extended retry delays");
+            const makeStream = () => underlying(model, context, {
                 ...options,
                 headers: {
                     ...options?.headers,
                     ...dynamicHeaders,
                 },
             });
-            const lastResult = await lastStream.result();
-            return buildReplayStream(lastResult);
+            return createRetryingStream(makeStream, cronJob);
         };
     },
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ynhcj/xiaoyi-channel",
-  "version": "0.0.94-beta",
+  "version": "0.0.95-beta",
   "description": "OpenClaw Xiaoyi Channel plugin - Xiaoyi A2A protocol integration",
   "type": "module",
   "main": "./dist/index.js",