mnfst-publish 0.1.1 → 0.1.4

package/manifest.publish.mjs

@@ -109,6 +109,35 @@ function prerenderOutputDir(root) {
 
 // --- MCP JSON-RPC over Streamable HTTP -------------------------------------
 
+const sleep = (ms) => new Promise((r) => setTimeout(r, ms));
+
+// Retry transient failures — a thrown fetch (network blip, worker cold-start /
+// redeploy window) or a 5xx — with a short backoff, so a single hiccup doesn't
+// surface as a raw "fetch failed". Publish/promote/upload are idempotent enough
+// that a repeat is harmless. 4xx are NOT retried (they're real client errors).
+async function fetchRetry(url, init, { tries = 4, label = 'the server' } = {}) {
+  let lastErr;
+  for (let attempt = 1; attempt <= tries; attempt++) {
+    try {
+      const res = await fetch(url, init);
+      if (res.status >= 500 && attempt < tries) {
+        log(`  ${label} had a hiccup (HTTP ${res.status}) — retrying (${attempt}/${tries - 1})…`);
+        await sleep(500 * attempt);
+        continue;
+      }
+      return res;
+    } catch (e) {
+      lastErr = e;
+      if (attempt < tries) {
+        log(`  couldn't reach ${label} — retrying (${attempt}/${tries - 1})…`);
+        await sleep(500 * attempt);
+        continue;
+      }
+    }
+  }
+  throw lastErr;
+}
+
 async function mcp(url, key, method, params, sessionId) {
   const headers = {
     'content-type': 'application/json',
@@ -116,7 +145,11 @@ async function mcp(url, key, method, params, sessionId) {
     'x-api-key': key,
   };
   if (sessionId) headers['mcp-session-id'] = sessionId;
-  const res = await fetch(url, { method: 'POST', headers, body: JSON.stringify({ jsonrpc: '2.0', id: 1, method, params }) });
+  const res = await fetchRetry(
+    url,
+    { method: 'POST', headers, body: JSON.stringify({ jsonrpc: '2.0', id: 1, method, params }) },
+    { label: 'Manifest' },
+  );
   const sid = res.headers.get('mcp-session-id') || sessionId;
   const text = await res.text();
   // Response may be a JSON object or an SSE "data: {...}" line.
@@ -165,7 +198,6 @@ export function isExcludedPath(rel) {
   const base = parts[parts.length - 1];
   if (base === '.env' || base.startsWith('.env.')) return true; // .env, .env.local, .env.prod …
   if (base === '.npmrc' || base === '.dev.vars' || base === 'id_rsa' || base === '.DS_Store') return true;
-  if (base === '.mnfst-render-complete') return true; // internal render sentinel — not part of the site
   if (/\.(pem|key|p12|pfx)$/i.test(base)) return true;
   return false;
 }
@@ -298,16 +330,19 @@ export async function main() {
     if (r.status !== 0) fail('render failed — fix the errors above and try again.');
   }
 
-  // For a render project, confirm the output is a COMPLETE render before shipping
-  // it. mnfst-render writes a .mnfst-render-complete sentinel as its final step
-  // (atomic swap), so its absence means the dir is stale or half-built — refuse
-  // rather than publish a broken/empty site.
+  // For a render project, sanity-check there's actually a built site to ship —
+  // an output folder with an index.html. (When we just ran the render above, its
+  // exit code was already checked; this also covers --no-render and hand-built
+  // output.) Deliberately forgiving: no hard dependency on any marker file, so a
+  // good build always publishes regardless of which mnfst-render produced it.
   if (source === 'render') {
     const outDir = prerenderOutputDir(root);
-    if (!existsSync(join(root, outDir, '.mnfst-render-complete'))) {
+    if (!existsSync(join(root, outDir, 'index.html'))) {
       fail(
-        `the "${outDir}" directory isn't a complete render` +
-          (opts.render === false ? ' — re-run without --no-render.' : ' — run `npx mnfst-render` and try again.'),
+        `the "${outDir}" folder doesn't have a built site yet (no index.html). ` +
+          (opts.render === false
+            ? 'Remove --no-render so it builds first, or run `npx mnfst-render`, then publish again.'
+            : 'Run `npx mnfst-render` first, then publish again.'),
       );
     }
   }
@@ -335,11 +370,11 @@ export async function main() {
   const zip = buildZip(root, rels);
   log(`Uploading ${rels.length} files (${(zip.length / 1048576).toFixed(1)} MB)…`);
 
-  const up = await fetch(uploadUrl, {
-    method: 'POST',
-    headers: { 'content-type': 'application/zip' },
-    body: zip,
-  });
+  const up = await fetchRetry(
+    uploadUrl,
+    { method: 'POST', headers: { 'content-type': 'application/zip' }, body: zip },
+    { label: 'the upload server' },
+  );
   const upText = await up.text();
   let upJson = null;
   try {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "mnfst-publish",
-  "version": "0.1.1",
+  "version": "0.1.4",
   "description": "One-command managed publishing for Manifest projects — render, zip, upload, live URL.",
   "type": "module",
   "bin": {