Skip to content

Commit 86a8224

Browse files
committed
docs(fin): close the DeepSeek Responses wire unit
All three fixes are on origin/dev and verified in the remote file content, and the 14 tests of the unit's suite ran green on ubuntu-latest and macos-latest. Records why the windows-latest job is not a blocker for this unit: every DeepSeek-named assertion in that job passed and 2107 tests had already run before it went silent after the SSE/relay tests and was cancelled by timeout. The three runs that landed before these commits reached the branch had already failed, one with a Bun internal assertion crash. That is the open Windows Bun stability work, not this unit.
1 parent a9926d0 commit 86a8224

6 files changed

Lines changed: 25 additions & 3 deletions

File tree

devlog/_plan/260801_deepseek_responses_wire/000_research.md renamed to devlog/_fin/260801_deepseek_responses_wire/000_research.md

File renamed without changes.

devlog/_plan/260801_deepseek_responses_wire/001_plan.md renamed to devlog/_fin/260801_deepseek_responses_wire/001_plan.md

File renamed without changes.

devlog/_plan/260801_deepseek_responses_wire/010_phase1_endpoint_contract.md renamed to devlog/_fin/260801_deepseek_responses_wire/010_phase1_endpoint_contract.md

File renamed without changes.

devlog/_plan/260801_deepseek_responses_wire/020_phase2_inbound_aware_wire.md renamed to devlog/_fin/260801_deepseek_responses_wire/020_phase2_inbound_aware_wire.md

File renamed without changes.

devlog/_plan/260801_deepseek_responses_wire/030_phase3_stateless_sanitisation.md renamed to devlog/_fin/260801_deepseek_responses_wire/030_phase3_stateless_sanitisation.md

File renamed without changes.

devlog/_plan/260801_deepseek_responses_wire/040_outcome.md renamed to devlog/_fin/260801_deepseek_responses_wire/040_outcome.md

Lines changed: 25 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -76,6 +76,28 @@ rather than by provider capability. Phase 2 widened that set to include DeepSeek
7676
correct fix narrows the write site; it was left out of scope here rather than expanded
7777
into mid-phase.
7878

79-
## Not pushed
80-
81-
Eight commits sit on local `dev`. Pushing needs explicit approval and was not given.
79+
## Pushed and verified on the remote
80+
81+
All nine commits are on `origin/dev` (each confirmed with
82+
`git merge-base --is-ancestor`), and the remote file content carries all three fixes:
83+
`responsesPath: "/responses"`, the inbound-scoped `modelWireDefaults` entry, and
84+
`statelessResponses: true`. The clone-isolation guard is on the remote too.
85+
86+
CI on `5f9434ab2` — the last SHA whose code tree is identical to HEAD, since the only
87+
later commit touches a devlog file — ran all 14 tests of
88+
`tests/deepseek-inbound-wire.test.ts` green on both `ubuntu-latest` and
89+
`macos-latest`, along with the three `npm-global` jobs.
90+
91+
The `windows-latest` job did not report. It is a pre-existing platform problem, not a
92+
regression from this unit:
93+
94+
- The DeepSeek suite is not implicated — every DeepSeek-named assertion in that job
95+
passed, and 2107 tests had already passed when the job stalled.
96+
- The job went silent after the SSE/relay tests at 22:04 and was cancelled by timeout
97+
at 22:18.
98+
- The three runs immediately BEFORE this unit's commits reached the branch
99+
(`8c8832137`, `18545f87e`, `89cb08fc9`) already failed, `8c8832137` with
100+
`panic(thread 8988): Internal assertion failure` / `oh no: Bun has crashed`.
101+
102+
That belongs to the open Windows Bun stability work
103+
(`devlog/_plan/260731_windows_bun_stability/`), not here.

0 commit comments

Comments
 (0)