feat: blades bot (#213)

* feat: blades bot

* fix bugs

* fix blades

* fix

* fix

* fix

* fix

* fix bug

* fix bug

* fix struct

* fix bug

* feat: add YAML parser and emitter implementation

- Introduced a new YAML parser and emitter in `yamlh.go` and `yamlprivateh.go` files, providing comprehensive support for YAML document processing.
- Implemented various data structures and constants for handling YAML tokens, events, nodes, and documents.
- Added utility functions for character checks and buffer management to facilitate parsing and emitting processes.
- Updated `modules.txt` to reflect new dependencies and versions, including updates to `github.com/go-kratos/blades` and `gopkg.in/yaml.v3`.

* fix: update middleware import paths to use recipe package

* fix

* fix

* fix: remove leftover merge conflict markers

* chore: align non-cmd files with origin/main

* refactor(app): update import paths and use unified middleware package

* fix bug

* fix bug

* fix

* fix

* fix

* add weixin

* fix weixin

* fix

* feat: enhance tool descriptions and improve context handling in runner

* feat: enhance error handling in agent tools and add context control option

---------

Co-authored-by: chenzhihui <zhihui_chen@foxmail.com>

Author: daemon365

agent.go

@@ -2,6 +2,7 @@ package blades
 
 import (
 	"context"
+	"errors"
 	"fmt"
 	"html/template"
 	"strings"
@@ -295,13 +296,18 @@ func (a *agent) handleTools(ctx context.Context, invocation *Invocation, part To
 		if tool.Name() == part.Name {
 			response, err := tool.Handle(ctx, part.Request)
 			if err != nil {
-				return part, err
+				if errors.Is(err, context.Canceled) || errors.Is(err, context.DeadlineExceeded) {
+					return part, err
+				}
+				part.Response = "Tool error: " + err.Error()
+				return part, nil
 			}
 			part.Response = response
 			return part, nil
 		}
 	}
-	return part, fmt.Errorf("agent: tool %s not found", part.Name)
+	part.Response = fmt.Sprintf("Tool error: agent: tool %s not found", part.Name)
+	return part, nil
 }
 
 // executeTools executes the tools specified in the tool parts.
@@ -438,6 +444,9 @@ func (a *agent) handle(ctx context.Context, session Session, invocation *Invocat
 					yield(nil, err)
 					return
 				}
+				if _, ok := toolMessage.Actions[tools.ActionLoopExit]; ok {
+					return
+				}
 				// In stateless mode, accumulate the tool result into the local slice
 				// so the next iteration can supply it to the model without loading
 				// session history. executeTools mutates finalMessage in-place and

agent_exit_tool_test.go

@@ -0,0 +1,55 @@
+package blades
+
+import (
+	"context"
+	"testing"
+
+	bladestools "github.com/go-kratos/blades/tools"
+)
+
+type exitToolLoopModel struct {
+	calls int
+}
+
+func (m *exitToolLoopModel) Name() string { return "exit-tool-loop" }
+
+func (m *exitToolLoopModel) Generate(_ context.Context, _ *ModelRequest) (*ModelResponse, error) {
+	m.calls++
+	msg := NewAssistantMessage(StatusCompleted)
+	msg.Role = RoleTool
+	msg.Parts = append(msg.Parts, NewToolPart("call_exit", "exit", `{"reason":"done"}`))
+	return &ModelResponse{Message: msg}, nil
+}
+
+func (m *exitToolLoopModel) NewStreaming(context.Context, *ModelRequest) Generator[*ModelResponse, error] {
+	return nil
+}
+
+func TestAgentStopsImmediatelyAfterExitTool(t *testing.T) {
+	t.Parallel()
+
+	model := &exitToolLoopModel{}
+	agent, err := NewAgent(
+		"review",
+		WithModel(model),
+		WithTools(bladestools.NewExitTool()),
+		WithMaxIterations(5),
+	)
+	if err != nil {
+		t.Fatalf("new agent: %v", err)
+	}
+
+	msg, err := NewRunner(agent).Run(context.Background(), UserMessage("review this"))
+	if err != nil {
+		t.Fatalf("runner run: %v", err)
+	}
+	if got, want := model.calls, 1; got != want {
+		t.Fatalf("model calls = %d, want %d", got, want)
+	}
+	if got, want := msg.Role, RoleTool; got != want {
+		t.Fatalf("message role = %q, want %q", got, want)
+	}
+	if _, ok := msg.Actions[bladestools.ActionLoopExit]; !ok {
+		t.Fatalf("expected %q action on final tool message", bladestools.ActionLoopExit)
+	}
+}

agent_session_context_test.go

@@ -2,6 +2,7 @@ package blades
 
 import (
 	"context"
+	"fmt"
 	"testing"
 
 	bladestools "github.com/go-kratos/blades/tools"
@@ -192,3 +193,70 @@ func TestAgentRunWithSessionAndNilPreparedContextKeepsInvocationMessagesAcrossTo
 		t.Fatalf("tool response = %q, want %q", got, want)
 	}
 }
