refactor: replace bool map with set-style map for internal channels

Use map[string]struct{} and comma-ok idiom for clearer and more idiomatic membership checks.
This commit is contained in:
Ruslan Semagin 2026-02-19 11:10:37 +03:00
parent 56a060ff61
commit a38fbf289f

View file

@ -1,15 +1,16 @@
// Package constants provides shared constants across the codebase. // Package constants provides shared constants across the codebase.
package constants package constants
// InternalChannels defines channels that are used for internal communication // internalChannels defines channels that are used for internal communication
// and should not be exposed to external users or recorded as last active channel. // and should not be exposed to external users or recorded as last active channel.
var InternalChannels = map[string]bool{ var internalChannels = map[string]struct{}{
"cli": true, "cli": {},
"system": true, "system": {},
"subagent": true, "subagent": {},
} }
// IsInternalChannel returns true if the channel is an internal channel. // IsInternalChannel returns true if the channel is an internal channel.
func IsInternalChannel(channel string) bool { func IsInternalChannel(channel string) bool {
return InternalChannels[channel] _, ok := internalChannels[channel]
return ok
} }