fix: sync.Map copy in go vet — use pointer assignment

molty3000 · molty3000 · commit 14631f8507dc · 2026-05-23T23:32:39.000+02:00
_ = toolMsgIDs copies the sync.Map value, triggering:
  'assignment copies lock value to _: sync.Map contains sync.noCopy'

Using _ = &amp;toolMsgIDs avoids the copy while still suppressing the
unused-variable warning.
diff --git a/cmd/odek/telegram_test.go b/cmd/odek/telegram_test.go
@@ -489,8 +489,8 @@ func TestEngagingMode_UsesEdits(t *testing.T) {
 	}
 
 	iterCallback := func(info loop.IterationInfo) {
+		_ = &toolMsgIDs
 		_ = truncateWords
-		_ = toolMsgIDs
 		_ = truncate
 		if info.IsPreTool && info.ReasoningContent != "" {
 			bot.SendMessage(chatID, "💭 "+truncateWords(info.ReasoningContent, 50),

Original file line number	Diff line number	Diff line change
`@@ -489,8 +489,8 @@ func TestEngagingMode_UsesEdits(t *testing.T) {`
`489`	`489`	`}`
`490`	`490`
`491`	`491`	`iterCallback := func(info loop.IterationInfo) {`
	`492`	`+ _ = &toolMsgIDs`
`492`	`493`	`_ = truncateWords`
`493`		`- _ = toolMsgIDs`
`494`	`494`	`_ = truncate`
`495`	`495`	`if info.IsPreTool && info.ReasoningContent != "" {`
`496`	`496`	`bot.SendMessage(chatID, "💭 "+truncateWords(info.ReasoningContent, 50),`