@djangocfg/layouts 2.1.597 → 2.1.599

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@djangocfg/layouts",
-  "version": "2.1.597",
+  "version": "2.1.599",
   "description": "Simple, straightforward layout components for Next.js - import and use with props",
   "keywords": [
     "layouts",
@@ -91,11 +91,11 @@
     "test": "vitest run"
   },
   "peerDependencies": {
-    "@djangocfg/analytics": "^2.1.597",
-    "@djangocfg/api": "^2.1.597",
-    "@djangocfg/devtools": "^2.1.597",
-    "@djangocfg/i18n": "^2.1.597",
-    "@djangocfg/ui-core": "^2.1.597",
+    "@djangocfg/analytics": "^2.1.599",
+    "@djangocfg/api": "^2.1.599",
+    "@djangocfg/devtools": "^2.1.599",
+    "@djangocfg/i18n": "^2.1.599",
+    "@djangocfg/ui-core": "^2.1.599",
     "@hookform/resolvers": "^5.2.2",
     "consola": "^3.4.2",
     "lucide-react": "^0.545.0",
@@ -125,12 +125,12 @@
     "uuid": "^11.1.1"
   },
   "devDependencies": {
-    "@djangocfg/analytics": "^2.1.597",
-    "@djangocfg/api": "^2.1.597",
-    "@djangocfg/devtools": "^2.1.597",
-    "@djangocfg/i18n": "^2.1.597",
-    "@djangocfg/typescript-config": "^2.1.597",
-    "@djangocfg/ui-core": "^2.1.597",
+    "@djangocfg/analytics": "^2.1.599",
+    "@djangocfg/api": "^2.1.599",
+    "@djangocfg/devtools": "^2.1.599",
+    "@djangocfg/i18n": "^2.1.599",
+    "@djangocfg/typescript-config": "^2.1.599",
+    "@djangocfg/ui-core": "^2.1.599",
     "@storybook/react-vite": "^10.5.0",
     "@types/node": "^25.9.5",
     "@types/react": "19.2.15",

package/src/layouts/SettingsLayout/SettingsDialog.tsx

@@ -13,6 +13,7 @@ import { useIsMobile } from '@djangocfg/ui-core/hooks';
 import { cn } from '@djangocfg/ui-core/lib';
 
 import { SettingsForm } from './SettingsForm';
+import { useGuestIntentRedirect } from './hooks/useGuestIntentRedirect';
 import { useSettingsUrl } from './hooks/useSettingsUrl';
 import { useSettingsCopy } from './i18n';
 import { buildBuiltinSections } from './sections';
@@ -77,6 +78,7 @@ export const SettingsDialog: React.FC<SettingsDialogProps> = ({
   );
 
   useSettingsUrl({ urlKey, enabled: syncUrl && isAuthenticated, resolveSection });
+  useGuestIntentRedirect(urlKey ?? 'settings', syncUrl !== false);
 
   const resolvedTitle = title ?? copy.nav.title;
 

package/src/layouts/SettingsLayout/hooks/useGuestIntentRedirect.ts

@@ -0,0 +1,31 @@
+'use client';
+
+/**
+ * useGuestIntentRedirect — a guest who arrives on `#settings/<section>?<intent>` is sent to sign-in and brought back.
+ *
+ * The dialog only opens for a signed-in user, so without this an assistant's consent link (`#settings/mcp?request=…`)
+ * lands on a page where nothing happens. The full address, fragment included, is saved as the post-sign-in
+ * destination, so the dialog reads the same intent once the session exists. A plain `#settings` (no intent) is left
+ * alone: a guest opening it has nothing pending.
+ */
+
+import { useEffect, useRef } from 'react';
+
+import { useAuth, useCfgRouter } from '@djangocfg/api/auth';
+
+import { carriesIntent } from '../intent/parseSettingsHash';
+
+export function useGuestIntentRedirect(urlKey: string, enabled: boolean): void {
+  const { isAuthenticated, isLoading, saveRedirectUrl, routes } = useAuth();
+  const router = useCfgRouter();
+  const sent = useRef(false);
+
+  useEffect(() => {
+    if (!enabled || isLoading || isAuthenticated || sent.current) return;
+    if (!carriesIntent(window.location.hash.slice(1), urlKey)) return;
+    sent.current = true;
+    const { pathname, search, hash } = window.location;
+    saveRedirectUrl(`${pathname}${search}${hash}`);
+    router.push(routes.auth);
+  }, [enabled, isLoading, isAuthenticated, saveRedirectUrl, router, routes.auth, urlKey]);
+}

package/src/layouts/SettingsLayout/intent/parseSettingsHash.test.ts

@@ -1,6 +1,6 @@
 import { describe, expect, it } from 'vitest';
 
-import { buildSettingsHash, parseSettingsHash, sectionToOpen } from './parseSettingsHash';
+import { buildSettingsHash, carriesIntent, parseSettingsHash, sectionToOpen } from './parseSettingsHash';
 
 describe('parseSettingsHash', () => {
   it('is closed for an empty or foreign fragment', () => {
@@ -52,3 +52,13 @@ describe('sectionToOpen', () => {
     expect(sectionToOpen(null, null)).toBeNull();
   });
 });
+
+describe('carriesIntent', () => {
+  it('is true only for a settings link with a query', () => {
+    expect(carriesIntent('settings/mcp?request=abc&client=Claude', 'settings')).toBe(true);
+    expect(carriesIntent('settings/mcp', 'settings')).toBe(false);
+    expect(carriesIntent('settings', 'settings')).toBe(false);
+    expect(carriesIntent('other/mcp?request=abc', 'settings')).toBe(false);
+    expect(carriesIntent('', 'settings')).toBe(false);
+  });
+});

package/src/layouts/SettingsLayout/intent/parseSettingsHash.ts

@@ -53,3 +53,9 @@ export function buildSettingsHash(urlKey: string, section: string | null): strin
 export function sectionToOpen(requested: string | null, resolved: string | null): string | null {
   return resolved ?? requested;
 }
+
+/** True when the fragment is a Settings deep link that carries an arrival intent (`#settings/mcp?request=…`). */
+export function carriesIntent(fragment: string, urlKey: string): boolean {
+  const parsed = parseSettingsHash(fragment, urlKey);
+  return parsed.open && Object.keys(parsed.params).length > 0;
+}