Skip to content

🧠 🐛 fix correct mem memory param in mem.yaml example#105

Merged
rumpl merged 2 commits intodocker:mainfrom
marcellodesales:bugfix/examples/fix-mem-example-toolset
Sep 5, 2025
Merged

🧠 🐛 fix correct mem memory param in mem.yaml example#105
rumpl merged 2 commits intodocker:mainfrom
marcellodesales:bugfix/examples/fix-mem-example-toolset

Conversation

@marcellodesales
Copy link
Copy Markdown
Contributor

@marcellodesales marcellodesales commented Sep 5, 2025

🐛 Current Bug

  • The current version doesn't parse the yaml file, so no linting output.
Screenshot 2025-09-04 at 11 44 33 PM
$ find . -name funny.db

✅ Fixed Demo

The correct version is specified in the docs

   root:
     model: openai/gpt-4o
     description: Humorous AI with persistent memory
-    memory:
-      path: ./funny.db
+    toolsets:
+      - type: memory
+        path: "./agent_memory.db"
     instruction: You are a funny chatter

 metadata:
$ ls -la agent_memory.db
-rw-r--r--@ 1 marcellodesales  staff  12288 Sep  4 23:41 agent_memory.db

~/dev/github.com/marcellodesales/cagent-multi-agent-docker on  main! ⌚ 23:43:39
$ file agent_memory.db
agent_memory.db: SQLite 3.x database, last written using SQLite version 3050004, file counter 2, database pages 3, cookie 0x1, schema 4, UTF-8, version-valid-for 2
Screenshot 2025-09-04 at 11 41 51 PM Screenshot 2025-09-04 at 11 41 41 PM Screenshot 2025-09-04 at 11 41 21 PM

The current version doesn't parse the yaml file, so no linting output.

The correct version is specified in the docs
@marcellodesales marcellodesales marked this pull request as ready for review September 5, 2025 06:52
@marcellodesales marcellodesales changed the title 🧠 🔧 fix correct mem memory param in mem.yaml example 🧠 🐛 fix correct mem memory param in mem.yaml example Sep 5, 2025
Copy link
Copy Markdown
Member

@rumpl rumpl left a comment

Choose a reason for hiding this comment

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

Thanks!

@rumpl rumpl merged commit 1e05e29 into docker:main Sep 5, 2025
4 checks passed
@marcellodesales marcellodesales deleted the bugfix/examples/fix-mem-example-toolset branch September 5, 2025 21:30
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