@mkterswingman/5mghost-wonder 0.0.19 → 0.0.20

package/dist/commands/read.js

@@ -3,8 +3,9 @@
 // Phase 1: URL routing, export, metadata output.
 // Phase 2: --tab cell/merge/image parsing.
 import { parseWecomUrl } from "../wecom/url.js";
-import { exportWecomDoc, ExportError } from "../wecom/export.js";
-import { loadCookies, getCookieStatus } from "../wecom/cookies.js";
+import { exportWecomDoc, ExportError, isExportAuthError } from "../wecom/export.js";
+import { loadCookies, getCookieStatus, saveCookies, validateCookies } from "../wecom/cookies.js";
+import { collectWecomCookiesViaBrowser } from "../wecom/browser.js";
 import { resolveWonderPaths } from "../platform/paths.js";
 import { lookupCachedExport, saveExportToCache } from "../wecom/cache.js";
 import { parseTab } from "../xlsx/parse-tab.js";
@@ -32,6 +33,7 @@ export async function runReadCommand(args, context) {
         writeError({ error: "missing_url", message: "用法：wonder read <url> [--save <dir>]" });
         return { exitCode: 1, telemetry: { outcome: "failure", errorKind: "missing_url" } };
     }
+    const sourceUrl = url;
     // --- Parse URL ---
     const parsed = parseWecomUrl(url);
     if (!parsed.ok) {
@@ -72,14 +74,12 @@ export async function runReadCommand(args, context) {
         return { exitCode: 1, telemetry: { outcome: "failure", errorKind: "cookie_invalid" } };
     }
     // Load cookies and convert Record<string,string> → CookieEntry[]
-    const rawCookies = loadCookies(paths.cookiesPath);
-    const cookies = rawCookies
-        ? Object.entries(rawCookies).map(([name, value]) => ({ name, value }))
-        : [];
+    let rawCookies = loadCookies(paths.cookiesPath);
+    let cookies = toCookieEntries(rawCookies);
     // --- Resolve save dir ---
     const resolvedSaveDir = saveDir ?? paths.defaultSaveDir;
     // --- Export (with local cache) ---
-    const tokValue = rawCookies?.["TOK"] ?? "";
+    let tokValue = rawCookies?.["TOK"] ?? "";
     const cacheDir = paths.cacheDir;
     const noCache = args.includes("--no-cache");
     let result = null;
@@ -113,17 +113,7 @@ export async function runReadCommand(args, context) {
     }
     if (!result) {
         try {
-            const fresh = await exportWecomDoc({
-                docId,
-                docType,
-                sourceUrl: url,
-                cookies,
-                saveDir: resolvedSaveDir,
-                onProgress: (pct) => {
-                    // Progress → stderr only, so stdout stays valid JSON for pipes.
-                    context.io.stderr(`Exporting… ${Math.max(1, Math.min(99, pct))}%`);
-                },
-            });
+            const fresh = await exportWithCurrentCookies();
             if (!noCache && tokValue) {
                 try {
                     saveExportToCache({
@@ -149,19 +139,57 @@ export async function runReadCommand(args, context) {
         }
         catch (err) {
             if (err instanceof ExportError) {
+                if (isExportAuthError(err)) {
+                    const retry = await refreshCookiesAndRetryExport();
+                    if (retry.ok) {
+                        const fresh = retry.fresh;
+                        if (!noCache && tokValue) {
+                            try {
+                                saveExportToCache({
+                                    cacheDir,
+                                    docId,
+                                    tokValue,
+                                    sourceFilePath: fresh.filePath,
+                                    fileName: fresh.fileName,
+                                    fileSizeBytes: fresh.fileSizeBytes,
+                                    docType,
+                                });
+                            }
+                            catch {
+                                // Cache save failure must not break the command.
+                            }
+                        }
+                        result = {
+                            filePath: fresh.filePath,
+                            fileName: fresh.fileName,
+                            fileSizeBytes: fresh.fileSizeBytes,
+                            docType,
+                        };
+                    }
+                    else {
+                        return retry.result;
+                    }
+                }
+                else {
+                    writeError({
+                        error: "export_failed",
+                        errorKind: err.kind,
+                        message: err.message,
+                    });
+                    return {
+                        exitCode: 1,
+                        telemetry: { outcome: "failure", errorKind: err.kind, errorMessage: err.message },
+                    };
+                }
+            }
+            else {
+                const msg = err instanceof Error ? err.message : String(err);
                 writeError({
                     error: "export_failed",
-                    errorKind: err.kind,
-                    message: err.message,
+                    message: msg,
                 });
-                return {
-                    exitCode: 1,
-                    telemetry: { outcome: "failure", errorKind: err.kind, errorMessage: err.message },
-                };
+                return { exitCode: 1, telemetry: { outcome: "failure", errorKind: "unknown", errorMessage: msg } };
             }
-            const msg = err instanceof Error ? err.message : String(err);
-            writeError({ error: "export_failed", message: msg });
-            return { exitCode: 1, telemetry: { outcome: "failure", errorKind: "unknown", errorMessage: msg } };
         }
     }
     // --- Build output ---
@@ -199,4 +227,91 @@ export async function runReadCommand(args, context) {
     const outputType = docType === "doc" ? "doc" : "slide";
     context.io.stdout(JSON.stringify({ type: outputType, path: result.filePath }));
     return { exitCode: 0, telemetry: { outcome: "success" } };
+    function exportWithCurrentCookies() {
+        return exportWecomDoc({
+            docId,
+            docType,
+            sourceUrl,
+            cookies,
+            saveDir: resolvedSaveDir,
+            onProgress: (pct) => {
+                // Progress → stderr only, so stdout stays valid JSON for pipes.
+                context.io.stderr(`Exporting… ${Math.max(1, Math.min(99, pct))}%`);
+            },
+        });
+    }
+    async function refreshCookiesAndRetryExport() {
+        context.io.stderr("WeCom export returned 401/403. Opening browser to refresh cookies, then retrying once...");
+        let refreshed;
+        try {
+            refreshed = await collectWecomCookiesViaBrowser({
+                chromeProfilePath: paths.chromeProfilePath,
+                io: context.io,
+            });
+        }
+        catch (refreshErr) {
+            const msg = refreshErr instanceof Error ? refreshErr.message : String(refreshErr);
+            writeError({ error: "cookie_refresh_failed", message: msg });
+            return {
+                ok: false,
+                result: {
+                    exitCode: 1,
+                    telemetry: { outcome: "failure", errorKind: "cookie_refresh_failed", errorMessage: msg },
+                },
+            };
+        }
+        saveCookies(paths.cookiesPath, refreshed);
+        const valid = await validateCookies(refreshed);
+        if (!valid) {
+            const msg = "Cookies were refreshed but WeCom still returned 401/403.";
+            writeError({ error: "cookie_refresh_invalid", message: msg });
+            return {
+                ok: false,
+                result: {
+                    exitCode: 1,
+                    telemetry: { outcome: "failure", errorKind: "cookie_refresh_invalid", errorMessage: msg },
+                },
+            };
+        }
+        rawCookies = refreshed;
+        cookies = toCookieEntries(rawCookies);
+        tokValue = rawCookies["TOK"] ?? "";
+        try {
+            return { ok: true, fresh: await exportWithCurrentCookies() };
+        }
+        catch (retryErr) {
+            if (retryErr instanceof ExportError) {
+                writeError({
+                    error: "export_failed",
+                    errorKind: retryErr.kind,
+                    message: retryErr.message,
+                });
+                return {
+                    ok: false,
+                    result: {
+                        exitCode: 1,
+                        telemetry: {
+                            outcome: "failure",
+                            errorKind: retryErr.kind,
+                            errorMessage: retryErr.message,
+                        },
+                    },
+                };
+            }
+            const msg = retryErr instanceof Error ? retryErr.message : String(retryErr);
+            writeError({ error: "export_failed", message: msg });
+            return {
+                ok: false,
+                result: {
+                    exitCode: 1,
+                    telemetry: { outcome: "failure", errorKind: "unknown", errorMessage: msg },
+                },
+            };
+        }
+    }
+}
+function toCookieEntries(rawCookies) {
+    return rawCookies
+        ? Object.entries(rawCookies).map(([name, value]) => ({ name, value }))
+        : [];
 }

package/dist/wecom/export.js

@@ -20,6 +20,27 @@ export class ExportError extends Error {
         this.name = "ExportError";
     }
 }
+export function isExportAuthError(err) {
+    if (containsAuthFailure(err.detail))
+        return true;
+    return /(?:\b(?:401|403)\b|用户身份认证失败|not authenticated|unauthorized|forbidden)/i.test(err.message);
+}
+function containsAuthFailure(value) {
+    if (value === null || value === undefined)
+        return false;
+    if (typeof value === "number")
+        return value === 401 || value === 403;
+    if (typeof value === "string") {
+        return /(?:\b(?:401|403)\b|用户身份认证失败|not authenticated|unauthorized|forbidden)/i.test(value);
+    }
+    if (typeof value !== "object")
+        return false;
+    for (const entry of Object.values(value)) {
+        if (containsAuthFailure(entry))
+            return true;
+    }
+    return false;
+}
 // ---------------------------------------------------------------------------
 // Internal helpers
 // ---------------------------------------------------------------------------

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mkterswingman/5mghost-wonder",
-  "version": "0.0.19",
+  "version": "0.0.20",
   "description": "企微文档读取 CLI — WeCom document reader",
   "type": "module",
   "engines": {

package/skills/setup-5mghost-wonder/SKILL.md

@@ -7,9 +7,9 @@ description: Use this skill when the user wants to install or set up wonder, say
 
 ## Skill version
 
-This skill matches **wonder 0.0.19**.
+This skill matches **wonder 0.0.20**.
 
-Once the CLI is installed in Step 1, run `wonder --version`. If the output does not equal `0.0.19`, the CLI on disk has drifted from the skill text loaded in this session. Ask the user to run `/update-5mghost-wonder`, then **start a fresh AI session** (`/exit` and re-enter, or open a new chat) — skill text already loaded into a running session does not refresh after `wonder update`, even though the file on disk has been replaced.
+Once the CLI is installed in Step 1, run `wonder --version`. If the output does not equal `0.0.20`, the CLI on disk has drifted from the skill text loaded in this session. Ask the user to run `/update-5mghost-wonder`, then **start a fresh AI session** (`/exit` and re-enter, or open a new chat) — skill text already loaded into a running session does not refresh after `wonder update`, even though the file on disk has been replaced.
 
 After a successful first install, also remind the user to start a fresh AI session before invoking `/use-5mghost-wonder` for the first time. The skill files were just written to disk; the current session never loaded them.
 

package/skills/update-5mghost-wonder/SKILL.md

@@ -7,7 +7,7 @@ description: Use this skill when the user wants to update or upgrade wonder, say
 
 ## Skill version
 
-This skill matches **wonder 0.0.19**.
+This skill matches **wonder 0.0.20**.
 
 ---
 

package/skills/use-5mghost-wonder/SKILL.md

@@ -10,10 +10,10 @@ the referenced workflow files needed for the current task.
 
 ## Version Gate
 
-This skill matches **wonder 0.0.19**.
+This skill matches **wonder 0.0.20**.
 
 On first use in a session, follow `references/session-init.md`. If the installed
-CLI version differs from `0.0.19`, stop and ask the user to run
+CLI version differs from `0.0.20`, stop and ask the user to run
 `/update-5mghost-wonder`, then start a fresh AI session.
 
 ## Hard Rules