felo-ai 0.2.1 → 0.2.3

package/felo-slides/README.md

@@ -6,7 +6,7 @@ Generate presentation slides with the Felo PPT Task API (asynchronous workflow).
 
 - Generate a PPT deck from a single prompt
 - Poll task status automatically until completion/failure/timeout
-- Return `live_doc_url` immediately when the task is completed
+- Return `ppt_url` immediately when the task is completed (fallback to `live_doc_url`)
 - Return `task_id` for follow-up tracking
 
 ## Quick Start
@@ -54,12 +54,12 @@ Based on Felo v2 PPT Task API:
 2. Query status (optional): `GET /v2/tasks/{task_id}/status`
 3. Query historical/result: `GET /v2/tasks/{task_id}/historical`
 
-The skill polls every 10 seconds (max wait 600 seconds). It stops immediately on `COMPLETED`/`SUCCESS` and returns `live_doc_url`.
+The skill polls every 10 seconds (max wait 1800 seconds). It stops immediately on `COMPLETED`/`SUCCESS` and returns `ppt_url` (fallback `live_doc_url`).
 
 Internal script example:
 
 ```bash
-node felo-slides/scripts/run_ppt_task.mjs --query "Felo product intro, 3 slides" --interval 10 --max-wait 600
+node felo-slides/scripts/run_ppt_task.mjs --query "Felo product intro, 3 slides" --interval 10 --max-wait 1800
 ```
 
 ## Troubleshooting
