Skip to content

Comments

fix: update messageProvider to use resolvedProvider in agent handlers#389

Closed
imfing wants to merge 3 commits intoopenclaw:mainfrom
imfing:fix-message-provider
Closed

fix: update messageProvider to use resolvedProvider in agent handlers#389
imfing wants to merge 3 commits intoopenclaw:mainfrom
imfing:fix-message-provider

Conversation

@imfing
Copy link
Contributor

@imfing imfing commented Jan 7, 2026

Summary

  • Gateway agent RPC hardcoded messageProvider: "voicewake", so provider-specific tools (like WhatsApp reactions) were filtered out even when delivering to WhatsApp.

  • Fix gateway agent calls to pass the resolved delivery provider as messageProvider so provider-specific tools (e.g., WhatsApp reactions) are included.

  • Add a gateway test assertion to validate messageProvider is set when routing via the last WhatsApp session.


Tested with my WhatsApp setup:

[whatsapp] Sending reaction "👍" -> message 3CD65DDF438EAD5BFDDE
[whatsapp] Sent reaction "👍" -> message 3CD65DDF438EAD5BFDDE

@steipete
Copy link
Contributor

steipete commented Jan 7, 2026

Thank you Xin!

@steipete
Copy link
Contributor

steipete commented Jan 7, 2026

Landed via temp branch and rebased onto main. Added messageProvider assertions + changelog entry. Tests: pnpm lint && pnpm build && pnpm test. Commit: da5481e. Thanks @imfing!

@steipete steipete closed this Jan 7, 2026
@imfing
Copy link
Contributor Author

imfing commented Jan 7, 2026

Landed via temp branch and rebased onto main. Added messageProvider assertions + changelog entry. Tests: pnpm lint && pnpm build && pnpm test. Commit: da5481e. Thanks @imfing!

🚀🚀🚀
ship ship ship

@imfing imfing deleted the fix-message-provider branch January 11, 2026 00:14
dgarson added a commit to dgarson/clawdbot that referenced this pull request Feb 9, 2026
* feat: Graphiti health probe + search-manager test fix

- Add periodic Graphiti health probe (configurable interval, 3-failure threshold)
- Gate entity-extract and graph writes behind health probe check
- Wire probe start/stop into gateway lifecycle
- Add healthProbeIntervalMinutes config option to MemoryGraphitiConfig
- Fix pre-existing test failure in search-manager.test.ts (ComposableMemoryManager
  creates new instances each call, so toBe reference check was incorrect)
- Add 10 comprehensive tests for health-probe.ts

* fix: minor pr comment
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.

2 participants