Skip to content

p-1 reliability branch merge - #12

Merged
PG1204 merged 4 commits into
p1-reliabilityfrom
main
Jul 7, 2026
Merged

p-1 reliability branch merge#12
PG1204 merged 4 commits into
p1-reliabilityfrom
main

Conversation

@PG1204

@PG1204 PG1204 commented Jul 7, 2026

Copy link
Copy Markdown
Owner

No description provided.

PG1204 and others added 4 commits June 15, 2026 12:24
…rgetBranch instead of baseBranch which was required for the test runs)
- Add GlobalExceptionHandler (@RestControllerAdvice): a malformed or empty
  request body now returns HTTP 200 with a PrAnalysisResponse envelope
  (overallRisk=PARSING_ERROR) instead of Spring's default 400, honoring the
  documented always-200 graceful-degradation contract.
- Extract PayloadTooLargeException to a public top-level class; the advice maps
  an over-limit chunked body to 413, consistent with the P0 size filter (this
  case previously leaked through as a 400).
- Tests: GlobalExceptionHandlerTest (unit, both branches) and a @WebMvcTest
  covering malformed/empty -> 200 envelope and valid -> service delegation.

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

github-actions Bot commented Jul 7, 2026

Copy link
Copy Markdown

💥 Blast Radius Analysis

Overall Risk: ❓ UNKNOWN

🔍 Impact Areas

  • Analysis service was unreachable — this check is advisory and does not block the PR.

🧪 Suggested Tests

  • None suggested

Analyzed by Blast Radius (Spring Boot + Groq LLM) • 2026-07-07

@PG1204
PG1204 merged commit bb87dca into p1-reliability Jul 7, 2026
2 checks passed
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