freertc 0.1.19 → 0.1.21

package/README.md

@@ -216,7 +216,7 @@ Quick checks:
 Expected `/health` response includes JSON like:
 
 ```json
-{"ok":true,"version":"0.1.19","protocol_version":"1.0","peers":0}
+{"ok":true,"version":"0.1.21","protocol_version":"1.0","peers":0}
 ```
 
 ## Auto WebRTC two-tab test

package/bin/freertc.mjs

@@ -107,6 +107,11 @@ function extractCurrentWorkerName(configText) {
   return match?.[1] || null;
 }
 
+function extractDeployHost(configPath) {
+  const text = fs.readFileSync(configPath, 'utf8');
+  return extractRouteHost(text) || extractRelayUrlHost(text) || extractDatabaseDomainHost(text);
+}
+
 function extractRelayUrlHost(configText) {
   const relayUrlMatch = configText.match(/"RELAY_URL"\s*:\s*"([^"]*)"/);
   if (!relayUrlMatch) return null;
@@ -350,25 +355,41 @@ function runDeployWithAutoRouteRecovery(wrangler, configPath, extraArgs = []) {
     if (result.stderr) process.stderr.write(result.stderr);
 
     if (typeof result.status === 'number' && result.status === 0) {
-      process.exit(0);
+      return { ok: true, status: 0 };
     }
 
     const combinedOutput = `${result.stdout || ''}\n${result.stderr || ''}`;
     const conflictOwner = extractRouteConflictOwner(combinedOutput);
     if (!conflictOwner || attemptedOwners.has(conflictOwner)) {
-      process.exit(typeof result.status === 'number' ? result.status : 1);
+      return { ok: false, status: typeof result.status === 'number' ? result.status : 1 };
     }
 
     attemptedOwners.add(conflictOwner);
     const patched = patchWranglerScriptName(configPath, conflictOwner);
     if (!patched) {
-      process.exit(typeof result.status === 'number' ? result.status : 1);
+      return { ok: false, status: typeof result.status === 'number' ? result.status : 1 };
     }
 
     console.log(`Detected route owned by worker "${conflictOwner}". Retrying deploy with that worker name automatically.`);
   }
 
