Skip to content

feat(openclaw): add X-Memoria-Tool header to API requests - #108

Merged
mergify[bot] merged 2 commits into
matrixorigin:mainfrom
ytw1005:codex/openclaw-tool-header
Mar 24, 2026
Merged

mergify[bot] merged 2 commits into
matrixorigin:mainfrom
ytw1005:codex/openclaw-tool-header

Conversation

@ytw1005

@ytw1005 ytw1005 commented Mar 24, 2026

Copy link
Copy Markdown
Contributor

What

Adds X-Memoria-Tool: openclaw header to all HTTP requests from the OpenClaw plugin to the Memoria API.

Why

This allows the Memoria backend to identify traffic originating from OpenClaw, enabling:

  • Per-tool usage tracking on the dashboard (/v1/tool-usage)
  • Connection status / "last seen" display for OpenClaw in the agents view

Changes

  • plugins/openclaw/openclaw/http-client.ts: Added X-Memoria-Tool: openclaw to the headers() method
  • plugins/openclaw/openclaw/__tests__/http-client.test.ts: Added test 3.8b asserting the header is present on every request

Testing

All 101 tests pass across 5 test files (vitest). No regressions.

ytw1005 and others added 2 commits March 24, 2026 16:58
Adds X-Memoria-Tool: openclaw to all HTTP requests from the OpenClaw
plugin, enabling per-tool usage tracking and connection status display
on the Memoria dashboard.

Includes regression test (3.8b) asserting the header is present.
@XuPeng-SH
XuPeng-SH self-requested a review March 24, 2026 09:08

@randomradio randomradio left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@mergify
mergify Bot merged commit 1d013ed into matrixorigin:main Mar 24, 2026
5 checks passed
@mergify

mergify Bot commented Mar 24, 2026

Copy link
Copy Markdown
Contributor

Merge Queue Status

  • ✅ Entered queue — 2026-03-24 09:15 UTC · Rule: main
  • ✅ Checks skipped · PR is already up-to-date
  • ✅ Merged — 2026-03-24 09:15 UTC · at a2d3125d74f4b106202832caf6456753cac82c8f

This pull request spent 7 seconds in the queue, with no time running CI.

Required conditions to merge

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.

3 participants