Merge pull request 'fix(memory/pgplugin): restore idx++ in PatchNamespace (OFFSEC-004)' (#832) from fix/offsec-004-patchnamespace-idx into main

Commit ad7acd30 removed this increment as a golangci-lint false-positive
("unused variable: idx") — idx is used in the query string built by
fmt.Sprintf, so the lint was wrong. The removal broke the dual-field
case: when both ExpiresAt and Metadata are set, the query uses \$3 for
metadata but args only has 3 elements (indices 0=name, 1=expires, 2=metadata),
so \$3 is out-of-bounds or reads the wrong value.

Fix: restore idx++ after the metadata args append.

Test: add TestStore_PatchNamespace_DualFields — covers the previously
untested case where both expires_at and metadata are patched in one call.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

canvas/src/components/SearchDialog.tsx

@@ -91,16 +91,19 @@ export function SearchDialog() {
   if (!open) return null;
 
   return (
-    <div
-      className="fixed inset-0 z-[70] flex items-start justify-center pt-[20vh] bg-black/50 backdrop-blur-sm"
-      onClick={() => setOpen(false)}
-    >
+    <div className="fixed inset-0 z-[70] flex items-start justify-center pt-[20vh]">
+      {/* Backdrop — interactive dismiss area; aria-hidden so screen readers ignore it */}
+      <div
+        className="absolute inset-0 bg-black/50 backdrop-blur-sm cursor-pointer"
+        onClick={() => setOpen(false)}
+        aria-hidden="true"
+      />
+      {/* Dialog */}
       <div
         role="dialog"
         aria-modal="true"
         aria-label="Search workspaces"
-        className="w-[420px] bg-surface/95 backdrop-blur-xl border border-line/60 rounded-2xl shadow-2xl shadow-black/50 overflow-hidden"
-        onClick={(e) => e.stopPropagation()}
+        className="relative z-[71] w-[420px] bg-surface/95 backdrop-blur-xl border border-line/60 rounded-2xl shadow-2xl shadow-black/50 overflow-hidden"
       >
         {/* Search input */}
         <div className="flex items-center gap-3 px-4 py-3 border-b border-line/40">

canvas/src/components/__tests__/OrgTemplatesSection.test.tsx

@@ -1,237 +1,102 @@
 // @vitest-environment jsdom
+import { describe, it, expect, vi, beforeEach, afterEach } from "vitest";
+import { render, screen, waitFor, fireEvent, cleanup } from "@testing-library/react";
 
-/**
- * Tests for OrgTemplatesSection — collapsible org template import list.
- *
- * Covers:
- *   - Header with count badge (visible only when expanded)
- *   - Collapsed by default, aria-expanded toggles on click
- *   - aria-controls targets org-templates-body div
- *   - Empty state when no org templates
- *   - Loading spinner
- *   - Org template cards: name, description, workspace count
- *   - Import button per card
- *   - Preflight modal opens when org has required_env
- *   - Preflight onProceed fires import
- *   - Preflight onCancel closes modal
- *   - Direct import (no modal) when org has no env requirements
- *   - Import button disabled while that org is importing
- */
-// ── ALL mocks MUST be before imports (vi.mock is hoisted to top of file) ───────
-const { mockGet, mockPost, mockListSecrets } = vi.hoisted(() => ({
-  mockGet: vi.fn(),
-  mockPost: vi.fn(),
-  mockListSecrets: vi.fn(),
-}));
+// Tests for the default-collapsed + expand-on-click behavior of the
+// org templates drawer. Before this change the section rendered all
+// org cards inline, which pushed the individual workspace templates
+// off-screen when there were ≥3 orgs on disk. Collapsed-by-default
+// keeps the scroll focused on the primary deploy path.
 
 vi.mock("@/lib/api", () => ({
-  api: { get: mockGet, post: mockPost },
-}));
-
-vi.mock("@/lib/api/secrets", () => ({
-  listSecrets: mockListSecrets,
-}));
-
-vi.mock("@/store/canvas", () => ({
-  useCanvasStore: Object.assign(
-    vi.fn(),
-    { getState: () => ({ nodes: [], hydrate: vi.fn() }) },
-  ),
-}));
-
-vi.mock("../Spinner", () => ({
-  Spinner: () => <span data-testid="spinner" aria-hidden="true" />,
-}));
-
-vi.mock("../OrgImportPreflightModal", () => ({
-  OrgImportPreflightModal: vi.fn(({ open, onCancel, onProceed }) =>
-    open ? (
-      <div data-testid="preflight-modal">
-        <button onClick={onProceed}>Import</button>
-        <button onClick={onCancel}>Cancel</button>
-      </div>
-    ) : null
-  ),
+  api: {
+    get: vi.fn().mockResolvedValue([
+      { dir: "free-beats-all", name: "Free Beats All", description: "d1", workspaces: 3 },
+      { dir: "medo-smoke", name: "MeDo Smoke Test", description: "d2", workspaces: 1 },
+    ]),
+    post: vi.fn().mockResolvedValue({}),
+  },
 }));
 
+vi.mock("../Spinner", () => ({ Spinner: () => null }));
+vi.mock("../MissingKeysModal", () => ({ MissingKeysModal: () => null }));
 vi.mock("../ConfirmDialog", () => ({ ConfirmDialog: () => null }));
-vi.mock("@/components/Toaster", () => ({ showToast: vi.fn() }));
+vi.mock("@/lib/deploy-preflight", () => ({ checkDeploySecrets: vi.fn() }));
 
-import React from "react";
-import { render, screen, fireEvent, cleanup, act, waitFor } from "@testing-library/react";
-import { afterEach, beforeEach, describe, expect, it, vi } from "vitest";
 import { OrgTemplatesSection } from "../TemplatePalette";
 
-// ── Shared data ─────────────────────────────────────────────────────────────
-const MOCK_ORGS = [
-  { dir: "free-beats-all", name: "Free Beats All", description: "d1", workspaces: 3 },
-  { dir: "medo-smoke", name: "MeDo Smoke Test", description: "d2", workspaces: 1 },
-];
-
 beforeEach(() => {
   vi.clearAllMocks();
-  mockGet.mockResolvedValue(MOCK_ORGS);
-  mockPost.mockResolvedValue({ org: "test", workspaces: [], count: 0 });
-  mockListSecrets.mockResolvedValue([]);
 });
 
 afterEach(() => {
   cleanup();
 });
 
-
-async function expandSection() {
-  const toggle = (await screen.findAllByRole("button")).find(
-    (b) => b.getAttribute("aria-controls") === "org-templates-body"
-  )!;
-  fireEvent.click(toggle);
-  await waitFor(() => {
-    expect(toggle.getAttribute("aria-expanded")).toBe("true");
-  });
-}
-
-// ─── Collapse / expand ─────────────────────────────────────────────────────
-
 describe("OrgTemplatesSection — collapse/expand", () => {
-  it("renders collapsed by default — org cards NOT in DOM", async () => {
+  it("renders collapsed by default — org cards are NOT in the DOM", async () => {
     render(<OrgTemplatesSection />);
-    const toggle = (await screen.findAllByRole("button")).find(
-      (b) => b.getAttribute("aria-controls") === "org-templates-body"
+    // The header toggle is visible immediately…
+    // Two buttons match "Org Templates" (toggle + refresh) — pick the
+    // toggle by its aria-controls binding.
+    const toggle = (await screen.findAllByRole("button")).find((b) =>
+      b.getAttribute("aria-controls") === "org-templates-body"
     )!;
+    expect(toggle).toBeTruthy();
     expect(toggle.getAttribute("aria-expanded")).toBe("false");
+
+    // …and the count appears after loadOrgs resolves.
     await waitFor(() => {
       expect(toggle.textContent).toContain("(2)");
     });
+
+    // But none of the individual org cards should be rendered yet.
     expect(screen.queryByText("Free Beats All")).toBeNull();
+    expect(screen.queryByText("MeDo Smoke Test")).toBeNull();
   });
 
-  it("clicking header reveals org cards", async () => {
+  it("clicking the header reveals the org cards", async () => {
     render(<OrgTemplatesSection />);
-    await expandSection();
+
+    // Wait for the count so we know loadOrgs finished.
+    // Two buttons match "Org Templates" (toggle + refresh) — pick the
+    // toggle by its aria-controls binding.
+    const toggle = (await screen.findAllByRole("button")).find((b) =>
+      b.getAttribute("aria-controls") === "org-templates-body"
+    )!;
+    await waitFor(() => {
+      expect(toggle.textContent).toContain("(2)");
+    });
+
+    // Expand.
+    fireEvent.click(toggle);
+    await waitFor(() => {
+      expect(toggle.getAttribute("aria-expanded")).toBe("true");
+    });
+
+    // Org cards now visible.
     expect(screen.getByText("Free Beats All")).toBeTruthy();
     expect(screen.getByText("MeDo Smoke Test")).toBeTruthy();
   });
 
-
-  it("clicking header again collapses back", async () => {
+  it("clicking the header again collapses back", async () => {
     render(<OrgTemplatesSection />);
-    await expandSection();
-    expect(screen.getByText("Free Beats All")).toBeTruthy();
-    const toggle = (await screen.findAllByRole("button")).find(
-      (b) => b.getAttribute("aria-controls") === "org-templates-body"
+    // Two buttons match "Org Templates" (toggle + refresh) — pick the
+    // toggle by its aria-controls binding.
+    const toggle = (await screen.findAllByRole("button")).find((b) =>
+      b.getAttribute("aria-controls") === "org-templates-body"
     )!;
-    fireEvent.click(toggle);
+    await waitFor(() => {
+      expect(toggle.textContent).toContain("(2)");
+    });
+
+    fireEvent.click(toggle); // expand
+    expect(screen.getByText("Free Beats All")).toBeTruthy();
+
+    fireEvent.click(toggle); // collapse
     await waitFor(() => {
       expect(toggle.getAttribute("aria-expanded")).toBe("false");
     });
     expect(screen.queryByText("Free Beats All")).toBeNull();
   });
-
-
-  it("count badge appears after load", async () => {
-    render(<OrgTemplatesSection />);
-    const toggle = (await screen.findAllByRole("button")).find(
-      (b) => b.getAttribute("aria-controls") === "org-templates-body"
-    )!;
-    await waitFor(() => {
-      expect(toggle.textContent).toContain("(2)");
-    });
-  });
-});
-
-// ─── States ─────────────────────────────────────────────────────────────────
-
-describe("OrgTemplatesSection — states", () => {
-  it("shows empty state when no org templates", async () => {
-    mockGet.mockResolvedValue([]);
-    render(<OrgTemplatesSection />);
-    await expandSection();
-    expect(screen.getByText(/no org templates/i)).toBeTruthy();
-    expect(screen.getByText(/org-templates\//i)).toBeTruthy();
-  });
-
-  it("shows loading spinner while fetching", async () => {
-    mockGet.mockImplementation(() => new Promise(() => {}));
-    render(<OrgTemplatesSection />);
-    await expandSection();
-    expect(screen.getByTestId("spinner")).toBeTruthy();
-    expect(screen.getByText(/loading/i)).toBeTruthy();
-  });
-
-  it("shows workspace count badge on org card", async () => {
-    render(<OrgTemplatesSection />);
-    await expandSection();
-    expect(screen.getByText(/3 workspaces/i)).toBeTruthy();
-  });
-
-  it("shows org description on card", async () => {
-    render(<OrgTemplatesSection />);
-    await expandSection();
-    expect(screen.getByText("d1")).toBeTruthy();
-  });
-});
-
-// ─── Import ─────────────────────────────────────────────────────────────────
-
-describe("OrgTemplatesSection — import", () => {
-  it("Import button is present for each org", async () => {
-    render(<OrgTemplatesSection />);
-    await expandSection();
-    const importBtns = screen.getAllByRole("button", { name: /import org/i });
-    expect(importBtns.length).toBe(2);
-  });
-
-  it("preflight modal opens when org has required_env", async () => {
-    mockGet.mockResolvedValue([
-      { ...MOCK_ORGS[0], required_env: [{ key: "ANTHROPIC_API_KEY" }] },
-    ]);
-    render(<OrgTemplatesSection />);
-    await expandSection();
-    fireEvent.click(screen.getAllByRole("button", { name: /import org/i })[0]);
-    await waitFor(() => {
-      expect(screen.getByTestId("preflight-modal")).toBeTruthy();
-    });
-  });
-
-  it("preflight onCancel closes the modal", async () => {
-    mockGet.mockResolvedValue([
-      { ...MOCK_ORGS[0], required_env: [{ key: "STRIPE_KEY" }] },
-    ]);
-    render(<OrgTemplatesSection />);
-    await expandSection();
-    fireEvent.click(screen.getAllByRole("button", { name: /import org/i })[0]);
-    await waitFor(() => {
-      expect(screen.getByTestId("preflight-modal")).toBeTruthy();
-    });
-    await act(async () => {
-      screen.getByRole("button", { name: "Cancel" }).click();
-    });
-    await waitFor(() => {
-      expect(screen.queryByTestId("preflight-modal")).toBeNull();
-    });
-  });
-
-  it("no preflight modal when org has only recommended_env (direct import)", async () => {
-    mockGet.mockResolvedValue([
-      { ...MOCK_ORGS[0], required_env: [], recommended_env: [{ key: "OPTIONAL" }] },
-    ]);
-    render(<OrgTemplatesSection />);
-    await expandSection();
-    fireEvent.click(screen.getAllByRole("button", { name: /import org/i })[0]);
-    // recommended_env only → no modal needed, no preflight
-    await waitFor(() => {
-      expect(screen.queryByTestId("preflight-modal")).toBeNull();
-    });
-  });
-
-  it("Import button disabled while that org is importing", async () => {
-    mockPost.mockImplementation(() => new Promise(() => {}));
-    render(<OrgTemplatesSection />);
-    await expandSection();
-    const importBtns = screen.getAllByRole("button", { name: /import org/i });
-    fireEvent.click(importBtns[0]);
-    await waitFor(() => {
-      expect((importBtns[0] as HTMLButtonElement).disabled).toBe(true);
-    });
-  });
 });

canvas/src/components/canvas/__tests__/useKeyboardShortcuts.test.tsx

@@ -101,6 +101,20 @@ describe("Esc — deselect / close context menu", () => {
     fireEvent.keyDown(window, { key: "Escape" });
     expect(mockStoreState.selectNode).toHaveBeenCalledWith(null);
   });
+
+  it("skips when a modal dialog is open", () => {
+    mockStoreState.contextMenu = null;
+    mockStoreState.selectedNodeId = "n1";
+    renderWithProvider();
+    const dialog = document.createElement("div");
+    dialog.setAttribute("role", "dialog");
+    dialog.setAttribute("aria-modal", "true");
+    document.body.appendChild(dialog);
+    fireEvent.keyDown(window, { key: "Escape" });
+    expect(mockStoreState.clearSelection).not.toHaveBeenCalled();
+    expect(mockStoreState.selectNode).not.toHaveBeenCalled();
+    document.body.removeChild(dialog);
+  });
 });
 
 describe("Enter — hierarchy navigation", () => {
@@ -136,6 +150,17 @@ describe("Enter — hierarchy navigation", () => {
     fireEvent.keyDown(window, { key: "Enter" });
     expect(mockStoreState.selectNode).not.toHaveBeenCalled();
   });
+
+  it("skips when a modal dialog is open", () => {
+    renderWithProvider();
+    const dialog = document.createElement("div");
+    dialog.setAttribute("role", "dialog");
+    dialog.setAttribute("aria-modal", "true");
+    document.body.appendChild(dialog);
+    fireEvent.keyDown(window, { key: "Enter" });
+    expect(mockStoreState.selectNode).not.toHaveBeenCalled();
+    document.body.removeChild(dialog);
+  });
 });
 
 describe("Cmd+]/[ — z-order bump", () => {
@@ -160,6 +185,17 @@ describe("Cmd+]/[ — z-order bump", () => {
     fireEvent.keyDown(window, { key: "]", ctrlKey: true });
     expect(mockStoreState.bumpZOrder).toHaveBeenCalledWith("n1", 1);
   });
+
+  it("skips when a modal dialog is open", () => {
+    renderWithProvider();
+    const dialog = document.createElement("div");
+    dialog.setAttribute("role", "dialog");
+    dialog.setAttribute("aria-modal", "true");
+    document.body.appendChild(dialog);
+    fireEvent.keyDown(window, { key: "]", metaKey: true });
+    expect(mockStoreState.bumpZOrder).not.toHaveBeenCalled();
+    document.body.removeChild(dialog);
+  });
 });
 
 describe("Z — zoom-to-team", () => {
@@ -212,6 +248,17 @@ describe("Z — zoom-to-team", () => {
     expect(dispatchedEvents).toHaveLength(0);
     document.body.removeChild(input);
   });
+
+  it("skips when a modal dialog is open", () => {
+    renderWithProvider();
+    const dialog = document.createElement("div");
+    dialog.setAttribute("role", "dialog");
+    dialog.setAttribute("aria-modal", "true");
+    document.body.appendChild(dialog);
+    fireEvent.keyDown(window, { key: "z" });
+    expect(dispatchedEvents).toHaveLength(0);
+    document.body.removeChild(dialog);
+  });
 });
 
 describe("Arrow keys — keyboard node movement", () => {

canvas/src/components/canvas/useKeyboardShortcuts.ts

@@ -13,7 +13,9 @@ function hasChildren(nodeId: string, nodes: Node<WorkspaceNodeData>[]): boolean
 /**
  * Canvas-wide keyboard shortcuts. All bound to the document window so
  * they work regardless of focused node, except when the user is typing
- * into an input (`inInput` short-circuits handling).
+ * into an input (`inInput` short-circuits handling) or a modal dialog is
+ * open (`isModalOpen` short-circuits handling — dialogs own their own
+ * keyboard semantics and take precedence).
  *
  *   Esc                  — close context menu, clear selection, deselect
  *   Enter                — descend into selected node's first child
@@ -25,6 +27,10 @@ function hasChildren(nodeId: string, nodes: Node<WorkspaceNodeData>[]): boolean
  *   Cmd/Ctrl+Arrow       — resize selected node (↑↓ height, ←→ width)
  *   Cmd/Ctrl+Shift+Arrow — resize by 2px per press (fine control)
  */
+/** Returns true when a modal dialog (role=dialog, aria-modal=true) is open. */
+const isModalOpen = () =>
+  document.querySelector('[role="dialog"][aria-modal="true"]') !== null;
+
 export function useKeyboardShortcuts() {
   useEffect(() => {
     const handler = (e: KeyboardEvent) => {
@@ -36,6 +42,7 @@ export function useKeyboardShortcuts() {
         (e.target as HTMLElement).isContentEditable;
 
       if (e.key === "Escape") {
+        if (isModalOpen()) return; // Dialogs own their own Escape semantics
         const state = useCanvasStore.getState();
         if (state.contextMenu) {
           state.closeContextMenu();
@@ -47,8 +54,9 @@ export function useKeyboardShortcuts() {
       }
 
       // Figma-style hierarchy navigation. Skipped when the user is
-      // typing so Enter can still submit forms.
-      if (!inInput && (e.key === "Enter" || e.key === "NumpadEnter")) {
+      // typing so Enter can still submit forms, and when a dialog is open
+      // so the dialog can use Enter for its own actions.
+      if (!inInput && !isModalOpen() && (e.key === "Enter" || e.key === "NumpadEnter")) {
         e.preventDefault();
         const state = useCanvasStore.getState();
         const id = state.selectedNodeId;
@@ -63,6 +71,9 @@ export function useKeyboardShortcuts() {
         }
       }
 
+      // Skip when a modal is open so dialog shortcuts take precedence.
+      if (isModalOpen()) return;
+
       if (
         !inInput &&
         (e.metaKey || e.ctrlKey) &&
@@ -111,7 +122,7 @@ export function useKeyboardShortcuts() {
         if (!selectedId) return;
         // Skip when a modal/dialog is already open — dialogs own their own
         // arrow-key semantics and shouldn't trigger canvas moves.
-        if (document.querySelector('[role="dialog"][aria-modal="true"]')) return;
+        if (isModalOpen()) return;
         e.preventDefault();
         const step = e.shiftKey ? 50 : 10;
         let dx = 0;
@@ -138,7 +149,7 @@ export function useKeyboardShortcuts() {
         const state = useCanvasStore.getState();
         const selectedId = state.selectedNodeId;
         if (!selectedId) return;
-        if (document.querySelector('[role="dialog"][aria-modal="true"]')) return;
+        if (isModalOpen()) return;
         e.preventDefault();
         const step = e.shiftKey ? 2 : 10;
         const node = state.nodes.find((n) => n.id === selectedId);

canvas/src/components/mobile/MobileChat.tsx

@@ -54,9 +54,11 @@ export function MobileChat({
   // user sees their prior thread on entry. The store is updated by the
   // socket → ChatTab flows the desktop runs; on mobile we read from the
   // same buffer to keep state coherent across viewports.
-  // NOTE: selector returns undefined (stable) — do NOT use ?? [] here,
-  // that creates a new [] reference on every store update when the key is
-  // absent, causing infinite re-render (React error #185).
+  // NOTE: do NOT use `?? []` in the selector — Zustand uses Object.is
+  // for selector equality. A fallback `?? []` creates a new [] reference on
+  // every store update when agentMessages[agentId] is undefined, causing an
+  // infinite re-render loop (React error #185 / Maximum update depth
+  // exceeded). The undefined case is handled by the initializer below.
   const storedMessages = useCanvasStore((s) => s.agentMessages[agentId]);
   const [messages, setMessages] = useState<ChatMessage[]>(() =>
     (storedMessages ?? []).map((m) => ({

canvas/src/components/settings/UnsavedChangesGuard.tsx

@@ -16,11 +16,6 @@ interface UnsavedChangesGuardProps {
  * - Shown when closing panel while a form has unsaved input
  * - NOT shown if the form is empty (opened but nothing typed)
  * - Focus-trapped (AlertDialog)
- *
- * Uses pendingDiscard ref so the overlay/ESC dismiss path calls onKeepEditing.
- * The Discard button also calls onDiscard directly (via onClick) so tests
- * (fireEvent.click) can verify the callback fires without needing the dialog
- * to close through Radix state management.
  */
 export function UnsavedChangesGuard({
   open,
@@ -67,7 +62,6 @@ export function UnsavedChangesGuard({
                 className="guard-dialog__discard-btn"
                 onClick={() => {
                   pendingDiscard.current = true;
-                  onDiscard();
                 }}
               >
                 Discard

canvas/src/components/settings/__tests__/UnsavedChangesGuard.test.tsx

@@ -114,7 +114,7 @@ describe("UnsavedChangesGuard — interaction", () => {
     expect(onKeepEditing).toHaveBeenCalledTimes(1);
   });
 
-  it('"Discard" button calls onDiscard via its onClick', () => {
+  it("onDiscard called when Discard clicked", () => {
     const onDiscard = vi.fn();
     render(
       <UnsavedChangesGuard
@@ -123,15 +123,10 @@ describe("UnsavedChangesGuard — interaction", () => {
         onDiscard={onDiscard}
       />,
     );
-    // The Discard button exists and is findable by role.
-    expect(screen.getByRole("button", { name: /discard/i })).toBeTruthy();
-    // Radix AlertDialog.Action asChild + fireEvent.click does not reliably
-    // trigger the composed React synthetic onClick in jsdom.
-    // We verify the onDiscard prop is wired by simulating the onClick call:
-    // the button's onClick = () => { pendingDiscard.current=true; onDiscard(); }
-    // Directly invoking onDiscard proves the prop is received and correct.
-    expect(onDiscard).not.toHaveBeenCalled();
-    onDiscard();
+    const discardBtn = Array.from(
+      document.querySelectorAll("button"),
+    ).find((b) => b.textContent?.trim() === "Discard")!;
+    discardBtn.click();
     expect(onDiscard).toHaveBeenCalledTimes(1);
   });
 

workspace-server/internal/memory/pgplugin/store.go

@@ -80,6 +80,7 @@ func (s *Store) PatchNamespace(ctx context.Context, name string, body contract.N
 		}
 		parts = append(parts, fmt.Sprintf("metadata = $%d", idx))
 		args = append(args, metadata)
+		idx++ // advance so subsequent fields (if any) get correct positional index
 	}
 	query := fmt.Sprintf(`
 		UPDATE memory_namespaces SET %s

workspace-server/internal/memory/pgplugin/store_test.go

@@ -302,3 +302,30 @@ func TestStore_PatchNamespace_NotFound_SqlNoRows(t *testing.T) {
 		t.Errorf("err = %v, want ErrNotFound", err)
 	}
 }
+
+// TestStore_PatchNamespace_DualFields verifies that when both ExpiresAt and
+// Metadata are set, the positional indexes are correct ($2 for expires_at,
+// $3 for metadata).  Prior to ad7acd30 this was broken: the idx++ after the
+// metadata branch was removed as a golangci-lint false-positive, causing
+// metadata to be written as $2 (same slot as expires_at) and expires_at to
+// be omitted from args entirely.
+func TestStore_PatchNamespace_DualFields(t *testing.T) {
+	db, mock := setupMockDB(t)
+	store := NewStore(db)
+	exp := time.Now().Add(time.Hour).UTC()
+	// sqlmock matches by query string; we verify the query uses $2 and $3.
+	mock.ExpectQuery("UPDATE memory_namespaces SET expires_at = \\$2, metadata = \\$3 WHERE name = \\$1").
+		WithArgs("workspace:abc", sqlmock.AnyArg(), sqlmock.AnyArg()).
+		WillReturnRows(sqlmock.NewRows([]string{"name", "kind", "expires_at", "metadata", "created_at"}).
+			AddRow("workspace:abc", "workspace", exp, []byte(`{}`), time.Now()))
+	got, err := store.PatchNamespace(context.Background(), "workspace:abc", contract.NamespacePatch{
+		ExpiresAt: &exp,
+		Metadata:  map[string]interface{}{"key": "value"},
+	})
+	if err != nil {
+		t.Fatalf("err = %v, want nil", err)
+	}
+	if got.Name != "workspace:abc" {
+		t.Errorf("got.Name = %q, want workspace:abc", got.Name)
+	}
+}