@leavepulse/ui 0.18.1 → 0.18.3

package/dist/chunks/{LpAppShell-CiHAD6PN.js → LpAppShell-BvAVgVLW.js}

@@ -38,11 +38,17 @@ const _sfc_main = /* @__PURE__ */ defineComponent({
     const flatItems = computed(
       () => props.sections ? props.sections.flatMap((s) => s.items) : props.items ?? []
     );
-    function defaultActive(item) {
-      if (props.path == null) return item.id === props.modelValue;
+    function matchesPath(item) {
       if (item.id === "/") return props.path === "/";
       return props.path === item.id || props.path.startsWith(`${item.id}/`);
     }
+    function defaultActive(item) {
+      if (props.path == null) return item.id === props.modelValue;
+      if (!matchesPath(item)) return false;
+      return !flatItems.value.some(
+        (other) => other.id !== item.id && other.id.length > item.id.length && matchesPath(other)
+      );
+    }
     const activeTest = computed(() => props.isActive ?? defaultActive);
     const pageTitle = computed(() => {
       if (props.title != null) return props.title;

package/dist/components/LpAppShell.vue.js

@@ -1,4 +1,4 @@
-import { _ as _sfc_main } from "../chunks/LpAppShell-CiHAD6PN.js";
+import { _ as _sfc_main } from "../chunks/LpAppShell-BvAVgVLW.js";
 export {
   _sfc_main as default
 };

package/dist/index.js

@@ -3,7 +3,7 @@ import { _ as _2 } from "./chunks/LpAddressInput-Dfyn88Dd.js";
 import { _ as _3 } from "./chunks/LpMapPicker-DFKrd1Ne.js";
 import { OSM_TILES, createNominatimProvider, emptyAddress, formatAddress, project, tileUrl, unproject } from "./components/address.js";
 import { _ as _4 } from "./chunks/LpAlert-BAYHDSS1.js";
-import { _ as _5 } from "./chunks/LpAppShell-CiHAD6PN.js";
+import { _ as _5 } from "./chunks/LpAppShell-BvAVgVLW.js";
 import { _ as _6 } from "./chunks/LpAutocomplete-DIgXmtCA.js";
 import { _ as _7 } from "./chunks/LpAvatar-B1Ez0ZAx.js";
 import { _ as _8 } from "./chunks/LpBadge-CXzBPnwO.js";

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@leavepulse/ui",
-  "version": "0.18.1",
+  "version": "0.18.3",
   "type": "module",
   "description": "LeavePulse token-driven Vue component kit.",
   "repository": {

package/src/components/LpAppShell.vue

@@ -83,10 +83,23 @@ const flatItems = computed<SidebarItem[]>(() =>
 
 // Default active test: exact match for "/", prefix match (on a segment
 // boundary) otherwise — so "/billing" doesn't light up on "/billing-x".
+function matchesPath(item: SidebarItem): boolean {
+  if (item.id === "/") return props.path === "/"
+  return props.path === item.id || props.path!.startsWith(`${item.id}/`)
+}
+
+// Only the LONGEST match lights up. A parent whose id prefixes its children —
+// "/dcim" over "/dcim/power" — otherwise stays lit alongside whichever child is
+// open, and two highlighted items read as two current pages.
 function defaultActive(item: SidebarItem): boolean {
   if (props.path == null) return item.id === props.modelValue
-  if (item.id === "/") return props.path === "/"
-  return props.path === item.id || props.path.startsWith(`${item.id}/`)
+  if (!matchesPath(item)) return false
+  return !flatItems.value.some(
+    (other) =>
+      other.id !== item.id &&
+      other.id.length > item.id.length &&
+      matchesPath(other),
+  )
 }
 const activeTest = computed(() => props.isActive ?? defaultActive)
 

package/src/runtime/theme.plugin.ts

@@ -16,11 +16,9 @@
  * holds the full token values, including themes the server cannot name — keeps
  * winning.
  */
-// Imported through the package barrel on purpose. This file is shipped as
-// source and compiled by the CONSUMING app, where "@leavepulse/ui" resolves to
-// dist — the same module instance the kit's components use. A relative "../theme/…"
-// would load a second copy of the theme singleton from src/, leaving components
-// subscribed to a different reactive state than the one this plugin drives.
+// Through the barrel, not "../theme/…": this file is compiled by the consuming
+// app, where the barrel resolves to the same dist instance the components use.
+// A relative path would load a second copy of the theme singleton.
 import {
   bootstrapTheme,
   leavepulse,
@@ -30,10 +28,6 @@ import {
   themeToCssRule,
   type TokenSet,
 } from "@leavepulse/ui"
-// "#imports" is Nuxt's virtual module: it resolves in the consuming app, which
-// is what compiles this file. (No @ts-expect-error here — the kit's own build
-// excludes src/runtime, so the directive would be unused there and error in the
-// consumer, where the import type-checks fine.)
 import { defineNuxtPlugin, useCookie, useHead, useRuntimeConfig } from "#imports"
 
 export default defineNuxtPlugin(() => {
@@ -45,8 +39,6 @@ export default defineNuxtPlugin(() => {
   const fallback =
     themes.find((theme) => theme.name === options.defaultTheme) ?? leavepulse
 
-  // Read-only here: the cookie is written by cacheTheme() in the browser, which
-  // is the one place a theme choice is recorded.
   const cookie = useCookie<string | null>(THEME_COOKIE, { readonly: true })
   const initial = themeFromCookie(cookie.value, themes) ?? fallback