Skip to content

docs(blog): add the version rule to the agent-memory post - #744

Merged
mthines merged 19 commits into
mainfrom
ccr-76d0ee0b-989bjd
Nov 9, 2026
Merged

mthines merged 19 commits into
mainfrom
ccr-76d0ee0b-989bjd

Conversation

@mthines

@mthines mthines commented Nov 9, 2026 •

Copy link
Copy Markdown
Owner

A reader pointed out a gap in "Agent memory needs its OpenTelemetry moment": traces are append-only, but memories get corrected. A runbook from January is fixed in March, and without a version rule both copies look right. If the spec doesn't define that rule, every tool invents its own and the lock-in stays.

The post now:

  • asks a fifth question: which version is current
  • adds version and supersedes to the example record, and explains them under it
  • says which to use when: a fix to the same claim bumps version, and a different claim that replaces it is a new record, starting again at version 1, that supersedes the old one
  • adds "a superseded record reads as superseded" to the conformance tests
  • answers the objection in "What a skeptic would say", and concedes it's where the OpenTelemetry analogy is weakest

The post makes no new claim about LoreKit. The matching LoreKit change is in #745.

Verified: npx vitest run src/lib/blog src/lib/mdx passes, and the post compiles through the real MDX pipeline. The blog isn't part of llms.txt, so no other docs needed changes.

https://claude.ai/code/session_012YX3RPBKN75heHAMkL2Toa

claude and others added 18 commits November 7, 2026 06:59
A tour of how agent tools shape memory today (instruction files, the
Anthropic memory tool, Copilot Memory, Letta blocks, Mem0, LangGraph's
store, MCP and temporal knowledge graphs), what they agree on, why
LoreKit's record looks the way it does, and how the same record carries
a CI job's versioned JSON state.

Claude-Session: https://claude.ai/code/session_012YX3RPBKN75heHAMkL2Toa
Fix claims that did not match the code: TTL is opt-in and only a write
carrying a TTL resets it (LoreKit is permanent by default), rewrites
drop unknown frontmatter fields, kind is null unless inferred, and the
:: rationale (git forbids ':' in branch names). Add the existing
portability proposals (Portable Memory, Open Memory Protocol, COGX),
LangGraph's read-refreshed TTL, the GitHub-shaped repo grammar, and a
section answering the objections a skeptic would raise.

Claude-Session: https://claude.ai/code/session_012YX3RPBKN75heHAMkL2Toa
Four SVGs in the blog's figure style: a map of memory formats by who
writes them and how structured they are, export vs live addressing,
how Copilot, LangGraph and LoreKit decide when a memory expires, and
one record envelope carrying a prose lesson and CI JSON state.

Claude-Session: https://claude.ai/code/session_012YX3RPBKN75heHAMkL2Toa
…ng posts

Captures the workflow used for the open memory format post: check
LoreKit claims against code, outside claims against sources, run a
skeptic's review and plain-language rewrite, draw SVG figures in the
blog style (with a render script to inspect them), register the post,
and compile it through the real MDX pipeline.

Claude-Session: https://claude.ai/code/session_012YX3RPBKN75heHAMkL2Toa
Retitle to 'Agent memory needs its OpenTelemetry moment'. Lead with the
OpenTracing/OpenCensus to OpenTelemetry history and its four layers,
present LoreKit as one open-source proposal, and add a section listing
seven gaps in LoreKit's own format (stable id, open attributes, content
type, verifiable anchors, explicit lifecycle, supersession, host-neutral
address and version) with a sketch record. Add an OTel-parallel figure
and stop highlighting LoreKit in the format map.

Claude-Session: https://claude.ai/code/session_012YX3RPBKN75heHAMkL2Toa
Keep 'Agent memory needs its OpenTelemetry moment' (~8 min) focused on
the formats, the existing proposals and what a shared spec could look
like. Move LoreKit's record, its seven gaps, the sketch record and the
CI test into 'Seven gaps in LoreKit's memory format' (~7 min), dated
2026-11-14 so it stays a draft in production until then.

