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

Copy Source

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

P0-demo

CERTifAI P0 demo build tickets (CS-*)

bug

critical

enhancement

epic

Tracking issue spanning multiple sub-issues

high

infrastructure

onboarding

Multi-target onboarding wizard redesign

performance

security

v0.3.0

No labels

Milestone

No items

No Milestone

Projects

Clear projects

No projects

Assignees

Clear assignees

Benjamin_Boenisch sharang (Sharang Parnerkar)

No Assignees

1 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: sharang/compliance-scanner-agent#47