docs: attribute the write-ahead login work to 0.0.14 (WALM-649) - #956
Merged
Merged
Conversation
The write-ahead/reclaim entry (#793) was filed under 0.0.13, but that code landed after 0.0.13 was published and first ships in 0.0.14 — the published 0.0.13 tarball has no dist/recovery.js and 0.0.14-dev.0 does. Moves the bullet, and drops the matching clause from the 0.0.13 summary paragraph in the docs copy. The 0.0.14 entry also never mentioned the Streamable HTTP transport, so this adds a bullet naming MEMWAL_MCP_TRANSPORT. Both changelog pages' frontmatter summaries named the previous release as the latest while their bodies already opened on the current one — 0.0.13 against 0.0.14 for the MCP, 0.1.7 against 0.1.8 for the SDK.
Resolve the 0.0.14 changelog conflicts in packages/mcp/CHANGELOG.md and docs/mcp/changelog.mdx. Keep #921's unreleased/dev-pin note alongside this branch's Added section for the opt-in Streamable HTTP transport, follow #921 in dropping the accepted-then-failed report bullet attached to memwal_recall, and keep the write-ahead delegate keypair bullet under 0.0.14 rather than 0.0.13. The docs frontmatter answer takes #921's paragraph, because npm latest is still 0.0.13 and no stable 0.0.14 exists.
Collaborator
Author
|
Merged Three changelog hunks conflicted, across
A merge-tree dry run against |
ducnmm
approved these changes
Sep 21, 2026
ducnmm
left a comment
Collaborator
There was a problem hiding this comment.
LGTM, changelog updates accurately reflect 0.0.14 features and fixes.
This was referenced Sep 21, 2026
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.
Three changelog statements are wrong. The write-ahead/reclaim work (#793) is credited under
## 0.0.13but first ships in0.0.14; the0.0.14entry never mentions the Streamable HTTP transport that shipped with it; and both changelog pages' frontmatter summaries name the previous release as the latest while their own bodies already open on the current one. This fixes all three. No code changes, no manifest changes.Evidence: on
dev,The #793 code is commit
0c765a03, authored 2026-09-16T03:05Z. npm published0.0.13on 2026-09-15T08:50Z and0.0.14-dev.0on 2026-09-16T10:14Z, so that work could not have been in 0.0.13; the tarballs agree —tar tzf memwal-mcp-0.0.13.tgz | grep recoveryis empty and 0.0.14-dev.0 carriesdist/recovery.js.0.0.13's prose summary paragraph in the docs copy carried the same claim, so that clause is dropped there too.Impact: documentation only, but it misdirects. A 0.0.13 user reading this believes an interrupted sign-in reclaims its delegate key when that build cannot; these pages carry
answerfrontmatter written for AI search citation, so the wrong version is what gets quoted back.Test: none — prose.
Open question for you, not fixed here:
CHANGELOG.mdis not inpackages/mcp/package.json'sfilesarray, so none of this reaches the npm tarball; the same is true for the SDK. Whether to publish it is a release-policy call, so both manifests are untouched.scripts/verify-manual-sdk-release.mjsis also left alone — extending it to catch this class of drift is hardening, not this fix.Several open PRs (#921, #931, #870, #873, #892, #902) append to these same changelogs and will conflict textually.
Closes #927