Skip to content

diagnostic: probe C, trivial file on PR #4837 base (not for merge) - #4844

Closed
lidge-jun wants to merge 1 commit into
devfrom
codex/2580-macos-shard-probe-c
Closed

lidge-jun wants to merge 1 commit into
devfrom
codex/2580-macos-shard-probe-c

Conversation

@lidge-jun

Copy link
Copy Markdown
Owner

Summary

Diagnostic probe. Not for merge; will be closed once it has answered its question.

Same trivial one-assertion file as #4839, at the same sorted position, but based on e2304ce0e3 — the base of PR #4837 — instead of current dev.

#4839 (trivial file on current dev) and #4840's companion probe (PR #4837's real test file on current dev) both left macos 1/2 green at 663 files. PR #4837 itself wedges that shard twice: 149 files, then 17 minutes of silence, then the 20-minute wall. The remaining difference is the base. dev has since landed #4835, which fixed Worker-teardown causes and re-enabled storage-worker-teardown-isolate — the same isolate/Worker-boundary family as this wedge.

If macos 1/2 wedges here, the defect belongs to the base and not to anything in #4837.

Verification

No local suite, typecheck, or build was run. The hosted macos 1/2 job on this head is the experiment.

Checklist

  • Scope stays focused and avoids unrelated cleanup.
  • Docs or release notes were updated when needed.
  • Security-sensitive changes were reviewed for secrets, auth, and unsafe defaults.

@coderabbitai

coderabbitai Bot commented Sep 16, 2026

Copy link
Copy Markdown
Contributor

Important

Draft PR not reviewed

Draft PRs are not automatically reviewed by default.

  • Trigger a manual review

To automatically review draft PRs, update your CodeRabbit configuration:

reviews:
  auto_review:
    drafts: true

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

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

@github-actions

Copy link
Copy Markdown
Contributor

✅ Deterministic PR hygiene checks passed.

@lidge-jun

Copy link
Copy Markdown
Owner Author

Diagnostic probe complete. Result recorded in the forensics table on #4837; closing.

@lidge-jun lidge-jun closed this Sep 16, 2026
@lidge-jun
lidge-jun deleted the codex/2580-macos-shard-probe-c branch September 16, 2026 22:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant