@ogygia/devtools 0.11.4 → 0.13.0

package/dist/profiler/ui/press.js

@@ -1,3 +1,4 @@
+import { scope } from "@ogygia/machine";
 // src/profiler/ui/press.ts
 /** The keys alone, as an `onkeydown` handler: for an element whose markup already gives it its
 *  role and `tabindex="0"`, so the compiler sees the keyboard path next to the click. */
@@ -13,7 +14,8 @@ const press_key = (e) => {
 };
 const pressable = (node) => {
 	if (node.tabIndex < 0) node.tabIndex = 0;
-	node.addEventListener("keydown", press_key);
-	return () => node.removeEventListener("keydown", press_key);
+	const s = scope();
+	s.on(node, "keydown", press_key);
+	return () => s.close();
 };
 export { press_key, pressable };

package/dist/profiler/ui/row-anchor.svelte.d.ts

@@ -1,9 +1,3 @@
-/**
- * ROW ANCHORS — a finding links to the row it is about (`#comp=<name>`, `#fn=<key>`); the table
- * island opens that row and scrolls to it, on load and on every hash change. Shared by the two
- * tables: give it the anchor kind and a getter for the open state, get back the row id maker and
- * the effect to install.
- */
 export type RowKind = "comp" | "fn" | "island" | "seed";
 export declare function row_id(kind: RowKind, key: string): string;
 /** The `href` a finding uses to reach a row. */

package/dist/profiler/ui/row-anchor.svelte.js

@@ -1,4 +1,11 @@
+import { scope } from "@ogygia/machine";
 // src/profiler/ui/row-anchor.svelte.ts
