docs(lane1): reverify recording-exporters --help item — already fixed on main (PR #145) - #180
Conversation
… on main (PR #145) The four recording-batch exporters (teleprompter / rehearsal / cockpit / queue) that previously ignored --help and rewrote their artifacts in prospects/ were fixed in a01b261 / fc44b42 (PR #145): handleHelp() runs before any work, --output= / --html= are routed through resolveOutputPath(), and scripts/test-active-operator-surfaces.mjs enforces exit 0 + usage for both --help and -h. Re-verified live against fresh origin/main: all four probes print usage and exit 0 without writing any artifact, and the full operator-surface suite passes. No source change is possible or needed; this report documents the verification. Co-authored-by: CommandCodeBot <noreply@commandcode.ai>
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
![greptile-apps[bot]](https://avatars.githubusercontent.com/in/867647?s=60&v=4){kind=small}
There was a problem hiding this comment.
Your trial has ended. Reactivate Greptile to resume code reviews.
|
Warning Review limit reached
Next review available in: 50 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: defaults Review profile: CHILL Plan: Free Run ID: 📒 Files selected for processing (1)
Note 🎁 Summarized by CodeRabbit FreeYour organization is on the Free plan. CodeRabbit will generate a high-level summary and a walkthrough for each pull request. For a comprehensive line-by-line review, please upgrade your subscription to CodeRabbit Pro by visiting https://app.coderabbit.ai/login. Comment |
There was a problem hiding this comment.
Your trial has ended. Reactivate Greptile to resume code reviews.
1 participant
Lane 1 reverify — [unreviewed-by-opus] Recording exporters ignore --help
Verdict: already fixed and merged on main (PR #145). No code change was possible or needed; this run reverified the shipped fix against fresh
origin/main(b8cb27e).Evidence
a01b261/fc44b42("fix(ops): honor --help in the 4 recording-batch exporters (fix(ops): honor --help in the 4 recording-batch exporters #145)") are ancestors of freshorigin/main(git merge-base --is-ancestor fc44b42 b8cb27e→ true).handleHelp(args, "Usage: ...")immediately after parsingprocess.argvand before any file work, and route--output=/--html=throughresolveOutputPath(..., { fallback }):scripts/export-recording-teleprompter.mjs(lines 10-16)scripts/export-recording-rehearsal-check.mjs(lines 11-20)scripts/export-recording-cockpit.mjs(lines 11-17)scripts/export-recording-queue.mjs(lines 8-14)scripts/test-active-operator-surfaces.mjsregisters all four inremainingHelpSurface(lines 133-136) and enforces exit 0 +Usage:on stdout for both--helpand-h.node scripts/test-active-operator-surfaces.mjs→Active operator surface checks passed.(exit 0); all four scripts'--helpprobes print usage and exit 0 with no artifact written (noprospects/created,git statusclean for sources).Files
None changed (verification-only). Report:
.lane/reports/docs-lane1-recording-exporters-help-reverify-20260815.md