test(handlers): add coverage for plugins_listing.go

Add 21 tests for the plugins registry listing surface:
- parseManifestYAML: full YAML, minimal fields, missing yaml,
  bad YAML, partial fields (wrong types for arrays)
- listRegistryFiltered: empty dir, nonexistent dir, no yaml,
  valid yaml, files-ignored, runtime filter matches/excludes,
  unspecified-runtime always included, multiple matching
- ListRegistry (GET /plugins): no filter, with runtime filter,
  empty on no matches
- ListAvailableForWorkspace (GET /workspaces/:id/plugins/available):
  runtimeLookup returns runtime, errors, nil lookup, unspecified
  runtime always included

No handler logic changed; all tests pass.

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

canvas/src/components/mobile/MobileChat.tsx

@@ -2,8 +2,11 @@
 
 // 04 · Chat — message thread + composer + sub-tabs.
 // Wired to the same /workspaces/:id/a2a (method message/send) endpoint
-// that the desktop ChatTab uses, but with a slimmer surface: no
-// attachments, no A2A topology overlay, no conversation tracing.
+// that the desktop ChatTab uses. Render parity with desktop ChatTab is
+// achieved by reusing its renderers rather than forking a reduced
+// mobile path: the Agent Comms sub-tab mounts the same AgentCommsPanel,
+// and message attachments route through the same AttachmentPreview
+// dispatch the desktop My-Chat bubble uses (#231/#232).
 
 import { useEffect, useMemo, useRef, useState } from "react";
 import ReactMarkdown from "react-markdown";
@@ -16,6 +19,9 @@ import {
   useChatSend,
   useChatSocket,
 } from "@/components/tabs/chat/hooks";
+import { AgentCommsPanel } from "@/components/tabs/chat/AgentCommsPanel";
+import { AttachmentPreview } from "@/components/tabs/chat/AttachmentPreview";
+import { downloadChatFile } from "@/components/tabs/chat/uploads";
 
 import { toMobileAgent } from "./components";
 import { MOBILE_FONT_MONO, MOBILE_FONT_SANS, usePalette } from "./palette";
