Skip to content

perf: 디렉토리 스냅샷 재사용으로 중복 I/O 제거#158

Closed
seonghobae wants to merge 1 commit into
masterfrom
fix/html4tree-directory-snapshot
Closed

perf: 디렉토리 스냅샷 재사용으로 중복 I/O 제거#158
seonghobae wants to merge 1 commit into
masterfrom
fix/html4tree-directory-snapshot

Conversation

@seonghobae

Copy link
Copy Markdown
Collaborator

PR #152의 고유 최적화를 최신 master에서 보안·CSP·coverage 회귀 없이 재구축한 통제 브랜치입니다.

기존 Jules 브랜치는 exact lease 복구와 복구 커밋 뒤에도 오래된 커밋을 반복 재적용하여 nonce CSP를 unsafe-inline으로 되돌리고 경로·심볼릭 링크·읽기권한 방어 및 테스트를 삭제했습니다. 따라서 반복 재작성되는 원 브랜치를 더 이상 병합 대상으로 사용하지 않습니다.

변경:

  • 디렉토리마다 불변 파일 스냅샷과 ignore 결과를 한 번 생성
  • 렌더링, ignore 매칭, 하위 탐색에서 동일 스냅샷 재사용
  • 기존 단일 인자 API는 기본 인자로 호환성 유지
  • 생성 시점 이후 파일이 포함되지 않는 스냅샷 일관성 테스트 2개 추가

검증:

  • TDD red: 기존 함수에서 새 스냅샷 인자가 컴파일 실패
  • focused 테스트 통과
  • 전체 test, jacocoTestReport, jacocoTestCoverageVerification 통과
  • current master CI/JaCoCo 통과
  • Trivy 온라인 실패는 Maven Central 429와 Retry-After 676으로 로그에 명시됐고 offline secret scan 0건

PR #152를 이 PR로 supersede하고 current-head 중앙 OSV/Trivy/OpenCode 결과를 추적한 뒤 직접 squash merge합니다.

@seonghobae

Copy link
Copy Markdown
Collaborator Author

중앙 OpenCode review queue cadence 수정(ContextualWisdomLab/.github#495)이 병합되었습니다. 코드나 HEAD를 변경하지 않고, 이전 장시간 model-pool 실행을 종료하고 새 10분 provider / 30분 step 예산으로 current HEAD 검증을 다시 시작합니다.

@seonghobae seonghobae closed this Jul 12, 2026
@seonghobae seonghobae reopened this Jul 12, 2026

@opencode-agent opencode-agent Bot left a comment

Copy link
Copy Markdown

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: 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.

@opencode-agent opencode-agent Bot left a comment

Copy link
Copy Markdown

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: 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.

@seonghobae

Copy link
Copy Markdown
Collaborator Author

현재 master10ed4f6 (#164)에 이 PR의 핵심인 디렉토리 목록 단일 스냅샷 재사용이 이미 병합되어 중복 I/O 제거가 적용되었습니다.

이 PR의 오래된 HEAD를 현재 base에 그대로 재적용하면 이후 병합된 다음 보호를 되돌립니다.

  • #159의 확장된 민감 파일 기본 제외 목록과 referrer=no-referrer
  • #162의 긴 파일명 줄바꿈·아이콘 접근성 UI
  • 최신 테스트 기대값

따라서 승인됐지만 충돌 중인 #158을 병합하는 대신, 대체 구현 #164가 반영된 것으로 정리합니다. 최신 master의 #164 커밋은 관련 테스트와 100% JaCoCo 근거를 포함해 병합됐습니다. 로컬 ./gradlew test 재실행은 현재 머신 JDK와 이 저장소의 구형 Gradle/Groovy 조합 불일치(NoClassDefFoundError: org.codehaus.groovy.vmplugin.v7.Java7)로 시작 단계에서 실패했으며, 이는 소스 테스트 실패가 아니라 도구체인 호환성 로그로 기록합니다.

@seonghobae seonghobae closed this Jul 12, 2026
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