CI: gate heavy reviews on self-review (Doc consistency); comment-trigger; disable TTL heavy reviews #159
Closed
rodin
wants to merge 8 commits from
ci-selfreview-gate into main
pull from: ci-selfreview-gate
merge into: :main
:main
:ci/cleanup
:ci-selfreview-gate
:issue-150
:issue-157
:issue-141
:issue-154
:review-bot-dev-loop
:issue-143
:issue-146
:pr-153
:review-bot-issue-130-work
:issue-148
:issue-139
:issue-137
:review-bot-fixes
:review-bot-issue-133
:review-bot-issue-130
:issue-130
:github-support
:issue-123-work
:issue-123
:review-bot-issue-120
:fix/125-readme-cli-example
:issue-125
:issue-124
:issue-120
:feature/github-support
:review-bot-issue-116
:review-bot-issue-115
:review-bot-issue-114
:review-bot-issue-96
:review-bot-issue-107
:review-bot-issue-82
:review-bot-issue-95
:review-bot-issue-92
:review-bot-issue-94
:review-bot-issue-81
:review-bot-issue-91
:review-bot-issue-97
:issue-80-c-file-reader
:issue-80-b-pr-reader
:issue-80-a-client
:review-bot-issue-80
:review-bot-issue-87
:review-bot-issue-79
:review-bot-issue-84
:review-bot-issue-78
:issue-73
:issue-70
:issue-68
:issue-66
:issue-64
:issue-60-remote-personas
:issue-60
:issue-57
:allow-deps
:feat/aicore-provider-v2
:issue-51
:ci/pr-ready-gate
:fix/stale-commit-check
:feat/aicore-provider
:fix/response-body-truncation
:fix/json-repair
:fix/sonnet-reviewer
:fix/consistent-path-escape
:feat/inline-review-comments
:feat/6-update-existing-review
:fix/19-context-overflow
:feat/18-anthropic-api
:fix/url-escaping-and-shadow
:fix/quick-wins
:fix/context-and-encapsulation
:docs/code-review-report
:ci/release-workflow
8
Commits
| Author | SHA1 | Message | Date | |
|---|---|---|---|---|
|
|
951aa5d584 |
ci: add workflow-lint to sanity-check triggers and gates for ci.yml
PR Ready Gate / clear-labels (pull_request) Successful in 1s
CI / review-gate (pull_request) Successful in 7s
CI / test (pull_request) Successful in 32s
CI / review (anthropic--claude-4.6-sonnet, sonnet, SONNET_REVIEW_TOKEN) (pull_request) Successful in 39s
CI / review (gpt-5, security, ., rodin/security-patterns, SECURITY_REVIEW.md, SECURITY_REVIEW_TOKEN) (pull_request) Successful in 1m22s
CI / review (gpt-5, gpt, GPT_REVIEW_TOKEN) (pull_request) Successful in 2m5s
|
||
|
|
d1ce39bd7b |
fix(ci): escape regex \s and \b as JSON string literals in jq filter
PR Ready Gate / clear-labels (pull_request) Successful in 2s
CI / review-gate (pull_request) Successful in 7s
CI / review (anthropic--claude-4.6-sonnet, sonnet, SONNET_REVIEW_TOKEN) (pull_request) Has been skipped
CI / review (gpt-5, gpt, GPT_REVIEW_TOKEN) (pull_request) Has been skipped
CI / review (gpt-5, security, ., rodin/security-patterns, SECURITY_REVIEW.md, SECURITY_REVIEW_TOKEN) (pull_request) Has been skipped
CI / test (pull_request) Successful in 22s
jq parses the test() argument as a JSON string, so \s and \b must be double-escaped (\\s, \\b) to produce literal \s and \b after JSON string parsing. Single backslash forms are invalid JSON escapes and cause a compile error. |
||
|
|
97b688f95f |
ci: install jq in review-gate job to ensure JSON parsing succeeds
PR Ready Gate / clear-labels (pull_request) Successful in 2s
CI / review-gate (pull_request) Failing after 6s
CI / review (anthropic--claude-4.6-sonnet, sonnet, SONNET_REVIEW_TOKEN) (pull_request) Has been skipped
CI / review (gpt-5, gpt, GPT_REVIEW_TOKEN) (pull_request) Has been skipped
CI / review (gpt-5, security, ., rodin/security-patterns, SECURITY_REVIEW.md, SECURITY_REVIEW_TOKEN) (pull_request) Has been skipped
CI / test (pull_request) Successful in 32s
|
||
|
|
b716aed914 |
ci: trigger review-gate on self-review comments; run heavy review on issue_comment when reason=='self-review'
CI / review-gate (pull_request) Failing after 1s
PR Ready Gate / clear-labels (pull_request) Successful in 1s
CI / review (anthropic--claude-4.6-sonnet, sonnet, SONNET_REVIEW_TOKEN) (pull_request) Has been skipped
CI / review (gpt-5, gpt, GPT_REVIEW_TOKEN) (pull_request) Has been skipped
CI / review (gpt-5, security, ., rodin/security-patterns, SECURITY_REVIEW.md, SECURITY_REVIEW_TOKEN) (pull_request) Has been skipped
CI / test (pull_request) Successful in 28s
|
||
|
|
a129f062a2 |
ci: require reason=='self-review' before running heavy review (disable TTL-triggered heavy reviews)
CI / review-gate (pull_request) Successful in 1s
PR Ready Gate / clear-labels (pull_request) Successful in 2s
CI / test (pull_request) Successful in 17s
CI / review (anthropic--claude-4.6-sonnet, sonnet, SONNET_REVIEW_TOKEN) (pull_request) Has been skipped
CI / review (gpt-5, gpt, GPT_REVIEW_TOKEN) (pull_request) Has been skipped
CI / review (gpt-5, security, ., rodin/security-patterns, SECURITY_REVIEW.md, SECURITY_REVIEW_TOKEN) (pull_request) Has been skipped
|
||
|
|
3d0c84fa6e |
ci: correct triggers and add self-review gate with TTL; gate review job
CI / test (pull_request) Successful in 17s
CI / review-gate (pull_request) Successful in 2s
PR Ready Gate / clear-labels (pull_request) Successful in 2s
CI / review (anthropic--claude-4.6-sonnet, sonnet, SONNET_REVIEW_TOKEN) (pull_request) Has been skipped
CI / review (gpt-5, gpt, GPT_REVIEW_TOKEN) (pull_request) Has been skipped
CI / review (gpt-5, security, ., rodin/security-patterns, SECURITY_REVIEW.md, SECURITY_REVIEW_TOKEN) (pull_request) Has been skipped
|
||
|
|
6b75201c1e |
ci: fix triggers (on:) and normalize review if condition
CI / review-gate (pull_request) Failing after 2s
PR Ready Gate / clear-labels (pull_request) Successful in 1s
CI / test (pull_request) Successful in 17s
CI / review (anthropic--claude-4.6-sonnet, sonnet, SONNET_REVIEW_TOKEN) (pull_request) Has been skipped
CI / review (gpt-5, gpt, GPT_REVIEW_TOKEN) (pull_request) Has been skipped
CI / review (gpt-5, security, ., rodin/security-patterns, SECURITY_REVIEW.md, SECURITY_REVIEW_TOKEN) (pull_request) Has been skipped
|
||
|
|
0c6f46d279 | ci: gate review job on current self-review (with Doc consistency); TTL fallback via review-gate job |