Skip to content

Commit d74b73f

Browse files
committed
docs(devlog): record the sweep outcome, including what the audit gate stopped
Three PRs landed, two issues left the unit after four falsified designs, and one stays draft behind the security-review gate. Zero backlog issues turned out to be already-fixed, which was the second premise the research round overturned.
1 parent b9d3434 commit d74b73f

1 file changed

Lines changed: 76 additions & 0 deletions

File tree

Lines changed: 76 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,76 @@
1+
# 060 — Outcome
2+
3+
## What landed
4+
5+
| PR | Issue | Merge | Note |
6+
|---|---|---|---|
7+
| #912 @DevMello | #888 | `1b620266b` | Tool-result images to vision models. Audit PASS, ablation-verified. |
8+
| #924 (rebase of #837/#616) || `df8243154` | Hosted image tool preferences. 4 defects fixed on top. |
9+
| #927 | #913 | `87c479006` | Compact alternate-account attempt. 2 activation tests contributed. |
10+
11+
## What did not land, and why that is the result
12+
13+
**#914 and #919 left the unit entirely.** Four audit rounds, four rejected
14+
designs, all falsified before a line of code was written:
15+
16+
1. Match Node error codes on the fetch rejection — Bun does not emit them. The
17+
guard would have been green against an injected `code` and dead in
18+
production.
19+
2. Resolve the hostname to decide neutrality — Bun's labels alternate as it
20+
evicts its DNS cache, and a resolving host can still refuse TCP behind a
21+
firewall or fake-IP proxy, which this repository explicitly supports.
22+
3. Use the boundary, "rejection means no header arrived" — Bun follows
23+
redirects by default, so a server can read `Authorization`, return 307 to a
24+
dead host, and reject after.
25+
4. For #919, read the synthetic/mid-stream label — it records who manufactured
26+
the terminal event, not who caused the failure, and the eager path's catch
27+
wraps our own inspection and rewriting too.
28+
29+
The fourth round also found the decision record:
30+
`devlog/_fin/260722_issue_bug_sweep/030_patch_s_sticky_502.md` introduced #919's
31+
behavior deliberately, with `transient 실패 기록, affinity 해제` as the stated
32+
expected outcome. It is a policy, not a defect.
33+
34+
Both moved to `devlog/_plan/260803_transport_attribution/` with the full
35+
history. @luvs01 then opened #922 with a fourth approach that includes Bun's
36+
`ConnectionRefused` — the thing design 1 missed. Reviewed with the probe
37+
evidence and the two post-header counterexamples attached.
38+
39+
**#916 stays draft.** Five hunks worth salvaging, three that must not land —
40+
including a fallback that strips shell-exported credentials on the documented
41+
direct-Bun path. It touches authentication and the durable launcher, so
42+
`MAINTAINERS.md` requires human security review. An agent audit does not
43+
substitute for one.
44+
45+
## Backlog disposition
46+
47+
Twelve issues, zero already-fixed. That was the second overturned premise: the
48+
working assumption was that recent merges had closed several by side effect,
49+
and none had.
50+
51+
Closed: #888, #913 (by merges above), #553 (environmental — the reporter's own
52+
captures showed a fake-IP DNS route returning a NetEase certificate).
53+
54+
Evidence-commented with file:line: #907, #586, #545, #241, #92, #915, #919,
55+
#893, #417.
56+
57+
Information requested, each naming the specific capture that would settle it:
58+
#796, #418, #904.
59+
60+
## Verification
61+
62+
On dev tip `b9d3434d9`:
63+
64+
- `bun x tsc --noEmit` — exit 0
65+
- `bun run test` — 7587 pass, 8 skip, 0 fail, 504 files
66+
- `bun run privacy:scan` — passed
67+
- Sharded CI run `30790453146` — success
68+
69+
## What the audit gate was worth
70+
71+
Five rounds, four FAIL. Every rejection was empirical: a runtime probe, a read
72+
of the actual test being proposed for rewrite, or an archived decision record.
73+
Two of the four would have shipped code that passed CI and did nothing.
74+
75+
The cost was one work-phase of planning. The alternative was three merged
76+
"fixes" for problems that needed a policy decision instead.

0 commit comments

Comments
 (0)