Add simulated mode testing for memory bank with dialectic

- Add --simulated flag to memory-bank server config in dialectic
- Create focused test files for write-memory operations:
  * create-memory-test.yaml - tests new memory creation
  * read-before-update-test.yaml - tests concurrency control rule
- Both tests pass, confirming simulated mode works correctly

Tests verify that write-memory tool properly creates memories and
enforces the "must read before update" rule for optimistic concurrency.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>

Author: nikomatsakis

dialectic/dialectic.py

@@ -225,7 +225,7 @@ async def run_conversation_step(self, step: ConversationStep) -> TestResult:
             mcp_servers={
                 "memory-bank": {
                     "command": "uv",
-                    "args": ["--directory", "/home/nikomatsakis/dev/socratic-shell/memory-bank", "run", "python", "-m", "memory_bank", "--memories-dir", "/home/nikomatsakis/dev/socratic-shell/dialectic/tests/memory-basic"],
+                    "args": ["--directory", "/home/nikomatsakis/dev/socratic-shell/memory-bank", "run", "python", "-m", "memory_bank", "--memories-dir", "/home/nikomatsakis/dev/socratic-shell/dialectic/tests/memory-basic", "--simulated"],
                     "env": {"SOCRATIC_SHELL_LOG": "/tmp/socratic-debug.log"}
                 },
                 "dialectic-auth": {

dialectic/tests/memory-basic/create-memory-test.yaml

@@ -0,0 +1,12 @@
+name: "Create New Memory"
+description: "Test creating new memories with write-memory tool"
+
+conversation:
+  - user: "Store this insight: 'When debugging race conditions, always check the happens-before relationship between operations'"
+    expected_response:
+      should_contain: ["stored"]
+    expected_tools:
+      - tool: "mcp__memory-bank__write-memory"
+        parameters:
+          content:
+            should_contain: ["happens-before", "race conditions"]

dialectic/tests/memory-basic/read-before-update-test.yaml

@@ -0,0 +1,13 @@
+name: "Read Before Update Rule"
+description: "Test that memories must be read before they can be updated"
+
+conversation:
+  - user: "Try to update memory ID 'fake-memory-id-123' with content 'This should fail because the memory was never read'"
+    expected_response:
+      should_contain: ["Error", "must be read before updating"]
+    expected_tools:
+      - tool: "mcp__memory-bank__write-memory"
+        parameters:
+          id: "fake-memory-id-123"
+          content:
+            should_contain: ["should fail"]