@@ -304,6 +310,17 @@ export function MobileChat({
   const removePendingFile = (index: number) =>
     setPendingFiles((prev) => prev.filter((_, i) => i !== index));
 
+  // Route attachment downloads through the same authenticated helper
+  // the desktop ChatTab uses (downloadChatFile) so platform-scheme
+  // URIs get a real Blob with auth headers instead of about:blank.
+  const downloadAttachment = (att: ChatAttachment) => {
+    downloadChatFile(agentId, att).catch(() => {
+      // AttachmentPreview's own error affordance covers the in-bubble
+      // failure state; matches ChatTab's behaviour of not double-
+      // reporting a download failure.
+    });
+  };
+
   const send = async () => {
     const text = draft.trim();
     if ((!text && pendingFiles.length === 0) || sending || !reachable) return;
@@ -433,7 +450,19 @@ export function MobileChat({
         </div>
       </div>
 
+      {/* Agent Comms — reuse the desktop AgentCommsPanel verbatim so
+          mobile renders the identical peer/A2A + delegation feed
+          (history GET + live socket events) instead of a placeholder
+          (#231). The panel owns its own scroll/load/error/empty
+          states, matching ChatTab's agent-comms tabpanel. */}
+      {tab === "a2a" && (
+        <div style={{ flex: 1, minHeight: 0, overflow: "hidden" }}>
+          <AgentCommsPanel workspaceId={agentId} />
+        </div>
+      )}
+
       {/* Messages */}
+      {tab === "my" && (
       <div
         ref={scrollRef}
         style={{
@@ -445,18 +474,6 @@ export function MobileChat({
           gap: 8,
         }}
       >
-        {tab === "a2a" && (
-          <div
-            style={{
-              padding: "20px 4px",
-              textAlign: "center",
-              color: p.text3,
-              fontSize: 13,
-            }}
-          >
-            Agent Comms — peer-to-peer A2A traffic surfaces in the Comms tab.
-          </div>
-        )}
         {tab === "my" && historyLoading && (
           <div style={{ padding: "20px 4px", textAlign: "center", color: p.text3, fontSize: 13 }}>
             Loading chat history…
@@ -521,9 +538,31 @@ export function MobileChat({
                     overflowWrap: "anywhere",
                   }}
                 >
-                  <MarkdownBubble dark={dark} accent={p.accent}>
-                    {m.content}
-                  </MarkdownBubble>
+                  {m.content && (
+                    <MarkdownBubble dark={dark} accent={p.accent}>
+                      {m.content}
+                    </MarkdownBubble>
+                  )}
+                  {m.attachments && m.attachments.length > 0 && (
+                    <div
+                      style={{
+                        display: "flex",
+                        flexWrap: "wrap",
+                        gap: 4,
+                        marginTop: m.content ? 6 : 0,
+                      }}
+                    >
+                      {m.attachments.map((att, i) => (
+                        <AttachmentPreview
+                          key={`${m.id}-${i}`}
+                          workspaceId={agentId}
+                          attachment={att}
+                          onDownload={downloadAttachment}
+                          tone={mine ? "user" : "agent"}
+                        />
+                      ))}
+                    </div>
+                  )}
                   <div
                     style={{
                       fontSize: 10,
@@ -554,7 +593,13 @@ export function MobileChat({
           </div>
         )}
       </div>
+      )}
 
+      {/* Footer ID + composer belong to My Chat only. The Agent Comms
+          tab is a read-only peer/A2A feed (parity with desktop
+          ChatTab, where the agent-comms tabpanel has no composer). */}
+      {tab === "my" && (
+      <>
       {/* Footer ID */}
       <div
         style={{
@@ -746,6 +791,8 @@ export function MobileChat({
           </button>
         </div>
       </div>
+      </>
+      )}
     </div>
   );
 }

canvas/src/components/mobile/__tests__/MobileChat.test.tsx

@@ -21,6 +21,14 @@ import { MobileChat } from "../MobileChat";
 vi.mock("@/lib/api");
 import { api } from "@/lib/api";
 
+// AgentCommsPanel (mounted by the Agent Comms sub-tab, #231) subscribes
+// to the global socket via useSocketEvent. Stub it to a no-op so the
+// panel mounts without the real ReconnectingSocket — the parity tests
+// only assert the panel renders (vs the old static placeholder).
+vi.mock("@/hooks/useSocketEvent", () => ({
+  useSocketEvent: vi.fn(),
+}));
+
 // ─── Mock store ───────────────────────────────────────────────────────────────
 
 const mockAgentId = "ws-chat-test";
@@ -155,6 +163,12 @@ beforeEach(() => {
   mockOnBack.mockClear();
   mockStoreState.nodes = [];
   mockStoreState.agentMessages = {};
+  // jsdom doesn't implement scrollIntoView. The Agent Comms tab now
+  // mounts AgentCommsPanel (#231), which scrolls its feed to bottom on
+  // arrival; a no-op stub keeps the panel from throwing under jsdom
+  // (same stub AgentCommsPanel's own render test installs).
+  Element.prototype.scrollIntoView =
+    vi.fn() as unknown as Element["scrollIntoView"];
   // Set up spies on the real api methods. Tests override these per-call.
   const getSpy = vi.spyOn(api, "get");
   const postSpy = vi.spyOn(api, "post");
@@ -474,3 +488,146 @@ describe("MobileChat — chat history", () => {
     expect(getSpy).toHaveBeenCalledTimes(2);
   });
 });
+
+// ─── #232 · Attachment render parity with desktop ChatTab ────────────────────
+//
+// Regression for the CTO-reported mobile bug: MobileChat used to render
+// only m.content (no attachment surface), so files sent/received in a
+// conversation were invisible on mobile while desktop showed them. The
+// fix routes m.attachments through the same AttachmentPreview the
+// desktop ChatTab bubble uses.
+
+describe("MobileChat — attachment render parity (#232)", () => {
+  beforeEach(() => {
+    mockStoreState.nodes = [onlineNode];
+  });
+
+  it("renders an attachment from a history message via AttachmentPreview", async () => {
+    const getSpy = vi.spyOn(api, "get");
+    // useChatHistory reads { messages, reached_end }.
+    getSpy.mockResolvedValueOnce({
+      messages: [
+        {
+          id: "m-att-1",
+          role: "agent",
+          content: "Here is the report",
+          attachments: [
+            {
+              name: "report.csv",
+              uri: "workspace://out/report.csv",
+              mimeType: "text/csv",
+              size: 2048,
+            },
+          ],
+          timestamp: new Date().toISOString(),
+        },
+      ],
+      reached_end: true,
+    });
+
+    let rr: ReturnType<typeof renderChat>;
+    await act(async () => {
+      rr = renderChat(mockAgentId);
+    });
+    const { container } = rr!;
+
+    // A non-image attachment renders the AttachmentChip download button
+    // with title="Download <name>" — same component the desktop bubble
+    // dispatches through AttachmentPreview.
+    await waitFor(() => {
+      const chip = container.querySelector('[title="Download report.csv"]');
+      expect(chip).toBeTruthy();
+    });
+    expect(container.textContent ?? "").toContain("report.csv");
+  });
+});
+
+// ─── #231 · Agent Comms (A2A/peer) render parity with desktop ChatTab ────────
+//
+// Regression for the CTO-reported mobile bug: the Agent Comms sub-tab
+// rendered a static placeholder string ("peer-to-peer A2A traffic
+// surfaces in the Comms tab") instead of the real feed. The fix mounts
+// the same AgentCommsPanel the desktop ChatTab agent-comms tabpanel
+// uses, so peer/A2A + delegation activity is visible on mobile.
+
+describe("MobileChat — Agent Comms render parity (#231)", () => {
+  beforeEach(() => {
+    mockStoreState.nodes = [onlineNode];
+  });
+
+  it("mounts AgentCommsPanel on the Agent Comms tab (not the old placeholder)", async () => {
+    const getSpy = vi.spyOn(api, "get");
+    // 1st GET: useChatHistory (My Chat) on mount.
+    getSpy.mockResolvedValueOnce({ messages: [], reached_end: true });
+    // 2nd GET: AgentCommsPanel's activity load when the tab is shown.
+    // Empty list → panel renders its own empty state, which still
+    // proves AgentCommsPanel mounted (vs. the removed placeholder).
+    getSpy.mockResolvedValueOnce([]);
+
+    let rr: ReturnType<typeof renderChat>;
+    await act(async () => {
+      rr = renderChat(mockAgentId);
+    });
+    const { container } = rr!;
+
+    const commsTab = Array.from(container.querySelectorAll("button")).find(
+      (b) => b.textContent?.trim() === "Agent Comms",
+    );
+    expect(commsTab).toBeTruthy();
+    await act(async () => {
+      commsTab!.click();
+    });
+
+    await waitFor(() => {
+      const text = container.textContent ?? "";
+      // The panel's empty state — proves AgentCommsPanel mounted.
+      expect(text).toContain("No agent-to-agent communications yet.");
+    });
+    // The old hard-coded placeholder must be gone.
+    expect(container.textContent ?? "").not.toContain(
+      "peer-to-peer A2A traffic surfaces in the Comms tab",
+    );
+    // The panel hit its activity endpoint.
+    expect(getSpy).toHaveBeenCalledWith(
+      expect.stringContaining(`/workspaces/${mockAgentId}/activity`),
+    );
+  });
+
+  it("renders a peer message on the Agent Comms tab", async () => {
+    const getSpy = vi.spyOn(api, "get");
+    getSpy.mockResolvedValueOnce({ messages: [], reached_end: true });
+    // a2a_receive from a peer → AgentCommsPanel.toCommMessage maps it
+    // to an inbound bubble with the request text.
+    getSpy.mockResolvedValueOnce([
+      {
+        id: "act-1",
+        activity_type: "a2a_receive",
+        source_id: "peer-ws-uuid",
+        target_id: mockAgentId,
+        method: "message/send",
+        summary: "peer asked something",
+        request_body: { task: "Please review PR 42" },
+        response_body: null,
+        status: "ok",
+        created_at: new Date().toISOString(),
+      },
+    ]);
+
+    let rr: ReturnType<typeof renderChat>;
+    await act(async () => {
+      rr = renderChat(mockAgentId);
+    });
+    const { container } = rr!;
+
+    const commsTab = Array.from(container.querySelectorAll("button")).find(
+      (b) => b.textContent?.trim() === "Agent Comms",
+    );
+    await act(async () => {
+      commsTab!.click();
+    });
+
+    await waitFor(() => {
+      expect(container.textContent ?? "").toContain("Please review PR 42");
+    });
+  });
+});

canvas/src/components/tabs/chat/__tests__/resolveWorkspaceName.test.ts

@@ -0,0 +1,102 @@
+import { describe, it, expect, beforeEach } from "vitest";
+import { useCanvasStore } from "@/store/canvas";
+import { resolveWorkspaceName } from "../hooks/resolveWorkspaceName";
+
+beforeEach(() => {
+  // Reset store to a clean slate between tests so node lookup is deterministic.
+  useCanvasStore.setState({ nodes: [] });
+});
+
+describe("resolveWorkspaceName", () => {
+  it("returns the workspace name when a node with that ID exists", () => {
+    useCanvasStore.setState({
+      nodes: [
+        {
+          id: "ws-alpha-001",
+          type: "workspace",
+          data: { name: "Alpha Agent" },
+          position: { x: 0, y: 0 },
+        },
+      ],
+    });
+
+    expect(resolveWorkspaceName("ws-alpha-001")).toBe("Alpha Agent");
+  });
+
+  it("falls back to the first 8 chars of the ID when no matching node exists", () => {
+    expect(resolveWorkspaceName("ws-zzz-not-found")).toBe("ws-zzz-n");
+  });
+
+  it("falls back to the first 8 chars when the node exists but has no name", () => {
+    useCanvasStore.setState({
+      nodes: [
+        {
+          id: "ws-no-name",
+          type: "workspace",
+          // data.name is deliberately absent
+          data: {},
+          position: { x: 0, y: 0 },
+        },
+      ],
+    });
+
+    expect(resolveWorkspaceName("ws-no-name")).toBe("ws-no-na");
+  });
+
+  it("returns the first 8 chars for a very short ID", () => {
+    expect(resolveWorkspaceName("ab")).toBe("ab");
+  });
+
+  it("returns the first 8 chars when the ID is exactly 8 characters", () => {
+    // slice(0,8) of an 8-char string is the full string
+    const id = "12345678";
+    expect(resolveWorkspaceName(id)).toBe(id);
+  });
+
+  it("picks the right node when multiple workspaces share a prefix", () => {
+    useCanvasStore.setState({
+      nodes: [
+        {
+          id: "00000000-0000-0000-0000-000000000001",
+          type: "workspace",
+          data: { name: "Backend Agent" },
+          position: { x: 0, y: 0 },
+        },
+        {
+          id: "00000000-0000-0000-0000-000000000002",
+          type: "workspace",
+          data: { name: "Frontend Agent" },
+          position: { x: 100, y: 0 },
+        },
+      ],
+    });
+
+    expect(resolveWorkspaceName("00000000-0000-0000-0000-000000000002")).toBe(
+      "Frontend Agent"
+    );
+    expect(resolveWorkspaceName("00000000-0000-0000-0000-000000000001")).toBe(
+      "Backend Agent"
+    );
+  });
+
+  it("does not mutate store state between calls", () => {
+    useCanvasStore.setState({
+      nodes: [
+        {
+          id: "stable-id",
+          type: "workspace",
+          data: { name: "Stable Workspace" },
+          position: { x: 0, y: 0 },
+        },
+      ],
+    });
+
+    resolveWorkspaceName("stable-id");
+    resolveWorkspaceName("unknown-id");
+
+    // Store nodes must be unchanged — resolveWorkspaceName is read-only.
+    const nodes = useCanvasStore.getState().nodes;
+    expect(nodes).toHaveLength(1);
+    expect((nodes[0] as { id: string }).id).toBe("stable-id");
+  });
+});

workspace-server/internal/handlers/plugins_install_test.go

@@ -0,0 +1,53 @@
+package handlers
+
+// plugins_install_test.go — additional coverage for plugins_install.go.
+//
+// Gaps filled vs. existing test files:
+//   - plugins_install_external_test.go: Install + Uninstall 422 (external runtime) ✓ covered
+//   - plugins_test.go:                 Install 400 (missing source, invalid body, etc.) ✓ covered
+//                                    Uninstall 400 (invalid plugin name, empty name) ✓ covered
+//                                    Download auth gate ✓ covered
+//   - org_import_helpers_test.go:      countWorkspaces, envRequirementKey, sanitizeEnvMembers,
+//                                        flattenAndSortRequirements, collectOrgEnv ✓ covered
+//
+// New test added here:
+//   - Uninstall 503: container not running, no SaaS dispatch.
+//
+// NOTE: validateWorkspaceID is not called inside the Install/Uninstall handlers.
+// UUID validation is the responsibility of the WorkspaceAuth middleware, so no
+// 400 test is needed here for UUID format.
+
+import (
+	"encoding/json"
+	"net/http"
+	"net/http/httptest"
+	"testing"
+
+	"github.com/gin-gonic/gin"
+	"github.com/stretchr/testify/require"
+)
+
+// TestPluginUninstall_ContainerNotRunning_Returns503 exercises the 503 path
+// where neither a local Docker container nor a SaaS instance-id dispatch
+// resolves. The handler must return "workspace container not running" — NOT a
+// generic 500 or a misleading 422 (external-runtime) message.
+func TestPluginUninstall_ContainerNotRunning_Returns503(t *testing.T) {
+	// No docker client + no instance-id lookup → falls through to 503.
+	h := NewPluginsHandler(t.TempDir(), nil, nil)
+
+	w := httptest.NewRecorder()
+	c, _ := gin.CreateTestContext(w)
+	c.Params = gin.Params{
+		{Key: "id", Value: "550e8400-e29b-41d4-a716-446655440000"},
+		{Key: "name", Value: "some-plugin"},
+	}
+	c.Request = httptest.NewRequest("DELETE",
+		"/workspaces/550e8400-e29b-41d4-a716-446655440000/plugins/some-plugin", nil)
+
+	h.Uninstall(c)
+
+	require.Equal(t, http.StatusServiceUnavailable, w.Code)
+	var body map[string]string
+	json.Unmarshal(w.Body.Bytes(), &body)
+	require.Equal(t, "workspace container not running", body["error"])
+}

workspace-server/internal/handlers/plugins_listing_test.go

@@ -0,0 +1,472 @@
+package handlers
+
+// Unit tests for plugins_listing.go:
+//   - parseManifestYAML: full YAML, missing fields, empty YAML
+//   - listRegistryFiltered: empty/missing dir, no yaml, valid yaml, runtime filter
+//   - ListRegistry (GET /plugins): no filter, with runtime filter
+//   - ListAvailableForWorkspace (GET /workspaces/:id/plugins/available): runtimeLookup stub
+
+import (
+	"encoding/json"
+	"errors"
+	"net/http"
+	"net/http/httptest"
+	"os"
+	"path/filepath"
+	"testing"
+
+	"github.com/gin-gonic/gin"
+)
+
+// -------- parseManifestYAML --------
+
+func TestParseManifestYAML_FullPlugin(t *testing.T) {
+	data := []byte(`
+name: molecule-audit
+version: 1.2.3
+description: Security audit plugin for Claude Code
+author: Molecule AI
+tags:
+  - security
+  - audit
+skills:
+  - security-scan
+  - compliance-check
+runtimes:
+  - claude_code
+  - hermes
+`)
+	info := parseManifestYAML("fallback-name", data)
+	if info.Name != "fallback-name" {
+		t.Errorf("Name = %q; want fallback-name", info.Name)
+	}
+	if info.Version != "1.2.3" {
+		t.Errorf("Version = %q; want 1.2.3", info.Version)
+	}
+	if info.Description != "Security audit plugin for Claude Code" {
+		t.Errorf("Description = %q; want full description", info.Description)
+	}
+	if info.Author != "Molecule AI" {
+		t.Errorf("Author = %q; want Molecule AI", info.Author)
+	}
+	if len(info.Tags) != 2 || info.Tags[0] != "security" || info.Tags[1] != "audit" {
+		t.Errorf("Tags = %v; want [security audit]", info.Tags)
+	}
+	if len(info.Skills) != 2 || info.Skills[0] != "security-scan" || info.Skills[1] != "compliance-check" {
+		t.Errorf("Skills = %v; want [security-scan compliance-check]", info.Skills)
+	}
+	if len(info.Runtimes) != 2 || info.Runtimes[0] != "claude_code" || info.Runtimes[1] != "hermes" {
+		t.Errorf("Runtimes = %v; want [claude_code hermes]", info.Runtimes)
+	}
+}
+
+func TestParseManifestYAML_MinimalFields(t *testing.T) {
+	// Only name field; all others should be zero-value.
+	data := []byte(`name: minimal-plugin`)
+	info := parseManifestYAML("fallback", data)
+	if info.Name != "fallback" {
+		t.Errorf("Name = %q; want fallback", info.Name)
+	}
+	if info.Version != "" {
+		t.Errorf("Version = %q; want empty", info.Version)
+	}
+	if info.Description != "" {
+		t.Errorf("Description = %q; want empty", info.Description)
+	}
+	if len(info.Tags) != 0 {
+		t.Errorf("Tags = %v; want []", info.Tags)
+	}
+	if len(info.Skills) != 0 {
+		t.Errorf("Skills = %v; want []", info.Skills)
+	}
+	if len(info.Runtimes) != 0 {
+		t.Errorf("Runtimes = %v; want []", info.Runtimes)
+	}
+}
+
+func TestParseManifestYAML_MissingPluginYAML(t *testing.T) {
+	// No plugin.yaml present → returns fallback name only.
+	info := parseManifestYAML("no-file", nil)
+	if info.Name != "no-file" {
+		t.Errorf("Name = %q; want no-file", info.Name)
+	}
+}
+
+func TestParseManifestYAML_BadYAML(t *testing.T) {
+	// Malformed YAML → returns fallback name only (no panic).
+	info := parseManifestYAML("bad-yaml", []byte("not: [yaml: at all"))
+	if info.Name != "bad-yaml" {
+		t.Errorf("Name = %q; want bad-yaml", info.Name)
+	}
+	if info.Version != "" {
+		t.Errorf("Version = %q; want empty after bad YAML", info.Version)
+	}
+}
+
+func TestParseManifestYAML_PartialFields(t *testing.T) {
+	// Present tags/skills/runtimes that are not []interface{} (e.g. wrong type)
+	// should not panic and should leave the field empty.
+	data := []byte(`
+name: partial
+tags: "not-an-array"
+skills: 123
+runtimes: true
+`)
+	info := parseManifestYAML("partial", data)
+	if info.Name != "partial" {
+		t.Errorf("Name = %q; want partial", info.Name)
+	}
+	if len(info.Tags) != 0 {
+		t.Errorf("Tags = %v; want [] (wrong type)", info.Tags)
+	}
+	if len(info.Skills) != 0 {
+		t.Errorf("Skills = %v; want [] (wrong type)", info.Skills)
+	}
+	if len(info.Runtimes) != 0 {
+		t.Errorf("Runtimes = %v; want [] (wrong type)", info.Runtimes)
+	}
+}
+
+// -------- listRegistryFiltered --------
+
+func makeTestHandler(t *testing.T, pluginsDir string) *PluginsHandler {
+	// Construct a minimal PluginsHandler with a nil docker client
+	// (filesystem paths are tested directly; container-dependent paths are
+	// tested separately or skipped in this file).
+	h := &PluginsHandler{pluginsDir: pluginsDir}
+	return h
+}
+
+func writePluginYAML(t *testing.T, dir, name, content string) {
+	path := filepath.Join(dir, name, "plugin.yaml")
+	if err := os.MkdirAll(filepath.Dir(path), 0o755); err != nil {
+		t.Fatalf("mkdir: %v", err)
+	}
+	if err := os.WriteFile(path, []byte(content), 0o644); err != nil {
+		t.Fatalf("writeFile: %v", err)
+	}
+}
+
+func TestListRegistryFiltered_EmptyDir(t *testing.T) {
+	dir := t.TempDir()
+	h := makeTestHandler(t, dir)
+	got := h.listRegistryFiltered("")
+	if len(got) != 0 {
+		t.Errorf("expected empty list for empty dir; got %d plugins", len(got))
+	}
+}
+
+func TestListRegistryFiltered_NonExistentDir(t *testing.T) {
+	h := makeTestHandler(t, "/does/not/exist")
+	got := h.listRegistryFiltered("")
+	if len(got) != 0 {
+		t.Errorf("expected empty list for nonexistent dir; got %d plugins", len(got))
+	}
+}
+
+func TestListRegistryFiltered_NoPluginYAML(t *testing.T) {
+	// Plugin directory exists but has no plugin.yaml → fallback name only.
+	dir := t.TempDir()
+	writePluginYAML(t, dir, "no-manifest-plugin", "")
+	h := makeTestHandler(t, dir)
+	got := h.listRegistryFiltered("")
+	if len(got) != 1 {
+		t.Fatalf("expected 1 plugin; got %d", len(got))
+	}
+	if got[0].Name != "no-manifest-plugin" {
+		t.Errorf("Name = %q; want no-manifest-plugin", got[0].Name)
+	}
+}
+
+func TestListRegistryFiltered_ValidPlugin(t *testing.T) {
+	dir := t.TempDir()
+	writePluginYAML(t, dir, "molecule-audit", `
+name: molecule-audit
+version: 1.0.0
+description: Security audit plugin
+author: Molecule AI
+tags:
+  - security
+skills:
+  - audit
+runtimes:
+  - hermes
+`)
+	h := makeTestHandler(t, dir)
+	got := h.listRegistryFiltered("")
+	if len(got) != 1 {
+		t.Fatalf("expected 1 plugin; got %d", len(got))
+	}
+	if got[0].Name != "molecule-audit" {
+		t.Errorf("Name = %q; want molecule-audit", got[0].Name)
+	}
+	if got[0].Version != "1.0.0" {
+		t.Errorf("Version = %q; want 1.0.0", got[0].Version)
+	}
+	if len(got[0].Tags) != 1 || got[0].Tags[0] != "security" {
+		t.Errorf("Tags = %v; want [security]", got[0].Tags)
+	}
+}
+
+func TestListRegistryFiltered_FilesIgnored(t *testing.T) {
+	// Regular files in pluginsDir are skipped (only directories are scanned).
+	dir := t.TempDir()
+	writePluginYAML(t, dir, "real-plugin", `
+name: real-plugin
+version: 1.0.0
+`)
+	f, err := os.Create(filepath.Join(dir, "not-a-plugin.txt"))
+	if err != nil {
+		t.Fatal(err)
+	}
+	f.Close()
+	h := makeTestHandler(t, dir)
+	got := h.listRegistryFiltered("")
+	if len(got) != 1 || got[0].Name != "real-plugin" {
+		t.Errorf("expected only real-plugin; got %v", got)
+	}
+}
+
+func TestListRegistryFiltered_RuntimeFilterMatches(t *testing.T) {
+	dir := t.TempDir()
+	writePluginYAML(t, dir, "cc-plugin", `
+name: cc-plugin
+runtimes: [claude_code]
+`)
+	writePluginYAML(t, dir, "hermes-plugin", `
+name: hermes-plugin
+runtimes: [hermes]
+`)
+	h := makeTestHandler(t, dir)
+
+	// With hermes filter → only hermes-plugin returned.
+	got := h.listRegistryFiltered("hermes")
+	if len(got) != 1 || got[0].Name != "hermes-plugin" {
+		t.Errorf("expected [hermes-plugin]; got %v", got)
+	}
+
+	// With claude-code filter → hyphen normalises to underscore → cc-plugin returned.
+	got2 := h.listRegistryFiltered("claude-code")
+	if len(got2) != 1 || got2[0].Name != "cc-plugin" {
+		t.Errorf("expected [cc-plugin] with claude-code filter; got %v", got2)
+	}
+}
+
+func TestListRegistryFiltered_RuntimeFilterExcludes(t *testing.T) {
+	// Plugin declares hermes; query asks for claude-code → plugin excluded.
+	dir := t.TempDir()
+	writePluginYAML(t, dir, "hermes-only", `
+name: hermes-only
+runtimes: [hermes]
+`)
+	h := makeTestHandler(t, dir)
+	got := h.listRegistryFiltered("claude_code")
+	if len(got) != 0 {
+		t.Errorf("expected empty list for mismatched runtime; got %v", got)
+	}
+}
+
+func TestListRegistryFiltered_UnspecifiedRuntimeIncluded(t *testing.T) {
+	// Plugin with no runtimes field is included in any filtered query
+	// ("unspecified = try it" contract).
+	dir := t.TempDir()
+	writePluginYAML(t, dir, "universal-plugin", `
+name: universal-plugin
+runtimes: []
+`)
+	h := makeTestHandler(t, dir)
+	got := h.listRegistryFiltered("any-runtime")
+	if len(got) != 1 || got[0].Name != "universal-plugin" {
+		t.Errorf("expected [universal-plugin] with any runtime filter; got %v", got)
+	}
+}
+
+func TestListRegistryFiltered_MultipleMatching(t *testing.T) {
+	dir := t.TempDir()
+	for _, name := range []string{"plugin-a", "plugin-b", "plugin-c"} {
+		writePluginYAML(t, dir, name, `name: `+name+`
+runtimes: [hermes, claude_code]
+`)
+	}
+	h := makeTestHandler(t, dir)
+	got := h.listRegistryFiltered("hermes")
+	if len(got) != 3 {
+		t.Errorf("expected 3 plugins; got %d: %v", len(got), got)
+	}
+}
+
+// -------- ListRegistry (GET /plugins) --------
+
+func listRegistryReq(runtime string) (*http.Request, *httptest.ResponseRecorder, *gin.Context) {
+	url := "/plugins"
+	if runtime != "" {
+		url += "?runtime=" + runtime
+	}
+	w := httptest.NewRecorder()
+	c, _ := gin.CreateTestContext(w)
+	c.Request = httptest.NewRequest("GET", url, nil)
+	return c.Request, w, c
+}
+
+func TestListRegistry_NoFilter(t *testing.T) {
+	dir := t.TempDir()
+	writePluginYAML(t, dir, "test-plugin", `
+name: test-plugin
+version: 0.1.0
+`)
+	h := makeTestHandler(t, dir)
+	_, w, c := listRegistryReq("")
+	h.ListRegistry(c)
+	if w.Code != http.StatusOK {
+		t.Errorf("expected 200; got %d: %s", w.Code, w.Body.String())
+	}
+	var resp []map[string]interface{}
+	if err := json.Unmarshal(w.Body.Bytes(), &resp); err != nil {
+		t.Fatalf("unmarshal: %v", err)
+	}
+	if len(resp) != 1 || resp[0]["name"] != "test-plugin" {
+		t.Errorf("unexpected response: %v", resp)
+	}
+}
+
+func TestListRegistry_WithRuntimeFilter(t *testing.T) {
+	dir := t.TempDir()
+	writePluginYAML(t, dir, "hermes-plugin", `
+name: hermes-plugin
+runtimes: [hermes]
+`)
+	writePluginYAML(t, dir, "cc-plugin", `
+name: cc-plugin
+runtimes: [claude_code]
+`)
+	h := makeTestHandler(t, dir)
+	_, w, c := listRegistryReq("hermes")
+	h.ListRegistry(c)
+	if w.Code != http.StatusOK {
+		t.Errorf("expected 200; got %d", w.Code)
+	}
+	var resp []map[string]interface{}
+	json.Unmarshal(w.Body.Bytes(), &resp)
+	if len(resp) != 1 || resp[0]["name"] != "hermes-plugin" {
+		t.Errorf("expected [hermes-plugin]; got %v", resp)
+	}
+}
+
+func TestListRegistry_EmptyOnNoMatches(t *testing.T) {
+	dir := t.TempDir()
+	writePluginYAML(t, dir, "cc-plugin", `name: cc-plugin
+runtimes: [claude_code]
+`)
+	h := makeTestHandler(t, dir)
+	_, w, c := listRegistryReq("nonexistent")
+	h.ListRegistry(c)
+	if w.Code != http.StatusOK {
+		t.Errorf("expected 200; got %d", w.Code)
+	}
+	var resp []map[string]interface{}
+	json.Unmarshal(w.Body.Bytes(), &resp)
+	if len(resp) != 0 {
+		t.Errorf("expected empty list; got %v", resp)
+	}
+}
+
+// -------- ListAvailableForWorkspace (GET /workspaces/:id/plugins/available) --------
+
+func listAvailableReq(workspaceID string) (*http.Request, *httptest.ResponseRecorder, *gin.Context) {
+	w := httptest.NewRecorder()
+	c, _ := gin.CreateTestContext(w)
+	c.Params = gin.Params{{Key: "id", Value: workspaceID}}
+	c.Request = httptest.NewRequest("GET", "/workspaces/"+workspaceID+"/plugins/available", nil)
+	return c.Request, w, c
+}
+
+func TestListAvailableForWorkspace_RuntimeLookupReturnsRuntime(t *testing.T) {
+	dir := t.TempDir()
+	writePluginYAML(t, dir, "hermes-plugin", `
+name: hermes-plugin
+runtimes: [hermes]
+`)
+	writePluginYAML(t, dir, "cc-plugin", `
+name: cc-plugin
+runtimes: [claude_code]
+`)
+	h := makeTestHandler(t, dir)
+	h.runtimeLookup = func(workspaceID string) (string, error) {
+		return "hermes", nil
+	}
+	_, w, c := listAvailableReq("00000000-0000-0000-0000-000000000001")
+	h.ListAvailableForWorkspace(c)
+	if w.Code != http.StatusOK {
+		t.Errorf("expected 200; got %d", w.Code)
+	}
+	var resp []map[string]interface{}
+	json.Unmarshal(w.Body.Bytes(), &resp)
+	if len(resp) != 1 || resp[0]["name"] != "hermes-plugin" {
+		t.Errorf("expected [hermes-plugin]; got %v", resp)
+	}
+}
+
+func TestListAvailableForWorkspace_RuntimeLookupErrors(t *testing.T) {
+	// runtimeLookup error → runtime="" → full registry returned.
+	dir := t.TempDir()
+	writePluginYAML(t, dir, "plugin-a", `name: plugin-a
+runtimes: [hermes]
+`)
+	writePluginYAML(t, dir, "plugin-b", `name: plugin-b
+runtimes: [claude_code]
+`)
+	h := makeTestHandler(t, dir)
+	h.runtimeLookup = func(workspaceID string) (string, error) {
+		return "", errors.New("runtime lookup failed")
+	}
+	_, w, c := listAvailableReq("00000000-0000-0000-0000-000000000002")
+	h.ListAvailableForWorkspace(c)
+	if w.Code != http.StatusOK {
+		t.Errorf("expected 200; got %d", w.Code)
+	}
+	var resp []map[string]interface{}
+	json.Unmarshal(w.Body.Bytes(), &resp)
+	if len(resp) != 2 {
+		t.Errorf("expected 2 plugins (full registry fallback); got %d: %v", len(resp), resp)
+	}
+}
+
+func TestListAvailableForWorkspace_NoRuntimeLookup(t *testing.T) {
+	// runtimeLookup nil → full registry (no filter).
+	dir := t.TempDir()
+	writePluginYAML(t, dir, "plugin-x", `name: plugin-x`)
+	h := makeTestHandler(t, dir)
+	// runtimeLookup is nil by default from makeTestHandler.
+	_, w, c := listAvailableReq("00000000-0000-0000-0000-000000000003")
+	h.ListAvailableForWorkspace(c)
+	if w.Code != http.StatusOK {
+		t.Errorf("expected 200; got %d", w.Code)
+	}
+	var resp []map[string]interface{}
+	json.Unmarshal(w.Body.Bytes(), &resp)
+	if len(resp) != 1 || resp[0]["name"] != "plugin-x" {
+		t.Errorf("expected [plugin-x]; got %v", resp)
+	}
+}
+
+func TestListAvailableForWorkspace_UnspecifiedRuntimePluginsAlwaysIncluded(t *testing.T) {
+	// Plugins with empty runtimes list should always be included
+	// regardless of workspace runtime.
+	dir := t.TempDir()
+	writePluginYAML(t, dir, "universal", `name: universal
+runtimes: []
+`)
+	writePluginYAML(t, dir, "cc-only", `name: cc-only
+runtimes: [claude_code]
+`)
+	h := makeTestHandler(t, dir)
+	h.runtimeLookup = func(id string) (string, error) { return "hermes", nil }
+	_, w, c := listAvailableReq("ws-001")
+	h.ListAvailableForWorkspace(c)
+	var resp []map[string]interface{}
+	json.Unmarshal(w.Body.Bytes(), &resp)
+	// "universal" has no runtimes (try-it); "cc-only" doesn't support hermes.
+	if len(resp) != 1 || resp[0]["name"] != "universal" {
+		t.Errorf("expected [universal]; got %v", resp)
+	}
+}

workspace-server/internal/handlers/workspace_abilities_test.go

@@ -0,0 +1,193 @@
+package handlers
+
+import (
+	"bytes"
+	"database/sql"
+	"encoding/json"
+	"net/http"
+	"net/http/httptest"
+	"testing"
+
+	"github.com/DATA-DOG/go-sqlmock"
+	"github.com/gin-gonic/gin"
+)
+
+// patchReq builds a gin context for a PATCH request to /workspaces/:id/abilities.
+func patchReq(id, body string) (*http.Request, *httptest.ResponseRecorder, *gin.Context) {
+	w := httptest.NewRecorder()
+	c, _ := gin.CreateTestContext(w)
+	c.Params = gin.Params{{Key: "id", Value: id}}
+	c.Request = httptest.NewRequest("PATCH", "/workspaces/"+id+"/abilities", bytes.NewBufferString(body))
+	c.Request.Header.Set("Content-Type", "application/json")
+	return c.Request, w, c
+}
+
+func TestPatchAbilities_InvalidWorkspaceID(t *testing.T) {
+	setupTestDB(t)
+
+	// "not-a-uuid" fails validateWorkspaceID
+	_, w, c := patchReq("not-a-uuid", `{"broadcast_enabled":true}`)
+	PatchAbilities(c)
+
+	if w.Code != http.StatusBadRequest {
+		t.Errorf("expected 400, got %d: %s", w.Code, w.Body.String())
+	}
+}
+
+func TestPatchAbilities_EmptyBody(t *testing.T) {
+	setupTestDB(t)
+	id := "00000000-0000-0000-0000-000000000001"
+
+	// Empty JSON object — no ability fields present
+	_, w, c := patchReq(id, `{}`)
+	PatchAbilities(c)
+
+	if w.Code != http.StatusBadRequest {
+		t.Errorf("expected 400, got %d: %s", w.Code, w.Body.String())
+	}
+
+	var resp map[string]string
+	json.Unmarshal(w.Body.Bytes(), &resp)
+	if resp["error"] != "at least one ability field required" {
+		t.Errorf("expected 'at least one ability field required', got %v", resp["error"])
+	}
+}
+
+func TestPatchAbilities_WorkspaceNotFound(t *testing.T) {
+	mock := setupTestDB(t)
+	id := "00000000-0000-0000-0000-000000000002"
+
+	// SELECT EXISTS returns false (workspace does not exist)
+	mock.ExpectQuery(`SELECT EXISTS\(SELECT 1 FROM workspaces WHERE id = \$1 AND status != 'removed'\)`).
+		WithArgs(id).
+		WillReturnRows(sqlmock.NewRows([]string{"exists"}).AddRow(false))
+
+	_, w, c := patchReq(id, `{"broadcast_enabled":true}`)
+	PatchAbilities(c)
+
+	if w.Code != http.StatusNotFound {
+		t.Errorf("expected 404, got %d: %s", w.Code, w.Body.String())
+	}
+}
+
+func TestPatchAbilities_SetBroadcastEnabledTrue(t *testing.T) {
+	mock := setupTestDB(t)
+	id := "00000000-0000-0000-0000-000000000003"
+
+	// SELECT EXISTS → true
+	mock.ExpectQuery(`SELECT EXISTS\(SELECT 1 FROM workspaces WHERE id = \$1 AND status != 'removed'\)`).
+		WithArgs(id).
+		WillReturnRows(sqlmock.NewRows([]string{"exists"}).AddRow(true))
+
+	// UPDATE broadcast_enabled = true
+	mock.ExpectExec(`UPDATE workspaces SET broadcast_enabled = \$2, updated_at = now\(\) WHERE id = \$1`).
+		WithArgs(id, true).
+		WillReturnResult(sqlmock.NewResult(0, 1))
+
+	_, w, c := patchReq(id, `{"broadcast_enabled":true}`)
+	PatchAbilities(c)
+
+	if w.Code != http.StatusOK {
+		t.Errorf("expected 200, got %d: %s", w.Code, w.Body.String())
+	}
+
+	var resp map[string]string
+	json.Unmarshal(w.Body.Bytes(), &resp)
+	if resp["status"] != "updated" {
+		t.Errorf("expected status=updated, got %v", resp["status"])
+	}
+}
+
+func TestPatchAbilities_SetTalkToUserEnabledFalse(t *testing.T) {
+	mock := setupTestDB(t)
+	id := "00000000-0000-0000-0000-000000000004"
+
+	// SELECT EXISTS → true
+	mock.ExpectQuery(`SELECT EXISTS\(SELECT 1 FROM workspaces WHERE id = \$1 AND status != 'removed'\)`).
+		WithArgs(id).
+		WillReturnRows(sqlmock.NewRows([]string{"exists"}).AddRow(true))
+
+	// UPDATE talk_to_user_enabled = false
+	mock.ExpectExec(`UPDATE workspaces SET talk_to_user_enabled = \$2, updated_at = now\(\) WHERE id = \$1`).
+		WithArgs(id, false).
+		WillReturnResult(sqlmock.NewResult(0, 1))
+
+	_, w, c := patchReq(id, `{"talk_to_user_enabled":false}`)
+	PatchAbilities(c)
+
+	if w.Code != http.StatusOK {
+		t.Errorf("expected 200, got %d: %s", w.Code, w.Body.String())
+	}
+}
+
+func TestPatchAbilities_BothFields(t *testing.T) {
+	mock := setupTestDB(t)
+	id := "00000000-0000-0000-0000-000000000005"
+
+	// SELECT EXISTS → true
+	mock.ExpectQuery(`SELECT EXISTS\(SELECT 1 FROM workspaces WHERE id = \$1 AND status != 'removed'\)`).
+		WithArgs(id).
+		WillReturnRows(sqlmock.NewRows([]string{"exists"}).AddRow(true))
+
+	// UPDATE broadcast_enabled = false
+	mock.ExpectExec(`UPDATE workspaces SET broadcast_enabled = \$2, updated_at = now\(\) WHERE id = \$1`).
+		WithArgs(id, false).
+		WillReturnResult(sqlmock.NewResult(0, 1))
+
+	// UPDATE talk_to_user_enabled = true
+	mock.ExpectExec(`UPDATE workspaces SET talk_to_user_enabled = \$2, updated_at = now\(\) WHERE id = \$1`).
+		WithArgs(id, true).
+		WillReturnResult(sqlmock.NewResult(0, 1))
+
+	_, w, c := patchReq(id, `{"broadcast_enabled":false,"talk_to_user_enabled":true}`)
+	PatchAbilities(c)
+
+	if w.Code != http.StatusOK {
+		t.Errorf("expected 200, got %d: %s", w.Code, w.Body.String())
+	}
+}
+
+func TestPatchAbilities_BroadcastUpdateFails(t *testing.T) {
+	mock := setupTestDB(t)
+	id := "00000000-0000-0000-0000-000000000006"
+
+	// SELECT EXISTS → true
+	mock.ExpectQuery(`SELECT EXISTS\(SELECT 1 FROM workspaces WHERE id = \$1 AND status != 'removed'\)`).
+		WithArgs(id).
+		WillReturnRows(sqlmock.NewRows([]string{"exists"}).AddRow(true))
+
+	// UPDATE fails
+	mock.ExpectExec(`UPDATE workspaces SET broadcast_enabled = \$2, updated_at = now\(\) WHERE id = \$1`).
+		WithArgs(id, true).
+		WillReturnError(sql.ErrConnDone)
+
+	_, w, c := patchReq(id, `{"broadcast_enabled":true}`)
+	PatchAbilities(c)
+
+	if w.Code != http.StatusInternalServerError {
+		t.Errorf("expected 500, got %d: %s", w.Code, w.Body.String())
+	}
+}
+
+func TestPatchAbilities_TalkToUserUpdateFails(t *testing.T) {
+	mock := setupTestDB(t)
+	id := "00000000-0000-0000-0000-000000000007"
+
+	// SELECT EXISTS → true
+	mock.ExpectQuery(`SELECT EXISTS\(SELECT 1 FROM workspaces WHERE id = \$1 AND status != 'removed'\)`).
+		WithArgs(id).
+		WillReturnRows(sqlmock.NewRows([]string{"exists"}).AddRow(true))
+
+	// UPDATE broadcast_enabled skipped (not in payload)
+	// UPDATE talk_to_user_enabled fails
+	mock.ExpectExec(`UPDATE workspaces SET talk_to_user_enabled = \$2, updated_at = now\(\) WHERE id = \$1`).
+		WithArgs(id, false).
+		WillReturnError(sql.ErrConnDone)
+
+	_, w, c := patchReq(id, `{"talk_to_user_enabled":false}`)
+	PatchAbilities(c)
+
+	if w.Code != http.StatusInternalServerError {
+		t.Errorf("expected 500, got %d: %s", w.Code, w.Body.String())
+	}
+}