Show the subagents in the clients

[onechat] / turn.go

diff --git a/turn.go b/turn.go
index 84f788337e80a7ea79b21461337b62ca68c24141..deb47d9150cdc4d2fdda5df299edad9df7a50248 100644 (file)
--- a/turn.go
+++ b/turn.go
@@ -3,6 +3,7 @@ package main
 import (
        "context"
        "errors"
 import (
        "context"
        "errors"

+       "fmt"

        "path/filepath"
        "sync"
        "time"
        "path/filepath"
        "sync"
        "time"


@@ -27,10 +28,24 @@ func newTurnLoop(agent *Agent, memory *Memory, subagents *Subagents, client Chai
                wake:   make(chan struct{}, 1),
        }
 
                wake:   make(chan struct{}, 1),
        }
 

+       l.runs = &SubagentRuns{
+               agent:     agent,
+               memory:    memory,
+               subagents: subagents,
+               layout:    layout,
+               client:    client,
+               state:     l.state,
+               report:    l.Report,
+               note:      l.note,
+               notify:    notify,
+               running:   make(map[string]*SubagentRun),
+       }
+

        l.tools = &Toolbox{
                log:       l,
                chat:      memory,
                subagents: subagents,
        l.tools = &Toolbox{
                log:       l,
                chat:      memory,
                subagents: subagents,

+               runs:      l.runs,

                workdir:   agent.config.Workdir,
        }
 
                workdir:   agent.config.Workdir,
        }
 


@@ -38,15 +53,18 @@ func newTurnLoop(agent *Agent, memory *Memory, subagents *Subagents, client Chai
 }
 
 // Event tells the clients what the agent does as it happens. Kind is the kind
 }
 
 // Event tells the clients what the agent does as it happens. Kind is the kind

-// of a logged message (user, reply, tool, echo), or one of ready, busy, idle,
-// and error. ID is the ID of a logged message. Source names the client that
-// sent a user message, such as terminal or telegram, so that each client can
-// show the messages of the others.
+// of a logged message (user, reply, tool, echo, note, work), or one of ready,
+// busy, idle, error, and subagent, which tells that the subagent Text starts a
+// run. ID is the ID of a logged message. Source names the client that sent a
+// user message, such as terminal or telegram, so that each client can show the
+// messages of the others, or the subagent of a report. Status tells how the run
+// of a report ended: finished, stopped, or failed.

 type Event struct {
        Kind   string `json:"kind"`
        ID     int    `json:"id,omitempty"`
        Text   string `json:"text,omitempty"`
        Source string `json:"source,omitempty"`
 type Event struct {
        Kind   string `json:"kind"`
        ID     int    `json:"id,omitempty"`
        Text   string `json:"text,omitempty"`
        Source string `json:"source,omitempty"`

+       Status string `json:"status,omitempty"`

 }
 
 // TurnLoop runs the turns of one agent. A user message goes to the log when it
 }
 
 // TurnLoop runs the turns of one agent. A user message goes to the log when it


