@leavittsoftware/web 11.3.0 → 11.4.1

package/CLAUDE.md

@@ -31,6 +31,8 @@ When bumping `@leavittsoftware/web` in a downstream project, read every entry **
 - `AppRoute` / `PageRoute` / `RedirectRoute` imports from `titanium/helpers/route`
 - App-shell `#routes` tables and `before` handlers on page routes
 - `'page' in route` / `'redirect' in route` discriminators used to collect or execute matches
+- **`window.navigation.navigate(`** anywhere inside `#route`, a route `before` handler, or a home-routing helper — this is the renderer-crash pattern below
+- **`event.canIntercept`** / `event.navigationType === 'reload'` guard bodies hand-rolled in `#onNavigate`
 - `md-menu-item[inert]` style overrides / `--md-menu-item-leading-icon-color` on single-select subclasses
 
 **Adopt when upgrading to this version:**
@@ -38,6 +40,28 @@ When bumping `@leavittsoftware/web` in a downstream project, read every entry **
 - **`titanium-multi-select-base`** — new generic multi select combobox: selected items render as chips inside the field and the search input lives in the popover above the selectable items. Replaces the `titanium-chip-multi-select` + separate single-select + add-button pattern for "pick several from a list" inputs. Use it directly with `.items` (local fuse.js search) and optional `.sections` grouping, or extend it and override `onInputChanged` / `showSuggestions` for remote search — same hook names as `titanium-single-select-base`. Fires `selected-changed`; read the new value from `event.target.selected`.
 - **Middleware + halt pipeline** — `AppRoute` now includes `MiddlewareRoute` (`pattern` + required `before`, no `page`/`redirect`). Matching destinations run **all** matching middleware (and page `before` handlers) in declaration order; handlers continue by default and may return `ROUTE_HALT` / `'halt'` (`RouteHandlerResult`) to stop the pipeline without rendering. Rework app routers from “first match wins immediately” to the two-pass collect-then-execute pattern (see skeleton.leavitt.com `develop` / leavittbook `my-app.ts`). Prefer importing `ROUTE_HALT` from `titanium/helpers/route` instead of the string literal.
 - **`titanium-single-select-base` shaped menu tokens** — shaped open menus now set `--md-menu-item-leading-icon-color` / `--md-menu-item-trailing-icon-color` to `inverse-on-surface` (alongside the existing label/supporting-text tokens). Inert section-header `md-menu-item` styling (spacing + shaped inverse colors) lives on the base — remove duplicate `md-menu-item[inert]` / leading-icon color overrides from subclasses once on this build.
+- **`getInterceptableUrl` + `resolveRoute` — required for any app with `redirect` routes.** Resolving a redirect with `window.navigation.navigate()` **crashes the renderer process**. A navigation started during boot, or from inside an intercept handler, produces a `NavigateEvent` whose handler settles once it is no longer the ongoing event, tripping `CHECK_EQ(this, window->navigation()->ongoing_navigate_event_)` in Blink's `NavigateEvent::ReactDone` (`navigate_event.cc`). The tab dies with `STATUS_BREAKPOINT`; it is not catchable from JS. Confirmed on Chrome 151 and the two prior stable builds.
+
+  `resolveRoute(routes, url)` follows redirects by rewriting the address bar with `history.replaceState` instead, and returns `{ url, matches }` where `matches` contains only middleware and page routes for the final URL. `getInterceptableUrl(event)` returns the destination URL to intercept or `null`, and encapsulates the intercept guards. The two coordinate through module-private state so the router does not intercept its own rewrite — `replaceState` dispatches a `navigate` event, and intercepting it starts a second, racing routing pass that reaches the same CHECK. **Use both together; do not hand-roll either half.**
+
+  ```ts
+  #onNavigate = (event: NavigateEvent) => {
+    const url = getInterceptableUrl(event);
+    if (!url) return;
+    event.intercept({ handler: () => this.#route(url) });
+  };
+
+  async #route(requestedUrl: URL) {
+    const { url, matches } = resolveRoute(this.#routes, requestedUrl);
+    if (!matches.some(({ route }) => 'page' in route)) return this.#showErrorPage();
+    for (const { route, params } of matches) {
+      if ((await route.before?.(params, url)) === ROUTE_HALT) return;
+      if ('page' in route) return this.#changePage(route.page, route.import);
+    }
+  }
+  ```
+
+  **Behaviour note:** a redirect means the intermediate URL is not real, so middleware matching it no longer runs — only the final URL's routes execute. An unresolvable redirect cycle (10 hops) returns no matches, which callers should treat as a 404.
 
 **Renamed / API changes:**
 
