docs(claude-md): require agent-memory changes to ship with their PR - #84
Merged
Merged
Conversation
Strengthen the existing `.claude/agent-memory/` Gotcha to explicitly require that any agent-authored memory files be committed alongside the code change that produced them. Prevents accidental loss of cross-session agent learnings when PRs are squashed or when authors stage selectively. Also flushes the two silent-failure-hunter memory files that the agent updated after PR #82 merged — keeps `main` in sync with the current agent-memory state.
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
Contributor
There was a problem hiding this comment.
No issues found across 3 files
Auto-approved: Documentation-only changes and agent-memory updates with no impact on source code or infrastructure.
Architecture diagram
sequenceDiagram
participant Agent as Claude Agent
participant FS as Local File System
participant Dev as Developer (PR Author)
participant Git as Version Control (GitHub)
Note over Agent,FS: During active coding/debugging session
Agent->>FS: Perform task & identify project "gotchas"
Agent->>FS: CHANGED: Update memory files in .claude/agent-memory/**
Note over Dev,Git: PR Creation Process
Dev->>FS: git add (stage code changes)
alt NEW: Compliant with CLAUDE.md "MUST be committed" rule
Dev->>FS: Stage agent-memory changes
Dev->>Git: git commit (code + learnings)
Git-->>Git: Session learnings persisted to main branch
else Non-compliant (Missing memory)
Dev->>Git: git commit (code only)
Note right of Git: Agent learnings are lost across sessions
end
Note over Agent,Git: Future sessions (Persistence)
Git->>FS: git checkout main
Agent->>FS: Read .claude/agent-memory/
Note over Agent: Agent remembers previous fixes (e.g. ensureCheckout)
Deploying ask-registry with
|
| Latest commit: |
965dc69
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://2586b782.ask-registry.pages.dev |
| Branch Preview URL: | https://amondnet-claude-md-agent-mem.ask-registry.pages.dev |
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.
Summary
Motivation
Context: PR #82 merged with agent-memory included, but the prior note ("IS committed to git — it persists agent learnings") was a passive statement. Making it an imperative rule makes it harder to skip inadvertently.
Test plan
Summary by cubic
Make committing agent-authored memory files required: update
CLAUDE.mdto state that any changes under.claude/agent-memory/**must ship in the same PR as the code that generated them to avoid losing cross-session learnings. Also sync thereview-review-silent-failure-huntermemory files to reflect the resolved ensureCheckout fallback-ref issue and current agent-memory state.Written for commit 965dc69. Summary will update on new commits.