@@ -76,7 +76,7 @@ The key is invalid or revoked. Generate a new key from [felo.ai](https://felo.ai
 
 The task may still be processing. Retry later with the same context, or run the script with `--verbose`.
 
-### Task completed but no `live_doc_url`
+### Task completed but no `ppt_url` / `live_doc_url`
 
 Use the returned `task_id` to query historical endpoint again.
 

package/felo-slides/SKILL.md

@@ -1,6 +1,6 @@
 ---
 name: felo-slides
-description: "Generate PPT/slides with Felo PPT Task API in Claude Code. Use when users ask to create/make/generate/export presentations or slide decks, or when explicit commands like /felo-slides are used. Handles API key check, task creation, polling, and final live_doc_url output."
+description: "Generate PPT/slides with Felo PPT Task API in Claude Code. Use when users ask to create/make/generate/export presentations or slide decks, or when explicit commands like /felo-slides are used. Handles API key check, task creation, polling, and final ppt_url output."
 ---
 
 # Felo Slides Skill
@@ -67,7 +67,7 @@ Use the bundled script (no `jq` dependency):
 node felo-slides/scripts/run_ppt_task.mjs \
   --query "USER_PROMPT_HERE" \
   --interval 10 \
-  --max-wait 600 \
+  --max-wait 1800 \
   --timeout 60
 ```
 
@@ -77,7 +77,7 @@ Script behavior:
 - Treats `COMPLETED`/`SUCCESS` as success terminal (case-insensitive)
 - Treats `FAILED`/`ERROR` as failure terminal
 - Stops polling immediately on terminal status
-- Prints `live_doc_url` on success
+- Prints `ppt_url` on success (fallback: `live_doc_url`)
 
 Optional debug output:
 
@@ -85,7 +85,7 @@ Optional debug output:
 node felo-slides/scripts/run_ppt_task.mjs \
   --query "USER_PROMPT_HERE" \
   --interval 10 \
-  --max-wait 600 \
+  --max-wait 1800 \
   --json \
   --verbose
 ```
@@ -93,6 +93,7 @@ node felo-slides/scripts/run_ppt_task.mjs \
 This outputs structured JSON including:
 - `task_id`
 - `task_status`
+- `ppt_url`
 - `live_doc_url`
 - `livedoc_short_id`
 - `ppt_business_id`
@@ -100,7 +101,7 @@ This outputs structured JSON including:
 ### Step 4: Return structured result
 
 On success, return:
-- `live_doc_url` immediately (script default output)
+- `ppt_url` immediately (script default output, fallback `live_doc_url`)
 - if `--json` is used, also include `task_id`, terminal status, and optional metadata
 
 ## Output Format
@@ -111,7 +112,8 @@ Use this response structure:
 ## PPT Generation Result
 - Task ID: <task_id>
 - Status: <status>
-- Live Doc URL: <live_doc_url>
+- PPT URL: <ppt_url>
+- Live Doc URL: <live_doc_url or N/A>
 
 ## Notes
 - livedoc_short_id: <value or N/A>

package/felo-slides/scripts/run_ppt_task.mjs

@@ -2,7 +2,7 @@
 
 const DEFAULT_API_BASE = 'https://openapi.felo.ai';
 const DEFAULT_INTERVAL_SEC = 10;
-const DEFAULT_MAX_WAIT_SEC = 600;
+const DEFAULT_MAX_WAIT_SEC = 1800;
 const DEFAULT_TIMEOUT_SEC = 60;
 
 function usage() {
@@ -14,7 +14,7 @@ function usage() {
       'Options:',
       '  --query <text>        PPT prompt (required)',
       '  --interval <seconds>  Poll interval, default 10',
-      '  --max-wait <seconds>  Max wait time, default 600',
+      '  --max-wait <seconds>  Max wait time, default 1800',
       '  --timeout <seconds>   Request timeout, default 60',
       '  --json                Print JSON output',
       '  --verbose             Print polling status to stderr',
@@ -114,12 +114,18 @@ async function fetchJson(url, init, timeoutMs) {
   }
 }
 
-function extractLiveDocUrl(historicalData, createData) {
-  const url = historicalData?.live_doc_url;
-  if (url) return url;
-  const sid = historicalData?.live_doc_short_id || historicalData?.livedoc_short_id || createData?.livedoc_short_id;
-  if (sid) return `https://felo.ai/livedoc/${sid}`;
-  return '';
+function extractTaskUrls(historicalData, createData) {
+  const pptUrl = historicalData?.ppt_url || '';
+  const liveDocUrl =
+    historicalData?.live_doc_url ||
+    (historicalData?.live_doc_short_id || historicalData?.livedoc_short_id || createData?.livedoc_short_id
+      ? `https://felo.ai/livedoc/${historicalData?.live_doc_short_id || historicalData?.livedoc_short_id || createData?.livedoc_short_id}`
+      : '');
+  return {
+    pptUrl,
+    liveDocUrl,
+    displayUrl: pptUrl || liveDocUrl,
+  };
 }
 
 async function createTask(apiKey, apiBase, query, timeoutMs) {
@@ -192,7 +198,7 @@ async function main() {
   while (Date.now() - startAt <= maxWaitMs) {
     const historicalData = await queryHistorical(apiKey, apiBase, taskId, timeoutMs);
     const taskStatus = normalizeStatus(historicalData.task_status || historicalData.status);
-    const liveDocUrl = extractLiveDocUrl(historicalData, createData);
+    const urls = extractTaskUrls(historicalData, createData);
     lastStatus = taskStatus || 'UNKNOWN';
 
     if (args.verbose) {
@@ -201,8 +207,8 @@ async function main() {
     }
 
     if (taskStatus === 'COMPLETED' || taskStatus === 'SUCCESS') {
-      if (!liveDocUrl) {
-        throw new Error('Task completed but live_doc_url is missing');
+      if (!urls.displayUrl) {
+        throw new Error('Task completed but no ppt_url/live_doc_url is available');
       }
       if (args.json) {
         console.log(
@@ -212,7 +218,8 @@ async function main() {
               data: {
                 task_id: taskId,
                 task_status: taskStatus,
-                live_doc_url: liveDocUrl,
+                ppt_url: urls.pptUrl || null,
+                live_doc_url: urls.liveDocUrl || null,
                 livedoc_short_id: createData.livedoc_short_id ?? historicalData.live_doc_short_id ?? historicalData.livedoc_short_id ?? null,
                 ppt_business_id: createData.ppt_business_id ?? null,
               },
@@ -222,7 +229,7 @@ async function main() {
           )
         );
       } else {
-        console.log(liveDocUrl);
+        console.log(urls.displayUrl);
       }
       return;
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "felo-ai",
-  "version": "0.2.1",
+  "version": "0.2.3",
   "description": "Felo AI CLI - real-time search from the terminal",
   "type": "module",
   "main": "src/cli.js",

package/src/cli.js

@@ -9,6 +9,31 @@ import * as config from './config.js';
 const require = createRequire(import.meta.url);
 const pkg = require('../package.json');
 
+/** Delay (ms) before process.exit to let Windows libuv finish handle cleanup. */
+const EXIT_DELAY_MS = 50;
+
+/**
+ * Flush stdout then stderr, then exit after a short delay. Avoids Node.js
+ * Windows UV_HANDLE_CLOSING assertion when process.exit() runs while streams
+ * or other handles are still closing.
+ * @param {number} code - Exit code.
+ */
+function flushStdioThenExit(code) {
+  const doExit = () => setTimeout(() => process.exit(code), EXIT_DELAY_MS);
+  const flushStderr = () => {
+    if (process.stderr?.writable && !process.stderr.destroyed) {
+      process.stderr.write('', () => doExit());
+    } else {
+      doExit();
+    }
+  };
+  if (process.stdout?.writable && !process.stdout.destroyed) {
+    process.stdout.write('', () => flushStderr());
+  } else {
+    flushStderr();
+  }
+}
+
 const program = new Command();
 
 program
@@ -31,8 +56,7 @@ program
       timeoutMs: Number.isNaN(timeoutMs) ? 60000 : timeoutMs,
     });
     process.exitCode = code;
-    // Defer exit so stdout/stderr can flush; avoids Node.js Windows UV_HANDLE_CLOSING assertion
-    setTimeout(() => process.exit(code), 0);
+    flushStdioThenExit(code);
   });
 
 program
@@ -53,8 +77,7 @@ program
       pollTimeoutMs: Number.isNaN(pollTimeoutMs) ? 1_200_000 : pollTimeoutMs,
     });
     process.exitCode = code;
-    // Defer exit so stderr can flush; reduces Node.js Windows assertion (UV_HANDLE_CLOSING)
-    setTimeout(() => process.exit(code), 0);
+    flushStdioThenExit(code);
   });
 
 const configCmd = program
@@ -68,9 +91,10 @@ configCmd
     try {
       await config.setConfig(key, value);
       console.log(`Set ${key}`);
+      flushStdioThenExit(0);
     } catch (e) {
       console.error('Error:', e.message);
-      process.exit(1);
+      flushStdioThenExit(1);
     }
   });
 
