mirror of
https://github.com/sipeed/picoclaw.git
synced 2026-06-12 18:08:54 +00:00
5db008f384
* fix(channels): dismiss tool feedback animation when turn ends via ResponseHandled When a tool sets ResponseHandled=true (e.g., send_file), the turn ends without producing a final assistant response. This meant no outbound message triggered FinalizeToolFeedbackMessage, leaving the animation goroutine running indefinitely — editing the Feishu card every 3 seconds with "." / ".." suffixes long after the tool had finished. Fix: call DismissToolFeedback at "Tool output satisfied delivery" so the tracker is cleared and the animation goroutine is stopped immediately. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com> * fix(adapters): add DismissToolFeedback to channelManagerAdapter The adapter must implement the new interface method added in the previous commit, otherwise the package fails to compile. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com> * fix(channels): pass InboundContext to DismissToolFeedback for topic-aware keys Telegram forum topics use scoped tracker keys like "chatID/topicID", resolved via ToolFeedbackMessageChatID with the InboundContext. The previous nil context caused the lookup to fall back to the raw chatID, missing the topic-scoped entry and leaving the animation goroutine orphaned in forum-topic conversations. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com> * style: wrap long function signatures for golines Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
52 lines
1.5 KiB
Go
52 lines
1.5 KiB
Go
// PicoClaw - Ultra-lightweight personal AI agent
|
|
|
|
package adapters
|
|
|
|
import (
|
|
"context"
|
|
|
|
"github.com/sipeed/picoclaw/pkg/agent/interfaces"
|
|
"github.com/sipeed/picoclaw/pkg/bus"
|
|
"github.com/sipeed/picoclaw/pkg/channels"
|
|
)
|
|
|
|
// channelManagerAdapter wraps *channels.Manager to implement interfaces.ChannelManager.
|
|
type channelManagerAdapter struct {
|
|
inner *channels.Manager
|
|
}
|
|
|
|
// NewChannelManager creates an adapter for *channels.Manager.
|
|
func NewChannelManager(inner *channels.Manager) interfaces.ChannelManager {
|
|
return &channelManagerAdapter{inner: inner}
|
|
}
|
|
|
|
func (a *channelManagerAdapter) GetChannel(name string) (channels.Channel, bool) {
|
|
return a.inner.GetChannel(name)
|
|
}
|
|
|
|
func (a *channelManagerAdapter) GetEnabledChannels() []string {
|
|
return a.inner.GetEnabledChannels()
|
|
}
|
|
|
|
func (a *channelManagerAdapter) InvokeTypingStop(channel, chatID string) {
|
|
a.inner.InvokeTypingStop(channel, chatID)
|
|
}
|
|
|
|
func (a *channelManagerAdapter) SendMessage(ctx context.Context, msg bus.OutboundMessage) error {
|
|
return a.inner.SendMessage(ctx, msg)
|
|
}
|
|
|
|
func (a *channelManagerAdapter) SendMedia(ctx context.Context, msg bus.OutboundMediaMessage) error {
|
|
return a.inner.SendMedia(ctx, msg)
|
|
}
|
|
|
|
func (a *channelManagerAdapter) SendPlaceholder(ctx context.Context, channel, chatID string) bool {
|
|
return a.inner.SendPlaceholder(ctx, channel, chatID)
|
|
}
|
|
|
|
func (a *channelManagerAdapter) DismissToolFeedback(
|
|
ctx context.Context, channel, chatID string, outboundCtx *bus.InboundContext,
|
|
) {
|
|
a.inner.DismissToolFeedback(ctx, channel, chatID, outboundCtx)
|
|
}
|