@@ -57,12 +75,14 @@ type Event struct {
 // The view goes out as the messages of a ViewLayout, so the first request of a
 // turn reads the view of the turn before from the prompt cache. User messages
 // that come during a turn join the next request of its chain after the tool
 // The view goes out as the messages of a ViewLayout, so the first request of a
 // turn reads the view of the turn before from the prompt cache. User messages
 // that come during a turn join the next request of its chain after the tool

-// results.
+// results, and so do the reports of the subagents. The subagents run until the
+// loop ends.

 type TurnLoop struct {
        agent  *Agent
        memory *Memory
        client ChainClient
        tools  *Toolbox
 type TurnLoop struct {
        agent  *Agent
        memory *Memory
        client ChainClient
        tools  *Toolbox

+       runs   *SubagentRuns

        layout *ViewLayout
        clock  func() time.Time
        notify func(Event)
        layout *ViewLayout
        clock  func() time.Time
        notify func(Event)


@@ -71,6 +91,7 @@ type TurnLoop struct {
        mutex   sync.Mutex
        pending []Message
        cancel  context.CancelFunc
        mutex   sync.Mutex
        pending []Message
        cancel  context.CancelFunc

+       failure error

 }
 
 // Submit logs a user message from the client source and wakes the loop. The
 }
 
 // Submit logs a user message from the client source and wakes the loop. The


@@ -94,6 +115,44 @@ func (l *TurnLoop) Submit(source, text string) (Message, error) {
        return message, nil
 }
 
        return message, nil
 }
 

+// Report logs the report text of the subagent name as a work message and tells
+// the clients, with the status of the run. Unless the subagent was stopped, the
+// report joins the waiting messages and wakes the loop, as a user message of
+// Submit does. A failed write stops the loop, because the log refuses all
+// writes after it.
+func (l *TurnLoop) Report(name, text, status string) {
+       message, err := l.memory.Append("work", text)
+       if err == nil {
+               l.notify(Event{
+                       Kind:   "work",
+                       ID:     message.ID,
+                       Text:   text,
+                       Source: name,
+                       Status: status,
+               })
+       }
+
+       l.mutex.Lock()
+       switch {
+       case err != nil:
+               l.failure = fmt.Errorf("%w: %w", errChainLogWrite, err)
+       case status != subagentStopped:
+               l.pending = append(l.pending, message)
+       }
+       l.mutex.Unlock()
+
+       select {
+       case l.wake <- struct{}{}:
+       default:
+       }
+}
+
+// StopSubagent ends the run of the subagent name and tells if it ran. Its
+// report is logged, but it starts no turn.
+func (l *TurnLoop) StopSubagent(name string) (bool, error) {
+       return l.runs.Stop(name)
+}
+

 // Interrupt ends the running turn. A running tool is killed, and the turn sends
 // no further request.
 func (l *TurnLoop) Interrupt() bool {
 // Interrupt ends the running turn. A running tool is killed, and the turn sends
 // no further request.
 func (l *TurnLoop) Interrupt() bool {


@@ -128,9 +187,12 @@ func (l *TurnLoop) Flush(ctx context.Context) error {
        return l.memory.Flush(ctx)
 }
 
        return l.memory.Flush(ctx)
 }
 

-// Run runs a turn each time user messages wait, until ctx ends. It returns the
-// error of a failed log write, which stops the agent.
+// Run runs a turn each time user messages or reports wait, until ctx ends. It
+// returns the error of a failed log write, which stops the agent. At its end
+// it ends the runs of the subagents and waits for them.

 func (l *TurnLoop) Run(ctx context.Context) error {
 func (l *TurnLoop) Run(ctx context.Context) error {

+       defer l.runs.Close()
+

        for {
                for l.waiting() {
                        err := l.turn(ctx)
        for {
                for l.waiting() {
                        err := l.turn(ctx)


@@ -144,9 +206,21 @@ func (l *TurnLoop) Run(ctx context.Context) error {
                case <-ctx.Done():
                        return ctx.Err()
                }
                case <-ctx.Done():
                        return ctx.Err()
                }

+
+               err := l.failed()
+               if err != nil {
+                       return err
+               }

        }
 }
 
        }
 }
 

+// note logs a note and tells the clients. A note starts no turn.
+func (l *TurnLoop) note(text string) error {
+       _, err := l.record("note", "", text)
+
+       return err
+}
+

 // record logs a message and tells the clients, with the client source of a user
 // message.
 func (l *TurnLoop) record(kind, source, text string) (Message, error) {
 // record logs a message and tells the clients, with the client source of a user
 // message.
 func (l *TurnLoop) record(kind, source, text string) (Message, error) {


@@ -165,7 +239,7 @@ func (l *TurnLoop) record(kind, source, text string) (Message, error) {
        return message, nil
 }
 
        return message, nil
 }
 

-// waiting tells if user messages wait for a request.
+// waiting tells if user messages or reports wait for a request.

 func (l *TurnLoop) waiting() bool {
        l.mutex.Lock()
        defer l.mutex.Unlock()
 func (l *TurnLoop) waiting() bool {
        l.mutex.Lock()
        defer l.mutex.Unlock()


@@ -173,6 +247,14 @@ func (l *TurnLoop) waiting() bool {
        return len(l.pending) > 0
 }
 
        return len(l.pending) > 0
 }
 

+// failed returns the error of a failed write of a report.
+func (l *TurnLoop) failed() error {
+       l.mutex.Lock()
+       defer l.mutex.Unlock()
+
+       return l.failure
+}
+

 // turn runs one turn. A failed or interrupted model call ends the turn and
 // tells the clients. The messages of the turn stay in the log and the view, and
 // the next turn starts only with a new message, so a lasting failure such as a
 // turn runs one turn. A failed or interrupted model call ends the turn and
 // tells the clients. The messages of the turn stay in the log and the view, and
 // the next turn starts only with a new message, so a lasting failure such as a