review-bot

rodin/review-bot

Fork 0

Files

T

History

Rodin 02dfc12141

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

Details

CI / test (pull_request) Successful in 17s

Details

CI / review (anthropic--claude-4.6-sonnet, sonnet, SONNET_REVIEW_TOKEN) (pull_request) Successful in 30s

Details

CI / review (gpt-5, gpt, GPT_REVIEW_TOKEN) (pull_request) Failing after 49s

Details

CI / review (gpt-5, security, ., rodin/security-patterns, SECURITY_REVIEW.md, SECURITY_REVIEW_TOKEN) (pull_request) Failing after 1m11s

Details

fix(#143 ): skip local doc-map validation when --doc-map-trusted-ref is set

When --doc-map-trusted-ref is provided, the --doc-map value is used as a
VCS API path parameter, not a local filesystem path. The early call to
validateWorkspacePath (which requires the file to exist locally) blocked
the trusted-ref code path when the doc-map did not exist in the local
checkout — defeating the feature's purpose in sparse checkouts or when
the file is only on the default branch.

Fix: guard the early validation with `&& *docMapTrustedRef == ""`.

Also fixes:
- review/docmap.go: correct ParseDocMapConfigContent godoc example to
  match actual source format "owner/repo@ref:path"
- cmd/review-bot/main_test.go: add TestMainSubprocess_DocMapTrustedRefSkipsLocalValidation
  to prevent regression

2026-05-15 12:08:13 +00:00

personas

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

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

docmap_test.go

feat(#143 ): fetch doc-map config from trusted VCS ref

2026-05-15 12:08:13 +00:00

docmap.go

fix(#143 ): skip local doc-map validation when --doc-map-trusted-ref is set

2026-05-15 12:08:13 +00:00

formatter_test.go

feat(persona): add role-based review personas