Repository navigation
Install a bundled buzz-memory skill on desktop startup - #686
Draft
salman1993 wants to merge 1 commit into
Draft
salman1993 wants to merge 1 commit into
salman1993 wants to merge 1 commit into
Conversation
Agents carry memory procedures only in the base prompt and the CLI skill's patch section. Install a dedicated buzz-memory skill alongside buzz-cli so harnesses can load memory conventions on demand: the core index, inline entry metadata, [[slug]] cross-links, hash-checked edits, and owner-approved hygiene. The skill gets its own .skill-version marker and Claude/Codex/Goose discovery links under the same preserve/repair policy as buzz-cli. Each skill installs independently, so a blocked CLI migration does not stop the memory skill. Only buzz-cli migrates the legacy Claude-only layout; a real .claude/skills/buzz-memory directory is a custom skill and is preserved. Adapted from block/buzz#8127. Its base-prompt and onboarding-nudge edits live in the pinned buzz-acp and are out of scope here. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Signed-off-by: Salman Mohammed <smohammed@squareup.com>
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
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.
Why
Agents learn memory procedures only from the base prompt and a short patch section in the CLI skill. They miss conventions that keep memory usable as it grows: a core index, source metadata, cross-links, and safe edits.
What
Install a bundled
buzz-memoryskill next tobuzz-clion startup. Agents load it on demand through each harness's skill discovery. Adapted from block/buzz#8127.How
The skill text is #8127's
nest_memory_skill.md. Each CLI claim was checked againstbuzz memandbuzz messages thread --linkat the pinned Buzz revision (4f51b9e). Core-memory loading at session creation was checked against the pinnedbuzz-acp.ensure_buzz_cli_skillbecomesensure_buzz_skills. It installs each skill with its own.skill-versionmarker and Claude, Codex and Goose links, using the same preserve and repair policy as #667. Each skill installs independently, so a blocked CLI migration does not stop the memory skill. Onlybuzz-climigrates the old Claude-only layout. A real.claude/skills/buzz-memorydirectory is treated as a custom skill and preserved.Not carried over from #8127:
buzz-acp, so they would need a Buzz change and a pin bump.buzz_cli_skill.md. It stays a verbatim upstream copy that shares its version marker with old Buzz. The overlap is consistent with the new skill.Risk
Startup now also writes
~/.buzz/.agents/skills/buzz-memoryand its provider links. Installation errors are logged and do not stop the app from opening. Custom agent workspaces are untouched.Testing
cargo test -p buzz-agent-controller skills::: 8 passed. New coverage checks fresh install and discovery, refresh on the skill's own marker without touching current CLI edits, preservation of a custom memory directory with supporting files, and isolation from a failed CLI migration.native_host_provisions_skills_before_agent_controls_are_usedpasses: native host startup installs both skills in an isolated workspace.Deferred: agent review, human desktop acceptance, and Windows execution. To check, run
bin/just desktopfrom this branch. Confirm that~/.buzz/.agents/skills/buzz-memory/SKILL.mdexists and resolves through.claude,.codexand.goose/skills/buzz-memory. Then ask a managed agent to recall a prior decision and confirm it loadsbuzz-memorybefore usingbuzz mem. Development builds write to the real~/.buzz.🤖 Generated with Claude Code