Name the third ring MemorySmith - #6
Merged
Merged
Conversation
SPEC.md uses "vault" on nearly every page for the thing a person keeps: the folder resolution may not cross, the boundary an edge stays inside, the place the attribute vocabulary belongs to. Section 2 used the same word for a layer of the specification, and the reader had to tell the two apart from context. The ring is not a property of anybody's vault. It is the notation this profile declares on top of CommonMark and GFM, and it is the only ring with no source above it. The other two rings are named after the specification that governs them, and the one that governs this ring is this document. profile.json carries "ring": "memorysmith" on all 29 entries and the schema accepts it in place of vault, which is a break for anything reading that field. In 0.x it arrives as a minor version, recorded under Changed. The released entries of the changelog keep the old name: they are dated records of what the ring was called then, and the entry of this cycle is what connects the two. Closes #5 Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_015AqyzSutJgPQiub6mc5AZg
This was referenced Sep 6, 2026
Merged
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 of changes
SPEC.mduses "vault" on nearly every page for the thing a person keeps — the folder resolution may not cross, the boundary an edge stays inside, the place the attribute vocabulary belongs to. § 2 used the same word for a layer of the specification, and the reader had to tell the two apart from context.The ring is not a property of anybody's vault. It is the notation this profile declares on top of CommonMark and GFM, and it is the only ring with no source above it. The other two rings are named after the specification that governs them; the one that governs this ring is this document.
SPEC.md§ 2 names the ring MemorySmith, and § 1.1 with it.profile.jsoncarries"ring": "memorysmith"on all 29 entries.schema/profile.schema.jsonacceptsmemorysmithin place ofvault.README.mdnames it in the table of three rings.ring:vaultlabel is renamed toring:memorysmith.The released entries of
CHANGELOG.mdare not rewritten. They are dated records of what the ring was called then, and theChangedentry of this cycle is what connects the two names.Closes #5
What an implementation has to do
Read
memorysmithwhere it readvaultin theringfield ofprofile.json. Nothing else changed: no notation was added, removed or given a different effect, no conformance case changed, and no note written in this profile parses differently.memorysmithapp/memorysmithapp pins a version of the profile and is unaffected until it moves to the next one. That is a separate issue in that repository, to open after this merges.
The three files
SPEC.md, because the prose of a notation changed — § 1.1 and § 2, the name of the ringprofile.json, because an entry was added, changed or removed — 29 entries, theringfield onlytests/conformance.json, because a case was added or its expectation changed — no case names a ringCHANGELOG.md, in the same commit as the change it documentsnpm run checkpasses locally: yes — 31 notations, 35 cases, version 0.2.0.Version
Minor. It changes the value of a published field, which is a break for anything reading
ring, and in0.xa break arrives as a minor version. It waits in[Unreleased]underChangedfor the cycle to close, perCONTRIBUTING.md§ 6.AI productivity analysis
2026-09-06to2026-09-06)Estimated human effort (without AI assistance)
The edit is mechanical and the work is not. The cost is separating the two senses of "vault" through a document that uses the word legitimately on nearly every page —
~x~and^x^render in "no vault editor" stays, "within the same vault" stays — and deciding not to rewrite the released changelog entries, which is the difference between a rename and a falsified record.🤖 Generated with Claude Code
https://claude.ai/code/session_015AqyzSutJgPQiub6mc5AZg