@stigmer/react 3.1.17 → 3.2.0

package/src/channel/__tests__/ChannelConversationsDialog.test.tsx

@@ -0,0 +1,114 @@
+import { describe, it, expect, vi, beforeEach, afterEach } from "vitest";
+import { render, screen, cleanup, waitFor } from "@testing-library/react";
+import type { ReactNode } from "react";
+import type { AgentChannel } from "@stigmer/protos/ai/stigmer/agentic/agentchannel/v1/api_pb";
+import { StigmerContext } from "../../context";
+import { FetchCacheContext } from "../../internal/FetchCacheProvider";
+import { ChannelConversationsDialog } from "../ChannelConversationsDialog";
+
+const CHANNEL = {
+  metadata: { id: "ach_1", org: "acme", name: "Leftbin Slack", slug: "leftbin-slack" },
+  spec: { enabled: true },
+} as unknown as AgentChannel;
+
+function makeChannelSession(id: string, subject: string, externalUser?: string) {
+  return {
+    metadata: {
+      id,
+      org: "acme",
+      labels: {
+        "stigmer.ai/channel-id": "ach_1",
+        ...(externalUser
+          ? { "stigmer.ai/channel-external-user-key": externalUser }
+          : {}),
+      },
+    },
+    spec: { subject },
+  };
+}
+
+function renderDialog(
+  listByChannel: (input: unknown) => Promise<unknown>,
+  props?: { sessionHref?: (id: string) => string },
+) {
+  const client = { session: { listByChannel } } as never;
+  function Wrapper({ children }: { children: ReactNode }) {
+    return (
+      <FetchCacheContext.Provider value={null}>
+        <StigmerContext.Provider value={client}>{children}</StigmerContext.Provider>
+      </FetchCacheContext.Provider>
+    );
+  }
+  return render(
+    <ChannelConversationsDialog
+      open
+      onOpenChange={vi.fn()}
+      channel={CHANNEL}
+      modal={false}
+      {...props}
+    />,
+    { wrapper: Wrapper },
+  );
+}
+
+describe("ChannelConversationsDialog", () => {
+  beforeEach(() => {
+    vi.restoreAllMocks();
+  });
+
+  afterEach(() => {
+    cleanup();
+  });
+
+  it("lists conversations with subject, external user, and session links", async () => {
+    const listByChannel = vi.fn().mockResolvedValue({
+      entries: [
+        makeChannelSession("ses_1", "What is a skill?", "U0AB12CD3"),
+        makeChannelSession("ses_2", "Deploy help"),
+      ],
+    });
+
+    renderDialog(listByChannel, { sessionHref: (id) => `/sessions/${id}` });
+
+    await waitFor(() =>
+      expect(screen.getByText("What is a skill?")).toBeDefined(),
+    );
+    expect(screen.getByText("U0AB12CD3")).toBeDefined();
+
+    const link = screen.getByRole("link", { name: /What is a skill\?/ });
+    expect(link.getAttribute("href")).toBe("/sessions/ses_1");
+  });
+
+  it("renders rows without links when the host provides no session route", async () => {
+    const listByChannel = vi.fn().mockResolvedValue({
+      entries: [makeChannelSession("ses_1", "What is a skill?", "U0AB12CD3")],
+    });
+
+    renderDialog(listByChannel);
+
+    await waitFor(() =>
+      expect(screen.getByText("What is a skill?")).toBeDefined(),
+    );
+    expect(screen.queryAllByRole("link")).toHaveLength(0);
+  });
+
+  it("shows the empty state when nobody has messaged the channel", async () => {
+    renderDialog(vi.fn().mockResolvedValue({ entries: [] }));
+
+    await waitFor(() =>
+      expect(screen.getByText("No conversations yet")).toBeDefined(),
+    );
+  });
+
+  it("surfaces a denial verbatim (caller cannot view the channel)", async () => {
+    renderDialog(
+      vi.fn().mockRejectedValue(new Error("unauthorized to list channel conversations")),
+    );
+
+    await waitFor(() =>
+      expect(
+        screen.getByText(/unauthorized to list channel conversations/),
+      ).toBeDefined(),
+    );
+  });
+});

