✨ navigate_tab 工具、researcher 页面只读能力、精简 task 工具

- 新增 navigate_tab 工具：导航已有标签页到新 URL，支持等待加载完成
- researcher 子代理增加页面只读能力（get_tab_content/open_tab/list_tabs/close_tab/navigate_tab），但不允许 DOM 交互
- 移除 get_task 和 delete_task 工具，list_tasks 改为返回完整任务信息
- 页面交互指南仅在同时拥有 get_tab_content 和 execute_script 时显示

Author: CodFrm

src/app/service/agent/core/sub_agent_types.test.ts

@@ -37,29 +37,31 @@ describe("Sub-Agent 类型系统", () => {
       "open_tab",
       "close_tab",
       "activate_tab",
+      "navigate_tab",
       "ask_user",
       "agent",
       "create_task",
       "update_task",
-      "get_task",
       "list_tasks",
-      "delete_task",
     ];
 
-    it.concurrent("researcher 类型排除 tab 工具和其他不在白名单中的工具", () => {
+    it.concurrent("researcher 类型排除 DOM 交互工具和其他不在白名单中的工具", () => {
       const config = SUB_AGENT_TYPES.researcher;
       const excluded = getExcludeToolsForType(config, allTools);
 
-      // researcher 不包含 tab 工具、execute_script、ask_user、agent
-      expect(excluded).toContain("get_tab_content");
-      expect(excluded).toContain("list_tabs");
-      expect(excluded).toContain("open_tab");
-      expect(excluded).toContain("close_tab");
+      // researcher 不包含 DOM 交互工具（execute_script、activate_tab）、ask_user、agent
       expect(excluded).toContain("activate_tab");
       expect(excluded).toContain("execute_script");
       expect(excluded).toContain("ask_user");
       expect(excluded).toContain("agent");
 
+      // researcher 可以读取页面（get_tab_content + tab 管理）
+      expect(excluded).not.toContain("get_tab_content");
+      expect(excluded).not.toContain("open_tab");
+      expect(excluded).not.toContain("list_tabs");
+      expect(excluded).not.toContain("close_tab");
+      expect(excluded).not.toContain("navigate_tab");
+
       // task 工具始终可用（ALWAYS_ALLOWED_TOOLS）
       expect(excluded).not.toContain("create_task");
       expect(excluded).not.toContain("update_task");
@@ -80,10 +82,11 @@ describe("Sub-Agent 类型系统", () => {
       expect(excluded).toContain("ask_user");
       expect(excluded).toContain("agent");
 
-      // 应该保留 tab 工具
+      // 应该保留 tab 工具（含 navigate_tab）
       expect(excluded).not.toContain("get_tab_content");
       expect(excluded).not.toContain("list_tabs");
       expect(excluded).not.toContain("open_tab");
+      expect(excluded).not.toContain("navigate_tab");
       expect(excluded).not.toContain("execute_script");
       expect(excluded).not.toContain("web_fetch");
 

src/app/service/agent/core/sub_agent_types.ts

@@ -11,26 +11,40 @@ export interface SubAgentTypeConfig {
 }
 
 // 所有子代理类型都默认可用的工具（task 工具用于与主 agent 共享任务进度）
-const ALWAYS_ALLOWED_TOOLS = ["create_task", "update_task", "get_task", "list_tasks", "delete_task"];
+const ALWAYS_ALLOWED_TOOLS = ["create_task", "update_task", "list_tasks"];
 
 // 内置子代理类型
 export const SUB_AGENT_TYPES: Record<string, SubAgentTypeConfig> = {
   researcher: {
     name: "researcher",
-    description: "Web search/fetch, data analysis, no tab interaction",
-    allowedTools: ["web_fetch", "web_search", "opfs_read", "opfs_write", "opfs_list", "opfs_delete"],
+    description: "Web search/fetch, page reading (read-only, no DOM interaction)",
+    allowedTools: [
+      "web_fetch",
+      "web_search",
+      "get_tab_content",
+      "open_tab",
+      "list_tabs",
+      "close_tab",
+      "navigate_tab",
+      "opfs_read",
+      "opfs_write",
+      "opfs_list",
+      "opfs_delete",
+    ],
     maxIterations: 20,
     timeoutMs: 600_000,
     systemPromptAddition: `## Role: Researcher
 
 You are a research-focused sub-agent. Your job is to search, fetch, read, and summarize information.
 
-**Capabilities:** Web search, URL fetching, OPFS file storage.
-**Limitations:** You cannot interact with browser tabs (no navigation, clicking, or form filling). You cannot ask the user questions.
+**Capabilities:** Web search, URL fetching, page reading (open tabs and read rendered content with get_tab_content), OPFS file storage.
+**Limitations:** You cannot interact with page DOM (no clicking, form filling, or script execution). You cannot ask the user questions.
 
 **Guidelines:**
-- Use web_search to find relevant sources, then web_fetch to read them.
+- Use web_search to find relevant sources, then web_fetch or get_tab_content to read them.
+- For JavaScript-rendered pages (SPAs), prefer get_tab_content over web_fetch — it reads the rendered DOM.
 - Synthesize information from multiple sources when possible.
+- Close tabs you no longer need to avoid clutter.
 - Return structured, concise results that the parent agent can act on.
 - If you cannot find the information, say so clearly rather than guessing.`,
   },
@@ -44,6 +58,7 @@ You are a research-focused sub-agent. Your job is to search, fetch, read, and su
       "open_tab",
       "close_tab",
       "activate_tab",
+      "navigate_tab",
       "execute_script",
       "web_fetch",
       "opfs_read",

src/app/service/agent/core/system_prompt.test.ts

@@ -109,14 +109,16 @@ describe("buildSubAgentSystemPrompt", () => {
     expect(result).not.toContain("## Sub-Agent");
   });
 
-  it.concurrent("researcher 类型不包含 tab 工具的描述", () => {
+  it.concurrent("researcher 类型包含页面读取工具但不包含页面交互工作流", () => {
     const config = SUB_AGENT_TYPES.researcher;
     const tools = config.allowedTools || [];
     const result = buildSubAgentSystemPrompt(config, tools);
 
-    expect(result).not.toContain("get_tab_content");
+    expect(result).toContain("get_tab_content");
     expect(result).toContain("web_fetch");
     expect(result).toContain("web_search");
+    // researcher 没有 execute_script，不应包含页面交互工作流段
+    expect(result).not.toContain("### Page Interaction Workflow");
   });
 
   it.concurrent("researcher 类型包含角色说明", () => {

src/app/service/agent/core/system_prompt.ts

@@ -95,7 +95,7 @@ Any task that involves 2+ tool calls (web searching, page reading, page interact
 
 ### Sub-Agent Types
 
-- **researcher** — Web search/fetch, data analysis. No tab interaction. Use for: information gathering, comparison research, content summarization, finding URLs/data.
+- **researcher** — Web search/fetch, page reading (read-only, no DOM interaction). Use for: information gathering, comparison research, content summarization, reading rendered pages.
 - **page_operator** — Browser tab interaction, page automation. Use for: navigating pages, filling forms, extracting page data, clicking buttons, writing content into editors.
 - **general** (default) — All tools. Use when the task spans both research and page interaction.
 
@@ -164,7 +164,7 @@ Use task tools **only** when tracking progress genuinely helps the user understa
 **Workflow:**
 1. **Plan** — Call \`list_tasks\` to check for existing tasks, then \`create_task\` for each step with a clear imperative subject and enough description for context.
 2. **Execute** — Before starting each task, call \`update_task\` with \`status: "in_progress"\`. When done, set \`status: "completed"\`.
-3. **Adapt** — If a completed task reveals follow-up work, create new tasks. If a task becomes irrelevant, use \`delete_task\` to clean up.
+3. **Adapt** — If a completed task reveals follow-up work, create new tasks.
 
 **Important:** Do not create tasks just to log what you already did or are about to do in the same response.`;
 
@@ -364,7 +364,8 @@ export function buildSystemPrompt(options: BuildSystemPromptOptions): string {
 export function buildSubAgentSystemPrompt(typeConfig: SubAgentTypeConfig, availableToolNames: string[]): string {
   const nameSet = new Set(availableToolNames);
   const hasOpfs = nameSet.has("opfs_read") || nameSet.has("opfs_write");
-  const hasTabTools = nameSet.has("get_tab_content");
+  // 页面交互指南需要同时具备 get_tab_content 和 execute_script
+  const hasPageInteraction = nameSet.has("get_tab_content") && nameSet.has("execute_script");
 
   const sections: string[] = [
     SUB_AGENT_SECTION_INTRO,
@@ -374,8 +375,8 @@ export function buildSubAgentSystemPrompt(typeConfig: SubAgentTypeConfig, availa
     SUB_AGENT_SECTION_TOOL_USAGE,
   ];
 
-  // 有 tab 工具时才包含页面交互验证指南
-  if (hasTabTools) {
+  // 同时拥有页面读取和 DOM 交互工具时才包含页面交互工作流指南
+  if (hasPageInteraction) {
     sections.push(SUB_AGENT_SECTION_PAGE_INTERACTION);
   }
 

src/app/service/agent/core/tool_call_guard.ts

@@ -31,7 +31,7 @@ function isNullResult(result: string): boolean {
 }
 
 // 不参与通用重复计数的查询类工具
-const QUERY_TOOLS = new Set(["list_tasks", "get_task", "list_tabs"]);
+const QUERY_TOOLS = new Set(["list_tasks", "list_tabs"]);
 
 /**
  * 检测：完全相同的 tool + args 被调用2次

src/app/service/agent/core/tools/sub_agent.ts

@@ -41,7 +41,7 @@ export const SUB_AGENT_DEFINITION: ToolDefinition = {
         type: "string",
         enum: SUB_AGENT_TYPE_NAMES,
         description:
-          "Sub-agent type. 'researcher' (web search/fetch, data analysis, no tab interaction), 'page_operator' (browser tab interaction, page automation), 'general' (all tools, default). Choose the most specific type for better results.",
+          "Sub-agent type. 'researcher' (web search/fetch, page reading — read-only, no DOM interaction), 'page_operator' (browser tab interaction, DOM manipulation, page automation), 'general' (all tools, default). Choose the most specific type for better results.",
       },
       to: {
         type: "string",

src/app/service/agent/core/tools/tab_tools.test.ts

@@ -11,6 +11,9 @@ const mockTabsUpdate = vi.fn();
 const mockTabsGet = vi.fn();
 // mock chrome.windows
 const mockWindowsUpdate = vi.fn();
+// mock chrome.tabs.onUpdated
+const mockOnUpdatedAddListener = vi.fn();
+const mockOnUpdatedRemoveListener = vi.fn();
 
 // mock offscreen extractHtmlWithSelectors 的返回值
 let mockExtractReturn: string | null = "Extracted content with selectors for testing";
@@ -42,6 +45,10 @@ beforeEach(() => {
     remove: mockTabsRemove,
     update: mockTabsUpdate,
     get: mockTabsGet,
+    onUpdated: {
+      addListener: mockOnUpdatedAddListener,
+      removeListener: mockOnUpdatedRemoveListener,
+    },
   };
   (chrome as any).windows = { update: mockWindowsUpdate };
 });
@@ -58,15 +65,16 @@ function getExecutor(name: string) {
 }
 
 describe("createTabTools", () => {
-  it("should create 5 tools", () => {
+  it("should create 6 tools", () => {
     const { tools } = makeTools();
-    expect(tools).toHaveLength(5);
+    expect(tools).toHaveLength(6);
     const names = tools.map((t) => t.definition.name);
     expect(names).toContain("get_tab_content");
     expect(names).toContain("list_tabs");
     expect(names).toContain("open_tab");
     expect(names).toContain("close_tab");
     expect(names).toContain("activate_tab");
+    expect(names).toContain("navigate_tab");
   });
 });
 
@@ -436,3 +444,82 @@ describe("activate_tab", () => {
     expect(mockWindowsUpdate).not.toHaveBeenCalled();
   });
 });
+
+describe("navigate_tab", () => {
+  it("should throw when tab_id is missing", async () => {
+    const executor = getExecutor("navigate_tab");
+    await expect(executor.execute({ url: "https://example.com" })).rejects.toThrow("tab_id is required");
+  });
+
+  it("should throw when url is missing", async () => {
+    const executor = getExecutor("navigate_tab");
+    await expect(executor.execute({ tab_id: 42 })).rejects.toThrow("url is required");
+  });
+
+  it("should navigate and wait for load by default", async () => {
+    mockTabsUpdate.mockResolvedValue({ id: 42 });
+    mockOnUpdatedAddListener.mockImplementation((listener: Function) => {
+      listener(42, { status: "complete" });
+    });
+    mockTabsGet.mockResolvedValue({
+      id: 42,
+      url: "https://new-page.com",
+      title: "New Page",
+      status: "complete",
+    });
+
+    const executor = getExecutor("navigate_tab");
+    const raw = (await executor.execute({ tab_id: 42, url: "https://new-page.com" })) as string;
+    const result = JSON.parse(raw);
+
+    expect(result.id).toBe(42);
+    expect(result.url).toBe("https://new-page.com");
+    expect(result.title).toBe("New Page");
+    expect(result.status).toBe("complete");
+    expect(mockTabsUpdate).toHaveBeenCalledWith(42, { url: "https://new-page.com" });
+    expect(mockOnUpdatedAddListener).toHaveBeenCalled();
+    expect(mockOnUpdatedRemoveListener).toHaveBeenCalled();
+  });
+
+  it("should skip waiting when wait_until_loaded is false", async () => {
+    mockTabsUpdate.mockResolvedValue({ id: 42 });
+    mockTabsGet.mockResolvedValue({
+      id: 42,
+      url: "https://new-page.com",
+      title: "",
+      status: "loading",
+    });
+
+    const executor = getExecutor("navigate_tab");
+    const raw = (await executor.execute({
+      tab_id: 42,
+      url: "https://new-page.com",
+      wait_until_loaded: false,
+    })) as string;
+    const result = JSON.parse(raw);
+
+    expect(result.status).toBe("loading");
+    expect(mockOnUpdatedAddListener).not.toHaveBeenCalled();
+  });
+
+  it("should ignore updates from other tabs", async () => {
+    mockTabsUpdate.mockResolvedValue({ id: 42 });
+    mockOnUpdatedAddListener.mockImplementation((listener: Function) => {
+      listener(99, { status: "complete" }); // 其他 tab
+      listener(42, { status: "loading" }); // 目标 tab 还在加载
+      listener(42, { status: "complete" }); // 目标 tab 加载完成
+    });
+    mockTabsGet.mockResolvedValue({
+      id: 42,
+      url: "https://new-page.com",
+      title: "Done",
+      status: "complete",
+    });
+
+    const executor = getExecutor("navigate_tab");
+    const raw = (await executor.execute({ tab_id: 42, url: "https://new-page.com" })) as string;
+    const result = JSON.parse(raw);
+
+    expect(result.title).toBe("Done");
+  });
+});

src/app/service/agent/core/tools/tab_tools.ts

@@ -85,6 +85,23 @@ const ACTIVATE_TAB_DEFINITION: ToolDefinition = {
   },
 };
 
+const NAVIGATE_TAB_DEFINITION: ToolDefinition = {
+  name: "navigate_tab",
+  description: "Navigate an existing tab to a new URL. Waits for page load completion by default.",
+  parameters: {
+    type: "object",
+    properties: {
+      tab_id: { type: "number", description: "Target tab ID" },
+      url: { type: "string", description: "URL to navigate to" },
+      wait_until_loaded: {
+        type: "boolean",
+        description: "Wait for page to finish loading (default: true). Set false to return immediately.",
+      },
+    },
+    required: ["tab_id", "url"],
+  },
+};
+
 // ---- Factory ----
 
 export function createTabTools(deps: {
@@ -288,13 +305,53 @@ export function createTabTools(deps: {
     },
   };
 
+  const navigateTabExecutor: ToolExecutor = {
+    execute: async (args: Record<string, unknown>) => {
+      const tabId = args.tab_id as number;
+      const url = args.url as string;
+      const waitUntilLoaded = (args.wait_until_loaded as boolean | undefined) ?? true;
+
+      if (tabId == null) throw new Error("tab_id is required");
+      if (!url) throw new Error("url is required");
+
+      await chrome.tabs.update(tabId, { url });
+
+      if (waitUntilLoaded) {
+        await new Promise<void>((resolve) => {
+          let timeoutId: ReturnType<typeof setTimeout>;
+          const listener = (updatedTabId: number, changeInfo: { status?: string }) => {
+            if (updatedTabId === tabId && changeInfo.status === "complete") {
+              chrome.tabs.onUpdated.removeListener(listener);
+              clearTimeout(timeoutId);
+              resolve();
+            }
+          };
+          chrome.tabs.onUpdated.addListener(listener);
+          timeoutId = setTimeout(() => {
+            chrome.tabs.onUpdated.removeListener(listener);
+            resolve();
+          }, 30_000);
+        });
+      }
+
+      const tab = await chrome.tabs.get(tabId);
+      return JSON.stringify({
+        id: tab.id,
+        url: tab.url || tab.pendingUrl || url,
+        title: tab.title || "",
+        status: tab.status || "unknown",
+      });
+    },
+  };
+
   return {
     tools: [
       { definition: GET_TAB_CONTENT_DEFINITION, executor: getTabContentExecutor },
       { definition: LIST_TABS_DEFINITION, executor: listTabsExecutor },
       { definition: OPEN_TAB_DEFINITION, executor: openTabExecutor },
       { definition: CLOSE_TAB_DEFINITION, executor: closeTabExecutor },
       { definition: ACTIVATE_TAB_DEFINITION, executor: activateTabExecutor },
+      { definition: NAVIGATE_TAB_DEFINITION, executor: navigateTabExecutor },
     ],
   };
 }