diff options
| author | amankumarsingh77 <amankumarsingh7702@gmail.com> | 2026-07-28 00:01:32 +0530 |
|---|---|---|
| committer | Aman Kumar <145339995+amankumarsingh77@users.noreply.github.com> | 2026-08-02 19:12:46 +0530 |
| commit | 5cfa7ccc8f4082db111bc0186c5fa3d44b767ad3 (patch) | |
| tree | 05720833665398725d960aad67496ff6159540eb /src/domains/navigate.ts | |
| parent | 753c2ad615fa9827e168be256e1a7c7384605855 (diff) | |
refactor(tools): declare concurrency in the tool definition
serialized: true was applied in registry.ts, away from the tools it
describes. Now a required concurrency field on each definition, with a
test pinning the serialized set so the classes cannot drift.
The lock check defaults to taking the mutex for any class that is not
explicitly "parallel", so a definition that bypasses defineBrowserTool at
runtime fails safe rather than racing on shared CDP state.
The test script gains --conditions=import: pi-coding-agent publishes no
"require" condition, so importing any tool definition from a CJS-loaded
test could not resolve it.
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Diffstat (limited to 'src/domains/navigate.ts')
| -rw-r--r-- | src/domains/navigate.ts | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/src/domains/navigate.ts b/src/domains/navigate.ts index 98a2c13..ba86751 100644 --- a/src/domains/navigate.ts +++ b/src/domains/navigate.ts @@ -28,6 +28,7 @@ export const navigateTool = defineBrowserTool({ "Note: Google and some sites with strict anti-bot detection may reject CDP navigation. Use browser_http_get for search results.", ], parameters: NavigateArgs, + concurrency: "serialized", renderCall: (a) => new Text(`🌐 Navigate to ${a.url}`, 0, 0), async handler(args, { client }): Promise<Result<ToolOk, ToolErr>> { const tabs = await client.listTabs(false); @@ -82,6 +83,7 @@ export const openUrlsTool = defineBrowserTool({ "Use browser_wait_for_load on a tab before extracting data from SPAs.", ], parameters: OpenUrlsArgs, + concurrency: "serialized", renderCall: (a) => new Text(`🌐 Opening ${a.urls.length} URL${a.urls.length !== 1 ? "s" : ""}…`, 0, 0), async handler(args, { client, onUpdate }): Promise<Result<ToolOk, ToolErr>> { const total = args.urls.length; |