review-bot

rodin/review-bot

Fork 0

Files

T

History

Rodin 10cd6203d4

PR Ready Gate / clear-labels (pull_request) Successful in 2s

Details

CI / test (pull_request) Successful in 9m31s

Details

CI / review (anthropic--claude-4.6-sonnet, sonnet, SONNET_REVIEW_TOKEN) (pull_request) Successful in 9m54s

Details

CI / review (gpt-5, security, SECURITY_REVIEW.md, SECURITY_REVIEW_TOKEN) (pull_request) Successful in 10m40s

Details

CI / review (gpt-5, gpt, GPT_REVIEW_TOKEN) (pull_request) Successful in 11m27s

Details

fix: address remaining PR #58 review findings

1. Remove dead JSON fallback in LoadBuiltinPersona
   - The embed directive only includes *.yaml files
   - JSON fallback code could never succeed
   - Simplified function to only try YAML

2. JSON parsing now rejects unknown fields
   - Switched from json.Unmarshal to json.Decoder
   - DisallowUnknownFields() matches YAML's KnownFields(true)
   - Added test coverage for JSON unknown field rejection

3. Documented symlink support in LoadPersona
   - os.Stat follows symlinks, so symlinks to regular files work
   - Added doc comment explaining the behavior
   - Added test for symlink support

2026-05-10 17:53:42 -07:00

personas

feat: add YAML support for persona files (#57 )

2026-05-10 14:16:41 -07:00

formatter_test.go

feat(persona): add role-based review personas

2026-05-10 09:14:48 -07:00

formatter.go

fix: address review feedback on persona feature

2026-05-10 10:01:34 -07:00

parser_test.go

fix: repair unescaped quotes in LLM JSON responses (#45 )