@kolkrabbi/kol-shell 0.37.0 → 0.37.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@kolkrabbi/kol-shell",
-  "version": "0.37.0",
+  "version": "0.37.1",
   "private": false,
   "description": "KOL application shell — fixed 48px NavRail + AppShell layout root, PageShell/PageHeader scaffolds, ContentFilters catalog organism, GridCard, SettingsScaffold, WalkthroughPanel, ShortcutsOverlay. App chrome (kol-framework owns site chrome). Nav items, content, shortcuts and settings are consumer-injected. Sits above @kolkrabbi/kol-{theme,component,framework}.",
   "license": "MIT",
@@ -20,9 +20,9 @@
     "react-dom": "^18.3.0 || ^19.0.0"
   },
   "devDependencies": {
-    "@kolkrabbi/kol-component": "^0.154.1",
-    "@kolkrabbi/kol-icons": "^0.25.0",
     "@kolkrabbi/kol-framework": "^0.36.0",
+    "@kolkrabbi/kol-component": "^0.156.0",
+    "@kolkrabbi/kol-icons": "^0.25.0",
     "@kolkrabbi/kol-theme": "^0.122.0"
   },
   "files": [

package/src/AppShell.jsx

@@ -99,6 +99,13 @@ const CODE_FOR_KEY = {
   '-': 'Minus', '=': 'Equal',
 }
 
+/* A STABLE DEFAULT (ShellDrawerOpenOnUnstableDep, kol-mirror 2026-09-01 —
+ * found by the user on his phone): `drawerOpenOn = []` in the signature was a
+ * fresh array every render, and it sat in an effect's deps, so the route effect
+ * re-ran on every render and closed the drawer straight after every tap. The
+ * trigger did nothing on every consumer taking the default. */
+const NO_PATHS = []
+
 export default function AppShell({
   items,
   bottomItems,
@@ -115,7 +122,7 @@ export default function AppShell({
   settingsPath,
   settingsKey,
   drawerBelow = 768,
-  drawerOpenOn = [],
+  drawerOpenOn = NO_PATHS,
   children,
 }) {
   const [navHidden, setNavHidden] = useState(false)
@@ -205,10 +212,14 @@ export default function AppShell({
   /* …unless the destination is one where nav IS the page (`drawerOpenOn`):
    * there the entry OPENS it. Runs on mount, on the fold and on navigation
    * alike — `drawer` is a dep — so a phone arriving on home gets the rail. */
+  /* Keyed on the BOOLEAN, not the array: a consumer's inline `['/']` is a new
+   * identity every render too (the showcase set passes one), and the effect
+   * must fire on route entry only — never on a re-render, or it undoes the tap. */
+  const opensHere = drawerOpenOn.some((p) => (p === '/' ? currentPath === '/' : currentPath.startsWith(p)))
   useEffect(() => {
-    if (drawer) setDrawerOpen(drawerOpenOn.some((p) => (p === '/' ? currentPath === '/' : currentPath.startsWith(p))))
+    if (drawer) setDrawerOpen(opensHere)
     else setNavHidden(false)
-  }, [currentPath, drawer, drawerOpenOn])
+  }, [currentPath, drawer, opensHere])
 
   /* THE WASH ALSO GOES ON THE ROOT (2026-08-30). It is set on the content
    * wrapper below, which every page inherits — but a PORTALLED surface does