@@ -45,6 +69,7 @@ When bumping `@leavittsoftware/web` in a downstream project, read every entry **
 - `AppRoute` = `MiddlewareRoute | PageRoute | RedirectRoute`
 - `PageRoute.before` return type is now `RouteHandlerResult | Promise<RouteHandlerResult>` (may return `ROUTE_HALT` / `'halt'`)
 - New exports: `MiddlewareRoute`, `RouteHandlerResult`, `ROUTE_HALT`
+- New exports from `titanium/helpers/route`: `getInterceptableUrl`, `resolveRoute`, and types `RouteMatch<T>`, `ResolvedRoute`
 
 ### 10.0.0
 

package/leavitt/app/app-main-content-container.js

@@ -44,6 +44,14 @@ let LeavittAppContentContainer = class LeavittAppContentContainer extends LitEle
         scrollbar-color: var(--md-sys-color-surface-container-highest) transparent;
         border-radius: 28px;
       }
+
+      /* The host squares off in drawer mode, so the scroller has to as well — its
+         rounded overflow clip is otherwise visible at the page's top corners once
+         there is enough content to scroll. Safari does not like nested selectors on
+         host, so this stays flat. */
+      :host([main-menu-position='drawer']) scroll-container {
+        border-radius: 0;
+      }
     `,
     ]; }
     render() {

package/leavitt/app/app-main-content-container.js.map

@@ -1 +1 @@
-{"version":3,"file":"app-main-content-container.js","sourceRoot":"","sources":["app-main-content-container.ts"],"names":[],"mappings":";AAAA,OAAO,kEAAkE,CAAC;AAE1E,OAAO,EAAE,UAAU,EAAE,GAAG,EAAE,IAAI,EAAE,MAAM,KAAK,CAAC;AAC5C,OAAO,EAAE,aAAa,EAAE,QAAQ,EAAE,KAAK,EAAE,MAAM,mBAAmB,CAAC;AACnE,OAAO,EAAE,uBAAuB,EAAE,MAAM,uCAAuC,CAAC;AAChF,OAAO,EAAE,OAAO,EAAE,MAAM,cAAc,CAAC;AAGhC,IAAM,0BAA0B,GAAhC,MAAM,0BAA2B,SAAQ,UAAU;IAAnD;;QAEwC,6CAAsC,IAAI,CAAC;QAGjF,qBAAgB,GAAW,MAAM,CAAC;IAuC3C,CAAC;IA3CqC,kCAAwC;IAAxC,IAAA,eAAe,qDAAyB;IAAxC,IAAA,eAAe,2DAAyB;IAC/B,sCAA2C;IAA3C,IAAA,mBAAmB,yDAAwB;IAA3C,IAAA,mBAAmB,+DAAwB;aAKjF,WAAM,GAAG;QACd,GAAG,CAAA;;;;;;;;;;;;;;;;;;;;;;;;;;;KA2BF;KACF,AA7BY,CA6BX;IAEF,MAAM;QACJ,OAAO,IAAI,CAAA;yFAC0E,IAAI,CAAC,mBAAmB;;KAE5G,CAAC;IACJ,CAAC;;AA1CmC;IAAnC,KAAK,CAAC,kBAAkB,CAAC;iEAAkD;AAC/B;IAA5C,QAAQ,CAAC,EAAE,IAAI,EAAE,MAAM,EAAE,CAAC;qEAA6D;AAGjF;IAFN,OAAO,CAAC,EAAE,OAAO,EAAE,uBAAuB,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC;IAC9D,QAAQ,CAAC,EAAE,IAAI,EAAE,MAAM,EAAE,OAAO,EAAE,IAAI,EAAE,SAAS,EAAE,oBAAoB,EAAE,CAAC;oEAClC;AAL9B,0BAA0B;IADtC,aAAa,CAAC,oCAAoC,CAAC;GACvC,0BAA0B,CA4CtC"}
+{"version":3,"file":"app-main-content-container.js","sourceRoot":"","sources":["app-main-content-container.ts"],"names":[],"mappings":";AAAA,OAAO,kEAAkE,CAAC;AAE1E,OAAO,EAAE,UAAU,EAAE,GAAG,EAAE,IAAI,EAAE,MAAM,KAAK,CAAC;AAC5C,OAAO,EAAE,aAAa,EAAE,QAAQ,EAAE,KAAK,EAAE,MAAM,mBAAmB,CAAC;AACnE,OAAO,EAAE,uBAAuB,EAAE,MAAM,uCAAuC,CAAC;AAChF,OAAO,EAAE,OAAO,EAAE,MAAM,cAAc,CAAC;AAGhC,IAAM,0BAA0B,GAAhC,MAAM,0BAA2B,SAAQ,UAAU;IAAnD;;QAEwC,6CAAsC,IAAI,CAAC;QAGjF,qBAAgB,GAAW,MAAM,CAAC;IA+C3C,CAAC;IAnDqC,kCAAwC;IAAxC,IAAA,eAAe,qDAAyB;IAAxC,IAAA,eAAe,2DAAyB;IAC/B,sCAA2C;IAA3C,IAAA,mBAAmB,yDAAwB;IAA3C,IAAA,mBAAmB,+DAAwB;aAKjF,WAAM,GAAG;QACd,GAAG,CAAA;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;KAmCF;KACF,AArCY,CAqCX;IAEF,MAAM;QACJ,OAAO,IAAI,CAAA;yFAC0E,IAAI,CAAC,mBAAmB;;KAE5G,CAAC;IACJ,CAAC;;AAlDmC;IAAnC,KAAK,CAAC,kBAAkB,CAAC;iEAAkD;AAC/B;IAA5C,QAAQ,CAAC,EAAE,IAAI,EAAE,MAAM,EAAE,CAAC;qEAA6D;AAGjF;IAFN,OAAO,CAAC,EAAE,OAAO,EAAE,uBAAuB,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC;IAC9D,QAAQ,CAAC,EAAE,IAAI,EAAE,MAAM,EAAE,OAAO,EAAE,IAAI,EAAE,SAAS,EAAE,oBAAoB,EAAE,CAAC;oEAClC;AAL9B,0BAA0B;IADtC,aAAa,CAAC,oCAAoC,CAAC;GACvC,0BAA0B,CAoDtC"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@leavittsoftware/web",
-  "version": "11.3.0",
+  "version": "11.4.1",
   "license": "Apache-2.0",
   "description": "",
   "files": [
@@ -42,5 +42,5 @@
     "url": "https://github.com/LeavittSoftware/titanium-elements/issues"
   },
   "homepage": "https://github.com/LeavittSoftware/titanium-elements/#readme",
-  "gitHead": "ab05ee189c2c85baa247e7d5460e74ef42ce1441"
+  "gitHead": "dc6b74566e849d9501a0212dee9c59a29b2ac56c"
 }

package/titanium/helpers/route.d.ts

@@ -26,4 +26,44 @@ export type RedirectRoute = {
     redirect: string | ((params: Record<string, string>) => string);
 };
 export type AppRoute = MiddlewareRoute | PageRoute | RedirectRoute;
+/** A matched route together with the params pulled out of its pattern. */
+export type RouteMatch<T extends AppRoute = AppRoute> = {
+    route: T;
+    params: Record<string, string>;
+};
+/** The URL to route, after redirects, and the routes to execute for it. */
+export type ResolvedRoute = {
+    url: URL;
+    matches: RouteMatch<MiddlewareRoute | PageRoute>[];
+};
+/**
+ * The destination URL when the app router should take this navigation over, or `null`
+ * to leave it to the browser.
+ *
+ * Use in a `navigate` listener:
+ *
+ * ```ts
+ * const url = getInterceptableUrl(event);
+ * if (!url) return;
+ * event.intercept({ handler: () => this.#route(url) });
+ * ```
+ */
+export declare function getInterceptableUrl(event: NavigateEvent): URL | null;
+/**
+ * Follows any redirect routes for `url` by rewriting the address bar, then returns the
+ * final URL along with the routes to execute for it — matching middleware in declaration
+ * order, ending at the first page route. Redirect routes are never returned; they are
+ * already resolved.
+ *
+ * Redirects deliberately do **not** call `navigation.navigate()`. A navigation started
+ * during boot, or from inside an intercept handler, produces a `NavigateEvent` whose
+ * handler settles once it is no longer the ongoing event, tripping
+ * `CHECK_EQ(this, window->navigation()->ongoing_navigate_event_)` in Blink's
+ * `NavigateEvent::ReactDone` — which kills the renderer process, not just the navigation.
+ *
+ * Because a redirect means "this URL is not real", middleware matching an intermediate
+ * URL does not run; only the final URL's routes are returned. An unresolvable redirect
+ * cycle yields no matches, which callers should treat as a 404.
+ */
+export declare function resolveRoute(routes: AppRoute[], url: URL): ResolvedRoute;
 //# sourceMappingURL=route.d.ts.map

package/titanium/helpers/route.js

@@ -3,4 +3,91 @@
  * Prefer this constant over the `'halt'` string when writing `before` handlers.
  */
 export const ROUTE_HALT = 'halt';
+/** Guards against a redirect cycle spinning the tab forever. */
+const MAX_REDIRECT_HOPS = 10;
+/**
+ * Set only while `resolveRoute` rewrites the address bar, so `getInterceptableUrl`
+ * can tell the router's own rewrite apart from a real navigation. Module-private on
+ * purpose: the two halves of this must never be separated by a consumer.
+ */
+let isRewritingUrl = false;
+/**
+ * The destination URL when the app router should take this navigation over, or `null`
+ * to leave it to the browser.
+ *
+ * Use in a `navigate` listener:
+ *
+ * ```ts
+ * const url = getInterceptableUrl(event);
+ * if (!url) return;
+ * event.intercept({ handler: () => this.#route(url) });
+ * ```
+ */
+export function getInterceptableUrl(event) {
+    // A redirect rewrite from resolveRoute — whoever triggered it is already routing.
+    if (isRewritingUrl) {
+        return null;
+    }
+    if (!event.canIntercept || event.hashChange || event.downloadRequest !== null) {
+        return null;
+    }
+    // Dev-server live reload and user refresh must fetch fresh modules — don't SPA-handle reloads.
+    if (event.navigationType === 'reload') {
+        return null;
+    }
+    const url = new URL(event.destination.url);
+    return url.origin === location.origin ? url : null;
+}
+/**
+ * Follows any redirect routes for `url` by rewriting the address bar, then returns the
+ * final URL along with the routes to execute for it — matching middleware in declaration
+ * order, ending at the first page route. Redirect routes are never returned; they are
+ * already resolved.
+ *
+ * Redirects deliberately do **not** call `navigation.navigate()`. A navigation started
+ * during boot, or from inside an intercept handler, produces a `NavigateEvent` whose
+ * handler settles once it is no longer the ongoing event, tripping
+ * `CHECK_EQ(this, window->navigation()->ongoing_navigate_event_)` in Blink's
+ * `NavigateEvent::ReactDone` — which kills the renderer process, not just the navigation.
+ *
+ * Because a redirect means "this URL is not real", middleware matching an intermediate
+ * URL does not run; only the final URL's routes are returned. An unresolvable redirect
+ * cycle yields no matches, which callers should treat as a 404.
+ */
+export function resolveRoute(routes, url) {
+    for (let hop = 0; hop <= MAX_REDIRECT_HOPS; hop++) {
+        const matches = [];
+        let redirect = null;
+        for (const route of routes) {
+            const match = route.pattern.exec({ pathname: url.pathname });
+            if (!match) {
+                continue;
+            }
+            const params = match.pathname.groups;
+            if ('redirect' in route) {
+                redirect = typeof route.redirect === 'function' ? route.redirect(params) : route.redirect;
+                break; // terminal
+            }
+            matches.push({ route, params });
+            if ('page' in route) {
+                break; // terminal — first matching page wins
+            }
+            // middleware entry — keep collecting
+        }
+        if (redirect === null) {
+            return { url, matches };
+        }
+        url = new URL(redirect, location.origin);
+        // replaceState dispatches a navigate event synchronously; the flag keeps the router
+        // from intercepting its own rewrite and starting a second, racing routing pass.
+        isRewritingUrl = true;
+        try {
+            history.replaceState({}, '', url.pathname + url.search + url.hash);
+        }
+        finally {
+            isRewritingUrl = false;
+        }
+    }
+    return { url, matches: [] };
+}
 //# sourceMappingURL=route.js.map

package/titanium/helpers/route.js.map

@@ -1 +1 @@
-{"version":3,"file":"route.js","sourceRoot":"","sources":["route.ts"],"names":[],"mappings":"AAAA;;;GAGG;AACH,MAAM,CAAC,MAAM,UAAU,GAAG,MAAe,CAAC"}
+{"version":3,"file":"route.js","sourceRoot":"","sources":["route.ts"],"names":[],"mappings":"AAAA;;;GAGG;AACH,MAAM,CAAC,MAAM,UAAU,GAAG,MAAe,CAAC;AA0C1C,gEAAgE;AAChE,MAAM,iBAAiB,GAAG,EAAE,CAAC;AAE7B;;;;GAIG;AACH,IAAI,cAAc,GAAG,KAAK,CAAC;AAE3B;;;;;;;;;;;GAWG;AACH,MAAM,UAAU,mBAAmB,CAAC,KAAoB;IACtD,kFAAkF;IAClF,IAAI,cAAc,EAAE,CAAC;QACnB,OAAO,IAAI,CAAC;IACd,CAAC;IACD,IAAI,CAAC,KAAK,CAAC,YAAY,IAAI,KAAK,CAAC,UAAU,IAAI,KAAK,CAAC,eAAe,KAAK,IAAI,EAAE,CAAC;QAC9E,OAAO,IAAI,CAAC;IACd,CAAC;IACD,+FAA+F;IAC/F,IAAI,KAAK,CAAC,cAAc,KAAK,QAAQ,EAAE,CAAC;QACtC,OAAO,IAAI,CAAC;IACd,CAAC;IACD,MAAM,GAAG,GAAG,IAAI,GAAG,CAAC,KAAK,CAAC,WAAW,CAAC,GAAG,CAAC,CAAC;IAC3C,OAAO,GAAG,CAAC,MAAM,KAAK,QAAQ,CAAC,MAAM,CAAC,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,IAAI,CAAC;AACrD,CAAC;AAED;;;;;;;;;;;;;;;GAeG;AACH,MAAM,UAAU,YAAY,CAAC,MAAkB,EAAE,GAAQ;IACvD,KAAK,IAAI,GAAG,GAAG,CAAC,EAAE,GAAG,IAAI,iBAAiB,EAAE,GAAG,EAAE,EAAE,CAAC;QAClD,MAAM,OAAO,GAA8C,EAAE,CAAC;QAC9D,IAAI,QAAQ,GAAkB,IAAI,CAAC;QAEnC,KAAK,MAAM,KAAK,IAAI,MAAM,EAAE,CAAC;YAC3B,MAAM,KAAK,GAAG,KAAK,CAAC,OAAO,CAAC,IAAI,CAAC,EAAE,QAAQ,EAAE,GAAG,CAAC,QAAQ,EAAE,CAAC,CAAC;YAC7D,IAAI,CAAC,KAAK,EAAE,CAAC;gBACX,SAAS;YACX,CAAC;YACD,MAAM,MAAM,GAAG,KAAK,CAAC,QAAQ,CAAC,MAAgC,CAAC;YAC/D,IAAI,UAAU,IAAI,KAAK,EAAE,CAAC;gBACxB,QAAQ,GAAG,OAAO,KAAK,CAAC,QAAQ,KAAK,UAAU,CAAC,CAAC,CAAC,KAAK,CAAC,QAAQ,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,KAAK,CAAC,QAAQ,CAAC;gBAC1F,MAAM,CAAC,WAAW;YACpB,CAAC;YACD,OAAO,CAAC,IAAI,CAAC,EAAE,KAAK,EAAE,MAAM,EAAE,CAAC,CAAC;YAChC,IAAI,MAAM,IAAI,KAAK,EAAE,CAAC;gBACpB,MAAM,CAAC,sCAAsC;YAC/C,CAAC;YACD,qCAAqC;QACvC,CAAC;QAED,IAAI,QAAQ,KAAK,IAAI,EAAE,CAAC;YACtB,OAAO,EAAE,GAAG,EAAE,OAAO,EAAE,CAAC;QAC1B,CAAC;QAED,GAAG,GAAG,IAAI,GAAG,CAAC,QAAQ,EAAE,QAAQ,CAAC,MAAM,CAAC,CAAC;QACzC,oFAAoF;QACpF,gFAAgF;QAChF,cAAc,GAAG,IAAI,CAAC;QACtB,IAAI,CAAC;YACH,OAAO,CAAC,YAAY,CAAC,EAAE,EAAE,EAAE,EAAE,GAAG,CAAC,QAAQ,GAAG,GAAG,CAAC,MAAM,GAAG,GAAG,CAAC,IAAI,CAAC,CAAC;QACrE,CAAC;gBAAS,CAAC;YACT,cAAc,GAAG,KAAK,CAAC;QACzB,CAAC;IACH,CAAC;IAED,OAAO,EAAE,GAAG,EAAE,OAAO,EAAE,EAAE,EAAE,CAAC;AAC9B,CAAC"}