moodle-cli 0.7.0-alpha.6 → 0.7.0-alpha.7

package/ONBOARDING.md

@@ -19,7 +19,7 @@ Wait for the answer before doing setup work. Do not ask the student to find the
 
 Once the student shares a URL, complete the setup on their computer:
 
-- Check for Node.js 22+ and Bun. Install `moodle-cli` with the runtime available on the computer. Use `bunx --bun moodle-cli` as the command prefix when Bun is the only runtime.
+- Check for Node.js 22.13+ and Bun (older Node cannot read browser cookies). Install `moodle-cli` with the runtime available on the computer. Use `bunx --bun moodle-cli` as the command prefix when Bun is the only runtime.
 - Follow redirects and resolve the URL to the final Moodle origin in the form `https://host`. Remove the path, query, and fragment. Confirm that the origin serves Moodle before saving it.
 - Read `~/.config/moodle-cli/config.yaml` if it exists. Set `base_url` to the verified origin and preserve the other settings.
 - Tell the student that Moodle may open in their browser and that you will wait while they complete their university sign-in. Run `moodle auth login`. If a browser opens, let the student finish SSO there, then continue when the command returns.

package/README.md

@@ -6,7 +6,7 @@ Let it keep up with deadlines and grades, fetch course files, and search forum d
 
 [![npm version](https://img.shields.io/npm/v/moodle-cli?logo=npm)](https://www.npmjs.com/package/moodle-cli)
 [![CI](https://github.com/bunizao/moodle-cli/actions/workflows/ci.yml/badge.svg)](https://github.com/bunizao/moodle-cli/actions/workflows/ci.yml)
-[![Node.js 22+](https://img.shields.io/badge/Node.js-22%2B-339933?logo=nodedotjs&logoColor=white)](https://nodejs.org/)
+[![Node.js 22.13+](https://img.shields.io/badge/Node.js-22.13%2B-339933?logo=nodedotjs&logoColor=white)](https://nodejs.org/)
 [![Bun](https://img.shields.io/badge/Bun-supported-fbf0df?logo=bun&logoColor=black)](https://bun.sh/)
 [![MIT License](https://img.shields.io/badge/license-MIT-blue.svg)](LICENSE)
 
@@ -32,7 +32,7 @@ Your agent asks for your Moodle URL and opens your university's sign-in page whe
 
 ### Install and sign in manually
 
-Use Node.js 22+ or Bun:
+Use Node.js 22.13+ (needed for `node:sqlite`, which reads browser cookies) or Bun:
 
 ```bash
 # npm
@@ -176,7 +176,7 @@ moodle mcp bridge
 
 The default client connection uses `moodle mcp bridge`, which keeps the Bearer token out of client configuration. Use `moodle mcp connect CLIENT --mode remote` for clients that support authenticated remote MCP headers.
 
-Alpha version `0.7.0-alpha.6` supports MCP `2026-07-28` and a stateless compatibility lane for `2025-11-25` clients.
+Alpha version `0.7.0-alpha.7` supports MCP `2026-07-28` and a stateless compatibility lane for `2025-11-25` clients.
 
 ### Configuration
 

package/dist/moodle.js

@@ -406,15 +406,22 @@ async function loadSessionsFromOktaCli(baseUrl, options = {}, forceLogin = false
   return refreshed.length ? refreshed : stored;
 }
 var COOKIE_ACCESS_DENIED = /EPERM|EACCES|operation not permitted|permission denied/i;
+var COOKIE_SQLITE_UNAVAILABLE = /No such built-in module: node:sqlite/i;
+var MINIMUM_NODE_FOR_BROWSER_COOKIES = "22.13.0";
 function cookieAccessBlocked(warnings) {
-  return warnings.some((warning) => COOKIE_ACCESS_DENIED.test(warning));
+  return warnings.some((warning) => COOKIE_ACCESS_DENIED.test(warning) || COOKIE_SQLITE_UNAVAILABLE.test(warning));
 }
 function cookieAccessHint(warnings, platform = process.platform) {
   const grant = platform === "darwin" ? "Grant Full Disk Access to the application running this command (System Settings > Privacy & Security > Full Disk Access), then restart it." : "Run this command as the user that owns the browser profile, or grant it read access to the browser cookie store.";
+  const remedy = warnings.some((warning) => COOKIE_SQLITE_UNAVAILABLE.test(warning)) ? [
+    `This Node.js runtime has no node:sqlite, which is needed to read browser cookies. Use Node.js ${MINIMUM_NODE_FOR_BROWSER_COOKIES} or newer, or run the CLI with Bun (bunx --bun moodle-cli).`
+  ] : [
+    "If this runs inside a sandboxed app (an IDE or agent terminal), rerun it from a regular terminal first.",
+    grant
+  ];
   return [
     "The browser cookie store could not be read, so the session could not be detected.",
-    "If this runs inside a sandboxed app (an IDE or agent terminal), rerun it from a regular terminal first.",
-    grant,
+    ...remedy,
     `Alternatively set ${ENV_MOODLE_SESSION} to a valid MoodleSession cookie value.`,
     "",
     "Cookie store diagnostics:",
@@ -3880,7 +3887,7 @@ function escapeXml(value) {
 }
 
 // src/version.ts
-var VERSION = "0.7.0-alpha.6";
+var VERSION = "0.7.0-alpha.7";
 
 // src/forum.ts
 function parseDiscussionReference(value) {
@@ -5347,8 +5354,8 @@ var DeploymentPlanError = class extends Error {
   code;
 };
 var DeploymentApplyError = class extends Error {
-  constructor(code, message) {
-    super(message);
+  constructor(code, message, options) {
+    super(message, options);
     this.code = code;
     this.name = "DeploymentApplyError";
   }
@@ -5377,7 +5384,7 @@ var ManagedMcpDeployment = class {
     const receipt = replacingExisting || remote === null ? null : matchingReceipt;
     const existing = remote && receipt ? { ...remote, productionEndpoint: receipt.productionEndpoint, releaseDigest: receipt.releaseDigest } : remote;
     const rotate = intent.rotateToken === true && credentials !== null;
-    const releaseChanged = existing?.releaseDigest !== intent.releaseDigest;
+    const releaseChanged = existing?.releaseDigest !== intent.releaseDigest || receipt?.restoredRelease === true;
     const uploadCandidate = !existing || replacingExisting || releaseChanged || intent.repair === true || rotate;
     return {
       intent: { ...intent },
@@ -5424,20 +5431,13 @@ var ManagedMcpDeployment = class {
       yield completed(activeStage);
       activeStage = "upload_private_credentials";
       yield started(activeStage);
-      if (plan.uploadCandidate) {
-        if (plan.operation === "create") {
-          initializedWorker = await this.dependencies.wrangler.initializeWorker({
-            accountId: plan.intent.accountId,
-            workerName: plan.intent.workerName,
-            configPath: prepared.wranglerConfigPath,
-            releaseDigest: plan.intent.releaseDigest
-          });
-        }
-        await this.dependencies.wrangler.uploadSecrets({
+      if (plan.uploadCandidate && plan.operation === "create") {
+        initializedWorker = await this.dependencies.wrangler.initializeWorker({
           accountId: plan.intent.accountId,
           workerName: plan.intent.workerName,
           configPath: prepared.wranglerConfigPath,
-          secretsFilePath: prepared.secretsFilePath
+          secretsFilePath: prepared.secretsFilePath,
+          releaseDigest: plan.intent.releaseDigest
         });
         secretsUploaded = true;
       }
@@ -5453,9 +5453,11 @@ var ManagedMcpDeployment = class {
           accountId: plan.intent.accountId,
           workerName: plan.intent.workerName,
           configPath: prepared.wranglerConfigPath,
+          secretsFilePath: prepared.secretsFilePath,
           releaseDigest: plan.intent.releaseDigest,
           productionEndpoint: productionEndpoint2
         });
+        secretsUploaded = true;
         if (!candidate.previewEndpoint) {
           await this.dependencies.wrangler.promote({
             accountId: plan.intent.accountId,
@@ -5605,10 +5607,9 @@ var ManagedMcpDeployment = class {
     let readinessReasonCode = null;
     let sessionRevision = null;
     if (worker && credentials) {
-      const remoteReadiness = await this.dependencies.worker.getReadiness({
-        endpoint: receipt.productionEndpoint,
-        sessionSyncToken: credentials.sessionSyncToken
-      });
+      const target = { endpoint: receipt.productionEndpoint, sessionSyncToken: credentials.sessionSyncToken };
+      await this.dependencies.worker.touchSession(target);
+      const remoteReadiness = await this.dependencies.worker.getReadiness(target);
       readiness = remoteReadiness.status;
       readinessReasonCode = remoteReadiness.reasonCode;
       sessionRevision = remoteReadiness.revision;
@@ -5676,6 +5677,7 @@ var ManagedMcpDeployment = class {
       await this.dependencies.receipts.write({
         ...receipt,
         productionVersionId: worker.previousHealthyVersionId,
+        ...swappedDigests(receipt),
         sessionRevision: upload.revision
       });
       await this.reconcileLocalIntegrations(profile);
@@ -5722,7 +5724,8 @@ var ManagedMcpDeployment = class {
     }
     await this.dependencies.receipts.write({
       ...receipt,
-      productionVersionId: previousVersionId
+      productionVersionId: previousVersionId,
+      ...swappedDigests(receipt)
     });
     return { status: "restored", versionId: previousVersionId };
   }
@@ -5829,6 +5832,7 @@ function makeReceipt(plan, candidate, sessionRevision) {
       productionEndpoint: candidate.productionEndpoint,
       productionVersionId: candidate.versionId,
       releaseDigest: plan.intent.releaseDigest,
+      ...previousDigest(plan.existing?.releaseDigest),
       sessionRevision
     };
   }
@@ -5844,9 +5848,16 @@ function makeReceipt(plan, candidate, sessionRevision) {
     productionEndpoint: plan.existing.productionEndpoint,
     productionVersionId: plan.existing.productionVersionId,
     releaseDigest: plan.existing.releaseDigest,
+    ...previousDigest(plan.receipt?.previousReleaseDigest),
     sessionRevision
   };
 }
+function previousDigest(digest2) {
+  return digest2 === void 0 ? {} : { previousReleaseDigest: digest2 };
+}
+function swappedDigests(receipt) {
+  return { releaseDigest: receipt.previousReleaseDigest ?? "", previousReleaseDigest: receipt.releaseDigest, restoredRelease: true };
+}
 function started(stageId) {
   return event(stageId, "started");
 }
@@ -5867,7 +5878,8 @@ function asDeploymentError(error) {
   if (error instanceof DeploymentApplyError) {
     return error;
   }
-  return new DeploymentApplyError("DEPLOYMENT_FAILED", "The managed Moodle MCP deployment failed");
+  const detail = error instanceof Error && error.message ? `: ${error.message}` : "";
+  return new DeploymentApplyError("DEPLOYMENT_FAILED", `The managed Moodle MCP deployment failed${detail}`, { cause: error });
 }
 
 // src/mcp/deployment/node-adapters.ts
@@ -6015,6 +6027,7 @@ function macOSPlan(options, intervalMinutes) {
   const label = `com.moodle-cli.mcp-renewal.${options.profile}`;
   const path4 = `${trimEnd(options.homeDirectory, "/")}/Library/LaunchAgents/${label}.plist`;
   const target = `gui/${options.uid}`;
+  const logPath = `${trimEnd(options.homeDirectory, "/")}/Library/Logs/${label}.log`;
   const plist = [
     '<?xml version="1.0" encoding="UTF-8"?>',
     '<!DOCTYPE plist PUBLIC "-//Apple//DTD PLIST 1.0//EN" "http://www.apple.com/DTDs/PropertyList-1.0.dtd">',
@@ -6025,6 +6038,8 @@ function macOSPlan(options, intervalMinutes) {
     "</array>",
     `<key>StartInterval</key><integer>${intervalMinutes * 60}</integer>`,
     "<key>RunAtLoad</key><true/>",
+    `<key>StandardOutPath</key><string>${xml(logPath)}</string>`,
+    `<key>StandardErrorPath</key><string>${xml(logPath)}</string>`,
     "</dict></plist>",
     ""
   ].join("\n");
@@ -6319,7 +6334,7 @@ var NodeDeploymentCommandRunner = class {
 };
 var WranglerCommandError = class extends Error {
   constructor(exitCode, stdout, stderr) {
-    super("Packaged Wrangler command failed");
+    super(wranglerFailureMessage(stderr, stdout));
     this.exitCode = exitCode;
     this.stdout = stdout;
     this.stderr = stderr;
@@ -6329,6 +6344,13 @@ var WranglerCommandError = class extends Error {
   stdout;
   stderr;
 };
+function wranglerFailureMessage(stderr, stdout) {
+  const lines = `${stderr}
+${stdout}`.replace(/\u001B\[[0-9;]*m/gu, "").split(/\r?\n/u).map((line) => line.trim());
+  const errorIndex = lines.findIndex((line) => line.includes("[ERROR]"));
+  const detail = errorIndex >= 0 ? lines.slice(errorIndex).filter((line) => line && !/^(To learn more|If you think this is a bug|Logs were written)/u.test(line)).slice(0, 3).join(" ").replace(/^.*\[ERROR\]\s*/u, "") : "";
+  return detail ? `Packaged Wrangler command failed: ${detail.slice(0, 600)}` : "Packaged Wrangler command failed";
+}
 var NodeWranglerDeploymentAdapter = class {
   wranglerBinPath;
   runner;
@@ -6371,8 +6393,8 @@ ${error.stderr}`)) {
       throw error;
     }
     const document = parseJsonOutput(result.stdout);
-    const versionIds = deploymentVersionIds(document);
-    if (!versionIds.length) {
+    const [current, previous] = deploymentHistory(document);
+    if (!current) {
       return null;
     }
     const productionEndpoint = firstWorkersDevUrl(document) ?? `https://${workerName}.workers.dev`;
@@ -6383,22 +6405,11 @@ ${error.stderr}`)) {
       deploymentId: deploymentId2,
       ownershipTag: deploymentId2,
       productionEndpoint,
-      productionVersionId: versionIds[0],
-      previousHealthyVersionId: versionIds[1] ?? null,
-      releaseDigest: releaseDigestFromDocument(document) ?? ""
+      productionVersionId: current.versionId,
+      previousHealthyVersionId: previous?.versionId ?? null,
+      releaseDigest: releaseDigestFromMessage(current.message) ?? ""
     };
   }
-  async uploadSecrets(input) {
-    await this.wrangler([
-      "secret",
-      "bulk",
-      input.secretsFilePath,
-      "--name",
-      input.workerName,
-      "--config",
-      input.configPath
-    ], input.accountId);
-  }
   async initializeWorker(input) {
     let result = null;
     try {
@@ -6408,6 +6419,8 @@ ${error.stderr}`)) {
         input.workerName,
         "--config",
         input.configPath,
+        "--secrets-file",
+        input.secretsFilePath,
         "--message",
         `moodle-cli-bootstrap:${input.releaseDigest}`
       ], input.accountId);
@@ -6442,6 +6455,8 @@ ${error.stderr}`)) {
         input.workerName,
         "--config",
         input.configPath,
+        "--secrets-file",
+        input.secretsFilePath,
         "--preview-alias",
         "moodle-cli-candidate",
         "--message",
@@ -6568,7 +6583,7 @@ var DefaultMoodleSessionSource = class {
   }
   options;
   async loadValidated(_profile, moodleOrigin) {
-    const session = this.options.interactive === false ? await getAuthenticatedSession(moodleOrigin, { ...this.options, nonInteractive: true }) : await getAuthenticatedSessionWithBrowserFallback(moodleOrigin, this.options);
+    const session = this.options.interactive === false ? await getAuthenticatedSession(moodleOrigin, { ...this.options, nonInteractive: true, noCache: true }) : await getAuthenticatedSessionWithBrowserFallback(moodleOrigin, this.options);
     return {
       moodleOrigin,
       cookieName: session.cookie.name,
@@ -6610,7 +6625,7 @@ var FetchManagedWorkerClient = class {
       return { revision: body.revision };
     }
     const code = isRecord8(body) && typeof body.code === "string" ? body.code : "SESSION_UPLOAD_FAILED";
-    throw new DeploymentApplyError(code, "The Worker rejected the Moodle session update");
+    throw new DeploymentApplyError(code, `The Worker rejected the Moodle session update (${code})`);
   }
   async getReadiness(input) {
     const response = await this.fetchWithRetry(endpointUrl(input.endpoint, "/readyz"), {
@@ -6628,6 +6643,12 @@ var FetchManagedWorkerClient = class {
     }
     return { status: "fail", reasonCode: null, revision: null };
   }
+  async touchSession(input) {
+    await this.fetchWithRetry(endpointUrl(input.endpoint, "/session/touch"), {
+      method: "POST",
+      headers: { authorization: `Bearer ${input.sessionSyncToken}` }
+    }, isRetryableWorkerRouting);
+  }
   async runSmoke(input) {
     const health = await this.fetchWithRetry(
       endpointUrl(input.endpoint, "/healthz"),
@@ -6819,6 +6840,9 @@ function isRetryableSessionUpload(status) {
 function isRetryableWorkerPropagation(status) {
   return status === 404 || status === 429 || status >= 500;
 }
+function isRetryableWorkerRouting(status) {
+  return status === 404 || status === 429;
+}
 async function safeJson(response) {
   try {
     return await response.json();
@@ -6871,30 +6895,35 @@ async function readWranglerVersionUpload(outputFilePath, workerName) {
   }
   throw new DeploymentApplyError("CANDIDATE_UPLOAD_INVALID", "Wrangler wrote unsupported candidate metadata");
 }
-function collectStrings(value, keys) {
-  const result = [];
-  visit(value, (key, item) => {
-    if (keys.has(key) && typeof item === "string") {
-      result.push(item);
+function deploymentHistory(value) {
+  const entries = [];
+  visit(value, (_key, item) => {
+    if (!isRecord8(item) || !Array.isArray(item.versions)) {
+      return;
     }
+    const versionId = activeVersionId(item.versions);
+    if (!versionId) {
+      return;
+    }
+    const createdOn = typeof item.created_on === "string" ? Date.parse(item.created_on) : Number.NaN;
+    const message = isRecord8(item.annotations) ? item.annotations["workers/message"] : void 0;
+    entries.push({
+      versionId,
+      message: typeof message === "string" ? message : null,
+      createdOn: Number.isNaN(createdOn) ? 0 : createdOn,
+      index: entries.length
+    });
   });
-  return result;
+  return entries.sort((a, b) => b.createdOn - a.createdOn || b.index - a.index).map(({ versionId, message }) => ({ versionId, message }));
 }
-function deploymentVersionIds(value) {
-  const ids = collectStrings(value, /* @__PURE__ */ new Set(["version_id", "versionId"]));
-  return [...new Set(ids)];
+function activeVersionId(versions) {
+  const records = versions.filter(isRecord8);
+  const active = records.find((version) => version.percentage === 100) ?? records[0];
+  const id = active?.version_id ?? active?.versionId;
+  return typeof id === "string" ? id : null;
 }
-function releaseDigestFromDocument(value) {
-  let digestValue = null;
-  visit(value, (_key, item) => {
-    if (!digestValue && typeof item === "string") {
-      const match = item.match(/moodle-cli-release:([a-zA-Z0-9._-]+)/u);
-      if (match?.[1]) {
-        digestValue = match[1];
-      }
-    }
-  });
-  return digestValue;
+function releaseDigestFromMessage(message) {
+  return message?.match(/moodle-cli-release:([a-zA-Z0-9._-]+)/u)?.[1] ?? null;
 }
 function collectAccountObjects(value) {
   const accounts = [];
@@ -8062,10 +8091,9 @@ var DefaultMcpCommandService = class {
       throw new UsageError(`No managed Moodle MCP deployment exists for profile ${profile}.`);
     }
     let receipt = storedReceipt;
-    const readiness = await this.worker.getReadiness({
-      endpoint: receipt.productionEndpoint,
-      sessionSyncToken: credentials.sessionSyncToken
-    });
+    const target = { endpoint: receipt.productionEndpoint, sessionSyncToken: credentials.sessionSyncToken };
+    await this.worker.touchSession(target);
+    const readiness = await this.worker.getReadiness(target);
     if (readiness.revision !== null && readiness.revision !== receipt.sessionRevision) {
       receipt = await this.writeRenewalRevision(receipt, readiness.revision);
     }
@@ -8077,6 +8105,7 @@ var DefaultMcpCommandService = class {
       agentInstalled: await this.renewal.inspect(profile)
     };
     let replacement = null;
+    let signInDetail;
     if (snapshot.remote === "expiring" || snapshot.remote === "expired") {
       try {
         replacement = await this.sessions.loadValidated(profile, receipt.moodleOrigin);
@@ -8086,6 +8115,7 @@ var DefaultMcpCommandService = class {
           throw error;
         }
         snapshot.replacement = { source: "mfa_required" };
+        signInDetail = [error.message, error.hint].filter(Boolean).join(" ");
       }
     }
     let uploaded = false;
@@ -8138,9 +8168,10 @@ var DefaultMcpCommandService = class {
         text: "Moodle MCP session renewed."
       };
     }
+    const detail = decision.state === "needs_sign_in" && signInDetail ? { detail: signInDetail } : {};
     return {
-      data: { profile, state: decision.state, reasonCode: decision.reasonCode, revision: receipt.sessionRevision },
-      text: renewalResultText(decision)
+      data: { profile, state: decision.state, reasonCode: decision.reasonCode, revision: receipt.sessionRevision, ...detail },
+      text: [renewalResultText(decision), signInDetail].filter(Boolean).join("\n")
     };
   }
   async writeRenewalRevision(receipt, revision) {

package/dist/worker/worker.js

@@ -20597,7 +20597,7 @@ function stringValue(value) {
 }
 
 // src/version.ts
-var VERSION = "0.7.0-alpha.6";
+var VERSION = "0.7.0-alpha.7";
 
 // src/worker/http.ts
 var HEALTH_PATH = "/healthz";

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "moodle-cli",
-  "version": "0.7.0-alpha.6",
+  "version": "0.7.0-alpha.7",
   "description": "Terminal-first CLI for Moodle LMS",
   "license": "MIT",
   "type": "module",
@@ -47,7 +47,7 @@
     "vitest": "^3.2.4"
   },
   "engines": {
-    "node": ">=22"
+    "node": ">=22.13"
   },
   "repository": {
     "type": "git",