Aaron Weiker aweiker
  • Joined on 2026-04-12
aweiker pushed to main at rodin/review-bot 2026-05-19 02:20:07 +00:00
0be601de2a Merge pull request 'Removing intermediate files' (#160) from ci/cleanup into main
e560781c87 Removing intermediate files
Compare 2 commits »
aweiker merged pull request rodin/review-bot#160 2026-05-19 02:20:06 +00:00
Removing intermediate files
aweiker created pull request rodin/review-bot#160 2026-05-19 02:17:42 +00:00
Removing intermediate files
aweiker merged pull request rodin/review-bot#158 2026-05-15 22:56:45 +00:00
fix(#157): add never-close constraint to spec, S9 invariant, and regression test
aweiker closed issue rodin/review-bot#157 2026-05-15 22:56:45 +00:00
Bug: dev-loop worker closed PR #156 autonomously
aweiker pushed to main at rodin/review-bot 2026-05-15 22:56:45 +00:00
7f31475330 Merge pull request 'fix(#157): add never-close constraint to spec, S9 invariant, and regression test' (#158) from issue-157 into main
ec6fdbff42 fix(#158): address bot feedback — correct S8/S10 description, fix §9 prose break
f883f39dbf fix(#158): address NIT feedback — clarify enforcement split, clean §9 prose
fb7d8d5e3b fix(#158): add S10 invariant to spec, fix enforcement wording in §8
6cefbb070e fix(#157): add S9 invariant and never-close constraint to dev-loop spec
Compare 5 commits »
aweiker pushed to main at rodin/review-bot 2026-05-15 21:28:43 +00:00
89596516d7 Merge pull request 'refactor(#154): extract baseSubprocessArgs helper in main_test.go subprocess tests' (#155) from issue-154 into main
282b6e0e86 nit(#154): add t.Fatal guard if baseSubprocessArgs flag not found
Compare 2 commits »
aweiker merged pull request rodin/review-bot#155 2026-05-15 21:28:42 +00:00
refactor(#154): extract baseSubprocessArgs helper in main_test.go subprocess tests
aweiker pushed to main at rodin/review-bot 2026-05-15 17:43:07 +00:00
d3b9027da3 Merge pull request 'feat(#141): validate-docmap subcommand' (#156) from issue-141 into main
bacb25e029 nit(#141): fix stale-docs error message phrasing
92efd1af2b fix(#141): strip leading './' from coverage-check paths
7adb296523 fix(#141): reject non-regular files in validateDocmapPath
Compare 4 commits »
aweiker merged pull request rodin/review-bot#156 2026-05-15 17:43:05 +00:00
feat(#141): validate-docmap subcommand
aweiker pushed to feature/github-support at rodin/review-bot 2026-05-14 01:49:58 +00:00
f79fb40bef Merge pull request 'fix(github): consolidate review.go and identity.go into reviews.go (#116)' (#119) from review-bot-issue-116 into feature/github-support
437e318240 nit: clarify truncation detection comment in ListReviews
2e2fcbabfc style: fix import ordering and restore nil-body comment
8e26c26f5f fix(github): add pagination tests and fix truncation warning logic
22b3ce8fef fix(github): consolidate review.go and identity.go into reviews.go (#116)
Compare 5 commits »
aweiker merged pull request rodin/review-bot#119 2026-05-14 01:49:57 +00:00
fix(github): consolidate review.go and identity.go into reviews.go (#116)
aweiker closed issue rodin/review-bot#116 2026-05-14 01:49:57 +00:00
Build broken: github/reviews.go duplicates github/review.go — redeclared types and methods
aweiker pushed to feature/github-support at rodin/review-bot 2026-05-14 01:49:35 +00:00
cb162c154b Merge pull request 'feat(vcs): add CommitID to ReviewRequest (#115)' (#118) from review-bot-issue-115 into feature/github-support
9a6298cc4f fix: address review NITs — readability, test dedup, consistent SHA var
be68e51898 fix(vcs): address self-review NITs - gofmt alignment and comment clarity
49db84fb82 fix(test): add missing blank line between test functions in adapter_test.go
08b5d4051b style: remove double blank lines in test files
Compare 6 commits »
aweiker merged pull request rodin/review-bot#118 2026-05-14 01:49:34 +00:00
feat(vcs): add CommitID to ReviewRequest (#115)
aweiker closed issue rodin/review-bot#115 2026-05-14 01:49:34 +00:00
PR #112: vcs.ReviewRequest.CommitID field missing — issue #107's interface-level fix not implemented