@kolkrabbi/kol-shell 0.36.0 → 0.37.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@kolkrabbi/kol-shell",
-  "version": "0.36.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.153.0",
-    "@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

@@ -56,6 +56,14 @@ import TouchDeviceOverlay, { useTouchPrimary } from './TouchDeviceOverlay.jsx'
  *                                        `bare` renders the children with NO shell on a coarse-pointer device unless
  *                                        localStorage `kol-desktop` is '1' (fxr's gate); `overlay` keeps the shell and
  *                                        mounts TouchDeviceOverlay once (monitor's)
+ * @param {string[]} [props.drawerOpenOn=[]]  paths whose ENTRY opens the drawer (ShellDrawerOpenOnRoute,
+ *                                        kol-mirror 2026-09-01 — user: "the rail should load open on home, not
+ *                                        everywhere"): a home that is a catalog IS navigation, and arriving there
+ *                                        with the nav folded hides the one thing the page is for. Matched like the
+ *                                        rail's active row — `'/'` exact, anything else by prefix. Every other path
+ *                                        keeps the close-on-navigate rule; above `drawerBelow` there is no drawer
+ *                                        and the list is inert. A list, not a boolean: the policy is per-route and
+ *                                        the shell already owns the route.
  * @param {string}  props.appName       TouchDeviceOverlay's subject
  * @param {boolean} props.navKeys       Option+1…9 navigates to the rail's nth ROW through `onNavigate` — with a
  *                                        `logomark` that is the mark ('/') then the items, which is the order on
@@ -91,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,
@@ -107,6 +122,7 @@ export default function AppShell({
   settingsPath,
   settingsKey,
   drawerBelow = 768,
+  drawerOpenOn = NO_PATHS,
   children,
 }) {
   const [navHidden, setNavHidden] = useState(false)
@@ -193,10 +209,17 @@ export default function AppShell({
    * `setNavHidden(false)` also made `navHidden` unusable as a consumer seam —
    * child effects run before parent effects, so a consumer hiding the rail on a
    * path change was overwritten in the same commit. */
+  /* …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(false)
+    if (drawer) setDrawerOpen(opensHere)
     else setNavHidden(false)
-  }, [currentPath, drawer])
+  }, [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