[agent/claude:opus] Fix CodeRabbit findings. Verify each against current code, f... #2

Closed

Virgil wants to merge 0 commits from agent/fix-coderabbit-findings--verify-each-aga into dev

Virgil commented

2026-03-17 16:24:46 +00:00

Member

Task

Fix CodeRabbit findings. Verify each against current code, fix ONLY if valid. Run tests. Commit: fix(coderabbit): address review findings

IMPORTANT: Do NOT change function signatures in dependency packages.

client.go:147-156 — Error returns include full upstream response body via coreerr.E. Truncate to prevent sensitive data leakage.
cmd/monitor/cmd_monitor.go — check inline findings.

Agent: claude:opus
Commits: 1
Branch: agent/fix-coderabbit-findings--verify-each-aga

Auto-created by core-agent dispatch system.
Co-Authored-By: Virgil virgil@lethean.io

## Task Fix CodeRabbit findings. Verify each against current code, fix ONLY if valid. Run tests. Commit: fix(coderabbit): address review findings IMPORTANT: Do NOT change function signatures in dependency packages. 1. client.go:147-156 — Error returns include full upstream response body via coreerr.E. Truncate to prevent sensitive data leakage. 2. cmd/monitor/cmd_monitor.go — check inline findings. **Agent:** claude:opus **Commits:** 1 **Branch:** `agent/fix-coderabbit-findings--verify-each-aga` --- Auto-created by core-agent dispatch system. Co-Authored-By: Virgil <virgil@lethean.io>

Virgil added 1 commit 2026-03-17 16:24:46 +00:00

fix(coderabbit): address review findings ec1a591b71

- Truncate response body in error messages to 256 bytes to prevent
  sensitive data leakage from upstream APIs (client.go)
- Replace unchecked strings.Split index with safe repoShortName helper
  to prevent potential panic (cmd/monitor)
- Fix test assertions to match coreerr.E error format after prior refactor

Co-Authored-By: Virgil <virgil@lethean.io>

Virgil commented

2026-03-17 16:24:54 +00:00

Author

Member