Skip to content

feat(automation): run html4tree hourly NVIDIA NIM review repair - #1097

Open
seonghobae wants to merge 1 commit into
mainfrom
feat/html4tree-hourly-nvidia-nim-review-repair
Open

feat(automation): run html4tree hourly NVIDIA NIM review repair#1097
seonghobae wants to merge 1 commit into
mainfrom
feat/html4tree-hourly-nvidia-nim-review-repair

Conversation

@seonghobae

@seonghobae seonghobae commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

Summary

html4tree directory-index heads never entered the hourly NVIDIA NIM repair loop because no product caller named the public fork or its protected master base. A buyer of governed listings would feel live defects sit: ContextualWisdomLab/html4tree#475, ContextualWisdomLab/html4tree#472, and ContextualWisdomLab/html4tree#454.

This increment adds a thin minute-15 caller that inspects at most 50 open PRs, dispatches at most one exact-head repair, and uses a two-hour same-head retry floor. The reusable scheduler stays product-neutral. The caller maps only PR_REVIEW_MERGE_TOKEN and OPENCODE_APPROVE_TOKEN, grants job-scoped id-token: write, and never receives NVIDIA_NIM_API_KEY or COPILOT_GITHUB_TOKEN.

Developer experience

  • Isolated worktree from origin/main; original workspace HEAD stays on fix/strix-agent-atomic-report-persist.
  • Local coverage run -m pytest tests plus coverage report and interrogate passed twice (1122 tests, 100% scripts/ci coverage, 100% docstrings).
  • Focused path filters rerun the hourly quality gate when the caller, doctoring, or contract test changes.

User experience

  • html4tree remains a standalone MIT directory-index module.
  • Mutation stays capability-gated per head on the public fork.
  • Independent non-author approval is not synthesized. Auto-merge is armed squash only.

Doctoring: docs/doctoring/html4tree-hourly-review-caller.md.

Summary by CodeRabbit

  • 새 기능

    • 보호된 master 브랜치의 PR을 매시 15분에 검토하고, 필요한 경우 제한된 복구 작업을 실행하는 자동화가 추가되었습니다.
    • 실행 범위, 재시도 횟수, 동시성 및 권한이 제한되어 안전하게 운영됩니다.
  • 문서

    • 운영 방식, 권한 정책, 검증 및 롤백 절차를 문서화했습니다.
  • 테스트

    • 워크플로 일정, 대상 경로, 권한 범위, 시크릿 사용 및 품질 검증 연계를 확인하는 계약 테스트를 추가했습니다.

Add a thin minute-15 caller for ContextualWisdomLab/html4tree against
protected master so directory-index heads receive the same sealed
exact-head RCA loop as Clearfolio, DiskSage, and fast-mlsirm.

The reusable scheduler stays product-neutral. The caller maps only
established scheduler credentials, grants job-scoped OIDC, and never
receives NVIDIA_NIM_API_KEY or COPILOT_GITHUB_TOKEN.
@cursor

cursor Bot commented Aug 17, 2026

Copy link
Copy Markdown

Bugbot is not enabled for your account, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

@coderabbitai

coderabbitai Bot commented Aug 17, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: c9ed6ec6-5832-4989-a341-7705568e9bf2

📥 Commits

Reviewing files that changed from the base of the PR and between c47afc2 and f6eef46.

📒 Files selected for processing (8)
  • .github/workflows/hourly-nvidia-nim-review-repair.yml
  • .github/workflows/html4tree-hourly-review-repair.yml
  • AGENTS.md
  • ARCHITECTURE.md
  • CHANGELOG.md
  • CLAUDE.md
  • docs/doctoring/html4tree-hourly-review-caller.md
  • tests/test_html4tree_hourly_review_caller.py

Included review availability: Your plan includes up to 1 review per rolling hour; 0 remain after this review.


📝 Walkthrough

Walkthrough

HTML4Tree용 시간별 NVIDIA NIM 리뷰·복구 호출자를 추가했습니다. 호출자는 보호된 master의 PR을 공유 스케줄러에 전달하고, 권한·시크릿·재시도·경로 계약을 문서와 테스트로 검증합니다.

Changes

HTML4Tree 시간별 리뷰 복구

Layer / File(s) Summary
시간별 호출자와 실행 계약
.github/workflows/html4tree-hourly-review-repair.yml, AGENTS.md, ARCHITECTURE.md, CHANGELOG.md, CLAUDE.md
매시 15분에 보호된 master의 PR을 검사하고, 최대 1건의 복구를 공유 스케줄러에 전달합니다. 동시 실행을 취소하지 않으며, OIDC와 제한된 시크릿 범위를 사용합니다.
운영·권한·복구 규칙 문서화
docs/doctoring/html4tree-hourly-review-caller.md
정확한 head 확인, 복구 조건, 2시간 재시도 제한, 자격 증명 경계, capability-gated mutation, 검증 및 롤백 절차를 기록합니다.
경로 연결과 계약 테스트
.github/workflows/hourly-nvidia-nim-review-repair.yml, tests/test_html4tree_hourly_review_caller.py
pull_request, push, compileall 경로를 갱신합니다. 호출자 설정, 권한, 시크릿, 문서 요구사항, 공유 스케줄러 분리와 경로 집합을 검증합니다.

Estimated code review effort: 3 (Moderate) | ~20 minutes

Merge Risk: ⚪ Minimal · up to f6eef

The hourly repair caller is merge-ready after normal checks and review; no actionable merge-blocking risk remains.

Sequence Diagram(s)

sequenceDiagram
  participant HourlyWorkflow as HTML4Tree hourly workflow
  participant Scheduler as pr-review-fix-scheduler.yml
  participant PullRequests as protected master PRs
  HourlyWorkflow->>Scheduler: Dispatch up to one bounded repair
  Scheduler->>PullRequests: Inspect up to 50 PRs
  Scheduler-->>HourlyWorkflow: Return repair status
Loading

Possibly related PRs

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed 제목은 html4tree에 시간별 NVIDIA NIM 리뷰 복구 자동화를 추가하는 주요 변경을 정확하고 간결하게 설명합니다.
Docstring Coverage ✅ Passed Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/html4tree-hourly-nvidia-nim-review-repair

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

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

Labels

None yet

Projects

Status: Todo

Development

Successfully merging this pull request may close these issues.

1 participant