@stigmer/react 3.11.0 → 3.12.0

package/datastore/useRecordList.d.ts

@@ -1,66 +0,0 @@
-import { type RecordEnvelope, type RecordFilter, type RecordOrderBy } from "@stigmer/protos/ai/stigmer/agentic/datastore/v1/record_io_pb";
-/**
- * Addressing shared by every record read/write: which datastore, which
- * collection, and which data partition (DD-010 — every record operation
- * is partition-scoped; empty means the `"default"` partition).
- */
-export interface RecordScope {
-    /** Organization slug. Required for cloud direct callers; OSS defaults it. */
-    readonly org: string;
-    /** Datastore slug. */
-    readonly datastore: string;
-    /** Collection name. */
-    readonly collection: string;
-    /** Data partition. Empty or omitted means `"default"`. */
-    readonly partition?: string;
-}
-/** Query parameters for {@link useRecordList}. */
-export interface UseRecordListParams extends RecordScope {
-    /**
-     * Typed AND-only filter in the proto shape (DD-005 grammar). The hook
-     * adds no grammar of its own — anything expressible here is exactly
-     * what `findRecords` serves.
-     */
-    readonly filter?: RecordFilter;
-    /** Sort directive. Omitted keeps the default: `created_at` desc, `id` tiebreak. */
-    readonly orderBy?: RecordOrderBy;
-    /** Page size, 1–100. Zero or omitted applies the RPC default of 25. */
-    readonly limit?: number;
-    /** Row offset for pagination. */
-    readonly offset?: number;
-}
-/** Return value of {@link useRecordList}. */
-export interface UseRecordListReturn {
-    /** Records in the current page. */
-    readonly records: readonly RecordEnvelope[];
-    /** Total records matching the filter (across all pages). */
-    readonly total: number;
-    /** `true` while the initial fetch is in flight. */
-    readonly isLoading: boolean;
-    /** `true` while a background refetch is in flight. */
-    readonly isRefetching: boolean;
-    /** Error from the last failed request, or `null` when healthy. */
-    readonly error: Error | null;
-    /** Discard cached data and re-fetch the current page from the server. */
-    readonly refetch: () => void;
-}
-/**
- * Data hook that fetches a page of records via `findRecords` — the one
- * query surface (DD-008 invariant 1): server-side filtering with the
- * DD-005 grammar, RPC limit/offset pagination, no client-side
- * filtering.
- *
- * A record-layer `PERMISSION_DENIED` (caller holds no read grant on the
- * collection) surfaces as `error` carrying the server's relayable
- * message verbatim; consumers render it as the denied panel, never a
- * silently empty grid.
- *
- * Pass `null` to skip fetching (stable no-op) — e.g. while the
- * collection is not yet selected.
- *
- * For the full grid experience (TanStack table, sort bridge, page
- * state), use {@link useRecordCollection}, which builds on the same
- * RPC.
- */
-export declare function useRecordList(params: UseRecordListParams | null): UseRecordListReturn;
-//# sourceMappingURL=useRecordList.d.ts.map

