fix: network monitor requestId parsing, largeResultToFile output, bump 0.3.4

- Parse requestId from top level of Network.requestWillBeSent event
  (not inside request object) — fixes requests overwriting each other
- Return nil typed output when saving to file via largeResultToFile,
  preventing the MCP SDK from also serializing the full data
- Bump version to 0.3.4

Author: nnemirovsky

.claude-plugin/marketplace.json

@@ -1,6 +1,6 @@
 {
   "name": "iwdp-mcp",
-  "version": "0.3.3",
+  "version": "0.3.4",
   "description": "iOS Safari debugging via ios-webkit-debug-proxy — MCP server with full WebKit Inspector Protocol support",
   "owner": {
     "name": "nnemirovsky"

.claude-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "iwdp-mcp",
-  "version": "0.3.3",
+  "version": "0.3.4",
   "description": "iOS Safari debugging via ios-webkit-debug-proxy — MCP server with full WebKit Inspector Protocol support",
   "mcpServers": {
     "iwdp-mcp": {

cmd/iwdp-mcp/main.go

@@ -58,7 +58,7 @@ func lookupInterceptStage(requestID string) string {
 func main() {
 	server := mcp.NewServer(&mcp.Implementation{
 		Name:    "iwdp-mcp",
-		Version: "0.3.3",
+		Version: "0.3.4",
 	}, nil)
 
 	registerTools(server)
@@ -668,7 +668,7 @@ func registerTools(server *mcp.Server) {
 		}
 		out := EvaluateScriptOutput{Result: result.Result, Type: result.Result.Type}
 		if fileResult, err := largeResultToFile(out, "eval"); err == nil && fileResult != nil {
-			return fileResult, out, nil
+			return fileResult, nil, nil
 		}
 		return nil, out, nil
 	})
@@ -686,7 +686,7 @@ func registerTools(server *mcp.Server) {
 		}
 		out := RawOutput{Result: result.Result}
 		if fileResult, err := largeResultToFile(out, "call-fn"); err == nil && fileResult != nil {
-			return fileResult, out, nil
+			return fileResult, nil, nil
 		}
 		return nil, out, nil
 	})
@@ -704,7 +704,7 @@ func registerTools(server *mcp.Server) {
 		}
 		out := RawOutput{Result: props}
 		if fileResult, err := largeResultToFile(out, "props"); err == nil && fileResult != nil {
-			return fileResult, out, nil
+			return fileResult, nil, nil
 		}
 		return nil, out, nil
 	})
@@ -723,7 +723,7 @@ func registerTools(server *mcp.Server) {
 		}
 		out := RawOutput{Result: doc}
 		if fileResult, err := largeResultToFile(out, "dom"); err == nil && fileResult != nil {
-			return fileResult, out, nil
+			return fileResult, nil, nil
 		}
 		return nil, out, nil
 	})
@@ -769,7 +769,7 @@ func registerTools(server *mcp.Server) {
 		}
 		out := GetOuterHTMLOutput{OuterHTML: html}
 		if fileResult, err := largeResultToFile(out, "html"); err == nil && fileResult != nil {
-			return fileResult, out, nil
+			return fileResult, nil, nil
 		}
 		return nil, out, nil
 	})
@@ -961,7 +961,7 @@ func registerTools(server *mcp.Server) {
 		}
 		out := RawOutput{Result: nm.GetRequests()}
 		if fileResult, err := largeResultToFile(out, "network"); err == nil && fileResult != nil {
-			return fileResult, out, nil
+			return fileResult, nil, nil
 		}
 		return nil, out, nil
 	})
@@ -979,7 +979,7 @@ func registerTools(server *mcp.Server) {
 		}
 		out := GetResponseBodyOutput{Body: body, Base64Encoded: b64}
 		if fileResult, err := largeResultToFile(out, "response"); err == nil && fileResult != nil {
-			return fileResult, out, nil
+			return fileResult, nil, nil
 		}
 		return nil, out, nil
 	})
@@ -1355,7 +1355,7 @@ func registerTools(server *mcp.Server) {
 		}
 		out := RawOutput{Result: cc.GetMessages()}
 		if fileResult, err := largeResultToFile(out, "console"); err == nil && fileResult != nil {
-			return fileResult, out, nil
+			return fileResult, nil, nil
 		}
 		return nil, out, nil
 	})
@@ -1636,7 +1636,7 @@ func registerTools(server *mcp.Server) {
 		}
 		out := RawOutput{Result: tc.GetEvents()}
 		if fileResult, err := largeResultToFile(out, "timeline"); err == nil && fileResult != nil {
-			return fileResult, out, nil
+			return fileResult, nil, nil
 		}
 		return nil, out, nil
 	})

internal/tools/network.go

@@ -83,17 +83,20 @@ func (m *NetworkMonitor) Start(ctx context.Context, client *webkit.Client) error
 
 	client.OnEvent("Network.requestWillBeSent", func(method string, params json.RawMessage) {
 		var evt struct {
-			Request webkit.NetworkRequest `json:"request"`
+			RequestID string                `json:"requestId"`
+			Request   webkit.NetworkRequest `json:"request"`
 		}
 		if err := json.Unmarshal(params, &evt); err != nil {
 			return
 		}
+		// requestId is at the top level of the event, not inside request.
+		evt.Request.RequestID = evt.RequestID
 		m.mu.Lock()
 		if len(m.requests) >= maxCollectorEntries {
 			m.mu.Unlock()
 			return
 		}
-		m.requests[evt.Request.RequestID] = &CapturedRequest{
+		m.requests[evt.RequestID] = &CapturedRequest{
 			Request: evt.Request,
 		}
 		m.mu.Unlock()

internal/tools/tools_test.go

@@ -303,8 +303,8 @@ func TestNetworkMonitor(t *testing.T) {
 
 	// Send a Network.requestWillBeSent event.
 	err := mock.SendEvent("Network.requestWillBeSent", map[string]interface{}{
+		"requestId": "req-1",
 		"request": map[string]interface{}{
-			"requestId": "req-1",
 			"url":       "https://example.com/api",
 			"method":    "GET",
 			"headers":   map[string]string{"Accept": "application/json"},