fix(desktop): update OpenCLI third-party notice - #1337
Conversation
|
Warning Review limit reached
More reviews will be available in 6 minutes and 32 seconds. Learn how PR review limits work. Your organization has used up its prepaid credits, and credit purchases are no longer available. Enable the review add-on in the billing tab to keep reviews running — you're only billed for reviews past your plan's rate limits ($0.25/file). ⌛ How to resolve this issue?After more reviews become available, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans include higher PR review limits than trial, open-source, and free plans. In all cases, reviews become available again over time. During sustained high-volume PR review activity, CodeRabbit may temporarily slow when the next review becomes available. Please see our Fair Usage Limits Policy for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (3)
✨ Finishing Touches🧪 Generate unit tests (beta)
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 |
There was a problem hiding this comment.
Code Review
This pull request updates the version of @jackwener/opencli from 1.8.3 to 1.8.4 in the THIRD_PARTY_NOTICES.md file. There are no review comments, and I have no feedback to provide.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
Summary
Update the root third-party notices entry for bundled OpenCLI from 1.8.3 to 1.8.4, and teach PR triage to route root third-party notice changes to the platform area.
This is a post-merge follow-up to #1315; there is no separate issue because the dev branch CI failure identified the missing notice update immediately after that merge.
Why
unit-desktopondevfailed becausepackages/opencode/package.jsonnow depends on@jackwener/opencli1.8.4 whileTHIRD_PARTY_NOTICES.mdstill reported 1.8.3.The follow-up PR itself also exposed that
THIRD_PARTY_NOTICES.mdhad no labeler routing rule, sopr-triageremoved manual routing labels and failed policy. Since the notices file is packaged by the desktop app, it belongs underplatform.Related Issue
Follow-up to #1315.
Human Review Status
Pending
Review Focus
Confirm the notice version matches the OpenCLI dependency bumped in #1315, and that root third-party notice changes are routed as platform PRs.
Risk Notes
No product behavior risk. This updates bundled third-party attribution text and a PR-triage labeler rule. No visible UI check is needed because the packaged notice file changes, not app UI copy.
How To Verify
Screenshots or Recordings
Not applicable; no visible UI changed.
Checklist
bug,enhancement,task,documentation. Type labels are author-added; the labeler bot does NOT assign them. Add the label in the GitHub UI, then tick this.app,ui,platform,harness,ci. The labeler bot assigns these on PR open based on changed paths. Confirm the bot's choice (or override if wrong), then tick this.P0,P1,P2,P3. The priority-triage bot suggests one on PR open. Confirm or override, then tick this.Pending,Approved by @<reviewer>, orNot required: <reason>(default isPending; "not required" is restricted to bot-authored low-risk PRs).dev, and my PR title and commit messages use Conventional Commits in English.