Skip to content

Include branch name in review submission payload - #141

Merged
lovestaco merged 1 commit into
mainfrom
fix/submit-branch-name
Aug 8, 2026
Merged

Include branch name in review submission payload#141
lovestaco merged 1 commit into
mainfrom
fix/submit-branch-name

Conversation

@lovestaco

Copy link
Copy Markdown
Contributor

The diff-review submission never sent the current git branch name to LiveReview, so CLI-sourced reviews always showed "-" for Branch in the dashboard. Resolve HEAD's branch and include it in the request.

Summary

Describe what changed and why.

Linked Issue

Link the agreed issue this PR fulfills.

Validation

Describe the most specific test or validation you ran.

Checklist

  • This PR fulfills an agreed issue.
  • I kept the change narrow and scoped.
  • I ran the most specific relevant validation and described it above.
  • If I changed behavior, I called that out clearly in this PR.
  • If I touched UI, I attached a GIF or video walkthrough. This is required.
  • If I changed storage/ responsibilities, I updated storage/storage_status.md and ran make check-status-doc.
  • If I changed network/ responsibilities, I updated network/network_status.md and ran make check-status-doc.
  • If this change touches security, disclosure flow, credentials, storage, network behavior, or remote data handling, I reviewed SECURITY.md and updated related documentation if needed.

Notes For Reviewers

Anything specific you want reviewers to focus on.

The diff-review submission never sent the current git branch name to
LiveReview, so CLI-sourced reviews always showed "-" for Branch in
the dashboard. Resolve HEAD's branch and include it in the request.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@lovestaco
lovestaco merged commit 2657572 into main Aug 8, 2026
16 checks passed
@lovestaco lovestaco self-assigned this Aug 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant