From 25cb55449e5ee975d0eca9c642d238d6b42a41bc Mon Sep 17 00:00:00 2001 From: claw Date: Wed, 13 May 2026 13:49:41 -0700 Subject: [PATCH] fix(nit): align CommitID field in vcs/types.go and document no-op in github/review.go --- github/review.go | 1 + vcs/types.go | 3 ++- 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/github/review.go b/github/review.go index 97fa4c1..1d043e5 100644 --- a/github/review.go +++ b/github/review.go @@ -108,6 +108,7 @@ func (c *Client) PostReview(ctx context.Context, owner, repo string, number int, } else if payload.CommitID != comment.CommitID { return nil, ErrConflictingCommitIDs } + // else: matching SHA is a no-op by design } payload.Comments = append(payload.Comments, reviewCommentEntry{ Path: comment.Path, diff --git a/vcs/types.go b/vcs/types.go index dff3e8c..f77df2c 100644 --- a/vcs/types.go +++ b/vcs/types.go @@ -97,6 +97,7 @@ type ReviewRequest struct { // CommitID anchors the review to a specific commit SHA. // If empty, the platform defaults to the current PR head. // Adapters use this as the primary commit anchor for the review submission. - CommitID string `json:"commit_id,omitempty"` + CommitID string `json:"commit_id,omitempty"` + Comments []ReviewComment `json:"comments,omitempty"` }