Skip to content

chore: acp-kernel 0.0.34, drop demoted mirror retry (kernel PR 112) - #125

Merged
ranxianglei merged 1 commit into
masterfrom
2026-08-22_kernel-0.0.34-drop-retry
Aug 22, 2026
Merged

chore: acp-kernel 0.0.34, drop demoted mirror retry (kernel PR 112)#125
ranxianglei merged 1 commit into
masterfrom
2026-08-22_kernel-0.0.34-drop-retry

Conversation

@ranxianglei

Copy link
Copy Markdown
Owner

Bumps acp-kernel 0.0.33 → 0.0.34 and removes the omp-side demoted-mirror retry that #124 added — the kernel now owns that problem.

Why

acp-kernel PR #112 (v0.0.34) makes openaiToCore split inline demoted <think>…</think> content, so hosts that demote thinking inline (glm / qwen3 / deepseek / kimi profiles) and hosts that use the reasoning_content field land in one identity space. Two serializations of the same turn now yield the same core sequence / fingerprints, therefore:

  • the demoteThinking mirror variant in viewToCoreStream is dead weight,
  • the primeFold refold retry in runtime.ts can never fire (condition: zero blocks replayed) — the default mirror replays fine on the first fold.

Protocol knowledge consolidates in the kernel per the architecture direction: omp stops hand-rolling a second serialization.

Changes

File Change
package.json / package-lock.json acp-kernel 0.0.34
src/runtime.ts delete retry block (~17 lines); openaiSystem stash + let r simplified
src/wire-fold.ts viewToCoreStream back to single mirror; opts.demoteThinking removed
tests/wire-transform.test.ts replaced variant unit test with id-space parity test: inline live wire ≡ reasoning_content mirror under kernel 0.0.34 (asserts the bump is load-bearing)
tests/prime-provider-restart.test.ts demoted e2e kept as-is (host still sends inline) — now passes via kernel normalization with no retry; comment updated

Tests

  • bun test 260/260 (demoted e2e restart test passes without retry)
  • tsc --noEmit + tsc -p tsconfig.test.json clean

Closes the omp side of the issue #64 demoted variant.

@ranxianglei
ranxianglei merged commit 837fe87 into master Aug 22, 2026
5 checks passed
@ranxianglei ranxianglei mentioned this pull request Aug 22, 2026
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