[mwbrooks]

Changelog

N/A - internal cleanup with no user-facing changes.

Summary

This pull request replaces a reverse index loop with slices.Backward in internal/manifest/validate.go for maintainers, from the Go modernize analyzer. Same behaviour, in the form current Go recommends.

• Uses range slices.Backward(slackError.Details) in HandleConnectorApprovalRequired instead of counting down from len(slackError.Details) - 1, and still deletes in place

Testing

1. Run go run golang.org/x/tools/go/analysis/passes/modernize/cmd/modernize@latest ./... and confirm internal/manifest/validate.go no longer reports slices.Backward
2. Run go build ./... && go test ./...

Requirements

• I've read and understood the Contributing Guidelines and have done my best effort to follow them.
• I've read and agree to the Code of Conduct.

[zimeg]

🧹 @mwbrooks LGTM! Clean swap to slices.Backward, same reverse-delete behavior preserved.

[mwbrooks]

Thanks @zimeg for the quick review! 🙇