-  process.exit(1);
+  return { ok: false, status: 1 };
+}
+
+async function fetchLiveHealthVersion(host) {
+  const normalizedHost = normalizeHost(host);
+  if (!normalizedHost) return null;
+  try {
+    const response = await fetch(`https://${normalizedHost}/health`, {
+      headers: { Accept: 'application/json' }
+    });
+    if (!response.ok) return null;
+    const payload = await response.json();
+    const version = payload?.version;
+    return typeof version === 'string' && version.trim() ? version.trim() : null;
+  } catch {
+    return null;
+  }
 }
 
 function compareVersions(left, right) {
@@ -496,7 +517,46 @@ async function main() {
     await maybeAlignWorkerNameToRouteOwner(configPath);
     validateWranglerConfigForDeploy(configPath);
     const wrangler = resolveWranglerCommand(PROJECT_ROOT);
-    runDeployWithAutoRouteRecovery(wrangler, configPath, rest);
+    const firstDeploy = runDeployWithAutoRouteRecovery(wrangler, configPath, rest);
+    if (!firstDeploy.ok) {
+      process.exit(firstDeploy.status);
+    }
+
+    const deployHost = extractDeployHost(configPath);
+    if (!deployHost) {
+      process.exit(0);
+    }
+
+    let liveVersion = await fetchLiveHealthVersion(deployHost);
+    if (liveVersion && compareVersions(liveVersion, CLI_VERSION) >= 0) {
+      process.exit(0);
+    }
+
+    // If live version is still stale, force-resolve route owner and redeploy once.
+    const routeOwner = await findRouteOwnerForHost(deployHost);
+    if (routeOwner) {
+      const currentText = fs.readFileSync(configPath, 'utf8');
+      const currentName = extractCurrentWorkerName(currentText);
+      if (currentName !== routeOwner) {
+        patchWranglerScriptName(configPath, routeOwner);
+        console.log(`Live health still stale on ${deployHost}. Retrying deploy against route-owner worker "${routeOwner}".`);
+      } else {
+        console.log(`Live health still stale on ${deployHost}. Retrying deploy once for route-owner worker "${routeOwner}".`);
+      }
+
+      const secondDeploy = runDeployWithAutoRouteRecovery(wrangler, configPath, rest);
+      if (!secondDeploy.ok) {
+        process.exit(secondDeploy.status);
+      }
+      liveVersion = await fetchLiveHealthVersion(deployHost);
+      if (liveVersion && compareVersions(liveVersion, CLI_VERSION) >= 0) {
+        process.exit(0);
+      }
+    }
+
+    console.error(`Deploy completed but live health on https://${deployHost}/health is still stale (version=${liveVersion || 'unknown'}, expected>=${CLI_VERSION}).`);
+    console.error('Refusing silent success. Please rerun deploy after Cloudflare route cache settles.');
+    process.exit(1);
   }
 
   if (subcommand === 'dev') {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "freertc",
-  "version": "0.1.19",
+  "version": "0.1.21",
   "description": "Cloudflare Worker signaling relay for WebRTC peers with D1 storage.",
   "keywords": [
     "webrtc",

package/scripts/non-cloudflare-server.mjs

@@ -7,7 +7,7 @@ import { fileURLToPath } from 'node:url';
 import { WebSocketServer } from 'ws';
 
 const PSP_VERSION = '1.0';
-const WORKER_VERSION = '0.1.19';
+const WORKER_VERSION = '0.1.21';
 const DEFAULT_TTL_MS = 30_000;
 const MAX_TTL_MS = 120_000;
 const MAX_MESSAGE_SIZE = 64 * 1024;

package/scripts/postinstall-message.mjs

@@ -52,7 +52,15 @@ async function maybeSelfHealInstall() {
   const projectRoot = process.env.INIT_CWD ? path.resolve(process.env.INIT_CWD) : null;
   if (!projectRoot) return;
   if (projectRoot === PACKAGE_ROOT) return;
-  if (!fs.existsSync(path.join(projectRoot, 'package.json'))) return;
+  const projectPackagePath = path.join(projectRoot, 'package.json');
+  if (!fs.existsSync(projectPackagePath)) return;
+
+  let hostPackage;
+  try {
+    hostPackage = JSON.parse(fs.readFileSync(projectPackagePath, 'utf8'));
+  } catch {
+    return;
+  }
 
   let latestVersion;
   try {
@@ -61,15 +69,30 @@ async function maybeSelfHealInstall() {
     return;
   }
 
-  if (compareVersions(CLI_VERSION, latestVersion) >= 0) {
+  let dependencyRewritten = false;
+  for (const section of ['dependencies', 'devDependencies', 'optionalDependencies']) {
+    const table = hostPackage?.[section];
+    if (table && typeof table === 'object' && Object.prototype.hasOwnProperty.call(table, 'freertc') && table.freertc !== 'latest') {
+      table.freertc = 'latest';
+      dependencyRewritten = true;
+    }
+  }
+
+  if (dependencyRewritten) {
+    fs.writeFileSync(projectPackagePath, `${JSON.stringify(hostPackage, null, 2)}\n`, 'utf8');
+    console.log('freertc postinstall updated host package.json dependency to "latest".');
+  }
+
+  const needsUpgrade = compareVersions(CLI_VERSION, latestVersion) < 0;
+  if (!needsUpgrade && !dependencyRewritten) {
     return;
   }
 
-  console.log(`freertc postinstall detected stale version ${CLI_VERSION}; upgrading host project to ${latestVersion}...`);
+  console.log(`freertc postinstall syncing host project to latest (${latestVersion})...`);
 
   const installResult = spawnSync(
     'npm',
-    ['install', `freertc@${latestVersion}`],
+    ['install'],
     {
       cwd: projectRoot,
       stdio: 'inherit',

package/src/index.js

@@ -1,5 +1,5 @@
 const PSP_VERSION = "1.0";
-const WORKER_VERSION = "0.1.19";
+const WORKER_VERSION = "0.1.21";
 
 const DISCOVERY_TYPES = new Set(["announce", "withdraw", "discover", "peer_list", "redirect"]);
 const NEGOTIATION_TYPES = new Set(["connect_request", "connect_accept", "connect_reject", "offer", "answer", "ice_candidate", "ice_end", "renegotiate"]);