@mycadet/cli 0.13.0-dev.202610181425.b8556ca → 0.13.0-dev.202610181631.3760abb

package/node_modules/@mycadet/environments/src/__tests__/cloud-assistant-hub-url.test.ts

@@ -2,7 +2,9 @@
  * Pins the environment-to-cloud-hub mapping in one place. Both consumers
  * (the Capacitor shell's `server.url` and the CLI remote-web edge's
  * `hubUrl`) read this helper, so this matrix is the single behavioral pin
- * for the served URLs.
+ * for the served URLs. (The remote-web edge additionally folds local → dev
+ * at its own site: clients served through a tunnel cannot reach the local
+ * seed's loopback.)
  */
 
 import { describe, expect, test } from "bun:test";
@@ -10,29 +12,19 @@ import { describe, expect, test } from "bun:test";
 import { cloudAssistantHubUrl, SEEDS } from "../index.js";
 
 describe("cloudAssistantHubUrl", () => {
-  test("cloud environments resolve to their own web origin", () => {
-    expect(cloudAssistantHubUrl("production")).toBe(
-      "https://mycadet.ai/assistant",
-    );
-    expect(cloudAssistantHubUrl("staging")).toBe(
-      "https://staging.mycadet.ai/assistant",
-    );
-  });
-
-  test("every other environment falls back to the dev SPA", () => {
-    for (const name of ["dev", "test", "local", "unknown-env", undefined]) {
+  test("every known environment resolves to its own web origin", () => {
+    for (const name of ["production", "staging", "dev", "test", "local"]) {
       expect(cloudAssistantHubUrl(name)).toBe(
-        "https://dev.mycadet.ai/assistant",
+        `${SEEDS[name].webUrl}/assistant`,
       );
     }
   });
 
-  test("cloud hub URLs derive from the seed web URLs", () => {
-    for (const name of ["production", "staging"]) {
+  test("unknown environments fall back to the dev SPA", () => {
+    for (const name of ["unknown-env", undefined]) {
       expect(cloudAssistantHubUrl(name)).toBe(
-        `${SEEDS[name].webUrl}/assistant`,
+        "https://dev.mycadet.ai/assistant",
       );
     }
-    expect(cloudAssistantHubUrl("local")).toBe(`${SEEDS.dev.webUrl}/assistant`);
   });
 });

package/node_modules/@mycadet/environments/src/seeds.ts

@@ -26,15 +26,11 @@ function portBlock(base: number): PortMap {
 
 /**
  * Base URL of the cloud-hosted assistant SPA (the "hub") for a deployment
- * environment. The Capacitor mobile shell bakes this into native builds as
- * `server.url`, and the CLI's remote-web edge stamps it into the served
- * config as `hubUrl`, so the environment-to-hub mapping lives here exactly
- * once.
- *
- * Only the cloud deployments (production, staging) serve the hosted SPA at
- * their own web origin; every other environment (dev, test, local, unknown)
- * falls back to the dev SPA, whose host serves remote clients that cannot
- * reach a local or per-developer web URL.
+ * environment: always `${SEEDS[env].webUrl}/assistant`. The Capacitor mobile
+ * shell bakes this into native builds as `server.url`, and the CLI's
+ * remote-web edge stamps it into the served config as `hubUrl`, so the
+ * environment-to-hub mapping lives here exactly once. Unknown environment
+ * names fall back to the dev SPA defensively.
  *
  * The `/assistant` suffix is deliberate: booting on the bare host lands on
  * the marketing page, whose CTA redirects to `mycadet.ai/assistant` and
@@ -46,10 +42,7 @@ function portBlock(base: number): PortMap {
  * imports type-only so it stays loadable on its own.
  */
 export function cloudAssistantHubUrl(envName: string | undefined): string {
-  const seed =
-    envName === "production" || envName === "staging"
-      ? SEEDS[envName]
-      : SEEDS.dev;
+  const seed = (envName && SEEDS[envName]) || SEEDS.dev;
   return `${seed.webUrl}/assistant`;
 }
 

package/node_modules/@mycadet/local-mode/node_modules/@mycadet/environments/src/__tests__/cloud-assistant-hub-url.test.ts

@@ -2,7 +2,9 @@
  * Pins the environment-to-cloud-hub mapping in one place. Both consumers
  * (the Capacitor shell's `server.url` and the CLI remote-web edge's
  * `hubUrl`) read this helper, so this matrix is the single behavioral pin
- * for the served URLs.
+ * for the served URLs. (The remote-web edge additionally folds local → dev
+ * at its own site: clients served through a tunnel cannot reach the local
+ * seed's loopback.)
  */
 
 import { describe, expect, test } from "bun:test";
@@ -10,29 +12,19 @@ import { describe, expect, test } from "bun:test";
 import { cloudAssistantHubUrl, SEEDS } from "../index.js";
 
 describe("cloudAssistantHubUrl", () => {
-  test("cloud environments resolve to their own web origin", () => {
-    expect(cloudAssistantHubUrl("production")).toBe(
-      "https://mycadet.ai/assistant",
-    );
-    expect(cloudAssistantHubUrl("staging")).toBe(
-      "https://staging.mycadet.ai/assistant",
-    );
-  });
-
-  test("every other environment falls back to the dev SPA", () => {
-    for (const name of ["dev", "test", "local", "unknown-env", undefined]) {
+  test("every known environment resolves to its own web origin", () => {
+    for (const name of ["production", "staging", "dev", "test", "local"]) {
       expect(cloudAssistantHubUrl(name)).toBe(
-        "https://dev.mycadet.ai/assistant",
+        `${SEEDS[name].webUrl}/assistant`,
       );
     }
   });
 
-  test("cloud hub URLs derive from the seed web URLs", () => {
-    for (const name of ["production", "staging"]) {
+  test("unknown environments fall back to the dev SPA", () => {
+    for (const name of ["unknown-env", undefined]) {
       expect(cloudAssistantHubUrl(name)).toBe(
-        `${SEEDS[name].webUrl}/assistant`,
+        "https://dev.mycadet.ai/assistant",
       );
     }
-    expect(cloudAssistantHubUrl("local")).toBe(`${SEEDS.dev.webUrl}/assistant`);
   });
 });

package/node_modules/@mycadet/local-mode/node_modules/@mycadet/environments/src/seeds.ts

@@ -26,15 +26,11 @@ function portBlock(base: number): PortMap {
 
 /**
  * Base URL of the cloud-hosted assistant SPA (the "hub") for a deployment
- * environment. The Capacitor mobile shell bakes this into native builds as
- * `server.url`, and the CLI's remote-web edge stamps it into the served
- * config as `hubUrl`, so the environment-to-hub mapping lives here exactly
- * once.
- *
- * Only the cloud deployments (production, staging) serve the hosted SPA at
- * their own web origin; every other environment (dev, test, local, unknown)
- * falls back to the dev SPA, whose host serves remote clients that cannot
- * reach a local or per-developer web URL.
+ * environment: always `${SEEDS[env].webUrl}/assistant`. The Capacitor mobile
+ * shell bakes this into native builds as `server.url`, and the CLI's
+ * remote-web edge stamps it into the served config as `hubUrl`, so the
+ * environment-to-hub mapping lives here exactly once. Unknown environment
+ * names fall back to the dev SPA defensively.
  *
  * The `/assistant` suffix is deliberate: booting on the bare host lands on
  * the marketing page, whose CTA redirects to `mycadet.ai/assistant` and
@@ -46,10 +42,7 @@ function portBlock(base: number): PortMap {
  * imports type-only so it stays loadable on its own.
  */
 export function cloudAssistantHubUrl(envName: string | undefined): string {
-  const seed =
-    envName === "production" || envName === "staging"
-      ? SEEDS[envName]
-      : SEEDS.dev;
+  const seed = (envName && SEEDS[envName]) || SEEDS.dev;
   return `${seed.webUrl}/assistant`;
 }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mycadet/cli",
-  "version": "0.13.0-dev.202610181425.b8556ca",
+  "version": "0.13.0-dev.202610181631.3760abb",
   "repository": {
     "type": "git",
     "url": "git+https://github.com/vargasjr-dev/mycadet-temp.git"

package/src/lib/nginx-ingress.ts

@@ -1269,11 +1269,19 @@ export async function startRemoteWebIngress(opts: {
   ) {
     listenPort = recorded.listenPort;
   }
+  // The hub mapping is per-environment, but every client served by THIS edge
+  // is remote (they arrived through the tunnel) and cannot reach a
+  // developer's loopback — so a local environment serves the dev hub, whose
+  // host is reachable from anywhere.
+  const currentEnvName = getCurrentEnvironment().name;
+  const hubUrl = cloudAssistantHubUrl(
+    currentEnvName === "local" ? "dev" : currentEnvName,
+  );
   // The SPA config the served index must carry, computed up front so the
   // reuse decision and the actual spawn share one value and cannot drift.
   const spaOptions = includeWebApp
     ? {
-        hubUrl: cloudAssistantHubUrl(getCurrentEnvironment().name),
+        hubUrl,
         ...(opts.assistantName ? { assistantName: opts.assistantName } : {}),
         ...(opts.assistantId ? { assistantId: opts.assistantId } : {}),
       }