Rodin rodin
0 Followers · 0 Following
  • Joined on 2026-04-23
Repositories
16
 Projects Packages Public Activity Starred Repositories
rodin pushed to main at rodin/model-research 2026-05-10 09:27:56 +00:00
b9036401c2 Finding #63: External System Assumptions Analysis
rodin closed issue rodin/review-bot#52 2026-05-10 08:58:42 +00:00
Investigate stale commit reviews on PR #687
rodin commented on issue rodin/review-bot#52 2026-05-10 08:58:37 +00:00
Investigate stale commit reviews on PR #687

Investigation Complete

Summary

This issue has been thoroughly investigated (see previous comments #12314 and #13592). The key findings:

  1. Root cause is NOT stale commits: Each…
rodin commented on pull request rodin/review-bot#54 2026-05-10 08:26:31 +00:00
feat: native SAP AI Core support

Self-Review Assessment

Reviewed findings from sonnet, gpt, and security bots. Here is my assessment:

Changes Made (commit 3276496)

  1. Security: Response body truncation — Added…
rodin pushed to feat/aicore-provider-v2 at rodin/review-bot 2026-05-10 08:26:18 +00:00
327649606b fix: address review findings (body truncation, unused field, whitespace)
rodin pushed to feat/aicore-provider-v2 at rodin/review-bot 2026-05-10 08:09:28 +00:00
6d4c33a7a4 ci: remove GPT models not deployed on AI Core
rodin closed pull request rodin/review-bot#50 2026-05-10 08:07:59 +00:00
feat: add native SAP AI Core support
rodin pushed to feat/aicore-provider-v2 at rodin/review-bot 2026-05-10 08:05:05 +00:00
5b3f6b1a44 docs: update README with AI Core configuration
rodin created pull request rodin/review-bot#54 2026-05-10 08:03:29 +00:00
feat: native SAP AI Core support
rodin pushed to feat/aicore-provider-v2 at rodin/review-bot 2026-05-10 08:03:22 +00:00
4931481a63 docs: update README with AI Core configuration
00c363a244 ci: switch to native AI Core provider
b1fdc35f70 feat: integrate AI Core client with review-bot
b62b52aab1 feat: add SAP AI Core client for Anthropic models
Compare 4 commits »
rodin created branch feat/aicore-provider-v2 in rodin/review-bot 2026-05-10 08:03:22 +00:00
rodin commented on pull request rodin/review-bot#53 2026-05-10 07:38:30 +00:00
fix: skip posting review when HEAD moves during evaluation

Reverted the CI changes (commit 6489870) that incorrectly removed model entries from the matrix.

The PR now contains only the stale-commit-check fix:

  • Re-fetch PR HEAD before posting review -…
rodin pushed to fix/stale-commit-check at rodin/review-bot 2026-05-10 07:38:25 +00:00
rodin commented on issue rodin/review-bot#52 2026-05-10 07:10:04 +00:00
Investigate stale commit reviews on PR #687

Follow-up Investigation: Stale Commit Reviews Analysis

Summary

After reviewing the evidence from PR #687 and PR #53, I can confirm and expand on the previous investigation findings.

###…

rodin pushed to feat/aicore-provider at rodin/review-bot 2026-05-10 06:48:28 +00:00
44c80c36cf fix: use bedrock-2023-05-31 for AI Core Anthropic version
rodin commented on pull request rodin/review-bot#53 2026-05-10 06:46:29 +00:00
fix: skip posting review when HEAD moves during evaluation

Self-review: PASS

Gates: ✅ make lint passes, make test passes

Coherence: ✅ Single-purpose change — adds stale commit detection before posting review. The PR description clearly…

rodin pushed to feat/aicore-provider at rodin/review-bot 2026-05-10 06:45:16 +00:00
f71f26fcff fix: remove anthropic_version from body - AI Core rejects it
rodin pushed to feat/aicore-provider at rodin/review-bot 2026-05-10 06:39:28 +00:00
8da8fca19d fix: add omitempty to model field so it's not sent when empty
rodin pushed to feat/aicore-provider at rodin/review-bot 2026-05-10 06:36:49 +00:00
b12df1a636 test: update Anthropic test to check anthropic_version instead of model
rodin pushed to main at rodin/model-research 2026-05-10 06:35:38 +00:00
ce4801e8a3 Add Finding #62: Boundary contract analysis (new analytical lens)