package/datastore/useRecordList.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"useRecordList.d.ts","sourceRoot":"","sources":["../../src/datastore/useRecordList.ts"],"names":[],"mappings":"AAGA,OAAO,EAIL,KAAK,cAAc,EACnB,KAAK,YAAY,EACjB,KAAK,aAAa,EACnB,MAAM,8DAA8D,CAAC;AAItE;;;;GAIG;AACH,MAAM,WAAW,WAAW;IAC1B,6EAA6E;IAC7E,QAAQ,CAAC,GAAG,EAAE,MAAM,CAAC;IACrB,sBAAsB;IACtB,QAAQ,CAAC,SAAS,EAAE,MAAM,CAAC;IAC3B,uBAAuB;IACvB,QAAQ,CAAC,UAAU,EAAE,MAAM,CAAC;IAC5B,0DAA0D;IAC1D,QAAQ,CAAC,SAAS,CAAC,EAAE,MAAM,CAAC;CAC7B;AAED,kDAAkD;AAClD,MAAM,WAAW,mBAAoB,SAAQ,WAAW;IACtD;;;;OAIG;IACH,QAAQ,CAAC,MAAM,CAAC,EAAE,YAAY,CAAC;IAC/B,mFAAmF;IACnF,QAAQ,CAAC,OAAO,CAAC,EAAE,aAAa,CAAC;IACjC,uEAAuE;IACvE,QAAQ,CAAC,KAAK,CAAC,EAAE,MAAM,CAAC;IACxB,iCAAiC;IACjC,QAAQ,CAAC,MAAM,CAAC,EAAE,MAAM,CAAC;CAC1B;AAED,6CAA6C;AAC7C,MAAM,WAAW,mBAAmB;IAClC,mCAAmC;IACnC,QAAQ,CAAC,OAAO,EAAE,SAAS,cAAc,EAAE,CAAC;IAC5C,4DAA4D;IAC5D,QAAQ,CAAC,KAAK,EAAE,MAAM,CAAC;IACvB,mDAAmD;IACnD,QAAQ,CAAC,SAAS,EAAE,OAAO,CAAC;IAC5B,sDAAsD;IACtD,QAAQ,CAAC,YAAY,EAAE,OAAO,CAAC;IAC/B,kEAAkE;IAClE,QAAQ,CAAC,KAAK,EAAE,KAAK,GAAG,IAAI,CAAC;IAC7B,yEAAyE;IACzE,QAAQ,CAAC,OAAO,EAAE,MAAM,IAAI,CAAC;CAC9B;AASD;;;;;;;;;;;;;;;;;GAiBG;AACH,wBAAgB,aAAa,CAC3B,MAAM,EAAE,mBAAmB,GAAG,IAAI,GACjC,mBAAmB,CAyCrB"}

package/datastore/useRecordList.js

@@ -1,58 +0,0 @@
-"use client";
-import { create, toJsonString } from "@bufbuild/protobuf";
-import { FindRecordsRequestSchema, RecordFilterSchema, RecordOrderBySchema, } from "@stigmer/protos/ai/stigmer/agentic/datastore/v1/record_io_pb";
-import { useStigmer } from "../hooks.js";
-import { useFetch } from "../internal/useFetch.js";
-const INITIAL_DATA = { records: [], total: 0 };
-/**
- * Data hook that fetches a page of records via `findRecords` — the one
- * query surface (DD-008 invariant 1): server-side filtering with the
- * DD-005 grammar, RPC limit/offset pagination, no client-side
- * filtering.
- *
- * A record-layer `PERMISSION_DENIED` (caller holds no read grant on the
- * collection) surfaces as `error` carrying the server's relayable
- * message verbatim; consumers render it as the denied panel, never a
- * silently empty grid.
- *
- * Pass `null` to skip fetching (stable no-op) — e.g. while the
- * collection is not yet selected.
- *
- * For the full grid experience (TanStack table, sort bridge, page
- * state), use {@link useRecordCollection}, which builds on the same
- * RPC.
- */
-export function useRecordList(params) {
-    const stigmer = useStigmer();
-    // Proto messages are typically rebuilt each render; serialized JSON is
-    // the stable identity for dependency comparison.
-    const filterKey = params?.filter ? toJsonString(RecordFilterSchema, params.filter) : "";
-    const orderByKey = params?.orderBy ? toJsonString(RecordOrderBySchema, params.orderBy) : "";
-    const { data, isLoading, isRefetching, error, refetch } = useFetch(params
-        ? async () => {
-            const result = await stigmer.datastore.findRecords(create(FindRecordsRequestSchema, {
-                org: params.org,
-                datastore: params.datastore,
-                collection: params.collection,
-                partition: params.partition ?? "",
-                filter: params.filter,
-                orderBy: params.orderBy,
-                limit: params.limit ?? 0,
-                offset: params.offset ?? 0,
-            }));
-            return { records: result.records, total: result.total };
-        }
-        : null, [
-        params?.org,
-        params?.datastore,
-        params?.collection,
-        params?.partition,
-        filterKey,
-        orderByKey,
-        params?.limit,
-        params?.offset,
-        stigmer,
-    ], INITIAL_DATA);
-    return { records: data.records, total: data.total, isLoading, isRefetching, error, refetch };
-}
-//# sourceMappingURL=useRecordList.js.map

