Skip to content

feat: add agent-memory-pack skill - #20692

Open
mrx-data wants to merge 2 commits into
NousResearch:mainfrom
mrx-data:add-agent-memory-pack-skill
Open

feat: add agent-memory-pack skill#20692
mrx-data wants to merge 2 commits into
NousResearch:mainfrom
mrx-data:add-agent-memory-pack-skill

Conversation

@mrx-data

@mrx-data mrx-data commented May 6, 2026

Copy link
Copy Markdown

What

Adds a new skill: agent-memory-pack — Portable Agent Memory Pack management for cross-agent knowledge handoff.

Why

When users work with multiple AI agents (Claude Code, Codex, Hermes, OpenCode), knowledge about user preferences, collaboration style, project context, and current state gets siloed in each platform's private memory system. This skill creates and maintains a platform-agnostic markdown folder that any agent can read to onboard in 2 minutes.

What it does

  • Creates a portable memory pack with 9 files: user profile, working style, global rules, project index, active state, handoff template, and response examples
  • Smart path management: asks on first use, remembers for subsequent calls
  • Audit-and-merge updates (not append-only) — cleans up stale entries, deduplicates, uses absolute dates
  • Mandatory change report after every sync — every file accounted for
  • Separates from neat-freak: this skill only manages the memory pack; neat-freak handles project docs/CLAUDE.md

Files

  • skills/software-development/agent-memory-pack/SKILL.md

Category

mrx-data added 2 commits May 6, 2026 19:29
Portable Agent Memory Pack management skill for cross-agent knowledge handoff.
@alt-glitch alt-glitch added type/feature New feature or request tool/skills Skills system (list, view, manage) P3 Low — cosmetic, nice to have labels May 6, 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 portable cross-agent handoff workflow. Current main does not contain an equivalent agent-memory-pack skill, so the idea is not superseded.

Problems

  • skills/software-development/agent-memory-pack/SKILL.md:3 violates the mandatory new-skill description rule in AGENTS.md:888-900: the folded description is far over 60 characters and is not one sentence ending in a period.
  • The instructions require persistent-memory and file operations (SKILL.md:44-96) without naming the corresponding native Hermes tools; AGENTS.md:902-914 requires these references to be explicit.
  • The declared pack has two example files (SKILL.md:71-73), but the mandatory report combines them as examples/ (SKILL.md:162) despite requiring every file to be accounted for (SKILL.md:176).
  • The PR changes only SKILL.md; add the required skill test described by AGENTS.md:948-950.

Suggested changes

  • Tighten the frontmatter and restructure around the modern skill sections.
  • Specify clarify, memory, read_file, and write_file at the relevant workflow steps.
  • Report both example files separately and add an invariant test for the nine-file pack/report.

This is an automated hermes-sweeper review.

@@ -0,0 +1,412 @@
---
name: agent-memory-pack
description: >

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.

New skills must use a single-sentence description of at most 60 characters ending in a period (AGENTS.md:888-900). Please replace this multi-line trigger list with a concise trigger-focused sentence; keep detailed triggers in the body.

用户选择默认 → 使用 `~/Desktop/agent-memory`
用户指定路径 → 使用用户指定的路径

将选定的路径记录到 Agent 的持久记忆中,格式:

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.

Please name the native memory tool and its intended target/action here. The workflow depends on persistent storage, and in-repo skill prose must explicitly reference the Hermes tools it requires (AGENTS.md:902-914).

| global-rules.md | 无变更 | — |
| agent-handoff-template.md | 无变更 | — |
| README.md | 无变更 | — |
| examples/ | 无变更 | — |

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.

This aggregates two required files into one row, so the report cannot satisfy the next requirement that every file be accounted for. List examples/good-responses.md and examples/bad-responses.md separately and keep the counts consistent with the nine-file structure.

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