Skip to content

ci(review): bundled PR review + incremental cache wiring #4320

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
yiliang114 wants to merge 48 commits into
base: main
Choose a base branch
from
Open

ci(review): bundled PR review + incremental cache wiring #4320

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
48 commits
Select commit Hold shift + click to select a range
ac27ac2
ci(review): use bundled PR review workflow
yiliang114 May 11, 2026
65b4111
fix(ci): correct misleading comment about quote stripping in review w…
yiliang114 May 12, 2026
7275f86
fix(ci): satisfy workflow yaml lint
yiliang114 May 12, 2026
c02ac19
ci(review): add manual dry-run mode
yiliang114 May 12, 2026
03e0dbc
fix(ci): harden bundled PR review against prompt injection
yiliang114 May 12, 2026
6818309
Merge remote-tracking branch 'origin/main' into codex/bundled-pr-revi…
yiliang114 May 12, 2026
79fd36c
style(ci): format review workflow action pins
yiliang114 May 12, 2026
9b86ab8
fix(ci): scope OpenAI secrets to the review step + document --ci
yiliang114 May 12, 2026
4cd8549
docs(review): align --ci command allowlist with what Step 9 actually …
yiliang114 May 12, 2026
596a780
ci(review): use review-specific OpenAI credentials
yiliang114 May 12, 2026
8d5e041
ci(review): add smoke mode for PR diff checks
yiliang114 May 12, 2026
0e1eb30
fix(ci): correct smoke review summary output
yiliang114 May 12, 2026
0919a11
fix(ci): tighten @qwen /review grep boundary and fix sed multi-line e…
yiliang114 May 13, 2026
dbb875b
refactor(ci): use QwenLM/qwen-code-action instead of building CLI fro…
yiliang114 May 13, 2026
0c9ccb9
fix(ci): align prompt indentation in review_prompt construction
yiliang114 May 13, 2026
bcb2127
refactor(ci): add checkout step, move review rules to .github/, rever…
yiliang114 May 14, 2026
596edce
fix(ci): strip leading blanks from additional_instructions and clarif…
yiliang114 May 14, 2026
11a491e
fix(ci): add reopened/ready_for_review triggers and edited comment types
yiliang114 May 14, 2026
f157309
fix(ci): align sed extraction boundary with grep for @qwen /review
yiliang114 May 14, 2026
7e83c5e
fix(ci): guard review-rules copy against missing file before merge
yiliang114 May 14, 2026
39a314a
refactor(ci): move review-rules to .qwen/, harden with cross-repo che…
yiliang114 May 14, 2026
db9bc44
refactor(ci): move review-rules to .qwen/, harden with cross-repo che…
yiliang114 May 14, 2026
b3c2361
docs(review): make product-direction gate blocking by default
yiliang114 May 14, 2026
204837c
fix(ci): prevent review workflow from approving PRs
yiliang114 May 14, 2026
f00f2af
docs(review): document code review automation design
yiliang114 May 18, 2026
2e77544
ci(review): add synchronize trigger and persist incremental review cache
yiliang114 May 18, 2026
fa3dbd9
fix(ci): include base SHA in review-cache key to prevent cross-base c…
yiliang114 May 18, 2026
5217988
fix(ci): use merge-base SHA in cache key and gate save on comment pub…
yiliang114 May 18, 2026
e542b0d
fix(ci): skip merge-base lookup for cross-repository PRs
yiliang114 May 18, 2026
e9ac834
fix(ci): address review workflow gate issues
yiliang114 May 18, 2026
fbcedd9
ci(review): drop cross-repository fork gate; dispatch checks out disp…
yiliang114 May 19, 2026
384c11d
docs(review): scope design docs to Phase 1-3 only
yiliang114 May 19, 2026
fd68575
docs(review): separate review criteria from workflow process in revie…
yiliang114 May 19, 2026
43e1ee6
docs(review): separate Phase 1-2 vs Phase 7 attribution in compare su…
yiliang114 May 19, 2026
6994bf2
docs(review): annotate the author-association gate as an intentional …
yiliang114 May 19, 2026
995a5bc
ci(review): add model attribution footer to the review summary comment
yiliang114 May 19, 2026
9a18c37
ci(review): raise job timeout 30->60min; deep review exceeded 30min i…
yiliang114 May 19, 2026
a3f5283
ci(review): align review-action invocation with main's proven-working…
yiliang114 May 19, 2026
847a61a
ci(review): stop restricting coreTools; force trust+yolo so bundled /…
yiliang114 May 19, 2026
8526038
ci(review): set up Node 22 before the review action (qwen-code requir…
yiliang114 May 19, 2026
d121125
ci(review): call qwen directly with streamed tee instead of qwen-code…
yiliang114 May 19, 2026
d8151dd
ci(review): npm-install qwen directly with cache; dry-run streams ful…
yiliang114 May 19, 2026
6fd4d6a
ci(review): stream qwen via --output-format stream-json (tee on text …
yiliang114 May 19, 2026
f2d3e24
ci(review): CI-lightweight dry-run prompt (skip Agent 7 build/test + …
yiliang114 May 19, 2026
adb88de
docs(review): require a Validation Evidence verdict; CI-lightweight f…
yiliang114 May 19, 2026
63cf5a6
ci(review): drop pull_request_review_comment / pull_request_review tr…
yiliang114 May 19, 2026
97f025c
ci(review): self-review cleanup — dedupe Validation Evidence, drop un…
yiliang114 May 20, 2026
bf7030c
ci(review): fix comment-posting permission + broaden fallback gate
yiliang114 May 20, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Loading
Loading