fix(pi): send tool_input/tool_output instead of input/output

[rohitg00]

Problem

Reported in #759. pi integration users see every conversation observation rendered as No content provided / No actual conversation content was captured in this observation in the viewer timeline.

Root cause

integrations/pi/index.ts agent_end handler emits post_tool_use observations with field names input / output. src/functions/observe.ts reads tool_input / tool_output. The field-name mismatch leaves raw.toolInput / raw.toolOutput undefined, so src/functions/compress.ts finds no content to summarize and falls back to the empty-content text.

Other integrations (openclaw, hermes, filesystem-watcher) already use the correct names — pi was the only one affected.

Fix

Rename input → tool_input, output → tool_output in the pi agent_end observation payload.

Closes

Closes #759.

Summary by CodeRabbit

Release Notes

• Chores
  • Improved the internal data structure used for recording agent conversations by updating property naming conventions to better organize tool-specific interaction data while maintaining existing message handling behavior for agent memory logging.

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
agentmemory	Ready	Preview, Comment	Jun 1, 2026 11:40pm

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: dd3fa122-0679-402e-b29c-cab43a255f44

📥 Commits

Reviewing files that changed from the base of the PR and between fd9e3bd and db4dc3b.

📒 Files selected for processing (1)

• integrations/pi/index.ts

---

📝 Walkthrough

Walkthrough

This PR updates the payload sent to the agentmemory server's observe endpoint in the agent_end handler. The property names for captured conversation data are renamed from input/output to tool_input/tool_output, with the same substring truncation logic preserved.

Changes

Agent Memory Observation Payload Update

Layer / File(s)	Summary
Agent end handler observation payload integrations/pi/index.ts	The observe endpoint payload renames conversation data field names from input/output to tool_input/tool_output while maintaining existing truncation behavior for lastPrompt and assistant text.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

A tiny hop through property names so small,
From input and output to tool_input calls,
The payload now speaks a more precise tongue,
While truncation stays true, no logic undone. 🐰

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title 'fix(pi): send tool_input/tool_output instead of input/output' accurately describes the main change: renaming fields in the pi integration's observation payload.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch fix/pi-observation-field-names

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}