@zerotal/devtools 1.5.0 → 1.6.0

package/CHANGELOG.md

@@ -8,6 +8,23 @@ follows the Zerotal monorepo's unified versioning.
 
 ## [Unreleased]
 
+## [1.5.1] — 2026-08-15
+
+### Fixed
+
+- **DevTools never activated.** The panel was missing from every app, in every mode,
+  however the environment was configured.
+
+  `DevtoolsProvider` gated itself on `isDevSurfaceAllowed(Bun.env["APP_ENV"])`, which fails
+  twice over. `APP_ENV` holds the _runtime mode_ by the time a provider boots — `setAppEnv()`
+  replaced it — so the check was asking whether `"web"` is a development environment. And it
+  was the one dev gate that did not honour `ZT_DEV`, the flag the dev orchestrator sets on
+  the server it supervises, so `zt dev` did not rescue it either.
+
+  It asks `devSurfacesEnabled()` now, which reads the preserved deployment name and honours
+  `ZT_DEV`. Still fails closed: an unset, `staging` or `production` deployment does not get
+  the unauthenticated trace inspector.
+
 ## [1.5.0] — 2026-08-15
 
 ### Changed

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@zerotal/devtools",
-  "version": "1.5.0",
+  "version": "1.6.0",
   "license": "MIT",
   "maturity": "stable",
   "private": false,
@@ -30,11 +30,11 @@
     "typecheck": "tsc --noEmit"
   },
   "dependencies": {
-    "@zerotal/core": "1.5.0"
+    "@zerotal/core": "1.6.0"
   },
   "devDependencies": {
     "typescript": "^5.8.0",
-    "@zerotal/orm": "1.5.0"
+    "@zerotal/orm": "1.6.0"
   },
   "description": "In-browser developer tools for Zerotal — request traces, an inspector panel, and an extensible tab registry.",
   "keywords": [

package/src/provider/DevtoolsProvider.ts

@@ -1,6 +1,6 @@
 import {
   ServiceProvider,
-  isDevSurfaceAllowed,
+  devSurfacesEnabled,
   type AppEnvironment,
   type HttpContext,
 } from "@zerotal/core";
@@ -71,10 +71,16 @@ export class DevtoolsProvider extends ServiceProvider {
   private _stopStream: (() => void) | null = null;
 
   override async onBooting(): Promise<void> {
-    const env = Bun.env["APP_ENV"] ?? "";
-    // Fail closed: only activate for explicitly non-prod envs. An unset or
-    // `staging` APP_ENV must NOT expose the unauthenticated trace inspector.
-    if (!isDevSurfaceAllowed(env)) return;
+    // Fail closed: only activate for explicitly non-prod environments. An unset or
+    // `staging` deployment must NOT expose the unauthenticated trace inspector.
+    //
+    // `devSurfacesEnabled()` rather than reading `APP_ENV` here, for two reasons, and
+    // together they meant devtools never activated at all:
+    //   - by the time a provider boots, `setAppEnv()` has replaced `APP_ENV` with the
+    //     runtime mode, so this was asking whether `"web"` is a development environment;
+    //   - it is the only dev gate that did not honour `ZT_DEV`, which is what the dev
+    //     orchestrator sets on the server it supervises — so `zt dev` did not help either.
+    if (!devSurfacesEnabled()) return;
     this._active = true;
 
     const config = this._config();
@@ -101,13 +107,13 @@ export class DevtoolsProvider extends ServiceProvider {
     // no package's spans ever reached the panel in a real app.
     this.app.container.value("devtools.trace", traceSink);
 
-    this.app.useOnce(DevtoolsInjectionMiddleware as never);
+    this.app.useOnce(DevtoolsInjectionMiddleware);
 
     // Auto-inject the in-page panel into HTML responses via the core dev
     // injector — no `DevTools.start()` needed in the app's own bundle. Also
     // register the injector so this works under a plain `serve` (not only
     // `serve --dev-worker`, where Application.enableDevWs() already adds it).
-    this.app.useOnce(DevReloadMiddleware as never);
+    this.app.useOnce(DevReloadMiddleware);
     registerDevHtmlSnippet("zerotal-devtools", (ctx: HttpContext) =>
       ctx.url.pathname.startsWith("/__zerotal")
         ? "" // don't inject the panel into the devtools' own pages