@djangocfg/layouts 2.1.541 → 2.1.543

package/src/layouts/types/providers.types.ts

@@ -1,7 +1,7 @@
 /**
  * Provider Configuration Types
  *
- * Configuration types for layout providers (theme, SWR, MCP chat, Centrifugo)
+ * Configuration types for layout providers (theme, SWR, MCP chat)
  * Note: Analytics, PWA, Push, and Error types are defined in their respective modules
  */
 
@@ -25,16 +25,3 @@ export interface SWRConfigOptions {
   revalidateOnReconnect?: boolean;
   dedupingInterval?: number;
 }
-
-// ============================================================================
-// Centrifugo Configuration
-// ============================================================================
-
-export interface CentrifugoConfig {
-  /** Enable Centrifugo (default: true) */
-  enabled?: boolean;
-  /** Auto-connect on mount (default: false) */
-  autoConnect?: boolean;
-  /** Centrifugo server URL */
-  url?: string;
-}

package/src/snippets/Breadcrumbs.tsx

@@ -42,18 +42,18 @@ const Breadcrumbs: React.FC<BreadcrumbsProps> = ({ items, className = "" }) => {
         {breadcrumbs.map((item: BreadcrumbItem, index: number) => (
           <li key={item.path} className="flex items-center">
             {index > 0 && (
-              <ChevronRight className="w-4 h-4 text-gray-400 mx-2" />
+              <ChevronRight className="w-4 h-4 text-muted-foreground mx-2" />
             )}
 
             {item.isActive ? (
-              <span className="text-gray-900 dark:text-white font-medium flex items-center gap-1">
+              <span className="text-foreground font-medium flex items-center gap-1">
                 {index === 0 && <Home className="w-4 h-4" />}
                 {item.label}
               </span>
             ) : (
               <Link
                 href={item.path}
-                className="text-gray-500 dark:text-gray-400 hover:text-gray-700 dark:hover:text-gray-200
+                className="text-muted-foreground hover:text-foreground
                          transition-colors duration-200 flex items-center gap-1 hover:underline"
               >
                 {index === 0 && <Home className="w-4 h-4" />}

package/src/testing/MockAuthFormProvider.tsx

@@ -3,7 +3,6 @@
 import React, { useMemo, useRef, useState } from 'react';
 
 import type {
-  AuthFormReturn,
   AuthStep,
   WebmailLink,
 } from '@djangocfg/api/auth';

package/src/utils/navItemsShallowEqual.ts

@@ -0,0 +1,34 @@
+import type { NavigationItem } from '../layouts/types/ui.types';
+import { shallowEqualBy } from './shallowEqualBy';
+
+/**
+ * Shallow equality for NavigationItem arrays. Compares href, label, external
+ * and one-level-deep sub-items so that nav label/badge changes re-render,
+ * but object-reference churn from parent is ignored.
+ *
+ * Lived as two byte-identical copies in NavbarShell and NavDesktopItems.
+ * The sub-item pass used `bi.items![j]`, an assertion resting on the length
+ * comparison two lines above it; `shallowEqualBy` makes that structural
+ * instead, so the non-null assertion is gone rather than relocated.
+ */
+export function navItemsShallowEqual(
+  a: NavigationItem[],
+  b: NavigationItem[],
+): boolean {
+  return shallowEqualBy(
+    a,
+    b,
+    (ai, bi) =>
+      ai.href === bi.href &&
+      ai.label === bi.label &&
+      ai.external === bi.external &&
+      shallowEqualBy(
+        ai.items,
+        bi.items,
+        (sub, bsub) =>
+          sub.href === bsub.href &&
+          sub.label === bsub.label &&
+          sub.external === bsub.external,
+      ),
+  );
+}

package/src/utils/shallowEqualBy.ts

@@ -0,0 +1,29 @@
+/**
+ * Pairwise array comparison for `React.memo` props.
+ *
+ * Four hand-rolled copies of this existed across the nav layouts — two of them
+ * (`navItemsShallowEqual` in NavbarShell and NavDesktopItems) byte-identical.
+ * Each wrote `const bi = b[i]` inside `a.every`, which is safe only because the
+ * length was compared on the line above; under `strict` that produced 26 of the
+ * package's 28 errors, all asking the same question about the same missing
+ * connection between the guard and the index.
+ *
+ * `entries()` carries the pair, so an element and its counterpart are produced
+ * together and neither can be `undefined`. The guard is not restated — it is
+ * expressed once, in the shape of the loop.
+ */
+export function shallowEqualBy<T>(
+  a: readonly T[] | undefined,
+  b: readonly T[] | undefined,
+  equal: (x: T, y: T) => boolean,
+): boolean {
+  if (a === b) return true;
+  if (!a || !b || a.length !== b.length) return false;
+  for (const [index, left] of a.entries()) {
+    const right = b[index];
+    // `index` comes from `a.entries()` and the lengths matched, so `right`
+    // exists. The check is what proves it to the compiler; it is not reachable.
+    if (right === undefined || !equal(left, right)) return false;
+  }
+  return true;
+}