@the-portland-company/shell 0.3.4 → 0.3.5

package/CHANGELOG.md

@@ -1,5 +1,12 @@
 # Changelog
 
+## 0.3.5 — 2026-05-14
+
+### Fixed
+
+- `ShellModeValue.getModeConfig` is now optional. If omitted, shell auto-resolves the active mode config via `modes.find(m => m.id === currentMode)`. Defangs the common `this`-binding mistake where consumers wrote `getModeConfig() { return this.modes[0] }` and shell internals destructured the function (losing `this`) and crashed the chrome.
+- All internal call sites (`<ModePill>`, `<UserMenu>`, `<NavItem>`) now use a defensive `resolveModeConfig(mode)` helper that try/catches the consumer-supplied function and falls back to derivation. A broken `getModeConfig` no longer blanks the page.
+
 ## 0.3.4 — 2026-05-14
 
 ### Fixed

package/dist/index.cjs

@@ -1275,6 +1275,20 @@ function getUserDisplayName(user) {
   if (!user) return "User";
   return user.profile?.full_name || user.profile?.name || user.user_metadata?.full_name || user.user_metadata?.name || (typeof user.email === "string" ? user.email.split("@")[0] : "") || user.email || "User";
 }
+
+// src/lib/resolveModeConfig.ts
+function resolveModeConfig(mode) {
+  if (!mode) return null;
+  const modes = mode.modes ?? [];
+  if (typeof mode.getModeConfig === "function") {
+    try {
+      const result = mode.getModeConfig();
+      if (result) return result;
+    } catch {
+    }
+  }
+  return modes.find((m) => m.id === mode.currentMode) ?? modes[0] ?? null;
+}
 function avatarUrlFromUser(user) {
   if (!user) return void 0;
   const meta = user.raw.user_metadata;
@@ -1299,7 +1313,7 @@ function UserMenu() {
   if (status !== "authenticated" || !user) return null;
   const displayName = getUserDisplayName(user.raw);
   const avatarUrl = userAvatarUrl ?? avatarUrlFromUser(user);
-  const modeConfig = mode?.getModeConfig() ?? null;
+  const modeConfig = resolveModeConfig(mode);
   const currentOrg = org?.currentOrg ?? null;
   const linkedAccounts = linked?.linkedAccounts ?? [];
   const activeRoleConfig = role?.activeRoleConfig ?? null;
@@ -1587,7 +1601,7 @@ function NavItem({
   const navigate = reactRouterDom.useNavigate();
   const mode = useShellMode();
   const currentMode = mode?.currentMode ?? null;
-  const modeConfig = mode?.getModeConfig() ?? null;
+  const modeConfig = resolveModeConfig(mode);
   const navPref = useShellNavigationPreference();
   const showSubMenu = navPref?.showSubMenu ?? true;
   const navHoverGradient = currentMode === MODE_CAMPAIGN ? "linear-gradient(90deg, #F2C94C 0%, #E63946 100%)" : currentMode === MODE_PETITION ? "linear-gradient(90deg, #F2C94C 0%, #007BFF 100%)" : "linear-gradient(90deg, #007BFF 0%, #E63946 100%)";
@@ -2134,8 +2148,8 @@ var OrgPill_default = OrgPill;
 function ModePill() {
   const mode = useShellMode();
   if (!mode) return null;
-  const { currentMode, modes, switchMode, getModeConfig } = mode;
-  const modeConfig = getModeConfig();
+  const { currentMode, modes, switchMode } = mode;
+  const modeConfig = resolveModeConfig(mode);
   if (!modeConfig) return null;
   const hasMultipleModes = modes.length > 1;
   const isDisabled = !hasMultipleModes;