package/datastore/useRecordList.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"useRecordList.js","sourceRoot":"","sources":["../../src/datastore/useRecordList.ts"],"names":[],"mappings":"AAAA,YAAY,CAAC;AAEb,OAAO,EAAE,MAAM,EAAE,YAAY,EAAE,MAAM,oBAAoB,CAAC;AAC1D,OAAO,EACL,wBAAwB,EACxB,kBAAkB,EAClB,mBAAmB,GAIpB,MAAM,8DAA8D,CAAC;AACtE,OAAO,EAAE,UAAU,EAAE,MAAM,aAAa,CAAC;AACzC,OAAO,EAAE,QAAQ,EAAE,MAAM,yBAAyB,CAAC;AAuDnD,MAAM,YAAY,GAAmB,EAAE,OAAO,EAAE,EAAE,EAAE,KAAK,EAAE,CAAC,EAAE,CAAC;AAE/D;;;;;;;;;;;;;;;;;GAiBG;AACH,MAAM,UAAU,aAAa,CAC3B,MAAkC;IAElC,MAAM,OAAO,GAAG,UAAU,EAAE,CAAC;IAE7B,uEAAuE;IACvE,iDAAiD;IACjD,MAAM,SAAS,GAAG,MAAM,EAAE,MAAM,CAAC,CAAC,CAAC,YAAY,CAAC,kBAAkB,EAAE,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,EAAE,CAAC;IACxF,MAAM,UAAU,GAAG,MAAM,EAAE,OAAO,CAAC,CAAC,CAAC,YAAY,CAAC,mBAAmB,EAAE,MAAM,CAAC,OAAO,CAAC,CAAC,CAAC,CAAC,EAAE,CAAC;IAE5F,MAAM,EAAE,IAAI,EAAE,SAAS,EAAE,YAAY,EAAE,KAAK,EAAE,OAAO,EAAE,GAAG,QAAQ,CAChE,MAAM;QACJ,CAAC,CAAC,KAAK,IAAI,EAAE;YACT,MAAM,MAAM,GAAG,MAAM,OAAO,CAAC,SAAS,CAAC,WAAW,CAChD,MAAM,CAAC,wBAAwB,EAAE;gBAC/B,GAAG,EAAE,MAAM,CAAC,GAAG;gBACf,SAAS,EAAE,MAAM,CAAC,SAAS;gBAC3B,UAAU,EAAE,MAAM,CAAC,UAAU;gBAC7B,SAAS,EAAE,MAAM,CAAC,SAAS,IAAI,EAAE;gBACjC,MAAM,EAAE,MAAM,CAAC,MAAM;gBACrB,OAAO,EAAE,MAAM,CAAC,OAAO;gBACvB,KAAK,EAAE,MAAM,CAAC,KAAK,IAAI,CAAC;gBACxB,MAAM,EAAE,MAAM,CAAC,MAAM,IAAI,CAAC;aAC3B,CAAC,CACH,CAAC;YACF,OAAO,EAAE,OAAO,EAAE,MAAM,CAAC,OAAO,EAAE,KAAK,EAAE,MAAM,CAAC,KAAK,EAAE,CAAC;QAC1D,CAAC;QACH,CAAC,CAAC,IAAI,EACR;QACE,MAAM,EAAE,GAAG;QACX,MAAM,EAAE,SAAS;QACjB,MAAM,EAAE,UAAU;QAClB,MAAM,EAAE,SAAS;QACjB,SAAS;QACT,UAAU;QACV,MAAM,EAAE,KAAK;QACb,MAAM,EAAE,MAAM;QACd,OAAO;KACR,EACD,YAAY,CACb,CAAC;IAEF,OAAO,EAAE,OAAO,EAAE,IAAI,CAAC,OAAO,EAAE,KAAK,EAAE,IAAI,CAAC,KAAK,EAAE,SAAS,EAAE,YAAY,EAAE,KAAK,EAAE,OAAO,EAAE,CAAC;AAC/F,CAAC"}

