Skip to content

fix(opencode): uv lock 검증과 리뷰 큐 cadence 제한#495

Merged
seonghobae merged 6 commits into
mainfrom
agent/uv-lock-check-and-opencode-cadence
Jul 12, 2026
Merged

fix(opencode): uv lock 검증과 리뷰 큐 cadence 제한#495
seonghobae merged 6 commits into
mainfrom
agent/uv-lock-check-and-opencode-cadence

Conversation

@seonghobae

@seonghobae seonghobae commented Jul 12, 2026

Copy link
Copy Markdown
Contributor

변경 사항

  • coverage evidence가 uv.lock을 사용하는 Python 프로젝트마다 uv lock --check를 먼저 실행해 stale lock 이유를 로그에 표시합니다.
  • pnpm coverage 인자 전달 수정이 포함된 최신 main을 병합했습니다.
  • 일반 OpenCode 모델 공급자 시도를 10분, 전체 모델 풀을 25분, 단계 30분, 작업 45분으로 제한합니다.
  • 후보 catalog는 한 번 순회하고 공급자별 실패·timeout 이유를 로그에 남긴 뒤 verdict가 없으면 fail closed 합니다.
  • zombie OpenCode 재실행 기준은 정상 작업 상한보다 큰 90분으로 조정해 중복 dispatch 없이 7시간 대기를 제거합니다.

검증

  • pytest 366개 통과
  • 모델 풀·스케줄러 집중 테스트 114개 통과
  • scheduler workflow actionlint 통과
  • shell 구문 검증 통과
  • Trivy Medium/High/Critical 0, secret 0
  • 대형 OpenCode workflow 단독 actionlint는 로컬 180초 timeout이므로 required GitHub actionlint/CodeQL 결과를 최종 확인합니다.

@seonghobae seonghobae changed the title fix(opencode): check uv lock consistency in coverage evidence fix(opencode): uv lock 검증과 리뷰 큐 cadence 제한 Jul 12, 2026
@seonghobae seonghobae enabled auto-merge (squash) July 12, 2026 18:15
…thub.com/ContextualWisdomLab/.github into agent/uv-lock-check-and-opencode-cadence

# Conflicts:
#	.github/workflows/opencode-review.yml
#	tests/test_opencode_agent_contract.py

@opencode-agent opencode-agent Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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: central OpenCode/Strix review-process
  • Changed files: 8
  • Model-pool outcome: exhausted
  • Head SHA: 32483c413793fdba04733f320f84686ec1371c5a
  • Workflow run: 29203616401
  • 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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant