fix: improve aiosandbox-backend reliability and docs

- Fix incorrect directory paths in README examples
- Fix relative link to deep-aiosandbox example
- Add URL validation for BaseURL (scheme/host required, http/https only)
- Use path package instead of filepath for cross-platform consistency
- Refactor grep to use ripgrep JSON output for reliable parsing
- Move sandbox-sdk-go to direct dependency

Author: Oanakiaja

adk/middleware/aiosandbox-backend/README.md

@@ -93,7 +93,7 @@ agent, err := deep.New(ctx, &deep.Config{
 ## Run the Example
 
 ```bash
-cd adk/middleware/aiosandbox-filesystem
+cd adk/middleware/aiosandbox-backend
 go run .
 ```
 
@@ -126,7 +126,7 @@ This example uses the **Middleware approach** with `filesystem.Backend`:
 - Includes built-in large result offloading
 - Clean interface-based design
 
-The [deep-aiosandbox](../multiagent/deep-aiosandbox) example uses the **Tool approach** with `commandline.Operator`:
+The [deep-aiosandbox](../../multiagent/deep-aiosandbox) example uses the **Tool approach** with `commandline.Operator`:
 - Manually registers tools: `bash`, `read_file`, `edit_file`, `tree`
 - More flexible for custom tool combinations
 - Requires more setup code

adk/middleware/aiosandbox-backend/README_zh.md

@@ -93,7 +93,7 @@ agent, err := deep.New(ctx, &deep.Config{
 ## 运行示例
 
 ```bash
-cd adk/middleware/aiosandbox-filesystem
+cd adk/middleware/aiosandbox-backend
 go run .
 ```
 
@@ -128,7 +128,7 @@ type ShellBackend interface {
 - 内置大结果分流处理
 - 简洁的基于接口的设计
 
-[deep-aiosandbox](../multiagent/deep-aiosandbox) 示例使用 **工具方式** 配合 `commandline.Operator`：
+[deep-aiosandbox](../../multiagent/deep-aiosandbox) 示例使用 **工具方式** 配合 `commandline.Operator`：
 - 手动注册工具：`bash`、`read_file`、`edit_file`、`tree`
 - 对自定义工具组合更灵活
 - 需要更多配置代码

adk/middleware/aiosandbox-backend/backend.go

@@ -18,10 +18,11 @@ package main
 
 import (
 	"context"
+	"encoding/json"
 	"fmt"
 	"net/http"
 	"net/url"
-	"path/filepath"
+	"path"
 	"strings"
 
 	sandboxsdk "github.com/agent-infra/sandbox-sdk-go"
@@ -71,6 +72,14 @@ func NewAIOSandboxBackend(ctx context.Context, config *AIOSandboxBackendConfig)
 		return nil, fmt.Errorf("invalid BaseURL: %w", err)
 	}
 
+	if parsedURL.Scheme == "" || parsedURL.Host == "" {
+		return nil, fmt.Errorf("invalid BaseURL: scheme and host are required")
+	}
+
+	if parsedURL.Scheme != "http" && parsedURL.Scheme != "https" {
+		return nil, fmt.Errorf("invalid BaseURL: only http and https schemes are supported")
+	}
+
 	baseURL := fmt.Sprintf("%s://%s%s", parsedURL.Scheme, parsedURL.Host, parsedURL.Path)
 
 	opts := []option.RequestOption{
@@ -160,8 +169,22 @@ func escapeShellArg(s string) string {
 	return strings.ReplaceAll(s, "'", "'\\''")
 }
 
+// rgJSONMatch represents the ripgrep JSON output for a match.
+type rgJSONMatch struct {
+	Type string `json:"type"`
+	Data struct {
+		Path struct {
+			Text string `json:"text"`
+		} `json:"path"`
+		Lines struct {
+			Text string `json:"text"`
+		} `json:"lines"`
+		LineNumber int `json:"line_number"`
+	} `json:"data"`
+}
+
 // GrepRaw searches for content matching the specified pattern in files.
-// Uses ripgrep (rg) for better performance.
+// Uses ripgrep (rg) with JSON output for reliable parsing.
 func (b *AIOSandboxBackend) GrepRaw(ctx context.Context, req *filesystem.GrepRequest) ([]filesystem.GrepMatch, error) {
 	if req.Pattern == "" {
 		return nil, nil
@@ -172,22 +195,19 @@ func (b *AIOSandboxBackend) GrepRaw(ctx context.Context, req *filesystem.GrepReq
 		searchPath = b.config.WorkDir
 	}
 
-	// Build rg command
-	// -n: show line numbers
+	// Build rg command with JSON output for reliable parsing
 	// -F: treat pattern as literal string
-	// --no-heading: show file path on each line
-	// --null: use \0 as separator to handle : in filenames
-	// Output format: filename\0linenum\0content
+	// --json: output in JSON format
 	pattern := escapeShellArg(req.Pattern)
 	searchPath = escapeShellArg(searchPath)
 
 	var cmd string
 	if req.Glob != "" {
 		glob := escapeShellArg(req.Glob)
-		cmd = fmt.Sprintf("rg -n -F --no-heading --null -g '%s' '%s' '%s' 2>/dev/null || true",
+		cmd = fmt.Sprintf("rg -F --json -g '%s' '%s' '%s' 2>/dev/null || true",
 			glob, pattern, searchPath)
 	} else {
-		cmd = fmt.Sprintf("rg -n -F --no-heading --null '%s' '%s' 2>/dev/null || true",
+		cmd = fmt.Sprintf("rg -F --json '%s' '%s' 2>/dev/null || true",
 			pattern, searchPath)
 	}
 
@@ -204,23 +224,24 @@ func (b *AIOSandboxBackend) GrepRaw(ctx context.Context, req *filesystem.GrepReq
 		return nil, nil
 	}
 
-	// Parse rg --null output: filename\0linenum\0content
+	// Parse rg --json output: each line is a JSON object
 	var matches []filesystem.GrepMatch
 	lines := strings.Split(*data.Output, "\n")
 	for _, line := range lines {
 		if line == "" {
 			continue
 		}
-		parts := strings.SplitN(line, "\x00", 3)
-		if len(parts) < 3 {
+		var m rgJSONMatch
+		if err := json.Unmarshal([]byte(line), &m); err != nil {
+			continue
+		}
+		if m.Type != "match" {
 			continue
 		}
-		lineNum := 0
-		fmt.Sscanf(parts[1], "%d", &lineNum)
 		matches = append(matches, filesystem.GrepMatch{
-			Path:    parts[0],
-			Line:    lineNum,
-			Content: parts[2],
+			Path:    m.Data.Path.Text,
+			Line:    m.Data.LineNumber,
+			Content: strings.TrimSuffix(m.Data.Lines.Text, "\n"),
 		})
 	}
 
@@ -319,11 +340,11 @@ func (b *AIOSandboxBackend) Edit(ctx context.Context, req *filesystem.EditReques
 	return nil
 }
 
-func (b *AIOSandboxBackend) resolvePath(path string) string {
-	if filepath.IsAbs(path) {
-		return path
+func (b *AIOSandboxBackend) resolvePath(p string) string {
+	if path.IsAbs(p) {
+		return p
 	}
-	return filepath.Join(b.config.WorkDir, path)
+	return path.Join(b.config.WorkDir, p)
 }
 
 // Execute runs a shell command in the sandbox.

go.mod

@@ -5,6 +5,7 @@ go 1.24.7
 toolchain go1.24.9
 
 require (
+	github.com/agent-infra/sandbox-sdk-go v0.0.3
 	github.com/alicebob/miniredis/v2 v2.35.0
 	github.com/bytedance/sonic v1.14.2
 	github.com/chromedp/chromedp v0.9.5
@@ -39,7 +40,6 @@ require (
 
 require (
 	github.com/PuerkitoBio/goquery v1.10.3 // indirect
-	github.com/agent-infra/sandbox-sdk-go v0.0.3 // indirect
 	github.com/andybalholm/cascadia v1.3.3 // indirect
 	github.com/aymerick/douceur v0.2.0 // indirect
 	github.com/bahlo/generic-list-go v0.2.0 // indirect