16 "github.com/stretchr/testify/require"
19 func TestNotificationWaitsForTheNextTurnAfterEveryActiveToolRequest(t *testing.T) {
20 server := newModelFake(t)
21 arrived := make(chan struct{})
22 release := make(chan struct{})
23 server.reply = func(w http.ResponseWriter, call ModelFakeCall) {
27 modelWriteEvents(w, modelCompletedEvent([]any{modelFunctionCallItem("fc_1", "call_1", "date", `{"id":0}`)}, 100, 0, 5))
32 modelWriteEvents(w, modelCompletedEvent([]any{modelMessageItem("")}, 100, 0, 5))
35 test := newDaemonMessageTest(t, t.TempDir(), server)
36 test.Start(t, 0, 1, 2)
38 _, err := test.loops[0].Submit(terminalSource, "Start a tool chain.")
39 require.NoError(t, err)
42 before := test.agents[0].memory.Len()
43 input := &NotifyInput{
46 Text: "Unique queued external data.",
49 receipt, err := test.messages.Notify(input)
50 require.NoError(t, err)
51 require.Equal(t, "accepted", receipt.Status)
52 require.Equal(t, before, test.agents[0].memory.Len())
53 require.True(t, test.loops[0].Busy())
55 test.events[0].WaitFor(t, "idle", 2)
56 require.Equal(t, 3, server.Calls())
57 require.NotContains(t, strings.Join(turnInputTexts(t, server.Call(1)), "\n"), input.Text)
58 require.Contains(t, strings.Join(turnInputTexts(t, server.Call(2)), "\n"), input.Text)
59 require.Equal(t, notifySource, test.events[0].LastEvent("work").Source)
60 require.Empty(t, test.Queue(t).Pending)
61 require.Len(t, test.Queue(t).Notifications, 1)
62 require.Zero(t, test.agents[1].memory.Len())
63 require.Zero(t, test.agents[2].memory.Len())
64 require.NotContains(t, test.logs.String(), input.Source)
65 require.NotContains(t, test.logs.String(), input.Text)
66 require.NotContains(t, test.logs.String(), input.ID)
69 func TestNotificationRetriesSurviveRestartBeforeAndAfterDelivery(t *testing.T) {
70 directory := t.TempDir()
71 server := newModelFake(t)
72 server.reply = func(w http.ResponseWriter, _ ModelFakeCall) {
73 modelWriteEvents(w, modelCompletedEvent([]any{modelMessageItem("")}, 100, 0, 5))
76 input := &NotifyInput{
79 Text: "A new result.",
82 first := newDaemonMessageTest(t, directory, server)
83 receipt, err := first.messages.Notify(input)
84 require.NoError(t, err)
85 require.Zero(t, first.agents[0].memory.Len())
88 second := newDaemonMessageTest(t, directory, server)
89 duplicate, err := second.messages.Notify(input)
90 require.NoError(t, err)
91 require.Equal(t, receipt.ID, duplicate.ID)
92 require.Equal(t, "duplicate", duplicate.Status)
93 require.Zero(t, second.agents[0].memory.Len())
95 second.events[0].WaitFor(t, "idle", 1)
96 require.Empty(t, second.Queue(t).Pending)
99 third := newDaemonMessageTest(t, directory, server)
100 duplicate, err = third.messages.Notify(input)
101 require.NoError(t, err)
102 require.Equal(t, "duplicate", duplicate.Status)
103 require.Equal(t, receipt.ID, duplicate.ID)
104 require.False(t, third.loops[0].waiting())
105 require.Equal(t, 1, server.Calls())
108 changed.Text = "A different result."
109 _, err = third.messages.Notify(&changed)
110 require.ErrorContains(t, err, "conflicts")
111 require.Len(t, third.Queue(t).Notifications, 1)
114 func TestNotificationAcceptanceIsSerializedForConcurrentRetries(t *testing.T) {
115 server := newModelFake(t)
116 test := newDaemonMessageTest(t, t.TempDir(), server)
117 input := &NotifyInput{
123 results := make(chan Event, 20)
124 failures := make(chan error, 20)
126 var workers sync.WaitGroup
134 receipt, err := test.messages.Notify(input)
146 for receipt := range results {
147 require.Equal(t, 1, receipt.ID)
149 if receipt.Status == "accepted" {
152 require.Equal(t, "duplicate", receipt.Status)
156 for err := range failures {
157 require.NoError(t, err)
160 require.Equal(t, 1, accepted)
161 require.Len(t, test.Queue(t).Pending, 1)
162 require.Len(t, test.Queue(t).Notifications, 1)
165 func TestNotificationStorageFailureReturnsNoAcceptedReceipt(t *testing.T) {
166 server := newModelFake(t)
167 test := newDaemonMessageTest(t, t.TempDir(), server)
168 test.messages.path = filepath.Join(t.TempDir(), "missing", "messages.json")
169 receipt, err := test.messages.Notify(&NotifyInput{
174 require.ErrorContains(t, err, "accept external event")
175 require.Empty(t, receipt.Kind)
176 require.False(t, test.loops[0].waiting())
177 require.Zero(t, test.agents[0].memory.Len())
179 _, err = test.messages.Notify(&NotifyInput{
182 Text: "Another result.",
184 require.ErrorContains(t, err, "queue stopped")
187 func TestNotificationFullStoreRejectsNewKeysWithoutRemovingReceipts(t *testing.T) {
188 server := newModelFake(t)
189 test := newDaemonMessageTest(t, t.TempDir(), server)
191 for number := range notifyReceipts {
192 test.messages.state.Notifications = append(test.messages.state.Notifications, NotifyReceipt{
194 ID: fmt.Sprint(number),
195 Digest: notifyDigest("data"),
200 test.messages.state.Next = notifyReceipts
201 receipt, err := test.messages.Notify(&NotifyInput{
206 require.ErrorContains(t, err, "store is full")
207 require.Empty(t, receipt.Kind)
208 require.Len(t, test.messages.state.Notifications, notifyReceipts)
209 require.Empty(t, test.messages.state.Pending)
211 receipt, err = test.messages.Notify(&NotifyInput{
216 require.NoError(t, err)
217 require.Equal(t, "duplicate", receipt.Status)
220 func TestNotificationBatchesKeepAcceptanceOrder(t *testing.T) {
221 server := newModelFake(t)
222 server.reply = func(w http.ResponseWriter, _ ModelFakeCall) {
223 modelWriteEvents(w, modelCompletedEvent([]any{modelMessageItem("")}, 100, 0, 5))
226 test := newDaemonMessageTest(t, t.TempDir(), server)
227 test.agents[0].memory.summarizer = newFakeSummarizer()
228 for number := range 21 {
229 _, err := test.messages.Notify(&NotifyInput{
231 ID: fmt.Sprint(number),
232 Text: fmt.Sprintf("result-%02d", number),
234 require.NoError(t, err)
238 test.events[0].WaitFor(t, "idle", 2)
239 require.Equal(t, 2, server.Calls())
241 first := turnInputTexts(t, server.Call(0))
242 require.Len(t, first, 22)
244 for number := range 20 {
245 require.Contains(t, first[number+2], fmt.Sprintf("result-%02d", number))
248 second := turnInputTexts(t, server.Call(1))
249 require.Contains(t, second[len(second)-1], "result-20")
250 require.Empty(t, test.Queue(t).Pending)
253 func TestNotificationRecoversItsPreparedLogIDWithoutAnotherAppend(t *testing.T) {
254 testCases := []struct {
259 name: "recovers a saved ID before the log append",
262 name: "reuses the log entry after the append",
267 for _, tc := range testCases {
268 t.Run(tc.name, func(t *testing.T) {
269 directory := t.TempDir()
270 server := newModelFake(t)
271 server.reply = func(w http.ResponseWriter, _ ModelFakeCall) {
272 modelWriteEvents(w, modelCompletedEvent([]any{modelMessageItem("")}, 100, 0, 5))
275 first := newDaemonMessageTest(t, directory, server)
276 _, err := first.messages.Notify(&NotifyInput{
281 require.NoError(t, err)
284 require.NoError(t, first.messages.Notifications(first.loops[0]))
287 first.messages.state.Pending[0].LogID = &id
288 require.NoError(t, first.messages.save())
293 second := newDaemonMessageTest(t, directory, server)
295 second.events[0].WaitFor(t, "idle", 1)
296 require.Equal(t, 1, server.Calls())
297 require.Equal(t, 1, second.agents[0].memory.Len())
298 require.Empty(t, second.Queue(t).Pending)
303 func TestPreparedNotificationKeepsItsLogIDBeforeStartupWriters(t *testing.T) {
304 testCases := []struct {
310 name: "keeps its reservation before a startup user message",
313 name: "keeps its reservation before a worker recovery note",
317 name: "keeps a child reservation before startup input",
321 name: "keeps a child reservation before a worker recovery note",
327 for _, tc := range testCases {
328 t.Run(tc.name, func(t *testing.T) {
334 directory := t.TempDir()
335 server := newModelFake(t)
336 server.reply = func(w http.ResponseWriter, _ ModelFakeCall) {
337 modelWriteEvents(w, modelCompletedEvent([]any{modelMessageItem("")}, 100, 0, 5))
340 first := newDaemonMessageTest(t, directory, server)
341 input := &NotifyInput{
342 AgentID: first.agents[index].agent.config.ID,
345 Text: "A result with a prepared log ID.",
348 receipt, err := first.messages.Notify(input)
349 require.NoError(t, err)
352 first.messages.state.Pending[0].LogID = &id
353 require.NoError(t, first.messages.save())
355 expected := first.Queue(t).Pending[0].Text
358 second := newDaemonMessageTest(t, directory, server)
359 require.False(t, second.loops[index].waiting())
360 require.Empty(t, second.events[index].Kinds())
363 path := filepath.Join(second.agents[index].agent.dir, subagentWorkFileName)
364 require.NoError(t, os.WriteFile(path, []byte(`{"running":["BlueFox"]}`), 0o600))
365 require.NoError(t, second.loops[index].runs.Resume(t.Context()))
367 note, err := second.agents[index].memory.Message(1)
368 require.NoError(t, err)
369 require.Equal(t, "note", note.Kind)
370 require.Contains(t, note.Text, "The daemon restarted, cutting BlueFox")
372 submitted, err := second.loops[index].Submit(terminalSource, "A user message during startup.")
373 require.NoError(t, err)
374 require.Equal(t, 1, submitted.ID)
377 restored, err := second.agents[index].memory.Message(id)
378 require.NoError(t, err)
379 require.Equal(t, "work", restored.Kind)
380 require.Equal(t, expected, restored.Text)
381 require.Len(t, second.Queue(t).Pending, 1)
383 second.Start(t, index)
384 second.events[index].WaitFor(t, "idle", 1)
385 require.Empty(t, second.Queue(t).Pending)
386 require.Equal(t, 1, server.Calls())
387 require.Equal(t, id, second.events[index].LastEvent("work").ID)
389 duplicate, err := second.messages.Notify(input)
390 require.NoError(t, err)
391 require.Equal(t, "duplicate", duplicate.Status)
392 require.Equal(t, receipt.ID, duplicate.ID)
393 require.Len(t, second.Queue(t).Notifications, 1)
398 func TestNotificationBatchMakesProgressWithEscapedControlCharacters(t *testing.T) {
399 server := newModelFake(t)
400 server.reply = func(w http.ResponseWriter, _ ModelFakeCall) {
401 modelWriteEvents(w, modelCompletedEvent([]any{modelMessageItem("")}, 100, 0, 5))
404 test := newDaemonMessageTest(t, t.TempDir(), server)
405 _, err := test.messages.Notify(&NotifyInput{
408 Text: "data" + strings.Repeat("\x00", notifyTextBytes-4),
410 require.NoError(t, err)
411 require.Greater(t, len(test.Queue(t).Pending[0].Text), 64<<10)
413 test.events[0].WaitFor(t, "idle", 1)
414 require.Empty(t, test.Queue(t).Pending)
417 func TestNotificationSelectorsBindOnePermanentIdentity(t *testing.T) {
418 server := newModelFake(t)
419 test := newDaemonMessageTest(t, t.TempDir(), server)
420 input := &NotifyInput{
421 AgentID: test.agents[1].agent.config.ID,
424 Text: "A result for one child.",
427 accepted, err := test.messages.Notify(input)
428 require.NoError(t, err)
430 queue := test.Queue(t)
431 require.Equal(t, test.agents[1].agent.config.ID, queue.Pending[0].TargetID)
432 require.Equal(t, queue.Pending[0].TargetID, queue.Notifications[0].TargetID)
433 require.Equal(t, "Garden", queue.Pending[0].Target)
434 require.Zero(t, test.agents[0].memory.Len())
435 require.Zero(t, test.agents[1].memory.Len())
436 require.Zero(t, test.agents[2].memory.Len())
438 input.AgentID = strings.ToUpper(test.agents[1].agent.config.ID)
439 duplicate, err := test.messages.Notify(input)
440 require.NoError(t, err)
441 require.Equal(t, accepted.ID, duplicate.ID)
442 require.Equal(t, "duplicate", duplicate.Status)
444 input.AgentID = test.agents[2].agent.config.ID
445 _, err = test.messages.Notify(input)
446 require.ErrorContains(t, err, "conflicts")
449 _, err = test.messages.Notify(input)
450 require.ErrorContains(t, err, "conflicts")
451 require.Equal(t, queue, test.Queue(t))
454 func TestNotificationRejectsUnknownWorkerAndMalformedIdentities(t *testing.T) {
455 server := newModelFake(t)
456 test := newDaemonMessageTest(t, t.TempDir(), server)
457 test.loops[0].runs.subagents.draw = func() string {
461 worker, err := test.loops[0].runs.subagents.Create(t.Context())
462 require.NoError(t, err)
464 testCases := []NotifyInput{
465 {AgentID: worker.name},
466 {AgentID: "Missing"},
468 {AgentID: "../Garden"},
469 {AgentID: strings.Repeat("f", 2*agentIDBytes)},
470 {AgentID: "invalid"},
473 for _, tc := range testCases {
476 tc.Text = "Unchanged data."
477 receipt, err := test.messages.Notify(&tc)
478 require.Error(t, err)
479 require.Empty(t, receipt.Kind)
482 require.Empty(t, test.messages.state.Pending)
483 require.Empty(t, test.messages.state.Notifications)
484 require.Zero(t, server.Calls())
487 func TestChildNotificationWaitsForTheWholeActiveChain(t *testing.T) {
488 server := newModelFake(t)
489 arrived := make(chan struct{})
490 release := make(chan struct{})
491 server.reply = func(w http.ResponseWriter, call ModelFakeCall) {
492 if call.number == 0 {
495 modelWriteEvents(w, modelCompletedEvent([]any{modelFunctionCallItem("fc_1", "call_1", "date", `{"id":0}`)}, 100, 0, 5))
500 modelWriteEvents(w, modelCompletedEvent([]any{modelMessageItem("")}, 100, 0, 5))
503 test := newDaemonMessageTest(t, t.TempDir(), server)
504 test.Start(t, 0, 1, 2)
505 require.NoError(t, test.messages.Tell("OneChat", "Garden", "Start the first chain."))
508 _, err := test.messages.Notify(&NotifyInput{
509 AgentID: test.agents[1].agent.config.ID,
512 Text: "Only the next child turn sees this.",
514 require.NoError(t, err)
515 require.Equal(t, 1, test.agents[1].memory.Len())
517 test.events[1].WaitFor(t, "idle", 2)
518 require.Equal(t, 3, server.Calls())
519 require.NotContains(t, strings.Join(turnInputTexts(t, server.Call(1)), "\n"), "Only the next child turn")
520 require.Contains(t, strings.Join(turnInputTexts(t, server.Call(2)), "\n"), "Only the next child turn")
521 require.Equal(t, notifySource, test.events[1].LastEvent("work").Source)
522 require.Zero(t, test.agents[0].memory.Len())
523 require.Zero(t, test.agents[2].memory.Len())
524 require.Empty(t, test.Queue(t).Pending)
527 func TestNotificationStableIdentitySurvivesOfflineNameChange(t *testing.T) {
528 directory := t.TempDir()
529 server := newModelFake(t)
530 server.reply = func(w http.ResponseWriter, _ ModelFakeCall) {
531 modelWriteEvents(w, modelCompletedEvent([]any{modelMessageItem("")}, 100, 0, 5))
534 first := newDaemonMessageTest(t, directory, server)
535 input := &NotifyInput{
536 AgentID: first.agents[1].agent.config.ID,
539 Text: "A saved child result.",
542 receipt, err := first.messages.Notify(input)
543 require.NoError(t, err)
546 first.messages.state.Pending[0].LogID = &id
547 require.NoError(t, first.messages.save())
549 envelope := first.Queue(t).Pending[0].Text
550 config := first.agents[1].agent.config
551 oldPath := first.agents[1].agent.dir
554 // Change only test fixtures to exercise recovery under the same
556 newPath := filepath.Join(filepath.Dir(oldPath), "Orchard")
557 require.NoError(t, os.Rename(oldPath, newPath))
559 config.Name = "Orchard"
560 agentWriteTestConfig(t, newPath, config)
562 second := newDaemonMessageTest(t, directory, server)
563 child := second.messages.Child("OneChat", "Orchard")
564 require.NotNil(t, child)
565 require.False(t, child.waiting())
567 restored, err := child.memory.Message(id)
568 require.NoError(t, err)
569 require.Equal(t, envelope, restored.Text)
571 duplicate, err := second.messages.Notify(input)
572 require.NoError(t, err)
573 require.Equal(t, receipt.ID, duplicate.ID)
574 require.Equal(t, "duplicate", duplicate.Status)
575 require.NoError(t, second.messages.Notifications(child))
576 require.NoError(t, child.turn(t.Context()))
577 require.Empty(t, second.Queue(t).Pending)
578 require.Zero(t, second.agents[0].memory.Len())
582 third := newDaemonMessageTest(t, directory, server)
583 duplicate, err = third.messages.Notify(input)
584 require.NoError(t, err)
585 require.Equal(t, "duplicate", duplicate.Status)
586 require.Empty(t, third.Queue(t).Pending)
587 require.Equal(t, 1, server.Calls())
590 func TestMissingNotificationIdentityStopsStartupWithoutFallback(t *testing.T) {
591 directory := t.TempDir()
592 server := newModelFake(t)
593 first := newDaemonMessageTest(t, directory, server)
594 _, err := first.messages.Notify(&NotifyInput{
595 AgentID: first.agents[1].agent.config.ID,
598 Text: "Do not redirect this result.",
600 require.NoError(t, err)
602 path := first.messages.path
603 before, err := os.ReadFile(path)
604 require.NoError(t, err)
606 childPath := first.agents[1].agent.dir
609 require.NoError(t, os.RemoveAll(childPath))
610 // A new agent with the same name must not receive the old identity's
612 daemonChildConfig(t, directory, "Garden")
614 agents, err := openDaemonAgents(t.Context(), directory, server.URL, ModelAPIKey(modelTestKey), defaultConfig().Compactor, newTestClock().Now, discardLog())
615 require.NoError(t, err)
617 var loops []*TurnLoop
619 for _, agent := range agents {
620 loop, err := agent.Loop(nil, func(Event) {
622 require.NoError(t, err)
624 loops = append(loops, loop)
627 _, err = openDaemonMessages(loops)
628 require.ErrorContains(t, err, "notification target is missing")
630 after, err := os.ReadFile(path)
631 require.NoError(t, err)
632 require.Equal(t, before, after)
633 require.Zero(t, agents[0].memory.Len())
634 require.Zero(t, agents[1].memory.Len())
635 require.NoError(t, agents.Close())
638 func TestCompletedNotificationReceiptDoesNotRequireItsTargetAtStartup(t *testing.T) {
639 directory := t.TempDir()
640 server := newModelFake(t)
641 server.reply = func(w http.ResponseWriter, _ ModelFakeCall) {
642 modelWriteEvents(w, modelCompletedEvent([]any{modelMessageItem("")}, 100, 0, 5))
645 first := newDaemonMessageTest(t, directory, server)
646 _, err := first.messages.Notify(&NotifyInput{
647 AgentID: first.agents[1].agent.config.ID,
649 ID: "completed-child",
650 Text: "A completed result.",
652 require.NoError(t, err)
653 require.NoError(t, first.messages.Notifications(first.loops[1]))
654 require.NoError(t, first.loops[1].turn(t.Context()))
655 require.Empty(t, first.Queue(t).Pending)
657 childPath := first.agents[1].agent.dir
659 require.NoError(t, os.RemoveAll(childPath))
661 second := newDaemonMessageTest(t, directory, server)
662 require.Empty(t, second.Queue(t).Pending)
663 require.Len(t, second.Queue(t).Notifications, 1)
664 require.Zero(t, second.agents[0].memory.Len())
667 func TestLegacyNotificationReceiptsStillBindTheRoot(t *testing.T) {
668 directory := t.TempDir()
669 server := newModelFake(t)
670 first := newDaemonMessageTest(t, directory, server)
671 input := &NotifyInput{
674 Text: "A root result.",
677 receipt, err := first.messages.Notify(input)
678 require.NoError(t, err)
680 first.messages.state.Pending[0].TargetID = ""
681 first.messages.state.Notifications[0].TargetID = ""
682 require.NoError(t, first.messages.save())
684 before, err := os.ReadFile(first.messages.path)
685 require.NoError(t, err)
688 second := newDaemonMessageTest(t, directory, server)
689 after, err := os.ReadFile(second.messages.path)
690 require.NoError(t, err)
691 require.Equal(t, before, after)
693 duplicate, err := second.messages.Notify(input)
694 require.NoError(t, err)
695 require.Equal(t, receipt.ID, duplicate.ID)
696 require.Equal(t, "duplicate", duplicate.Status)
698 input.AgentID = second.agents[0].agent.config.ID
699 duplicate, err = second.messages.Notify(input)
700 require.NoError(t, err)
701 require.Equal(t, "duplicate", duplicate.Status)
703 input.AgentID = second.agents[1].agent.config.ID
704 _, err = second.messages.Notify(input)
705 require.ErrorContains(t, err, "conflicts")
708 func TestChildNotificationRunsWhileTheRootIsBusy(t *testing.T) {
709 server := newModelFake(t)
710 arrived := make(chan struct{})
711 release := make(chan struct{})
712 server.reply = func(w http.ResponseWriter, call ModelFakeCall) {
713 if call.number == 0 {
718 modelWriteEvents(w, modelCompletedEvent([]any{modelMessageItem("")}, 100, 0, 5))
721 test := newDaemonMessageTest(t, t.TempDir(), server)
724 _, err := test.loops[0].Submit(terminalSource, "Keep the root busy.")
725 require.NoError(t, err)
728 _, err = test.messages.Notify(&NotifyInput{
729 AgentID: test.agents[1].agent.config.ID,
732 Text: "A result for the idle child.",
734 require.NoError(t, err)
735 test.events[1].WaitFor(t, "idle", 1)
736 require.True(t, test.loops[0].Busy())
737 require.Equal(t, 1, test.agents[0].memory.Len())
738 require.Empty(t, test.events[0].Last("work"))
740 test.events[0].WaitFor(t, "idle", 1)
741 require.Empty(t, test.Queue(t).Pending)
742 require.Equal(t, 2, server.Calls())
745 func TestNotificationBatchesStayOrderedForEachChild(t *testing.T) {
746 server := newModelFake(t)
747 server.reply = func(w http.ResponseWriter, _ ModelFakeCall) {
748 modelWriteEvents(w, modelCompletedEvent([]any{modelMessageItem("")}, 100, 0, 5))
751 test := newDaemonMessageTest(t, t.TempDir(), server)
752 test.agents[1].memory.summarizer = newFakeSummarizer()
753 test.agents[2].memory.summarizer = newFakeSummarizer()
755 for number := range 21 {
756 for index := 1; index <= 2; index++ {
757 _, err := test.messages.Notify(&NotifyInput{
758 AgentID: test.agents[index].agent.config.ID,
760 ID: fmt.Sprintf("%d-%d", index, number),
761 Text: fmt.Sprintf("target-%d-result-%02d", index, number),
763 require.NoError(t, err)
767 test.Start(t, 0, 1, 2)
768 test.events[1].WaitFor(t, "idle", 2)
769 test.events[2].WaitFor(t, "idle", 2)
770 require.Equal(t, 4, server.Calls())
771 require.Zero(t, test.agents[0].memory.Len())
772 require.Empty(t, test.Queue(t).Pending)
774 for index := 1; index <= 2; index++ {
775 for number := range 21 {
776 logged, err := test.agents[index].memory.Message(number)
777 require.NoError(t, err)
778 require.Contains(t, logged.Text, fmt.Sprintf("target-%d-result-%02d", index, number))
783 func TestChildNotificationReportsOnlyItsFinalReplyToTheRoot(t *testing.T) {
784 server := newModelFake(t)
785 server.reply = func(w http.ResponseWriter, call ModelFakeCall) {
787 if call.body["model"] == "gpt-test-garden" {
788 reply = "A result needs the user."
791 modelWriteEvents(w, modelCompletedEvent([]any{modelMessageItem(reply)}, 100, 0, 5))
794 test := newDaemonMessageTest(t, t.TempDir(), server)
795 test.Start(t, 0, 1, 2)
797 _, err := test.messages.Notify(&NotifyInput{
798 AgentID: test.agents[1].agent.config.ID,
801 Text: "Private child event data.",
803 require.NoError(t, err)
804 test.events[1].WaitFor(t, "idle", 1)
805 test.events[0].WaitFor(t, "idle", 1)
806 require.Equal(t, 2, server.Calls())
807 require.Equal(t, "[Garden] A result needs the user.", test.events[0].Last("work"))
808 require.Equal(t, 1, test.events[0].Count("work"))
809 require.Equal(t, notifySource, test.events[1].LastEvent("work").Source)
810 require.Zero(t, test.agents[2].memory.Len())
811 require.Empty(t, test.Queue(t).Pending)
814 func TestParentMessageWakesChildAndFinalReplyWakesParent(t *testing.T) {
815 server := newModelFake(t)
816 server.reply = func(w http.ResponseWriter, call ModelFakeCall) {
817 text := "I will ask the user."
818 if call.body["model"] == "gpt-test-garden" {
819 text = "Approve the gate size."
822 modelWriteEvents(w, modelCompletedEvent([]any{modelMessageItem(text)}, 100, 0, 5))
825 test := newDaemonMessageTest(t, t.TempDir(), server)
826 test.Start(t, 0, 1, 2)
827 require.NoError(t, test.messages.Tell("OneChat", "Garden", "Check the gate size."))
828 test.events[0].WaitFor(t, "idle", 1)
829 test.events[1].WaitFor(t, "idle", 1)
831 require.Equal(t, "OneChat", test.events[1].LastEvent("user").Source)
832 require.Equal(t, "Check the gate size.", test.events[1].Last("user"))
833 require.Equal(t, "Garden", test.events[0].LastEvent("work").Source)
834 require.Equal(t, "[Garden] Approve the gate size.", test.events[0].Last("work"))
835 require.Equal(t, []string{"user", "busy", "reply", "idle"}, test.events[1].Kinds())
836 require.Equal(t, []string{"work", "busy", "reply", "idle"}, test.events[0].Kinds())
837 require.Equal(t, 2, server.Calls())
838 require.Zero(t, test.agents[2].memory.Len())
839 require.Empty(t, test.events[2].Kinds())
841 first, err := test.agents[1].memory.Message(0)
842 require.NoError(t, err)
843 require.Equal(t, "user", first.Kind)
845 report, err := test.agents[0].memory.Message(0)
846 require.NoError(t, err)
847 require.Equal(t, "work", report.Kind)
848 require.Equal(t, "[Garden] Approve the gate size.", report.Text)
849 require.Contains(t, test.logs.String(), "turn started name=Garden")
850 require.Contains(t, test.logs.String(), "turn done name=Garden")
851 require.NotContains(t, test.logs.String(), "gate size")
852 require.Empty(t, test.Queue(t).Pending)
854 info, err := os.Stat(test.messages.path)
855 require.NoError(t, err)
856 require.Equal(t, os.FileMode(0o600), info.Mode().Perm())
859 func TestChildReportSendsATelegramCompletionBeforeTheParentReply(t *testing.T) {
860 server := newModelFake(t)
861 server.reply = func(w http.ResponseWriter, call ModelFakeCall) {
862 text := "I will ask the user."
863 if call.body["model"] == "gpt-test-garden" {
864 text = "Approve the gate size."
867 modelWriteEvents(w, modelCompletedEvent([]any{modelMessageItem(text)}, 100, 0, 5))
870 test := newDaemonMessageTest(t, t.TempDir(), server)
871 telegram := newTelegramFake(t)
872 bot := newTelegramBot(newTelegramAPI(telegram.URL, telegramTestToken), telegramTestOwner, filepath.Join(t.TempDir(), telegramOffsetFileName), discardLog())
873 test.loops[0].notify = func(event Event) {
874 test.events[0].Add(event)
878 ctx, cancel := context.WithCancel(t.Context())
881 done := make(chan struct{})
894 require.NoError(t, test.messages.Tell("OneChat", "Garden", "Check the gate size."))
895 test.events[0].WaitFor(t, "idle", 1)
896 require.Eventually(t, func() bool {
897 return len(telegram.Sent()) >= 2
898 }, 10*time.Second, time.Millisecond)
900 require.Equal(t, []string{"[Garden] finished", "I will ask the user."}, telegram.Sent())
901 require.Equal(t, subagentFinished, test.events[0].LastEvent("work").Status)
902 require.Equal(t, "[Garden] Approve the gate size.", test.events[0].Last("work"))
905 func TestQueuedAgentMessageFailureKeepsItsOperationAndCause(t *testing.T) {
906 server := newModelFake(t)
907 test := newDaemonMessageTest(t, t.TempDir(), server)
908 require.NoError(t, test.messages.Tell("OneChat", "Garden", "Keep the plan."))
910 failure := errors.New("fake queued message failure")
911 test.loops[1].fail(failure)
913 err := test.loops[1].Run(t.Context())
914 require.ErrorIs(t, err, failure)
915 require.ErrorContains(t, err, "check turn loop failure")
916 require.Zero(t, server.Calls())
919 func TestEmptyChildReplySendsNothingAndWakesNobody(t *testing.T) {
920 testCases := []struct {
925 name: "sends nothing for an empty final reply",
929 name: "sends nothing for a whitespace final reply",
934 for _, tc := range testCases {
935 t.Run(tc.name, func(t *testing.T) {
936 server := newModelFake(t)
937 server.reply = func(w http.ResponseWriter, _ ModelFakeCall) {
938 modelWriteEvents(w, modelCompletedEvent([]any{modelMessageItem(tc.text)}, 100, 0, 5))
941 test := newDaemonMessageTest(t, t.TempDir(), server)
942 test.Start(t, 0, 1, 2)
943 require.NoError(t, test.messages.Tell("OneChat", "Garden", "Keep the plan."))
944 test.events[1].WaitFor(t, "idle", 1)
945 require.Equal(t, 1, server.Calls())
946 require.Zero(t, test.agents[0].memory.Len())
947 require.Empty(t, test.events[0].Kinds())
948 require.False(t, test.loops[0].waiting())
949 require.Empty(t, test.events[2].Kinds())
950 require.Empty(t, test.Queue(t).Pending)
955 func TestBusyChildQueuesTheNextParentMessage(t *testing.T) {
956 server := newModelFake(t)
957 arrived := make(chan struct{})
958 release := make(chan struct{})
959 server.reply = func(w http.ResponseWriter, call ModelFakeCall) {
960 if call.number == 0 {
965 modelWriteEvents(w, modelCompletedEvent([]any{modelMessageItem("")}, 100, 0, 5))
968 test := newDaemonMessageTest(t, t.TempDir(), server)
970 require.NoError(t, test.messages.Tell("OneChat", "Garden", "Check the first plan."))
972 require.True(t, test.loops[1].Busy())
973 require.NoError(t, test.messages.Tell("OneChat", "Garden", "Check the second plan too."))
974 require.Equal(t, 1, server.Calls())
975 require.Len(t, test.Queue(t).Pending, 2)
977 test.events[1].WaitFor(t, "idle", 2)
978 require.Equal(t, 2, server.Calls())
980 texts := turnInputTexts(t, server.Call(1))
981 require.Equal(t, "Check the second plan too.", texts[len(texts)-1])
982 require.Empty(t, test.Queue(t).Pending)
985 func TestBusyParentQueuesItsChildReport(t *testing.T) {
986 server := newModelFake(t)
987 arrived := make(chan struct{})
988 release := make(chan struct{})
989 server.reply = func(w http.ResponseWriter, call ModelFakeCall) {
990 text := "The plan is ready."
991 if call.number == 0 {
995 text = "I will continue."
998 modelWriteEvents(w, modelCompletedEvent([]any{modelMessageItem(text)}, 100, 0, 5))
1001 test := newDaemonMessageTest(t, t.TempDir(), server)
1004 _, err := test.loops[0].Submit("terminal", "Start the work.")
1005 require.NoError(t, err)
1007 require.NoError(t, test.messages.Tell("OneChat", "Garden", "Prepare the plan."))
1008 test.events[1].WaitFor(t, "idle", 1)
1009 require.Equal(t, "[Garden] The plan is ready.", test.events[0].Last("work"))
1010 require.True(t, test.loops[0].Busy())
1011 require.Equal(t, 2, server.Calls())
1013 test.events[0].WaitFor(t, "idle", 2)
1014 require.Equal(t, 3, server.Calls())
1016 texts := turnInputTexts(t, server.Call(2))
1017 require.Equal(t, "[Garden] The plan is ready.", texts[len(texts)-1])
1018 require.Empty(t, test.Queue(t).Pending)
1021 func TestChildReportsOnlyTheFinalReplyAndTakesQueuedToolInput(t *testing.T) {
1022 server := newModelFake(t)
1023 arrived := make(chan struct{})
1024 release := make(chan struct{})
1025 server.reply = func(w http.ResponseWriter, call ModelFakeCall) {
1026 if call.number == 0 {
1029 modelWriteEvents(w, modelCompletedEvent([]any{
1030 modelMessageItem("I will check the plan."),
1031 modelFunctionCallItem("fc_1", "call_1", "date", `{"id":0}`),
1037 modelWriteEvents(w, modelCompletedEvent([]any{modelMessageItem("The plan needs approval.")}, 100, 0, 5))
1040 test := newDaemonMessageTest(t, t.TempDir(), server)
1042 require.NoError(t, test.messages.Tell("OneChat", "Garden", "Check the plan."))
1044 require.NoError(t, test.messages.Tell("OneChat", "Garden", "Also check the date."))
1046 test.events[1].WaitFor(t, "idle", 1)
1047 require.Equal(t, 2, server.Calls())
1048 require.Equal(t, 1, test.events[1].Count("busy"))
1049 require.Equal(t, 1, test.events[0].Count("work"))
1050 require.Equal(t, "[Garden] The plan needs approval.", test.events[0].Last("work"))
1051 require.NotContains(t, test.events[0].Last("work"), "I will check")
1053 texts := turnInputTexts(t, server.Call(1))
1054 require.Equal(t, "Also check the date.", texts[len(texts)-1])
1056 queue := test.Queue(t)
1057 require.Len(t, queue.Pending, 1)
1058 require.Equal(t, "work", queue.Pending[0].Kind)
1061 func TestQueuedParentMessageSurvivesDaemonRestart(t *testing.T) {
1063 server := newModelFake(t)
1064 server.reply = func(w http.ResponseWriter, _ ModelFakeCall) {
1065 modelWriteEvents(w, modelCompletedEvent([]any{modelMessageItem("")}, 100, 0, 5))
1068 first := newDaemonMessageTest(t, dir, server)
1069 require.NoError(t, first.messages.Tell("OneChat", "Garden", "Remember the garden plan."))
1070 require.Equal(t, 1, first.agents[1].memory.Len())
1073 second := newDaemonMessageTest(t, dir, server)
1074 require.Equal(t, 1, second.agents[1].memory.Len())
1075 require.Equal(t, "OneChat", second.events[1].LastEvent("user").Source)
1076 second.Start(t, 0, 1)
1077 second.events[1].WaitFor(t, "idle", 1)
1078 require.Equal(t, 1, server.Calls())
1079 require.Empty(t, second.Queue(t).Pending)
1082 third := newDaemonMessageTest(t, dir, server)
1083 require.Equal(t, 1, third.agents[1].memory.Len())
1084 require.False(t, third.loops[1].waiting())
1085 require.Empty(t, third.events[1].Kinds())
1088 func TestQueuedChildReportSurvivesDaemonRestart(t *testing.T) {
1090 server := newModelFake(t)
1091 first := newDaemonMessageTest(t, dir, server)
1093 require.NoError(t, first.messages.Tell("OneChat", "Garden", "Review the plan."))
1094 first.events[1].WaitFor(t, "idle", 1)
1095 require.Equal(t, "[Garden] a line", first.events[0].Last("work"))
1096 require.Len(t, first.Queue(t).Pending, 1)
1099 second := newDaemonMessageTest(t, dir, server)
1100 require.Equal(t, 1, second.agents[0].memory.Len())
1101 require.False(t, second.loops[1].waiting())
1102 second.Start(t, 0, 1)
1103 second.events[0].WaitFor(t, "idle", 1)
1104 require.Equal(t, 2, server.Calls())
1105 require.Equal(t, 2, second.agents[0].memory.Len())
1106 require.Empty(t, second.Queue(t).Pending)
1109 third := newDaemonMessageTest(t, dir, server)
1110 require.Equal(t, 2, third.agents[0].memory.Len())
1111 require.False(t, third.loops[0].waiting())
1112 require.Empty(t, third.events[0].Kinds())
1115 func TestInterruptedChildDeliveryResumesAfterDaemonRestart(t *testing.T) {
1117 server := newModelFake(t)
1118 arrived := make(chan struct{})
1119 release := make(chan struct{})
1120 server.reply = func(w http.ResponseWriter, call ModelFakeCall) {
1121 if call.number == 0 {
1126 modelWriteEvents(w, modelCompletedEvent([]any{modelMessageItem("")}, 100, 0, 5))
1129 first := newDaemonMessageTest(t, dir, server)
1131 require.NoError(t, first.messages.Tell("OneChat", "Garden", "Finish the garden plan."))
1136 second := newDaemonMessageTest(t, dir, server)
1137 require.Equal(t, 1, second.agents[1].memory.Len())
1139 second.events[1].WaitFor(t, "idle", 1)
1140 require.Equal(t, 2, server.Calls())
1141 require.Equal(t, 1, second.agents[1].memory.Len())
1142 require.Empty(t, second.Queue(t).Pending)
1145 func TestInterruptedParentReportResumesAfterDaemonRestart(t *testing.T) {
1147 server := newModelFake(t)
1148 arrived := make(chan struct{})
1149 release := make(chan struct{})
1150 server.reply = func(w http.ResponseWriter, call ModelFakeCall) {
1151 if call.number == 1 {
1156 modelWriteEvents(w, modelCompletedEvent([]any{modelMessageItem("The plan is ready.")}, 100, 0, 5))
1159 first := newDaemonMessageTest(t, dir, server)
1160 first.Start(t, 0, 1)
1161 require.NoError(t, first.messages.Tell("OneChat", "Garden", "Review the plan."))
1166 second := newDaemonMessageTest(t, dir, server)
1167 require.Equal(t, 1, second.agents[0].memory.Len())
1168 require.False(t, second.loops[1].waiting())
1169 second.Start(t, 0, 1)
1170 second.events[0].WaitFor(t, "idle", 1)
1171 require.Equal(t, 3, server.Calls())
1172 require.Equal(t, 2, second.agents[0].memory.Len())
1173 require.Empty(t, second.Queue(t).Pending)
1175 texts := turnInputTexts(t, server.Call(2))
1176 require.Equal(t, "[Garden] The plan is ready.", texts[len(texts)-1])
1179 func TestAgentMessageWriteFailureStopsDelivery(t *testing.T) {
1180 server := newModelFake(t)
1181 test := newDaemonMessageTest(t, t.TempDir(), server)
1182 require.NoError(t, os.Mkdir(test.messages.path, 0o700))
1183 require.ErrorContains(t, test.messages.Tell("OneChat", "Garden", "Keep the plan."), "save agent messages")
1184 require.ErrorContains(t, test.messages.Tell("OneChat", "Garden", "Do not send again."), "save agent messages")
1186 for _, loop := range test.loops {
1187 require.ErrorContains(t, loop.failed(), "save agent messages")
1188 require.Zero(t, loop.memory.Len())
1192 func TestAgentMessageSnapshotBlocksDelivery(t *testing.T) {
1193 server := newModelFake(t)
1194 test := newDaemonMessageTest(t, t.TempDir(), server)
1195 entered := make(chan struct{})
1196 release := make(chan struct{})
1200 defer once.Do(func() {
1204 snapshot := make(chan error, 1)
1206 snapshot <- test.messages.Snapshot(func() error {
1207 return daemonSnapshot(t.Context(), test.loops, func() error {
1218 sent := make(chan error, 1)
1219 started := make(chan struct{})
1223 sent <- test.messages.Tell("OneChat", "Garden", "Record the plan after the copy.")
1227 require.Never(t, func() bool {
1228 return len(sent) > 0
1229 }, 50*time.Millisecond, time.Millisecond)
1234 require.NoError(t, <-snapshot)
1235 require.NoError(t, <-sent)
1236 require.Equal(t, 1, test.agents[1].memory.Len())
1237 require.Len(t, test.Queue(t).Pending, 1)
1240 func TestParentCanMessageOnlyItsOwnChildren(t *testing.T) {
1241 server := newModelFake(t)
1243 test := newDaemonMessageTest(t, t.TempDir(), server)
1244 testCases := []struct {
1250 name: "no sibling messages",
1255 name: "no arbitrary child reports",
1260 name: "no self messages",
1265 name: "refuses an unknown sender",
1270 name: "refuses an unknown recipient",
1276 for _, tc := range testCases {
1277 t.Run(tc.name, func(t *testing.T) {
1278 require.ErrorContains(t, test.messages.Tell(tc.source, tc.target, "Do not deliver."), "not a child")
1282 for _, agent := range test.agents {
1283 require.Zero(t, agent.memory.Len())
1287 func TestInvalidAgentMessageQueueStopsStartup(t *testing.T) {
1288 server := newModelFake(t)
1289 testCases := []struct {
1294 name: "stops startup for broken JSON",
1298 name: "stops startup for unknown sender",
1299 file: `{"next":1,"pending":[{"id":1,"source":"Other","target":"Garden","kind":"user"}]}`,
1302 name: "stops startup for sibling delivery",
1303 file: `{"next":1,"pending":[{"id":1,"source":"Library","target":"Garden","kind":"user"}]}`,
1306 name: "stops startup for invalid sequence",
1307 file: `{"next":0,"pending":[{"id":1,"source":"OneChat","target":"Garden","kind":"user"}]}`,
1311 for _, tc := range testCases {
1312 t.Run(tc.name, func(t *testing.T) {
1314 test := newDaemonMessageTest(t, dir, server)
1315 path := test.messages.path
1317 require.NoError(t, os.WriteFile(path, []byte(tc.file), 0o600))
1319 agents, err := openDaemonAgents(t.Context(), dir, server.URL, ModelAPIKey(modelTestKey), defaultConfig().Compactor, newTestClock().Now, discardLog())
1320 require.NoError(t, err)
1323 require.NoError(t, agents.Close())
1326 var loops []*TurnLoop
1328 for _, agent := range agents {
1329 loop, err := agent.Loop(nil, func(Event) {
1331 require.NoError(t, err)
1333 loops = append(loops, loop)
1336 _, err = openDaemonMessages(loops)
1337 require.Error(t, err)
1339 for _, agent := range agents {
1340 require.Zero(t, agent.memory.Len())
1346 func TestChildScheduleWakesOnlyItsOwner(t *testing.T) {
1347 server := newModelFake(t)
1348 server.reply = func(w http.ResponseWriter, _ ModelFakeCall) {
1349 modelWriteEvents(w, modelCompletedEvent([]any{modelMessageItem("")}, 100, 0, 5))
1352 test := newDaemonMessageTest(t, t.TempDir(), server)
1353 child := test.agents[1]
1354 at := child.clock().Add(time.Minute).Format(scheduleAtLayout)
1355 output := test.loops[1].tools.run(t.Context(), "schedule_set", `{"text":"Check the plants.","at":"`+at+`","cron":null}`)
1356 require.Contains(t, output, "Schedule 1 set.")
1357 require.Equal(t, "Garden", child.schedules.sorted()[0].Owner)
1359 child.schedules.clock = func() time.Time {
1360 return child.clock().Add(time.Minute)
1362 require.ErrorContains(t, child.schedules.deliver(t.Context(), test.loops[0]), "does not match")
1363 require.NoError(t, child.schedules.deliver(t.Context(), test.loops[1]))
1364 test.Start(t, 0, 1, 2)
1365 test.events[1].WaitFor(t, "idle", 1)
1366 require.Equal(t, 1, server.Calls())
1367 require.Equal(t, child.agent.CacheKey(), server.Call(0).body["prompt_cache_key"])
1368 require.Zero(t, test.agents[0].memory.Len())
1369 require.Zero(t, test.agents[2].memory.Len())
1370 require.Contains(t, test.events[1].LastEvent("work").Text, "Check the plants.")
1371 require.Equal(t, scheduleSource, test.events[1].LastEvent("work").Source)
1374 func TestRoutedWhatsAppBatchesWakeOnlyTheirChild(t *testing.T) {
1375 testCases := []struct {
1380 name: "routes a private chat only to its child",
1382 ID: whatsappTestAna.String(),
1387 name: "routes an unmentioned group only to its child",
1389 ID: whatsappTestFamily.String(),
1390 Name: "Garden plans",
1396 for _, tc := range testCases {
1397 t.Run(tc.name, func(t *testing.T) {
1398 server := newModelFake(t)
1399 server.reply = func(w http.ResponseWriter, _ ModelFakeCall) {
1400 modelWriteEvents(w, modelCompletedEvent([]any{modelMessageItem("")}, 100, 0, 5))
1404 test := newDaemonMessageTest(t, dir, server)
1405 inbox := whatsappRouteTestInbox(t, dir)
1406 require.NoError(t, test.messages.routes.Set("whatsapp:"+tc.chat.ID, "Garden"))
1407 whatsappRouteTestAdd(t, inbox, tc.chat, "The gate needs a new latch.")
1408 test.Start(t, 0, 1, 2)
1409 require.NoError(t, inbox.deliver(t.Context(), test.loops[0], WhatsAppLink{}, true))
1410 test.events[1].WaitFor(t, "idle", 1)
1411 require.Equal(t, 1, server.Calls())
1412 require.Equal(t, "gpt-test-garden", server.Call(0).body["model"])
1413 require.Contains(t, string(server.Call(0).raw), "The gate needs a new latch.")
1414 require.Zero(t, test.agents[0].memory.Len())
1415 require.Empty(t, test.events[0].Kinds())
1416 require.Zero(t, test.agents[2].memory.Len())
1417 require.False(t, test.loops[0].waiting())
1418 require.Equal(t, whatsappSource, test.events[1].LastEvent("work").Source)
1419 require.Empty(t, test.events[1].LastEvent("work").Status)
1420 require.Equal(t, 1, test.agents[1].memory.Len())
1421 require.Empty(t, inbox.state.Closed())
1422 require.Empty(t, test.Queue(t).Pending)
1423 require.Contains(t, test.logs.String(), "batch routed name=Garden kind=whatsapp id=1 to=Garden")
1424 require.NotContains(t, test.logs.String(), tc.chat.ID)
1425 require.NotContains(t, test.logs.String(), "new latch")
1430 func TestWhatsAppUnrouteRestoresRootDelivery(t *testing.T) {
1432 test := newDaemonMessageTest(t, dir, newModelFake(t))
1433 inbox := whatsappRouteTestInbox(t, dir)
1434 chat := WhatsAppChat{
1435 ID: whatsappTestAna.String(),
1439 source := "whatsapp:" + chat.ID
1440 require.NoError(t, test.messages.routes.Set(source, "Garden"))
1441 whatsappRouteTestAdd(t, inbox, chat, "Keep the gate plan.")
1442 require.NoError(t, inbox.deliver(t.Context(), test.loops[0], WhatsAppLink{}, true))
1443 require.Zero(t, test.agents[0].memory.Len())
1444 require.True(t, test.loops[1].waiting())
1445 require.NoError(t, test.messages.routes.Set(source, ""))
1446 whatsappRouteTestAdd(t, inbox, chat, "Please check the other plan.")
1447 require.NoError(t, inbox.deliver(t.Context(), test.loops[0], WhatsAppLink{}, true))
1448 require.Equal(t, 1, test.agents[0].memory.Len())
1449 require.Equal(t, 1, test.agents[1].memory.Len())
1450 require.True(t, test.loops[0].waiting())
1451 require.Contains(t, test.events[0].Last("work"), "Please check the other plan.")
1452 require.NotContains(t, test.events[0].Last("work"), "Keep the gate plan.")
1453 require.Empty(t, inbox.state.Closed())
1456 func TestMissingWhatsAppRouteChildFallsBackToRoot(t *testing.T) {
1458 test := newDaemonMessageTest(t, dir, newModelFake(t))
1459 inbox := whatsappRouteTestInbox(t, dir)
1460 chat := WhatsAppChat{
1461 ID: whatsappTestAna.String(),
1465 require.NoError(t, test.messages.routes.Set("whatsapp:"+chat.ID, "Missing"))
1466 whatsappRouteTestAdd(t, inbox, chat, "Please check the plan.")
1467 require.NoError(t, inbox.deliver(t.Context(), test.loops[0], WhatsAppLink{}, true))
1468 require.Equal(t, 1, test.agents[0].memory.Len())
1469 require.True(t, test.loops[0].waiting())
1470 require.Zero(t, test.agents[1].memory.Len())
1471 require.False(t, test.loops[1].waiting())
1472 require.Empty(t, test.events[0].LastEvent("work").Status)
1473 require.Equal(t, whatsappSource, test.events[0].LastEvent("work").Source)
1474 require.Empty(t, inbox.state.Closed())
1477 func TestRoutedWhatsAppRestartDoesNotDeliverTheBatchTwice(t *testing.T) {
1478 testCases := []struct {
1483 name: "queued turn resumes",
1487 name: "finished turn stays finished",
1492 for _, tc := range testCases {
1493 t.Run(tc.name, func(t *testing.T) {
1495 server := newModelFake(t)
1496 server.reply = func(w http.ResponseWriter, _ ModelFakeCall) {
1497 modelWriteEvents(w, modelCompletedEvent([]any{modelMessageItem("")}, 100, 0, 5))
1500 first := newDaemonMessageTest(t, dir, server)
1501 inbox := whatsappRouteTestInbox(t, dir)
1502 chat := WhatsAppChat{
1503 ID: whatsappTestAna.String(),
1507 source := "whatsapp:" + chat.ID
1508 require.NoError(t, first.messages.routes.Set(source, "Garden"))
1509 whatsappRouteTestAdd(t, inbox, chat, "Remember the gate plan.")
1510 require.NoError(t, inbox.state.CloseAll())
1512 batch := inbox.state.Closed()[0]
1513 routed, err := first.messages.RouteWhatsApp(batch, batch.Text(time.UTC))
1514 require.NoError(t, err)
1515 require.True(t, routed)
1519 first.events[1].WaitFor(t, "idle", 1)
1520 require.Empty(t, first.Queue(t).Pending)
1525 second := newDaemonMessageTest(t, dir, server)
1526 require.Equal(t, source+" -> Garden", second.messages.routes.List())
1527 require.Equal(t, 1, second.agents[1].memory.Len())
1528 require.Equal(t, !tc.finish, second.loops[1].waiting())
1529 require.NoError(t, second.messages.routes.Set(source, ""))
1531 restored := whatsappRouteTestInbox(t, dir)
1532 require.Len(t, restored.state.Closed(), 1)
1533 require.NoError(t, restored.deliver(t.Context(), second.loops[0], WhatsAppLink{}, true))
1534 require.Empty(t, restored.state.Closed())
1535 require.Zero(t, second.agents[0].memory.Len())
1536 require.Equal(t, 1, second.agents[1].memory.Len())
1540 second.events[1].WaitFor(t, "idle", 1)
1543 require.Equal(t, 1, server.Calls())
1544 require.Empty(t, second.Queue(t).Pending)
1547 third := newDaemonMessageTest(t, dir, server)
1548 require.False(t, third.loops[1].waiting())
1549 require.Empty(t, third.events[1].Kinds())
1550 require.Equal(t, 1, third.agents[1].memory.Len())
1555 func TestQueuedWhatsAppBatchRecoversBeforeItsLogAppend(t *testing.T) {
1556 testCases := []struct {
1561 name: "recovers one message before its log ID is stored",
1565 name: "recovers one message after its log ID is stored",
1570 for _, tc := range testCases {
1571 t.Run(tc.name, func(t *testing.T) {
1573 server := newModelFake(t)
1574 first := newDaemonMessageTest(t, dir, server)
1575 path := first.messages.path
1578 file := DaemonMessageFile{
1580 WhatsAppBatches: []int{1},
1581 Pending: []DaemonMessage{{
1586 Text: "[WhatsApp] Keep the gate plan.",
1592 data, err := json.Marshal(file)
1593 require.NoError(t, err)
1594 require.NoError(t, os.WriteFile(path, data, 0o600))
1596 second := newDaemonMessageTest(t, dir, server)
1597 require.Equal(t, 1, second.agents[1].memory.Len())
1598 require.True(t, second.loops[1].waiting())
1599 require.Equal(t, "[WhatsApp] Keep the gate plan.", second.events[1].Last("work"))
1600 require.Equal(t, whatsappSource, second.events[1].LastEvent("work").Source)
1603 third := newDaemonMessageTest(t, dir, server)
1604 require.Equal(t, 1, third.agents[1].memory.Len())
1605 require.True(t, third.loops[1].waiting())
1606 require.Zero(t, third.agents[0].memory.Len())
1611 func TestQueuedWhatsAppBatchUsesRootIfItsChildIsRemoved(t *testing.T) {
1613 server := newModelFake(t)
1614 first := newDaemonMessageTest(t, dir, server)
1615 inbox := whatsappRouteTestInbox(t, dir)
1616 chat := WhatsAppChat{
1617 ID: whatsappTestAna.String(),
1621 require.NoError(t, first.messages.routes.Set("whatsapp:"+chat.ID, "Garden"))
1622 whatsappRouteTestAdd(t, inbox, chat, "Keep the gate plan.")
1623 require.NoError(t, inbox.deliver(t.Context(), first.loops[0], WhatsAppLink{}, true))
1625 require.NoError(t, os.RemoveAll(filepath.Join(dir, daemonAgentDirName, "children", "Garden")))
1627 second := newDaemonMessageTest(t, dir, server)
1628 require.Equal(t, 1, second.agents[0].memory.Len())
1629 require.True(t, second.loops[0].waiting())
1630 require.Equal(t, whatsappSource, second.events[0].LastEvent("work").Source)
1631 require.Empty(t, second.events[0].LastEvent("work").Status)
1632 require.Contains(t, second.events[0].Last("work"), "Keep the gate plan.")
1633 require.False(t, second.loops[1].waiting())
1636 third := newDaemonMessageTest(t, dir, server)
1637 require.Equal(t, 1, third.agents[0].memory.Len())
1638 require.True(t, third.loops[0].waiting())
1641 func TestRoutedWhatsAppBatchesCanCloseOutOfNumberOrder(t *testing.T) {
1643 test := newDaemonMessageTest(t, dir, newModelFake(t))
1644 inbox := whatsappRouteTestInbox(t, dir)
1645 first := WhatsAppChat{
1646 ID: whatsappTestAna.String(),
1650 second := WhatsAppChat{
1651 ID: whatsappTestFamily.String(),
1652 Name: "Garden plans",
1656 require.NoError(t, test.messages.routes.Set("whatsapp:"+first.ID, "Garden"))
1657 require.NoError(t, test.messages.routes.Set("whatsapp:"+second.ID, "Garden"))
1660 _, err := inbox.state.Add(first, WhatsAppMessage{
1662 Text: "The first chat is still open.",
1666 require.NoError(t, err)
1668 _, err = inbox.state.Add(second, WhatsAppMessage{
1670 Text: "The second chat is quiet.",
1673 }, now.Add(-time.Minute))
1674 require.NoError(t, err)
1675 require.NoError(t, inbox.deliver(t.Context(), test.loops[0], WhatsAppLink{}, false))
1676 require.Equal(t, 1, test.agents[1].memory.Len())
1677 require.Contains(t, test.events[1].Last("work"), "The second chat is quiet.")
1678 require.NoError(t, inbox.deliver(t.Context(), test.loops[0], WhatsAppLink{}, true))
1679 require.Equal(t, 2, test.agents[1].memory.Len())
1680 require.Contains(t, test.events[1].Last("work"), "The first chat is still open.")
1681 require.Zero(t, test.agents[0].memory.Len())
1682 require.Empty(t, test.Queue(t).WhatsAppBatches)
1685 func TestWhatsAppCatchupSeparatesChildAndRootBatches(t *testing.T) {
1687 test := newDaemonMessageTest(t, dir, newModelFake(t))
1688 inbox := whatsappRouteTestInbox(t, dir)
1689 garden := WhatsAppChat{
1690 ID: whatsappTestFamily.String(),
1691 Name: "Garden plans",
1695 library := WhatsAppChat{
1696 ID: whatsappTestBruno.String(),
1700 root := WhatsAppChat{
1701 ID: whatsappTestAna.String(),
1705 require.NoError(t, test.messages.routes.Set("whatsapp:"+garden.ID, "Garden"))
1706 require.NoError(t, test.messages.routes.Set("whatsapp:"+library.ID, "Library"))
1707 inbox.waitForCatchup()
1708 whatsappRouteTestAdd(t, inbox, garden, "Check the gate.")
1709 whatsappRouteTestAdd(t, inbox, library, "Check the book.")
1710 whatsappRouteTestAdd(t, inbox, root, "Check the mixed chat.")
1711 require.NoError(t, inbox.deliver(t.Context(), test.loops[0], WhatsAppLink{}, false))
1713 for _, agent := range test.agents {
1714 require.Zero(t, agent.memory.Len())
1718 require.NoError(t, inbox.deliver(t.Context(), test.loops[0], WhatsAppLink{}, false))
1720 for _, loop := range test.loops {
1721 require.Equal(t, 1, loop.memory.Len())
1722 require.True(t, loop.waiting())
1725 require.Contains(t, test.events[0].Last("work"), "Check the mixed chat.")
1726 require.Contains(t, test.events[1].Last("work"), "Check the gate.")
1727 require.Contains(t, test.events[2].Last("work"), "Check the book.")
1728 require.Empty(t, inbox.state.Closed())
1729 require.Empty(t, test.Queue(t).WhatsAppBatches)
1732 func TestRoutedWhatsAppQueueFailureKeepsTheInboxBatch(t *testing.T) {
1734 server := newModelFake(t)
1735 first := newDaemonMessageTest(t, dir, server)
1736 inbox := whatsappRouteTestInbox(t, dir)
1737 chat := WhatsAppChat{
1738 ID: whatsappTestAna.String(),
1742 require.NoError(t, first.messages.routes.Set("whatsapp:"+chat.ID, "Garden"))
1743 require.NoError(t, os.Mkdir(first.messages.path, 0o700))
1744 whatsappRouteTestAdd(t, inbox, chat, "Keep the gate plan.")
1745 require.ErrorContains(t, inbox.deliver(t.Context(), first.loops[0], WhatsAppLink{}, true), "save agent messages")
1746 require.Len(t, inbox.state.Closed(), 1)
1748 for _, loop := range first.loops {
1749 require.ErrorContains(t, loop.failed(), "save agent messages")
1750 require.Zero(t, loop.memory.Len())
1753 path := first.messages.path
1755 require.NoError(t, os.Remove(path))
1757 second := newDaemonMessageTest(t, dir, server)
1758 restored := whatsappRouteTestInbox(t, dir)
1759 require.NoError(t, restored.deliver(t.Context(), second.loops[0], WhatsAppLink{}, true))
1760 require.Equal(t, 1, second.agents[1].memory.Len())
1761 require.True(t, second.loops[1].waiting())
1762 require.Zero(t, second.agents[0].memory.Len())
1763 require.Empty(t, restored.state.Closed())
1766 func whatsappRouteTestInbox(t *testing.T, dir string) *WhatsAppInbox {
1769 state, err := openWhatsAppState(filepath.Join(dir, whatsappDirName, whatsappStateFileName))
1770 require.NoError(t, err)
1772 return newWhatsAppInbox(state, newTestClock().Now, newWhatsAppGroups(discardLog()), &ImageStore{dir: filepath.Join(dir, imageDirName)}, discardLog())
1775 func whatsappRouteTestAdd(t *testing.T, inbox *WhatsAppInbox, chat WhatsAppChat, text string) {
1778 at := time.Date(2026, 1, 1, 12, 0, 0, 0, time.UTC)
1779 _, err := inbox.state.Add(chat, WhatsAppMessage{
1785 require.NoError(t, err)