docs: retire superseded direct-NIM OpenCode hotfix - #1640
Conversation
|
Warning Review limit reachedNext included review available in 45 minutes. View limit detailsLimit details: You’ve used the included review currently available. You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Review configuration: ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Team Run ID: 📒 Files selected for processing (2)
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. Comment |
|
The remaining admission blocker is therefore the verified central Actions backlog rather than source/security disagreement. Proceeding with the documented minimum-scope bypass pinned to this exact SHA. No predecessor review/check evidence is transferred. |
![devin-ai-integration[bot]](https://avatars.githubusercontent.com/in/811515?s=60&v=4){kind=small}
| @@ -1,53 +0,0 @@ | |||
| # NVIDIA NIM OpenCode model priority (hotfix) | |||
There was a problem hiding this comment.
🔍 Baseline retains obsolete cleanup status
The direct-NIM audit still says the deleted document exists and cleanup was not attempted. Its reference and status are now stale.
Was this helpful? React with 👍 or 👎 to provide feedback.
|
|
||
| ## Verification | ||
|
|
||
| This record was created from protected `main@81b6f20d7f701bd2e50642ab107ab0f187ae6dc9`. At that revision, `opencode.jsonc` declares `enabled_providers: ["contextual-orchestrator"]`, uses `contextual-orchestrator/orchestrator/free`, and contains no live `nvidia-nim` provider block. The existing `docs/doctoring/hourly-nvidia-nim-autofix.md` already records the corrected gateway-only provider contract. |
There was a problem hiding this comment.
📝 Info: Dormant provider block stays inactive
A nvidia-nim definition remains in root configuration, but enabled_providers excludes it. The record accurately describes active routing rather than configuration inventory.
Was this helpful? React with 👍 or 👎 to provide feedback.
Preserve protected main #1640 documentation cleanup while retaining only the seven intended solo-maintainer governance paths. Fresh pre-merge comparison showed the writer one commit behind and no content delta outside those seven paths. No force-push or history rewrite.
Restore protected main #1640 documentation retirement byte-for-byte after ancestry integration exposed two predecessor documentation paths still present in the writer tree. Keep the final delta limited to the seven intended governance paths.
1 participant
Current-main cleanup
Protected
main@81b6f20d7f701bd2e50642ab107ab0f187ae6dc9already has the gateway-only runtime state that old PR #1479 was trying to establish:opencode.jsoncenables onlycontextual-orchestrator, routes throughcontextual-orchestrator/orchestrator/free, and has no livenvidia-nimprovider block. The old branch is 144 commits behind and no longer owns a safe current-main source delta.One contradiction remains on protected main:
docs/nvidia-nim-opencode-hotfix.mdstill instructs maintainers to configure a directnvidia-nimprovider, bindNVIDIA_API_KEY, and use a provider-specific admin-bypass hotfix window. That conflicts with the current gateway-only contract and the already-correcteddocs/doctoring/hourly-nvidia-nim-autofix.md/ ADR-0003 boundary.This successor performs only the still-needed current-main cleanup:
No runtime code, workflow, credential, routing policy, security threshold, review identity, or branch-protection setting changes. Supersedes the remaining documentation responsibility of #1479 without transferring any predecessor checks/reviews.