Regression from audit #109: rows.Err() checks were removed from four
functions between commits 3a30b073 and b25b4fb6. Without these checks,
a mid-stream query error (e.g. connection loss during row iteration)
is silently ignored and partial results are returned as success.
Added rows.Err() checks after every for rows.Next() loop:
- List: workspace secrets loop + global secrets loop
- Values: global secrets loop + workspace secrets loop
- ListGlobal: single loop
- restartAllAffectedByGlobalKey: affected workspaces loop
Each check logs the iteration error and continues (non-fatal, matching
the existing log.Printf pattern used elsewhere in the file).
Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
fullstack-engineer