+
+type recoverableToolErrorModel struct {
+	calls       int
+	secondInput []*Message
+}
+
+func (m *recoverableToolErrorModel) Name() string { return "recoverable-tool-error" }
+
+func (m *recoverableToolErrorModel) Generate(_ context.Context, req *ModelRequest) (*ModelResponse, error) {
+	m.calls++
+	if m.calls == 2 {
+		m.secondInput = append(m.secondInput[:0], req.Messages...)
+	}
+
+	msg := NewAssistantMessage(StatusCompleted)
+	if m.calls == 1 {
+		msg.Role = RoleTool
+		msg.Parts = append(msg.Parts, NewToolPart("call_1", "edit", `{"path":"IDENTITY.md"}`))
+		return &ModelResponse{Message: msg}, nil
+	}
+
+	msg.Parts = append(msg.Parts, TextPart{Text: "recovered"})
+	return &ModelResponse{Message: msg}, nil
+}
+
+func (m *recoverableToolErrorModel) NewStreaming(context.Context, *ModelRequest) Generator[*ModelResponse, error] {
+	return nil
+}
+
+func TestAgentRunKeepsGoingAfterRecoverableToolError(t *testing.T) {
+	t.Parallel()
+
+	model := &recoverableToolErrorModel{}
+	tool := bladestools.NewTool("edit", "edit", bladestools.HandleFunc(func(context.Context, string) (string, error) {
+		return "", fmt.Errorf("edit: edits[0] target not found")
+	}))
+	agent, err := NewAgent("tool-agent", WithModel(model), WithTools(tool))
+	if err != nil {
+		t.Fatalf("new agent: %v", err)
+	}
+
+	runner := NewRunner(agent)
+	session := NewSession()
+	output, err := runner.Run(context.Background(), UserMessage("set persona"), WithSession(session))
+	if err != nil {
+		t.Fatalf("runner run: %v", err)
+	}
+	if got, want := output.Text(), "recovered"; got != want {
+		t.Fatalf("output text = %q, want %q", got, want)
+	}
+	if got, want := model.calls, 2; got != want {
+		t.Fatalf("model calls = %d, want %d", got, want)
+	}
+	if got, want := len(model.secondInput), 2; got != want {
+		t.Fatalf("second input len = %d, want %d", got, want)
+	}
+	part, ok := model.secondInput[1].Parts[0].(ToolPart)
+	if !ok {
+		t.Fatalf("second input second part type = %T, want ToolPart", model.secondInput[1].Parts[0])
+	}
+	if got, want := part.Completed, true; got != want {
+		t.Fatalf("tool completed = %t, want %t", got, want)
+	}
+	if got, want := part.Response, "Tool error: edit: edits[0] target not found"; got != want {
+		t.Fatalf("tool response = %q, want %q", got, want)
+	}
+}

agent_tool_status_test.go

@@ -2,6 +2,8 @@ package blades
 
 import (
 	"context"
+	"errors"
+	"fmt"
 	"testing"
 
 	bladestools "github.com/go-kratos/blades/tools"
@@ -106,3 +108,48 @@ func TestAgentExecuteToolsSkipsCompletedToolPart(t *testing.T) {
 		t.Fatalf("tool response = %q, want %q", gotResp, want)
 	}
 }
+
+func TestAgentExecuteToolsCapturesToolErrorsInResponse(t *testing.T) {
+	t.Parallel()
+
+	tool := bladestools.NewTool("edit", "edit", bladestools.HandleFunc(func(context.Context, string) (string, error) {
+		return "", context.DeadlineExceeded
+	}))
+	invocation := &Invocation{Tools: []bladestools.Tool{tool}}
+	message := NewAssistantMessage(StatusCompleted)
+	message.Role = RoleTool
+	message.Parts = append(message.Parts, NewToolPart("call_1", "edit", `{"path":"IDENTITY.md"}`))
+
+	_, err := (&agent{}).executeTools(context.Background(), invocation, message)
+	if !errors.Is(err, context.DeadlineExceeded) {
+		t.Fatalf("executeTools error = %v, want deadline exceeded", err)
+	}
+}
+
+func TestAgentExecuteToolsTurnsRecoverableErrorsIntoCompletedToolResults(t *testing.T) {
+	t.Parallel()
+
+	tool := bladestools.NewTool("edit", "edit", bladestools.HandleFunc(func(context.Context, string) (string, error) {
+		return "", fmt.Errorf("edit: edits[0] target not found")
+	}))
+	invocation := &Invocation{Tools: []bladestools.Tool{tool}}
+	message := NewAssistantMessage(StatusCompleted)
+	message.Role = RoleTool
+	message.Parts = append(message.Parts, NewToolPart("call_1", "edit", `{"path":"IDENTITY.md"}`))
+
+	got, err := (&agent{}).executeTools(context.Background(), invocation, message)
+	if err != nil {
+		t.Fatalf("executeTools returned error: %v", err)
+	}
+
+	toolPart, ok := got.Parts[0].(ToolPart)
+	if !ok {
+		t.Fatalf("part type = %T, want ToolPart", got.Parts[0])
+	}
+	if got, want := toolPart.Completed, true; got != want {
+		t.Fatalf("tool completed = %t, want %t", got, want)
+	}
+	if got, want := toolPart.Response, "Tool error: edit: edits[0] target not found"; got != want {
+		t.Fatalf("tool response = %q, want %q", got, want)
+	}
+}