Repository navigation
test(dsh-lane): post-fold prefix-stability backstop for the uncovered plugin/openai lane (#2997) - #3002
Merged
Conversation
…ane (#2997) #2755/#2741 and their regression tests cover only the Responses-wire / proxy-mode lane. DSH-native is plugin mode over /v1/chat/completions (folds arrive via /__bili/plugin/tool) and had no post-fold cache-seam coverage. Add a hermetic backstop asserting that across a multi-disjoint-range fold the pinned first user message never moves, no summary precedes it, and consecutive steady-state builds are byte-identical except at the tail — the invariant whose violation produces #2997's early-prefix cache miss. Carrier-agnostic (holds under either the acp_summary or the client tool-pair carrier).
Contributor
📦 Built Package ArtifactBranch: Option A — Install this exact build (recommended)npm install -g billion-context@0.0.3400-pr.3002.541c1e3This exact version is immutable and reproducible. The rolling Option B — Download artifact
tar xzf billion-context-pr3002.tgz
npm install -g packageInstall instructions are refreshed on each push. |
Owner
Author
[bot] 🏷 Review 完成 — 结论:可合。 已拉取分支 我做了什么(证据)
结构化八问
两个非阻塞观察(记录在案,不拦合并)
边界确认:§5 的 PARTIAL-fold double-seam 成本问题确属另一失败域(commit-granularity 取舍,需设计决策),PR 正确地没有捆绑进来,继续在 #2997 下跟踪即可。 判定:可合。 测试有牙(变异验证通过)、范围干净、CI 全绿、triage 判定证据链完整。 |
Contributor
📦 Merged Build PublishedThis merge to master was published as npm install -g billion-context@0.0.391-master.026997b # this exact build
npm install -g billion-context@master # always the newest merged build
|
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Addresses the DSH-lane e2e-assertion ask in #2997 §4, and records the triage verdict for the reported post-fold follow-up excess.
What this adds
One hermetic regression test,
tests/openai-plugin-postfold-seam.test.ts, on the ONE compression lane #2755/#2741 never covered: DSH-native = plugin mode over/v1/chat/completions(folds arrive viaPOST /__bili/plugin/tool; rebuild pathprepare-openai). It drives a session, fires a multi-disjoint-range fold (production cadence), and asserts the cache-seam invariant §4 asks for:Carrier-agnostic: holds whether the seam carries the kernel
acp_summaryor the client's compress tool pair, so it guards the lane under either host behaviour.Triage verdict for #2997 (why the test PASSES)
A hermetic sweep over every fold shape I could construct — single, big-contiguous, multi-disjoint-range (up to 9 stacked blocks), and partial folds — shows the bili openai-chat plugin-mode rebuild is deterministic and prefix-stable: the pinned head is byte-identical across consecutive builds and no summary drifts ahead of the pin. The reported 12–17% follow-up hit therefore does not reproduce against the proxy's own output, which points the root cause OUTSIDE bili's rebuild:
compaction-basic, which stays ARMED unless the fix(dsh): profile 层 patch 其实可以关掉 web profile 的原生自动压缩(#1772 结论修正)+ 一个会静默清空整个 preset 的写法坑 #2474 override is applied (the launcher warns about this explicitly),None of those is modelable in a hermetic proxy unit test; isolating them needs the real dsh binary + real upstream.
Scope / non-goals
test:). No source changes, no config surface touched.Relates to #2997 (addresses §4).