feat(attention): open comment-author settings from GitHub watches

docs/plugin-architecture.md

@@ -340,6 +340,8 @@ plugin is replaced, so ask for it when acting rather than keeping it. Like any t
 
 Plugins can discover active Attention source registrations through `ctx.agents.attentionProviders()`. Each entry includes its `provider` key and optional `description` and JSON-object `configExample` supplied to `addAttentionSource`. The catalog omits executable callbacks and inactive registrations; discovery tools belong in external plugins.
 
+A source may also `present` its watches on the Agent Attention page. Given a watch and its display name, it returns the watch's kind and, optionally, an icon, a target with a subtitle, an external link, and a details component that replaces the name in the expanded row. When a provider has no `present`, or it throws, Buzz shows the watch by its name. The GitHub plugin presents pull request watches this way, linking to its comment author settings.
+
 Each Attention source `run` receives `spawn(processId, options)` for commands using that watch's owning agent. The host binds agent identity, watch destination and run lifetime; plugins cannot select another agent or obtain the raw authenticated connection. Declare the process in `host.processes` and request `host.mcp`, plus `host.files` for tools reading local paths. Options supply arguments, working directory, environment and output callbacks.
 
 Buzz revokes command access on return, failure, cancellation, agent stop, provider unload or community reconnection, and closes the process and its descendants. Successfully emitted output remains queued after a producer returns. The key and MCP connection stay in Buzz; commands retain the app user's operating-system privileges and can use the agent's tools while their run remains active.

src/bundled/agents/AgentsPage.test.tsx

@@ -372,6 +372,7 @@ function setup(
       for (const listener of listeners) listener();
     }),
     attentionProviders: () => [],
+    presentWatch: () => undefined,
     addAttentionSource: vi.fn(),
     storage: localStore(),
     resumeInterrupted: vi.fn(),

src/bundled/github/SoloPull.test.tsx

@@ -183,7 +183,10 @@ it("gives agents a github_link_pr tool for the Solo chat they work in", async ()
     key: "a".repeat(64),
     pubkey: "a".repeat(64),
   } as BuzzClient;
