From 9f7483e1d70313d8bb9f0ad72badfdba395d5266 Mon Sep 17 00:00:00 2001 From: Alex Rudenko Date: Thu, 30 Oct 2025 08:11:43 +0100 Subject: [PATCH 1/3] refactor: improve get_network_requests --- docs/tool-reference.md | 3 +-- src/tools/network.ts | 3 +-- 2 files changed, 2 insertions(+), 4 deletions(-) diff --git a/docs/tool-reference.md b/docs/tool-reference.md index 2fa445b2..0912269c 100644 --- a/docs/tool-reference.md +++ b/docs/tool-reference.md @@ -246,8 +246,7 @@ ### `get_network_request` -**Description:** Gets a network request by reqid. You can get all requests by calling [`list_network_requests`](#list_network_requests). -Get the request currently selected in the DevTools UI by ommitting reqid +**Description:** Gets a network request by an optional reqid, if omitted returns the request selected in DevTools UI. **Parameters:** diff --git a/src/tools/network.ts b/src/tools/network.ts index 1df94550..70bf8e74 100644 --- a/src/tools/network.ts +++ b/src/tools/network.ts @@ -84,8 +84,7 @@ export const listNetworkRequests = defineTool({ export const getNetworkRequest = defineTool({ name: 'get_network_request', - description: `Gets a network request by reqid. You can get all requests by calling ${listNetworkRequests.name}. -Get the request currently selected in the DevTools UI by ommitting reqid`, + description: `Gets a network request by an optional reqid, if omitted returns the request selected in DevTools UI.`, annotations: { category: ToolCategory.NETWORK, readOnlyHint: true, From b32f00d127420fe2e828256fc5f106db900b63de Mon Sep 17 00:00:00 2001 From: Alex Rudenko Date: Thu, 30 Oct 2025 11:35:32 +0100 Subject: [PATCH 2/3] Apply suggestion from @OrKoN --- src/tools/network.ts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/tools/network.ts b/src/tools/network.ts index 70bf8e74..e49a9cad 100644 --- a/src/tools/network.ts +++ b/src/tools/network.ts @@ -84,7 +84,7 @@ export const listNetworkRequests = defineTool({ export const getNetworkRequest = defineTool({ name: 'get_network_request', - description: `Gets a network request by an optional reqid, if omitted returns the request selected in DevTools UI.`, + description: `Gets a network request by an optional reqid, if omitted returns the currently selected request in the DevTools Network panel.`, annotations: { category: ToolCategory.NETWORK, readOnlyHint: true, From 2e3a594141fb57283812da77f308a93db736e231 Mon Sep 17 00:00:00 2001 From: Alex Rudenko Date: Thu, 30 Oct 2025 11:38:29 +0100 Subject: [PATCH 3/3] Refer to the specific panel --- docs/tool-reference.md | 4 ++-- src/formatters/networkFormatter.ts | 2 +- src/tools/network.ts | 4 ++-- tests/formatters/networkFormatter.test.ts | 2 +- 4 files changed, 6 insertions(+), 6 deletions(-) diff --git a/docs/tool-reference.md b/docs/tool-reference.md index 0912269c..b5106dcf 100644 --- a/docs/tool-reference.md +++ b/docs/tool-reference.md @@ -246,11 +246,11 @@ ### `get_network_request` -**Description:** Gets a network request by an optional reqid, if omitted returns the request selected in DevTools UI. +**Description:** Gets a network request by an optional reqid, if omitted returns the currently selected request in the DevTools Network panel. **Parameters:** -- **reqid** (number) _(optional)_: The reqid of the network request. If omitted, looks up the current request selected in DevTools UI. +- **reqid** (number) _(optional)_: The reqid of the network request. If omitted returns the currently selected request in the DevTools Network panel. --- diff --git a/src/formatters/networkFormatter.ts b/src/formatters/networkFormatter.ts index d3945130..74ba1184 100644 --- a/src/formatters/networkFormatter.ts +++ b/src/formatters/networkFormatter.ts @@ -16,7 +16,7 @@ export function getShortDescriptionForRequest( selectedInDevToolsUI = false, ): string { // TODO truncate the URL - return `reqid=${id} ${request.method()} ${request.url()} ${getStatusFromRequest(request)}${selectedInDevToolsUI ? ` [selected in DevTools UI]` : ''}`; + return `reqid=${id} ${request.method()} ${request.url()} ${getStatusFromRequest(request)}${selectedInDevToolsUI ? ` [selected in the DevTools Network panel]` : ''}`; } export function getStatusFromRequest(request: HTTPRequest): string { diff --git a/src/tools/network.ts b/src/tools/network.ts index e49a9cad..e2923c42 100644 --- a/src/tools/network.ts +++ b/src/tools/network.ts @@ -94,7 +94,7 @@ export const getNetworkRequest = defineTool({ .number() .optional() .describe( - 'The reqid of the network request. If omitted, looks up the current request selected in DevTools UI.', + 'The reqid of the network request. If omitted returns the currently selected request in the DevTools Network panel.', ), }, handler: async (request, response, context) => { @@ -106,7 +106,7 @@ export const getNetworkRequest = defineTool({ response.attachNetworkRequest(data?.requestId); } else { response.appendResponseLine( - `Nothing is currently selected in DevTools UI.`, + `Nothing is currently selected in the DevTools Network panel.`, ); } } diff --git a/tests/formatters/networkFormatter.test.ts b/tests/formatters/networkFormatter.test.ts index 2bd1bbd6..b6edd56a 100644 --- a/tests/formatters/networkFormatter.test.ts +++ b/tests/formatters/networkFormatter.test.ts @@ -78,7 +78,7 @@ describe('networkFormatter', () => { assert.equal( result, - 'reqid=1 GET http://example.com [pending] [selected in DevTools UI]', + 'reqid=1 GET http://example.com [pending] [selected in the DevTools Network panel]', ); }); });