Claude-Session: https://claude.ai/code/session_012YX3RPBKN75heHAMkL2Toa
The 'Seven gaps' post now describes the stable id, attributes, content_type
and supersedes as shipped (#736), adds a What changed section with the new
CLI flags, and keeps anchors, an explicit expiry rule and the address
grammar as the open gaps.

Claude-Session: https://claude.ai/code/session_012YX3RPBKN75heHAMkL2Toa
Removes the post, its registry entry and its two figures, and rewords the
three sentences in the OpenTelemetry post that pointed to it.

Claude-Session: https://claude.ai/code/session_012YX3RPBKN75heHAMkL2Toa
…r-own angle

Cuts about a third: folds the four-questions section into the spec section,
shortens the tour, drops the weakest objection and the map figure that
repeated the table. Fixes the shape counts, the OpenTelemetry figure
caption, the four-layers mismatch, an overstated MCP claim, and an
undefined use of 'anchors'. Adds an early paragraph on how many people are
building their own memory layer, and why that needs an agreement rather
than another tool.

Claude-Session: https://claude.ai/code/session_012YX3RPBKN75heHAMkL2Toa
The blog's example record is a markdown fence, and MDX_SHIKI_LANGS only
loads the grammars content declares, so rehype-shiki.spec failed.

Claude-Session: https://claude.ai/code/session_012YX3RPBKN75heHAMkL2Toa
A reader pointed out that traces are append-only but memories get
corrected, so a shared record needs a version rule or every tool invents
its own. The post now asks a fifth question (which version is current),
adds version and supersedes to the example record and the conformance
tests, and answers the objection in the skeptic section.

Claude-Session: https://claude.ai/code/session_012YX3RPBKN75heHAMkL2Toa
@github-actions

github-actions Bot commented Nov 9, 2026 •

Copy link
Copy Markdown
Contributor

The dashboard preview for this PR — redeployed on each push that changes the web app.

Project Deployment Actions Updated
lorekit Ready Ready Preview Nov 9, 2026 8:45am UTC

Preview always points at this PR's latest commit; Deployment is the immutable build of 079cc42. · Comment /web-preview to force a redeploy. · Workflow logs

mthines commented Nov 9, 2026 •

Copy link
Copy Markdown
Owner Author

✅ No issues found

Adds the version rule the prior run asked for: a fix to the same claim bumps version; a different claim that replaces it is a new record at version 1 that supersedes the old one.

<a href="https://app.dash0-dev.com/goto/agent0?auto_submit=true&amp;initial_prompt=%2Fpr-review%20https%3A%2F%2Fgithub.com%2Fmthines%2Florekit%2Fpull%2F744&amp;d0_source=github:pr-reviewer-re-review&amp;org=e8b3aed4-ae5f-4093-a1af-448a98405346"&gt;&lt;picture&gt;&lt;source media="(prefers-color-scheme: dark)" srcset="https://raw.githubusercontent.com/mthines/agent-skills/main/agents/pr-reviewer/assets/agent0-re-review-dark.svg">Re-review

Review details
Gate Details
Description vs. code ✅ The description matches the diff; the new commit adds the one-sentence version rule to the envelope paragraph.
Prior review feedback ✅ Earlier review comments are resolved.
Documentation ✅ A blog-post edit; the description explains the motivation and states that llms.txt and other docs are unaffected.
Self-review signals ✅ No debug logs, leftover TODOs, or unreviewed stubs.
Code review ✅ The multi-lens review found no blocking issues.
Run Details
Mode incremental-quick · 2 lines in delta · tier quick · depth checkout
Progress open review threads 0 → 0 across the last 2 reviews
Checked 1 of 1 changed file read · no possible issues found
Found produced 0 → posted inline 0 · cleared 0 · carried forward 0 · deferred 0 · below-bar 0
Integrations none
CI still pending: Harness tests + dry-run plans, Preflight — environment is clean, Web — Vercel preview (changed) / Deploy Vercel preview, Plugin smoke (install + enable), Typecheck, Test & Lint (affected), +2 more · 2 passed
Memories no relevance rules or lessons consulted

Nothing to report — standards (skipped), measurability (skipped), severity, reach (impact graph found no consumer or importer), 0 files skipped.

pr-reviewer · commit 079cc42 · incremental review, delta since e912ae8 · how these findings are produced · updated Nov 9, 2026 8:44am UTC


Generated by Claude Code

@mthines
mthines marked this pull request as ready for review November 9, 2026 08:49
@mthines
mthines merged commit 0e14d9a into main Nov 9, 2026
19 checks passed
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.

2 participants