Repository navigation
ci: centralize law-agent quality on dedicated self-hosted runners - #2581
seonghobae merged 12 commits into
Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configuration
You can disable this status message by setting the Use the checkbox below for a quick retry:
📝 Walkthrough
Merge Risk: 🟡 Moderate · up to The full test suite can fail on runners and developer machines without this local directory. Use a short temporary path that the test creates before merging. Security Architecture Review
Pre-merge checks |
|
|
15:15 KST 실제 follow-up: consumer full349passed/skip0 in Python3.12/3.14 and final caller independent PASS; ContextualWisdomLab/law-ai-agent#1 head02f0f6db51eb28801c6d0105592a68a0ff53962a points to this central9e482e14. Source/app code preserved. |
|
2026-10-05 15:33 KST: CodeRabbit portability finding repaired under strict TDD: no-Hermes-HOME RED2 → final39passed (normal and GITHUB_ACTIONS=true). Independent final static review deleg_8376bba3 PASS; final test SHA256e36b38d67befd4f591c2362fe2306430af8843d8028d7d13e3deb5429bfece79 and reportaa2a376 matched before staging. Coordinator re-ran39passed, then pushed67bf8ded11e7d6feb4e7c0cda83217bc36bc2965. Working tree clean; helper/workflow bytes unchanged. No stale approval reused for the new test. |
|
2026-10-05 16:32 KST 새 확인. 대상은 이 PR의 현재 head
소유자 인계 요청: 현재 helper의 불확실한 instance metadata에서는 shutdown 성공과 recovery evidence 제거를 보류하는 경계를 검토해 주십시오. 정상/초기 실패/실제 stop 실패/metadata 유실을 구분하는 회귀 검증이 필요합니다. runner ACL·결제·signing·control gate 수정은 이 요청에 포함하지 않습니다. |
|
Drafted due to a reproducible cleanup trust-boundary defect on exact head
Required repair before Ready:
No merge/bypass is justified while this substantive filesystem-boundary failure exists. |
Current authority — exact head
|
Preserve shared base bytes exactly. Reviewed-Tree: 0e4f548 Local full gate: 5993 passed, 4 skipped, 86 subtests; Python coverage/docstrings 100%. Source publication only; runner, inference, formal App approval and protected merge remain open.
| | `.github/workflows/law-ai-agent-ci.yml` | `3aac714edfb5438c1f7c0eb6420fcff2f43b39692324cd6d7e86c4c103cfc164` | | ||
| | `scripts/ci/law_ai_agent_ci.sh` | `7ee3d7d4f7083d87b921530f4ee3c7ce7dd8041e9611f4429390aa58f9e4e935` | | ||
| | `tests/test_law_ai_agent_ci.py` | `8afd78b82f0c2a270375192eb35454735b1bb192b573bc89027d951b50900b35` | |
There was a problem hiding this comment.
There was a problem hiding this comment.
현재 f31 기준 문서 상태를 정정합니다. lines57–58의 helper7ee3d7d/test8afd78b8는 초기 frozen bytes의 역사적 receipt이며 현재 파일의 승인/실행으로 전이하지 않습니다. lines79의 “미게시” 및 line89의 “새 base 조합은 아직 실행하지 않았다”는 scratch 준비 시점의 문구로, 현재 상태를 설명하지 못하므로 철회합니다. 실제 f31d7e69f97187336c5cfdd7d45e109682d0a761은 parents a55eb2a + 362fed6, tree7707779ad171803411214d7631616011d86b0b2b로 ordinary 조합·non-force 게시 및 branch/pull/API readback을 완료했습니다. 현재 helperSHA67b7b11968eed60745537d9e000a5c834f1cfbc22409d8321f16191ac25355f6, 기존testSHAcbc5fa07cb6f004ff32f4f8a89f91d16dbcb4e944f9850af14d64c10e7311a25, 새testSHA736eb442735e1d40744edb554a65c087f825079eb91f5dc7650a679d6d3b8800입니다. 기존 peer의3파일 source APPROVE(문서 포함), patch25902ffe24be63974a460e4cae5a1c36ba8496f0cc41d4a0b890c8f092450778가 이 bytes에 결속됩니다. 76 targeted PASS는 부모 관측 tool 결과이며 native PostgreSQL startup·새base전체suite·Linux격리·App 승인으로 확대하지 않습니다. 이전 a55 전체5993pass도 변경된f31의full acceptance가 아닙니다. 이 thread reply가 상태정정 원문이며 historical receipts는 보존합니다. 새로운 source-neutral wake commit을 만들지 않습니다. egress finding은 별도로 미해결 유지합니다.
| runs-on: | ||
| group: CWL law CI | ||
| labels: [self-hosted, Linux, X64, law-ai-agent-ci] |
There was a problem hiding this comment.
🟨 Pull request code reaches unfiltered runner network
For same-repository PRs, quality executes the PR head on a runner whose network permits internal access. The runner label does not isolate egress, exposing reachable internal services to PR-controlled code.
Was this helpful? React with 👍 or 👎 to provide feedback.
Reconcile the reviewed law lane with current base 362fed6 and keep the base-owned Noema/App producer delta. Quote the PostgreSQL server socket option and libpq host value for RUNNER_TEMP paths containing spaces. Source publication only; runtime, App approval and protected merge remain separate.
75bdd69
into
fix/sdp-ghas-analyses-pagination-20261002
현재 source와 검토 요청
Closes #2580.
ContextualWisdomLab/law-ai-agent의 품질 실행을 중앙 reusable workflow로 정의합니다.CWL law CI전용 group을 사용하며 private 제품 source를 중앙 저장소에 복사하지 않습니다.a55eb2a949d42a49e7e1e9f3f3b7e6bcd0b86d070e4f548c5bd58bc7b124f111f7600ee0fb1539d88dff0538c55b99df3f6f19b25a06633fc72b38ef+32eced9f3586534e9a9fc5ef33e01ca52f3efbffci/sdp-all-self-hosted-20261003(PR ci: stage all workflows on isolated self-hosted runners #2565)Ready는 source-only review 요청입니다. 운영 activation, runner 격리, canary, inference, 비용·retention admission, App 승인 또는 merge 승인이 아닙니다. 기존 self-HOLD의 runtime 준비 조건을 운영 gate로 분리합니다. #2555 및 #2565의 기존 담당자와 보호 gate는 변경하지 않습니다.
수리 범위
postmaster.pid까지 symlink 경계를 거부합니다. PID 누락, stop 실패 또는 stop 이후 PID 잔존이면 증거를 보존합니다.[1,2]대신 exact 제품 source의 전체 SQL inventory를 사용합니다. 제품의read_text().encode()checksum 의미를 보존하고 DB ledger와 전체 version/checksum을 비교합니다.74fa5d7c309ef83687d2cfed2c3adf6c5a24ad4c8bdc5007c9b693f2ea4d4f48cbc5fa07cb6f004ff32f4f8a89f91d16dbcb4e944f9850af14d64c10e7311a25해당 tree의 로컬 검증
Python 3.14.5 / pytest 9.0.3의 로컬 실행입니다. Linux Actions나 실제 제품 DB/wheel acceptance를 대신하지 않습니다.
coverage run --branch -m pytest tests -q -W error: 5,993 passed / 4 skipped, subtests 86개 별도 통과, exit 0.scripts/ciPython coverage: statements 19,279/19,279, branches 7,916/7,916, missing/partial 0, 100%, exit 0. Bash helper coverage를 주장하지 않습니다.GITHUB_ACTIONS=true124 passed. 합성 경계/source 계약 검증입니다.2b7726067175f0c3b3ba252017db90cf30465daf24b243af9688344f7fb7369b55faf5734e3c20e90cd51c4cfe01f45c4edd047c2643f57980283f0c974c2df2051085b2e251b33167ef34feacedc4a14e325ed60350d6ed755a7aa074cf315b이전 PR body의
431c575fhead와 5,536-test 수치는 역사이며 현재 acceptance가 아닙니다. 이전 cleanup RED/GREEN과 후속 symlink 반례·재수리 이력은 보존합니다. 새 Ready 전이를 위해 unchanged suite나 reviewer를 반복하지 않았습니다.현재 원격 실패와 미완료 조건
Noema run
38259231556, job114846985934의 실제 GraphQL annotation:Failed to CreateArtifact: Artifact storage quota has been hit. Unable to upload any new artifacts.PR is draft; Noema model review skipped before sidecar provisioning.이는 artifact transport failure이며 substantive model verdict가 아닙니다. App token 준비, Draft skip, CodeRabbit skipped SUCCESS, zero-step 또는 skipped checks를 실제 review/approval로 계산하지 않습니다. 현재 candidate에는 Draft/current-producer evidence 수리가 이미 있지만 실제 실행 source binding과 protected producer 채택은 별도입니다. artifact 삭제, quota flag 조작, 유료 storage 변경 또는 blind dispatch를 하지 않습니다.
운영 및 정상 merge gate — 유지
APPROVED. 실제 requested-changes finding은 수리 후 재검토합니다.auto의 실제 인증 inference, 유효 비용·private retention admission. 기존 key custody를 유지하며 새 key, Secret overwrite 또는 허위 zero-cost/ZDR attestation을 만들지 않습니다.조건을 충족한 뒤에만 normal merge를 검토합니다. PR2581의 feature-base delivery와 protected default-main 배포는 서로 다릅니다. Ready는 위 조건의 면제나 사전 승인, admin bypass, dispatch 또는 운영 activation이 아닙니다.