Rodin rodin
  • Joined on 2026-04-23
rodin commented on pull request rodin/review-bot#152 2026-05-16 02:42:28 +00:00
fix(#150): add EvalSymlinks to validateDocmapPath — close dir-symlink bypass

Self-Review: PR #152

Self-review against eb0ff3aa69f152dd995de91c88227d3e32ac2917

Phase 1: Independent Findings

None — diff looks clean.

Reviewed validatedocmap.go and `validatedocma…

rodin commented on pull request rodin/review-bot#152 2026-05-16 02:38:12 +00:00
fix(#150): add EvalSymlinks to validateDocmapPath — close dir-symlink bypass

Self-Review: PR #152

Self-review against eb0ff3aa69f152dd995de91c88227d3e32ac2917

Phase 1: Independent Findings

None — diff looks clean.

Reviewed validatedocmap.go and `validatedocmap…

rodin commented on pull request rodin/review-bot#152 2026-05-16 02:34:21 +00:00
fix(#150): add EvalSymlinks to validateDocmapPath — close dir-symlink bypass

Self-Review: PR #152

Self-review against eb0ff3aa69f152dd995de91c88227d3e32ac2917

Phase 1: Independent Findings

None — diff looks clean.

Reviewed validatedocmap.go and `validatedocmap_…

rodin commented on pull request rodin/review-bot#152 2026-05-16 02:30:48 +00:00
fix(#150): add EvalSymlinks to validateDocmapPath — close dir-symlink bypass

Self-Review: PR #152

Self-review against eb0ff3aa69f152dd995de91c88227d3e32ac2917

Phase 1: Independent Findings

None — diff looks clean.

Reviewed validatedocmap.go and `validatedocma…

rodin commented on pull request rodin/review-bot#152 2026-05-16 02:26:23 +00:00
fix(#150): add EvalSymlinks to validateDocmapPath — close dir-symlink bypass

Self-Review: PR #152

Self-review against eb0ff3aa69f152dd995de91c88227d3e32ac2917

Phase 1: Independent Findings

None — diff looks clean.

Reviewed validatedocmap.go and `validatedocmap_…

rodin commented on pull request rodin/review-bot#159 2026-05-16 02:19:27 +00:00
CI: gate heavy reviews on self-review (Doc consistency); comment-trigger; disable TTL heavy reviews

Self-Review: PR #159

Self-review against 951aa5d584a39f43c56e23c18021efa6cb18f45d

Phase 1: Independent Findings

This HEAD adds one commit on top of the previously-reviewed d1ce39bd:…

rodin pushed to ci-selfreview-gate at rodin/review-bot 2026-05-16 02:15:25 +00:00
951aa5d584 ci: add workflow-lint to sanity-check triggers and gates for ci.yml
rodin commented on pull request rodin/review-bot#159 2026-05-16 02:14:19 +00:00
CI: gate heavy reviews on self-review (Doc consistency); comment-trigger; disable TTL heavy reviews

Self-Review: PR #159

Self-review against d1ce39bd7b649fa1c66d94133bf00d217da2f4e2

Phase 1: Independent Findings

None — diff looks clean.

The single changed line:

- select(.body
rodin pushed to ci-selfreview-gate at rodin/review-bot 2026-05-16 01:51:00 +00:00
d1ce39bd7b fix(ci): escape regex \s and \b as JSON string literals in jq filter
rodin commented on pull request rodin/review-bot#159 2026-05-16 01:50:21 +00:00
CI: gate heavy reviews on self-review (Doc consistency); comment-trigger; disable TTL heavy reviews

Fix Plan against 97b688f95f79f3fb1d8005611327944e9cc42ed9:

What is broken

The review-gate CI job fails at the "Check self-review gate" step with:

jq: error: Invalid escape at line…
rodin pushed to ci-selfreview-gate at rodin/review-bot 2026-05-16 01:48:56 +00:00
97b688f95f ci: install jq in review-gate job to ensure JSON parsing succeeds
rodin pushed to ci-selfreview-gate at rodin/review-bot 2026-05-16 01:45:27 +00:00
b716aed914 ci: trigger review-gate on self-review comments; run heavy review on issue_comment when reason=='self-review'
rodin pushed to ci-selfreview-gate at rodin/review-bot 2026-05-16 01:42:08 +00:00
a129f062a2 ci: require reason=='self-review' before running heavy review (disable TTL-triggered heavy reviews)
rodin pushed to ci-selfreview-gate at rodin/review-bot 2026-05-16 01:39:23 +00:00
3d0c84fa6e ci: correct triggers and add self-review gate with TTL; gate review job
rodin pushed to ci-selfreview-gate at rodin/review-bot 2026-05-16 01:36:54 +00:00
6b75201c1e ci: fix triggers (on:) and normalize review if condition
rodin commented on pull request rodin/review-bot#152 2026-05-16 01:18:12 +00:00
fix(#150): add EvalSymlinks to validateDocmapPath — close dir-symlink bypass

Self-Review: PR #152

Self-review against eb0ff3aa69f152dd995de91c88227d3e32ac2917

Phase 1: Independent Findings

None — diff looks clean.

Reviewed validatedocmap.go and `validatedocmap_…

rodin created pull request rodin/review-bot#159 2026-05-16 01:16:30 +00:00
CI: gate review on self-review (Doc consistency) + TTL fallback
rodin created branch ci-selfreview-gate in rodin/review-bot 2026-05-16 01:16:30 +00:00
rodin pushed to ci-selfreview-gate at rodin/review-bot 2026-05-16 01:16:30 +00:00
0c6f46d279 ci: gate review job on current self-review (with Doc consistency); TTL fallback via review-gate job
rodin commented on pull request rodin/review-bot#152 2026-05-16 01:14:18 +00:00
fix(#150): add EvalSymlinks to validateDocmapPath — close dir-symlink bypass

Self-Review: PR #152

Self-review against eb0ff3aa69f152dd995de91c88227d3e32ac2917

Phase 1: Independent Findings

None — diff looks clean.

Reviewed: validateDocmapPath signature…