Skip to content

feat(automation): run Inkspan hourly NVIDIA NIM review repair - #796

Draft
seonghobae wants to merge 11 commits into
fix/hourly-nvidia-nim-review-repair-mainfrom
feat/inkspan-hourly-review-repair
Draft

feat(automation): run Inkspan hourly NVIDIA NIM review repair#796
seonghobae wants to merge 11 commits into
fix/hourly-nvidia-nim-review-repair-mainfrom
feat/inkspan-hourly-review-repair

Conversation

@seonghobae

@seonghobae seonghobae commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

Purpose

Add the product-specific one-hour PR review → repair → revalidation caller for ContextualWisdomLab/inkspan while reusing the bounded NVIDIA NIM scheduler from prerequisite PR #782.

Exact current stacked head

a4af64e6866c591eecbaa63a0e22e4145c1a257e

The branch now contains a real merge parent for current prerequisite head b921e26854f1b0fd367c76a32af6db966374bcef; GitHub reports the stacked PR mergeable with exactly five product files. No force-push, temporary workflow, encoded patch, or competing branch writer was used.

Product and MSA behavior

  • Run .github/workflows/inkspan-hourly-review-repair.yml at minute 37 of every hour, offset from Clearfolio.
  • Target exact repository ContextualWisdomLab/inkspan and protected base main.
  • Permit at most one repair dispatch per invocation and one retry per exact head per hour.
  • Keep an Inkspan-specific single-flight concurrency group so product runs cannot cancel Clearfolio runs.
  • Call the product-neutral central scheduler rather than copying privileged implementation into Inkspan.
  • Preserve standalone Inkspan operation and naruon compose / ui.panel modular integration.

Credential, privacy, and compliance boundary

  • The caller maps only PR_REVIEW_MERGE_TOKEN and OPENCODE_APPROVE_TOKEN; it never uses secrets: inherit.
  • COPILOT_GITHUB_TOKEN, GitHub Models, and direct NVIDIA_NIM_API_KEY binding are absent from the caller.
  • The caller and reusable scheduler keep the workflow-generated GITHUB_TOKEN read-only and declare no job-level write elevation.
  • Cross-repository operations use only the explicit scheduler credentials, with no github.token mutation fallback.
  • The caller contains product/branch identity and bounded numeric limits, not business payloads, comment bodies, model output, or personal records. Separation and minimization preserve operational identifiers without masking them.
  • The repair plane cannot approve, merge, release, alter reviewer credentials, update branch protection, or convert a failed check into success.

Permanent five-file slice

  • .github/workflows/hourly-nvidia-nim-review-repair.yml
  • .github/workflows/inkspan-hourly-review-repair.yml
  • CHANGELOG.md
  • docs/doctoring/inkspan-hourly-review-caller.md
  • tests/test_pr_review_fix_hourly_contract.py

The focused workflow tracks both the caller and doctoring. The static contract validates the exact schedule/target, independent concurrency, one-dispatch and one-hour bounds, explicit secret mapping, read-only workflow-token boundary, no inherited or model secrets, and absence of product literals from the reusable engine.

Stacked activation boundary

This PR targets fix/hourly-nvidia-nim-review-repair-main, the head branch of prerequisite #782. It must not merge before #782. The schedule is inactive until both the prerequisite and this caller reach protected main.

After #782 merges:

  1. retarget this PR to main;
  2. verify the effective diff remains the exact five-file product slice;
  3. rerun every exact-head focused/full quality, 100% production statement/branch/docstring, security, supply-chain, CodeRabbit, OpenCode, Noema, independent-approval, unresolved-thread, and branch-protection gate;
  4. merge only without administrative bypass.

Keep Draft. Pending, queued, cancelled, skipped-required, status-only, author-only, predecessor-head, or synthetic evidence is not acceptance.

@coderabbitai

coderabbitai Bot commented Aug 6, 2026

Copy link
Copy Markdown

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 4f50a5e7-9645-4cf8-abc0-6d67e0fe34c1

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

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

Copy link
Copy Markdown
Contributor Author

Exact-current-head read-only review request after one-time prerequisite reconciliation.

Head: 1f2e56e0e517a5cbd9f9302eec8b58e81909231c
Current stacked base head: 68414ff8b24a3f8045482ccb501c935e731f821d

The focused Hourly NVIDIA NIM Review Repair workflow is successful on this exact head. The current base and head have the identical scripts/ci/redact_sensitive_log.py blob; GitHub still lists that upstream patch because the branch histories diverged, which is explicitly documented in the PR body and must not be mistaken for a second product change. The intended Inkspan caller slice remains the scheduler caller, focused contract, doctoring, and changelog integration.

@coderabbitai review
@opencode-agent review
@cwl-noema-review review
@strix review

Review read-only. Do not update either branch, create repair automation, synthesize approval, mark Ready, or merge. Anchor findings only to the exact head above; queued, stale-head, or predecessor evidence is not success.

@coderabbitai

coderabbitai Bot commented Aug 7, 2026

Copy link
Copy Markdown

@seonghobae I will review the PR in read-only mode.

I will anchor findings to head 1f2e56e0e517a5cbd9f9302eec8b58e81909231c. I will use base head 68414ff8b24a3f8045482ccb501c935e731f821d for stacked-diff context. I will not treat the identical scripts/ci/redact_sensitive_log.py blob as an Inkspan product change.

⚠️ Action not completed

Review rate limited.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

Copy link
Copy Markdown
Contributor Author

/oc Refetch the live PR and stop without writing unless exact head is 1f2e56e0e517a5cbd9f9302eec8b58e81909231c. Keep Draft. The stacked base branch fix/hourly-nvidia-nim-review-repair-main has advanced to prerequisite #782 head 1ce640629d8e308f163b6b038c4033c4c05b1d1f, while this PR still records base 68414ff8b24a3f8045482ccb501c935e731f821d and is non-mergeable.

Reconcile the current prerequisite head into the existing Inkspan branch with an ordinary non-force merge and preserve every newer fail-closed scheduler, NVIDIA NIM, exact-scope, credential-isolation, explicit push-destination, test, and doctoring contract from #782. Resolve overlap by retaining the prerequisite implementation verbatim and preserving only the intended Inkspan product caller slice: .github/workflows/inkspan-hourly-review-repair.yml, its permanent contract tests, Inkspan caller doctoring, focused workflow trigger coverage, and matching CHANGELOG entry. Remove no prerequisite behavior and do not duplicate the central worker.

After reconciliation, require the effective diff against the live stacked base to be product-only and explain any unavoidable ancestry-only file if GitHub still reports it. Run the focused Inkspan/hourly caller contracts, the complete hourly NVIDIA NIM quality suite, full repository tests, 100% production statement/branch coverage, 100% public docstrings, compileall, actionlint/shell syntax, and git diff --check. Do not mark Ready, approve, merge, release, retarget to main before #782 merges, force-push, create another branch/PR/workflow, or alter any reviewer/model credential; never introduce COPILOT_GITHUB_TOKEN.

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