Fix 0.3.1: get_logseq_guide crashed on literal braces (.format regression)

0.3.0 added a `{"project": "[[X]]"}` example to the guide, but render_guide used
GUIDE.format(prefix=...), which parsed the literal braces as a format field ->
KeyError: '"project"'. Switch to str.replace so literal braces are safe forever.

Also: clarify edit_block's old_content must be the block's raw_content (with the
TODO/DONE marker + property lines), not the cleaned text — a real gotcha hit in
practice. Adds tests/test_guide.py to guard the render against this class.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

Author: mandanya

pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "mcp-server-logseq"
-version = "0.3.0"
+version = "0.3.1"
 description = "An MCP server for LogSeq API"
 readme = "README.md"
 requires-python = ">=3.11"

src/mcp_server_logseq/guide.py

@@ -100,12 +100,20 @@
   as `old_content` — the edit is rejected if it doesn't match (you never read it, or
   it changed meanwhile). This is the block analogue of a file edit's old/new match and
   guards against clobbering a concurrent change.
+- `old_content` must be the block's **`raw_content`** (what read_block returns under
+  that key), NOT the cleaned `text`: raw_content keeps the leading task marker
+  (`TODO`/`DONE`) and any `key:: value` property lines. Passing the marker-stripped
+  `text` will fail the match.
 - Like other content writes it is confined to `{prefix}/`: a uuid on a page outside
   the agent namespace is refused. To change a task's status marker use
   `set_task_status`, not `edit_block`.
 """
 
 
 def render_guide(agent_prefix: str) -> str:
-    """Fill the guide with the deployment's actual agent write-prefix."""
-    return GUIDE.format(prefix=agent_prefix or "byAgent")
+    """Fill the guide with the deployment's actual agent write-prefix.
+
+    Uses str.replace (not .format) so literal braces in the guide text — e.g. a
+    `{"project": "[[X]]"}` example — don't get parsed as format fields.
+    """
+    return GUIDE.replace("{prefix}", agent_prefix or "byAgent")

tests/test_guide.py

@@ -0,0 +1,31 @@
+"""render_guide must fill {prefix} without choking on literal braces in the text.
+
+Regression: 0.3.0 added a `{"project": "[[X]]"}` example and rendered via
+str.format(), which parsed the literal braces as a format field -> KeyError.
+"""
+
+from __future__ import annotations
+
+from mcp_server_logseq.guide import render_guide
+
+
+def test_render_guide_does_not_raise_and_fills_prefix() -> None:
+    out = render_guide("byAgent")
+    assert "byAgent/" in out
+    assert "{prefix}" not in out  # every placeholder substituted
+
+
+def test_render_guide_preserves_literal_braces() -> None:
+    # a literal {"project": ...} example must survive verbatim (the 0.3.0 regression)
+    out = render_guide("byAgent")
+    assert '{"project"' in out
+
+
+def test_render_guide_custom_prefix() -> None:
+    out = render_guide("agentX")
+    assert "agentX/" in out
+
+
+def test_render_guide_empty_prefix_defaults() -> None:
+    out = render_guide("")
+    assert "byAgent/" in out