Refactor duplicated []string→[]interface{} conversions into strutil.StringsToAny

[Copilot]

This change removes three repeated inline conversion loops that transformed []string into []interface{} in guard/config paths. The conversion is now centralized to reduce boilerplate and keep future behavior changes in one place.

• New shared utility

  • Added StringsToAny([]string) []interface{} in internal/strutil/strutil.go.
  • Purpose: canonical conversion helper for JSON/generic-interface call paths.

• Guard payload cleanup

  • Replaced duplicate loops in BuildLabelAgentPayload (internal/guard/wasm_payload.go) for:
    • trusted-bots
    • trusted-users
  • Call sites now use strutil.StringsToAny(...).

• Guard policy normalization cleanup

  • Replaced the inline []string scope conversion in internal/config/guard_policy_validation.go with strutil.StringsToAny(scope) before normalizeAndValidateScopeArray(...).

• Focused coverage for helper

  • Added internal/strutil/strings_to_any_test.go covering:
    • nil input
    • empty input
    • ordered value conversion

// before
generic := make([]interface{}, len(scope))
for i := range scope {
    generic[i] = scope[i]
}
scopes, err := normalizeAndValidateScopeArray(generic)

// after
scopes, err := normalizeAndValidateScopeArray(strutil.StringsToAny(scope))

[Copilot]

Pull request overview

This PR reduces duplicated boilerplate by centralizing []string → []interface{} conversions into a shared internal/strutil.StringsToAny helper, then updating guard/config code paths to use it.

Changes:

• Added strutil.StringsToAny([]string) []interface{} as the canonical conversion helper.
• Replaced repeated inline conversion loops in guard label-agent payload building and guard policy normalization.
• Added a focused unit test covering conversion behavior.

Show a summary per file

File	Description
internal/strutil/strutil.go	Adds the new shared StringsToAny conversion helper.
internal/strutil/strings_to_any_test.go	Introduces unit tests for StringsToAny behavior.
internal/guard/wasm_payload.go	Replaces inline []string→[]interface{} loops for trusted-bots/users with StringsToAny.
internal/config/guard_policy_validation.go	Uses StringsToAny when normalizing allow-only.repos provided as []string.

Copilot's findings

Tip

Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

• Files reviewed: 4/4 changed files
• Comments generated: 1