Agent Memory System & Tool Execution Improvements#14
Merged
lxowalle merged 4 commits intosipeed:mainfrom Feb 11, 2026
Merged
Conversation
- Add MemoryStore for persistent long-term and daily notes - Add dynamic tool summary generation in system prompt - Fix YAML frontmatter parsing for nanobot skill format - Add GetSummaries() method to ToolRegistry - Fix DebugCF logging to use structured metadata - Improve web_search and shell tool descriptions
anatolykoptev
added a commit
to anatolykoptev/picoclaw
that referenced
this pull request
Feb 11, 2026
…ed#14 + SessionKey fix Cherry-picked from sipeed#14 (yinwm): - Memory system: persistent MEMORY.md + daily notes in memory/YYYYMM/ - Enhanced skills loader with YAML frontmatter + GetSummaries() - Structured logging throughout agent loop (InfoCF, DebugCF) - Tool improvements: edit tool, better shell/web descriptions - Dynamic tool summaries in system prompt - Subagent tool support Additional fix from sipeed#5: - SessionKey now uses "channel:chatID" format (was empty string) preventing all users from sharing the same session Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Collaborator
|
Nice! Thanks for the PR!! |
emadomedher
pushed a commit
to emadomedher/picoclaw
that referenced
this pull request
Feb 17, 2026
Agent Memory System & Tool Execution Improvements
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Agent Memory System & Tool Execution Improvements
Summary
This PR introduces a comprehensive memory system and improves the agent's tool execution capabilities with better logging, dynamic tool loading, and enhanced debugging support.
Key Changes
1. New Memory System (
pkg/agent/memory.go)memory/MEMORY.mdmemory/YYYYMM/YYYYMMDD.md2. Enhanced Skills System
GetSummaries()Method: ToolRegistry can provide tool descriptions for LLM3. Improved Logging & Debugging
InfoCFcalls throughout agent loop with context metadataDebugCFfor verbose system prompt loggingskills_availablefield (was showing<nil>)4. System Message Channel
processSystemMessage()Handler: Routes system channel messages properly5. Tool Registry Enhancements
6. Code Quality
BuildMessagessignature inconsistenciesFiles Changed
pkg/agent/memory.gopkg/agent/loop.gopkg/agent/context.gopkg/tools/registry.gopkg/skills/loader.gopkg/tools/edit.gopkg/tools/subagent.gocmd/picoclaw/main.goTest Plan
skills_availableshows correct countRelated Issues
N/A (feature enhancement)
Checklist
Additional Context:
This branch was created by cherry-picking 3 commits from the original
fix/tool-executionbranch onto the latestupstream/main, then adding a fix commit. It excludes the unrelated QQ channel changes to keep this PR focused on agent execution improvements.