-  expect(inject).toHaveBeenCalledWith(["agents", "host"], expect.any(Function));
+  expect(inject).toHaveBeenCalledWith(
+    ["agents", "host", "navigation"],
+    expect.any(Function),
+  );
   expect(tool?.name).toBe("github_link_pr");
   await expect(
     tool?.call(client, { channel: chat }, { url: "https://github.com/block" }),

src/bundled/github/attention.test.tsx

@@ -0,0 +1,51 @@
+// @vitest-environment jsdom
+import { cleanup, render, screen } from "@testing-library/react";
+import userEvent from "@testing-library/user-event";
+import { afterEach, expect, it, vi } from "vitest";
+import { GitPullRequestIcon } from "../../shared/design-system/icons";
+import { presentPull } from "./attention";
+
+afterEach(cleanup);
+
+const watch = (pr: unknown) => ({
+  type: "source" as const,
+  interest_id: "release",
+  enabled: true,
+  provider: "attention/github-pr",
+  config: { pr },
+});
+
+it("names the pull request and opens whose feedback wakes agents", async () => {
+  const open = vi.fn();
+  const shown = presentPull({ open })(
+    watch("https://github.com/block/buzz-app/pull/1578"),
+    "buzz-app#1578 Release checks",
+  );
+  expect(shown).toMatchObject({
+    type: "GitHub PR",
+    icon: GitPullRequestIcon,
+    target: "buzz-app#1578",
+    subtitle: "Release checks",
+    link: {
+      href: "https://github.com/block/buzz-app/pull/1578",
+      title: "Open on GitHub",
+    },
+  });
+  const Details = shown.details as NonNullable<typeof shown.details>;
+  render(<Details />);
+  await userEvent.click(
+    screen.getByRole("button", { name: "Choose who can wake agents" }),
+  );
+  expect(open).toHaveBeenCalledWith({
+    version: 1,
+    kind: "settings",
+    section: "buzz.github/authors",
+  });
+});
+
+it("leaves a watch without a pull request URL to its name", () => {
+  expect(presentPull({ open: vi.fn() })(watch(42), "Release")).toEqual({
+    type: "GitHub PR",
+    icon: GitPullRequestIcon,
+  });
+});

src/bundled/github/attention.tsx

@@ -0,0 +1,42 @@
+// How a pull request watch looks in Agent Attention.
+import { GitPullRequestIcon } from "../../shared/design-system/icons";
+import { Button } from "../../shared/design-system/ui/Button";
+import type { AttentionSource } from "../../features/agents/attention-sources";
+import type { Navigation } from "../../features/navigation/controller";
+import { pullOf } from "./watch";
+
+/** Names the pull request a watch follows, and opens the settings for whose
+ * feedback wakes agents. */
+export function presentPull(
+  navigation: Pick<Navigation, "open">,
+): NonNullable<AttentionSource["present"]> {
+  const Authors = () => (
+    <Button
+      variant="link"
+      size="inline"
+      type="button"
+      onClick={() =>
+        void navigation.open({
+          version: 1,
+          kind: "settings",
+          section: "buzz.github/authors",
+        })
+      }
+    >
+      Choose who can wake agents
+    </Button>
+  );
+  return (watch, title) => {
+    const pull =
+      typeof watch.config.pr === "string" ? pullOf(watch.config.pr) : undefined;
+    if (!pull) return { type: "GitHub PR", icon: GitPullRequestIcon };
+    return {
+      type: "GitHub PR",
+      icon: GitPullRequestIcon,
+      target: `${pull.name}#${pull.number}`,
+      subtitle: title.replace(/^(?:PR\s+\d+:?\s*|[^ ]*#\d+\s*)/i, ""),
+      link: { href: pull.url, title: "Open on GitHub" },
+      details: Authors,
+    };
+  };
+}

src/bundled/github/index.tsx

@@ -43,6 +43,7 @@ import { stateIcons } from "./pull-state-view";
 import { PullLink } from "./PullLink";
 import { SoloPullIcon, SoloPullLink, SoloPullMenu } from "./SoloPull";
 import { watchPulls } from "./watcher";
+import { presentPull } from "./attention";
 import { SubmitReview } from "./SubmitReview";
 import { AuthorSettings } from "./AuthorSettings";
 
@@ -184,7 +185,9 @@ export const apply: PluginModule["apply"] = (ctx) => {
   });
   addPullStateTool(ctx.agents, states);
   // Agents can watch pull requests where the app runs them.
-  ctx.inject(["agents", "host"], (inner) => watchPulls(inner, states, gh));
+  ctx.inject(["agents", "host", "navigation"], (scope) =>
+    watchPulls(scope, states, gh, presentPull(scope.navigation)),
+  );
   // Whose comments and reviews wake them.
   ctx.inject(["settingsCards"], (scope) => {
     scope.settingsCards.register({

src/bundled/github/watcher.test.ts

@@ -144,7 +144,10 @@ const asks = (app: ReturnType<typeof setup>, login: string) =>
     ([query]) => membershipRead(query) && query.includes(`login:"${login}"`),
   ).length;
 
-function setup(initial: Pull | null = answer()) {
+function setup(
+  initial: Pull | null = answer(),
+  present?: AttentionSource["present"],
+) {
   let gh = initial;
   let texts = true;
   let members: readonly string[] | null = [];
@@ -286,7 +289,7 @@ function setup(initial: Pull | null = answer()) {
     },
     effect: (fn: () => () => void) => disposers.push(fn()),
   } as unknown as Context;
-  watchPulls(ctx, states, github as unknown as GitHubCli);
+  watchPulls(ctx, states, github as unknown as GitHubCli, present);
   const tool = tools[0] as AddedTool;
   const call = (
     args: Record<string, unknown>,
@@ -355,10 +358,15 @@ describe("github_watch_pr", () => {
   });
   afterEach(() => vi.useRealTimers());
 
-  it("registers PR watching as an Attention source", () => {
-    const app = setup();
+  it("registers PR watching as an Attention source with its presenter", () => {
+    const present = vi.fn();
+    const app = setup(answer(), present);
     expect(app.addAttentionSource).toHaveBeenCalledWith(
-      expect.objectContaining({ id: "github-pr", run: expect.any(Function) }),
+      expect.objectContaining({
+        id: "github-pr",
+        run: expect.any(Function),
+        present,
+      }),
     );
     app.dispose();
   });

src/bundled/github/watcher.ts

@@ -3,6 +3,7 @@
 // and delivery.
 import type { Context } from "@deepseek-ai/cordis";
 import { callAttentionTool } from "../../buzz-mcp/attention";
+import type { AttentionSource } from "../../features/agents/attention-sources";
 import type { GitHubCli } from "./gh";
 import type { Feedback, PullStates } from "./pull-states";
 import {
@@ -91,6 +92,7 @@ export function watchPulls(
   ctx: Context,
   states: PullStates,
   gh: GitHubCli | undefined,
+  present?: AttentionSource["present"],
 ) {
   localStorage.removeItem(RETIRED_STORAGE_KEY);
   const cursors = load();
@@ -227,6 +229,7 @@ export function watchPulls(
     id: "github-pr",
     description: "Follow GitHub pull request feedback and CI",
     configExample: { pr: "https://github.com/OWNER/REPO/pull/NUMBER" },
+    ...(present && { present }),
     async run({ id, agent, watch: source, signal, emit }) {
       const pull =
         typeof source.config.pr === "string"

src/bundled/watching/Tab.test.tsx

@@ -112,6 +112,7 @@ it("shows one agent's watches, reacts to changes and surfaces save errors", asyn
   const ctx = {
     react: React,
     agents: {
+      presentWatch: () => undefined,
       snapshot: () => snapshot,
       subscribe: (listener: () => void) => {
         changed = listener;
@@ -271,6 +272,7 @@ it.each(["source", "event", "timer"] as const)(
     const ctx = {
       react: React,
       agents: {
+        presentWatch: () => undefined,
         snapshot: () => snapshot,
         subscribe: (listener: () => void) => {
           changed = listener;
@@ -399,7 +401,12 @@ it("keeps subscription controls disabled and the tab open while saving, and pres
   const save = vi.fn(() => held);
   const ctx = {
     react: React,
-    agents: { snapshot: () => snapshot, subscribe: () => () => {}, save },
+    agents: {
+      presentWatch: () => undefined,
+      snapshot: () => snapshot,
+      subscribe: () => () => {},
+      save,
+    },
     relay: { snapshot: () => disconnected, subscribe: () => () => {} },
   } as unknown as Context;
   const report = vi.fn();
@@ -737,6 +744,19 @@ it.each([false, true])(
     const ctx = {
       react: React,
       agents: {
+        presentWatch: (watch: { provider: string }) =>
+          watch.provider === "attention/github-pr"
+            ? {
+                type: "GitHub PR",
+                target: "buzz-app#1578",
+                subtitle: "Release checks",
+                link: {
+                  href: "https://github.com/block/buzz-app/pull/1578",
+                  title: "Open on GitHub",
+                },
+                details: () => React.createElement("p", null, "Who wakes it"),
+              }
+            : undefined,
         snapshot: () => snapshot,
         subscribe: (listener: () => void) => {
           changed = listener;
@@ -778,6 +798,7 @@ it.each([false, true])(
     );
     expect(screen.getByRole("heading", { name: "Recent events" })).toBeTruthy();
     expect(screen.getByText("CI completed: all checks passed")).toBeTruthy();
+    expect(screen.getByText("Who wakes it")).toBeTruthy();
     fireEvent.click(
       screen.getByRole("button", { name: "Details for Discussion" }),
     );
@@ -861,7 +882,12 @@ it("keeps overview and destination navigation in the selected community", async
   const open = vi.fn(),
     save = vi.fn();
   const ctx = {
-    agents: { snapshot: () => snapshot, subscribe: () => () => {}, save },
+    agents: {
+      presentWatch: () => undefined,
+      snapshot: () => snapshot,
+      subscribe: () => () => {},
+      save,
+    },
     relay: {
       snapshot: () => connection,
       subscribe: (listener: () => void) => {