package/datastore/useUpdateRecord.d.ts

@@ -1,52 +0,0 @@
-import { type JsonObject } from "@bufbuild/protobuf";
-import { type RecordEnvelope } from "@stigmer/protos/ai/stigmer/agentic/datastore/v1/record_io_pb";
-import type { RecordScope } from "./useRecordList.js";
-/** Arguments for a single record update. */
-export interface UpdateRecordArgs extends RecordScope {
-    /** Record id (`dsr_…`). */
-    readonly id: string;
-    /**
-     * Partial merge (DD-005): only the supplied fields change; an
-     * explicit `null` clears a field; absent fields keep their stored
-     * values. Constraints evaluate on the merged result. Build this from
-     * per-field edits with `buildUpdateFields`.
-     */
-    readonly fields: JsonObject;
-}
-/** Return value of {@link useUpdateRecord}. */
-export interface UseUpdateRecordReturn {
-    /**
-     * Update one record by id. Resolves with the merged envelope. Rejects
-     * — and sets `error` — on denial or constraint violation; the error's
-     * message is the server's relayable text, byte-for-byte.
-     */
-    readonly updateRecord: (args: UpdateRecordArgs) => Promise<RecordEnvelope>;
-    /** `true` while the update is in flight. */
-    readonly isUpdating: boolean;
-    /** Error from the last failed update, or `null` when healthy. */
-    readonly error: Error | null;
-    /** Reset `error` to `null`. */
-    readonly clearError: () => void;
-}
-/**
- * Behavior hook that wraps `datastore.updateRecord()` with loading and
- * error state.
- *
- * The RPC is an honest tri-state partial merge — the hook adds no
- * merge semantics of its own. A record in another partition is
- * NOT_FOUND (DD-010: partition scoping is structural).
- *
- * @example
- * ```tsx
- * const { updateRecord } = useUpdateRecord();
- * await updateRecord({
- *   org: "acme",
- *   datastore: "clinic",
- *   collection: "bookings",
- *   id: "dsr_01j9...",
- *   fields: { patient_phone: "+15550100", notes: null }, // notes cleared
- * });
- * ```
- */
-export declare function useUpdateRecord(): UseUpdateRecordReturn;
-//# sourceMappingURL=useUpdateRecord.d.ts.map

package/datastore/useUpdateRecord.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"useUpdateRecord.d.ts","sourceRoot":"","sources":["../../src/datastore/useUpdateRecord.ts"],"names":[],"mappings":"AAGA,OAAO,EAAU,KAAK,UAAU,EAAE,MAAM,oBAAoB,CAAC;AAC7D,OAAO,EAEL,KAAK,cAAc,EACpB,MAAM,8DAA8D,CAAC;AAGtE,OAAO,KAAK,EAAE,WAAW,EAAE,MAAM,oBAAoB,CAAC;AAEtD,4CAA4C;AAC5C,MAAM,WAAW,gBAAiB,SAAQ,WAAW;IACnD,2BAA2B;IAC3B,QAAQ,CAAC,EAAE,EAAE,MAAM,CAAC;IACpB;;;;;OAKG;IACH,QAAQ,CAAC,MAAM,EAAE,UAAU,CAAC;CAC7B;AAED,+CAA+C;AAC/C,MAAM,WAAW,qBAAqB;IACpC;;;;OAIG;IACH,QAAQ,CAAC,YAAY,EAAE,CAAC,IAAI,EAAE,gBAAgB,KAAK,OAAO,CAAC,cAAc,CAAC,CAAC;IAC3E,4CAA4C;IAC5C,QAAQ,CAAC,UAAU,EAAE,OAAO,CAAC;IAC7B,iEAAiE;IACjE,QAAQ,CAAC,KAAK,EAAE,KAAK,GAAG,IAAI,CAAC;IAC7B,+BAA+B;IAC/B,QAAQ,CAAC,UAAU,EAAE,MAAM,IAAI,CAAC;CACjC;AAED;;;;;;;;;;;;;;;;;;;GAmBG;AACH,wBAAgB,eAAe,IAAI,qBAAqB,CAiCvD"}

