feat(review): salvage findings from a length-capped response instead of discarding the batch - #511

Merged

devops-thiago merged 3 commits into

release/v0.6.0from

feat/500-truncation-recovery

Aug 10, 2026

SonarQubeCloud / SonarCloud Code Analysis succeeded Aug 10, 2026 in 58s

Quality Gate passed

Issues
4 New issues
0 Accepted issues

Measures
0 Security Hotspots
100.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

Annotations

Check warning on line 180 in src/main/java/dev/thiagogonzaga/thrillhousebot/review/ai/TruncatedResponseSalvager.java

sonarqubecloud / SonarCloud Code Analysis

Remove or merge the dangling Javadoc comment(s).

See more on https://sonarcloud.io/project/issues?id=devops-thiago_ThrillhouseBot&issues=AZ_td3kQZu4Ig3zpE5B3&open=AZ_td3kQZu4Ig3zpE5B3&pullRequest=511

Check warning on line 295 in src/test/java/dev/thiagogonzaga/thrillhousebot/review/ai/TruncatedResponseSalvagerTest.java

sonarqubecloud / SonarCloud Code Analysis

Use a static import for "doThrow".

See more on https://sonarcloud.io/project/issues?id=devops-thiago_ThrillhouseBot&issues=AZ_td3p-Zu4Ig3zpE5B5&open=AZ_td3p-Zu4Ig3zpE5B5&pullRequest=511

Check warning on line 294 in src/test/java/dev/thiagogonzaga/thrillhousebot/review/ai/TruncatedResponseSalvagerTest.java

sonarqubecloud / SonarCloud Code Analysis

Use a static import for "mock".

See more on https://sonarcloud.io/project/issues?id=devops-thiago_ThrillhouseBot&issues=AZ_td3p-Zu4Ig3zpE5B4&open=AZ_td3p-Zu4Ig3zpE5B4&pullRequest=511

Check warning on line 298 in src/test/java/dev/thiagogonzaga/thrillhousebot/review/ai/TruncatedResponseSalvagerTest.java

sonarqubecloud / SonarCloud Code Analysis

Use a static import for "verify".

See more on https://sonarcloud.io/project/issues?id=devops-thiago_ThrillhouseBot&issues=AZ_td3p-Zu4Ig3zpE5B6&open=AZ_td3p-Zu4Ig3zpE5B6&pullRequest=511

View more details on SonarQubeCloud