docs: backlog #292 and #294 — edit-note ## corruption, and mcp 2.1.1 reddening CI - #293
Open
CryptoJones wants to merge 2 commits into
Open
docs: backlog #292 and #294 — edit-note ## corruption, and mcp 2.1.1 reddening CI#293CryptoJones wants to merge 2 commits into
## corruption, and mcp 2.1.1 reddening CI#293CryptoJones wants to merge 2 commits into
Conversation
Mirrors the new issue into BACKLOG.md per the two-views rule, under a new "Bugs" subsection since every other Open entry is grouped by the review that produced it and this one came from hitting it in normal use. Found on 2026-08-31 while editing a project note through the MCP tool: content after the first `## ` in the details payload is reclassified as an extra section and re-emitted after `## References`, so `## Details` empties out, and a second edit leaves the stale copy alongside the new one. The note ended up asserting both a piece of guidance and its correction, with nothing failing at write time. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_0112ufce112zoeTEsrEJENbt
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Team Run ID: Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
…ery PR Found while waiting on CI for the #292 backlog PR: all 5 test matrix jobs fail on a markdown-only change. `mcp>=2.0.0,<3.0` resolves to 2.1.1 in CI, which replaces the underlying exception text with a generic "Error executing tool <name>", so the five test_server.py assertions that check WHY a call failed stop matching. Local venv is still on mcp 2.0.0, which is why it passes here and only CI sees it. Not caused by either backlog change — it will hit any PR opened today. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_0112ufce112zoeTEsrEJENbt
## heading in details## corruption, and mcp 2.1.1 reddening CI
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.
Mirrors two new issues into
BACKLOG.md, per the standing rule that the file andthe Issues tab are two views of one list. Docs-only — no code touched.
Both go under a new Bugs subsection: every other entry under
## Openisgrouped by the review that produced it, and these came from hitting them in
normal use.
#292 —
edit-notesilently guts a note whendetailscontains a##headingContent after the first
##is reclassified as an "extra" section and re-emittedafter
## References, leaving## Detailsempty — and a second edit leavesthe stale copy in the file alongside the new one.
I hit this editing a project note through the MCP tool on 2026-08-31: it ended up
carrying two contradictory bodies, including guidance I had just corrected still
reading as current. Both writes returned success and a fresh version token.
Reproduced without MCP, straight against
store.py:###is unaffected:_H2_RE(store.py:134) requires whitespace after##.#294 — mcp 2.1.1 masks
ToolErrormessages, so CI is red on every PRFound while waiting on CI for this very PR: all 5 test matrix jobs fail on a
markdown-only change.
mcp>=2.0.0,<3.0resolves to 2.1.1 in CI (confirmedin the job log), which replaces the underlying exception text with a generic
Error executing tool <name>. The fivetest_server.pyassertions that checkwhy a call failed stop matching. 5 failed, 1013 passed, on every OS and Python
version.
It passes locally because the venv is still on mcp 2.0.0 — that skew is why
only CI sees it. Neither backlog change caused it; it will hit any PR opened
today. Pinning
<2.1unblocks, but #294 argues the prior question is whetherthat detail is still reachable by callers at all — the version-conflict message
in particular is what tells an agent to re-read before writing.
CI status on this PR: everything except the
testmatrix is green (CodeQL,gitleaks, MCP conformance, wheel build + install smoke on macOS and Ubuntu). The
testfailures are #294 and are unrelated to this change.Ran the equivalent pre-commit hygiene checks by hand (no
.pre-commit-config.yamlin this repo): no trailing whitespace, single final newline, no CRLF, no conflict
markers, Nebraska footer intact.
Proudly Made in Nebraska. Go Big Red! 🌽 https://xkcd.com/2347/