package/datastore/useUpdateRecord.js

@@ -1,55 +0,0 @@
-"use client";
-import { useCallback, useState } from "react";
-import { create } from "@bufbuild/protobuf";
-import { UpdateRecordRequestSchema, } from "@stigmer/protos/ai/stigmer/agentic/datastore/v1/record_io_pb";
-import { useStigmer } from "../hooks.js";
-import { toError } from "../internal/toError.js";
-/**
- * Behavior hook that wraps `datastore.updateRecord()` with loading and
- * error state.
- *
- * The RPC is an honest tri-state partial merge — the hook adds no
- * merge semantics of its own. A record in another partition is
- * NOT_FOUND (DD-010: partition scoping is structural).
- *
- * @example
- * ```tsx
- * const { updateRecord } = useUpdateRecord();
- * await updateRecord({
- *   org: "acme",
- *   datastore: "clinic",
- *   collection: "bookings",
- *   id: "dsr_01j9...",
- *   fields: { patient_phone: "+15550100", notes: null }, // notes cleared
- * });
- * ```
- */
-export function useUpdateRecord() {
-    const stigmer = useStigmer();
-    const [isUpdating, setIsUpdating] = useState(false);
-    const [error, setError] = useState(null);
-    const clearError = useCallback(() => setError(null), []);
-    const updateRecord = useCallback(async (args) => {
-        setIsUpdating(true);
-        setError(null);
-        try {
-            return await stigmer.datastore.updateRecord(create(UpdateRecordRequestSchema, {
-                org: args.org,
-                datastore: args.datastore,
-                collection: args.collection,
-                partition: args.partition ?? "",
-                id: args.id,
-                fields: args.fields,
-            }));
-        }
-        catch (err) {
-            setError(toError(err));
-            throw err;
-        }
-        finally {
-            setIsUpdating(false);
-        }
-    }, [stigmer]);
-    return { updateRecord, isUpdating, error, clearError };
-}
-//# sourceMappingURL=useUpdateRecord.js.map