+/**
+* ROW ANCHORS — a finding links to the row it is about (`#comp=<name>`, `#fn=<key>`); the table
+* island opens that row and scrolls to it, on load and on every hash change. Shared by the two
+* tables: give it the anchor kind and a getter for the open state, get back the row id maker and
+* the effect to install.
+*/
 function row_id(kind, key) {
 	let h = 5381;
 	for (let i = 0; i < key.length; i++) h = (Math.imul(h, 33) ^ key.charCodeAt(i)) >>> 0;
@@ -11,6 +18,7 @@ function row_href(anchor) {
 }
 /** Install in a table island: opens the row named by the URL hash (if it is one of `keys`). */
 function follow_hash(kind, keys, open) {
+	const s = scope();
 	const apply = () => {
 		const m = /^#(comp|fn|island|seed)=(.*)$/.exec(location.hash);
 		if (!m || m[1] !== kind) return;
@@ -22,7 +30,7 @@ function follow_hash(kind, keys, open) {
 		}
 		if (!keys().includes(key)) return;
 		open(key);
-		setTimeout(() => {
+		s.timeout(() => {
 			document.getElementById(row_id(kind, key))?.scrollIntoView({
 				block: "center",
 				behavior: "smooth"
@@ -30,15 +38,12 @@ function follow_hash(kind, keys, open) {
 		}, 30);
 	};
 	apply();
-	window.addEventListener("hashchange", apply);
+	s.on(window, "hashchange", apply);
 	const again = (e) => {
 		const href = (e.target?.closest?.("a"))?.getAttribute("href");
-		if (href && href === location.hash) setTimeout(apply, 0);
-	};
-	document.addEventListener("click", again);
-	return () => {
-		window.removeEventListener("hashchange", apply);
-		document.removeEventListener("click", again);
+		if (href && href === location.hash) s.timeout(apply, 0);
 	};
+	s.on(document, "click", again);
+	return () => s.close();
 }
 export { follow_hash, row_href, row_id };

package/dist/profiler/ui/sort.svelte.d.ts

@@ -1,14 +1,17 @@
 /**
  * Reactive column sort shared by the profiler's tables (was report.ts's TABLE_SORT_JS DOM hack).
- * Give it a getter for the rows and the initial numeric key; it owns the sort state and returns the
+ * Give it a getter for the rows and the initial numeric key; it owns the sort state and holds the
  * sorted view + a click handler. Svelte runes in a `.svelte.ts` module — imported by the table
  * islands, one instance each.
  */
-export interface Sortable<T> {
-  readonly key: string;
-  readonly sorted: T[];
+export declare class Sortable<T> {
+  #private;
+  constructor(rows: () => T[], initial_key: string);
+  get key(): string;
+  get sorted(): T[];
   click(k: string): void;
+  /** The little arrow for a header: ▲/▼ when it's the active column, else empty. */
   arrow(k: string): string;
+  /** The header's `aria-sort`: what a screen reader says the arrow means. */
   aria(k: string): "ascending" | "descending" | "none";
-}
-export declare function sortable<T>(rows: () => T[], initial_key: string): Sortable<T>;
+}

package/dist/profiler/ui/sort.svelte.js

@@ -1,37 +1,43 @@
 // src/profiler/ui/sort.svelte.ts
-function sortable(rows, initial_key) {
-	let key = $state(initial_key);
-	let dir = $state("desc");
-	const sorted = $derived([...rows()].sort((a, b) => {
-		const x = Number(a[key]) || 0;
-		const y = Number(b[key]) || 0;
-		return dir === "asc" ? x - y : y - x;
+/**
+* Reactive column sort shared by the profiler's tables (was report.ts's TABLE_SORT_JS DOM hack).
+* Give it a getter for the rows and the initial numeric key; it owns the sort state and holds the
+* sorted view + a click handler. Svelte runes in a `.svelte.ts` module — imported by the table
+* islands, one instance each.
+*/
+var Sortable = class {
+	#rows;
+	#key = $state("");
+	#dir = $state("desc");
+	#sorted = $derived.by(() => [...this.#rows()].sort((a, b) => {
+		const x = Number(a[this.#key]) || 0;
+		const y = Number(b[this.#key]) || 0;
+		return this.#dir === "asc" ? x - y : y - x;
 	}));
-	function click(k) {
-		if (key === k) dir = dir === "asc" ? "desc" : "asc";
+	constructor(rows, initial_key) {
+		this.#rows = rows;
+		this.#key = initial_key;
+	}
+	get key() {
+		return this.#key;
+	}
+	get sorted() {
+		return this.#sorted;
+	}
+	click(k) {
+		if (this.#key === k) this.#dir = this.#dir === "asc" ? "desc" : "asc";
 		else {
-			key = k;
-			dir = "desc";
+			this.#key = k;
+			this.#dir = "desc";
 		}
 	}
 	/** The little arrow for a header: ▲/▼ when it's the active column, else empty. */
-	function arrow(k) {
-		return key === k ? dir === "asc" ? " ▲" : " ▼" : "";
+	arrow(k) {
+		return this.#key === k ? this.#dir === "asc" ? " ▲" : " ▼" : "";
 	}
 	/** The header's `aria-sort`: what a screen reader says the arrow means. */
-	function aria(k) {
-		return key === k ? dir === "asc" ? "ascending" : "descending" : "none";
+	aria(k) {
+		return this.#key === k ? this.#dir === "asc" ? "ascending" : "descending" : "none";
 	}
-	return {
-		get key() {
-			return key;
-		},
-		get sorted() {
-			return sorted;
-		},
-		click,
-		arrow,
-		aria
-	};
-}
-export { sortable };
+};
+export { Sortable };

package/dist/profiler/visit.d.ts

@@ -195,7 +195,6 @@ export interface Visit {
     as: string;
     crossorigin: string | null;
     vary?: string;
-    hole?: true;
   }[];
   /** the `@font-face` rules behind the fonts it fetched: family, font-display, the fetched files */
   font_faces?: {

package/dist/profiler/visit.js

@@ -311,8 +311,7 @@ function parse_visit(page, raw) {
 			type,
 			bytes,
 			as: str(m.as, 20) ?? "",
-			crossorigin: typeof m.crossorigin === "string" ? m.crossorigin.slice(0, 20) : null,
-			...m.hole === true ? { hole: true } : {}
+			crossorigin: typeof m.crossorigin === "string" ? m.crossorigin.slice(0, 20) : null
 		});
 	}
 	if (preload_misses.length) visit.preload_misses = preload_misses;

package/dist/server.js

@@ -1,4 +1,7 @@
+import { set_devtools_app } from "./app.js";
 import { profiler_demand_skip } from "./profiler/on-demand.js";
+import { OgygiaError } from "./messages/runtime.js";
+import { STALE, flights } from "@ogygia/machine";
 import { defineServer } from "@ogygia/core/plugin";
 import { want_request_stats } from "@ogygia/core/internal/server";
 // src/server.ts
@@ -9,31 +12,45 @@ import { want_request_stats } from "@ogygia/core/internal/server";
 * path — loads on the first request through a dynamic import, so an app pays for it only when it is
 * on; with `onDemand`, only once someone opens the profiler.
 */
-/** the mounted profiler; `undefined` until the first request resolves it, `null` when unavailable */
-let mounted;
-async function mount(options) {
-	if (mounted !== void 0) return mounted;
-	try {
+/** the mounted profiler, once a mount succeeded */
+let mounted = null;
+/** The process's one profiler mount: the first requests that arrive together share it. A mount
+*  that fails (an edge runtime without node:inspector, a dev server mid-edit) is not kept, so the
+*  next request tries again. */
+const mounting = flights.keyed()({
+	name: "profiler mount",
+	key: [],
+	ignore: { options: "one mount for the process: the first request's options are the ones it mounts with" },
+	run: async ({ options }) => {
 		const { profiler } = await import("./profiler/index.js");
 		const { getProfilerStore } = await import("./profiler/storage/index.js").catch(() => ({ getProfilerStore: () => void 0 }));
 		const store = getProfilerStore();
-		mounted = profiler({
+		return mounted = profiler({
 			...options,
 			...store ? { store } : {}
 		});
-	} catch {
-		mounted = null;
 	}
-	return mounted;
+});
+/** a failed mount was said (once for the process: an edge runtime fails every request the same way) */
+let mount_failure_said = false;
+async function mount(options) {
+	return mounting.get({ options }).then((m) => m === STALE ? null : m, (error) => {
+		if (!mount_failure_said) {
+			mount_failure_said = true;
+			console.error(new OgygiaError("devtools_profiler_mount_failed"), error);
+		}
+		return null;
+	});
 }
 const server = defineServer({
 	setup(app) {
+		set_devtools_app(app);
 		if (app.options.profiler) want_request_stats();
 	},
 	async handle({ event, options }, next) {
 		const config = options.profiler;
 		if (!config) return next();
-		if (profiler_demand_skip(!!config.onDemand, mounted !== void 0, event.url.pathname, config.path || "/__profiler")) return next();
+		if (profiler_demand_skip(!!config.onDemand, mounted !== null, event.url.pathname, config.path || "/__profiler")) return next();
 		const prof = await mount(config);
 		if (!prof) return next();
 		return prof({

package/dist/vite.js

@@ -1,3 +1,4 @@
+import { warn } from "./messages/build.js";
 import fs from "node:fs";
 import path from "node:path";
 import { fileURLToPath } from "node:url";
@@ -58,8 +59,8 @@ function devtools_vite(dock, profiler) {
 		config(_config, env) {
 			dev = env.command === "serve";
 			if (env.command === "build" && isMainThread) {
-				if (dock) console.warn("[ogygia] devtools is ON for this build: its pages carry the devtools launcher (the dock loads only when opened). Keep it off for production: list `devtools()` only when `mode !== 'production'`.");
-				if (profiler && "secret" in profiler && !profiler.secret) console.warn("[ogygia] profiler: `profiler.secret` is EMPTY in this build — the variable it reads is not set in the BUILD environment. The UI will be DISABLED in production unless OGYGIA_PROFILER_SECRET exists at runtime (on Amplify / Lambda it does not): /__profiler will answer \"installed but disabled\". Set the variable where the build runs, and keep the `profiler` option unconditional.");
+				if (dock) warn("devtools_on_in_build");
+				if (profiler && "secret" in profiler && !profiler.secret) warn("devtools_profiler_secret_empty");
 			}
 			const out = {};
 			if (dock && env.command === "serve") out.optimizeDeps = { include: ["@ogygia/devtools > @neodrag/svelte", "@ogygia/devtools > @neodrag/svelte/resize"] };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ogygia/devtools",
-  "version": "0.11.4",
+  "version": "0.13.0",
   "description": "Devtools for ogygia: the in-page dock, and the SSR profiler with its report, client marks and storage adapters",
   "keywords": [
     "devtools",
@@ -22,6 +22,9 @@
     "README.md"
   ],
   "type": "module",
+  "sideEffects": [
+    "**/*.css"
+  ],
   "exports": {
     ".": {
       "types": "./dist/devtools/index.d.ts",
@@ -59,6 +62,10 @@
     "./profiler/storage/postgres": {
       "types": "./dist/profiler/storage/postgres.d.ts",
       "default": "./dist/profiler/storage/postgres.js"
+    },
+    "./internal/messages": {
+      "types": "./dist/messages/index.d.ts",
+      "default": "./dist/messages/index.js"
     }
   },
   "publishConfig": {
@@ -66,10 +73,15 @@
   },
   "dependencies": {
     "@neodrag/svelte": "3.0.0-next.12",
-    "devalue": "^5.9.3",
-    "@ogygia/router": "^0.11.0"
+    "@ogygia/machine": "^0.13.0",
+    "@ogygia/router": "^0.13.0",
+    "devalue": "^5.9.3"
   },
   "devDependencies": {
+    "@ogygia/config": "0.0.0",
+    "@ogygia/core": "0.13.0",
+    "@ogygia/tooling": "0.0.0",
+    "@ogygia/tsdown-plugin-svelte": "1.1.1",
     "@playwright/test": "1.62.1",
     "@sveltejs/kit": "^2.70.2",
     "@sveltejs/vite-plugin-svelte": "^7.2.0",
@@ -80,16 +92,13 @@
     "tsdown": "^0.23.4",
     "typescript": "^6.0.3",
     "vite": "^8.3.5",
-    "vitest": "^4.1.12",
-    "@ogygia/core": "0.11.4",
-    "@ogygia/tooling": "0.0.0",
-    "@ogygia/tsdown-plugin-svelte": "1.1.0"
+    "vitest": "^4.1.12"
   },
   "peerDependencies": {
+    "@ogygia/core": "^0.13.0",
     "@sveltejs/kit": ">=2.70.2 <4",
     "svelte": "^5.56.8",
-    "vite": "^8.2.4",
-    "@ogygia/core": "^0.11.4"
+    "vite": "^8.2.4"
   },
   "scripts": {
     "build": "tsdown",