fix(trail): clarify finding-batch decode error message · Entire
fix(trail): clarify finding-batch decode error message
5360643→main·
dipree·1mo ago·1 file·+1 added/-1 removed
Addresses PR review: the decode failure here unmarshals the batch create response, not a single finding, so name it accordingly.
Co-Authored-By: Claude Opus 4.8 (1M context) noreply@anthropic.com
Sessions
893b69821084View transcript
[?
Align Trail Commands with Updated APIClaude Code·Opus 4.8[1m]·1 step](/content/gh/entireio/cli/session/224b2676-67b4-4e2d-afd4-d2186950b9d1#timeline-893b69821084/index.html)
Changes
1
cmd/entire/cli
Mtrail_review_cmd.go+1/-1
722 unmodified lines
723
724
725
726
726
727
728
729
722 unmodified lines
}
var batch api.TrailReviewCommentBatchResponse
if err := api.DecodeJSON(resp, &batch); err != nil {
return api.TrailReviewComment{}, fmt.Errorf("decode created finding: %w", err)
return api.TrailReviewComment{}, fmt.Errorf("decode finding batch response: %w", err)
}
if len(batch.Results) == 0 {
return api.TrailReviewComment{}, errors.New("create finding: server returned no results")
Mcmd/entire/cli/trail_review_cmd.go+1/-1