@chessceo/mcp 0.29.0 → 0.31.0

package/dist/index.js

@@ -15,8 +15,8 @@ import { Chess } from "chess.js";
 import { parsePGN } from "./pgn/parser.js";
 import { exportPGN } from "./pgn/exporter.js";
 import { describePosition } from "./pgn/describe.js";
-import { addMove, deleteSubtree, MutationError, promoteVariation, setAnnotations, setCeoEval, setComment, setNags, setTag, } from "./pgn/mutations.js";
-import { PathError } from "./pgn/paths.js";
+import { addLine, addMove, deleteSubtree, MutationError, promoteVariation, setAnnotations, setCeoEval, setComment, setNags, setTag, } from "./pgn/mutations.js";
+import { buildIdIndex, NodeIdError, PathError, resolveNodeId, ROOT_ID } from "./pgn/paths.js";
 import { Server } from "@modelcontextprotocol/sdk/server/index.js";
 import { StdioServerTransport } from "@modelcontextprotocol/sdk/server/stdio.js";
 import { StreamableHTTPServerTransport } from "@modelcontextprotocol/sdk/server/streamableHttp.js";
@@ -71,6 +71,7 @@ const AUTHED_TOOLS = new Set([
     "create_prep_file",
     "delete_prep_file",
     "add_move",
+    "add_line",
     "set_comment",
     "set_nags",
     "set_annotations",
@@ -79,6 +80,7 @@ const AUTHED_TOOLS = new Set([
     "set_tag",
     "apply_mutations",
     "auto_evaluate",
+    "quote_engine_eval",
     "predict_human_move",
 ]);
 function isAuthedToolCall(body) {
@@ -193,7 +195,7 @@ const TOOLS = [
             "Reading the response — CRITICAL:\n" +
             "• Win % is one weight, not a verdict. Recommend 1.b3 over 1.d4 because 60% > 50% is wrong. Sample size matters (3 games at 66% is noise; 300 at 55% is signal); avgWhite / avgBlack per move show the rating context (a big score often means a rating gap, not repertoire truth).\n" +
             "• Prep is symmetric information — both sides see the same history. Assume the opponent knows the weakness you spotted; a weak opponent won't patch it, a strong or improving one already has (but structural weaknesses like 'bad in Catalan structures' hold anyway).\n" +
-            "• Recency > career. The last 12-24 months dominate. This endpoint's compact/LLM view deliberately omits per-move `hotness` — at the individual level it's trailing noise. The general DB endpoint keeps it (there it's fashion signal).\n" +
+            "• Recency > career. The last 12-24 months dominate. This endpoint's compact/LLM view deliberately omits per-move `fashionScore` — at the individual level it's trailing noise. The general DB endpoint (`get_position_stats`) keeps it, where it's real fashion signal (what the top field is playing this month).\n" +
             "• Opponent will deviate early. Prep is a tree, not a line — cover the 2 most likely replies at each real branching point, not one 20-move line.\n" +
             "• Surprise is a scalpel. Don't tell a lifelong 1.e4 player to switch to 1.d4 — meta-signal screams prep. Rare secondary lines within the user's existing repertoire (e.g. 6.Bc4 instead of usual 6.Bg5 vs the Najdorf) are where surprise is real.\n\n" +
             "For the full guide call the `read_prep_strategy_guide` tool.",
@@ -206,17 +208,25 @@ const TOOLS = [
                     enum: ["white", "black"],
                     description: "Which colour the player is analysed with.",
                 },
+                file_id: {
+                    type: "string",
+                    description: "Prep file id to address the position from. Combine with `node_id` — the server derives the FEN from the tree, so you don't paste a FEN string. **Prefer this over `fen`/`line`/`moves` when a prep file is open.**",
+                },
+                node_id: {
+                    type: "string",
+                    description: "Node id inside `file_id` whose position to query. Root is 'r'. When set, overrides `fen`/`line`/`moves`.",
+                },
                 line: {
                     type: "string",
-                    description: "Move sequence in SAN from the starting position, space-separated, no move numbers required. Example: 'e4 e5 Nf3'. Leave empty for startpos. Alias: `moves` (same thing).",
+                    description: "Move sequence in SAN from the starting position, space-separated, no move numbers required. Example: 'e4 e5 Nf3'. Leave empty for startpos. Alias: `moves` (same thing). Only used if `node_id` is not set.",
                 },
                 moves: {
                     type: "string",
-                    description: "SAN moves to apply on top of `fen` (or on top of startpos if no fen). Same shape as `line`. Use this when you have a starting FEN and want to walk from there — e.g. fen='<game tabiya>', moves='b4 a5 c3' to explore that continuation. Wins over `line` if both are set.",
+                    description: "SAN moves to apply on top of `fen` (or on top of startpos if no fen). Same shape as `line`. Wins over `line` if both are set. Only used if `node_id` is not set.",
                 },
                 fen: {
                     type: "string",
-                    description: "Starting position as FEN. Combine with `moves` to walk from there, or use alone.",
+                    description: "Starting position as FEN. Combine with `moves` to walk from there, or use alone. Only used if `node_id` is not set.",
                 },
                 limit: {
                     type: "integer",
@@ -236,21 +246,29 @@ const TOOLS = [
             "- `gm-classical` — GM classical games (both players ≥2500, real thinking-time). BEST for opening prep — every move is signal, avgElo ~2600 across all listed moves.\n" +
             "- `main` — the whole 11.7M-game DB. Widest coverage but noisiest (includes 1000-Elo blunder-fests in the move stats). Use as fallback when gm-classical's totalCount is too small to be informative.\n\n" +
             "Game movetext is trimmed to the moves AFTER the queried position (using each game's plyNumber). Saves ~70% of the bytes vs full movetext.\n\n" +
-            "AUTO-EVAL: if a cloud combo instance is running, the response includes `.eval` with a compact Stockfish + Lc0 read and the corresponding NAG. Do NOT fire cloud_analyse separately for the same FEN.",
+            "AUTO-EVAL: if a cloud combo instance is running, the response includes `.eval` with a compact Stockfish + Lc0 read and the corresponding NAG. Do NOT fire cloud_analyse separately for the same FEN. When called with `file_id`+`node_id`, the eval is also auto-stored on that node's `ceoEval` — later readable via quote_engine_eval.",
         inputSchema: {
             type: "object",
             properties: {
+                file_id: {
+                    type: "string",
+                    description: "Prep file id. **Prefer file_id+node_id over `fen`** when a prep file is open — the server derives the FEN from the tree.",
+                },
+                node_id: {
+                    type: "string",
+                    description: "Node id inside `file_id`. Root is 'r'. When set, overrides `fen`/`moves`/`line`.",
+                },
                 fen: {
                     type: "string",
-                    description: "Starting position as FEN. Combine with `moves` to walk from there.",
+                    description: "Starting position as FEN. Combine with `moves`. Only used if `node_id` is not set.",
                 },
                 moves: {
                     type: "string",
-                    description: "Optional SAN moves to apply on top of `fen` (or on top of startpos). Example: fen='<tabiya>', moves='b4 a5'.",
+                    description: "Optional SAN moves on top of `fen` (or startpos). Only used if `node_id` is not set.",
                 },
                 line: {
                     type: "string",
-                    description: "Synonym for `moves` from the starting position; kept for compatibility.",
+                    description: "Synonym for `moves` from startpos; kept for compatibility.",
                 },
                 limit: {
                     type: "integer",
@@ -271,12 +289,14 @@ const TOOLS = [
         description: "Structured facts about a chess position: piece placements per colour (SAN-style letters), material balance in pawn units, list of every contested piece (attackers + defenders), hanging pieces, checkers if in check, castling rights, en passant square, side to move, and the full list of LEGAL MOVES for the side to move. Pure computation — no engine, ~1 ms per call.\n\n" +
             "USE THIS BEFORE COMMENTING ON A POSITION. LLMs are not reliable at reading FEN strings — you'll misplace pieces or invent captures. This tool gives you the same board state a human sees.\n\n" +
             "ALSO USE THIS if `add_move` rejects an illegal SAN — the `.legalMoves` array shows exactly what's playable from that position.\n\n" +
-            "Position input is flexible — pass `fen`, or `moves` from startpos, or `fen + moves` to walk from there.",
+            "Position input: prefer `file_id`+`node_id` if you're inside a prep file. Otherwise `fen`, `moves` from startpos, or `fen + moves`.",
         inputSchema: {
             type: "object",
             properties: {
-                fen: { type: "string", description: "Starting position as FEN (defaults to startpos)." },
-                moves: { type: "string", description: "Optional SAN moves to apply on top of `fen`." },
+                file_id: { type: "string", description: "Prep file id. When combined with `node_id`, describes that node's position." },
+                node_id: { type: "string", description: "Node id inside `file_id`. Root is 'r'." },
+                fen: { type: "string", description: "Starting position as FEN (defaults to startpos). Only used if `node_id` is not set." },
+                moves: { type: "string", description: "Optional SAN moves to apply on top of `fen`. Only used if `node_id` is not set." },
             },
         },
     },
@@ -290,14 +310,16 @@ const TOOLS = [
             "• After you've found what the opponent SHOULD do (with the engine), check what they'll ACTUALLY do at their rating. If the top human move is a mistake, you have a real practical advantage.\n" +
             "• The WDL head is rating-aware — a 400-point gap will show up as a big win probability even in equal positions (the model has learned that human errors compound).\n" +
             "• Pass `prev_fen` (most recent first) when analysing mid-trade positions — without history the model treats the position as quiet.\n\n" +
-            "Position input is flexible — pass `fen`, or `moves` from startpos, or `fen + moves`. Default rating 2400 both sides. ~1-2s per call. **Premium (or admin/moderator) only** — anonymous calls get 402.",
+            "Position input: prefer `file_id`+`node_id` when inside a prep file. Otherwise `fen`, `moves` from startpos, or `fen + moves`. Default rating 2400 both sides. ~1-2s per call. **Premium (or admin/moderator) only** — anonymous calls get 402.",
         inputSchema: {
             type: "object",
             properties: {
-                fen: { type: "string", description: "Starting position as FEN." },
+                file_id: { type: "string", description: "Prep file id. Combine with `node_id` to point at a tree node's position." },
+                node_id: { type: "string", description: "Node id inside `file_id`. Root is 'r'." },
+                fen: { type: "string", description: "Starting position as FEN. Only used if `node_id` is not set." },
                 moves: {
                     type: "string",
-                    description: "Optional SAN moves to apply on top of `fen` (or startpos).",
+                    description: "Optional SAN moves to apply on top of `fen` (or startpos). Only used if `node_id` is not set.",
                 },
                 white_elo: {
                     type: "integer",
@@ -418,14 +440,17 @@ const TOOLS = [
             "Contempt (`contempt`) skews Lc0 (only Lc0 — Stockfish always stays objective) toward White (positive) or Black (negative). Practical range -20..+20. Use it to find non-objective 'practical' ideas or when the user needs to steer toward fighting/solid lines with a specific colour. Do NOT quote a contempt-biased eval as objective — cross-check with Stockfish.\n\n" +
             "Also useful: pass `moves` on top of `fen` to explore a variation without computing FENs yourself (e.g. fen='<tabiya>', moves='b4 a5 c3'). And the flip-side-to-move threat check documented in the guide is a great free trick.\n\n" +
             "For the full guide including worked examples, call the `read_engine_usage_guide` tool.\n\n" +
-            "Not for casual questions — this costs real money per second. Use the free `analyse` tool (single Stockfish, 2s) or `get_position_stats` for anything that doesn't require deep prep.",
+            "Not for casual questions — this costs real money per second. Use `get_position_stats` for anything that doesn't require deep prep.\n\n" +
+            "**When called with `file_id`+`node_id` (preferred inside a prep file), the resulting eval is auto-stored on that node's `ceoEval` — you can then quote it with quote_engine_eval on any later call.** This is what makes engine attribution trustworthy: prose that says 'engines say X on node Y' can only be true if a call was actually made against node_id=Y.",
         inputSchema: {
             type: "object",
             properties: {
-                fen: { type: "string", description: "Starting position as FEN. Combine with `moves` to walk from there." },
+                file_id: { type: "string", description: "Prep file id. **Prefer file_id+node_id over fen** when inside a prep file — the FEN comes from the tree AND the result is stored on the node." },
+                node_id: { type: "string", description: "Node id inside `file_id`. Root is 'r'. When set, overrides `fen`/`moves`." },
+                fen: { type: "string", description: "Starting position as FEN. Only used if `node_id` is not set." },
                 moves: {
                     type: "string",
-                    description: "Optional SAN moves to apply on top of `fen` (or on top of startpos). Example: fen='<tabiya>', moves='b4 a5 c3' analyses the position after those three moves.",
+                    description: "Optional SAN moves to apply on top of `fen` (or startpos). Only used if `node_id` is not set.",
                 },
                 movetime_ms: {
                     type: "integer",
@@ -446,7 +471,6 @@ const TOOLS = [
                     description: "Lc0 contempt bias. 0 = objective (default). Positive favours White, negative favours Black; stay within -20..+20 in practice. Not applied to Stockfish. See engine_usage_primer prompt for when to use.",
                 },
             },
-            required: ["fen"],
         },
     },
     {
@@ -467,8 +491,9 @@ const TOOLS = [
     },
     {
         name: "read_prep_file",
-        description: "Read one prep file. Returns a compact tree (path-addressable, one node per move) plus tags and the `version` for optimistic locking on subsequent mutations. NO raw PGN — all edits go through the mutation tools (add_move / set_comment / set_nags / set_annotations / delete_subtree / promote_variation / set_tag), which validate SAN and structure for you.\n\n" +
-            "Path addressing: paths are arrays of child indices. `[]` = root position. `[0]` = the first mainline move. `[0, 1]` = the second variation branching after the first move. `[0, 0, 0]` = three plies deep on the mainline. Every mutation returns the effective path it landed at.",
+        description: "Read one prep file. Returns a compact tree (each node carries a stable `id`, `san`, `fen`, `ply`, optional `comment`/`nags`/`annotations`/`ceoEval`, and `children`) plus tags and the `version` for optimistic locking on subsequent mutations. NO raw PGN — all edits go through the mutation tools (add_move / add_line / set_comment / set_nags / set_annotations / delete_subtree / promote_variation / set_tag), which validate SAN and structure for you.\n\n" +
+            "**Node addressing.** Every node has a stable `id` — root is `'r'`, every other node is an 8-hex-char content hash derived from its parent's id + its SAN. Sibling insertions, deletions and variation promotions do NOT change any other node's id. Pass this id as `node_id` (or `parent_id` for add_move / add_line) to every mutation and engine/DB tool.\n\n" +
+            "**Every engine/DB tool accepts `file_id`+`node_id`** (get_position_stats, cloud_analyse, describe_position, predict_human_move, prep_snapshot, get_player_preparation, quote_engine_eval). Use it whenever a file is open — the server derives the FEN from the tree, so you can't 'analyse the wrong position' by mis-typing a FEN.",
         inputSchema: {
             type: "object",
             properties: {
@@ -505,56 +530,71 @@ const TOOLS = [
     },
     {
         name: "add_move",
-        description: "Append a move as a new child of the node at `path`. If the node already has children, the new move becomes a variation (appended at the end). Use promote_variation afterwards if you want it to be the mainline. SAN is validated against the position — illegal moves are rejected with a clear error.\n\n" +
-            "Auto-saves. Returns `{path, version}` — the effective path the new node landed at (which you pass to follow-up set_comment / set_nags / etc.) and the new file version for optimistic locking on your next mutation.",
+        description: "Append a move as a new child of the node identified by `parent_id`. If the parent already has children, the new move becomes a variation (appended at the end); use promote_variation afterwards to make it the mainline. SAN is validated against the position — illegal moves are rejected with a clear error.\n\n" +
+            "Auto-saves. Returns `{node_id, version}` — the id of the new node (pass this to follow-up set_comment / set_nags / etc.) and the new file version for optimistic locking. **Node ids are content-derived and stable** — sibling insertions, deletions, and promotions do NOT change any other node's id.",
         inputSchema: {
             type: "object",
             properties: {
                 id: { type: "string", description: "Prep file id." },
-                path: { type: "array", items: { type: "integer", minimum: 0 }, description: "Path to the parent node the move is played FROM." },
+                parent_id: { type: "string", description: "Node id of the parent (the position the move is played FROM). Root id is 'r'." },
                 san: { type: "string", description: "The move in SAN notation (e.g. 'Nf3', 'exd5', 'O-O', 'Qxf7+')." },
                 expected_version: { type: "integer", description: "Optimistic-lock check; pass the `version` from your last read." },
             },
-            required: ["id", "path", "san"],
+            required: ["id", "parent_id", "san"],
+        },
+    },
+    {
+        name: "add_line",
+        description: "Append a linear sequence of moves under `parent_id`. Each SAN in the list becomes the mainline child of the previous — one call instead of N add_move calls for a straight variation. If the parent already has other children, this whole line is appended as a variation (promote_variation the first move if you want it as the mainline).\n\n" +
+            "Auto-saves. Returns `{node_id, line: [{node_id, san}, ...], version}` — `node_id` is the last (leaf) node's id, `line` is every node created in order so you can address any of them next.",
+        inputSchema: {
+            type: "object",
+            properties: {
+                id: { type: "string", description: "Prep file id." },
+                parent_id: { type: "string", description: "Node id of the parent to build from. Root is 'r'." },
+                sans: { type: "array", items: { type: "string" }, minItems: 1, description: "SAN moves in order, e.g. ['e4','e5','Nf3','Nc6','Bb5']." },
+                expected_version: { type: "integer" },
+            },
+            required: ["id", "parent_id", "sans"],
         },
     },
     {
         name: "set_comment",
-        description: "Set (or clear, with empty string) the text comment on the node at `path`. Comments are for plans, prep-signal, and interpretation the app can't derive — NOT for describing moves that should be variations instead. Auto-saves.",
+        description: "Set (or clear, with empty string) the text comment on the node identified by `node_id`. Comments are for plans, prep-signal, and interpretation the app can't derive — NOT for describing moves that should be variations instead. Auto-saves.",
         inputSchema: {
             type: "object",
             properties: {
                 id: { type: "string", description: "Prep file id." },
-                path: { type: "array", items: { type: "integer", minimum: 0 } },
+                node_id: { type: "string", description: "Node id from read_prep_file / add_move." },
                 comment: { type: "string", description: "New comment text. Empty string clears." },
                 expected_version: { type: "integer" },
             },
-            required: ["id", "path", "comment"],
+            required: ["id", "node_id", "comment"],
         },
     },
     {
         name: "set_nags",
-        description: "Replace the list of NAGs on the node at `path`. Empty array clears them. See read_pgn_authoring_guide for the full NAG table and eval→NAG thresholds (|eval|<0.25 → $10; <0.6 → $14/$15; <1.3 → $16/$17; ≥1.3 → $18/$19). Auto-saves.",
+        description: "Replace the list of NAGs on the node identified by `node_id`. Empty array clears them. NAGs are your EDITORIAL call — see read_pgn_authoring_guide for the discipline (novelty $146, sharp choice $5, decisive $18/$19, etc.). Do NOT set $10 '=' on every equal position; that's board noise. Auto-saves.",
         inputSchema: {
             type: "object",
             properties: {
                 id: { type: "string" },
-                path: { type: "array", items: { type: "integer", minimum: 0 } },
+                node_id: { type: "string" },
                 nags: { type: "array", items: { type: "string", pattern: "^\\$\\d+$" }, description: "NAG list, e.g. ['$14'] or ['$146', '$44']." },
                 expected_version: { type: "integer" },
             },
-            required: ["id", "path", "nags"],
+            required: ["id", "node_id", "nags"],
         },
     },
     {
         name: "set_annotations",
-        description: "Replace the visual annotations (arrows + coloured squares) on the node at `path`. Passing empty arrays clears them.\n\n" +
+        description: "Replace the visual annotations (arrows + coloured squares) on the node identified by `node_id`. Passing empty arrays clears them.\n\n" +
             "Colours: green, red, yellow, light-blue, dark-blue, orange. Keep it LIGHT: 1-3 arrows and 2-3 squares per move maximum. Twenty arrows is noise, not signal. Auto-saves.",
         inputSchema: {
             type: "object",
             properties: {
                 id: { type: "string" },
-                path: { type: "array", items: { type: "integer", minimum: 0 } },
+                node_id: { type: "string" },
                 arrows: {
                     type: "array",
                     items: {
@@ -580,40 +620,40 @@ const TOOLS = [
                 },
                 expected_version: { type: "integer" },
             },
-            required: ["id", "path"],
+            required: ["id", "node_id"],
         },
     },
     {
         name: "delete_subtree",
-        description: "Delete the node at `path` and all its descendants. Refuses to delete the root. Auto-saves.",
+        description: "Delete the node identified by `node_id` and all its descendants. Refuses to delete the root. Auto-saves.",
         inputSchema: {
             type: "object",
             properties: {
                 id: { type: "string" },
-                path: { type: "array", items: { type: "integer", minimum: 0 } },
+                node_id: { type: "string" },
                 expected_version: { type: "integer" },
             },
-            required: ["id", "path"],
+            required: ["id", "node_id"],
         },
     },
     {
         name: "promote_variation",
-        description: "Make the node at `path` its parent's mainline (children[0]), demoting the current mainline (and any other siblings) into variation order. Silently no-op if already the mainline. Auto-saves.",
+        description: "Make the node identified by `node_id` its parent's mainline (children[0]), demoting the current mainline (and any other siblings) into variation order. Silently no-op if already the mainline. Auto-saves.",
         inputSchema: {
             type: "object",
             properties: {
                 id: { type: "string" },
-                path: { type: "array", items: { type: "integer", minimum: 0 }, description: "Path to the node to promote. Length ≥ 1." },
+                node_id: { type: "string", description: "Node id of the variation to promote. Cannot be root." },
                 expected_version: { type: "integer" },
             },
-            required: ["id", "path"],
+            required: ["id", "node_id"],
         },
     },
     {
         name: "apply_mutations",
         description: "Batch: apply a list of mutations in one call. One load-parse-mutate-export-save cycle for N ops, so building a 100-move repertoire costs one HTTP round-trip and one save instead of 100. This is the RIGHT way to build a file — use single mutations only for surgical follow-up edits.\n\n" +
-            "Each mutation is `{op, path, ...args}` where `op` is one of: add_move, set_comment, set_nags, set_annotations, delete_subtree, promote_variation, set_tag. Same arg shape as the individual tools. Ops apply in order; `path` is resolved against the tree AFTER preceding ops in the batch — so after `{op: 'add_move', path: [0], san: 'Nc3'}` the new node's path is `[0, N]` where N is the parent's previous children count (or `[0, 0]` if it was empty).\n\n" +
-            "Any op error aborts the batch (nothing saved). Response is `{ok, results: [{path}], version}` giving the effective path each op landed at plus the new file version.",
+            "Each mutation is `{op, node_id | parent_id, ...args}` where `op` is one of: add_move, add_line, set_comment, set_nags, set_annotations, delete_subtree, promote_variation, set_tag. Same arg shape as the individual tools. Ops apply in order; because node ids are content-derived (hash of parent_id + san), a node created by an early op has a deterministic id you can reference in later ops in the same batch.\n\n" +
+            "Any op error aborts the batch (nothing saved). Response is `{ok, results: [{node_id, line?}], version}` — one entry per op with the id it landed on (add_line also returns the full line array).",
         inputSchema: {
             type: "object",
             properties: {
@@ -625,9 +665,11 @@ const TOOLS = [
                     items: {
                         type: "object",
                         properties: {
-                            op: { type: "string", enum: ["add_move", "set_comment", "set_nags", "set_annotations", "delete_subtree", "promote_variation", "set_tag"] },
-                            path: { type: "array", items: { type: "integer", minimum: 0 } },
+                            op: { type: "string", enum: ["add_move", "add_line", "set_comment", "set_nags", "set_annotations", "delete_subtree", "promote_variation", "set_tag"] },
+                            node_id: { type: "string" },
+                            parent_id: { type: "string" },
                             san: { type: "string" },
+                            sans: { type: "array", items: { type: "string" } },
                             comment: { type: "string" },
                             nags: { type: "array", items: { type: "string", pattern: "^\\$\\d+$" } },
                             arrows: { type: "array" },
@@ -644,21 +686,35 @@ const TOOLS = [
     },
     {
         name: "auto_evaluate",
-        description: "Walk the tree from `path` (default `[]` = whole file) and auto-assign NAGs to every node based on the Stockfish eval from cloud_analyse. Uses the standard eval → NAG thresholds (|eval|<0.25 → $10, <0.6 → $14/$15, <1.3 → $16/$17, ≥1.3 → $18/$19). Requires a running cloud combo instance.\n\n" +
-            "This is the automate-the-boring-part tool. Instead of hand-annotating each move with a NAG, you build the tree via apply_mutations (no NAGs needed), then call auto_evaluate once and every node gets the right glyph.\n\n" +
-            "Costs real money — hits cloud_analyse per node. A 200-node tree at 1.5s/node = ~5 min of engine time. Use `only_missing=true` (default) to skip nodes that already have a NAG on subsequent runs. Runs 4 evaluations in parallel to save wall time.",
+        description: "Walk the tree from `node_id` (default `'r'` = whole file) and populate the persistent `ceoEval` (Stockfish + Lc0 numbers) on every descendant via cloud_analyse. Requires a running cloud combo instance.\n\n" +
+            "**Does NOT set visible NAGs.** NAG placement is your call, not the engine's — an opening tree full of 0.00 positions doesn't need a `$10` (=) glyph on every move (that's just noise on the board), and a `!` on a novelty or a `?!` on a risky committal is a judgment call the engine can't make. Use this tool to persist the raw numbers on every node, then re-read the file and hand-pick NAGs on the moves where a glyph carries real signal.\n\n" +
+            "On re-read every evaluated node carries `ceoEval: { sf: {cp, depth}, lc0: {cp, depth} }` — the numbers travel with the file. Read those back with quote_engine_eval before writing prose that references engine numbers. Use `only_missing=true` (default) to skip nodes already evaluated on repeat runs.\n\n" +
+            "Costs real money — hits cloud_analyse per node. A 200-node tree at 1.5s/node = ~5 min of engine time. Runs 4 evaluations in parallel to save wall time.",
         inputSchema: {
             type: "object",
             properties: {
                 id: { type: "string" },
-                path: { type: "array", items: { type: "integer", minimum: 0 }, description: "Subtree root (default = whole file)." },
-                only_missing: { type: "boolean", description: "Skip nodes that already carry a NAG (default true)." },
+                node_id: { type: "string", description: "Subtree root (default 'r' = whole file)." },
+                only_missing: { type: "boolean", description: "Skip nodes that already carry a stored ceoEval (default true)." },
                 movetime_ms: { type: "integer", minimum: 500, maximum: 5000, description: "Per-node cloud_analyse think time (default 1500)." },
                 expected_version: { type: "integer" },
             },
             required: ["id"],
         },
     },
+    {
+        name: "quote_engine_eval",
+        description: "Return the stored engine eval for a node, or null if that node was never analysed. **Call this before writing prose or NAGs that quote engine numbers** — if it returns null, you have no measurement to cite. Do NOT infer an eval for the node from siblings or children; either analyse it (cloud_analyse with node_id) or omit the number from your prose.\n\n" +
+            "Response: `{ ceoEval: { sf: {cp, depth}, lc0: {cp, depth}, nag } | null }`. `cp` is White-POV centipawns as an integer (+20 = +0.20). `nag` is the threshold-derived glyph as a SUGGESTION — promote to a visible NAG via set_nags only when a glyph on that move carries editorial signal.",
+        inputSchema: {
+            type: "object",
+            properties: {
+                id: { type: "string", description: "Prep file id." },
+                node_id: { type: "string", description: "Node id whose stored eval you want to quote." },
+            },
+            required: ["id", "node_id"],
+        },
+    },
     {
         name: "set_tag",
         description: "Set or clear a game-level PGN tag (Event, Site, Date, White, Black, Result, or any custom tag). Passing empty string removes the tag. Auto-saves.",
@@ -703,13 +759,15 @@ const TOOLS = [
                 fide_id_me: { type: "integer", description: "Your FIDE ID." },
                 fide_id_opponent: { type: "integer", description: "Opponent's FIDE ID." },
                 my_color: { type: "string", enum: ["white", "black"], description: "The colour YOU will play." },
+                file_id: { type: "string", description: "Prep file id. **Prefer file_id+node_id** when inside a prep file." },
+                node_id: { type: "string", description: "Node id inside `file_id`. When set, overrides `line`/`fen`." },
                 line: {
                     type: "string",
-                    description: "Move sequence in SAN, space-separated. Empty = starting position. Example: 'e4 c5 Nf3'. Either line or fen (or neither for the starting position).",
+                    description: "Move sequence in SAN, space-separated. Empty = starting position. Only used if `node_id` is not set.",
                 },
                 fen: {
                     type: "string",
-                    description: "Alternative to line — raw FEN of the target position.",
+                    description: "Alternative to line — raw FEN of the target position. Only used if `node_id` is not set.",
                 },
             },
             required: ["fide_id_me", "fide_id_opponent", "my_color"],
@@ -798,56 +856,70 @@ function convertCloudSnapshotResponse(raw, startFen) {
     }
     return raw;
 }
-// Path arg parsing for the mutation tools. Rejects malformed input
-// with a clear message the LLM can act on.
-function argPath(args) {
-    const raw = args.path;
-    if (!Array.isArray(raw))
-        throw new Error("`path` must be an array of non-negative integers");
-    const out = [];
-    for (let i = 0; i < raw.length; i++) {
-        const n = raw[i];
-        if (typeof n !== "number" || !Number.isInteger(n) || n < 0) {
-            throw new Error(`path[${i}] must be a non-negative integer (got ${JSON.stringify(n)})`);
-        }
-        out.push(n);
+// Extract a node id from the args. Accepts either `node_id` or a
+// `parent_id` alias for the add-style tools. Throws with a helpful
+// message if malformed.
+function argNodeId(args, key = "node_id") {
+    const raw = args[key];
+    if (typeof raw !== "string" || raw.length === 0) {
+        throw new Error(`\`${key}\` is required (call read_prep_file to get valid node ids)`);
     }
-    return out;
+    return raw.trim();
 }
-// Dispatch table for the batch tool: name → mutator that returns the
-// standard { file, path } shape. Reused for individual and batch calls.
-function dispatchMutation(file, op) {
+// Dispatch table for the batch tool: name → mutator that returns
+// { file, id } where id is the node the mutation touched. The batch
+// caller rebuilds the id → path index between ops so newly-created
+// nodes are addressable within the same batch.
+function dispatchMutation(file, idIndex, op) {
     const kind = String(op.op);
-    const path = Array.isArray(op.path) ? op.path : [];
+    // Small local helper — resolves a node_id (or parent_id) op field to
+    // a path against the CURRENT tree state.
+    const nodeIdField = (key) => {
+        const raw = op[key];
+        if (typeof raw !== "string" || raw.length === 0) {
+            throw new Error(`\`${key}\` required on op ${kind}`);
+        }
+        return raw.trim();
+    };
+    const resolve = (id) => resolveNodeId(idIndex, id);
     switch (kind) {
         case "add_move":
-            return addMove(file, path, String(op.san));
+            return addMove(file, resolve(nodeIdField("parent_id")), String(op.san));
+        case "add_line": {
+            const sans = Array.isArray(op.sans) ? op.sans.map(String) : [];
+            const parentPath = resolve(nodeIdField("parent_id"));
+            const step = addLine(file, parentPath, sans);
+            const lastId = step.line.length > 0 ? step.line[step.line.length - 1].id : nodeIdField("parent_id");
+            return { file: step.file, id: lastId, results: step.line };
+        }
         case "set_comment":
-            return setComment(file, path, typeof op.comment === "string" ? op.comment : "");
+            return setComment(file, resolve(nodeIdField("node_id")), typeof op.comment === "string" ? op.comment : "");
         case "set_nags":
-            return setNags(file, path, Array.isArray(op.nags) ? op.nags.map(String) : []);
+            return setNags(file, resolve(nodeIdField("node_id")), Array.isArray(op.nags) ? op.nags.map(String) : []);
         case "set_annotations": {
             const arrows = Array.isArray(op.arrows) ? op.arrows : [];
             const highlights = Array.isArray(op.highlights) ? op.highlights : [];
             const ann = arrows.length === 0 && highlights.length === 0 ? null : { arrows, highlights };
-            return setAnnotations(file, path, ann);
+            return setAnnotations(file, resolve(nodeIdField("node_id")), ann);
         }
         case "set_ceo_eval": {
             const ev = op.ceoEval;
-            return setCeoEval(file, path, ev ?? null);
+            return setCeoEval(file, resolve(nodeIdField("node_id")), ev ?? null);
         }
         case "delete_subtree":
-            return deleteSubtree(file, path);
+            return deleteSubtree(file, resolve(nodeIdField("node_id")));
         case "promote_variation":
-            return promoteVariation(file, path);
+            return promoteVariation(file, resolve(nodeIdField("node_id")));
         case "set_tag":
-            return { file: setTag(file, String(op.key), String(op.value ?? "")), path: [] };
+            return { file: setTag(file, String(op.key), String(op.value ?? "")), id: ROOT_ID };
         default:
             throw new Error(`unknown mutation op: ${kind}`);
     }
 }
-// Batch: load, parse, apply N mutations in order, export, save. All-or-nothing —
-// any error aborts and nothing is saved.
+// Batch: load, parse, apply N mutations in order, export, save.
+// All-or-nothing — any error aborts and nothing is saved. The id index
+// is rebuilt after each op so nodes created earlier in the batch can be
+// addressed by later ops via their newly-derived node_id.
 async function applyBatchMutations(args) {
     const id = String(args.id);
     const mutations = Array.isArray(args.mutations) ? args.mutations : [];
@@ -858,13 +930,15 @@ async function applyBatchMutations(args) {
     if (typeof g.pgnContent !== "string")
         throw new Error("prep file missing pgnContent");
     let file = parsePGN(g.pgnContent);
+    let idIndex = buildIdIndex(file.root);
     const results = [];
     for (let i = 0; i < mutations.length; i++) {
         const op = mutations[i];
         try {
-            const step = dispatchMutation(file, op);
+            const step = dispatchMutation(file, idIndex, op);
             file = step.file;
-            results.push({ path: step.path });
+            idIndex = buildIdIndex(file.root);
+            results.push({ node_id: step.id, ...(step.results !== undefined ? { line: step.results } : {}) });
         }
         catch (err) {
             const msg = err instanceof Error ? err.message : String(err);
@@ -887,7 +961,9 @@ async function applyBatchMutations(args) {
 // mutation at the end so a 200-node evaluate is one save.
 async function autoEvaluate(args) {
     const id = String(args.id);
-    const startPath = Array.isArray(args.path) ? args.path : [];
+    const startNodeId = typeof args.node_id === "string" && args.node_id.length > 0
+        ? String(args.node_id)
+        : ROOT_ID;
     const onlyMissing = args.only_missing !== false; // default true
     const movetimeMs = typeof args.movetime_ms === "number" ? args.movetime_ms : 1500;
     const raw = await authedRequest("GET", `/api/agent/prep-files/${encodeURIComponent(id)}`);
@@ -895,18 +971,22 @@ async function autoEvaluate(args) {
     if (typeof g.pgnContent !== "string")
         throw new Error("prep file missing pgnContent");
     const file = parsePGN(g.pgnContent);
+    const idIndex = buildIdIndex(file.root);
+    const startPath = resolveNodeId(idIndex, startNodeId);
     const targets = [];
-    const walk = (node, path) => {
-        if (path.length > 0) {
+    const walk = (node, isStartAndRoot) => {
+        if (!isStartAndRoot) {
             if (!onlyMissing || !node.ceoEval) {
-                targets.push({ path, fen: node.fen });
+                targets.push({ nodeId: node.id, fen: node.fen });
             }
         }
-        for (let i = 0; i < node.children.length; i++)
-            walk(node.children[i], [...path, i]);
+        for (const child of node.children)
+            walk(child, false);
     };
-    const startNode = pathIntoTree(file.root, startPath);
-    walk(startNode, startPath);
+    const startNode = getNodeByPath(file.root, startPath);
+    // If the caller anchored at the root, skip evaluating the root itself
+    // (no move); otherwise the anchor node IS a real move and gets evaluated.
+    walk(startNode, startNode.id === ROOT_ID);
     if (targets.length === 0)
         return { ok: true, evaluated: 0, skipped: 0, version: g.version };
     // Dispatch cloud_analyse in parallel with a 4-way cap so we don't
@@ -922,7 +1002,7 @@ async function autoEvaluate(args) {
                 const analysis = await authedRequest("POST", "/api/agent/cloud-engines/analyse", { fen: t.fen, movetime_ms: movetimeMs, multipv: 1 });
                 const ev = analysisToStoredEval(analysis, t.fen);
                 if (ev)
-                    stored.push({ path: t.path, ev });
+                    stored.push({ nodeId: t.nodeId, ev });
             }
             catch {
                 // best-effort — a single failed node shouldn't kill the batch
@@ -932,11 +1012,16 @@ async function autoEvaluate(args) {
     await Promise.all(Array.from({ length: CONCURRENCY }, () => worker()));
     if (stored.length === 0)
         return { ok: true, evaluated: 0, skipped: targets.length, version: g.version };
+    // Persist the raw numbers only — never touch visible NAGs. NAG glyphs
+    // are the LLM's editorial call (novelty, sharp choice, real mistake),
+    // not an automatic mapping from the engine number. Stamping `$10` on
+    // every 0.00 position in an opening tree just clutters the board.
+    // The threshold-derived NAG still lives INSIDE `ceoEval.nag` so the
+    // LLM can read it back and decide whether to promote it to a visible
+    // NAG on a case-by-case basis.
     const batchMutations = [];
     for (const s of stored) {
-        batchMutations.push({ op: "set_ceo_eval", path: s.path, ceoEval: s.ev });
-        if (s.ev.nag)
-            batchMutations.push({ op: "set_nags", path: s.path, nags: [s.ev.nag] });
+        batchMutations.push({ op: "set_ceo_eval", node_id: s.nodeId, ceoEval: s.ev });
     }
     const saveResult = await applyBatchMutations({ id, mutations: batchMutations, expected_version: g.version });
     const sr = saveResult;
@@ -1031,7 +1116,9 @@ function storedEvalToCompact(ev, analysis) {
 }
 // Load-mutate-save: fetch current PGN, parse, apply mutation, re-export,
 // save with optimistic lock. Auto-saves so every tool call is atomic;
-// the LLM never sees intermediate state.
+// the LLM never sees intermediate state. The mutator is called with
+// both the parsed file and its id → path index, so the mutation can
+// resolve node_ids without rebuilding the index itself.
 async function applyMutation(args, mutator) {
     const id = String(args.id);
     const raw = await authedRequest("GET", `/api/agent/prep-files/${encodeURIComponent(id)}`);
@@ -1039,12 +1126,13 @@ async function applyMutation(args, mutator) {
     if (typeof g.pgnContent !== "string")
         throw new Error("prep file missing pgnContent");
     const file = parsePGN(g.pgnContent);
+    const idIndex = buildIdIndex(file.root);
     let result;
     try {
-        result = mutator(file);
+        result = mutator(file, idIndex);
     }
     catch (err) {
-        if (err instanceof MutationError || err instanceof PathError) {
+        if (err instanceof MutationError || err instanceof PathError || err instanceof NodeIdError) {
             throw new Error(`mutation rejected: ${err.message}`);
         }
         throw err;
@@ -1058,10 +1146,52 @@ async function applyMutation(args, mutator) {
     const savedRow = saved;
     return {
         ok: true,
-        path: result.path,
+        node_id: result.id,
+        ...(result.results !== undefined ? { line: result.results } : {}),
         version: savedRow.version,
     };
 }
+// Strip cruft the LLM doesn't need from the DB-position response.
+// Called AFTER trimGamesMovetext so plyNumber survives long enough to
+// slice each game's movetext. Also renames the `transpositions` field
+// to something the LLM can parse without knowing chess-DB jargon.
+function stripPositionResponse(r) {
+    if (!r || typeof r !== "object")
+        return;
+    const t = r;
+    delete t.hash; // internal zobrist string
+    delete t.source; // internal "database" marker; we overwrite with our own .source
+    delete t.totalGames; // duplicates statistics.totalCount often; hasMore covers pagination
+    if (Array.isArray(t.moves)) {
+        for (const m of t.moves) {
+            if (typeof m.transpositions === "number") {
+                m.reachedViaTransposition = m.transpositions;
+                delete m.transpositions;
+            }
+            // Backend calls it "hotness" — a 0-100 time-decayed popularity score
+            // (recent + played often = high). Rename to something an LLM can read
+            // without guessing it means "on a winning streak".
+            if (typeof m.hotness === "number") {
+                m.fashionScore = m.hotness;
+                delete m.hotness;
+            }
+        }
+    }
+    if (Array.isArray(t.games)) {
+        for (const g of t.games) {
+            delete g.gameId;
+            delete g.whiteTitle;
+            delete g.blackTitle;
+            delete g.whiteTeam;
+            delete g.blackTeam;
+            delete g.round;
+            delete g.plyNumber;
+            delete g.relevance;
+            delete g.site;
+            delete g.ply;
+        }
+    }
+}
 // Trim every game's `moves` field to just the plies AFTER the queried
 // position, using each game's `plyNumber`. Massive token save — a game
 // 80 plies long queried at ply 12 drops to ~68 plies of movetext. Ports
@@ -1168,6 +1298,65 @@ function resolveFenFromArgs(args) {
     }
     return board.fen();
 }
+// Async resolver used by every engine / DB tool. If the caller supplied
+// file_id + node_id (preferred), load the file, resolve the node, and
+// return both the FEN and a handle we can use to persist ceoEval later.
+// Otherwise fall back to the raw fen/moves/line inputs.
+async function resolveFromNodeOrFen(args) {
+    const fileId = typeof args.file_id === "string" ? args.file_id.trim() : "";
+    const nodeId = typeof args.node_id === "string" ? args.node_id.trim() : "";
+    if (fileId && nodeId) {
+        const raw = await authedRequest("GET", `/api/agent/prep-files/${encodeURIComponent(fileId)}`);
+        const g = raw;
+        if (typeof g.pgnContent !== "string")
+            throw new Error("prep file missing pgnContent");
+        const parsedFile = parsePGN(g.pgnContent);
+        const idIndex = buildIdIndex(parsedFile.root);
+        const nodePath = resolveNodeId(idIndex, nodeId);
+        const node = getNodeByPath(parsedFile.root, nodePath);
+        return {
+            fen: node.fen,
+            file: {
+                id: fileId,
+                version: g.version ?? 0,
+                parsedFile,
+                idIndex,
+                nodePath,
+                fen: node.fen,
+            },
+        };
+    }
+    return { fen: resolveFenFromArgs(args) };
+}
+// Local wrapper — the mutation module re-exports paths.getNode so this
+// import stays consistent with the rest of the file's imports.
+function getNodeByPath(root, path) {
+    let cur = root;
+    for (const idx of path) {
+        if (idx < 0 || idx >= cur.children.length)
+            throw new Error(`invalid node path segment ${idx}`);
+        cur = cur.children[idx];
+    }
+    return cur;
+}
+// Persist a fresh ceoEval on the node referenced by the file handle.
+// Best-effort — if the file version raced (another agent saved
+// between our GET and our PUT), we silently drop the store rather
+// than fail the analysis the LLM actually asked for. The eval is
+// still returned in the response either way.
+async function storeEvalOnNode(handle, ev) {
+    try {
+        const step = setCeoEval(handle.parsedFile, handle.nodePath, ev);
+        const newPgn = exportPGN(step.file);
+        await authedRequest("PUT", `/api/agent/prep-files/${encodeURIComponent(handle.id)}`, {
+            pgn: newPgn,
+            expected_version: handle.version,
+        });
+    }
+    catch {
+        // Best-effort — the analysis result is what the LLM asked for.
+    }
+}
 function stringifyForLog(v) {
     let s;
     try {
@@ -1204,31 +1393,21 @@ async function callToolInner(name, args) {
         case "get_player_profile":
             return get("/api/chess/players/profile", { fideId: Number(args.fide_id) });
         case "get_player_preparation": {
-            // Prefer sending `line` to the backend when the caller specified moves
-            // WITHOUT a fen — the backend then attaches the cumulative-line SAN to
-            // each move in the response (nicer for the LLM's follow-up walks).
-            // When a fen was supplied, always resolve the effective position
-            // client-side and send just fen — the backend can't reconstruct
-            // history it didn't see anyway.
-            const fenArg = typeof args.fen === "string" ? args.fen.trim() : "";
-            const movesArg = typeof args.moves === "string" ? args.moves.trim() : "";
-            const lineArg = typeof args.line === "string" ? args.line.trim() : "";
+            // Node-first: if file_id+node_id was given, derive the FEN from the
+            // tree (never trust an LLM-supplied FEN when a node reference is
+            // available). Otherwise fall back to fen/moves/line.
+            const resolved = await resolveFromNodeOrFen(args);
+            const effectiveFen = resolved.fen;
             const params = {
                 fideId: Number(args.fide_id),
                 color: String(args.color),
                 compact: "true",
+                fen: effectiveFen,
             };
-            if (fenArg || movesArg) {
-                params.fen = resolveFenFromArgs(args);
-            }
-            else if (lineArg) {
-                params.line = lineArg;
-            }
             if (typeof args.limit === "number")
                 params.limit = args.limit;
             if (typeof args.offset === "number")
                 params.offset = args.offset;
-            const effectiveFen = resolveFenFromArgs(args);
             const [raw, ev] = await Promise.all([
                 get("/api/chess/prep/by-player", params),
                 fetchCompactEval(effectiveFen),
@@ -1236,13 +1415,15 @@ async function callToolInner(name, args) {
             const converted = convertAvailableMovesToSAN(raw, effectiveFen);
             if (converted && typeof converted === "object") {
                 trimGamesMovetext(converted);
+                stripPositionResponse(converted);
                 if (ev)
                     converted.eval = ev;
             }
             return converted;
         }
         case "get_position_stats": {
-            const fen = resolveFenFromArgs(args);
+            const resolved = await resolveFromNodeOrFen(args);
+            const fen = resolved.fen;
             const rawSource = typeof args.source === "string" ? args.source : "gm-classical";
             const source = rawSource === "main" ? "main" : "gm-classical";
             const [raw, ev] = await Promise.all([
@@ -1256,16 +1437,20 @@ async function callToolInner(name, args) {
             const converted = convertAvailableMovesToSAN(raw, fen);
             if (converted && typeof converted === "object") {
                 trimGamesMovetext(converted);
+                stripPositionResponse(converted);
                 converted.source = source;
                 if (ev)
                     converted.eval = ev;
             }
             return converted;
         }
-        case "describe_position":
-            return describePosition(resolveFenFromArgs(args));
+        case "describe_position": {
+            const resolved = await resolveFromNodeOrFen(args);
+            return describePosition(resolved.fen);
+        }
         case "predict_human_move": {
-            const fen = resolveFenFromArgs(args);
+            const resolved = await resolveFromNodeOrFen(args);
+            const fen = resolved.fen;
             const qs = new URLSearchParams();
             qs.set("fen", fen);
             if (typeof args.white_elo === "number")
@@ -1307,7 +1492,8 @@ async function callToolInner(name, args) {
         case "stop_cloud_engine":
             return authedRequest("DELETE", `/api/agent/cloud-engines/${encodeURIComponent(String(args.contract_id))}`);
         case "cloud_analyse": {
-            const fen = resolveFenFromArgs(args);
+            const resolved = await resolveFromNodeOrFen(args);
+            const fen = resolved.fen;
             const body = { fen };
             if (typeof args.movetime_ms === "number")
                 body.movetime_ms = args.movetime_ms;
@@ -1316,7 +1502,19 @@ async function callToolInner(name, args) {
             if (typeof args.contempt === "number")
                 body.contempt = args.contempt;
             const raw = await authedRequest("POST", "/api/agent/cloud-engines/analyse", body);
-            return convertCloudSnapshotResponse(raw, fen);
+            const converted = convertCloudSnapshotResponse(raw, fen);
+            // Node-addressed calls: persist the result on the node's ceoEval
+            // so a later quote_engine_eval can cite this measurement. This is
+            // the anti-hallucination hinge — prose that says "engines say X
+            // on node Y" can only trace back to a call actually made against
+            // node_id=Y, because the store only fires when file_id+node_id
+            // was supplied and the eval survives via the [%ceo-eval] escape.
+            if (resolved.file) {
+                const ev = analysisToStoredEval(converted, fen);
+                if (ev)
+                    await storeEvalOnNode(resolved.file, ev);
+            }
+            return converted;
         }
         case "list_prep_files":
             return authedRequest("GET", "/api/agent/prep-files");
@@ -1342,32 +1540,53 @@ async function callToolInner(name, args) {
         case "delete_prep_file":
             return authedRequest("DELETE", `/api/agent/prep-files/${encodeURIComponent(String(args.id))}`);
         case "add_move":
-            return applyMutation(args, file => addMove(file, argPath(args), String(args.san)));
+            return applyMutation(args, (file, idIndex) => addMove(file, resolveNodeId(idIndex, argNodeId(args, "parent_id")), String(args.san)));
+        case "add_line":
+            return applyMutation(args, (file, idIndex) => {
+                const sans = Array.isArray(args.sans) ? args.sans.map(String) : [];
+                const parentPath = resolveNodeId(idIndex, argNodeId(args, "parent_id"));
+                const step = addLine(file, parentPath, sans);
+                const lastId = step.line.length > 0 ? step.line[step.line.length - 1].id : argNodeId(args, "parent_id");
+                return { file: step.file, id: lastId, results: step.line };
+            });
         case "set_comment":
-            return applyMutation(args, file => setComment(file, argPath(args), typeof args.comment === "string" ? args.comment : ""));
+            return applyMutation(args, (file, idIndex) => setComment(file, resolveNodeId(idIndex, argNodeId(args)), typeof args.comment === "string" ? args.comment : ""));
         case "set_nags":
-            return applyMutation(args, file => setNags(file, argPath(args), Array.isArray(args.nags) ? args.nags.map(String) : []));
+            return applyMutation(args, (file, idIndex) => setNags(file, resolveNodeId(idIndex, argNodeId(args)), Array.isArray(args.nags) ? args.nags.map(String) : []));
         case "set_annotations": {
             const arrowsRaw = Array.isArray(args.arrows) ? args.arrows : [];
             const highlightsRaw = Array.isArray(args.highlights) ? args.highlights : [];
             const ann = (arrowsRaw.length === 0 && highlightsRaw.length === 0)
                 ? null
                 : { arrows: arrowsRaw, highlights: highlightsRaw };
-            return applyMutation(args, file => setAnnotations(file, argPath(args), ann));
+            return applyMutation(args, (file, idIndex) => setAnnotations(file, resolveNodeId(idIndex, argNodeId(args)), ann));
         }
         case "delete_subtree":
-            return applyMutation(args, file => deleteSubtree(file, argPath(args)));
+            return applyMutation(args, (file, idIndex) => deleteSubtree(file, resolveNodeId(idIndex, argNodeId(args))));
         case "promote_variation":
-            return applyMutation(args, file => promoteVariation(file, argPath(args)));
+            return applyMutation(args, (file, idIndex) => promoteVariation(file, resolveNodeId(idIndex, argNodeId(args))));
         case "set_tag":
             return applyMutation(args, file => ({
                 file: setTag(file, String(args.key), String(args.value ?? "")),
-                path: [],
+                id: ROOT_ID,
             }));
         case "apply_mutations":
             return applyBatchMutations(args);
         case "auto_evaluate":
             return autoEvaluate(args);
+        case "quote_engine_eval": {
+            const fileId = String(args.id);
+            const nodeId = argNodeId(args);
+            const raw = await authedRequest("GET", `/api/agent/prep-files/${encodeURIComponent(fileId)}`);
+            const g = raw;
+            if (typeof g.pgnContent !== "string")
+                throw new Error("prep file missing pgnContent");
+            const file = parsePGN(g.pgnContent);
+            const idIndex = buildIdIndex(file.root);
+            const path = resolveNodeId(idIndex, nodeId);
+            const node = getNodeByPath(file.root, path);
+            return { ceoEval: node.ceoEval ?? null };
+        }
         case "read_engine_usage_guide":
             return { guide: ENGINE_USAGE_DOC };
         case "read_prep_strategy_guide":
@@ -1381,35 +1600,15 @@ async function callToolInner(name, args) {
             const opp = Number(args.fide_id_opponent);
             const myColor = String(args.my_color);
             const oppColor = myColor === "white" ? "black" : "white";
-            const line = typeof args.line === "string" ? args.line.trim() : "";
-            let fen = typeof args.fen === "string" ? args.fen.trim() : "";
-            // General DB lookup needs a FEN. If we only have a line, compute it
-            // locally with chess.js — one dep, keeps the three data-fetches truly
-            // parallel instead of doing a preliminary round-trip.
-            if (!fen) {
-                const board = new Chess();
-                if (line.length > 0) {
-                    for (const raw of line.split(/\s+/)) {
-                        // Tolerant of move-number tokens like "1." / "12..." that some
-                        // clients include; chess.js rejects those outright.
-                        const san = raw.replace(/^\d+\.+/, "");
-                        if (!san)
-                            continue;
-                        try {
-                            board.move(san);
-                        }
-                        catch {
-                            throw new Error(`bad SAN token '${raw}' in line`);
-                        }
-                    }
-                }
-                fen = board.fen();
-            }
+            // Prefer file_id+node_id — derives FEN from the tree, no LLM-typed
+            // FEN in the loop. Falls back to line/fen for scratch positions.
+            const resolved = await resolveFromNodeOrFen(args);
+            const fen = resolved.fen;
             const prepParams = (fideId, color) => ({
                 fideId,
                 color,
                 compact: "true",
-                ...(line.length > 0 ? { line } : { fen }),
+                fen,
             });
             const [opponent, you, general, ev] = await Promise.all([
                 get("/api/chess/prep/by-player", prepParams(opp, oppColor)),
@@ -1418,7 +1617,7 @@ async function callToolInner(name, args) {
                 fetchCompactEval(fen),
             ]);
             return {
-                position: { line, fen, my_color: myColor },
+                position: { fen, my_color: myColor, ...(resolved.file ? { node_id: typeof args.node_id === "string" ? args.node_id : undefined } : {}) },
                 eval: ev,
                 opponent: convertAvailableMovesToSAN(opponent, fen),
                 you: convertAvailableMovesToSAN(you, fen),