fix: add schema_version to writeback method (closes OB1 writeback failure)#290
fix: add schema_version to writeback method (closes OB1 writeback failure)#290cosmo-sage wants to merge 1 commit into
Conversation
|
Hey @cosmo-sage — welcome to Open Brain Source! 👋 Thanks for submitting your first PR. The automated review will run shortly and check things like metadata, folder structure, and README completeness. If anything needs fixing, the review comment will tell you exactly what. Once the automated checks pass, a human admin will review for quality and clarity. Expect a response within a few days. If you have questions, check out CONTRIBUTING.md or open an issue. |
|
I would suggest putting vs |
|
Thanks for the contribution, and for tracking down the writeback failure. The fix is on the right track. Worth folding in before this moves forward: @jonbjornn's comment above is a valid catch. As written, That is a one-line reorder. Recommend author refresh with that change before this goes to a maintainer. — Alan (community reviewer; non-binding) |
Bug Summary
The
nbj-ob1-agent-memoryplugin v0.1.1writeback()method silently fails because it doesn't sendschema_versionin the request body.The agent-memory-api server expects
schema_version: "openbrain.openclaw.writeback.v1"as a required field. Without it, writeback calls fail validation.The Fix
Added
schema_version: "openbrain.openclaw.writeback.v1"to the writeback body object, mirroring the pattern already used in therecall()method.Impact
recall()works because schema_version is included therewriteback()broken — session-end capture fails silentlyTest