Fix S1011 staticcheck lint failure in central_slash_command_workflow.go

[Copilot]

Bug Fix

What was the bug?

golangci-lint (staticcheck S1011) was failing CI on pkg/workflow/central_slash_command_workflow.go — a for loop was appending slice elements one-by-one instead of using the idiomatic append(...spread) form.

How did you fix it?

Replaced the inner loop with a single append call using the spread operator:

// Before
for _, route := range buildCentralizedRoutes(wd, routeEvents) {
    routesByCommand[commandName] = append(routesByCommand[commandName], route)
}

// After
routesByCommand[commandName] = append(routesByCommand[commandName], buildCentralizedRoutes(wd, routeEvents)...)

No behavioral change — purely a style/lint conformance fix.

[Copilot]

Pull request overview

Fixes a golangci-lint / staticcheck S1011 issue by replacing a per-element append loop with an idiomatic append(dst, src...) when aggregating centralized slash-command routes.

Changes:

• Simplified route aggregation in collectCentralSlashCommandRoutes using a single spread append.
• Minor whitespace-only change in the generated agentic_commands.yml workflow file.

Show a summary per file

File	Description
pkg/workflow/central_slash_command_workflow.go	Replaces an inner append loop with append(..., buildCentralizedRoutes(...)...) to satisfy S1011.
.github/workflows/agentic_commands.yml	Adds a blank line in the permissions block (whitespace-only change).

Copilot's findings

Tip

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

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