i?3 dZddlmZddlZddlZddlZddlZddlZddlZddl Z ddl Z ddl m Z m Z ddl mZeejj!deeej'j(j(Zedz dz Zedz d z d z Zgd Zgd Zd ZdZddZdZddZdddZddZ d dZ!dd!dZ"d"d#dZ#d$dZ$d%dZ%d&dZ&e'dk(re jPe&yy)'ugemini_review_gate.py — Gemini Review를 GitHub status check로 강제하는 게이트. CI에서 호출: 1) status=pending 으로 check run 생성/갱신 (리뷰 진행 중) 2) Gemini API 호출 → blocking 패턴 detect 3) status=success/failure 로 check run 갱신 4) 결과를 stdout에 JSON으로 출력 + memory/logs/gemini-calls.jsonl append 블로킹 패턴 (대소문자 무시 + 부분 매칭): en: blocking, critical issue, must fix, request changes ko: 차단, 필수 수정, 심각, blocker 차단 결정: - Gemini 응답에 위 패턴 1개 이상 → conclusion="failure" - 0건 → conclusion="success" - Gemini API 키 없거나 호출 실패 → conclusion="neutral" (CI 차단 X) Rate limit / debounce: - 같은 SHA 중복 호출 차단 (cache 파일) - PR별 debounce 120초 CLI: python3 scripts/gemini_review_gate.py --pr-number 123 --commit-sha abc... [--status pending|success|failure] [--repo OWNER/REPO] [--diff-file path/to/diff] [--mode call|publish-only] ) annotationsN)datetimetimezone)Path WORKSPACEmemorycachelogszgemini-calls.jsonl)blockingzcritical issuezmust fixzrequest changesblocker)u차단u 필수 수정u심각r zgemini-review-gatexcdtjtjj SN)rnowrutc isoformatK/home/jay/workspace/.worktrees/task-2464-dev6/scripts/gemini_review_gate.py_now_isor6s << % / / 11r) z no blockingz no blockerz not blockingz no criticalz no must fixzno request changesu 차단 없음u필수 수정 없음u심각한 문제 없음c|sgS|j}|}tD]"}|j|jd}$g}tD]O}t j dt j |jd|s?|j|QtD]&}|j|vs|j|(|S)u{Return matched blocking patterns (case-insensitive, word-aware). - Strips out common negation phrases ("no blocking issues", "차단 없음" 등) so that `LGTM — no blocking issues` 가 false positive를 일으키지 않는다. - English patterns use word-boundary regex. - Korean patterns use plain substring (한글 단어 경계는 \b가 안 통함).  z\b) lower_NEGATION_PHRASESreplaceBLOCKING_PATTERNS_ENresearchescapeappendBLOCKING_PATTERNS_KO)textloweredcleanednegmatchespats rdetect_blockingr(Gs  jjlGG 4//#))+s34G#  99299SYY[12"5w ? NN3  $  99;' ! NN3   Nrcddg|z}tj|dd|d}|j|j|jfS)NghapiT)capture_outputr"inputtimeout) subprocessrun returncodestdoutstderr)args input_datacmdprocs r gh_api_jsonr9_s@ -$ C >>#dZY[ \D ??DKK 44rctjddtdtj|j j dz }|j rdd|fStd|dz }|j rV t|jjxsd }tj|z tkr dd td fSy #t$rd }Y8wxYw)z7Rate-limit + dedup gate. Returns (should_call, reason).Tparentsexist_okgemini-.jsonFzduplicate SHA cached at gemini-pr- .lasttime0gz debounce (zs))Tok) CACHE_DIRmkdirhashlibsha1encode hexdigestexistsfloat read_textstrip ValueErrortimeDEBOUNCE_SECONDS) pr_number commit_sha sha_cache debounce_pathlasts rshould_call_geminirVes OOD4O0ggll:3D3D3F&G&Q&Q&S%TTYZZI0 <<<*YKy AAM 00288:AcBD 99;  0 0J'7&8;; ;   D s +C C-,C-ctjjddtjdd5}|j t j |ddzdddy#1swYyxYw) NTr;autf-8encodingF ensure_ascii )LOG_PATHparentrEopenwritejsondumps)recordfs rwrite_call_logrgvs\ OO$6 sW -?  66=>???s *A,,A5c tjjd}|r tj|}d|jdd|jdt |dz|jdt |jdddz|jdd d Stjjd}|s d dd d d dd S d dl}d dl}|xs tjjdd}d|d|}d|dd} tjddd| igigijd} |jj|| ddi} tj} |jj| d5} tj| j!j#d}dddt%tj| z d z}d} d!d d"dd d}|jd#ixsi}d|t%|jd$d t%|jd%d |d S#t $r}d dd d d d |d cYd}~Sd}~wwxYw#t$r}d dd d d d|d cYd}~Sd}~wwxYw#1swYxYw#t&t(f$rtj}YwxYw#|j*j,$rN}d dd d t%tj| z d zd&|j.d'|j0d cYd}~Sd}~w|j*j2$rA}d dd d t%tj| z d zd(|j0d cYd}~Sd}~wt $r7}d dd d t%tj| z d zd)|d cYd}~Sd}~wwxYw)*zLightweight Gemini API call. Stub-friendly; mock by setting GEMINI_REVIEW_MOCK=. Returns: {"ok": bool, "text": str, "tokens_in": int, "tokens_out": int, "latency_ms": int, "error": str?} GEMINI_REVIEW_MOCKTr" tokens_in tokens_out latency_msr)rCr"rkrmrnFz mock parse: )rCr"rkrmrnerrorNGEMINI_API_KEYzGEMINI_API_KEY missingzurllib import: GEMINI_MODELzgemini-2.5-proz8https://generativelanguage.googleapis.com/v1beta/models/z:generateContent?key=u You are a strict code reviewer. Given the following diff, list any BLOCKING issues using the literal string 'blocking' or 'must fix' (English) or '차단'/'필수 수정' (Korean) in your response. If the diff is acceptable, respond with 'no blocking issues'. DIFF: i}contentspartsrYz Content-Typezapplication/json)dataheaders<)r/i candidatescontent usageMetadatapromptTokenCountcandidatesTokenCountzhttp z: zurl: zcall: )osenvirongetrcloadslen Exceptionurllib.request urllib.error ImportErrorrdrHrequestRequestrOurlopenreaddecodeintKeyError IndexErrorro HTTPErrorcodereasonURLError) diff_textmodel mock_jsonpayloadeapi_keyurllib use_modelurlpromptbodyreqt0resprtrnr"usages r call_geminir|sQ  34I |jj+G FB/$[[c)n6IJ%kk,GKKPReii(>BC$   E |!1\]jvwxvyhz{ { | {RaqXYfuvwuxdyzz{  ; ; *% $::d#D $ << ! !_RaqX[]a]f]f]hkm]mqu\uXvDIJKJPJPIQQSTUT\T\S]B^_ _ << URaqX[]a]f]f]hkm]mqu\uXvDIJKJRJRISBTU U ORaqX[]a]f]f]hkm]mqu\uXvDJKLJMBNO OOsBII40K 3J-K .J"AK  I1 I,&I1,I14 J= JJJJK "$K K K  K O %AL.(O .O  6NO  O ,O>O O ct||t|xs|dddd}|r||d<|r||dd<tj|}dd d |d d d g}t||\} } } | | | dS)zCreate or update gemini-review-gate check run via gh api. status: queued | in_progress | completed conclusion (only when completed): success | failure | neutral | cancelled | timed_out | action_required N)titlesummary)namehead_shastatusoutput conclusionrr"z-XPOSTzrepos/z /check-runsz--input-)r6)rcr3r4) CHECK_NAMErcrdr9) reposharrrdetailsrrr5routerrs rpublish_check_runrs&73KgdsmL G  * $+&! ::g D &F4& 4i EDt5LBSs 33rctjdddt|d|gddd}|jdk(r |jSy ) zEBest-effort PR diff fetch. Falls back to empty string if unavailable.r*prdiff--repoTr,)r-r"r/rrj)r0r1strr2r3)rrQr8s r fetch_pr_diffrsF >> tVS^Xt<$ D !{{ rc |jxs tjjdd}|jxsd}|j }t |||td}|jdk(r|js1ttjdditjy t|||jd vrd nd |jd vr |jnd|j xs |j }||d<ttj|d|ddk(rdSdS|s1ttjdditjy |jdk(r5t||d dd|d<ttj|dyt#||\}}|sJ|j$s>d|d<||d<t||d dd| ttj|dyd}|j&r& t)|j&j+d}|s|r t/||}t1|} t3| jd!d} | d"sd} d#| jdd$} n| rd%} d&| } nd'} d(} t||| jd)d| jd*d| jd+d| | | d"d, }t5|| d"rt6d-t9j:|j=j?d.z }|j@jCd/d/0|jEtj|dt6d1|d2z }|jEtGtIjH|jJr!t||d | | | jd!d3| |d<| |d4<| jd+d|d+<ttj|d| d%k(rdSdS#t,$r>} ttjdd | itjYd} ~ 'd} ~ wwxYw)5NGH_REPOJonghyukJeon/dev_workspacer)rrrrts publish-onlyroz&--status required in publish-only mode)file)successfailureneutral completed in_progress)rrrpublishFr\rz--commit-sha requiredpendingzGemini review pending)rrstateskippedrrz skipped: rjrYrZwarnz diff read: r"rCzgemini call failed: unknownrzblocking matches: rzno blocking issuesrkrmrn) timestamprQrRrkrmrnrr&rCr>r?Tr;r@rA)rrrrr&)&rr|r}r~rQrRrrmoderprintrcrdsysr4rrrVforce diff_filerrLOSErrorrrr(rgrDrFrGrHrIr`rE write_textrrO publish_check)r5rrQrRrresultproceedrrr gemini_resultr&rrrerSrTs rgaters 99 O y2NOD#!IJ#9ZQU]e]ghC yyN"{{ $**g'OPQX[XbXb c" *"&++1R"R;Xe&*kk5V&Vt{{\`LL/DKK    I djj5124LA%q,1,  djj'#:;<3::N {{i$ =Jab G  djj512(J?OGV 4:: G H $ ;9`ijpiq^rs djj512I ~~ LT^^,666HI !$ 2  *Mm//;rs6#  '  Sh1G1G1I1P1P1W1W-XY Z  7 * x & (+? ?_G ! 2 05 "? ?OD4,U/p  z CHHTVr