review: triage Greg Brockman and Boris Cherny open work

[safal207]

Summary

Preserves a read-only public review of open work associated with Greg Brockman (gdb) and Boris Cherny (bcherny).

Main decisions

Greg

No current public OpenAI PR authored by gdb or open issue assigned to him was found in the reviewed public OpenAI repositories.

The most actionable open technical item in his public repos is gdb/tee-output#3:

• report and current source support a shutdown-drain race;
• close() restores fds and immediately interrupts the child tee process;
• stdout/stderr are not explicitly flushed before fd restoration;
• pending output can be lost silently;
• proposed fix: flush → restore fds → close writers → natural EOF drain → bounded terminate/kill fallback;
• severity candidate: P1/P2 reliability for logging and audit output.

Boris

The three open PRs in bcherny/openclaw are stale fork-local shells. They were already merged upstream:

Fork PR	Upstream PR	Upstream merge
#1	openclaw/openclaw#58036	f6380ae4b7886f0cb5cc7dca45e9457017864c39
#2	openclaw/openclaw#58037	bc16b9dccf87e662a966e2c49dfb5a6923ae4e88
#3	openclaw/openclaw#58038	af81c437fafc97808e17af771aa9fbfb0fff83b7

Verdict for all three: CLOSE_AS_SUPERSEDED.

Upstream later corrected overstated prompt-cache comments in b474e098d15d8a0936153118adb6e28255b9071e; the fixes remain useful, but their guarantees should use the narrower upstream wording.

Three Claude Code issues assigned to Boris were also reviewed:

• #21151: keep open and define a collapsed file-operation visibility/a11y contract;
• #4937: verify current installed-CLI parity, then close as implemented or narrow the missing behavior;
• #1554: require a current-version reproduction and classify CPU spin, blocked I/O, MCP wait, hook wait, or memory pressure before resolution.

Files

• docs/reviews/GREG_BROCKMAN_BORIS_CHERNY_OPEN_WORK_REVIEW_2026-07-20.md
• audits/reviews/greg-boris-open-work-result-2026-07-20.json

Boundary

Public GitHub evidence only. No third-party comments, approvals, closes, labels, assignments, or code changes were made. Evidence and recommendations only; no external authority is granted.

[coderabbitai]

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: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 82563928-4352-4285-932a-5bee9e7b9de0

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

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch agent/greg-boris-open-work-review-2026-07-20

---

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.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

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