chore: scrub private-repo references ahead of open-sourcing - #997
Conversation
Removes links to private trackers (inbox, infra) from a script and a doc comment — including one in a runtime CI error annotation that would print the URL into public logs. The factual observations (the #869 hang timing, the arm64 dispatch bug) are kept; only the private references are dropped. #869 stays as a same-repo reference. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
|
Warning Review limit reachedYou’ve reached a temporary PR review limit under our Fair Usage Limits Policy. Next review available in: 23 minutes Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available. How can I continue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews. How do review limits work?CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability. For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (2)
Comment |
Closes #792 (final remediation for the open-source security review — all other items verified clean, see the issue's status comment).
Part of #792 (open-source security review, item 4: cross-repo references to internal repos).
Removes the only private-tracker references in non-vendored code/docs:
scripts/bulk-upload-e2e.sh— twoinbox#335links, one of which sat inside a runtime::error::annotation that would print the private URL into public CI logs.crates/decode/src/builds.rs— agominimal/infra#18doc-comment reference (also dropped internalres-serverterminology).The factual content is preserved (the #869 hang and its observed timing; the arm64 dispatch bug and the eager→deferred fix).
#869stays — it is a same-repo minimal issue.Full-history gitleaks scan came back clean (5 findings, all false positives); this was the remaining internal-reference cleanup.
🤖 Generated with Claude Code
Note
Remove private issue references from doc comments and scripts ahead of open-sourcing
Scrubs internal issue URLs from builds.rs and bulk-upload-e2e.sh to prepare the repo for public release. No logic changes; only comments and error message text are affected.
Macroscope summarized bbfc437.