@dooer/dooer-test-env 1.0.1 → 1.0.2

package/lib/compose/generate.js

@@ -275,9 +275,44 @@ function frontendPort(name, index) {
   return name === 'frontend-hq' ? HQ_HOST_PORT : 8081 + index
 }
 
+// {frontend name → published host port}, deterministic over the sorted frontend list.
+function frontendPortMap(frontends) {
+  const map = {}
+  ;(frontends || []).forEach((fe, i) => {
+    map[fe.name] = frontendPort(fe.name, i)
+  })
+  return map
+}
+
+// Cross-frontend base-URL env vars → the frontend that serves them. These are browser-facing links (e.g. HQ's
+// "open frontend-sumify-neue" button), so each must point at that frontend's OWN localhost port — NOT the
+// current app's port. Keyed by env-var NAME (robust: DOOER_XRAYS_BASE_URL's staging value is xray.s-e032.com
+// while the route host is xrays.s-e032.com, so hostname matching would miss it).
+const FRONTEND_URL_ENV = {
+  DOOER_PUBLIC_FRONTEND_BASE_URL: 'frontend-sumify-neue', // go.dooer.com
+  DOOER_SUMIFY_HOST: 'frontend-sumify-neue',
+  DOOER_PUBLIC_WEBSITE_BASE_URL: 'frontend-website',
+  DOOER_TASKS_BASE_URL: 'frontend-tasks',
+  DOOER_XRAYS_BASE_URL: 'frontend-xrays',
+  DOOER_BACKOFFICE_BASE_URL: 'frontend-back-office-neue',
+  DOOER_AI_DATA_STUDIO_BASE_URL: 'frontend-ai-data-studio',
+}
+
+// Repoint every browser-facing URL in `env` at the local ports: the GraphQL facade → gateway, and each
+// cross-frontend base URL → its frontend's own host port. Only rewrites keys already present (from the
+// manifest) whose target frontend has a known port. Applied to services AND frontends.
+function rewriteBrowserUrls(env, portMap) {
+  if ('DOOER_PUBLIC_FACADE_HOST' in env) env.DOOER_PUBLIC_FACADE_HOST = `http://localhost:${GATEWAY_HOST_PORT}`
+  for (const [key, frontendName] of Object.entries(FRONTEND_URL_ENV)) {
+    if (key in env && portMap[frontendName]) env[key] = `http://localhost:${portMap[frontendName]}`
+  }
+  return env
+}
+
 // Turn the parsed manifests into the compose object.
 function buildCompose(services, frontends, { profile } = {}) {
   const wantedSvc = profile ? services.filter((s) => profilesFor(s.name).includes(profile)) : services
+  const portMap = frontendPortMap(frontends)
 
   const composeServices = infraServices()
   wantedSvc.forEach((svc) => {
@@ -289,7 +324,7 @@ function buildCompose(services, frontends, { profile } = {}) {
       // Fall back to ./CMD-OPF.sh when a manifest sets none, so we never hit the image default ./CMD.sh
       // (Consul path) which hangs on consul-template.
       command: svc.command || ['./CMD-OPF.sh'],
-      environment: { ...buildServiceEnv(svc), ...(SERVICE_ENV_OVERRIDES[svc.name] || {}) },
+      environment: rewriteBrowserUrls({ ...buildServiceEnv(svc), ...(SERVICE_ENV_OVERRIDES[svc.name] || {}) }, portMap),
       // publish the GraphQL gateway on the host so the frontends' browsers can reach it at localhost
       ...(svc.name === 'service-graphql' ? { ports: [`${GATEWAY_HOST_PORT}:3000`] } : {}),
       depends_on: ['postgres', 'redis', 'discovery-router'],
@@ -301,16 +336,19 @@ function buildCompose(services, frontends, { profile } = {}) {
   // frontends: served (SSR/static) on :3000, published to a host port. The BROWSER loads them from
   // localhost and calls the GraphQL facade — repointed to the local gateway. Cross-frontend base URLs are
   // repointed to each frontend's own localhost port where we run it (others stay as-is).
-  ;(frontends || []).forEach((fe, i) => {
-    const hostPort = frontendPort(fe.name, i)
+  ;(frontends || []).forEach((fe) => {
+    const hostPort = portMap[fe.name]
     const env = buildServiceEnv(fe)
-    env.DOOER_PUBLIC_FACADE_HOST = `http://localhost:${GATEWAY_HOST_PORT}`
     // dooer-frontend's config validation rejects an EMPTY sentryEndpoint for the `local` environment
     // ("No value specified for [sentryEndpoint] ... (ENV: 'DOOER_SENTRY_ENDPOINT')"), even though staging
     // ships it as "". We don't use Sentry (Jimmy 2026-09-01) — inject a well-formed but inert DSN so the
     // client SDK initialises and any error POST just fails silently against the loopback host.
     env.DOOER_SENTRY_ENDPOINT = 'https://local@localhost/0'
-    if (env.DOOER_PUBLIC_FRONTEND_BASE_URL) env.DOOER_PUBLIC_FRONTEND_BASE_URL = `http://localhost:${hostPort}`
+    // Repoint the facade + every cross-frontend link at the right local ports. Each base URL points at the
+    // frontend that SERVES it (e.g. DOOER_PUBLIC_FRONTEND_BASE_URL → frontend-sumify-neue's port), never at
+    // this app's own port — that was the bug that sent HQ's "open go.dooer.com" button to localhost:8080.
+    if (!('DOOER_PUBLIC_FACADE_HOST' in env)) env.DOOER_PUBLIC_FACADE_HOST = `http://localhost:${GATEWAY_HOST_PORT}`
+    rewriteBrowserUrls(env, portMap)
     composeServices[fe.name] = {
       image: fe.image,
       platform: 'linux/amd64',

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dooer/dooer-test-env",
-  "version": "1.0.1",
+  "version": "1.0.2",
   "description": "Run the whole Dooer backend locally (staging DB minus customers), copy/purge customers between environments, and shred — one CLI.",
   "license": "UNLICENSED",
   "repository": "Dooer/cli-dooer-test-env",