perf: 디렉토리 스냅샷 재사용으로 중복 I/O 제거#158
Conversation
|
중앙 OpenCode review queue cadence 수정(ContextualWisdomLab/.github#495)이 병합되었습니다. 코드나 HEAD를 변경하지 않고, 이전 장시간 model-pool 실행을 종료하고 새 10분 provider / 30분 step 예산으로 current HEAD 검증을 다시 시작합니다. |
There was a problem hiding this comment.
Pull request overview
OpenCode model providers were unavailable for this same-head run, but deterministic current-head evidence is clean: coverage evidence passed, peer GitHub Checks are complete, medium-or-higher code-scanning alerts are clear, mergeability is clean, and reviewer threads are resolved or outdated.
Findings
No blocking findings.
Evidence
- Result: APPROVE
- Reason: current-head model-unavailable evidence fallback; coverage, docstring, peer GitHub Checks, code-scanning alerts, mergeability, and review threads were clear for current head.
- Scope:
unsupported - Changed files:
0 - Model-pool outcome:
exhausted - Head SHA:
7bcf23fb979a23b84daae2475438862f4cfd2f33 - Workflow run: 29202513860
- Workflow attempt: 1
This fallback does not suppress failed checks, medium-or-higher code-scanning alerts, merge conflicts, unresolved reviewer threads, or failed coverage evidence; any of those conditions still publish REQUEST_CHANGES or leave the approval state unchanged.
There was a problem hiding this comment.
Pull request overview
OpenCode model providers were unavailable for this same-head run, but deterministic current-head evidence is clean: coverage evidence passed, peer GitHub Checks are complete, medium-or-higher code-scanning alerts are clear, mergeability is clean, and reviewer threads are resolved or outdated.
Findings
No blocking findings.
Evidence
- Result: APPROVE
- Reason: current-head model-unavailable evidence fallback; coverage, docstring, peer GitHub Checks, code-scanning alerts, mergeability, and review threads were clear for current head.
- Scope:
unsupported - Changed files:
0 - Model-pool outcome:
exhausted - Head SHA:
7bcf23fb979a23b84daae2475438862f4cfd2f33 - Workflow run: 29204715097
- Workflow attempt: 1
This fallback does not suppress failed checks, medium-or-higher code-scanning alerts, merge conflicts, unresolved reviewer threads, or failed coverage evidence; any of those conditions still publish REQUEST_CHANGES or leave the approval state unchanged.
|
현재 이 PR의 오래된 HEAD를 현재 base에 그대로 재적용하면 이후 병합된 다음 보호를 되돌립니다.
따라서 승인됐지만 충돌 중인 #158을 병합하는 대신, 대체 구현 #164가 반영된 것으로 정리합니다. 최신 master의 #164 커밋은 관련 테스트와 100% JaCoCo 근거를 포함해 병합됐습니다. 로컬 |
PR #152의 고유 최적화를 최신 master에서 보안·CSP·coverage 회귀 없이 재구축한 통제 브랜치입니다.
기존 Jules 브랜치는 exact lease 복구와 복구 커밋 뒤에도 오래된 커밋을 반복 재적용하여 nonce CSP를 unsafe-inline으로 되돌리고 경로·심볼릭 링크·읽기권한 방어 및 테스트를 삭제했습니다. 따라서 반복 재작성되는 원 브랜치를 더 이상 병합 대상으로 사용하지 않습니다.
변경:
검증:
PR #152를 이 PR로 supersede하고 current-head 중앙 OSV/Trivy/OpenCode 결과를 추적한 뒤 직접 squash merge합니다.