package/src/channel/__tests__/ChannelCredentialsDialog.test.tsx

@@ -0,0 +1,240 @@
+import { describe, it, expect, vi, beforeAll, afterEach } from "vitest";
+import { render, screen, fireEvent, waitFor, cleanup } from "@testing-library/react";
+import type { ReactNode } from "react";
+import { create } from "@bufbuild/protobuf";
+import { AgentSchema } from "@stigmer/protos/ai/stigmer/agentic/agent/v1/api_pb";
+import { ApiResourceVisibility } from "@stigmer/protos/ai/stigmer/commons/apiresource/enum_pb";
+import type { AgentChannelInput } from "@stigmer/sdk";
+import { StigmerContext } from "../../context";
+import { FetchCacheContext } from "../../internal/FetchCacheProvider";
+import { DeploymentModeContext } from "../../deployment-mode";
+import { ChannelCredentialsDialog } from "../ChannelCredentialsDialog";
+
+// happy-dom does not implement the native dialog show/close methods.
+beforeAll(() => {
+  HTMLDialogElement.prototype.showModal = function showModal() {
+    this.open = true;
+  };
+  HTMLDialogElement.prototype.close = function close() {
+    this.open = false;
+  };
+});
+
+afterEach(cleanup);
+
+function makeAgent(withTools = true) {
+  return create(AgentSchema, {
+    metadata: { id: "agt_1", org: "acme", slug: "support-agent", name: "Support Agent" },
+    spec: {
+      instructions: "help",
+      mcpServerUsages: withTools
+        ? [{ mcpServerRef: { org: "acme", slug: "github" } }]
+        : [],
+    },
+  });
+}
+
+function makeChannel(environmentRefs: { org: string; slug: string }[] = []) {
+  return {
+    metadata: {
+      id: "ach_1",
+      name: "Support Slack",
+      slug: "support-slack",
+      org: "acme",
+      labels: {},
+    },
+    spec: {
+      agentRef: { org: "acme", slug: "support-agent" },
+      enabled: true,
+      providerConfig: { case: "slack", value: {} },
+      environmentRefs,
+    },
+    status: { installState: 2 },
+  } as never;
+}
+
+function createMockStigmer(overrides: {
+  apply?: (input: AgentChannelInput) => Promise<unknown>;
+} = {}) {
+  return {
+    agentChannel: {
+      apply: overrides.apply ?? vi.fn().mockResolvedValue({}),
+    },
+    environment: {
+      list: vi.fn().mockResolvedValue({
+        items: [
+          {
+            metadata: {
+              slug: "github-credentials",
+              name: "GitHub Credentials",
+              org: "acme",
+              visibility: ApiResourceVisibility.visibility_org,
+            },
+            spec: {},
+          },
+          {
+            metadata: {
+              slug: "private-creds",
+              name: "Private Creds",
+              org: "acme",
+              visibility: ApiResourceVisibility.visibility_private,
+            },
+            spec: {},
+          },
+        ],
+        totalCount: 2,
+      }),
+      getByReference: vi.fn().mockImplementation(({ slug }: { slug: string }) =>
+        Promise.resolve({
+          metadata: {
+            visibility:
+              slug === "github-credentials"
+                ? ApiResourceVisibility.visibility_org
+                : ApiResourceVisibility.visibility_private,
+          },
+        }),
+      ),
+    },
+  } as never;
+}
+
+function Providers({
+  client,
+  children,
+}: {
+  client: unknown;
+  children: ReactNode;
+}) {
+  return (
+    <FetchCacheContext.Provider value={null}>
+      <DeploymentModeContext.Provider value="cloud">
+        <StigmerContext.Provider value={client as never}>
+          {children}
+        </StigmerContext.Provider>
+      </DeploymentModeContext.Provider>
+    </FetchCacheContext.Provider>
+  );
+}
+
+describe("ChannelCredentialsDialog", () => {
+  it("offers only org-shared environments in the picker", async () => {
+    render(
+      <Providers client={createMockStigmer()}>
+        <ChannelCredentialsDialog
+          open
+          onOpenChange={() => {}}
+          agent={makeAgent()}
+          channel={makeChannel()}
+        />
+      </Providers>,
+    );
+
+    // Org-shared env offered; private env filtered out (the runtime
+    // merge would silently skip it).
+    expect(
+      await screen.findByRole("option", { name: "GitHub Credentials" }),
+    ).toBeTruthy();
+    expect(screen.queryByRole("option", { name: "Private Creds" })).toBeNull();
+  });
+
+  it("saves the full input with the updated bindings — nothing else is dropped", async () => {
+    const apply = vi.fn().mockResolvedValue({});
+    const onSaved = vi.fn();
+
+    render(
+      <Providers client={createMockStigmer({ apply })}>
+        <ChannelCredentialsDialog
+          open
+          onOpenChange={() => {}}
+          agent={makeAgent()}
+          channel={makeChannel()}
+          onSaved={onSaved}
+        />
+      </Providers>,
+    );
+
+    // Bind the org-shared environment through the picker.
+    const select = await screen.findByLabelText("Add environment");
+    fireEvent.change(select, { target: { value: "github-credentials" } });
+
+    fireEvent.click(screen.getByRole("button", { name: "Save" }));
+
+    // The save is a full-input apply: the agent reference and provider
+    // marker survive alongside the new binding (apply semantics would
+    // silently unbind whatever a partial input omitted).
+    await waitFor(() =>
+      expect(apply).toHaveBeenCalledWith(
+        expect.objectContaining({
+          name: "Support Slack",
+          org: "acme",
+          agentRef: { org: "acme", slug: "support-agent" },
+          enabled: true,
+          slack: {},
+          environmentRefs: [{ org: "acme", slug: "github-credentials" }],
+        }),
+      ),
+    );
+    expect(onSaved).toHaveBeenCalled();
+  });
+
+  it("saves an emptied list as an explicit unbind", async () => {
+    const apply = vi.fn().mockResolvedValue({});
+
+    render(
+      <Providers client={createMockStigmer({ apply })}>
+        <ChannelCredentialsDialog
+          open
+          onOpenChange={() => {}}
+          agent={makeAgent()}
+          channel={makeChannel([{ org: "acme", slug: "github-credentials" }])}
+        />
+      </Providers>,
+    );
+
+    fireEvent.click(
+      await screen.findByRole("button", { name: /remove github credentials/i }),
+    );
+    fireEvent.click(screen.getByRole("button", { name: "Save" }));
+
+    await waitFor(() =>
+      expect(apply).toHaveBeenCalledWith(
+        expect.objectContaining({ environmentRefs: [] }),
+      ),
+    );
+  });
+
+  it("shows the needs-credentials hint for a tool-using agent with no bindings", async () => {
+    render(
+      <Providers client={createMockStigmer()}>
+        <ChannelCredentialsDialog
+          open
+          onOpenChange={() => {}}
+          agent={makeAgent()}
+          channel={makeChannel()}
+        />
+      </Providers>,
+    );
+
+    expect(
+      await screen.findByText(/no credentials are bound to this channel/i),
+    ).toBeTruthy();
+  });
+
+  it("stays silent for agents without tools", async () => {
+    render(
+      <Providers client={createMockStigmer()}>
+        <ChannelCredentialsDialog
+          open
+          onOpenChange={() => {}}
+          agent={makeAgent(false)}
+          channel={makeChannel()}
+        />
+      </Providers>,
+    );
+
+    await screen.findByLabelText("Add environment");
+    expect(
+      screen.queryByText(/no credentials are bound to this channel/i),
+    ).toBeNull();
+  });
+});