@@ -85,9 +109,10 @@ configCmd
       } else {
         console.log(config.maskValueForDisplay(key, value));
       }
+      flushStdioThenExit(0);
     } catch (e) {
       console.error('Error:', e.message);
-      process.exit(1);
+      flushStdioThenExit(1);
     }
   });
 
@@ -100,12 +125,13 @@ configCmd
       const keys = Object.keys(c);
       if (keys.length === 0) {
         console.log('No config set. Use: felo config set FELO_API_KEY <key>');
-        return;
+      } else {
+        keys.forEach((k) => console.log(k));
       }
-      keys.forEach((k) => console.log(k));
+      flushStdioThenExit(0);
     } catch (e) {
       console.error('Error:', e.message);
-      process.exit(1);
+      flushStdioThenExit(1);
     }
   });
 
@@ -116,9 +142,10 @@ configCmd
     try {
       await config.unsetConfig(key);
       console.log(`Unset ${key}`);
+      flushStdioThenExit(0);
     } catch (e) {
       console.error('Error:', e.message);
-      process.exit(1);
+      flushStdioThenExit(1);
     }
   });
 
@@ -127,6 +154,7 @@ configCmd
   .description('Show config file path')
   .action(() => {
     console.log(config.getConfigPath());
+    flushStdioThenExit(0);
   });
 
 program
@@ -135,7 +163,7 @@ program
   .argument('[input]', 'text or URL to summarize')
   .action(() => {
     console.error('summarize: not yet implemented. Use felo search for now.');
-    process.exit(1);
+    flushStdioThenExit(1);
   });
 
 program
@@ -144,7 +172,7 @@ program
   .argument('[text]', 'text to translate')
   .action(() => {
     console.error('translate: not yet implemented. Use felo search for now.');
-    process.exit(1);
+    flushStdioThenExit(1);
   });
 
 program.parse();

package/src/search.js

@@ -38,7 +38,6 @@ async function fetchWithTimeoutAndRetry(url, options, timeoutMs = DEFAULT_TIMEOU
         ...options,
         signal: controller.signal,
       });
