@the-portland-company/shell 0.25.0 → 0.27.0

package/CHANGELOG.md

@@ -1,3 +1,46 @@
+## 0.27.0 — 2026-07-23
+
+VRM native-chrome parity (shell-nav-handoff §1–§3). All three were verified in
+this package's source before changing anything, and all three are additive —
+forms / surveys / polls / contacts behavior is unchanged.
+
+### Added — `NavAppShell contentWrapper`
+
+`'padded'` (default) keeps the existing `<main class="px-6 py-7">` + `mx-auto
+max-w-7xl`. `'bare'` renders children directly. VRM wraps every page in the
+shell's own `PageContainer` — which emits its OWN `<main>` and max-width — so
+the default gave it a **nested `<main>`** (invalid HTML) and a doubled gutter on
+Org Settings, Email Broadcast and SMS/MMS Blast.
+
+### Added — nav group headers can navigate as well as expand
+
+A group header always toggles; when the item also carries an `onClick`, that now
+runs too, so a parent row can open its section dashboard AND expand in one
+click. Existing apps pass no parent `onClick`, so nothing changes for them.
+
+### Added — `NavItem.disabled` + `NavItem.disabledTitle`
+
+Mirrors `NavTab`'s existing contract: a gated row renders as a non-interactive,
+dimmed `aria-disabled` span with its reason as a tooltip. VRM gates Email / SMS
+until setup is complete and previously had no way to express that, so gated rows
+rendered fully clickable.
+
+## 0.26.0 — 2026-07-23
+
+### Added — `PolitogyAppFrame` data plumbing (so Contacts can adopt it)
+
+`PolitogyFrameData` (the optional `data` prop) carries the things that differ by
+**where an app's data comes from**, never by how its chrome looks:
+
+- `fetchJson` / `devNotes` — for apps that authenticate client-side instead of
+  through a same-origin `/api/chrome/*` proxy (Contacts).
+- `org` — controlled active-org state for apps that own switching.
+- `contactsItem` — app-owned children for a standard nav entry the app owns.
+- `fallbackAvatarUrl` — avatar fallback (stored memoji / default) when the
+  profile has no uploaded photo.
+
+Adding a *visual* option here would re-open the drift this component closes.
+
 ## 0.25.0 — 2026-07-23
 
 ### Added — `PolitogyAppFrame`: one component owns the whole app frame

package/dist/app-chrome.cjs

@@ -1313,6 +1313,17 @@ function NavRow({
       item.label
     ] })
   ] });
+  if (item.disabled) {
+    return /* @__PURE__ */ jsxRuntime.jsx(
+      "span",
+      {
+        "aria-disabled": "true",
+        title: item.disabledTitle,
+        className: `${wrapperClasses} cursor-not-allowed opacity-50`,
+        children: inner
+      }
+    );
+  }
   if (item.onClick && !item.href) {
     return /* @__PURE__ */ jsxRuntime.jsx("button", { type: "button", onClick: item.onClick, className: `w-full text-left ${wrapperClasses}`, children: inner });
   }
@@ -1346,7 +1357,10 @@ function NavGroup({
       "button",
       {
         type: "button",
-        onClick: onToggle,
+        onClick: () => {
+          onToggle();
+          item.onClick?.();
+        },
         "aria-expanded": open,
         className: headerClasses,
         children: [
@@ -1866,14 +1880,20 @@ function useCloseNavDetailsOnOutsideClick() {
     };
   }, []);
 }
-function NavAppShell({ sidebar, topBar, betaBanner, children }) {
+function NavAppShell({
+  sidebar,
+  topBar,
+  betaBanner,
+  contentWrapper = "padded",
+  children
+}) {
   useCloseNavDetailsOnOutsideClick();
   return /* @__PURE__ */ jsxRuntime.jsxs("div", { className: "min-h-screen bg-[color:var(--background)] text-[color:var(--foreground)]", children: [
     /* @__PURE__ */ jsxRuntime.jsx(NavSidebar, { ...sidebar }),
     /* @__PURE__ */ jsxRuntime.jsxs("div", { className: "lg:pl-[240px]", children: [
       /* @__PURE__ */ jsxRuntime.jsx(NavTopBar, { ...topBar }),
       betaBanner ? /* @__PURE__ */ jsxRuntime.jsx(NavBetaBanner, { ...betaBanner }) : null,
-      /* @__PURE__ */ jsxRuntime.jsx("main", { className: "px-6 py-7", children: /* @__PURE__ */ jsxRuntime.jsx("div", { className: "mx-auto max-w-7xl", children }) })
+      contentWrapper === "bare" ? children : /* @__PURE__ */ jsxRuntime.jsx("main", { className: "px-6 py-7", children: /* @__PURE__ */ jsxRuntime.jsx("div", { className: "mx-auto max-w-7xl", children }) })
     ] })
   ] });
 }
@@ -1963,6 +1983,7 @@ function PolitogyAppFrame({
   session,
   onSignOut,
   buildMeta,
+  data,
   children
 }) {
   const meta = POLITOGY_APPS[app];
@@ -1970,6 +1991,9 @@ function PolitogyAppFrame({
     PolitogyAppChrome,
     {
       storagePrefix: meta.storagePrefix,
+      fetchJson: data?.fetchJson,
+      devNotes: data?.devNotes,
+      org: data?.org,
       identity: {
         userId: session.userId,
         userName: session.userName ?? null,
@@ -1986,6 +2010,7 @@ function PolitogyAppFrame({
           session,
           onSignOut,
           buildMeta,
+          data,
           children
         }
       )
@@ -2000,6 +2025,7 @@ function FrameBody({
   session,
   onSignOut,
   buildMeta,
+  data,
   children
 }) {
   const meta = POLITOGY_APPS[app];
@@ -2016,8 +2042,14 @@ function FrameBody({
   const activeCoreOrg = coreOrgs.find((o) => o.id === chrome?.orgId) ?? null;
   const navDisplay = chrome?.navDisplay ?? "both";
   const navItems = react.useMemo(
-    () => standardPolitogyNav({ coreBaseUrl: POLITOGY_APPS.vrm.origin, icons: standardNavIcons }),
-    []
+    () => standardPolitogyNav({
+      coreBaseUrl: POLITOGY_APPS.vrm.origin,
+      icons: standardNavIcons,
+      // An app that OWNS a standard nav entry (Contacts) nests its own routes
+      // under it. This is routing data, not chrome configuration.
+      contactsItem: data?.contactsItem
+    }),
+    [data?.contactsItem]
   );
   const breadcrumbs = react.useMemo(
     () => deriveBreadcrumbs(navItems, pathname, { search, origin: meta.origin }),
@@ -2056,7 +2088,9 @@ function FrameBody({
         user: {
           name: identity.userName,
           email: identity.email,
-          avatarUrl: identity.avatarUrl,
+          // The chrome API's avatar wins; the app may supply a fallback chain
+          // (e.g. a stored memoji) for users with no uploaded photo.
+          avatarUrl: identity.avatarUrl ?? data?.fallbackAvatarUrl ?? null,
           role: session.role
         },
         userMenu,