fix(handlers): add rows.Err() checks after rows.Next() loops #412

Merged
core-be merged 3 commits from fix/delegations-rows-err-check into main 2026-05-11 06:54:27 +00:00

3 Commits

Author SHA1 Message Date
150bf84b0b ci: re-trigger CI for fresh PR
All checks were successful
Secret scan / Scan diff for credential-shaped strings (pull_request) Successful in 20s
sop-tier-check / tier-check (pull_request) Bypass: infra#241 runner OOM; code review + core-qa APPROVE on record
audit-force-merge / audit (pull_request) Successful in 3s
Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
2026-05-11 06:42:24 +00:00
8d4a9a184f ci: re-trigger after runner stall
Some checks failed
sop-tier-check / tier-check (pull_request) Failing after 16s
Secret scan / Scan diff for credential-shaped strings (pull_request) Successful in 18s
audit-force-merge / audit (pull_request) Failing after 12m31s
Force a fresh sop-tier-check run to check if runners have recovered
from infra#241 OOM cascade.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
2026-05-11 06:24:01 +00:00
aa49dbc728 fix(handlers): add rows.Err() checks after rows.Next() loops
Some checks failed
Secret scan / Scan diff for credential-shaped strings (pull_request) Successful in 10s
sop-tier-check / tier-check (pull_request) Failing after 10s
Add deferred error checks following rows.Next() iteration in:
- ListDelegations (delegation.go): log on error, continue serving results
- org import reconcile orphan query (org.go): log + append to reconcileErrs

Fixes the rows.Err() gap identified in the delegated rows.Err() check PR
(#302, closed; replaced by this PR).  Two additional files already had
the check (activity.go, memories.go) — pattern applied consistently here.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
2026-05-11 06:15:42 +00:00