-      clearTimeout(timeoutId);
       // Retry on 5xx (server errors)
       if (res.status >= 500 && attempt < MAX_RETRIES) {
         const delay = RETRY_BASE_MS * Math.pow(2, attempt);
@@ -47,7 +46,6 @@ async function fetchWithTimeoutAndRetry(url, options, timeoutMs = DEFAULT_TIMEOU
       }
       return res;
     } catch (err) {
-      clearTimeout(timeoutId);
       lastError = err;
       if (err.name === 'AbortError') {
         throw new Error(`Request timed out after ${timeoutMs / 1000}s`);
@@ -58,6 +56,8 @@ async function fetchWithTimeoutAndRetry(url, options, timeoutMs = DEFAULT_TIMEOU
         continue;
       }
       throw lastError;
+    } finally {
+      clearTimeout(timeoutId);
     }
   }
   throw lastError;

package/src/slides.js

@@ -80,7 +80,7 @@ async function createPptTask(apiKey, query, timeoutMs, apiBase) {
 }
 
 /**
- * Get task historical info. Returns { task_status, live_doc_url?, live_doc_short_id? } or throws.
+ * Get task historical info. Returns { task_status, ppt_url?, ppt_biz_id?, live_doc_url?, live_doc_short_id? } or throws.
  * Uses fetchWithTimeoutAndRetry for 5xx retry (per PPT Task API error codes).
  */
 async function getTaskHistorical(apiKey, taskId, timeoutMs, apiBase) {
@@ -116,6 +116,8 @@ async function getTaskHistorical(apiKey, taskId, timeoutMs, apiBase) {
 
   return {
     task_status: payload.task_status ?? payload.status,
+    ppt_url: payload.ppt_url,
+    ppt_biz_id: payload.ppt_biz_id,
     live_doc_url: payload.live_doc_url,
     live_doc_short_id: payload.live_doc_short_id ?? payload.livedoc_short_id,
   };
@@ -166,19 +168,25 @@ export async function slides(query, options = {}) {
       const normalizedStatus = normalizeTaskStatus(historical.task_status);
       lastStatus = normalizedStatus || historical.task_status;
 
+      const elapsed = Math.floor((Date.now() - startTime) / 1000);
       if (useLiveStatus) {
-        const elapsed = Math.floor((Date.now() - startTime) / 1000);
         spinIndex = (spinIndex + 1) % SPINNER_FRAMES.length;
         writeStatusLine(SPINNER_FRAMES[spinIndex], elapsed);
+      } else if (!options.verbose && !options.json) {
+        process.stderr.write(`  Generating... ${elapsed}s\n`);
       }
 
       const done = normalizedStatus === 'COMPLETED' || normalizedStatus === 'SUCCESS';
       if (done) {
         if (useLiveStatus) clearStatusLine();
-        const url =
+        const pptUrl =
+          historical.ppt_url ||
+          (historical.ppt_biz_id ? `https://dev.felo.ai/slides/${historical.ppt_biz_id}` : null);
+        const liveDocUrl =
           historical.live_doc_url ||
           (historical.live_doc_short_id ? `https://felo.ai/livedoc/${historical.live_doc_short_id}` : null) ||
           (createResult.livedoc_short_id ? `https://felo.ai/livedoc/${createResult.livedoc_short_id}` : null);
+        const url = pptUrl || liveDocUrl;
         if (options.json) {
           console.log(
             JSON.stringify(
@@ -187,8 +195,10 @@ export async function slides(query, options = {}) {
                 data: {
                   task_id: taskId,
                   task_status: normalizedStatus || historical.task_status,
-                  live_doc_url: url,
-                  livedoc_short_id: createResult.livedoc_short_id,
+                  ppt_url: pptUrl,
+                  ppt_biz_id: historical.ppt_biz_id ?? createResult.ppt_business_id,
+                  live_doc_url: liveDocUrl,
+                  livedoc_short_id: historical.live_doc_short_id ?? createResult.livedoc_short_id,
                   ppt_business_id: createResult.ppt_business_id,
                 },
               },
@@ -198,10 +208,13 @@ export async function slides(query, options = {}) {
           );
         } else {
           if (url) {
-            console.log(url);
+            if (pptUrl && !options.json) {
+              process.stderr.write('PPT ready. Open this link to preview:\n');
+            }
+            console.log(pptUrl || liveDocUrl);
           } else {
             if (useLiveStatus) clearStatusLine();
-            console.error('Error: Completed but no live_doc_url in response');
+            console.error('Error: Completed but no ppt_url or live_doc_url in response');
             return 1;
           }
         }