@rubytech/create-maxy 1.0.784 → 1.0.786

package/dist/index.js

@@ -1358,6 +1358,19 @@ function buildPlatform() {
     // server/package.json but NOT shipped as pre-built node_modules — npm pack silently
     // strips files from nested node_modules (e.g. rxjs/package.json), breaking require().
     // Install fresh on device to guarantee a complete dependency tree.
+    //
+    // On upgrade, wipe `node_modules` first so npm extracts a clean tree. Without
+    // this, an interrupted previous install (network blip, operator cancellation,
+    // power loss) can leave nested package.json files half-truncated — the most
+    // common manifestation is `Error: Invalid package config .../rxjs/package.json`
+    // at server startup, which loops the brand service indefinitely. The wipe
+    // adds ~30 s to upgrades but eliminates a class of unrecoverable customer
+    // states; reliability wins over speed for a one-shot install path.
+    const serverNodeModules = join(INSTALL_DIR, "server", "node_modules");
+    if (existsSync(serverNodeModules)) {
+        console.log("  Wiping previous server/node_modules for a clean reinstall...");
+        rmSync(serverNodeModules, { recursive: true, force: true });
+    }
     console.log(`  Installing server dependencies (${join(INSTALL_DIR, "server")})...`);
     shellRetry("npm", ["install", "--omit=dev", ...NPM_NET_FLAGS], { cwd: join(INSTALL_DIR, "server") }, 3, 15);
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@rubytech/create-maxy",
-  "version": "1.0.784",
+  "version": "1.0.786",
   "description": "Install Maxy — AI for Productive People",
   "bin": {
     "create-maxy": "./dist/index.js"

package/payload/platform/scripts/wifi-provision-server/server.js

@@ -292,12 +292,10 @@ function portalHTML() {
       <div class="success-icon">&#10003;</div>
       <h1>Connected!</h1>
       <p class="hint">${escapedBrandName} is now online. Redirecting in <span id="redirect-countdown">15</span>s…</p>
-      <div class="address-box">
-        <a id="device-link" href="#"></a>
-      </div>
-      <p class="hint" id="manual-hint" style="display:none">If your phone doesn't redirect automatically, open this in your normal browser:</p>
-      <div class="address-box" id="manual-box" style="display:none">
-        <span id="device-link-text" style="user-select:all"></span>
+      <a id="open-btn" href="#" class="btn" style="text-decoration:none;display:flex;align-items:center;justify-content:center">Open ${escapedBrandName} →</a>
+      <p class="hint" style="margin-top:16px">Or copy this URL into your browser:</p>
+      <div class="address-box" style="user-select:all" id="manual-box">
+        <span id="device-link-text"></span>
       </div>
       <p class="hint">This access point will close shortly.<br>Your phone will reconnect to your WiFi automatically.</p>
     </div>
@@ -421,21 +419,11 @@ function portalHTML() {
     var ipAddr = data.ip ? "http://" + data.ip + devicePort : null;
     // Auto-navigate target: prefer the IP because mDNS .local resolution
     // is flaky on Android (notably Brave) and the redirect dead-ends.
-    // The displayed link uses the same URL so what the user sees is what
-    // they get if they tap.
     var redirectAddr = ipAddr || hostnameAddr;
-    var link = document.getElementById("device-link");
-    link.href = redirectAddr;
-    link.textContent = redirectAddr;
-    // Always surface a manual fallback line — captive webviews on some
-    // phone OSes silently swallow JS navigation, and the user needs to
-    // be able to copy-paste the URL into a real browser.
+    var openBtn = document.getElementById("open-btn");
+    if (openBtn) openBtn.href = redirectAddr;
     var manualText = document.getElementById("device-link-text");
-    if (manualText) {
-      manualText.textContent = redirectAddr;
-      document.getElementById("manual-hint").style.display = "block";
-      document.getElementById("manual-box").style.display = "block";
-    }
+    if (manualText) manualText.textContent = redirectAddr;
     showScreen("success-screen");
     // 15s countdown: the Pi keeps the AP up for ~10s after writing the
     // success result so this poll can land, then tears it down; the

package/payload/platform/scripts/wifi-provision.sh

@@ -40,15 +40,21 @@ PLATFORM_ROOT="${MAXY_PLATFORM_ROOT:-$(dirname "$SCRIPT_DIR")}"
 BRAND_JSON="${PLATFORM_ROOT}/config/brand.json"
 CONFIG_DIR=".maxy"
 PRODUCT_NAME="Maxy"
-HOSTNAME_NAME="maxy"
 if [ -f "$BRAND_JSON" ] && command -v jq >/dev/null 2>&1; then
   _dir=$(jq -r '.configDir // empty' "$BRAND_JSON" 2>/dev/null) || true
   [ -n "$_dir" ] && CONFIG_DIR="$_dir"
   _name=$(jq -r '.productName // empty' "$BRAND_JSON" 2>/dev/null) || true
   [ -n "$_name" ] && PRODUCT_NAME="$_name"
-  _host=$(jq -r '.hostname // empty' "$BRAND_JSON" 2>/dev/null) || true
-  [ -n "$_host" ] && HOSTNAME_NAME="$_host"
 fi
+# HOSTNAME_NAME must be the **actual system hostname** (the name Avahi
+# advertises and the post-connect URL must use), not the brand default
+# from brand.json. The operator's --hostname flag rewrites the system
+# hostname but never touches brand.json's hostname field, so reading
+# from the brand mis-routed the captive portal's success URL to a name
+# that doesn't resolve. `hostname -s` is the canonical answer Avahi
+# itself follows when avahi-daemon.conf has `host-name` left empty,
+# so this is the same source of truth.
+HOSTNAME_NAME="$(hostname -s 2>/dev/null || hostname 2>/dev/null || echo maxy)"
 
 # Determine the home directory of the installing user. The service runs as
 # root, but logs and config belong to the user who installed the platform.