Rodin rodin
  • Joined on 2026-04-23
rodin pushed to main at rodin/review-bot 2026-05-14 23:08:59 +00:00
fccfdd2ff7 [dev-loop] Add tests for fetchFileContext, fetchPatterns, and persona edge cases
rodin pushed to main at rodin/review-bot 2026-05-14 22:54:01 +00:00
e3fb19fa1b chore: dev-loop cleanup — go fmt and go mod tidy at 2026-05-14 22:53 UTC
rodin closed pull request rodin/review-bot#132 2026-05-14 22:53:55 +00:00
feat: implement GitHub API methods for PR review
rodin closed pull request rodin/review-bot#136 2026-05-14 22:53:46 +00:00
test: fix cleanEnv VCS_ leak, add githubAPIURL tests, add GitHub integration test
rodin closed issue rodin/review-bot#135 2026-05-14 22:53:44 +00:00
PR #131: cleanEnv() does not strip VCS_TYPE or VCS_URL env vars
rodin pushed to main at rodin/review-bot 2026-05-14 22:53:44 +00:00
a1bbab406d test: fix cleanEnv VCS_ leak, add githubAPIURL tests, add GitHub integration test (#136)
4d48917e36 refactor(test): remove misleading t.Setenv in GitHub integration test
bd516cd044 nit: remove extra blank line before TestEnvOrDefault
1f67954da7 test(#133,#134,#135): fix cleanEnv, add githubAPIURL tests, add GitHub integration test
Compare 4 commits »
rodin closed issue rodin/review-bot#133 2026-05-14 22:53:44 +00:00
PR #131: integration test not extended for GitHub (issue #130 acceptance criterion)
rodin closed issue rodin/review-bot#134 2026-05-14 22:53:44 +00:00
PR #131: githubAPIURL() function has no unit tests
rodin commented on pull request rodin/review-bot#136 2026-05-14 22:34:14 +00:00
test: fix cleanEnv VCS_ leak, add githubAPIURL tests, add GitHub integration test

Self-review against 4d48917e3601461d819ee3f0430cc4ea9dd8e139

Assessment: Clean

No issues found — ready for human review.

Review notes:

  • #135 fix: VCS_ prefix strips both VCS_TYPE and…
rodin pushed to review-bot-issue-133 at rodin/review-bot 2026-05-14 22:33:52 +00:00
4d48917e36 refactor(test): remove misleading t.Setenv in GitHub integration test
rodin created pull request rodin/review-bot#136 2026-05-14 22:33:00 +00:00
test: fix cleanEnv VCS_ leak, add githubAPIURL tests, add GitHub integration test
rodin pushed to review-bot-issue-133 at rodin/review-bot 2026-05-14 22:32:45 +00:00
bd516cd044 nit: remove extra blank line before TestEnvOrDefault
rodin pushed to review-bot-issue-133 at rodin/review-bot 2026-05-14 22:32:18 +00:00
1f67954da7 test(#133,#134,#135): fix cleanEnv, add githubAPIURL tests, add GitHub integration test
rodin created branch review-bot-issue-133 in rodin/review-bot 2026-05-14 22:32:18 +00:00
rodin commented on pull request rodin/review-bot#132 2026-05-14 22:22:52 +00:00
feat: implement GitHub API methods for PR review

Rebase complete, conflict resolved.

This branch had a merge conflict because PR #131 was merged first with an equivalent (but architecturally different) implementation of the same feature (issue…

rodin pushed to review-bot-issue-130 at rodin/review-bot 2026-05-14 22:22:34 +00:00
d396599d05 chore: dev-loop health check — status at 2026-05-15 02:10 UTC
9f3f32174b chore: update dev-loop status after issue-130 merge
c53a07b230 feat: implement GitHub API methods and VCS routing (issue #130) (#131)
Compare 3 commits »
rodin opened issue rodin/review-bot#135 2026-05-14 22:16:23 +00:00
PR #131: cleanEnv() does not strip VCS_TYPE or VCS_URL env vars
rodin opened issue rodin/review-bot#134 2026-05-14 22:16:13 +00:00
PR #131: githubAPIURL() function has no unit tests
rodin opened issue rodin/review-bot#133 2026-05-14 22:16:03 +00:00
PR #131: integration test not extended for GitHub (issue #130 acceptance criterion)
rodin pushed to main at rodin/review-bot 2026-05-14 22:11:15 +00:00
d396599d05 chore: dev-loop health check — status at 2026-05-15 02:10 UTC