review-bot

Author	SHA1	Message	Date
Rodin	5132a2028d	fix: address review findings (body truncation, unused field, whitespace) CI / test (pull_request) Successful in 15s Details CI / review (anthropic--claude-4.6-sonnet, sonnet, SONNET_REVIEW_TOKEN) (pull_request) Successful in 32s Details CI / review (gpt-5, gpt, GPT_REVIEW_TOKEN) (pull_request) Successful in 1m34s Details CI / review (gpt-5, security, SECURITY_REVIEW.md, SECURITY_REVIEW_TOKEN) (pull_request) Successful in 2m28s Details Self-review fixes for PR #54: - Add truncateBody helper to limit error message body length (200 chars) Addresses security bot finding about potential information leakage in error messages that include upstream response bodies - Remove unused deployment.ID field from deployment struct Now stores just the URL string directly in the deployments map Addresses sonnet finding about unused struct field - Add doc comment noting deployment cache limitation Documents that cache is never invalidated, acceptable for CI use case - Fix trailing whitespace in action.yml aicore-resource-group default All existing tests pass.	2026-05-10 08:28:49 -07:00
Rodin	065b8417dd	ci: remove GPT models not deployed on AI Core gpt-4.1, gpt-4.1-mini, and gpt-5-mini are not deployed on SAP AI Core. Only gpt-5 and anthropic--claude-4.6-sonnet are available. Removed matrix entries for non-existent deployments to fix CI failures.	2026-05-10 08:28:49 -07:00
Rodin	9a40b1baf4	docs: update README with AI Core configuration	2026-05-10 08:28:49 -07:00
Rodin	f40584e1de	ci: switch to native AI Core provider - Remove HAI proxy dependency - Use aicore provider with secrets - Update model names to AI Core format (anthropic--claude-4.6-sonnet)	2026-05-10 08:28:49 -07:00
Rodin	b0ff007aa2	feat: integrate AI Core client with review-bot Adds aicore provider option that uses native SAP AI Core instead of OpenAI-compatible proxy. Configurable via: - AICORE_CLIENT_ID - AICORE_CLIENT_SECRET - AICORE_AUTH_URL - AICORE_API_URL - AICORE_RESOURCE_GROUP	2026-05-10 08:28:25 -07:00
Rodin	6379e303ba	feat: add SAP AI Core client for Anthropic models Implements native AI Core support with: - OAuth2 token refresh - Deployment discovery via /v2/lm/deployments - Anthropic Messages API via /invoke endpoint - Uses bedrock-2023-05-31 API version (AI Core uses Bedrock format) - Model field omitted from body (deployment URL specifies model) - Retry logic with exponential backoff Tested via integration tests against live AI Core endpoint.	2026-05-10 08:28:25 -07:00

Compare commits

Diff Content Not Available