Skip to content

feat(automation): run life-os hourly NVIDIA NIM review repair - #1087

Open
seonghobae wants to merge 3 commits into
mainfrom
feat/life-os-hourly-nvidia-nim-review-repair
Open

feat(automation): run life-os hourly NVIDIA NIM review repair#1087
seonghobae wants to merge 3 commits into
mainfrom
feat/life-os-hourly-nvidia-nim-review-repair

Conversation

@seonghobae

@seonghobae seonghobae commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

Buyer-perceivable gap

Hourly NVIDIA NIM review repair never scanned ContextualWisdomLab/life-os. Live LifeOS pull requests such as ContextualWisdomLab/life-os#206, ContextualWisdomLab/life-os#205, ContextualWisdomLab/life-os#199, ContextualWisdomLab/life-os#198, and ContextualWisdomLab/life-os#145 target protected main and stalled while Clearfolio, DiskSage, and fast-mlsirm received heartbeats.

Change

  • Thin scheduled caller at minute 6, base_branch: main, one dispatch, two-hour same-head retry.
  • Job-scoped id-token: write so the reusable scheduler can mint the OpenCode App fallback from GitHub OIDC.
  • Maps only PR_REVIEW_MERGE_TOKEN and OPENCODE_APPROVE_TOKEN. Never NVIDIA_NIM_API_KEY on the caller. Never COPILOT_GITHUB_TOKEN.
  • Independent pull_request / push / compileall path contracts.
  • Local 2× pytest + coverage + interrogate at 100% on this head (1122 passed).

Do not self-approve. Merge remains subject to two-approval + last-pusher ruleset. GitHub review/Checks wait is not a blocker.

Refs ContextualWisdomLab/life-os#206, ContextualWisdomLab/life-os#205, ContextualWisdomLab/life-os#199, ContextualWisdomLab/life-os#198, and ContextualWisdomLab/life-os#145.


Open in Devin Review

@cursor

cursor Bot commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

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

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: 8f9e3bdc-dd38-4dd6-97cc-991e77cf64d7


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.

@seonghobae
seonghobae enabled auto-merge (squash) August 17, 2026 07:12
@opencode-agent
opencode-agent Bot disabled auto-merge August 18, 2026 12:58
@seonghobae
seonghobae force-pushed the feat/life-os-hourly-nvidia-nim-review-repair branch from 21f2c40 to 34f1530 Compare August 19, 2026 09:52
@seonghobae seonghobae moved this from Todo to In Progress in naruon Platform Roadmap Aug 19, 2026
@seonghobae

Copy link
Copy Markdown
Contributor Author

Rebased onto current main 9e9f59f3; resolved shared hourly-contract conflicts by preserving both current-main and life-os paths. Caller tests passed (6 passed); actionlint passed on both changed workflows; diff check passed. Project #1 is In Progress.

@seonghobae
seonghobae force-pushed the feat/life-os-hourly-nvidia-nim-review-repair branch from 34f1530 to d744b80 Compare August 19, 2026 14:45
@seonghobae

Copy link
Copy Markdown
Contributor Author

Current-head evidence: exact head d744b80, based on current main bbedc1a. Rebased the original Life OS hourly caller. Caller tests: 34 passed; hourly workflow actionlint and git diff --check passed. Hosted protected checks remain; no bypass used.

@seonghobae
seonghobae force-pushed the feat/life-os-hourly-nvidia-nim-review-repair branch from d744b80 to 8246a49 Compare August 20, 2026 01:10
@seonghobae

Copy link
Copy Markdown
Contributor Author

Rebased onto current origin/main c47bee5 and preserved the existing hourly caller path set.

Current HEAD: 8246a49

Validation on this HEAD:

  • pytest -q tests/test_life_os_hourly_review_caller.py: 6 passed
  • actionlint: caller and hourly dispatcher passed
  • compileall and git diff --check passed

@OpenCode review this exact HEAD after the rebase.

Give LifeOS the same bounded exact-head repair heartbeat as other
owned products so live goals, plugin-origin, and data-rights heads
enter hourly NVIDIA NIM repair instead of stalling on Clearfolio-only
scan coverage.
@seonghobae
seonghobae force-pushed the feat/life-os-hourly-nvidia-nim-review-repair branch from 8246a49 to cdb5d77 Compare August 20, 2026 04:43
@seonghobae

Copy link
Copy Markdown
Contributor Author

@opencode-agent review the exact current head $head_oid against protected base 2cce96f. Rebased this hourly caller onto current main; contract tests, changed-workflow actionlint, and diff --check were run on the exact rebased source. The caller keeps explicit target identity, bounded dispatch/retry, OIDC/approved secret scope, and no COPILOT token forwarding. Treat predecessor reviews/checks as historical and do not approve from queued or status-only evidence.

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

✅ Devin Review: No Issues Found

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

View in Devin Review to see 1 additional finding.

Open in Devin Review

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

Labels

None yet

Projects

Status: In Progress

Development

Successfully merging this pull request may close these issues.

1 participant