refactor(tests): improve comments for clarity in ChannelFiltering test case
This commit is contained in:
parent
e7518e8e41
commit
de6a04bf9c
2 changed files with 5 additions and 2 deletions
|
|
@ -384,7 +384,11 @@ func TestSpawnStatusTool_ChannelFiltering_NoContext(t *testing.T) {
|
|||
|
||||
tool := NewSpawnStatusTool(manager)
|
||||
|
||||
// No tool context (e.g. CLI) — callerChannel and callerChatID are both "".
|
||||
// No ToolContext injected (e.g. a direct programmatic call that bypasses
|
||||
// WithToolContext entirely) — callerChannel and callerChatID are both "".
|
||||
// Note: the normal CLI path uses ProcessDirectWithChannel("cli", "direct"),
|
||||
// which *does* inject a non-empty context; this test covers the case where
|
||||
// no context injection happens at all.
|
||||
// The filter conditions require a non-empty caller value, so all tasks pass through.
|
||||
result := tool.Execute(context.Background(), map[string]any{})
|
||||
if result.IsError {
|
||||
|
|
|
|||
|
|
@ -111,7 +111,6 @@ func (sm *SubagentManager) Spawn(
|
|||
func (sm *SubagentManager) runTask(ctx context.Context, task *SubagentTask, callback AsyncCallback) {
|
||||
sm.mu.Lock()
|
||||
task.Status = "running"
|
||||
task.Created = time.Now().UnixMilli()
|
||||
sm.mu.Unlock()
|
||||
|
||||
// Build system prompt for subagent
|
||||
|
|
|
|||
Loading…
Add table
Reference in a new issue