Skip to content

fix(agent): normalize legacy Kimi K3 tool IDs - #756

Merged
AnthonyRonning merged 1 commit into
masterfrom
codex-maple-agent-order-maple
Aug 9, 2026
Merged

fix(agent): normalize legacy Kimi K3 tool IDs#756
AnthonyRonning merged 1 commit into
masterfrom
codex-maple-agent-order-maple

Conversation

@AnthonyRonning

Copy link
Copy Markdown
Contributor

Summary

  • normalize response-local Kimi K3 tool-call IDs at the Maple provider boundary before Goose observes them
  • preserve same-inference duplicate IDs so Goose's duplicate-call guard remains intact
  • leave fixed-format IDs and all other models unchanged, with an explicit removal note for vLLM #50420

Root cause

Tinfoil's current attested Kimi K3 image contains the pre-fix vLLM parser, which emits message-scoped IDs such as read:0. Goose preserves those IDs, while Maple's timeline treats them as session-unique, so later calls replace earlier rows and adjacent reasoning coalesces.

Upstream:

Validation

  • pre-commit hook: Prettier, frontend production build, 524 frontend tests, and the Rust unit suite
  • focused provider tests: 22 passed
  • full Rust suite: 279 passed, 2 ignored
  • exact packaged macOS app/full local stack: six sequential Kimi K3 tool calls remained distinct and chronological
  • persisted Goose session: six distinct normalized request IDs, each paired with its matching response; restart/reload preserved all six rows

Notes

  • temporary Maple-side compatibility patch; remove after Tinfoil's attested K3 image includes vLLM #50420
  • prevents new collisions; it does not reconstruct already-collapsed historical rows

@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying maple with  Cloudflare Pages  Cloudflare Pages

Latest commit: f2ed731
Status: ✅  Deploy successful!
Preview URL: https://536e3243.maple-ca8.pages.dev
Branch Preview URL: https://codex-maple-agent-order-mapl.maple-ca8.pages.dev

View logs

@AnthonyRonning
AnthonyRonning merged commit 87f8784 into master Aug 9, 2026
19 checks passed
@AnthonyRonning
AnthonyRonning deleted the codex-maple-agent-order-maple branch August 9, 2026 19:38
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