Skip to content

docs(memory): consolidate stranded agent memories and promoted records from stale worktrees - #925

Merged
drmoisan merged 1 commit into
mainfrom
documentationandmemories
Sep 28, 2026
Merged

drmoisan merged 1 commit into
mainfrom
documentationandmemories

Conversation

@drmoisan

Copy link
Copy Markdown
Owner

Suggested title

docs(memory): consolidate stranded agent memories and promoted records from stale worktrees

Summary

Why

A /cleanup-merged-worktrees run removed 9 worktrees and 46 merged branches, but 15 merged worktrees were blocked as dirty and several branches are unmerged. A read-only triage checked every dirty or unmerged entry against main, both at the same path and by topic. Entries that main already holds, or that are superseded or contradicted, were discarded. The entries in this PR are the ones classified GENUINELY_NEW or STILL_RELEVANT. They must reach main before the worktrees and branches that hold them are deleted.

What Changed

Agent memory (.claude/agent-memory/**)

  • New lesson files for atomic-executor, atomic-planner, orchestrator, prd-feature, task-researcher, feature-review, parallel-orchestrator and parallel-planner, each with one index line in that directory's MEMORY.md.
  • Additive sections appended to existing lesson files. Only the new text was taken; stale whole-file MEMORY.md rewrites from the worktrees were not copied.
  • prd-feature/feedback_ac_gates_verify_satisfiability.md: two new rules, added as 7 and 8. The existing rule 6 keeps its number.
  • Trimmed before commit:
    • the Deedle/runsettings paragraph in project_coverage_runner_throws_before_postprocessing.md;
    • the batch-budget and clean-tree paragraphs in project_873_evidence_projection_plan_seams.md;
    • two links to user-level memories in invoke-mstest-with-coverage-three-traps.md;
    • one host short-path token in feature-review/project_bash-heredoc-backslash-and-tmp-traps.md.

Promoted potential records (docs/features/potential/promoted/)

  • Four 2026-09-13 records from chor/defect-filings-2026-09-13.
  • Five records dated 2026-09-14 to 2026-09-20 from the session root.

Parallel run records (docs/features/parallel/bugs-2026-09-11/)

  • parallel.md, parallel-kickoff.md, parallel-status.md.

Architecture / How It Fits Together

Not applicable. These are Markdown records read by agents and maintainers, and no code path consumes them.

Verification

Completed

  • The source for every file was re-read from its worktree or branch before copying.
  • Whole-file replacements were taken only after confirming that main's blob equals the worktree's HEAD blob.
  • Additive changes were applied with git apply --3way: 14 patches, all clean.
  • Every new MEMORY.md index line was checked to resolve to an existing file.
  • The added content was scanned for conflict markers and absolute host paths. One short-path token was found and replaced with <user>.

Recommended

  • git diff origin/main...documentationandmemories --stat and spot-read the MEMORY.md diffs, which should be additions only.

Backward Compatibility / Migration Notes

None. Only additions, plus one existing memory line whose description was extended and one existing lesson file whose rules were renumbered.

Risks and Mitigations

  • Risk: a preserved lesson duplicates one on main under a different name. Mitigation: each entry was searched by topic across main's .claude/agent-memory and docs. Any duplicates are harmless and can be removed later.
  • Rollback: revert the single commit.

Review Guide

  1. .claude/agent-memory/*/MEMORY.md: index additions.
  2. .claude/agent-memory/atomic-planner/project_501_r3_preflight_seams.md: the correction.
  3. .claude/agent-memory/prd-feature/feedback_ac_gates_verify_satisfiability.md: rule numbering.
  4. New lesson files and promoted records, which can be skimmed.

Follow-ups

  • After this PR merges, run the cleanup deletion pass for the 15 dirty merged worktrees and the unmerged branches that were triaged as safe to delete.

GitHub Auto-close

  • None

🤖 Generated with Claude Code

https://claude.ai/code/session_01GXh9UqqzGiTGcnJiBDsfVG

…s from stale worktrees

Preserve content found by the cleanup-merged-worktrees dirty-worktree triage
before the stale worktrees and branches are deleted.

- Agent memory: add lessons that exist only as uncommitted files in merged
  agent worktrees and the 2026-09-12T10-15 session root (atomic-executor,
  atomic-planner, orchestrator, prd-feature, task-researcher, feature-review,
  parallel-orchestrator, parallel-planner), with their MEMORY.md index lines.
  Only additive sections of modified lesson files were taken; stale MEMORY.md
  rewrites were not copied.
- Trim superseded content: the Deedle/runsettings paragraph (#877 fixed by
  PR 880), the #873 batch-budget and clean-tree paragraphs, two dangling
  user-memory links, and one host short-path token.
- Correct atomic-planner/project_501_r3_preflight_seams.md: the five cited
  [Ignore] sites are not compiled or no longer exist on main.
- Promoted potential records for open issues #884-#887 (from the unmerged
  chor/defect-filings-2026-09-13 branch) and #899, #905, #906, #912, #914
  (uncommitted in the session root).
- Parallel run records for bugs-2026-09-11 (from the unmerged
  parallel/bugs-2026-09-11-plan branch).

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01GXh9UqqzGiTGcnJiBDsfVG
@drmoisan
drmoisan merged commit cb280fc into main Sep 28, 2026
6 checks passed
drmoisan added a commit that referenced this pull request Sep 28, 2026
…trees

Second cleanup-merged-worktrees consolidation, covering the two worktrees
that the first pass (PR #925) left in place.

- orchestrator/delegate-may-lack-bash-tool-verify-its-git-claims.md: add
  the task-researcher no-Bash lesson (fetch gh issue history yourself into
  an evidence artifact before delegating research). The draft's claim that
  WebFetch cannot read the repository was corrected: the repository is
  public and the WebFetch route is documented in
  task-researcher/reference_github_issue_search_without_gh.md.
- prd-feature/project_671_projections_only_evidence.md (new) plus its
  MEMORY.md index line: fixed-filename Markdown projections and full
  feature-relative evidence paths, never bare evidence/ paths.
- prd-feature/feedback_backticked_paths_are_the_change_footprint.md: add
  the evidence-paths-as-second-permitted-backtick-site rule alongside the
  space-in-path rule added by PR #925.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01GXh9UqqzGiTGcnJiBDsfVG
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant