Skip to content

Add git-as-memory skill for Git-backed agent memory - #28636

Open
femto wants to merge 2 commits into
NousResearch:mainfrom
femto:add-git-as-memory-skill
Open

Add git-as-memory skill for Git-backed agent memory#28636
femto wants to merge 2 commits into
NousResearch:mainfrom
femto:add-git-as-memory-skill

Conversation

@femto

@femto femto commented May 19, 2026

Copy link
Copy Markdown

Summary

  • Adds a new memory category under skills
  • Adds the git-as-memory skill which provides Git-backed durable memory for agents
  • Supports OpenClaw, Hermes, Claude Code, Codex, and other agents that can run shell commands

What is git-as-memory?

A memory layer that stores agent memory directly in Git refs (not the working tree), providing:

  • Append-only history through commits
  • Auditable provenance
  • No working tree changes
  • Versioned storage namespace

Test plan

  • Verify skill loads correctly in Hermes
  • Test basic gam commands (init, write, read, search)

🤖 Generated with Claude Code

Adds a new memory category with the git-as-memory skill, which provides
a durable, auditable memory layer for agents using Git refs.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@alt-glitch alt-glitch added type/feature New feature or request P3 Low — cosmetic, nice to have tool/skills Skills system (list, view, manage) tool/memory Memory tool and memory providers labels May 19, 2026

@teknium1 teknium1 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.

Thanks for the contribution. A git-ref-backed memory workflow is a plausible fit for the skill layer, and the PR avoids adding a new core tool.

Problems

  • skills/memory/git-as-memory/agents/openai.yaml:1 appears inert in current main. I found the skill tooling only allow-lists supporting files under references/, templates/, scripts/, and assets/ (tools/skill_manager_tool.py:171, tools/skill_manager_tool.py:427), and a repo-wide search for openai.yaml / default_prompt found no skill loader path consuming this metadata.
  • The new SKILL.md has only name and description frontmatter. The in-repo authoring guide says bundled peer skills normally include version, author, license, and metadata.hermes (skills/software-development/hermes-agent-skill-authoring/SKILL.md:40, skills/software-development/hermes-agent-skill-authoring/SKILL.md:56).
  • This creates a new top-level skills/memory/ category. The authoring guide says to pick the closest existing category and not invent new top-level categories casually (skills/software-development/hermes-agent-skill-authoring/SKILL.md:95, skills/software-development/hermes-agent-skill-authoring/SKILL.md:103).

Suggested changes

  • Drop or relocate agents/openai.yaml unless a real current-main consumer is added.
  • Add the standard bundled-skill metadata frontmatter.
  • Reconsider the category placement or note why memory should become a new top-level category.

Automated hermes-sweeper review.

@@ -0,0 +1,4 @@
interface:

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.

I couldn't find any current-main skill loader that consumes agents/openai.yaml or default_prompt for bundled skills; supporting files are normally under references/, templates/, scripts/, or assets/, so this file looks inert unless a consumer is added.

Comment thread skills/memory/git-as-memory/SKILL.md Outdated
@@ -0,0 +1,244 @@
---

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.

Bundled in-repo skills normally include version, author, license, and metadata.hermes frontmatter; adding those would match the repository's skill-authoring conventions.

@femto

femto commented Jun 15, 2026

Copy link
Copy Markdown
Author

please check.

@femto
femto force-pushed the add-git-as-memory-skill branch from 8e07dec to e492470 Compare June 16, 2026 05:08
@teknium1 teknium1 added sweeper:risk-platform-windows Sweeper risk: may break or behave differently on native Windows sweeper:blast-contained Sweeper blast radius: contained — one narrow path / opt-in / few users area/memory Memory subsystem: store, providers, sync, background reviews labels Jul 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/memory Memory subsystem: store, providers, sync, background reviews P3 Low — cosmetic, nice to have sweeper:blast-contained Sweeper blast radius: contained — one narrow path / opt-in / few users sweeper:risk-platform-windows Sweeper risk: may break or behave differently on native Windows tool/memory Memory tool and memory providers tool/skills Skills system (list, view, manage) type/feature New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants