@igstack/app-catalog-frontend-core 6.8.1 → 6.8.2

package/dist/esm/ui/components/header/Header.js

@@ -10,6 +10,7 @@ import { AppCatalogContext } from "../../../modules/appCatalog/context/AppCatalo
 import { AttributionFooter } from "../../../modules/appCatalog/ui/catalog/AttributionFooter.js";
 import { useAuth, useIsAuthenticated, useUser } from "../../../modules/auth/AuthContext.js";
 import { Button } from "../../button.js";
+import { useAuthModal } from "../../../modules/auth/AuthModalContext.js";
 import { useAuthActions } from "../../../modules/auth/useAuthActions.js";
 import { DropdownMenu, DropdownMenuTrigger, DropdownMenuContent, DropdownMenuSeparator, DropdownMenuItem } from "../../dropdown-menu.js";
 import { Popover, PopoverTrigger, PopoverContent, PopoverClose } from "../../popover.js";
@@ -117,6 +118,7 @@ function Header({ middle }) {
   const isAuthenticated = useIsAuthenticated();
   const user = useUser();
   const { logout, devLogin } = useAuthActions();
+  const { open: openLoginModal } = useAuthModal();
   const appCatalogContextMaybe = use(AppCatalogContext);
   const trpc = useTRPC();
   const { data: providersData } = useQuery(
@@ -129,6 +131,10 @@ function Header({ middle }) {
       console.error("Logout failed:", error);
     }
   };
+  const handleLoginClick = () => {
+    const currentUrl = window.location.pathname + window.location.search;
+    openLoginModal(currentUrl);
+  };
   return /* @__PURE__ */ jsxs("div", { className: "flex items-center mb-4 pb-4 gap-4 border-b", children: [
     /* @__PURE__ */ jsx("div", { className: "flex items-center gap-4", children: /* @__PURE__ */ jsxs("div", { className: "flex items-center gap-2", children: [
       /* @__PURE__ */ jsx(Link, { to: "/", children: /* @__PURE__ */ jsx(SvgAppCatalog, { className: "h-16 w-16" }) }),
@@ -141,24 +147,29 @@ function Header({ middle }) {
     /* @__PURE__ */ jsxs("div", { className: "flex items-center gap-3 ml-auto", children: [
       /* @__PURE__ */ jsx(ThemeSwitcher, {}),
       isLoading ? /* @__PURE__ */ jsx(Skeleton, { className: "w-8 h-8 rounded-full" }) : !isAuthenticated ? (
-        // No Login button: the catalog is fully browsable without signing in,
-        // so it mostly advertised a flow most visitors have no reason to
-        // enter. Auth itself is untouched -- the modal, the login route, the
-        // client and session handling all still work, so anyone who reaches
-        // the flow by URL or by calling `useAuthModal().open()` can sign in,
-        // and this branch flips to the user menu below once they do. `git
-        // revert` of the commit that removed it restores the button as-is.
-        // DEV Login stays: it is gated behind `devLoginEnabled`, so it never
-        // reaches a real deployment.
-        (providersData == null ? void 0 : providersData.devLoginEnabled) && /* @__PURE__ */ jsx(
-          "button",
-          {
-            type: "button",
-            onClick: devLogin,
-            className: "text-xs font-medium px-2 py-1 rounded bg-amber-500/20 text-amber-600 dark:text-amber-400 hover:bg-amber-500/30 border border-amber-500/30 cursor-pointer",
-            children: "DEV Login"
-          }
-        )
+        // The catalog is browsable without signing in; Login is for the parts
+        // that need an identity (comments, admin). DEV Login is gated behind
+        // `devLoginEnabled`, so it never reaches a real deployment.
+        /* @__PURE__ */ jsxs("div", { className: "flex items-center gap-2", children: [
+          (providersData == null ? void 0 : providersData.devLoginEnabled) && /* @__PURE__ */ jsx(
+            "button",
+            {
+              type: "button",
+              onClick: devLogin,
+              className: "text-xs font-medium px-2 py-1 rounded bg-amber-500/20 text-amber-600 dark:text-amber-400 hover:bg-amber-500/30 border border-amber-500/30 cursor-pointer",
+              children: "DEV Login"
+            }
+          ),
+          /* @__PURE__ */ jsx(
+            "button",
+            {
+              type: "button",
+              onClick: handleLoginClick,
+              className: "text-sm font-medium text-muted-foreground hover:text-foreground cursor-pointer",
+              children: "Login"
+            }
+          )
+        ] })
       ) : /* @__PURE__ */ jsxs(DropdownMenu, { children: [
         /* @__PURE__ */ jsx(DropdownMenuTrigger, { asChild: true, children: /* @__PURE__ */ jsx(
           Button,

package/dist/esm/ui/components/header/Header.js.map

@@ -1 +1 @@
-{"version":3,"file":"Header.js","sources":["../../../../../src/ui/components/header/Header.tsx"],"sourcesContent":["import type {\n  AppVersionInfo,\n  VersionInfo,\n} from '@igstack/app-catalog-backend-core'\nimport { Link } from '@tanstack/react-router'\nimport { useQuery } from '@tanstack/react-query'\nimport { LogOut, X } from 'lucide-react'\nimport React, { use } from 'react'\nimport AppCatalogLogo from '~/assets/app-catalog.svg?react'\nimport { useTRPC } from '~/api/infra/trpc'\nimport { ThemeSwitcher } from '~/components/ThemeSwitcher'\nimport { AppCatalogContext } from '~/modules/appCatalog/context/AppCatalogContext'\nimport { AttributionFooter } from '~/modules/appCatalog/ui/catalog/AttributionFooter'\nimport {\n  useAuth,\n  useAuthActions,\n  useIsAuthenticated,\n  useUser,\n} from '~/modules/auth'\nimport { Button } from '~/ui/button'\nimport {\n  DropdownMenu,\n  DropdownMenuContent,\n  DropdownMenuItem,\n  DropdownMenuSeparator,\n  DropdownMenuTrigger,\n} from '~/ui/dropdown-menu'\nimport {\n  Popover,\n  PopoverClose,\n  PopoverContent,\n  PopoverTrigger,\n} from '~/ui/popover'\nimport { Skeleton } from '~/ui/skeleton'\n\nfunction ShaSuffix({ version }: { version: VersionInfo }) {\n  if (!version.sha) return null\n  if (version.shaUrl) {\n    return (\n      <>\n        {' '}\n        <a\n          href={version.shaUrl}\n          target=\"_blank\"\n          rel=\"noopener noreferrer\"\n          className=\"text-muted-foreground hover:text-primary\"\n          onClick={(e) => e.stopPropagation()}\n        >\n          ({version.sha})\n        </a>\n      </>\n    )\n  }\n  return <span className=\"text-muted-foreground\"> ({version.sha})</span>\n}\n\nfunction VersionItem({\n  label,\n  version,\n}: {\n  label: string\n  version: VersionInfo\n}) {\n  return (\n    <div className=\"flex items-center justify-between gap-4 py-1\">\n      <span className=\"text-xs text-muted-foreground\">{label}</span>\n      <span className=\"text-sm font-medium text-right\">\n        {version.url ? (\n          <a\n            href={version.url}\n            target=\"_blank\"\n            rel=\"noopener noreferrer\"\n            className=\"text-primary hover:text-primary/80 transition-colors\"\n            onClick={(e) => e.stopPropagation()}\n          >\n            {version.displayName}\n          </a>\n        ) : (\n          version.displayName\n        )}\n        <ShaSuffix version={version} />\n      </span>\n    </div>\n  )\n}\n\nexport function VersionDisplay({ versions }: { versions: AppVersionInfo }) {\n  const isLocal =\n    versions.backend?.displayName === 'local' && !versions.coreVersion\n\n  if (isLocal) {\n    return (\n      <span className=\"text-xs text-muted-foreground\" title=\"Local\">\n        local\n      </span>\n    )\n  }\n\n  return (\n    <div className=\"flex flex-col divide-y\">\n      {versions.backend && (\n        <VersionItem label=\"Pipeline\" version={versions.backend} />\n      )}\n      {versions.coreVersion && (\n        <VersionItem label=\"Core\" version={versions.coreVersion} />\n      )}\n      {versions.frontend && (\n        <VersionItem label=\"FE\" version={versions.frontend} />\n      )}\n    </div>\n  )\n}\n\n/**\n * Version detail moved behind an on-demand popover (same click-to-reveal\n * pattern as PersonBadge's approver chips) instead of always-visible text —\n * keeps the header compact. Content is unchanged: same Pipeline/Core/FE\n * lines, links, and SHA suffix as before, just revealed on click.\n */\nexport function VersionPopover({ versions }: { versions: AppVersionInfo }) {\n  const isLocal =\n    versions.backend?.displayName === 'local' && !versions.coreVersion\n\n  if (isLocal) {\n    return (\n      <span className=\"text-xs text-muted-foreground\" title=\"Local\">\n        local\n      </span>\n    )\n  }\n\n  return (\n    <Popover>\n      <PopoverTrigger asChild>\n        <button\n          type=\"button\"\n          className=\"inline-flex items-center gap-1 rounded-full bg-muted px-2 py-0.5 text-[10px] font-mono leading-none text-muted-foreground hover:bg-primary/15 hover:text-primary transition-colors\"\n          aria-label=\"Show version details\"\n          title=\"Version details\"\n          data-testid=\"version-info-trigger\"\n        >\n          {versions.backend?.displayName ?? '?'}\n        </button>\n      </PopoverTrigger>\n      <PopoverContent align=\"start\" className=\"w-72 p-0 overflow-hidden\">\n        <div className=\"flex items-center justify-between px-3 py-2 border-b bg-muted/40\">\n          <span className=\"text-sm font-semibold\">Version details</span>\n          <PopoverClose\n            className=\"rounded p-1 text-muted-foreground hover:text-foreground hover:bg-muted transition-colors\"\n            aria-label=\"Close\"\n          >\n            <X className=\"size-3.5\" />\n          </PopoverClose>\n        </div>\n        <div className=\"px-3 py-1\">\n          <VersionDisplay versions={versions} />\n        </div>\n        <AttributionFooter variant=\"popover\" />\n      </PopoverContent>\n    </Popover>\n  )\n}\n\nexport interface HeaderProps {\n  middle?: React.ReactNode\n}\n\nexport function Header({ middle }: HeaderProps) {\n  const { isLoading } = useAuth()\n  const isAuthenticated = useIsAuthenticated()\n  const user = useUser()\n  const { logout, devLogin } = useAuthActions()\n  const appCatalogContextMaybe = use(AppCatalogContext)\n  const trpc = useTRPC()\n  const { data: providersData } = useQuery(\n    trpc.auth.getProviders.queryOptions(),\n  )\n\n  const handleLogout = async () => {\n    try {\n      await logout()\n    } catch (error) {\n      console.error('Logout failed:', error)\n    }\n  }\n\n  return (\n    <div className=\"flex items-center mb-4 pb-4 gap-4 border-b\">\n      <div className=\"flex items-center gap-4\">\n        <div className=\"flex items-center gap-2\">\n          <Link to=\"/\">\n            <AppCatalogLogo className=\"h-16 w-16\" />\n          </Link>\n          <div className=\"flex flex-col items-start\">\n            <Link to=\"/\" className=\"font-serif text-lg font-semibold\">\n              App Catalog\n            </Link>\n            {appCatalogContextMaybe?.versions && (\n              <VersionPopover versions={appCatalogContextMaybe.versions} />\n            )}\n          </div>\n        </div>\n      </div>\n      {middle && <div className=\"flex-1\">{middle}</div>}\n      <div className=\"flex items-center gap-3 ml-auto\">\n        <ThemeSwitcher />\n        {isLoading ? (\n          <Skeleton className=\"w-8 h-8 rounded-full\" />\n        ) : !isAuthenticated ? (\n          // No Login button: the catalog is fully browsable without signing in,\n          // so it mostly advertised a flow most visitors have no reason to\n          // enter. Auth itself is untouched -- the modal, the login route, the\n          // client and session handling all still work, so anyone who reaches\n          // the flow by URL or by calling `useAuthModal().open()` can sign in,\n          // and this branch flips to the user menu below once they do. `git\n          // revert` of the commit that removed it restores the button as-is.\n          // DEV Login stays: it is gated behind `devLoginEnabled`, so it never\n          // reaches a real deployment.\n          providersData?.devLoginEnabled && (\n            <button\n              type=\"button\"\n              onClick={devLogin}\n              className=\"text-xs font-medium px-2 py-1 rounded bg-amber-500/20 text-amber-600 dark:text-amber-400 hover:bg-amber-500/30 border border-amber-500/30 cursor-pointer\"\n            >\n              DEV Login\n            </button>\n          )\n        ) : (\n          <DropdownMenu>\n            <DropdownMenuTrigger asChild>\n              <Button\n                variant=\"ghost\"\n                size=\"sm\"\n                className=\"w-8 h-8 rounded-full p-0 hover:bg-accent hover:text-accent-foreground cursor-pointer\"\n                aria-label={`User menu for ${user?.name || 'User'}`}\n                data-testid=\"user-avatar-button\"\n              >\n                {user?.image ? (\n                  <img\n                    src={user.image}\n                    alt={user.name || 'User'}\n                    className=\"w-full h-full rounded-full object-cover\"\n                  />\n                ) : (\n                  <div className=\"flex items-center justify-center w-full h-full rounded-full bg-primary text-primary-foreground text-xs font-semibold\">\n                    {(user?.name || user?.email || '?').charAt(0).toUpperCase()}\n                  </div>\n                )}\n              </Button>\n            </DropdownMenuTrigger>\n            <DropdownMenuContent align=\"end\" className=\"w-56\">\n              <div className=\"px-2 py-3 flex items-center gap-3\">\n                {user?.image ? (\n                  <img\n                    src={user.image}\n                    alt={user.name || 'User'}\n                    className=\"w-10 h-10 rounded-full object-cover shrink-0\"\n                  />\n                ) : (\n                  <div className=\"flex items-center justify-center w-10 h-10 rounded-full bg-primary text-primary-foreground text-sm font-semibold shrink-0\">\n                    {(user?.name || user?.email || '?').charAt(0).toUpperCase()}\n                  </div>\n                )}\n                <div className=\"flex flex-col min-w-0\">\n                  <p className=\"text-xs text-muted-foreground truncate\">\n                    @{user?.email?.split('@')[0] || 'user'}\n                  </p>\n                  <p className=\"text-sm font-medium truncate\">\n                    {user?.name || 'User'}\n                  </p>\n                </div>\n              </div>\n              <DropdownMenuSeparator />\n              <DropdownMenuItem onClick={handleLogout}>\n                <LogOut className=\"h-4 w-4 mr-2\" />\n                <span>Sign out</span>\n              </DropdownMenuItem>\n            </DropdownMenuContent>\n          </DropdownMenu>\n        )}\n      </div>\n    </div>\n  )\n}\n"],"names":["AppCatalogLogo"],"mappings":";;;;;;;;;;;;;;;;AAmCA,SAAS,UAAU,EAAE,WAAqC;AACxD,MAAI,CAAC,QAAQ,IAAK,QAAO;AACzB,MAAI,QAAQ,QAAQ;AAClB,WACE,qBAAA,UAAA,EACG,UAAA;AAAA,MAAA;AAAA,MACD;AAAA,QAAC;AAAA,QAAA;AAAA,UACC,MAAM,QAAQ;AAAA,UACd,QAAO;AAAA,UACP,KAAI;AAAA,UACJ,WAAU;AAAA,UACV,SAAS,CAAC,MAAM,EAAE,gBAAA;AAAA,UACnB,UAAA;AAAA,YAAA;AAAA,YACG,QAAQ;AAAA,YAAI;AAAA,UAAA;AAAA,QAAA;AAAA,MAAA;AAAA,IAChB,GACF;AAAA,EAEJ;AACA,SAAO,qBAAC,QAAA,EAAK,WAAU,yBAAwB,UAAA;AAAA,IAAA;AAAA,IAAG,QAAQ;AAAA,IAAI;AAAA,EAAA,GAAC;AACjE;AAEA,SAAS,YAAY;AAAA,EACnB;AAAA,EACA;AACF,GAGG;AACD,SACE,qBAAC,OAAA,EAAI,WAAU,gDACb,UAAA;AAAA,IAAA,oBAAC,QAAA,EAAK,WAAU,iCAAiC,UAAA,OAAM;AAAA,IACvD,qBAAC,QAAA,EAAK,WAAU,kCACb,UAAA;AAAA,MAAA,QAAQ,MACP;AAAA,QAAC;AAAA,QAAA;AAAA,UACC,MAAM,QAAQ;AAAA,UACd,QAAO;AAAA,UACP,KAAI;AAAA,UACJ,WAAU;AAAA,UACV,SAAS,CAAC,MAAM,EAAE,gBAAA;AAAA,UAEjB,UAAA,QAAQ;AAAA,QAAA;AAAA,MAAA,IAGX,QAAQ;AAAA,MAEV,oBAAC,aAAU,QAAA,CAAkB;AAAA,IAAA,EAAA,CAC/B;AAAA,EAAA,GACF;AAEJ;AAEO,SAAS,eAAe,EAAE,YAA0C;;AACzE,QAAM,YACJ,cAAS,YAAT,mBAAkB,iBAAgB,WAAW,CAAC,SAAS;AAEzD,MAAI,SAAS;AACX,+BACG,QAAA,EAAK,WAAU,iCAAgC,OAAM,SAAQ,UAAA,SAE9D;AAAA,EAEJ;AAEA,SACE,qBAAC,OAAA,EAAI,WAAU,0BACZ,UAAA;AAAA,IAAA,SAAS,WACR,oBAAC,aAAA,EAAY,OAAM,YAAW,SAAS,SAAS,SAAS;AAAA,IAE1D,SAAS,eACR,oBAAC,aAAA,EAAY,OAAM,QAAO,SAAS,SAAS,aAAa;AAAA,IAE1D,SAAS,YACR,oBAAC,aAAA,EAAY,OAAM,MAAK,SAAS,SAAS,SAAA,CAAU;AAAA,EAAA,GAExD;AAEJ;AAQO,SAAS,eAAe,EAAE,YAA0C;;AACzE,QAAM,YACJ,cAAS,YAAT,mBAAkB,iBAAgB,WAAW,CAAC,SAAS;AAEzD,MAAI,SAAS;AACX,+BACG,QAAA,EAAK,WAAU,iCAAgC,OAAM,SAAQ,UAAA,SAE9D;AAAA,EAEJ;AAEA,8BACG,SAAA,EACC,UAAA;AAAA,IAAA,oBAAC,gBAAA,EAAe,SAAO,MACrB,UAAA;AAAA,MAAC;AAAA,MAAA;AAAA,QACC,MAAK;AAAA,QACL,WAAU;AAAA,QACV,cAAW;AAAA,QACX,OAAM;AAAA,QACN,eAAY;AAAA,QAEX,YAAA,cAAS,YAAT,mBAAkB,gBAAe;AAAA,MAAA;AAAA,IAAA,GAEtC;AAAA,IACA,qBAAC,gBAAA,EAAe,OAAM,SAAQ,WAAU,4BACtC,UAAA;AAAA,MAAA,qBAAC,OAAA,EAAI,WAAU,oEACb,UAAA;AAAA,QAAA,oBAAC,QAAA,EAAK,WAAU,yBAAwB,UAAA,mBAAe;AAAA,QACvD;AAAA,UAAC;AAAA,UAAA;AAAA,YACC,WAAU;AAAA,YACV,cAAW;AAAA,YAEX,UAAA,oBAAC,GAAA,EAAE,WAAU,WAAA,CAAW;AAAA,UAAA;AAAA,QAAA;AAAA,MAC1B,GACF;AAAA,0BACC,OAAA,EAAI,WAAU,aACb,UAAA,oBAAC,gBAAA,EAAe,UAAoB,GACtC;AAAA,MACA,oBAAC,mBAAA,EAAkB,SAAQ,UAAA,CAAU;AAAA,IAAA,EAAA,CACvC;AAAA,EAAA,GACF;AAEJ;AAMO,SAAS,OAAO,EAAE,UAAuB;;AAC9C,QAAM,EAAE,UAAA,IAAc,QAAA;AACtB,QAAM,kBAAkB,mBAAA;AACxB,QAAM,OAAO,QAAA;AACb,QAAM,EAAE,QAAQ,SAAA,IAAa,eAAA;AAC7B,QAAM,yBAAyB,IAAI,iBAAiB;AACpD,QAAM,OAAO,QAAA;AACb,QAAM,EAAE,MAAM,cAAA,IAAkB;AAAA,IAC9B,KAAK,KAAK,aAAa,aAAA;AAAA,EAAa;AAGtC,QAAM,eAAe,YAAY;AAC/B,QAAI;AACF,YAAM,OAAA;AAAA,IACR,SAAS,OAAO;AACd,cAAQ,MAAM,kBAAkB,KAAK;AAAA,IACvC;AAAA,EACF;AAEA,SACE,qBAAC,OAAA,EAAI,WAAU,8CACb,UAAA;AAAA,IAAA,oBAAC,SAAI,WAAU,2BACb,UAAA,qBAAC,OAAA,EAAI,WAAU,2BACb,UAAA;AAAA,MAAA,oBAAC,QAAK,IAAG,KACP,8BAACA,eAAA,EAAe,WAAU,aAAY,EAAA,CACxC;AAAA,MACA,qBAAC,OAAA,EAAI,WAAU,6BACb,UAAA;AAAA,QAAA,oBAAC,MAAA,EAAK,IAAG,KAAI,WAAU,oCAAmC,UAAA,eAE1D;AAAA,SACC,iEAAwB,aACvB,oBAAC,gBAAA,EAAe,UAAU,uBAAuB,SAAA,CAAU;AAAA,MAAA,EAAA,CAE/D;AAAA,IAAA,EAAA,CACF,EAAA,CACF;AAAA,IACC,UAAU,oBAAC,OAAA,EAAI,WAAU,UAAU,UAAA,QAAO;AAAA,IAC3C,qBAAC,OAAA,EAAI,WAAU,mCACb,UAAA;AAAA,MAAA,oBAAC,eAAA,EAAc;AAAA,MACd,YACC,oBAAC,UAAA,EAAS,WAAU,uBAAA,CAAuB,IACzC,CAAC;AAAA;AAAA;AAAA;AAAA;AAAA;AAAA;AAAA;AAAA;AAAA;AAAA,SAUH,+CAAe,oBACb;AAAA,UAAC;AAAA,UAAA;AAAA,YACC,MAAK;AAAA,YACL,SAAS;AAAA,YACT,WAAU;AAAA,YACX,UAAA;AAAA,UAAA;AAAA,QAAA;AAAA,+BAKF,cAAA,EACC,UAAA;AAAA,QAAA,oBAAC,qBAAA,EAAoB,SAAO,MAC1B,UAAA;AAAA,UAAC;AAAA,UAAA;AAAA,YACC,SAAQ;AAAA,YACR,MAAK;AAAA,YACL,WAAU;AAAA,YACV,cAAY,kBAAiB,6BAAM,SAAQ,MAAM;AAAA,YACjD,eAAY;AAAA,YAEX,wCAAM,SACL;AAAA,cAAC;AAAA,cAAA;AAAA,gBACC,KAAK,KAAK;AAAA,gBACV,KAAK,KAAK,QAAQ;AAAA,gBAClB,WAAU;AAAA,cAAA;AAAA,YAAA,IAGZ,oBAAC,OAAA,EAAI,WAAU,wHACX,YAAA,6BAAM,UAAQ,6BAAM,UAAS,KAAK,OAAO,CAAC,EAAE,cAAY,CAC5D;AAAA,UAAA;AAAA,QAAA,GAGN;AAAA,QACA,qBAAC,qBAAA,EAAoB,OAAM,OAAM,WAAU,QACzC,UAAA;AAAA,UAAA,qBAAC,OAAA,EAAI,WAAU,qCACZ,UAAA;AAAA,aAAA,6BAAM,SACL;AAAA,cAAC;AAAA,cAAA;AAAA,gBACC,KAAK,KAAK;AAAA,gBACV,KAAK,KAAK,QAAQ;AAAA,gBAClB,WAAU;AAAA,cAAA;AAAA,YAAA,IAGZ,oBAAC,OAAA,EAAI,WAAU,6HACX,YAAA,6BAAM,UAAQ,6BAAM,UAAS,KAAK,OAAO,CAAC,EAAE,eAChD;AAAA,YAEF,qBAAC,OAAA,EAAI,WAAU,yBACb,UAAA;AAAA,cAAA,qBAAC,KAAA,EAAE,WAAU,0CAAyC,UAAA;AAAA,gBAAA;AAAA,kBAClD,kCAAM,UAAN,mBAAa,MAAM,KAAK,OAAM;AAAA,cAAA,GAClC;AAAA,kCACC,KAAA,EAAE,WAAU,gCACV,WAAA,6BAAM,SAAQ,OAAA,CACjB;AAAA,YAAA,EAAA,CACF;AAAA,UAAA,GACF;AAAA,8BACC,uBAAA,EAAsB;AAAA,UACvB,qBAAC,kBAAA,EAAiB,SAAS,cACzB,UAAA;AAAA,YAAA,oBAAC,QAAA,EAAO,WAAU,eAAA,CAAe;AAAA,YACjC,oBAAC,UAAK,UAAA,WAAA,CAAQ;AAAA,UAAA,EAAA,CAChB;AAAA,QAAA,EAAA,CACF;AAAA,MAAA,EAAA,CACF;AAAA,IAAA,EAAA,CAEJ;AAAA,EAAA,GACF;AAEJ;"}
+{"version":3,"file":"Header.js","sources":["../../../../../src/ui/components/header/Header.tsx"],"sourcesContent":["import type {\n  AppVersionInfo,\n  VersionInfo,\n} from '@igstack/app-catalog-backend-core'\nimport { Link } from '@tanstack/react-router'\nimport { useQuery } from '@tanstack/react-query'\nimport { LogOut, X } from 'lucide-react'\nimport React, { use } from 'react'\nimport AppCatalogLogo from '~/assets/app-catalog.svg?react'\nimport { useTRPC } from '~/api/infra/trpc'\nimport { ThemeSwitcher } from '~/components/ThemeSwitcher'\nimport { AppCatalogContext } from '~/modules/appCatalog/context/AppCatalogContext'\nimport { AttributionFooter } from '~/modules/appCatalog/ui/catalog/AttributionFooter'\nimport {\n  useAuth,\n  useAuthActions,\n  useIsAuthenticated,\n  useUser,\n} from '~/modules/auth'\nimport { useAuthModal } from '~/modules/auth/AuthModalContext'\nimport { Button } from '~/ui/button'\nimport {\n  DropdownMenu,\n  DropdownMenuContent,\n  DropdownMenuItem,\n  DropdownMenuSeparator,\n  DropdownMenuTrigger,\n} from '~/ui/dropdown-menu'\nimport {\n  Popover,\n  PopoverClose,\n  PopoverContent,\n  PopoverTrigger,\n} from '~/ui/popover'\nimport { Skeleton } from '~/ui/skeleton'\n\nfunction ShaSuffix({ version }: { version: VersionInfo }) {\n  if (!version.sha) return null\n  if (version.shaUrl) {\n    return (\n      <>\n        {' '}\n        <a\n          href={version.shaUrl}\n          target=\"_blank\"\n          rel=\"noopener noreferrer\"\n          className=\"text-muted-foreground hover:text-primary\"\n          onClick={(e) => e.stopPropagation()}\n        >\n          ({version.sha})\n        </a>\n      </>\n    )\n  }\n  return <span className=\"text-muted-foreground\"> ({version.sha})</span>\n}\n\nfunction VersionItem({\n  label,\n  version,\n}: {\n  label: string\n  version: VersionInfo\n}) {\n  return (\n    <div className=\"flex items-center justify-between gap-4 py-1\">\n      <span className=\"text-xs text-muted-foreground\">{label}</span>\n      <span className=\"text-sm font-medium text-right\">\n        {version.url ? (\n          <a\n            href={version.url}\n            target=\"_blank\"\n            rel=\"noopener noreferrer\"\n            className=\"text-primary hover:text-primary/80 transition-colors\"\n            onClick={(e) => e.stopPropagation()}\n          >\n            {version.displayName}\n          </a>\n        ) : (\n          version.displayName\n        )}\n        <ShaSuffix version={version} />\n      </span>\n    </div>\n  )\n}\n\nexport function VersionDisplay({ versions }: { versions: AppVersionInfo }) {\n  const isLocal =\n    versions.backend?.displayName === 'local' && !versions.coreVersion\n\n  if (isLocal) {\n    return (\n      <span className=\"text-xs text-muted-foreground\" title=\"Local\">\n        local\n      </span>\n    )\n  }\n\n  return (\n    <div className=\"flex flex-col divide-y\">\n      {versions.backend && (\n        <VersionItem label=\"Pipeline\" version={versions.backend} />\n      )}\n      {versions.coreVersion && (\n        <VersionItem label=\"Core\" version={versions.coreVersion} />\n      )}\n      {versions.frontend && (\n        <VersionItem label=\"FE\" version={versions.frontend} />\n      )}\n    </div>\n  )\n}\n\n/**\n * Version detail moved behind an on-demand popover (same click-to-reveal\n * pattern as PersonBadge's approver chips) instead of always-visible text —\n * keeps the header compact. Content is unchanged: same Pipeline/Core/FE\n * lines, links, and SHA suffix as before, just revealed on click.\n */\nexport function VersionPopover({ versions }: { versions: AppVersionInfo }) {\n  const isLocal =\n    versions.backend?.displayName === 'local' && !versions.coreVersion\n\n  if (isLocal) {\n    return (\n      <span className=\"text-xs text-muted-foreground\" title=\"Local\">\n        local\n      </span>\n    )\n  }\n\n  return (\n    <Popover>\n      <PopoverTrigger asChild>\n        <button\n          type=\"button\"\n          className=\"inline-flex items-center gap-1 rounded-full bg-muted px-2 py-0.5 text-[10px] font-mono leading-none text-muted-foreground hover:bg-primary/15 hover:text-primary transition-colors\"\n          aria-label=\"Show version details\"\n          title=\"Version details\"\n          data-testid=\"version-info-trigger\"\n        >\n          {versions.backend?.displayName ?? '?'}\n        </button>\n      </PopoverTrigger>\n      <PopoverContent align=\"start\" className=\"w-72 p-0 overflow-hidden\">\n        <div className=\"flex items-center justify-between px-3 py-2 border-b bg-muted/40\">\n          <span className=\"text-sm font-semibold\">Version details</span>\n          <PopoverClose\n            className=\"rounded p-1 text-muted-foreground hover:text-foreground hover:bg-muted transition-colors\"\n            aria-label=\"Close\"\n          >\n            <X className=\"size-3.5\" />\n          </PopoverClose>\n        </div>\n        <div className=\"px-3 py-1\">\n          <VersionDisplay versions={versions} />\n        </div>\n        <AttributionFooter variant=\"popover\" />\n      </PopoverContent>\n    </Popover>\n  )\n}\n\nexport interface HeaderProps {\n  middle?: React.ReactNode\n}\n\nexport function Header({ middle }: HeaderProps) {\n  const { isLoading } = useAuth()\n  const isAuthenticated = useIsAuthenticated()\n  const user = useUser()\n  const { logout, devLogin } = useAuthActions()\n  const { open: openLoginModal } = useAuthModal()\n  const appCatalogContextMaybe = use(AppCatalogContext)\n  const trpc = useTRPC()\n  const { data: providersData } = useQuery(\n    trpc.auth.getProviders.queryOptions(),\n  )\n\n  const handleLogout = async () => {\n    try {\n      await logout()\n    } catch (error) {\n      console.error('Logout failed:', error)\n    }\n  }\n\n  const handleLoginClick = () => {\n    // Preserve the current URL for redirect after login\n    const currentUrl = window.location.pathname + window.location.search\n    openLoginModal(currentUrl)\n  }\n\n  return (\n    <div className=\"flex items-center mb-4 pb-4 gap-4 border-b\">\n      <div className=\"flex items-center gap-4\">\n        <div className=\"flex items-center gap-2\">\n          <Link to=\"/\">\n            <AppCatalogLogo className=\"h-16 w-16\" />\n          </Link>\n          <div className=\"flex flex-col items-start\">\n            <Link to=\"/\" className=\"font-serif text-lg font-semibold\">\n              App Catalog\n            </Link>\n            {appCatalogContextMaybe?.versions && (\n              <VersionPopover versions={appCatalogContextMaybe.versions} />\n            )}\n          </div>\n        </div>\n      </div>\n      {middle && <div className=\"flex-1\">{middle}</div>}\n      <div className=\"flex items-center gap-3 ml-auto\">\n        <ThemeSwitcher />\n        {isLoading ? (\n          <Skeleton className=\"w-8 h-8 rounded-full\" />\n        ) : !isAuthenticated ? (\n          // The catalog is browsable without signing in; Login is for the parts\n          // that need an identity (comments, admin). DEV Login is gated behind\n          // `devLoginEnabled`, so it never reaches a real deployment.\n          <div className=\"flex items-center gap-2\">\n            {providersData?.devLoginEnabled && (\n              <button\n                type=\"button\"\n                onClick={devLogin}\n                className=\"text-xs font-medium px-2 py-1 rounded bg-amber-500/20 text-amber-600 dark:text-amber-400 hover:bg-amber-500/30 border border-amber-500/30 cursor-pointer\"\n              >\n                DEV Login\n              </button>\n            )}\n            <button\n              type=\"button\"\n              onClick={handleLoginClick}\n              className=\"text-sm font-medium text-muted-foreground hover:text-foreground cursor-pointer\"\n            >\n              Login\n            </button>\n          </div>\n        ) : (\n          <DropdownMenu>\n            <DropdownMenuTrigger asChild>\n              <Button\n                variant=\"ghost\"\n                size=\"sm\"\n                className=\"w-8 h-8 rounded-full p-0 hover:bg-accent hover:text-accent-foreground cursor-pointer\"\n                aria-label={`User menu for ${user?.name || 'User'}`}\n                data-testid=\"user-avatar-button\"\n              >\n                {user?.image ? (\n                  <img\n                    src={user.image}\n                    alt={user.name || 'User'}\n                    className=\"w-full h-full rounded-full object-cover\"\n                  />\n                ) : (\n                  <div className=\"flex items-center justify-center w-full h-full rounded-full bg-primary text-primary-foreground text-xs font-semibold\">\n                    {(user?.name || user?.email || '?').charAt(0).toUpperCase()}\n                  </div>\n                )}\n              </Button>\n            </DropdownMenuTrigger>\n            <DropdownMenuContent align=\"end\" className=\"w-56\">\n              <div className=\"px-2 py-3 flex items-center gap-3\">\n                {user?.image ? (\n                  <img\n                    src={user.image}\n                    alt={user.name || 'User'}\n                    className=\"w-10 h-10 rounded-full object-cover shrink-0\"\n                  />\n                ) : (\n                  <div className=\"flex items-center justify-center w-10 h-10 rounded-full bg-primary text-primary-foreground text-sm font-semibold shrink-0\">\n                    {(user?.name || user?.email || '?').charAt(0).toUpperCase()}\n                  </div>\n                )}\n                <div className=\"flex flex-col min-w-0\">\n                  <p className=\"text-xs text-muted-foreground truncate\">\n                    @{user?.email?.split('@')[0] || 'user'}\n                  </p>\n                  <p className=\"text-sm font-medium truncate\">\n                    {user?.name || 'User'}\n                  </p>\n                </div>\n              </div>\n              <DropdownMenuSeparator />\n              <DropdownMenuItem onClick={handleLogout}>\n                <LogOut className=\"h-4 w-4 mr-2\" />\n                <span>Sign out</span>\n              </DropdownMenuItem>\n            </DropdownMenuContent>\n          </DropdownMenu>\n        )}\n      </div>\n    </div>\n  )\n}\n"],"names":["AppCatalogLogo"],"mappings":";;;;;;;;;;;;;;;;;AAoCA,SAAS,UAAU,EAAE,WAAqC;AACxD,MAAI,CAAC,QAAQ,IAAK,QAAO;AACzB,MAAI,QAAQ,QAAQ;AAClB,WACE,qBAAA,UAAA,EACG,UAAA;AAAA,MAAA;AAAA,MACD;AAAA,QAAC;AAAA,QAAA;AAAA,UACC,MAAM,QAAQ;AAAA,UACd,QAAO;AAAA,UACP,KAAI;AAAA,UACJ,WAAU;AAAA,UACV,SAAS,CAAC,MAAM,EAAE,gBAAA;AAAA,UACnB,UAAA;AAAA,YAAA;AAAA,YACG,QAAQ;AAAA,YAAI;AAAA,UAAA;AAAA,QAAA;AAAA,MAAA;AAAA,IAChB,GACF;AAAA,EAEJ;AACA,SAAO,qBAAC,QAAA,EAAK,WAAU,yBAAwB,UAAA;AAAA,IAAA;AAAA,IAAG,QAAQ;AAAA,IAAI;AAAA,EAAA,GAAC;AACjE;AAEA,SAAS,YAAY;AAAA,EACnB;AAAA,EACA;AACF,GAGG;AACD,SACE,qBAAC,OAAA,EAAI,WAAU,gDACb,UAAA;AAAA,IAAA,oBAAC,QAAA,EAAK,WAAU,iCAAiC,UAAA,OAAM;AAAA,IACvD,qBAAC,QAAA,EAAK,WAAU,kCACb,UAAA;AAAA,MAAA,QAAQ,MACP;AAAA,QAAC;AAAA,QAAA;AAAA,UACC,MAAM,QAAQ;AAAA,UACd,QAAO;AAAA,UACP,KAAI;AAAA,UACJ,WAAU;AAAA,UACV,SAAS,CAAC,MAAM,EAAE,gBAAA;AAAA,UAEjB,UAAA,QAAQ;AAAA,QAAA;AAAA,MAAA,IAGX,QAAQ;AAAA,MAEV,oBAAC,aAAU,QAAA,CAAkB;AAAA,IAAA,EAAA,CAC/B;AAAA,EAAA,GACF;AAEJ;AAEO,SAAS,eAAe,EAAE,YAA0C;;AACzE,QAAM,YACJ,cAAS,YAAT,mBAAkB,iBAAgB,WAAW,CAAC,SAAS;AAEzD,MAAI,SAAS;AACX,+BACG,QAAA,EAAK,WAAU,iCAAgC,OAAM,SAAQ,UAAA,SAE9D;AAAA,EAEJ;AAEA,SACE,qBAAC,OAAA,EAAI,WAAU,0BACZ,UAAA;AAAA,IAAA,SAAS,WACR,oBAAC,aAAA,EAAY,OAAM,YAAW,SAAS,SAAS,SAAS;AAAA,IAE1D,SAAS,eACR,oBAAC,aAAA,EAAY,OAAM,QAAO,SAAS,SAAS,aAAa;AAAA,IAE1D,SAAS,YACR,oBAAC,aAAA,EAAY,OAAM,MAAK,SAAS,SAAS,SAAA,CAAU;AAAA,EAAA,GAExD;AAEJ;AAQO,SAAS,eAAe,EAAE,YAA0C;;AACzE,QAAM,YACJ,cAAS,YAAT,mBAAkB,iBAAgB,WAAW,CAAC,SAAS;AAEzD,MAAI,SAAS;AACX,+BACG,QAAA,EAAK,WAAU,iCAAgC,OAAM,SAAQ,UAAA,SAE9D;AAAA,EAEJ;AAEA,8BACG,SAAA,EACC,UAAA;AAAA,IAAA,oBAAC,gBAAA,EAAe,SAAO,MACrB,UAAA;AAAA,MAAC;AAAA,MAAA;AAAA,QACC,MAAK;AAAA,QACL,WAAU;AAAA,QACV,cAAW;AAAA,QACX,OAAM;AAAA,QACN,eAAY;AAAA,QAEX,YAAA,cAAS,YAAT,mBAAkB,gBAAe;AAAA,MAAA;AAAA,IAAA,GAEtC;AAAA,IACA,qBAAC,gBAAA,EAAe,OAAM,SAAQ,WAAU,4BACtC,UAAA;AAAA,MAAA,qBAAC,OAAA,EAAI,WAAU,oEACb,UAAA;AAAA,QAAA,oBAAC,QAAA,EAAK,WAAU,yBAAwB,UAAA,mBAAe;AAAA,QACvD;AAAA,UAAC;AAAA,UAAA;AAAA,YACC,WAAU;AAAA,YACV,cAAW;AAAA,YAEX,UAAA,oBAAC,GAAA,EAAE,WAAU,WAAA,CAAW;AAAA,UAAA;AAAA,QAAA;AAAA,MAC1B,GACF;AAAA,0BACC,OAAA,EAAI,WAAU,aACb,UAAA,oBAAC,gBAAA,EAAe,UAAoB,GACtC;AAAA,MACA,oBAAC,mBAAA,EAAkB,SAAQ,UAAA,CAAU;AAAA,IAAA,EAAA,CACvC;AAAA,EAAA,GACF;AAEJ;AAMO,SAAS,OAAO,EAAE,UAAuB;;AAC9C,QAAM,EAAE,UAAA,IAAc,QAAA;AACtB,QAAM,kBAAkB,mBAAA;AACxB,QAAM,OAAO,QAAA;AACb,QAAM,EAAE,QAAQ,SAAA,IAAa,eAAA;AAC7B,QAAM,EAAE,MAAM,eAAA,IAAmB,aAAA;AACjC,QAAM,yBAAyB,IAAI,iBAAiB;AACpD,QAAM,OAAO,QAAA;AACb,QAAM,EAAE,MAAM,cAAA,IAAkB;AAAA,IAC9B,KAAK,KAAK,aAAa,aAAA;AAAA,EAAa;AAGtC,QAAM,eAAe,YAAY;AAC/B,QAAI;AACF,YAAM,OAAA;AAAA,IACR,SAAS,OAAO;AACd,cAAQ,MAAM,kBAAkB,KAAK;AAAA,IACvC;AAAA,EACF;AAEA,QAAM,mBAAmB,MAAM;AAE7B,UAAM,aAAa,OAAO,SAAS,WAAW,OAAO,SAAS;AAC9D,mBAAe,UAAU;AAAA,EAC3B;AAEA,SACE,qBAAC,OAAA,EAAI,WAAU,8CACb,UAAA;AAAA,IAAA,oBAAC,SAAI,WAAU,2BACb,UAAA,qBAAC,OAAA,EAAI,WAAU,2BACb,UAAA;AAAA,MAAA,oBAAC,QAAK,IAAG,KACP,8BAACA,eAAA,EAAe,WAAU,aAAY,EAAA,CACxC;AAAA,MACA,qBAAC,OAAA,EAAI,WAAU,6BACb,UAAA;AAAA,QAAA,oBAAC,MAAA,EAAK,IAAG,KAAI,WAAU,oCAAmC,UAAA,eAE1D;AAAA,SACC,iEAAwB,aACvB,oBAAC,gBAAA,EAAe,UAAU,uBAAuB,SAAA,CAAU;AAAA,MAAA,EAAA,CAE/D;AAAA,IAAA,EAAA,CACF,EAAA,CACF;AAAA,IACC,UAAU,oBAAC,OAAA,EAAI,WAAU,UAAU,UAAA,QAAO;AAAA,IAC3C,qBAAC,OAAA,EAAI,WAAU,mCACb,UAAA;AAAA,MAAA,oBAAC,eAAA,EAAc;AAAA,MACd,YACC,oBAAC,UAAA,EAAS,WAAU,uBAAA,CAAuB,IACzC,CAAC;AAAA;AAAA;AAAA;AAAA,QAIH,qBAAC,OAAA,EAAI,WAAU,2BACZ,UAAA;AAAA,WAAA,+CAAe,oBACd;AAAA,YAAC;AAAA,YAAA;AAAA,cACC,MAAK;AAAA,cACL,SAAS;AAAA,cACT,WAAU;AAAA,cACX,UAAA;AAAA,YAAA;AAAA,UAAA;AAAA,UAIH;AAAA,YAAC;AAAA,YAAA;AAAA,cACC,MAAK;AAAA,cACL,SAAS;AAAA,cACT,WAAU;AAAA,cACX,UAAA;AAAA,YAAA;AAAA,UAAA;AAAA,QAED,EAAA,CACF;AAAA,+BAEC,cAAA,EACC,UAAA;AAAA,QAAA,oBAAC,qBAAA,EAAoB,SAAO,MAC1B,UAAA;AAAA,UAAC;AAAA,UAAA;AAAA,YACC,SAAQ;AAAA,YACR,MAAK;AAAA,YACL,WAAU;AAAA,YACV,cAAY,kBAAiB,6BAAM,SAAQ,MAAM;AAAA,YACjD,eAAY;AAAA,YAEX,wCAAM,SACL;AAAA,cAAC;AAAA,cAAA;AAAA,gBACC,KAAK,KAAK;AAAA,gBACV,KAAK,KAAK,QAAQ;AAAA,gBAClB,WAAU;AAAA,cAAA;AAAA,YAAA,IAGZ,oBAAC,OAAA,EAAI,WAAU,wHACX,YAAA,6BAAM,UAAQ,6BAAM,UAAS,KAAK,OAAO,CAAC,EAAE,cAAY,CAC5D;AAAA,UAAA;AAAA,QAAA,GAGN;AAAA,QACA,qBAAC,qBAAA,EAAoB,OAAM,OAAM,WAAU,QACzC,UAAA;AAAA,UAAA,qBAAC,OAAA,EAAI,WAAU,qCACZ,UAAA;AAAA,aAAA,6BAAM,SACL;AAAA,cAAC;AAAA,cAAA;AAAA,gBACC,KAAK,KAAK;AAAA,gBACV,KAAK,KAAK,QAAQ;AAAA,gBAClB,WAAU;AAAA,cAAA;AAAA,YAAA,IAGZ,oBAAC,OAAA,EAAI,WAAU,6HACX,YAAA,6BAAM,UAAQ,6BAAM,UAAS,KAAK,OAAO,CAAC,EAAE,eAChD;AAAA,YAEF,qBAAC,OAAA,EAAI,WAAU,yBACb,UAAA;AAAA,cAAA,qBAAC,KAAA,EAAE,WAAU,0CAAyC,UAAA;AAAA,gBAAA;AAAA,kBAClD,kCAAM,UAAN,mBAAa,MAAM,KAAK,OAAM;AAAA,cAAA,GAClC;AAAA,kCACC,KAAA,EAAE,WAAU,gCACV,WAAA,6BAAM,SAAQ,OAAA,CACjB;AAAA,YAAA,EAAA,CACF;AAAA,UAAA,GACF;AAAA,8BACC,uBAAA,EAAsB;AAAA,UACvB,qBAAC,kBAAA,EAAiB,SAAS,cACzB,UAAA;AAAA,YAAA,oBAAC,QAAA,EAAO,WAAU,eAAA,CAAe;AAAA,YACjC,oBAAC,UAAK,UAAA,WAAA,CAAQ;AAAA,UAAA,EAAA,CAChB;AAAA,QAAA,EAAA,CACF;AAAA,MAAA,EAAA,CACF;AAAA,IAAA,EAAA,CAEJ;AAAA,EAAA,GACF;AAEJ;"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@igstack/app-catalog-frontend-core",
-  "version": "6.8.1",
+  "version": "6.8.2",
   "description": "Frontend core library for App Catalog",
   "homepage": "https://github.com/lislon/app-catalog",
   "repository": {
@@ -118,7 +118,7 @@
     "tailwindcss": "^4.1.7",
     "tiptap-markdown": "^0.8.0",
     "tslib": "^2.3.0",
-    "@igstack/app-catalog-shared-core": "6.8.1"
+    "@igstack/app-catalog-shared-core": "6.8.2"
   },
   "devDependencies": {
     "@tailwindcss/vite": "^4.0.5",
@@ -146,7 +146,7 @@
     "vite-plugin-static-copy": "^3.1.4",
     "vite-plugin-svgr": "^4.2.0",
     "vitest": "^4.1.2",
-    "@igstack/app-catalog-backend-core": "6.8.1"
+    "@igstack/app-catalog-backend-core": "6.8.2"
   },
   "peerDependencies": {
     "@tanstack/react-router": "1.170.41",
@@ -156,7 +156,7 @@
     "vite": "^6.3.5",
     "vite-plugin-svgr": "^4.2.0"
   },
-  "gitHead": "e850b173b27ae74c0935d72db50359d1ad036fa2",
+  "gitHead": "622217f4de5765e49a41d89c2e87592dccad8a23",
   "scripts": {
     "build": "vite build",
     "build:lenient": "vite build --mode lenient",

package/src/__tests__/ui/components/header/{Header.loginHidden.test.tsx → Header.login.test.tsx}

@@ -1,10 +1,11 @@
 import { render, screen } from '@testing-library/react'
+import userEvent from '@testing-library/user-event'
 import { beforeEach, describe, expect, it, vi } from 'vitest'
 import '@testing-library/jest-dom/vitest'
 
 // The header pulls in the router, the theme switcher, an svgr asset and the
 // whole auth/tRPC stack. None of that is under test here, so it is stubbed down
-// to the two things these cases drive: session state and `devLoginEnabled`.
+// to what these cases drive: session state, `devLoginEnabled` and the modal.
 const state = vi.hoisted(() => ({
   isLoading: false,
   isAuthenticated: false,
@@ -37,6 +38,10 @@ vi.mock('~/modules/auth', () => ({
   useUser: vi.fn(() => state.user),
   useAuthActions: vi.fn(() => ({ logout: vi.fn(), devLogin: vi.fn() })),
 }))
+const openLoginModal = vi.hoisted(() => vi.fn())
+vi.mock('~/modules/auth/AuthModalContext', () => ({
+  useAuthModal: vi.fn(() => ({ open: openLoginModal })),
+}))
 
 const { Header } = await import('~/ui/components/header/Header')
 
@@ -45,15 +50,20 @@ beforeEach(() => {
   state.isAuthenticated = false
   state.devLoginEnabled = false
   state.user = null
+  openLoginModal.mockClear()
 })
 
-describe('Header login affordance', () => {
-  it('offers an anonymous visitor no way into the login flow', () => {
+describe('Header login button', () => {
+  it('offers an anonymous visitor a Login button that opens the sign-in modal', async () => {
+    window.history.pushState({}, '', '/app/sample-review-app?tab=access')
     render(<Header />)
 
-    // Catches a plain "Login" and any other login-ish control alike.
-    expect(screen.queryByRole('button', { name: /login/i })).toBeNull()
-    expect(screen.queryByText('Login')).not.toBeInTheDocument()
+    await userEvent.click(screen.getByRole('button', { name: 'Login' }))
+
+    // Comes back to where the visitor was after signing in.
+    expect(openLoginModal).toHaveBeenCalledWith(
+      '/app/sample-review-app?tab=access',
+    )
   })
 
   // Deliberately kept: already gated behind `devLoginEnabled`, so it never
@@ -65,10 +75,10 @@ describe('Header login affordance', () => {
     expect(
       screen.getByRole('button', { name: 'DEV Login' }),
     ).toBeInTheDocument()
-    expect(screen.queryByText('Login')).not.toBeInTheDocument()
+    expect(screen.getByRole('button', { name: 'Login' })).toBeInTheDocument()
   })
 
-  it('leaves no empty wrapper where the button used to be', () => {
+  it('leaves no empty flex child in the header row', () => {
     const { container } = render(<Header />)
 
     // An empty flex child still eats the parent's `gap-3`, which is the
@@ -85,5 +95,6 @@ describe('Header login affordance', () => {
     render(<Header />)
 
     expect(screen.getByTestId('user-avatar-button')).toBeInTheDocument()
+    expect(screen.queryByRole('button', { name: 'Login' })).toBeNull()
   })
 })

package/src/ui/components/header/Header.tsx

@@ -17,6 +17,7 @@ import {
   useIsAuthenticated,
   useUser,
 } from '~/modules/auth'
+import { useAuthModal } from '~/modules/auth/AuthModalContext'
 import { Button } from '~/ui/button'
 import {
   DropdownMenu,
@@ -170,6 +171,7 @@ export function Header({ middle }: HeaderProps) {
   const isAuthenticated = useIsAuthenticated()
   const user = useUser()
   const { logout, devLogin } = useAuthActions()
+  const { open: openLoginModal } = useAuthModal()
   const appCatalogContextMaybe = use(AppCatalogContext)
   const trpc = useTRPC()
   const { data: providersData } = useQuery(
@@ -184,6 +186,12 @@ export function Header({ middle }: HeaderProps) {
     }
   }
 
+  const handleLoginClick = () => {
+    // Preserve the current URL for redirect after login
+    const currentUrl = window.location.pathname + window.location.search
+    openLoginModal(currentUrl)
+  }
+
   return (
     <div className="flex items-center mb-4 pb-4 gap-4 border-b">
       <div className="flex items-center gap-4">
@@ -207,24 +215,27 @@ export function Header({ middle }: HeaderProps) {
         {isLoading ? (
           <Skeleton className="w-8 h-8 rounded-full" />
         ) : !isAuthenticated ? (
-          // No Login button: the catalog is fully browsable without signing in,
-          // so it mostly advertised a flow most visitors have no reason to
-          // enter. Auth itself is untouched -- the modal, the login route, the
-          // client and session handling all still work, so anyone who reaches
-          // the flow by URL or by calling `useAuthModal().open()` can sign in,
-          // and this branch flips to the user menu below once they do. `git
-          // revert` of the commit that removed it restores the button as-is.
-          // DEV Login stays: it is gated behind `devLoginEnabled`, so it never
-          // reaches a real deployment.
-          providersData?.devLoginEnabled && (
+          // The catalog is browsable without signing in; Login is for the parts
+          // that need an identity (comments, admin). DEV Login is gated behind
+          // `devLoginEnabled`, so it never reaches a real deployment.
+          <div className="flex items-center gap-2">
+            {providersData?.devLoginEnabled && (
+              <button
+                type="button"
+                onClick={devLogin}
+                className="text-xs font-medium px-2 py-1 rounded bg-amber-500/20 text-amber-600 dark:text-amber-400 hover:bg-amber-500/30 border border-amber-500/30 cursor-pointer"
+              >
+                DEV Login
+              </button>
+            )}
             <button
               type="button"
-              onClick={devLogin}
-              className="text-xs font-medium px-2 py-1 rounded bg-amber-500/20 text-amber-600 dark:text-amber-400 hover:bg-amber-500/30 border border-amber-500/30 cursor-pointer"
+              onClick={handleLoginClick}
+              className="text-sm font-medium text-muted-foreground hover:text-foreground cursor-pointer"
             >
-              DEV Login
+              Login
             </button>
-          )
+          </div>
         ) : (
           <DropdownMenu>
             <DropdownMenuTrigger asChild>