fix: check Gitea API response status and fallback for PR reviews #47

Merged

sharang merged 5 commits from fix/gitea-pr-review-error-handling into main 2026-03-25 16:26:09 +00:00

Conversation 0 Commits 5 Files Changed 5 +148 -8

sharang commented 2026-03-18 22:42:11 +00:00

Owner

Copy Link

Summary

• Add HTTP response status checking to all Gitea tracker methods that were silently swallowing errors
• Add fallback in create_pr_review: if inline comments fail, retry as plain PR comment

Test plan

• Deploy and trigger a PR review, check logs for actual error details
• Verify fallback posts summary comment when inline comments fail

## Summary - Add HTTP response status checking to all Gitea tracker methods that were silently swallowing errors - Add fallback in create_pr_review: if inline comments fail, retry as plain PR comment ## Test plan - [ ] Deploy and trigger a PR review, check logs for actual error details - [ ] Verify fallback posts summary comment when inline comments fail

sharang added 1 commit 2026-03-18 22:42:12 +00:00

fix: check Gitea API response status and fallback for PR reviews ... c62e9fdcd4

The Gitea tracker was silently swallowing HTTP error responses — only
network failures were caught, so 4xx/5xx from Gitea went unnoticed and
the agent logged success even when reviews weren't posted.

- Add response status checking to create_pr_review, update_issue_status,
  and add_comment
- Add fallback in create_pr_review: if inline comments fail, retry as a
  plain PR comment so the summary still gets posted

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

sharang added 1 commit 2026-03-20 21:57:06 +00:00

feat: add health check for MCP servers on dashboard ... fad8bbbd65

Add /health endpoint to the MCP server and probe it from the dashboard
on page load to update server status (running/stopped) in real time.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

sharang added 1 commit 2026-03-25 15:33:52 +00:00

fix: avoid unwrap in MCP health check to pass clippy ... e5c14636a7

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

sharang added 1 commit 2026-03-25 15:48:10 +00:00

style: apply cargo fmt formatting ... f310a3e0a2

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

sharang added 1 commit 2026-03-25 16:08:21 +00:00

fix: update rustls-webpki to 0.103.10 (RUSTSEC-2026-0049) ... af4760baf5

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

sharang merged commit 745ad8a441 into main 2026-03-25 16:26:09 +00:00

sharang referenced this issue from a commit 2026-03-25 16:26:10 +00:00

fix: check Gitea API response status and fallback for PR reviews (#47)

Sign in to join this conversation.

Reviewers

No Reviewers

Labels

Clear labels

bug

critical

enhancement

high

infrastructure

performance

security

v0.3.0

No Label

Milestone

No items

No Milestone

Projects

Clear projects

No project

Assignees

Clear assignees

sharang

No Assignees

1 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: sharang/compliance-scanner-agent#47