Skip to content

ci: add semantic-data-portal hourly review-repair caller - #1343

Merged
seonghobae merged 11 commits into
mainfrom
feat/semantic-data-portal-hourly-loop
Aug 26, 2026
Merged

ci: add semantic-data-portal hourly review-repair caller#1343
seonghobae merged 11 commits into
mainfrom
feat/semantic-data-portal-hourly-loop

Conversation

@seonghobae

@seonghobae seonghobae commented Aug 25, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Adds the protected hourly caller for ContextualWisdomLab/semantic-data-portal at minute 31 — a free slot across every existing product caller (2, 10, 14, 16, 21, 23, 27, 37, 43, 49, 53, 58) and clear of the minute-zero surge.
  • Delegates to the central pr-review-fix-scheduler.yml: max 50 PRs inspected, one bounded dispatch, two-hour same-head retry floor, non-cancelling single-flight.
  • Ships docs/doctoring/semantic-data-portal-hourly-review-caller.md (RCA + remediation feasibility rationale, cadence/concurrency, credential boundary, authority boundaries: Keyverse identity, GRC policy truth, naruon document-KG writes, LineageWeave lineage reconstruction).
  • Caller contract tests mirror the disksage pattern: exact cron, read-only caller scope, explicit secret mapping, no NVIDIA_NIM_API_KEY, no COPILOT_GITHUB_TOKEN, no secrets: inherit.

Fulfills the hourly-trigger requirement of semantic-data-portal#77 (Autonomous Data Management Commercial Loop) using the established central scheduler; review-agent credential contracts untouched.

Validation

  • uv run --with pytest python -m pytest tests/test_semantic_data_portal_hourly_review_caller.py → 4 passed.
  • Existing hourly contract suites still green (21 passed).

Open in Devin Review

Summary by CodeRabbit

  • 새 기능

    • Semantic Data Portal의 리뷰·수정 점검이 매시간 자동 실행됩니다.
    • 실행 중인 작업을 취소하지 않고, 제한된 범위에서 안전하게 수정 작업을 시도합니다.
    • 동일한 변경 사항에 대한 반복 재시도가 제한됩니다.
  • 문서

    • 자동 점검 운영 절차, 검증 기준, 복구 조건 및 보안 경계를 문서화했습니다.
  • 테스트

    • 실행 주기, 권한 보호, 중복 실행 방지, 비밀 정보 노출 방지를 검증하는 테스트를 추가했습니다.

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, you can upgrade your account or add credits to your account and enable them for code reviews in your settings.

@coderabbitai

coderabbitai Bot commented Aug 25, 2026

Copy link
Copy Markdown

Review Change Stack

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 3b008a0b-b81d-4d6c-a760-de25cadd2cef

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: 71e2efa5-b937-49cd-81eb-186e266b8575

📥 Commits

Reviewing files that changed from the base of the PR and between cd3e1c2 and 6be8660.

📒 Files selected for processing (2)
  • .github/workflows/semantic-data-portal-hourly-review-repair.yml
  • docs/doctoring/semantic-data-portal-hourly-review-caller.md
🚧 Files skipped from review as they are similar to previous changes (1)
  • .github/workflows/semantic-data-portal-hourly-review-repair.yml

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


📝 Walkthrough

Walkthrough

Semantic Data Portal에 매시 59분 실행되는 리뷰 수정 호출자를 추가했다. 호출자는 중앙 스케줄러에 작업을 위임하고, 정확한 HEAD, 권한, 시크릿 및 재시도 정책을 적용한다. 운영 문서와 계약 테스트도 추가했다.

Changes

Semantic Data Portal 리뷰 수정

Layer / File(s) Summary
시간별 호출자 및 스케줄러 위임
.github/workflows/semantic-data-portal-hourly-review-repair.yml, docs/doctoring/semantic-data-portal-hourly-review-caller.md
워크플로가 매시 59분에 실행된다. 중앙 스케줄러에 대상 저장소, 브랜치, 처리 한도, 재시도 간격 및 명시적 시크릿을 전달한다. 동시 실행을 취소하지 않는다.
정확한 HEAD 및 보안 경계
docs/doctoring/semantic-data-portal-hourly-review-caller.md
정확한 HEAD 재확인, 단일 수정 제한, 2시간 재시도 제한, 읽기 전용 권한, 허용된 시크릿 매핑 및 실패 처리 조건을 정의한다.
호출자 계약 검증
tests/test_semantic_data_portal_hourly_review_caller.py
cron, 동시성, 실행 한도, 권한, 시크릿 경계, 호출자 충돌 및 운영 문서의 필수 계약을 검증한다.

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

Merge Risk: ⚪ Minimal · up to 6be86

The PR adds the hourly caller and its stated minute-31 schedule; no actionable merge-blocking risk remains beyond normal review of the documented scheduling rationale.

🚥 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 제목은 semantic-data-portal용 시간별 review-repair caller를 CI에 추가하는 변경 사항을 정확하고 간결하게 설명합니다.
Docstring Coverage ✅ Passed Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 6 functions across 1 files. (2 skipped: 2 …
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.
Full details: Docstring Coverage

Explanation

Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 6 functions across 1 files. (2 skipped: 2 unsupported.)

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/semantic-data-portal-hourly-loop

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.

@devin-ai-integration devin-ai-integration 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.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Open in Devin Review

@seonghobae
seonghobae force-pushed the feat/semantic-data-portal-hourly-loop branch from e291c81 to ca94993 Compare August 25, 2026 19:21
@seonghobae
seonghobae enabled auto-merge (squash) August 25, 2026 19:21
coderabbitai[bot]

This comment was marked as resolved.

@opencode-agent
opencode-agent Bot disabled auto-merge August 25, 2026 19:38
Adds the protected hourly caller at minute 31 (free slot across all other
product callers), delegating to the central pr-review-fix-scheduler with the
standard one-dispatch budget and two-hour same-head retry floor. Ships the
doctoring record (RCA/feasibility, cadence, credential boundary, authority
boundaries incl. Keyverse/GRC/naruon/LineageWeave) and caller contract tests
mirroring the disksage pattern. Closes ContextualWisdomLab/semantic-data-portal#77's
hourly-trigger requirement without touching COPILOT_GITHUB_TOKEN or
review-agent key contracts.
@seonghobae
seonghobae force-pushed the feat/semantic-data-portal-hourly-loop branch from a5841c5 to 4189e6a Compare August 25, 2026 20:06
@seonghobae
seonghobae enabled auto-merge (squash) August 25, 2026 20:07
devin-ai-integration[bot]

This comment was marked as resolved.

@opencode-agent
opencode-agent Bot disabled auto-merge August 25, 2026 20:19
@seonghobae
seonghobae enabled auto-merge (squash) August 25, 2026 21:30
devin-ai-integration[bot]

This comment was marked as resolved.

@opencode-agent
opencode-agent Bot disabled auto-merge August 25, 2026 21:34
@seonghobae
seonghobae enabled auto-merge (squash) August 25, 2026 22:54
@seonghobae
seonghobae merged commit 139c22f into main Aug 26, 2026
46 of 47 checks passed
@seonghobae
seonghobae deleted the feat/semantic-data-portal-hourly-loop branch August 26, 2026 17:18
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