Fix mapsloop lint violation in safe output app permission merge

[Copilot]

The lint-go job was failing on pkg/workflow/safe_outputs_app_config.go due to a modernize/mapsloop finding. The failure came from manually copying entries from a temporary permission map into permissionFields.

• What changed

  • Replaced the ad hoc for k, v := range ... merge with maps.Copy(...) when applying github-app.permissions overrides.
  • Added the standard library maps import required for the merge.

• Why this area

  • The affected code builds permission-* inputs for GitHub App token minting.
  • This keeps the existing override behavior intact while aligning the implementation with the lint rule enforced in CI.

• Code shape

  tempPerms := NewPermissionsFromMap(map[PermissionScope]PermissionLevel{
  	scope: PermissionLevel(level),
  })
  maps.Copy(permissionFields, convertPermissionsToAppTokenFields(tempPerms))

[Copilot]

Pull request overview

Fixes a modernize/mapsloop lint failure by replacing a manual map merge loop with the standard library maps.Copy when applying github-app.permissions overrides during GitHub App token mint-step generation.

Changes:

• Added the maps stdlib import to pkg/workflow/safe_outputs_app_config.go.
• Replaced the manual for k, v := range ... { ... } map merge with maps.Copy(...) for permission field overrides.

Show a summary per file

File	Description
pkg/workflow/safe_outputs_app_config.go	Uses maps.Copy to merge app permission override fields into permissionFields, resolving the lint finding.

Copilot's findings

Tip

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

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