@retasc/cli 1.3.0 → 1.3.1

package/dist/api.js

@@ -46,12 +46,16 @@ export function isAuthError(e) {
  *   1. If there is no stored token at all, the user was never signed in — this
  *      isn't an expiry, so rethrow the clean "sign in first" error rather than
  *      launching a surprise device flow under a misleading "session expired".
- *   2. `refreshSession()` — swap the refresh token for a fresh access token.
- *   3. If that fails (refresh token expired/invalid), fall back to a full
- *      `deviceLogin()` — but only when stdout is a TTY, since the device flow
- *      prints a code to stdout for a human to authorize; in a captured/piped
- *      context we surface a clear "run `retasc login`" error instead of hanging
- *      or spilling the prompt into someone's `$(retasc …)` capture.
+ *   2. `refreshSession()` — swap the refresh token for a fresh access token. A
+ *      transient backend failure here (network/5xx/masked "Server Error") is
+ *      rethrown by `refreshSession`, so it propagates out as a retryable error
+ *      instead of forcing a device-flow re-login on a valid session (RTSC-178).
+ *   3. If the refresh token is genuinely expired/invalid (a clean `false`, not a
+ *      throw), fall back to a full `deviceLogin()` — but only when stdout is a
+ *      TTY, since the device flow prints a code to stdout for a human to
+ *      authorize; in a captured/piped context we surface a clear "run `retasc
+ *      login`" error instead of hanging or spilling the prompt into someone's
+ *      `$(retasc …)` capture.
  * A second auth failure after recovery is real (rethrown) — we never loop.
  */
 async function withAuth(call) {

package/dist/auth.js

@@ -76,6 +76,31 @@ export async function deviceLogin() {
     }
     patchConfig({ token: tokens.token, refreshToken: tokens.refreshToken });
 }
+/**
+ * Does this thrown error clearly mean "this refresh token can never mint a new
+ * session" — as opposed to a transient backend blip? RTSC-178.
+ *
+ * Note the asymmetry in how Convex Auth's refresh grant fails:
+ *   - A genuinely expired/invalid/reused refresh token does NOT throw — the
+ *     server returns `{ tokens: null }`, which `refreshSession` already reads as
+ *     `false` (no exception reaches here).
+ *   - The `catch` only sees *thrown* errors: network failures, 5xx, and the
+ *     opaque masked "Server Error" prod returns for any uncaught server-side
+ *     throw — all transient — plus the rare corrupt/unparseable stored token
+ *     (`parseRefreshToken` throws "Can't parse refresh token …").
+ *
+ * So we default to "transient" and only classify as unrefreshable when the
+ * message is unmistakably refresh-token-shaped. In prod a corrupt-token throw is
+ * itself masked to "Server Error" and so reads as transient — the deliberate,
+ * safe bias (RTSC-165 saw exactly this masking): never drag a user with a valid
+ * session through the whole device flow on a one-off hiccup. A truly dead
+ * refresh token still reaches `deviceLogin` via the `{ tokens: null }` → `false`
+ * path above, unaffected by this classifier.
+ */
+export function isUnrefreshableRefreshToken(err) {
+    const msg = String(err?.message ?? err);
+    return /can'?t parse refresh token|cannot parse refresh token|invalid refresh token|expired refresh token|refresh token (?:is |has )?(?:invalid|expired)/i.test(msg);
+}
 /**
  * Silently mint a fresh access token from the stored refresh token and persist
  * the rotated pair to ~/.retasc/config.json.
@@ -88,10 +113,12 @@ export async function deviceLogin() {
  * tokens are single-use; the old one is invalidated), so we must write the new
  * pair back immediately and never reuse the old refresh token.
  *
- * Returns true on success (config now holds a fresh session), false if there is
- * no refresh token or the grant is rejected (expired/invalid/reused) — the
- * caller then falls back to a full `deviceLogin()`. Never throws and never
- * prints token values.
+ * Returns true on success (config now holds a fresh session), false when there
+ * is no refresh token or the grant is cleanly rejected (expired/invalid/reused,
+ * which the server signals as `{ tokens: null }`) — the caller then falls back
+ * to a full `deviceLogin()`. A *transient* failure (network/5xx/opaque backend
+ * blip) is RE-THROWN so the caller can surface a retryable error instead of
+ * forcing a needless device-flow re-login (RTSC-178). Never prints token values.
  */
 export async function refreshSession() {
     const cfg = loadConfig();
@@ -106,9 +133,14 @@ export async function refreshSession() {
         patchConfig({ token: tokens.token, refreshToken: tokens.refreshToken });
         return true;
     }
-    catch {
-        // Expired/invalid/reused refresh token, or a transient backend error. Treat
-        // as "cannot refresh" — the caller re-authenticates via the device flow.
-        return false;
+    catch (err) {
+        // A clearly refresh-token-shaped rejection means we genuinely cannot
+        // refresh — fall through to device login. Anything else (network, 5xx,
+        // masked prod "Server Error") is transient: rethrow so `withAuth` surfaces a
+        // retryable error rather than dragging a valid session through the device
+        // flow at an hourly access-token boundary.
+        if (isUnrefreshableRefreshToken(err))
+            return false;
+        throw err;
     }
 }

package/dist/config.js

@@ -1,6 +1,7 @@
 import { homedir } from "node:os";
 import { join } from "node:path";
-import { mkdirSync, readFileSync, writeFileSync, existsSync, chmodSync } from "node:fs";
+import { mkdirSync, readFileSync, writeFileSync, renameSync, unlinkSync, existsSync, chmodSync, } from "node:fs";
+import { randomUUID } from "node:crypto";
 // Production defaults. Overridable via env for dev/testing.
 //   RETASC_DEPLOYMENT_URL — Convex deployment (.cloud) for management calls
 //   RETASC_MCP_URL        — the MCP endpoint agents connect to
@@ -8,20 +9,41 @@ export const DEFAULTS = {
     deploymentUrl: process.env.RETASC_DEPLOYMENT_URL ?? "https://unique-lyrebird-934.convex.cloud",
     mcpUrl: process.env.RETASC_MCP_URL ?? "https://mcp.retasc.com/mcp",
 };
-const DIR = join(homedir(), ".retasc");
-const FILE = join(DIR, "config.json");
+/** The dir config lives in. RETASC_DIR overrides it (tests, sandboxes) — same
+ *  knob the keystore reads, so the two stay co-located. */
+function configDir() {
+    return process.env.RETASC_DIR || join(homedir(), ".retasc");
+}
 export function configPath() {
-    return FILE;
+    return join(configDir(), "config.json");
 }
 export function loadConfig() {
+    const FILE = configPath();
     let stored = {};
     if (existsSync(FILE)) {
+        let raw;
         try {
-            stored = JSON.parse(readFileSync(FILE, "utf8"));
+            raw = readFileSync(FILE, "utf8");
         }
         catch {
-            // Corrupt config — start fresh rather than crash.
-            stored = {};
+            // A read that FAILS (file lock, EIO, EMFILE, stale NFS handle) is transient,
+            // not corruption. Leave the file untouched and fall back to defaults for this
+            // run — self-heal next time — rather than renaming a possibly-valid config
+            // aside and turning a blip into a permanent logout.
+            raw = undefined;
+        }
+        if (raw !== undefined) {
+            try {
+                stored = JSON.parse(raw);
+            }
+            catch {
+                // Read succeeded but the bytes aren't valid JSON → genuinely corrupt/
+                // truncated (e.g. an interrupted write). Preserve them under a backup name
+                // rather than silently discarding — the file may still hold the only copy
+                // of the user's tokens, recoverable by hand. Then start fresh.
+                backupCorruptConfig(FILE);
+                stored = {};
+            }
         }
     }
     // Defaults fill in; a saved value always wins.
@@ -35,19 +57,49 @@ export function loadConfig() {
         defaultProjectPrefix: stored.defaultProjectPrefix,
     };
 }
-export function saveConfig(cfg) {
-    // Create the dir user-only and the file 0600 on the initial write, so tokens
-    // never exist world-readable even briefly (the chmod below only closes a
-    // pre-existing 0644 file — it can't undo a world-readable creation window).
-    mkdirSync(DIR, { recursive: true, mode: 0o700 });
-    writeFileSync(FILE, JSON.stringify(cfg, null, 2) + "\n", { encoding: "utf8", mode: 0o600 });
-    // Tokens live here — keep it user-only readable (covers overwriting an
-    // existing 0644 file, where the write mode above doesn't apply).
+/** Move a corrupt config aside to a unique sibling so a human can recover any
+ *  tokens it still holds. Best effort — never throw from the load path. */
+function backupCorruptConfig(file) {
     try {
-        chmodSync(FILE, 0o600);
+        renameSync(file, `${file}.corrupt-${randomUUID()}`);
     }
     catch {
-        /* best effort (e.g. Windows) */
+        /* best effort — if we can't back it up, saveConfig will overwrite it */
+    }
+}
+export function saveConfig(cfg) {
+    const dir = configDir();
+    const FILE = configPath();
+    // Create the dir user-only and the temp file 0600, so tokens never exist
+    // world-readable even briefly (the chmod below only closes a pre-existing
+    // 0644 file — it can't undo a world-readable creation window).
+    mkdirSync(dir, { recursive: true, mode: 0o700 });
+    // Atomic write: fully write a sibling temp file, then rename it over the real
+    // one. rename(2) is atomic on POSIX, so a crash or an interleaved write leaves
+    // either the old complete config or the new one — never a truncated file that
+    // loadConfig would read as "logged out". The temp lives in the same dir so the
+    // rename stays on one filesystem (a cross-device rename is not atomic).
+    const tmp = join(dir, `.config.json.${randomUUID()}.tmp`);
+    const body = JSON.stringify(cfg, null, 2) + "\n";
+    try {
+        writeFileSync(tmp, body, { encoding: "utf8", mode: 0o600 });
+        try {
+            chmodSync(tmp, 0o600); // keep user-only if a umask/prior file loosened it
+        }
+        catch {
+            /* best effort (e.g. Windows) */
+        }
+        renameSync(tmp, FILE);
+    }
+    catch (err) {
+        // Don't leave a stray temp file behind on failure.
+        try {
+            unlinkSync(tmp);
+        }
+        catch {
+            /* already gone */
+        }
+        throw err;
     }
 }
 export function patchConfig(patch) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@retasc/cli",
-  "version": "1.3.0",
+  "version": "1.3.1",
   "description": "Retasc CLI — sign in with GitHub, create projects, mint agent API keys, and wire your agent to the Retasc MCP server in one command.",
   "type": "module",
   "bin": {