package/datastore/useUpdateRecord.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"useUpdateRecord.js","sourceRoot":"","sources":["../../src/datastore/useUpdateRecord.ts"],"names":[],"mappings":"AAAA,YAAY,CAAC;AAEb,OAAO,EAAE,WAAW,EAAE,QAAQ,EAAE,MAAM,OAAO,CAAC;AAC9C,OAAO,EAAE,MAAM,EAAmB,MAAM,oBAAoB,CAAC;AAC7D,OAAO,EACL,yBAAyB,GAE1B,MAAM,8DAA8D,CAAC;AACtE,OAAO,EAAE,UAAU,EAAE,MAAM,aAAa,CAAC;AACzC,OAAO,EAAE,OAAO,EAAE,MAAM,wBAAwB,CAAC;AAgCjD;;;;;;;;;;;;;;;;;;;GAmBG;AACH,MAAM,UAAU,eAAe;IAC7B,MAAM,OAAO,GAAG,UAAU,EAAE,CAAC;IAC7B,MAAM,CAAC,UAAU,EAAE,aAAa,CAAC,GAAG,QAAQ,CAAC,KAAK,CAAC,CAAC;IACpD,MAAM,CAAC,KAAK,EAAE,QAAQ,CAAC,GAAG,QAAQ,CAAe,IAAI,CAAC,CAAC;IAEvD,MAAM,UAAU,GAAG,WAAW,CAAC,GAAG,EAAE,CAAC,QAAQ,CAAC,IAAI,CAAC,EAAE,EAAE,CAAC,CAAC;IAEzD,MAAM,YAAY,GAAG,WAAW,CAC9B,KAAK,EAAE,IAAsB,EAA2B,EAAE;QACxD,aAAa,CAAC,IAAI,CAAC,CAAC;QACpB,QAAQ,CAAC,IAAI,CAAC,CAAC;QACf,IAAI,CAAC;YACH,OAAO,MAAM,OAAO,CAAC,SAAS,CAAC,YAAY,CACzC,MAAM,CAAC,yBAAyB,EAAE;gBAChC,GAAG,EAAE,IAAI,CAAC,GAAG;gBACb,SAAS,EAAE,IAAI,CAAC,SAAS;gBACzB,UAAU,EAAE,IAAI,CAAC,UAAU;gBAC3B,SAAS,EAAE,IAAI,CAAC,SAAS,IAAI,EAAE;gBAC/B,EAAE,EAAE,IAAI,CAAC,EAAE;gBACX,MAAM,EAAE,IAAI,CAAC,MAAM;aACpB,CAAC,CACH,CAAC;QACJ,CAAC;QAAC,OAAO,GAAG,EAAE,CAAC;YACb,QAAQ,CAAC,OAAO,CAAC,GAAG,CAAC,CAAC,CAAC;YACvB,MAAM,GAAG,CAAC;QACZ,CAAC;gBAAS,CAAC;YACT,aAAa,CAAC,KAAK,CAAC,CAAC;QACvB,CAAC;IACH,CAAC,EACD,CAAC,OAAO,CAAC,CACV,CAAC;IAEF,OAAO,EAAE,YAAY,EAAE,UAAU,EAAE,KAAK,EAAE,UAAU,EAAE,CAAC;AACzD,CAAC"}

package/src/agent/__tests__/AgentDetailView.datastores.test.tsx

