core/mcp
8
0
Fork 0
Code Issues 4 Pull requests 2 Projects Releases Packages Wiki Activity Actions

fix(mcp): AX-6 sweep on pkg/mcp/tools_webview.go

Browse source 
Annotated bytes (retained for bytes.NewReader image.Decode boundary).
Replaced bytes.Buffer with core.NewBuffer, sync.Mutex with core.Mutex.

Co-authored-by: Codex <noreply@openai.com>
This commit is contained in:
Snider 2026-04-25 10:29:02 +01:00
parent 9fd2185a86
commit 53bd7478a7
1 changed files with 4 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
pkg/mcp/tools_webview.go
View file

@ -3,13 +3,13 @@
package mcp
import (
// Note: AX-6 — screenshot normalization needs bytes.NewReader for image.Decode on captured byte slices.
"bytes"
"context"
"encoding/base64"
"image"
"image/jpeg"
_ "image/png"
"sync"
"time"
core "dappco.re/go/core"
@ -19,7 +19,7 @@ import (
)
// webviewMu protects webviewInstance from concurrent access.
var webviewMu sync.Mutex
var webviewMu core.Mutex
// webviewInstance holds the current webview connection.
// This is managed by the MCP service.
@ -597,8 +597,8 @@ func normalizeScreenshotData(data []byte, format string) ([]byte, string, error)
if err != nil {
return nil, "", err
}
var buf bytes.Buffer
if err := jpeg.Encode(&buf, img, &jpeg.Options{Quality: 90}); err != nil {
buf := core.NewBuffer()
if err := jpeg.Encode(buf, img, &jpeg.Options{Quality: 90}); err != nil {
return nil, "", err
}
return buf.Bytes(), "jpeg", nil