Skip to content

fix(agent): disable hidden Goose tool-pair summarization#669

Merged
AnthonyRonning merged 1 commit into
masterfrom
codex-maple-disable-tool-pair-summarization-maple
Jul 21, 2026
Merged

fix(agent): disable hidden Goose tool-pair summarization#669
AnthonyRonning merged 1 commit into
masterfrom
codex-maple-disable-tool-pair-summarization-maple

Conversation

@AnthonyRonning

Copy link
Copy Markdown
Contributor

Summary

  • force GOOSE_TOOL_PAIR_SUMMARIZATION off whenever Maple initializes its embedded Goose runtime
  • remove any inherited environment override before writing Maple-owned Goose configuration
  • preserve exact tool evidence and prompt-cache continuity instead of silently replacing old tool pairs with hidden model-generated summaries
  • add a regression test proving Maple overwrites a stale enabled value

Why

Goose enables tool-pair summarization by default for ordinary providers. It performs an extra inference, marks the original tool request/result agent-invisible, and inserts a user-invisible summary. Maple does not surface or own that lifecycle. In addition to changing the inference prefix, the persistence path can abort a run after a tool result when a referenced message row is missing.

Full context compaction remains unchanged.

Validation

  • Maple pre-commit hook
    • Prettier check
    • TypeScript production build
    • 209 frontend tests
    • 135 Rust unit tests
  • cargo clippy --lib -- -D warnings

@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying maple with  Cloudflare Pages  Cloudflare Pages

Latest commit: 3ad14ec
Status: ✅  Deploy successful!
Preview URL: https://b7684ae9.maple-ca8.pages.dev
Branch Preview URL: https://codex-maple-disable-tool-pai.maple-ca8.pages.dev

View logs

@AnthonyRonning
AnthonyRonning merged commit e1a2c80 into master Jul 21, 2026
16 checks passed
@AnthonyRonning
AnthonyRonning deleted the codex-maple-disable-tool-pair-summarization-maple branch July 21, 2026 22:44
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