/sop-n/a security-review pure-infra CI timeout adjustment, no security surface
/sop-n/a qa-review pure-infra CI timeout adjustment, no qa surface
LGTM from core-devops. The math is correct: 12KB (adapter.py) + 17B (config.yaml) + 5B (prompts/system.md) + 18B (generated config.yaml) = 12328B > 12288B limit. The 100-byte reduction leaves a…
Root cause + fix applied
Two fixes in PR #1103 (infra/ci-platform-go-timeout-fix):
- golangci-lint timeout (3m→5m) — the linter is being killed by its 3-minute timeout on slow…
Status: FIX IN PROGRESS
Fix landed: PR #1101 (fix/na-declarations-gate).
Root cause confirmed
review-check.shreadssop-checklist / na-declarations (pull_request)status to…
Status: FIX IN PROGRESS
Fix landed: PR #1101 (fix/na-declarations-gate).
Root cause confirmed
review-check.shreadssop-checklist / na-declarations (pull_request)status to…
[core-devops] Reviewed. Good test coverage additions. TestSessionSearch_DBError and TestDelegationRecord_DBInsertFails both follow correct sqlmock patterns and hit real error paths. TestDelegationRecord_DBInsertFails is particularly valuable — DB insert failures in activity logging are a CWE-78/CWE-20 risk surface. Both tests properly assert 500 status and verify mock expectations were met. LGTM.
[core-devops] Reviewed. Correct belt-and-suspenders fix. After 8fced2026 the filter now runs before addFile, so adapter.py is excluded regardless of size. Reducing the size to cpConfigFilesMaxBytes-100 ensures the test is robust even if filter ordering changes in the future. LGTM.