Skip to content

Commit 3c7faa0

Browse files
committed
fix(telegram): gate skill log messages behind skills.verbose
The Telegram SkillEventHandler sent skill lifecycle messages (loaded, autoloaded, saved, deleted) to the chat regardless of the skills.verbose config setting. Now all four message types check skillsCfg.Verbose before sending, matching the behavior of the CLI Renderer which already respects this flag.
1 parent 2659a46 commit 3c7faa0

1 file changed

Lines changed: 14 additions & 6 deletions

File tree

cmd/odek/telegram.go

Lines changed: 14 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -837,15 +837,23 @@ func handleChatMessage(
837837
SkillEventHandler: func(event skills.SkillEvent) {
838838
switch event.Type {
839839
case "loaded":
840-
names := strings.Join(event.Skills, ", ")
841-
sendAsync(bot, chatID, "📚 Loaded skill: "+names, &telegram.SendOpts{ReplyToMessageID: messageID})
840+
if skillsCfg != nil && skillsCfg.Verbose {
841+
names := strings.Join(event.Skills, ", ")
842+
sendAsync(bot, chatID, "📚 Loaded skill: "+names, &telegram.SendOpts{ReplyToMessageID: messageID})
843+
}
842844
case "autoloaded":
843-
names := strings.Join(event.Skills, ", ")
844-
sendAsync(bot, chatID, "📚 Auto-loaded skills: "+names, &telegram.SendOpts{ReplyToMessageID: messageID})
845+
if skillsCfg != nil && skillsCfg.Verbose {
846+
names := strings.Join(event.Skills, ", ")
847+
sendAsync(bot, chatID, "📚 Auto-loaded skills: "+names, &telegram.SendOpts{ReplyToMessageID: messageID})
848+
}
845849
case "saved":
846-
sendAsync(bot, chatID, fmt.Sprintf("✓ Saved skill %q", event.SkillName), &telegram.SendOpts{ReplyToMessageID: messageID})
850+
if skillsCfg != nil && skillsCfg.Verbose {
851+
sendAsync(bot, chatID, fmt.Sprintf("✓ Saved skill %q", event.SkillName), &telegram.SendOpts{ReplyToMessageID: messageID})
852+
}
847853
case "deleted":
848-
sendAsync(bot, chatID, fmt.Sprintf("✗ Deleted skill %q", event.SkillName), &telegram.SendOpts{ReplyToMessageID: messageID})
854+
if skillsCfg != nil && skillsCfg.Verbose {
855+
sendAsync(bot, chatID, fmt.Sprintf("✗ Deleted skill %q", event.SkillName), &telegram.SendOpts{ReplyToMessageID: messageID})
856+
}
849857
case "suggested":
850858
replyMarkup := &telegram.InlineKeyboardMarkup{
851859
InlineKeyboard: [][]telegram.InlineKeyboardButton{

0 commit comments

Comments
 (0)