Skip to content

fix(openclaw): preserve plugin tool results - #1071

Open
kriptoburak wants to merge 1 commit into
mksglu:nextfrom
kriptoburak:codex/preserve-openclaw-plugin-results
Open

fix(openclaw): preserve plugin tool results#1071
kriptoburak wants to merge 1 commit into
mksglu:nextfrom
kriptoburak:codex/preserve-openclaw-plugin-results

Conversation

@kriptoburak

Copy link
Copy Markdown

What / Why / How

OpenClaw tools outside the built-in mapping fell through to a generic event. That event kept only the tool name, parameters, and duration. It discarded the normalized result, so plugin output disappeared from session continuity after compaction.

TweetClaw reproduces this with its tweetclaw tool and standard text-content result. Preserve the normalized result in the generic event. This also fixes the same data loss for other OpenClaw plugin tools without adding vendor-specific runtime logic.

Affected platforms

  • Claude Code
  • Cursor
  • VS Code Copilot (GitHub Copilot)
  • JetBrains Copilot
  • Gemini CLI
  • Qwen Code
  • OpenCode
  • KiloCode
  • Codex CLI
  • OpenClaw (Pi Agent)
  • Pi
  • Kiro
  • Antigravity
  • Zed
  • All platforms

Test plan

  • Added a regression using the real TweetClaw tool name, request shape, and result envelope.
  • Confirmed the test failed before the fix: 1 failed, 99 passed.
  • npm exec -- vitest run tests/plugins/openclaw.test.ts: 100 passed.
  • npm run typecheck: passed.
  • npm test: 213 files passed, 4,779 tests passed, 21 skipped. Build and bundle assertions passed.

Checklist

  • Tests added/updated (TDD: red → green)
  • npm test passes
  • npm run typecheck passes
  • Docs reviewed; no user-facing contract changed
  • No Windows path regressions (no paths changed)
  • Targets next branch (unless hotfix)
Cross-platform notes

This change adds no file paths or platform-specific behavior.

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