-
Notifications
You must be signed in to change notification settings - Fork 1k
devlog: record the community bug sweep #2636
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Changes from all commits
Commits
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
69 changes: 69 additions & 0 deletions
69
devlog/_plan/260825_owner_backlog_and_bugpr_closeout/150_community_bug_sweep.md
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,69 @@ | ||
| # 150 — community bug sweep | ||
|
|
||
| The plan's DONE criterion was zero `lidge-jun` issues and every bug PR terminal. Both were met | ||
| partway through. The request behind the plan was broader — no outstanding bugs — so the loop | ||
| continued into the `bug`-labelled community backlog rather than stopping at its own inventory. | ||
|
|
||
| Sixteen open, three left. Final `dev` at `87179e86a`: 15037 pass / 16 skip / 0 fail, typecheck | ||
| and `privacy:scan` green. | ||
|
|
||
| ## Six were already fixed by this same run | ||
|
|
||
| | Issue | Fixed by | Verified | | ||
| |---|---|---| | ||
| | #2499 Windows catalog-state latency | #2580 (`342911fec7`) | ancestry + 51 pass | | ||
| | #2545 Antigravity thought_signature | #2577 (`4d3d2716e`) | ancestry + 150 pass | | ||
| | #2210 Cursor stall timeout | `994e5ba87` | ancestry + 252 pass | | ||
| | #2156 Muse Spark mid-tool-call | `1a15d6292` + `08cc2ac89` | ancestry + 54 pass | | ||
| | #2300 Cursor slowness | #2307 | ancestry + falsification | | ||
| | #2509 stale pool account (core) | #2515 | ancestry + falsification | | ||
|
|
||
| Each was checked with `git merge-base --is-ancestor` **and** a covering test run before closing. | ||
| Closing on a report that something was fixed is how a backlog acquires issues that were never | ||
| actually resolved. | ||
|
|
||
| ## What the sweep found on its own | ||
|
|
||
| **#2458 — the reported cause was one layer off.** A 502 on an undeclared `get_video_duration` | ||
| tool call looked like a bridge problem. The real defect was that Chat Completions silently | ||
| discarded `video_url` (`src/chat/inbound.ts:56`), so the model reasoned about a video it had | ||
| never received, and the bridge refused the resulting call. OpenCodex was rejecting a symptom of | ||
| its own dropped input. | ||
|
|
||
| **#2509 — a second half nobody had noticed.** #2515 fixed selection to preview per candidate | ||
| quota scope and #2623 added the entitlement filter, but the encrypted-recovery path got the | ||
| scope and not the filter. Same stale-selection class, one layer over. | ||
|
|
||
| **#2459 — the health check was the worst part.** A bare npm reinstall under a live proxy leaves | ||
| `/healthz` answering 200 while every `/v1/responses` fails. A lying health check is worse than | ||
| an honest failure, because every supervisor believes it. | ||
|
|
||
| **A load flake, not a defect.** One full-suite failure at exactly 5003 ms — Bun's default 5s | ||
| timeout — on a case that starts a real server while its neighbours declare `SERVER_BUDGET_MS`. | ||
| It passed 3/3 in isolation on two machines. Fixed as a budget, not chased as a bug. | ||
|
|
||
| ## Three left open, each for a stated reason | ||
|
|
||
| **#2221 — native main token refresh.** The safe-looking half is not separable. A refresh rotates | ||
| the grant, `atomicWriteFile` has no compare-and-swap against an external writer, and the Codex | ||
| CLI writes the same `auth.json`. Refreshing without a crash-safe publisher can strand the login: | ||
| the current behavior fails one request, the naive fix can cost the credential. #2497 proposes a | ||
| publisher and is itself held under security review with a rename-before-link crash window. This | ||
| needs a designed protocol, not a smaller patch. | ||
|
|
||
| **#1527 — Cursor large-context.** Both mechanisms behind the reported symptoms changed: rate | ||
| limits are excluded from transport retry, and checkpoint continuation replaced full-history | ||
| replay — which was the credible differentiator in the reporter's own proxy-vs-direct comparison. | ||
| The residual claim needs a matched live probe, which only the reporter can run. | ||
|
|
||
| **#1419 — macOS SIGTRAP.** The reported offsets symbolicate to Bun's own crash handler and | ||
| nothing past it; the frames that would name the faulting subsystem are absent, and no upstream | ||
| release establishes that 1.4.0 fixes that signature. The runtime is pinned so it cannot regress; | ||
| the crash is unproven. Recorded alongside it: `ocx gui` starts an unref'd detached process, so a | ||
| native crash there strands the proxy with no supervisor, unlike an installed service. | ||
|
|
||
| ## The pattern worth keeping | ||
|
|
||
| Four of these closed as "already fixed" and three as "not fixable here, here is why". Neither is | ||
| a failure to do work. A backlog that only ever accepts code changes as outcomes accumulates | ||
| issues nobody can close, and closes issues nobody actually verified. | ||
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
When this tracked file is pushed, this sentence publishes the precise rename-before-link crash window for credential storage while #2497 is still unshipped and under security review. This is unreleased security analysis rather than a closed hardening record; remove the failure details from
devlog/and keep them in.tmp/until the fix or advisory is public, leaving only the already-public issue/PR pointer here.AGENTS.md reference: AGENTS.md:L97-L104
Useful? React with 👍 / 👎.