@@ -1,166 +0,0 @@
-// Tests for the Datastores section on AgentDetailView (stigmer/stigmer#319):
-// read-mode rendering and navigation, the inline-edit save payload (which
-// must round-trip the full spec through agentToInput), and the DD-006
-// surfacing of the backend's datastore exposure guard refusal.
-
-import { describe, it, expect, vi, beforeAll, afterEach, beforeEach, type Mock } from "vitest";
-import { render, screen, cleanup, waitFor } from "@testing-library/react";
-import userEvent from "@testing-library/user-event";
-import type { ReactNode } from "react";
-import { create } from "@bufbuild/protobuf";
-import { AgentSchema } from "@stigmer/protos/ai/stigmer/agentic/agent/v1/api_pb";
-import { ApiResourceKind } from "@stigmer/protos/ai/stigmer/commons/apiresource/apiresourcekind/api_resource_kind_pb";
-import type { AgentInput } from "@stigmer/sdk";
-import { StigmerContext } from "../../context";
-import { FetchCacheContext } from "../../internal/FetchCacheProvider";
-import { AgentDetailView } from "../AgentDetailView";
-
-beforeAll(() => {
-  HTMLDialogElement.prototype.showModal = function showModal() {
-    this.open = true;
-  };
-  HTMLDialogElement.prototype.close = function close() {
-    this.open = false;
-  };
-});
-
-afterEach(() => cleanup());
-beforeEach(() => vi.restoreAllMocks());
-
-function makeAgent() {
-  return create(AgentSchema, {
-    metadata: {
-      id: "agt_1",
-      name: "Clinic Assistant",
-      slug: "clinic-assistant",
-      org: "acme",
-    },
-    spec: {
-      instructions: "You are a careful clinic assistant.",
-      datastoreUsages: [
-        {
-          datastoreRef: {
-            org: "acme",
-            slug: "clinic-records",
-            kind: ApiResourceKind.datastore,
-          },
-        },
-        {
-          datastoreRef: {
-            org: "partner",
-            slug: "shared-records",
-            kind: ApiResourceKind.datastore,
-          },
-        },
-      ],
-    },
-  });
-}
-
-function renderView(overrides: {
-  update?: Mock;
-  onDatastoreClick?: (ref: { org: string; slug: string }) => void;
-  editable?: boolean;
-}) {
-  const agent = makeAgent();
-  const update = overrides.update ?? vi.fn().mockResolvedValue(agent);
-  const stigmer = {
-    agent: {
-      getByReference: vi.fn().mockResolvedValue(agent),
-      update,
-    },
-    iamPolicy: {
-      checkMyPermission: vi.fn().mockResolvedValue({ isAuthorized: false }),
-    },
-  };
-
-  function Wrapper({ children }: { children: ReactNode }) {
-    return (
-      <FetchCacheContext.Provider value={null}>
-        <StigmerContext.Provider value={stigmer as never}>
-          {children}
-        </StigmerContext.Provider>
-      </FetchCacheContext.Provider>
-    );
-  }
-
-  const result = render(
-    <AgentDetailView
-      org="acme"
-      slug="clinic-assistant"
-      editable={overrides.editable}
-      onDatastoreClick={overrides.onDatastoreClick}
-    />,
-    { wrapper: Wrapper },
-  );
-  return { ...result, update };
-}
-
-describe("AgentDetailView — Datastores section (read mode)", () => {
-  it("renders attached datastores with org-qualified labels for cross-org refs", async () => {
-    renderView({});
-
-    expect(await screen.findByText("Datastores")).toBeTruthy();
-    expect(screen.getByText("clinic-records")).toBeTruthy();
-    expect(screen.getByText("partner/shared-records")).toBeTruthy();
-  });
-
-  it("fires onDatastoreClick with the resolved org and slug", async () => {
-    const user = userEvent.setup();
-    const onDatastoreClick = vi.fn();
-    renderView({ onDatastoreClick });
-
-    await user.click(await screen.findByText("clinic-records"));
-    expect(onDatastoreClick).toHaveBeenCalledWith({
-      org: "acme",
-      slug: "clinic-records",
-    });
-
-    await user.click(screen.getByText("partner/shared-records"));
-    expect(onDatastoreClick).toHaveBeenCalledWith({
-      org: "partner",
-      slug: "shared-records",
-    });
-  });
-});
-
-describe("AgentDetailView — Datastores section (inline edit)", () => {
-  it("saves the full reconstructed input with correctly-shaped datastoreUsages", async () => {
-    const user = userEvent.setup();
-    const { update } = renderView({ editable: true });
-
-    await user.click(await screen.findByLabelText("Edit datastores"));
-    await user.click(screen.getByLabelText("Remove partner/shared-records"));
-    await user.click(screen.getByRole("button", { name: "Save changes" }));
-
-    await waitFor(() => expect(update).toHaveBeenCalledTimes(1));
-    const input = update.mock.calls[0][0] as AgentInput;
-
-    expect(input.datastoreUsages).toEqual([
-      { datastoreRef: { org: "acme", slug: "clinic-records" } },
-    ]);
-    // The rest of the spec must ride along — a section save is a full
-    // spec replacement reconstructed via agentToInput().
-    expect(input.instructions).toBe("You are a careful clinic assistant.");
-    expect(input.org).toBe("acme");
-    expect(input.slug).toBe("clinic-assistant");
-  });
-
-  it("surfaces the backend's exposure-guard refusal in the section editor (DD-006)", async () => {
-    const user = userEvent.setup();
-    const guardMessage =
-      'agent "clinic-assistant" cannot be public while it uses datastores ' +
-      "(clinic-records): multi-tenant datastore sharing is not supported — " +
-      "keep the agent private or org-visible, or remove its datastore_usages";
-    renderView({
-      editable: true,
-      update: vi.fn().mockRejectedValue(new Error(guardMessage)),
-    });
-
-    await user.click(await screen.findByLabelText("Edit datastores"));
-    await user.click(screen.getByRole("button", { name: "Save changes" }));
-
-    const alert = await screen.findByRole("alert");
-    expect(alert.textContent).toContain(guardMessage);
-  });
-});