From 0cef17f431396349205fb8cef53f5b3bae1618d1 Mon Sep 17 00:00:00 2001
From: "DESKTOP-02FN3OU\\80423" <804235820@qq.com>
Date: Fri, 29 May 2026 19:11:54 +0800
Subject: [PATCH 01/15] feat: improve file edit tool && chatui tool result
 display

---
 .../core/tools/computer_tools/edit_engine.py  | 651 ++++++++++++++++++
 astrbot/core/tools/computer_tools/fs.py       | 188 ++++-
 .../src/components/chat/ReasoningSidebar.vue  |  95 ++-
 dashboard/src/components/chat/ThreadPanel.vue |  83 ++-
 .../chat/message_list_comps/DiffPreview.vue   | 512 ++++++++++++++
 .../chat/message_list_comps/ToolCallCard.vue  | 279 ++++++--
 .../message_list_comps/ToolResultView.vue     | 485 +++++++++++++
 .../shared/ThemeAwareMarkdownCodeBlock.vue    |   8 +
 dashboard/src/utils/shikiLimitedBundle.js     |  15 +
 9 files changed, 2229 insertions(+), 87 deletions(-)
 create mode 100644 astrbot/core/tools/computer_tools/edit_engine.py
 create mode 100644 dashboard/src/components/chat/message_list_comps/DiffPreview.vue
 create mode 100644 dashboard/src/components/chat/message_list_comps/ToolResultView.vue

diff --git a/astrbot/core/tools/computer_tools/edit_engine.py b/astrbot/core/tools/computer_tools/edit_engine.py
new file mode 100644
index 0000000000..539f13056e
--- /dev/null
+++ b/astrbot/core/tools/computer_tools/edit_engine.py
@@ -0,0 +1,651 @@
+"""
+Robust file editing engine for AstrBot, inspired by opencode's multi-strategy replacer chain.
+
+Implements 9 fallback replacers to handle LLM-generated edits that may have:
+- indentation drift
+- whitespace normalization issues
+- escape sequence mismatches (\\n vs actual newline)
+- trailing/leading whitespace differences
+- block-level fuzzy matching via Levenshtein similarity
+
+"""
+
+from __future__ import annotations
+
+import asyncio
+import difflib
+import os
+import weakref
+from collections.abc import Callable, Iterator
+from dataclasses import dataclass
+from pathlib import Path
+from typing import Literal
+
+Replacer = Callable[[str, str], Iterator[str]]
+
+# File-level locks to prevent concurrent edits on the same file.
+# Use WeakValueDictionary so locks for deleted files can be garbage-collected.
+_locks: weakref.WeakValueDictionary[str, asyncio.Lock] = weakref.WeakValueDictionary()
+_locks_lock = asyncio.Lock()
+
+
+async def _get_lock(path: str) -> asyncio.Lock:
+    """Get or create an asyncio.Lock for the given file path."""
+    resolved = str(Path(path).resolve())
+    async with _locks_lock:
+        lock = _locks.get(resolved)
+        if lock is None:
+            lock = asyncio.Lock()
+            _locks[resolved] = lock
+        return lock
+
+
+# ---------------------------------------------------------------------------
+# Line-ending / BOM helpers (mirrors opencode src/tool/edit.ts)
+# ---------------------------------------------------------------------------
+
+
+def _normalize_line_endings(text: str) -> str:
+    """
+    Normalize actual CRLF line endings to LF.
+
+    ONLY handles real carriage-return + newline sequences (\\r\\n bytes).
+    Does NOT interpret escape sequences — literal \\n in file content
+    (e.g. Python string literals) must be preserved as-is.
+
+    Escape sequence handling for search strings is done by the
+    _escape_normalized_replacer in the replacer chain.
+    """
+    return text.replace("\r\n", "\n")
+
+
+def _detect_line_ending(text: str) -> Literal["\n", "\r\n"]:
+    return "\r\n" if "\r\n" in text else "\n"
+
+
+def _convert_to_line_ending(text: str, ending: Literal["\n", "\r\n"]) -> str:
+    if ending == "\n":
+        return text
+    # Convert standalone \n to \r\n, but avoid converting existing \r\n to \r\r\n
+    # by first normalizing any existing \r\n to \n, then converting all \n to \r\n
+    text = text.replace("\r\n", "\n")
+    return text.replace("\n", "\r\n")
+
+
+# ---------------------------------------------------------------------------
+# Levenshtein distance (for BlockAnchorReplacer)
+# ---------------------------------------------------------------------------
+
+
+def _levenshtein(a: str, b: str) -> int:
+    if a == "" or b == "":
+        return max(len(a), len(b))
+    # Use a single row DP to reduce memory
+    prev = list(range(len(b) + 1))
+    for i in range(1, len(a) + 1):
+        curr = [i]
+        ai = a[i - 1]
+        for j in range(1, len(b) + 1):
+            cost = 0 if ai == b[j - 1] else 1
+            curr.append(min(curr[-1] + 1, prev[j] + 1, prev[j - 1] + cost))
+        prev = curr
+    return prev[len(b)]
+
+
+# ---------------------------------------------------------------------------
+# Escape helpers
+# ---------------------------------------------------------------------------
+
+
+def _unescape(s: str) -> str:
+    """
+    Unescape common escape sequences in a string.
+
+    Handles: \\n, \\t, \\r, \\b, \\f, \\v, \\\\, \\", \\', \\`, \\$
+    Also handles \\xNN hex and \\uNNNN unicode escapes.
+    """
+    result = []
+    i = 0
+    while i < len(s):
+        if s[i] == "\\" and i + 1 < len(s):
+            nxt = s[i + 1]
+            if nxt == "n":
+                result.append("\n")
+            elif nxt == "t":
+                result.append("\t")
+            elif nxt == "r":
+                result.append("\r")
+            elif nxt == "b":
+                result.append("\b")
+            elif nxt == "f":
+                result.append("\f")
+            elif nxt == "v":
+                result.append("\v")
+            elif nxt == "x" and i + 3 < len(s):
+                # \xNN hex escape
+                try:
+                    val = int(s[i + 2 : i + 4], 16)
+                    result.append(chr(val))
+                    i += 4
+                    continue
+                except ValueError:
+                    result.append(s[i])
+                    result.append(nxt)
+            elif nxt == "u" and i + 5 < len(s):
+                # \uNNNN unicode escape
+                try:
+                    val = int(s[i + 2 : i + 6], 16)
+                    result.append(chr(val))
+                    i += 6
+                    continue
+                except ValueError:
+                    result.append(s[i])
+                    result.append(nxt)
+            elif nxt in ("'", '"', "`", "\\", "$"):
+                result.append(nxt)
+            else:
+                # Unknown escape: preserve both characters
+                result.append(s[i])
+                result.append(nxt)
+            i += 2
+        else:
+            result.append(s[i])
+            i += 1
+    return "".join(result)
+
+
+# ---------------------------------------------------------------------------
+# Replacer implementations
+# ---------------------------------------------------------------------------
+
+
+def _simple_replacer(content: str, find: str) -> Iterator[str]:
+    """Exact match."""
+    if not find:
+        return
+    yield find
+
+
+def _escape_normalized_replacer(content: str, find: str) -> Iterator[str]:
+    """
+    Handle escaped sequences like \\n, \\t in the find string.
+
+    This replacer tries two approaches:
+    1. Unescape the find string and look for it in content
+    2. If find contains literal backslash sequences, try matching content
+       blocks after unescaping them
+
+    Results are deduplicated to avoid yielding the same block twice.
+    """
+    if not find:
+        return
+
+    yielded = set()
+
+    # Approach 1: unescape find and search in content
+    unescaped_find = _unescape(find)
+    if unescaped_find != find and unescaped_find in content:
+        yielded.add(unescaped_find)
+        yield unescaped_find
+
+    # Approach 2: if find contains literal backslash sequences,
+    # try matching against content blocks after unescaping
+    if "\\" in find:
+        lines = content.split("\n")
+        find_lines = unescaped_find.split("\n")
+        for i in range(len(lines) - len(find_lines) + 1):
+            block = "\n".join(lines[i : i + len(find_lines)])
+            if block not in yielded and _unescape(block) == unescaped_find:
+                yielded.add(block)
+                yield block
+
+
+def _line_trimmed_replacer(content: str, find: str) -> Iterator[str]:
+    """Match blocks where each line matches after trim."""
+    if not find:
+        return
+    original_lines = content.split("\n")
+    search_lines = find.split("\n")
+    if search_lines and search_lines[-1] == "":
+        search_lines.pop()
+    if not search_lines:
+        return
+    for i in range(len(original_lines) - len(search_lines) + 1):
+        if all(
+            original_lines[i + j].strip() == search_lines[j].strip()
+            for j in range(len(search_lines))
+        ):
+            yield "\n".join(original_lines[i : i + len(search_lines)])
+
+
+def _block_anchor_replacer(content: str, find: str) -> Iterator[str]:
+    """
+    Use first and last line as anchors, then use Levenshtein similarity on middle lines.
+    Single candidate threshold: 0.0 (accept if anchors match)
+    Multiple candidates threshold: 0.3 (pick best)
+    """
+    if not find:
+        return
+    original_lines = content.split("\n")
+    search_lines = find.split("\n")
+    if len(search_lines) < 3:
+        return
+    if search_lines and search_lines[-1] == "":
+        search_lines.pop()
+    if len(search_lines) < 3:
+        return
+
+    first_anchor = search_lines[0].strip()
+    last_anchor = search_lines[-1].strip()
+    search_block_size = len(search_lines)
+
+    candidates: list[tuple[int, int]] = []
+    for i, line in enumerate(original_lines):
+        if line.strip() != first_anchor:
+            continue
+        for j in range(i + 2, len(original_lines)):
+            if original_lines[j].strip() == last_anchor:
+                candidates.append((i, j))
+                break
+
+    if not candidates:
+        return
+
+    def _similarity(start: int, end: int) -> float:
+        actual_size = end - start + 1
+        lines_to_check = min(search_block_size - 2, actual_size - 2)
+        if lines_to_check <= 0:
+            return 1.0
+        sim = 0.0
+        for k in range(1, lines_to_check + 1):
+            if start + k >= len(original_lines) or k >= len(search_lines) - 1:
+                break
+            ol = original_lines[start + k].strip()
+            sl = search_lines[k].strip()
+            max_len = max(len(ol), len(sl))
+            if max_len == 0:
+                continue
+            dist = _levenshtein(ol, sl)
+            sim += (1 - dist / max_len) / lines_to_check
+        return sim
+
+    if len(candidates) == 1:
+        start, end = candidates[0]
+        if _similarity(start, end) >= 0.0:
+            yield "\n".join(original_lines[start : end + 1])
+        return
+
+    best_match: tuple[int, int] | None = None
+    max_sim = -1.0
+    for start, end in candidates:
+        sim = _similarity(start, end)
+        if sim > max_sim:
+            max_sim = sim
+            best_match = (start, end)
+
+    if max_sim >= 0.3 and best_match:
+        start, end = best_match
+        yield "\n".join(original_lines[start : end + 1])
+
+
+def _whitespace_normalized_replacer(content: str, find: str) -> Iterator[str]:
+    """Collapse all whitespace sequences to a single space before matching."""
+    if not find:
+        return
+
+    def _norm(t: str) -> str:
+        return " ".join(t.split())
+
+    normalized_find = _norm(find)
+    if not normalized_find:
+        return
+
+    # Single-line matches
+    for line in content.split("\n"):
+        if _norm(line) == normalized_find:
+            yield line
+
+    # Multi-line block matches
+    lines = content.split("\n")
+    find_lines = find.split("\n")
+    if len(find_lines) > 1:
+        for i in range(len(lines) - len(find_lines) + 1):
+            block = "\n".join(lines[i : i + len(find_lines)])
+            if _norm(block) == normalized_find:
+                yield block
+
+
+def _indentation_flexible_replacer(content: str, find: str) -> Iterator[str]:
+    """
+    Match blocks where removing common indentation makes them equal to find.
+
+    Important: yields the ORIGINAL block (with original indentation),
+    not the de-indented version. This preserves the file's indentation
+    during replacement.
+    """
+    if not find:
+        return
+
+    def _remove_indent(text: str) -> str:
+        lines = text.split("\n")
+        non_empty = [ln for ln in lines if ln.strip()]
+        if not non_empty:
+            return text
+        min_indent = min(len(ln) - len(ln.lstrip()) for ln in non_empty if ln.strip())
+        return "\n".join(ln[min_indent:] if ln.strip() else ln for ln in lines)
+
+    normalized_find = _remove_indent(find)
+    content_lines = content.split("\n")
+    find_lines = find.split("\n")
+    for i in range(len(content_lines) - len(find_lines) + 1):
+        block = "\n".join(content_lines[i : i + len(find_lines)])
+        if _remove_indent(block) == normalized_find:
+            yield block
+
+
+def _trimmed_boundary_replacer(content: str, find: str) -> Iterator[str]:
+    """Match if the trimmed version of find exists in content."""
+    if not find:
+        return
+    trimmed = find.strip()
+    if trimmed == find:
+        return
+    if trimmed in content:
+        yield trimmed
+
+    lines = content.split("\n")
+    find_lines = find.split("\n")
+    for i in range(len(lines) - len(find_lines) + 1):
+        block = "\n".join(lines[i : i + len(find_lines)])
+        if block.strip() == trimmed:
+            yield block
+
+
+def _context_aware_replacer(content: str, find: str) -> Iterator[str]:
+    """
+    Use first and last line as context anchors, accept if >= 50% of middle lines match.
+    """
+    if not find:
+        return
+    find_lines = find.split("\n")
+    if len(find_lines) < 3:
+        return
+    if find_lines and find_lines[-1] == "":
+        find_lines.pop()
+    if len(find_lines) < 3:
+        return
+
+    first_line = find_lines[0].strip()
+    last_line = find_lines[-1].strip()
+    content_lines = content.split("\n")
+
+    for i, line in enumerate(content_lines):
+        if line.strip() != first_line:
+            continue
+        for j in range(i + 2, len(content_lines)):
+            if content_lines[j].strip() != last_line:
+                continue
+            block_lines = content_lines[i : j + 1]
+            if len(block_lines) != len(find_lines):
+                continue
+            matching = 0
+            total_nonempty = 0
+            for k in range(1, len(block_lines) - 1):
+                bl = block_lines[k].strip()
+                fl = find_lines[k].strip()
+                if bl or fl:
+                    total_nonempty += 1
+                    if bl == fl:
+                        matching += 1
+            if total_nonempty == 0 or matching / total_nonempty >= 0.5:
+                yield "\n".join(block_lines)
+                return  # Only first match
+            break
+
+
+def _multi_occurrence_replacer(content: str, find: str) -> Iterator[str]:
+    """Yield all exact matches (used with replace_all)."""
+    if not find:
+        return
+    start = 0
+    while True:
+        idx = content.find(find, start)
+        if idx == -1:
+            break
+        yield find
+        start = idx + len(find)
+
+
+# Ordered chain: most specific first, most lenient last.
+# Escape-normalized is placed early because it handles a common LLM issue
+# (using \\n instead of actual newlines) before more aggressive fuzzy matchers.
+_REPLACERS: list[Replacer] = [
+    _simple_replacer,
+    _escape_normalized_replacer,
+    _line_trimmed_replacer,
+    _block_anchor_replacer,
+    _whitespace_normalized_replacer,
+    _indentation_flexible_replacer,
+    _trimmed_boundary_replacer,
+    _context_aware_replacer,
+    _multi_occurrence_replacer,
+]
+
+
+# ---------------------------------------------------------------------------
+# Core replace function
+# ---------------------------------------------------------------------------
+
+
+def robust_replace(
+    content: str,
+    old_string: str,
+    new_string: str,
+    *,
+    replace_all: bool = False,
+) -> tuple[str, int]:
+    """
+    Replace old_string with new_string using the multi-strategy replacer chain.
+
+    Returns:
+        A tuple of (new_content, replacements_count).
+
+    Raises:
+        ValueError: If old_string cannot be found, or if multiple non-unique
+                    matches are found (when replace_all=False).
+    """
+    if old_string == new_string:
+        raise ValueError(
+            "No changes to apply: old_string and new_string are identical."
+        )
+
+    not_found = True
+
+    for replacer in _REPLACERS:
+        matches = list(replacer(content, old_string))
+        if not matches:
+            continue
+
+        # Collect all unique match positions
+        match_positions: list[tuple[int, str]] = []
+        for match in matches:
+            start = 0
+            while True:
+                idx = content.find(match, start)
+                if idx == -1:
+                    break
+                # Avoid duplicate positions from overlapping matches
+                if not any(pos <= idx < pos + len(m) for pos, m in match_positions):
+                    match_positions.append((idx, match))
+                start = idx + 1
+
+        if not match_positions:
+            continue
+
+        not_found = False
+
+        if replace_all:
+            # Replace all occurrences, from end to start to preserve indices
+            new_content = content
+            replacements = 0
+            for idx, match in sorted(match_positions, key=lambda x: x[0], reverse=True):
+                new_content = (
+                    new_content[:idx] + new_string + new_content[idx + len(match) :]
+                )
+                replacements += 1
+            return new_content, replacements
+
+        # Single replacement mode: require exactly one match
+        if len(match_positions) == 1:
+            idx, match = match_positions[0]
+            return content[:idx] + new_string + content[idx + len(match) :], 1
+
+        # Multiple matches found in single-replacement mode: continue to next replacer
+        # to try a more specific strategy
+        continue
+
+    if not_found:
+        raise ValueError(
+            "Could not find oldString in the file. It must match exactly, "
+            "including whitespace, indentation, and line endings. "
+            "Try providing more surrounding context to make the match unique."
+        )
+    raise ValueError(
+        "Found multiple matches for oldString. Provide more surrounding context "
+        "to make the match unique, or use replace_all=True to change every instance."
+    )
+
+
+# ---------------------------------------------------------------------------
+# Edit result model
+# ---------------------------------------------------------------------------
+
+
+@dataclass
+class EditResult:
+    success: bool
+    replacements: int = 0
+    diff: str = ""
+    error: str = ""
+    old_content: str = ""
+    new_content: str = ""
+
+
+# ---------------------------------------------------------------------------
+# Async file edit with locking and line-ending preservation (local runtime)
+# ---------------------------------------------------------------------------
+
+
+async def edit_file(
+    path: str,
+    old_string: str,
+    new_string: str,
+    *,
+    replace_all: bool = False,
+    encoding: str = "utf-8",
+) -> EditResult:
+    """
+    Edit a file using the robust multi-strategy replacer.
+
+    Features:
+    - File-level asyncio lock prevents concurrent edits
+    - Preserves original line endings (\\n vs \\r\\n)
+    - Preserves BOM if present
+    - Returns unified diff of changes
+    """
+    lock = await _get_lock(path)
+    async with lock:
+        try:
+            # Read file
+            raw_bytes = await asyncio.to_thread(_read_file_bytes, path)
+            has_bom = raw_bytes.startswith(b"\xef\xbb\xbf")
+            if has_bom:
+                raw_bytes = raw_bytes[3:]
+
+            old_content = raw_bytes.decode(encoding)
+            original_ending = _detect_line_ending(old_content)
+
+            # Normalize for matching: ONLY normalize actual CRLF line endings.
+            # Escape sequence handling (\n vs actual newline, \t vs tab, etc.)
+            # is deferred to the _escape_normalized_replacer in the replacer chain.
+            normalized_old = _normalize_line_endings(old_string)
+            normalized_new = _normalize_line_endings(new_string)
+
+            # Normalize file content to LF for matching (replacers work on LF)
+            normalized_content = _normalize_line_endings(old_content)
+            # Perform replacement
+            new_content, replacements = robust_replace(
+                normalized_content,
+                normalized_old,
+                normalized_new,
+                replace_all=replace_all,
+            )
+
+            # Convert back to original line endings
+            if original_ending == "\r\n":
+                new_content = _convert_to_line_ending(new_content, "\r\n")
+
+            # Re-add BOM if present
+            write_bytes = b""
+            if has_bom:
+                write_bytes += b"\xef\xbb\xbf"
+            write_bytes += new_content.encode(encoding)
+
+            # Write file
+            await asyncio.to_thread(_write_file_bytes, path, write_bytes)
+
+            # Generate unified diff
+            diff = build_unified_diff(
+                path,
+                old_content,
+                new_content,
+            )
+
+            return EditResult(
+                success=True,
+                replacements=replacements,
+                diff=diff,
+                old_content=old_content,
+                new_content=new_content,
+            )
+
+        except Exception as exc:
+            return EditResult(
+                success=False,
+                error=str(exc),
+            )
+
+
+def _read_file_bytes(path: str) -> bytes:
+    with open(path, "rb") as f:
+        return f.read()
+
+
+def _write_file_bytes(path: str, data: bytes) -> None:
+    # Ensure parent directory exists
+    os.makedirs(os.path.dirname(path) or ".", exist_ok=True)
+    with open(path, "wb") as f:
+        f.write(data)
+
+
+# ---------------------------------------------------------------------------
+# Public diff helper
+# ---------------------------------------------------------------------------
+
+
+def build_unified_diff(
+    old_path: str,
+    old_content: str,
+    new_content: str,
+) -> str:
+    """Generate a unified diff between two content strings."""
+    diff = list(
+        difflib.unified_diff(
+            old_content.splitlines(),
+            new_content.splitlines(),
+            fromfile=old_path,
+            tofile=old_path,
+            lineterm="",
+        )
+    )
+    return "\n".join(diff)
diff --git a/astrbot/core/tools/computer_tools/fs.py b/astrbot/core/tools/computer_tools/fs.py
index 5660022fd0..77a755c34f 100644
--- a/astrbot/core/tools/computer_tools/fs.py
+++ b/astrbot/core/tools/computer_tools/fs.py
@@ -55,6 +55,12 @@
 
 from ..registry import builtin_tool
 from . import util as computer_util
+from .edit_engine import (
+    EditResult,
+    build_unified_diff,
+    edit_file,
+    robust_replace,
+)
 from .util import (
     check_admin_permission,
     is_local_runtime,
@@ -368,22 +374,85 @@ async def call(
             return f"Error writing file: {exc}"
 
 
+def _format_result(
+    path: str,
+    result: EditResult,
+    *,
+    replace_all: bool,
+) -> str:
+    """Build the human-readable / LLM-readable result string with optional diff."""
+    if not result.success:
+        return f"Error editing file: {result.error}"
+
+    mode_text = "all matches" if replace_all else "first match"
+
+    lines = [
+        f"Edited {path}.",
+        f"Replaced {result.replacements} occurrence(s) using {mode_text} mode.",
+    ]
+
+    if result.diff:
+        diff_preview = result.diff
+        if len(diff_preview) > 2000:
+            diff_preview = diff_preview[:2000] + "\n... (diff truncated)"
+        lines.append("")
+        lines.append("Diff:")
+        lines.append("```diff")
+        lines.append(diff_preview)
+        lines.append("```")
+
+    return "\n".join(lines)
+
+
 @builtin_tool(config=_COMPUTER_RUNTIME_TOOL_CONFIG)
 @dataclass
 class FileEditTool(FunctionTool):
+    """
+    Enhanced file editing tool with robust fuzzy matching.
+
+    In local runtime it uses the full robust edit engine (BOM +
+    line-ending preservation, file locks); in sandbox runtimes it
+    mediates reads and writes through the booter's filesystem
+    abstraction while applying the same 9-strategy replacer chain.
+
+    This tool is designed to handle LLM-generated edits that may
+    have minor whitespace, indentation, or escape sequence
+    differences from the actual file content.
+    """
+
     name: str = "astrbot_file_edit_tool"
-    description: str = "Editing files."
+    description: str = (
+        "Editing files with robust fuzzy matching. "
+        "Supports exact match, escape-normalized match, line-trimmed match, block-anchor match, "
+        "whitespace-normalized match, indentation-flexible match, "
+        "trimmed-boundary match, context-aware match, "
+        "and multi-occurrence replacement. "
+        "When editing text from Read tool output, preserve the exact indentation "
+        "(tabs/spaces) as it appears AFTER the line number prefix. "
+        "The line number prefix format is: line number + colon + space (e.g., '1: '). "
+        "Everything after that space is the actual file content to match. "
+        "Never include any part of the line number prefix in oldString or newString. "
+        "The edit will FAIL if oldString is not found. "
+        "The edit will FAIL if oldString is found multiple times and replace_all is false. "
+        "Use replace_all for renaming variables or strings across the file."
+    )
     parameters: dict = field(
         default_factory=lambda: {
             "type": "object",
             "properties": {
                 "path": {
                     "type": "string",
-                    "description": "Path of the file to edit. If relative, will be in workspace root.",
+                    "description": (
+                        "Path of the file to edit. If relative, will be in workspace root."
+                    ),
                 },
                 "old": {
                     "type": "string",
-                    "description": "The exact old text to replace.",
+                    "description": (
+                        "The text to replace. Must be an exact substring of the file content, "
+                        "but the tool will try multiple matching strategies if exact match fails. "
+                        "Include sufficient surrounding context (3-5 lines) to make the match unique."
+                    ),
                 },
                 "new": {
                     "type": "string",
@@ -391,7 +460,10 @@ class FileEditTool(FunctionTool):
                 },
                 "replace_all": {
                     "type": "boolean",
-                    "description": "Whether to replace all matches. Defaults to false.",
+                    "description": (
+                        "Whether to replace all matches. Defaults to false. "
+                        "Useful for renaming variables or strings across the file."
+                    ),
                 },
             },
             "required": ["path", "old", "new"],
@@ -409,45 +481,97 @@ async def call(
         umo = str(context.context.event.unified_msg_origin)
         local_env = is_local_runtime(context)
         restricted = _is_restricted_env(context)
+
         try:
-            normalized_path = (
-                _normalize_rw_path(
+            # ── path resolution ────────────────────────────────────
+            if local_env:
+                normalized_path = _normalize_rw_path(
                     path,
                     restricted=restricted,
                     local_env=local_env,
                     umo=umo,
                     write=True,
                 )
-                if local_env
-                else path.strip()
-            )
+            else:
+                normalized_path = path.strip()
+
             if not normalized_path:
                 raise ValueError("`path` must be a non-empty string.")
-            normalized_old = _decode_escaped_text(old)
-            normalized_new = _decode_escaped_text(new)
-            sb = await get_booter(
-                context.context.context,
-                context.context.event.unified_msg_origin,
-            )
-            result = await sb.fs.edit_file(
-                path=normalized_path,
-                old_string=normalized_old,
-                new_string=normalized_new,
-                replace_all=replace_all,
-                encoding="utf-8",
-            )
-            if not result.get("success", False):
-                error_detail = str(result.get("error", "") or "").strip()
-                return (
-                    "Error editing file: "
-                    f"{error_detail or 'unknown filesystem edit error'}"
+
+            # ── execute edit ───────────────────────────────────────
+            if local_env:
+                # Local: full robust edit engine
+                #   BOM preservation  ✅
+                #   CRLF preservation ✅
+                #   file-level lock   ✅
+                #   diff output       ✅
+                result = await edit_file(
+                    path=normalized_path,
+                    old_string=old,
+                    new_string=new,
+                    replace_all=replace_all,
+                    encoding="utf-8",
+                )
+            else:
+                # Sandbox: booter-mediated I/O + robust_replace
+                #   fuzzy matching   ✅
+                #   diff output      ✅
+                #   BOM/CRLF         handled by sandbox runtime
+                #   concurrency      handled by sandbox runtime
+                sb = await get_booter(
+                    context.context.context,
+                    umo,
+                )
+
+                read_result = await sb.fs.read_file(path=normalized_path)
+                if not read_result.get("success", False):
+                    error_detail = str(read_result.get("error", "") or "").strip()
+                    return (
+                        "Error editing file: "
+                        f"{error_detail or 'unknown filesystem read error'}"
+                    )
+
+                old_content = read_result.get("content", "")
+
+                try:
+                    new_content, replacements = robust_replace(
+                        old_content,
+                        old,
+                        new,
+                        replace_all=replace_all,
+                    )
+                except ValueError as exc:
+                    return f"Error editing file: {exc}"
+
+                write_result = await sb.fs.write_file(
+                    path=normalized_path,
+                    content=new_content,
+                )
+                if not write_result.get("success", False):
+                    error_detail = str(write_result.get("error", "") or "").strip()
+                    return (
+                        "Error editing file: "
+                        f"{error_detail or 'unknown filesystem write error'}"
+                    )
+
+                diff = build_unified_diff(
+                    normalized_path,
+                    old_content,
+                    new_content,
+                )
+
+                result = EditResult(
+                    success=True,
+                    replacements=replacements,
+                    diff=diff,
                 )
-            replacements = int(result.get("replacements", 0) or 0)
-            mode_text = "all matches" if replace_all else "first match"
-            return (
-                f"Edited {normalized_path}. "
-                f"Replaced {replacements} occurrence(s) using {mode_text} mode."
+
+            return _format_result(
+                normalized_path,
+                result,
+                replace_all=replace_all,
             )
+
         except PermissionError as exc:
             return f"Error: {exc}"
         except Exception as exc:
diff --git a/dashboard/src/components/chat/ReasoningSidebar.vue b/dashboard/src/components/chat/ReasoningSidebar.vue
index 93cc0ed067..3486839b16 100644
--- a/dashboard/src/components/chat/ReasoningSidebar.vue
+++ b/dashboard/src/components/chat/ReasoningSidebar.vue
@@ -1,6 +1,17 @@
 <template>
   <transition name="slide-left">
-    <aside v-if="modelValue" class="reasoning-sidebar">
+    <aside
+      v-if="modelValue"
+      ref="sidebarRef"
+      class="reasoning-sidebar"
+      :style="{ width: sidebarWidth + 'px' }"
+    >
+      <!-- Drag handle -->
+      <div
+        class="reasoning-sidebar-resizer"
+        @mousedown="startResize"
+      />
+
       <div class="reasoning-sidebar-header">
         <div class="reasoning-sidebar-title">{{ tm("reasoning.thinking") }}</div>
         <v-btn icon="mdi-close" size="small" variant="text" @click="close" />
@@ -14,7 +25,7 @@
           :is-dark="isDark"
         />
         <div v-else class="reasoning-sidebar-empty">
-          {{ tm("reasoning.thinking") }}
+          {{ tm("reasoning.noThinking") }}
         </div>
       </div>
     </aside>
@@ -22,6 +33,7 @@
 </template>
 
 <script setup lang="ts">
+import { onBeforeUnmount, ref } from "vue";
 import type { MessagePart } from "@/composables/useMessages";
 import { useModuleI18n } from "@/i18n/composables";
 import ReasoningTimeline from "@/components/chat/message_list_comps/ReasoningTimeline.vue";
@@ -42,6 +54,52 @@ const { tm } = useModuleI18n("features/chat");
 function close() {
   emit("update:modelValue", false);
 }
+
+// ── Drag resize ────────────────────────────────────────────────────
+
+const MIN_WIDTH = 280;
+const MAX_WIDTH = 800;
+const DEFAULT_WIDTH = 380;
+
+const sidebarWidth = ref(DEFAULT_WIDTH);
+const sidebarRef = ref<HTMLElement | null>(null);
+let isResizing = false;
+
+function startResize(e: MouseEvent) {
+  e.preventDefault();
+  isResizing = true;
+  document.body.style.cursor = "ew-resize";
+  document.body.style.userSelect = "none";
+  document.addEventListener("mousemove", onMouseMove);
+  document.addEventListener("mouseup", onMouseUp);
+}
+
+function onMouseMove(e: MouseEvent) {
+  if (!isResizing || !sidebarRef.value) return;
+  // The sidebar is on the right side of the viewport.
+  // `sidebarRef` is positioned inside the parent's flex layout.
+  // Calculate width from the right edge of the viewport:
+  const rect = sidebarRef.value.parentElement?.getBoundingClientRect();
+  if (!rect) return;
+  const newWidth = rect.right - e.clientX;
+  sidebarWidth.value = Math.min(MAX_WIDTH, Math.max(MIN_WIDTH, newWidth));
+}
+
+function onMouseUp() {
+  if (!isResizing) return;
+  isResizing = false;
+  document.body.style.cursor = "";
+  document.body.style.userSelect = "";
+  document.removeEventListener("mousemove", onMouseMove);
+  document.removeEventListener("mouseup", onMouseUp);
+}
+
+onBeforeUnmount(() => {
+  if (isResizing) {
+    document.removeEventListener("mousemove", onMouseMove);
+    document.removeEventListener("mouseup", onMouseUp);
+  }
+});
 </script>
 
 <style scoped>
@@ -54,8 +112,29 @@ function close() {
   display: flex;
   flex-direction: column;
   flex-shrink: 0;
+  position: relative;
+}
+
+/* ── Drag handle ──────────────────────────────────────────────── */
+
+.reasoning-sidebar-resizer {
+  position: absolute;
+  left: 0;
+  top: 0;
+  bottom: 0;
+  width: 6px;
+  cursor: ew-resize;
+  z-index: 10;
+  transition: background 0.15s ease;
 }
 
+.reasoning-sidebar-resizer:hover,
+.reasoning-sidebar-resizer:active {
+  background: rgba(var(--v-theme-primary), 0.2);
+}
+
+/* ── Transition ───────────────────────────────────────────────── */
+
 .slide-left-enter-active,
 .slide-left-leave-active {
   transition: all 0.2s ease;
@@ -67,6 +146,8 @@ function close() {
   opacity: 0;
 }
 
+/* ── Header ───────────────────────────────────────────────────── */
+
 .reasoning-sidebar-header {
   display: flex;
   align-items: center;
@@ -81,6 +162,8 @@ function close() {
   color: rgb(var(--v-theme-on-surface));
 }
 
+/* ── Body ─────────────────────────────────────────────────────── */
+
 .reasoning-sidebar-body {
   flex: 1;
   min-height: 0;
@@ -96,16 +179,22 @@ function close() {
   font-size: 13px;
 }
 
+/* ── Mobile ───────────────────────────────────────────────────── */
+
 @media (max-width: 760px) {
   .reasoning-sidebar {
     position: fixed;
     inset: 0;
     z-index: 1300;
-    width: 100vw;
+    width: 100vw !important;
     height: 100dvh;
     border-left: 0;
   }
 
+  .reasoning-sidebar-resizer {
+    display: none;
+  }
+
   .reasoning-sidebar-header {
     min-height: 52px;
     padding: calc(10px + env(safe-area-inset-top)) 12px 8px;
diff --git a/dashboard/src/components/chat/ThreadPanel.vue b/dashboard/src/components/chat/ThreadPanel.vue
index bab1fe8d18..e2bfc41320 100644
--- a/dashboard/src/components/chat/ThreadPanel.vue
+++ b/dashboard/src/components/chat/ThreadPanel.vue
@@ -1,6 +1,17 @@
 <template>
   <transition name="slide-left">
-    <aside v-if="modelValue && thread" class="thread-panel">
+    <aside
+      v-if="modelValue && thread"
+      ref="panelRef"
+      class="thread-panel"
+      :style="{ width: panelWidth + 'px' }"
+    >
+      <!-- Drag handle -->
+      <div
+        class="thread-panel-resizer"
+        @mousedown="startResize"
+      />
+
       <div class="thread-panel-header">
         <div class="thread-panel-title">{{ tm("thread.title") }}</div>
         <div class="thread-panel-actions">
@@ -55,7 +66,7 @@
 </template>
 
 <script setup lang="ts">
-import { nextTick, ref, watch } from "vue";
+import { nextTick, onBeforeUnmount, ref, watch } from "vue";
 import axios from "axios";
 import {
   appendPlain,
@@ -108,6 +119,49 @@ function close() {
   emit("update:modelValue", false);
 }
 
+// ── Drag resize ────────────────────────────────────────────────────
+
+const MIN_WIDTH = 280;
+const MAX_WIDTH = 800;
+const DEFAULT_WIDTH = 380;
+
+const panelWidth = ref(DEFAULT_WIDTH);
+const panelRef = ref<HTMLElement | null>(null);
+let isResizing = false;
+
+function startResize(e: MouseEvent) {
+  e.preventDefault();
+  isResizing = true;
+  document.body.style.cursor = "ew-resize";
+  document.body.style.userSelect = "none";
+  document.addEventListener("mousemove", onMouseMove);
+  document.addEventListener("mouseup", onMouseUp);
+}
+
+function onMouseMove(e: MouseEvent) {
+  if (!isResizing || !panelRef.value) return;
+  const rect = panelRef.value.parentElement?.getBoundingClientRect();
+  if (!rect) return;
+  const newWidth = rect.right - e.clientX;
+  panelWidth.value = Math.min(MAX_WIDTH, Math.max(MIN_WIDTH, newWidth));
+}
+
+function onMouseUp() {
+  if (!isResizing) return;
+  isResizing = false;
+  document.body.style.cursor = "";
+  document.body.style.userSelect = "";
+  document.removeEventListener("mousemove", onMouseMove);
+  document.removeEventListener("mouseup", onMouseUp);
+}
+
+onBeforeUnmount(() => {
+  if (isResizing) {
+    document.removeEventListener("mousemove", onMouseMove);
+    document.removeEventListener("mouseup", onMouseUp);
+  }
+});
+
 async function loadThread(threadId: string) {
   try {
     const response = await axios.get("/api/chat/thread/get", {
@@ -337,8 +391,29 @@ function scrollToBottom() {
   display: flex;
   flex-direction: column;
   flex-shrink: 0;
+  position: relative;
 }
 
+/* ── Drag handle ──────────────────────────────────────────────── */
+
+.thread-panel-resizer {
+  position: absolute;
+  left: 0;
+  top: 0;
+  bottom: 0;
+  width: 6px;
+  cursor: ew-resize;
+  z-index: 10;
+  transition: background 0.15s ease;
+}
+
+.thread-panel-resizer:hover,
+.thread-panel-resizer:active {
+  background: rgba(var(--v-theme-primary), 0.2);
+}
+
+/* ── Transition ───────────────────────────────────────────────── */
+
 .slide-left-enter-active,
 .slide-left-leave-active {
   transition: all 0.2s ease;
@@ -446,6 +521,10 @@ function scrollToBottom() {
     border-left: 0;
   }
 
+  .thread-panel-resizer {
+    display: none;
+  }
+
   .thread-panel-header {
     min-height: 52px;
     padding: calc(10px + env(safe-area-inset-top)) 12px 8px;
diff --git a/dashboard/src/components/chat/message_list_comps/DiffPreview.vue b/dashboard/src/components/chat/message_list_comps/DiffPreview.vue
new file mode 100644
index 0000000000..e16a7025a0
--- /dev/null
+++ b/dashboard/src/components/chat/message_list_comps/DiffPreview.vue
@@ -0,0 +1,512 @@
+<template>
+  <div
+    class="diff-preview"
+    :class="{ 'is-dark': isDark, collapsed: isCollapsed }"
+  >
+    <!-- Summary header — always visible, clickable to toggle -->
+    <button
+      v-if="summary || filePath || statsText"
+      type="button"
+      class="diff-header"
+      @click="toggleCollapsed"
+    >
+      <div class="diff-header-left">
+        <v-icon size="16" class="diff-header-icon">mdi-file-document-edit-outline</v-icon>
+        <span v-if="filePath" class="diff-file-path">{{ filePath }}</span>
+      </div>
+      <div class="diff-header-right">
+        <span v-if="statsText" class="diff-stats">{{ statsText }}</span>
+        <v-icon
+          v-if="collapsible"
+          size="18"
+          class="diff-chevron"
+          :class="{ expanded: !isCollapsed }"
+        >
+          mdi-chevron-right
+        </v-icon>
+      </div>
+    </button>
+
+    <!-- Summary text (e.g. "Replaced 1 occurrence(s)...") -->
+    <div v-if="summary && !isCollapsed" class="diff-summary-text">
+      {{ summary }}
+    </div>
+
+    <!-- Diff hunks — hidden when collapsed -->
+    <div v-if="!isCollapsed" class="diff-body">
+      <div v-if="truncated" class="diff-truncation-warning">
+        ⚠ Diff truncated (showing first {{ maxChars.toLocaleString() }} characters)
+      </div>
+
+      <div
+        v-for="(hunk, hi) in parsedHunks"
+        :key="hi"
+        class="diff-hunk"
+      >
+        <div class="hunk-header">
+          @@ {{ hunk.header }} @@
+        </div>
+        <div
+          v-for="(line, li) in hunk.lines"
+          :key="li"
+          class="diff-line"
+          :class="line.type"
+        >
+          <span class="line-number old">{{ line.oldNo }}</span>
+          <span class="line-number new">{{ line.newNo }}</span>
+          <span class="line-prefix">{{ line.prefix }}</span>
+          <span class="line-content">{{ line.content }}</span>
+        </div>
+      </div>
+
+      <div v-if="collapsedOverflow > 0" class="diff-overflow-bar">
+        <button type="button" class="diff-show-more" @click="showAllLines = true">
+          Show all {{ totalLines.toLocaleString() }} lines
+          ({{ collapsedOverflow }} more)
+        </button>
+      </div>
+    </div>
+  </div>
+</template>
+
+<script setup lang="ts">
+import { computed, ref } from "vue";
+
+// ── Types ──────────────────────────────────────────────────────────
+
+interface DiffLine {
+  type: "add" | "del" | "ctx" | "header-file";
+  prefix: string;
+  content: string;
+  oldNo: string;
+  newNo: string;
+}
+
+interface DiffHunk {
+  header: string;
+  lines: DiffLine[];
+}
+
+// ── Props ──────────────────────────────────────────────────────────
+
+const props = withDefaults(
+  defineProps<{
+    content: string;
+    filePath?: string;
+    summary?: string;
+    maxLines?: number;
+    maxChars?: number;
+    collapsible?: boolean;
+    isDark?: boolean;
+  }>(),
+  {
+    filePath: "",
+    summary: "",
+    maxLines: 30,
+    maxChars: 2000,
+    collapsible: true,
+    isDark: false,
+  },
+);
+
+// ── State ──────────────────────────────────────────────────────────
+
+const isCollapsed = ref(false);
+const showAllLines = ref(false);
+const effectiveMaxLines = computed(() =>
+  showAllLines.value ? Infinity : props.maxLines,
+);
+
+const toggleCollapsed = () => {
+  if (props.collapsible) {
+    isCollapsed.value = !isCollapsed.value;
+  }
+};
+
+// ── Parse unified diff ─────────────────────────────────────────────
+
+const parsedHunks = computed<DiffHunk[]>(() => {
+  const text = extractDiffContent(props.content);
+  return parseUnifiedDiff(text, effectiveMaxLines.value);
+});
+
+const totalLines = computed(() =>
+  parsedHunks.value.reduce((sum, h) => sum + h.lines.length, 0),
+);
+
+const truncated = computed(() => {
+  const raw = extractDiffContent(props.content);
+  return raw.length > props.maxChars;
+});
+
+const collapsedOverflow = computed(() => {
+  if (showAllLines.value) return 0;
+  const fullHunks = parseUnifiedDiff(
+    extractDiffContent(props.content),
+    Infinity,
+  );
+  const fullTotal = fullHunks.reduce((sum, h) => sum + h.lines.length, 0);
+  return Math.max(0, fullTotal - totalLines.value);
+});
+
+// ── Stats ──────────────────────────────────────────────────────────
+
+const statsText = computed(() => {
+  let adds = 0;
+  let dels = 0;
+  for (const hunk of parsedHunks.value) {
+    for (const line of hunk.lines) {
+      if (line.type === "add") adds++;
+      if (line.type === "del") dels++;
+    }
+  }
+  if (adds === 0 && dels === 0) return "";
+  return `+${adds} −${dels}`;
+});
+
+// ── Helpers ────────────────────────────────────────────────────────
+
+function extractDiffContent(raw: string): string {
+  // If the text contains a ```diff ... ``` block, extract its content
+  const blockMatch = raw.match(/```diff\s*\n?([\s\S]*?)```/);
+  if (blockMatch) return blockMatch[1];
+
+  // Otherwise, try to strip leading "Diff:" / "Edited ..." lines
+  const diffIdx = raw.indexOf("@@");
+  if (diffIdx >= 0) return raw.slice(diffIdx);
+
+  return raw;
+}
+
+function parseUnifiedDiff(text: string, maxLines: number): DiffHunk[] {
+  const lines = text.split("\n");
+  const hunks: DiffHunk[] = [];
+  let currentHunk: DiffHunk | null = null;
+  let totalLines = 0;
+  let oldNo = 0;
+  let newNo = 0;
+
+  // Try to parse --- / +++ file headers to get old/new line numbers
+  for (const rawLine of lines) {
+    if (totalLines >= maxLines) break;
+
+    const hunkMatch = rawLine.match(
+      /^@@\s+-(\d+)(?:,(\d+))?\s+\+(\d+)(?:,(\d+))?\s+@@(.*)$/,
+    );
+    if (hunkMatch) {
+      // Flush previous hunk
+      if (currentHunk) hunks.push(currentHunk);
+
+      oldNo = parseInt(hunkMatch[1], 10);
+      newNo = parseInt(hunkMatch[3], 10);
+
+      currentHunk = {
+        header: rawLine.replace(/^@@\s+/, "").replace(/\s+@@.*$/, "").trim(),
+        lines: [],
+      };
+      continue;
+    }
+
+    if (!currentHunk) continue;
+
+    const ch = rawLine[0];
+    let type: DiffLine["type"];
+    let prefix: string;
+    let content: string;
+
+    if (ch === "+") {
+      type = "add";
+      prefix = "+";
+      content = rawLine.slice(1);
+    } else if (ch === "-") {
+      type = "del";
+      prefix = "−";
+      content = rawLine.slice(1);
+    } else if (ch === " ") {
+      type = "ctx";
+      prefix = " ";
+      content = rawLine.slice(1);
+    } else if (rawLine === "\\ No newline at end of file") {
+      type = "ctx";
+      prefix = " ";
+      content = rawLine;
+    } else {
+      // Could be --- or +++ header lines; skip or treat as ctx
+      if (rawLine.startsWith("---") || rawLine.startsWith("+++")) continue;
+      type = "ctx";
+      prefix = " ";
+      content = rawLine;
+    }
+
+    const line: DiffLine = {
+      type,
+      prefix,
+      content,
+      oldNo: type === "add" ? "" : String(oldNo),
+      newNo: type === "del" ? "" : String(newNo),
+    };
+
+    if (type !== "add") oldNo++;
+    if (type !== "del") newNo++;
+
+    currentHunk.lines.push(line);
+    totalLines++;
+  }
+
+  if (currentHunk) hunks.push(currentHunk);
+  return hunks;
+}
+</script>
+
+<style scoped>
+.diff-preview {
+  --diff-add-bg: rgba(70, 200, 70, 0.12);
+  --diff-add-border: rgba(70, 200, 70, 0.35);
+  --diff-del-bg: rgba(255, 100, 100, 0.12);
+  --diff-del-border: rgba(255, 100, 100, 0.35);
+  --diff-hunk-bg: #e8f0fe;
+  --diff-hunk-border: rgba(100, 150, 220, 0.3);
+  --diff-line-no: rgba(0, 0, 0, 0.35);
+  --diff-border: rgba(0, 0, 0, 0.08);
+
+  margin: 4px 0;
+  border: 1px solid var(--diff-border);
+  border-radius: 8px;
+  overflow: hidden;
+  font-family: ui-monospace, SFMono-Regular, Menlo, Monaco, Consolas, monospace;
+  font-size: 12px;
+  line-height: 1.55;
+}
+
+.diff-preview.is-dark {
+  --diff-add-bg: rgba(70, 200, 70, 0.16);
+  --diff-add-border: rgba(70, 200, 70, 0.3);
+  --diff-del-bg: rgba(255, 100, 100, 0.16);
+  --diff-del-border: rgba(255, 100, 100, 0.3);
+  --diff-hunk-bg: rgba(100, 150, 255, 0.12);
+  --diff-hunk-border: rgba(100, 150, 255, 0.2);
+  --diff-line-no: rgba(255, 255, 255, 0.35);
+  --diff-border: rgba(255, 255, 255, 0.1);
+}
+
+/* ── Header ─────────────────────────────────────────────────────── */
+
+.diff-header {
+  display: flex;
+  align-items: center;
+  justify-content: space-between;
+  width: 100%;
+  padding: 8px 12px;
+  background: rgba(0, 0, 0, 0.02);
+  border: 0;
+  cursor: pointer;
+  font: inherit;
+  color: inherit;
+  user-select: none;
+}
+
+.diff-preview.is-dark .diff-header {
+  background: rgba(255, 255, 255, 0.03);
+}
+
+.diff-header:hover {
+  background: rgba(0, 0, 0, 0.05);
+}
+
+.diff-preview.is-dark .diff-header:hover {
+  background: rgba(255, 255, 255, 0.06);
+}
+
+.diff-header-left {
+  display: flex;
+  align-items: center;
+  gap: 8px;
+  min-width: 0;
+}
+
+.diff-header-right {
+  display: flex;
+  align-items: center;
+  gap: 6px;
+  flex-shrink: 0;
+}
+
+.diff-header-icon {
+  color: rgba(var(--v-theme-on-surface), 0.55);
+  flex-shrink: 0;
+}
+
+.diff-file-path {
+  font-size: 12px;
+  font-weight: 600;
+  white-space: nowrap;
+  overflow: hidden;
+  text-overflow: ellipsis;
+  color: rgba(var(--v-theme-on-surface), 0.8);
+}
+
+.diff-stats {
+  font-size: 11px;
+  font-weight: 600;
+  font-family: ui-monospace, SFMono-Regular, Menlo, Monaco, Consolas, monospace;
+  color: rgba(var(--v-theme-on-surface), 0.55);
+  white-space: nowrap;
+}
+
+.diff-chevron {
+  color: rgba(var(--v-theme-on-surface), 0.45);
+  transition: transform 0.2s ease;
+}
+
+.diff-chevron.expanded {
+  transform: rotate(90deg);
+}
+
+/* ── Summary text ────────────────────────────────────────────────── */
+
+.diff-summary-text {
+  padding: 4px 12px 8px;
+  font-size: 11px;
+  color: rgba(var(--v-theme-on-surface), 0.55);
+  font-family: inherit;
+}
+
+/* ── Body ────────────────────────────────────────────────────────── */
+
+.diff-body {
+  border-top: 1px solid var(--diff-border);
+}
+
+.diff-truncation-warning {
+  padding: 6px 12px;
+  font-size: 11px;
+  color: rgba(var(--v-theme-on-surface), 0.55);
+  background: rgba(var(--v-theme-warning), 0.08);
+  border-bottom: 1px solid var(--diff-border);
+}
+
+/* ── Hunk ────────────────────────────────────────────────────────── */
+
+.diff-hunk + .diff-hunk {
+  border-top: 1px solid rgba(var(--v-theme-on-surface), 0.04);
+}
+
+.hunk-header {
+  padding: 4px 12px;
+  font-size: 11px;
+  font-weight: 600;
+  color: rgba(var(--v-theme-on-surface), 0.55);
+  background: var(--diff-hunk-bg);
+  border-bottom: 1px solid var(--diff-hunk-border);
+}
+
+/* ── Diff line ───────────────────────────────────────────────────── */
+
+.diff-line {
+  display: flex;
+  align-items: baseline;
+  padding: 1px 12px;
+  min-height: 20px;
+  transition: background 0.1s ease;
+}
+
+.diff-line:hover {
+  filter: brightness(0.94);
+}
+
+.diff-preview.is-dark .diff-line:hover {
+  filter: brightness(1.15);
+}
+
+.diff-line.add {
+  background: var(--diff-add-bg);
+  border-left: 3px solid var(--diff-add-border);
+}
+
+.diff-line.del {
+  background: var(--diff-del-bg);
+  border-left: 3px solid var(--diff-del-border);
+}
+
+.diff-line.ctx {
+  border-left: 3px solid transparent;
+}
+
+/* ── Line numbers ────────────────────────────────────────────────── */
+
+.line-number {
+  width: 36px;
+  flex-shrink: 0;
+  text-align: right;
+  padding-right: 8px;
+  color: var(--diff-line-no);
+  user-select: none;
+}
+
+.line-number.new {
+  padding-right: 0;
+  padding-left: 8px;
+}
+
+/* ── Prefix and content ──────────────────────────────────────────── */
+
+.line-prefix {
+  width: 14px;
+  flex-shrink: 0;
+  text-align: center;
+  font-weight: 700;
+  user-select: none;
+}
+
+.diff-line.add .line-prefix {
+  color: #2da44e;
+}
+
+.diff-preview.is-dark .diff-line.add .line-prefix {
+  color: #57ab5a;
+}
+
+.diff-line.del .line-prefix {
+  color: #cf222e;
+}
+
+.diff-preview.is-dark .diff-line.del .line-prefix {
+  color: #f47067;
+}
+
+.line-content {
+  white-space: pre-wrap;
+  word-break: break-all;
+  min-width: 0;
+  padding-left: 4px;
+}
+
+/* ── Show more ───────────────────────────────────────────────────── */
+
+.diff-overflow-bar {
+  border-top: 1px solid var(--diff-border);
+  padding: 6px 12px;
+}
+
+.diff-show-more {
+  background: none;
+  border: 0;
+  padding: 0;
+  font: inherit;
+  font-size: 11px;
+  color: rgba(var(--v-theme-on-surface), 0.5);
+  cursor: pointer;
+}
+
+.diff-show-more:hover {
+  color: rgba(var(--v-theme-on-surface), 0.75);
+  text-decoration: underline;
+}
+
+/* ── Collapsed state ─────────────────────────────────────────────── */
+
+.diff-preview.collapsed .diff-body,
+.diff-preview.collapsed .diff-summary-text {
+  display: none;
+}
+</style>
diff --git a/dashboard/src/components/chat/message_list_comps/ToolCallCard.vue b/dashboard/src/components/chat/message_list_comps/ToolCallCard.vue
index 7ba80905a6..19296d303d 100644
--- a/dashboard/src/components/chat/message_list_comps/ToolCallCard.vue
+++ b/dashboard/src/components/chat/message_list_comps/ToolCallCard.vue
@@ -23,26 +23,72 @@
         </code>
       </div>
 
-      <div class="tool-call-detail-row">
+      <!-- ── Args: key-value table ──────────────────────────── -->
+      <div v-if="argEntries.length" class="tool-call-detail-row">
         <span class="detail-label">Args:</span>
-        <pre class="detail-value detail-json">{{
-          JSON.stringify(toolCall.args, null, 2)
-        }}</pre>
+        <div class="args-table">
+          <div
+            v-for="(entry, i) in displayedArgEntries"
+            :key="i"
+            class="args-row"
+            :class="{ clickable: entry.long }"
+            @click="entry.long && toggleArgExpand(i)"
+          >
+            <span class="args-key">{{ entry.icon }}{{ entry.key }}</span>
+            <span class="args-value">{{ entry.display }}</span>
+            <span v-if="entry.long && !expandedArgs.has(i)" class="args-expand-hint">…</span>
+          </div>
+          <div
+            v-if="argEntries.length > maxVisibleArgs"
+            class="args-row args-more"
+            @click="showAllArgs = !showAllArgs"
+          >
+            <span class="args-key"></span>
+            <span class="args-value args-more-text">
+              {{ showAllArgs
+                ? 'Show fewer'
+                : `+${argEntries.length - maxVisibleArgs} more` }}
+            </span>
+          </div>
+        </div>
       </div>
 
-      <div v-if="toolCall.result" class="tool-call-detail-row">
+      <!-- ── Result ────────────────────────────────────────── -->
+      <div
+        v-if="isEditTool && editToolDiff"
+        class="tool-call-detail-row"
+      >
+        <span class="detail-label">Result:</span>
+        <DiffPreview
+          :content="editToolDiff"
+          :file-path="editToolFilePath"
+          :summary="editToolSummary"
+          :is-dark="isDark"
+          :max-lines="25"
+          :collapsible="false"
+        />
+      </div>
+
+      <div
+        v-else-if="toolCall.result"
+        class="tool-call-detail-row"
+      >
         <span class="detail-label">Result:</span>
-        <pre class="detail-value detail-json detail-result">{{
-          formattedResult
-        }}</pre>
+        <ToolResultView
+          :tool-name="toolCall.name ?? ''"
+          :result="toolCall.result ?? ''"
+          :tool-args="toolCall.args"
+        />
       </div>
     </div>
   </div>
 </template>
 
 <script setup>
-import { computed, onMounted, onUnmounted, ref } from "vue";
+import { computed, onMounted, onUnmounted, reactive, ref } from "vue";
 import { useModuleI18n } from "@/i18n/composables";
+import DiffPreview from "./DiffPreview.vue";
+import ToolResultView from "./ToolResultView.vue";
 
 const props = defineProps({
   toolCall: {
@@ -73,20 +119,114 @@ const elapsedTime = computed(() => {
 
 const displayToolName = computed(() => props.toolCall.name || "tool");
 
+// ── Icons ─────────────────────────────────────────────────────
+
 const toolCallIcon = computed(() => {
   const name = String(props.toolCall.name || "");
-  if (name === "astrbot_execute_ipython" || name === "astrbot_execute_python") {
-    return "mdi-code-json";
+  if (name === "astrbot_file_read_tool") return "mdi-file-document-outline";
+  if (name === "astrbot_file_write_tool") return "mdi-content-save-outline";
+  if (name === "astrbot_file_edit_tool") return "mdi-file-document-edit-outline";
+  if (name === "astrbot_grep_tool") return "mdi-magnify";
+  if (name === "astrbot_execute_shell") return "mdi-console-line";
+  if (name === "astrbot_execute_python" || name === "astrbot_execute_ipython") return "mdi-language-python";
+  if (name === "astrbot_upload_file") return "mdi-upload-outline";
+  if (name === "astrbot_download_file") return "mdi-download-outline";
+  if (name.includes("web_search") || name.includes("tavily")) return "mdi-web";
+  return "mdi-wrench";
+});
+
+// ── Args display ──────────────────────────────────────────────
+
+const maxVisibleArgs = 5;
+const showAllArgs = ref(false);
+const expandedArgs = reactive(new Set());
+
+const argEntries = computed(() => {
+  const args = props.toolCall.args;
+  if (!args || typeof args !== "object") return [];
+  return Object.entries(args).map(([key, value]) => {
+    const raw = value === null || value === undefined ? "—" : String(value);
+    const long = raw.length > 60;
+    return {
+      key,
+      raw,
+      long,
+      icon: argIcon(key),
+      display: long ? raw.slice(0, 60) : raw,
+    };
+  });
+});
+
+const displayedArgEntries = computed(() => {
+  const entries = argEntries.value;
+  if (showAllArgs.value) return entries;
+  return entries.slice(0, maxVisibleArgs);
+});
+
+function argIcon(key) {
+  const k = key.toLowerCase();
+  if (k.includes("path") || k === "file" || k.includes("dir")) return "📁 ";
+  if (k.includes("content") || k === "old" || k === "new" || k === "text") return "📝 ";
+  if (k.includes("pattern") || k.includes("query") || k.includes("search")) return "🔍 ";
+  if (k.includes("command") || k.includes("cmd")) return "⚡ ";
+  if (k.includes("code") || k.includes("python")) return "🐍 ";
+  if (k.includes("replace_all")) return "🔄 ";
+  if (k.includes("offset") || k.includes("limit") || k.includes("result_limit")) return "#⃣ ";
+  return "";
+}
+
+function toggleArgExpand(index) {
+  const entry = argEntries.value[index];
+  if (!entry || !entry.long) return;
+  if (expandedArgs.has(index)) {
+    expandedArgs.delete(index);
+  } else {
+    expandedArgs.add(index);
   }
-  if (name.includes("web_search") || name.includes("tavily")) {
-    return "mdi-web";
+  // Update display
+  const displayed = displayedArgEntries.value;
+  for (let i = 0; i < displayed.length; i++) {
+    const e = argEntries.value.find((x) => x.key === displayed[i].key);
+    if (e && e.long) {
+      displayed[i].display = expandedArgs.has(i) ? e.raw : e.raw.slice(0, 60);
+    }
   }
-  if (name === "astrbot_execute_shell") {
-    return "mdi-console-line";
+}
+
+// ── file_edit_tool diff rendering ─────────────────────────────
+
+const isEditTool = computed(
+  () => props.toolCall.name === "astrbot_file_edit_tool",
+);
+
+const editToolDiff = computed(() => {
+  if (!isEditTool.value) return "";
+  const raw = props.toolCall.result ?? "";
+  const match = raw.match(/```diff\s*\n?([\s\S]*?)```/);
+  return match ? match[1] : raw;
+});
+
+const editToolFilePath = computed(() => {
+  if (!isEditTool.value) return "";
+  const raw = props.toolCall.result ?? "";
+  const match = raw.match(/^Edited\s+(.+?)\./m);
+  return match ? match[1] : "";
+});
+
+const editToolSummary = computed(() => {
+  if (!isEditTool.value) return "";
+  const raw = props.toolCall.result ?? "";
+  const lines = raw.split("\n");
+  const statusParts = [];
+  for (const line of lines) {
+    if (line.startsWith("Diff:") || line.startsWith("```")) break;
+    if (line.trim()) statusParts.push(line.trim());
   }
-  return "mdi-wrench";
+  return statusParts.join("\n");
 });
 
+// ── Duration ──────────────────────────────────────────────────
+
 const toolCallDuration = computed(() => {
   const startTime = Number(props.toolCall.ts);
   if (!Number.isFinite(startTime) || startTime <= 0) return "";
@@ -96,16 +236,6 @@ const toolCallDuration = computed(() => {
   return elapsedTime.value;
 });
 
-const formattedResult = computed(() => {
-  if (!props.toolCall.result) return "";
-  try {
-    const parsed = JSON.parse(props.toolCall.result);
-    return JSON.stringify(parsed, null, 2);
-  } catch {
-    return props.toolCall.result;
-  }
-});
-
 const formatDuration = (seconds) => {
   if (!Number.isFinite(seconds) || seconds < 0) return "";
   if (seconds < 1) {
@@ -226,12 +356,81 @@ onUnmounted(() => {
 .detail-value {
   font-size: 12px;
   color: rgba(var(--v-theme-on-surface), 0.8);
-  background-color: transparent;
-  padding: 0;
+  word-break: break-all;
+}
+
+/* ── Args table ─────────────────────────────────────────────── */
+
+.args-table {
+  border: 1px solid rgba(var(--v-theme-on-surface), 0.08);
   border-radius: 4px;
+  overflow: hidden;
+}
+
+.args-row {
+  display: flex;
+  align-items: baseline;
+  padding: 3px 8px;
+  font-size: 11.5px;
+  line-height: 1.55;
+}
+
+.args-row + .args-row {
+  border-top: 1px solid rgba(var(--v-theme-on-surface), 0.05);
+}
+
+.args-row.clickable {
+  cursor: pointer;
+}
+
+.args-row.clickable:hover {
+  background: rgba(var(--v-theme-on-surface), 0.03);
+}
+
+.args-key {
+  flex-shrink: 0;
+  width: 110px;
+  font-family: ui-monospace, SFMono-Regular, Menlo, Monaco, Consolas, monospace;
+  font-size: 11px;
+  color: rgba(var(--v-theme-on-surface), 0.5);
+  font-weight: 600;
+  overflow: hidden;
+  text-overflow: ellipsis;
+  white-space: nowrap;
+  padding-right: 8px;
+}
+
+.args-value {
+  font-family: ui-monospace, SFMono-Regular, Menlo, Monaco, Consolas, monospace;
+  font-size: 11px;
+  color: rgba(var(--v-theme-on-surface), 0.8);
+  white-space: pre-wrap;
   word-break: break-all;
+  min-width: 0;
 }
 
+.args-expand-hint {
+  color: rgba(var(--v-theme-on-surface), 0.35);
+  font-style: italic;
+  margin-left: 4px;
+}
+
+.args-more {
+  cursor: pointer;
+  background: rgba(var(--v-theme-on-surface), 0.02);
+}
+
+.args-more:hover {
+  background: rgba(var(--v-theme-on-surface), 0.05);
+}
+
+.args-more-text {
+  color: rgba(var(--v-theme-on-surface), 0.45) !important;
+  font-style: italic;
+}
+
+/* ── Legacy detail rows ─────────────────────────────────────── */
+
 .detail-json {
   font-family: ui-monospace, SFMono-Regular, Menlo, Monaco, Consolas, monospace;
   white-space: pre-wrap;
@@ -240,32 +439,12 @@ onUnmounted(() => {
   margin: 0;
 }
 
-.detail-result {
-  max-height: 300px;
-  background-color: transparent;
-}
-
 .animate-fade-in {
   animation: fadeIn 0.2s ease-in-out;
 }
 
 @keyframes fadeIn {
-  from {
-    opacity: 0;
-  }
-
-  to {
-    opacity: 1;
-  }
-}
-
-@keyframes spin {
-  from {
-    transform: rotate(0deg);
-  }
-
-  to {
-    transform: rotate(360deg);
-  }
+  from { opacity: 0; }
+  to { opacity: 1; }
 }
 </style>
diff --git a/dashboard/src/components/chat/message_list_comps/ToolResultView.vue b/dashboard/src/components/chat/message_list_comps/ToolResultView.vue
new file mode 100644
index 0000000000..ff66bc49b7
--- /dev/null
+++ b/dashboard/src/components/chat/message_list_comps/ToolResultView.vue
@@ -0,0 +1,485 @@
+<template>
+  <div class="tool-result-view">
+    <!-- ── file_read_tool ──────────────────────────────────────── -->
+    <template v-if="toolName === 'astrbot_file_read_tool'">
+      <div class="result-header">
+        <v-icon size="14" class="result-header-icon">mdi-file-document-outline</v-icon>
+        <span class="result-header-text">{{ readFilePath }}</span>
+        <span v-if="readFileRange" class="result-header-meta">{{ readFileRange }}</span>
+      </div>
+      <div
+        v-if="shikiReady && detectedLanguage !== 'text'"
+        class="result-code result-code-shiki"
+        v-html="highlightedCode"
+      ></div>
+      <pre v-else class="result-code">{{ readFileContent }}</pre>
+    </template>
+
+    <!-- ── file_write_tool ─────────────────────────────────────── -->
+    <template v-else-if="toolName === 'astrbot_file_write_tool'">
+      <div class="result-status" :class="resultOk ? 'success' : 'error'">
+        <v-icon size="16">{{ resultOk ? 'mdi-check-circle' : 'mdi-alert-circle' }}</v-icon>
+        <span>{{ resultOk ? writeFilePath : resultText }}</span>
+      </div>
+    </template>
+
+    <!-- ── grep_tool ───────────────────────────────────────────── -->
+    <template v-else-if="toolName === 'astrbot_grep_tool'">
+      <div v-if="grepLines.length" class="result-header">
+        <v-icon size="14" class="result-header-icon">mdi-magnify</v-icon>
+        <span class="result-header-text">Found {{ grepLines.length }} match(es)</span>
+      </div>
+      <div v-if="grepLines.length" class="grep-results">
+        <div
+          v-for="(line, i) in grepLines"
+          :key="i"
+          class="grep-line"
+        >
+          <span v-if="line.file" class="grep-file">{{ line.file }}</span>
+          <span v-if="line.lineno" class="grep-lineno">{{ line.lineno }}</span>
+          <span class="grep-text">{{ line.text }}</span>
+        </div>
+      </div>
+      <div v-if="grepTruncated" class="result-truncated">{{ grepTruncated }}</div>
+    </template>
+
+    <!-- ── execute_shell ───────────────────────────────────────── -->
+    <template v-else-if="toolName === 'astrbot_execute_shell'">
+      <pre class="result-terminal" v-text="resultText"></pre>
+      <div v-if="shellExitCode != null" class="result-exit-code" :class="shellExitCode === 0 ? 'success' : 'error'">
+        Exit code: {{ shellExitCode }}
+      </div>
+    </template>
+
+    <!-- ── fallback ────────────────────────────────────────────── -->
+    <template v-else>
+      <div class="result-status" :class="resultOk ? 'success' : 'error'">
+        <v-icon size="16">{{ resultOk ? 'mdi-check-circle' : 'mdi-alert-circle' }}</v-icon>
+        <span>{{ resultText }}</span>
+      </div>
+    </template>
+
+    <!-- ── fallback ────────────────────────────────────────────── -->
+    <template v-else>
+      <pre class="result-raw">{{ formattedResult }}</pre>
+    </template>
+  </div>
+</template>
+
+<script setup lang="ts">
+import { computed, onMounted, ref } from "vue";
+import {
+  ensureShikiLanguages,
+  escapeHtml,
+  renderShikiCode,
+} from "@/utils/shiki";
+
+const props = defineProps<{
+  toolName: string;
+  result: string;
+  toolArgs?: Record<string, any>;
+}>();
+
+// ── Shiki syntax highlighting ─────────────────────────────────────
+
+const EXT_TO_LANG: Record<string, string> = {
+  ".py": "python",
+  ".js": "javascript",
+  ".mjs": "javascript",
+  ".cjs": "javascript",
+  ".ts": "typescript",
+  ".tsx": "tsx",
+  ".jsx": "jsx",
+  ".vue": "vue",
+  ".json": "json",
+  ".yaml": "yaml",
+  ".yml": "yaml",
+  ".sh": "bash",
+  ".bash": "bash",
+  ".zsh": "bash",
+  ".css": "css",
+  ".html": "html",
+  ".htm": "html",
+  ".xml": "xml",
+  ".svg": "xml",
+  ".md": "markdown",
+  ".sql": "sql",
+  ".java": "java",
+  ".ini": "ini",
+  ".diff": "diff",
+  ".patch": "diff",
+  ".ps1": "powershell",
+  ".dockerfile": "dockerfile",
+  ".txt": "text",
+  // C / C++ / Go / Rust
+  ".c": "c",
+  ".h": "c",
+  ".cpp": "cpp",
+  ".cc": "cpp",
+  ".cxx": "cpp",
+  ".hpp": "cpp",
+  ".c++": "cpp",
+  ".go": "go",
+  ".rs": "rust",
+};
+
+function detectLanguage(filePath: string): string {
+  const m = filePath.match(/\.([\w]+)$/i);
+  if (!m) return "text";
+  const key = "." + m[1].toLowerCase();
+  return EXT_TO_LANG[key] || "text";
+}
+
+const shikiHighlighter = ref<any>(null);
+const shikiReady = ref(false);
+
+// ── helpers ──────────────────────────────────────────────────────
+
+const resultText = computed(() => (props.result ?? "").trim());
+
+const resultOk = computed(() => {
+  const t = resultText.value.toLowerCase();
+  return t.includes("success") || t.includes("successfully") || t.startsWith("file written") || t.startsWith("file uploaded") || t.startsWith("file downloaded");
+});
+
+const formattedResult = computed(() => {
+  if (!props.result) return "";
+  try {
+    return JSON.stringify(JSON.parse(props.result), null, 2);
+  } catch {
+    return props.result;
+  }
+});
+
+// ── file_read_tool ──────────────────────────────────────────────
+
+const readFilePath = computed(() => {
+  // Prefer tool args when available
+  if (props.toolArgs?.path) return String(props.toolArgs.path);
+  // Try to parse from a formatted header like "path (lines 0-50, 82 lines total)"
+  const m = resultText.value.match(/^(.+?)\s*\(/m);
+  if (m) return m[1];
+  // fallback: first line of raw content
+  return resultText.value.split("\n")[0]?.slice(0, 120) || "";
+});
+
+const readFileRange = computed(() => {
+  // Build from args when offset/limit are provided
+  if (props.toolArgs) {
+    const offset = props.toolArgs.offset;
+    const limit = props.toolArgs.limit;
+    if (offset !== undefined || limit !== undefined) {
+      const parts: string[] = [];
+      if (offset !== undefined && offset !== null) parts.push(`offset ${offset}`);
+      if (limit !== undefined && limit !== null) parts.push(`limit ${limit}`);
+      if (parts.length) return parts.join(", ");
+    }
+  }
+  // Fallback: parse from a formatted header
+  const m = resultText.value.match(/\((Lines?\s+[\d–\-]+)/i);
+  return m ? m[1] : "";
+});
+
+const readFileContent = computed(() => {
+  const lines = resultText.value.split("\n");
+  // Detect if the first non-empty line looks like a path+range header
+  // e.g. "F:/path/to/file (Lines 0-50, 82 lines total)"
+  const firstLine = lines[0]?.trim() || "";
+  const hasHeader =
+    /^.+?\s*\((Lines?\s+[\d–\-]+)/i.test(firstLine);
+  if (hasHeader) {
+    // Strip the header line and any following blank separator
+    let i = 1;
+    while (i < lines.length && !lines[i].trim()) i++;
+    return lines.slice(i).join("\n");
+  }
+  // No recognizable header — show full content as-is
+  return resultText.value;
+});
+
+// ── Shiki highlighting (file_read_tool) ──────────────────────────
+
+const detectedLanguage = computed(() =>
+  detectLanguage(readFilePath.value),
+);
+
+const highlightedCode = computed(() => {
+  if (!shikiReady.value || !shikiHighlighter.value) return "";
+  const lang = detectedLanguage.value;
+  const code = readFileContent.value;
+  if (!code) return "";
+  try {
+    return renderShikiCode(
+      shikiHighlighter.value,
+      code,
+      lang,
+      // ToolResultView doesn't receive isDark directly; Shiki's
+      // default dual-theme (light/dark) output works through CSS
+      // media queries, so we pass "auto" here.
+      "auto",
+    );
+  } catch (err) {
+    console.error("Failed to highlight code with Shiki:", err);
+    return `<pre><code>${escapeHtml(code)}</code></pre>`;
+  }
+});
+
+onMounted(async () => {
+  try {
+    shikiHighlighter.value = await ensureShikiLanguages();
+    shikiReady.value = true;
+  } catch (err) {
+    console.error("Failed to initialize Shiki:", err);
+  }
+});
+
+// ── file_write_tool ─────────────────────────────────────────────
+
+const writeFilePath = computed(() => {
+  const m = resultText.value.match(/^File written successfully:\s*(.+)/i);
+  if (m) return m[1];
+  return resultText.value;
+});
+
+// ── grep_tool ──────────────────────────────────────────────────
+
+const grepLines = computed(() => {
+  const lines = resultText.value.split("\n");
+  const parsed: Array<{ file?: string; lineno?: string; text: string }> = [];
+  for (const line of lines) {
+    if (!line.trim()) continue;
+    if (line.startsWith("[Truncated")) continue;
+    // Try "file:lineno:text" pattern
+    const m = line.match(/^(.+?):(\d+):(.*)$/);
+    if (m) {
+      parsed.push({ file: m[1], lineno: m[2], text: m[3] });
+    } else {
+      // Try "file-path:text" or "file:text" (ripgrep with -n)
+      const m2 = line.match(/^(.+?):(\d+)[:\-](.*)$/);
+      if (m2) {
+        parsed.push({ file: m2[1], lineno: m2[2], text: m2[3] });
+      } else {
+        parsed.push({ text: line });
+      }
+    }
+  }
+  return parsed;
+});
+
+const grepTruncated = computed(() => {
+  const m = resultText.value.match(/\[Truncated.+\]/);
+  return m ? m[0] : "";
+});
+
+// ── execute_shell ──────────────────────────────────────────────
+
+const shellExitCode = computed(() => {
+  return null; // Shell output doesn't include exit code in result string
+  // We could try to parse it but it's not reliable
+});
+
+</script>
+
+<style scoped>
+.tool-result-view {
+  font-size: 12px;
+  line-height: 1.5;
+}
+
+/* ── Header ─────────────────────────────────────────────────── */
+
+.result-header {
+  display: flex;
+  align-items: center;
+  gap: 6px;
+  margin-bottom: 6px;
+  padding: 4px 8px;
+  border-radius: 4px;
+  background: rgba(var(--v-theme-on-surface), 0.04);
+  font-size: 11.5px;
+}
+
+.result-header-icon {
+  color: rgba(var(--v-theme-on-surface), 0.5);
+  flex-shrink: 0;
+}
+
+.result-header-text {
+  font-weight: 600;
+  color: rgba(var(--v-theme-on-surface), 0.75);
+  overflow: hidden;
+  text-overflow: ellipsis;
+  white-space: nowrap;
+  min-width: 0;
+}
+
+.result-header-meta {
+  color: rgba(var(--v-theme-on-surface), 0.45);
+  flex-shrink: 0;
+  margin-left: auto;
+}
+
+/* ── Code block ──────────────────────────────────────────────── */
+
+.result-code {
+  margin: 0;
+  padding: 8px 10px;
+  border-radius: 4px;
+  background: rgba(var(--v-theme-on-surface), 0.04);
+  font-family: ui-monospace, SFMono-Regular, Menlo, Monaco, Consolas, monospace;
+  font-size: 11.5px;
+  line-height: 1.55;
+  white-space: pre-wrap;
+  word-break: break-all;
+  max-height: 300px;
+  overflow-y: auto;
+}
+
+/* Shiki 高亮容器 — 继承 code block 布局，清除 Shiki 默认样式 */
+.result-code-shiki {
+  padding: 0;
+  border-radius: 6px;
+  overflow: hidden;
+}
+
+.result-code-shiki :deep(pre.shiki) {
+  margin: 0;
+  padding: 10px 12px;
+  border-radius: 6px;
+  overflow: auto;
+  max-height: 300px;
+  font-size: 11.5px;
+  line-height: 1.55;
+  tab-size: 4;
+}
+
+.result-code-shiki :deep(pre.shiki code) {
+  display: block;
+  padding: 0;
+  background: transparent;
+  font-family: inherit;
+  font-size: inherit;
+  line-height: inherit;
+}
+
+/* ── Terminal block ──────────────────────────────────────────── */
+
+.result-terminal {
+  margin: 0;
+  padding: 8px 10px;
+  border-radius: 4px;
+  background: rgba(0, 0, 0, 0.85);
+  color: #e0e0e0;
+  font-family: ui-monospace, SFMono-Regular, Menlo, Monaco, Consolas, monospace;
+  font-size: 11.5px;
+  line-height: 1.55;
+  white-space: pre-wrap;
+  word-break: break-all;
+  max-height: 300px;
+  overflow-y: auto;
+}
+
+/* ── Status badge ────────────────────────────────────────────── */
+
+.result-status {
+  display: flex;
+  align-items: center;
+  gap: 6px;
+  padding: 6px 10px;
+  border-radius: 4px;
+  font-size: 12px;
+}
+
+.result-status.success {
+  background: rgba(70, 200, 70, 0.08);
+  color: #2da44e;
+}
+
+.result-status.error {
+  background: rgba(255, 100, 100, 0.08);
+  color: #cf222e;
+}
+
+/* ── Exit code ───────────────────────────────────────────────── */
+
+.result-exit-code {
+  margin-top: 4px;
+  padding: 2px 8px;
+  border-radius: 3px;
+  font-size: 11px;
+  font-weight: 600;
+  display: inline-block;
+}
+
+.result-exit-code.success {
+  background: rgba(70, 200, 70, 0.1);
+  color: #2da44e;
+}
+
+.result-exit-code.error {
+  background: rgba(255, 100, 100, 0.1);
+  color: #cf222e;
+}
+
+/* ── Grep results ────────────────────────────────────────────── */
+
+.grep-results {
+  max-height: 300px;
+  overflow-y: auto;
+}
+
+.grep-line {
+  display: flex;
+  gap: 0;
+  padding: 1px 0;
+  font-family: ui-monospace, SFMono-Regular, Menlo, Monaco, Consolas, monospace;
+  font-size: 11.5px;
+  line-height: 1.55;
+}
+
+.grep-file {
+  color: rgba(var(--v-theme-on-surface), 0.45);
+  flex-shrink: 0;
+  overflow: hidden;
+  text-overflow: ellipsis;
+  max-width: 140px;
+  white-space: nowrap;
+  padding-right: 4px;
+}
+
+.grep-lineno {
+  color: rgba(var(--v-theme-on-surface), 0.35);
+  flex-shrink: 0;
+  min-width: 32px;
+  text-align: right;
+  padding-right: 6px;
+}
+
+.grep-text {
+  color: rgba(var(--v-theme-on-surface), 0.8);
+  white-space: pre-wrap;
+  word-break: break-all;
+  min-width: 0;
+}
+
+.result-truncated {
+  padding: 4px 0;
+  font-size: 11px;
+  color: rgba(var(--v-theme-on-surface), 0.45);
+  font-style: italic;
+}
+
+/* ── Raw fallback ────────────────────────────────────────────── */
+
+.result-raw {
+  margin: 0;
+  font-family: ui-monospace, SFMono-Regular, Menlo, Monaco, Consolas, monospace;
+  font-size: 11.5px;
+  line-height: 1.55;
+  white-space: pre-wrap;
+  word-break: break-all;
+  max-height: 300px;
+  overflow-y: auto;
+  color: rgba(var(--v-theme-on-surface), 0.8);
+}
+</style>
diff --git a/dashboard/src/components/shared/ThemeAwareMarkdownCodeBlock.vue b/dashboard/src/components/shared/ThemeAwareMarkdownCodeBlock.vue
index 902cc51113..608edae579 100644
--- a/dashboard/src/components/shared/ThemeAwareMarkdownCodeBlock.vue
+++ b/dashboard/src/components/shared/ThemeAwareMarkdownCodeBlock.vue
@@ -1,5 +1,12 @@
 <template>
+  <DiffPreview
+    v-if="node.lang === 'diff'"
+    :content="String((node as Record<string, unknown>).content ?? (node as Record<string, unknown>).code ?? '')"
+    :is-dark="effectiveIsDark"
+    :max-lines="30"
+  />
   <MarkdownCodeBlockNode
+    v-else
     :key="themeRenderKey"
     v-bind="forwardedBindings"
   >
@@ -16,6 +23,7 @@
 import { computed, inject, type Ref } from "vue";
 import { MarkdownCodeBlockNode } from "markstream-vue";
 import { useAttrs } from "vue";
+import DiffPreview from "@/components/chat/message_list_comps/DiffPreview.vue";
 
 defineOptions({
   inheritAttrs: false,
diff --git a/dashboard/src/utils/shikiLimitedBundle.js b/dashboard/src/utils/shikiLimitedBundle.js
index b0ece53879..89862f7cb4 100644
--- a/dashboard/src/utils/shikiLimitedBundle.js
+++ b/dashboard/src/utils/shikiLimitedBundle.js
@@ -1,9 +1,12 @@
 import { createHighlighterCore } from "shiki/core";
 import { createJavaScriptRegexEngine } from "shiki/engine/javascript";
 import bash from "shiki/langs/bash.mjs";
+import c from "shiki/langs/c.mjs";
+import cpp from "shiki/langs/cpp.mjs";
 import css from "shiki/langs/css.mjs";
 import diff from "shiki/langs/diff.mjs";
 import dockerfile from "shiki/langs/dockerfile.mjs";
+import go from "shiki/langs/go.mjs";
 import html from "shiki/langs/html.mjs";
 import ini from "shiki/langs/ini.mjs";
 import java from "shiki/langs/java.mjs";
@@ -13,6 +16,7 @@ import jsx from "shiki/langs/jsx.mjs";
 import markdown from "shiki/langs/markdown.mjs";
 import powershell from "shiki/langs/powershell.mjs";
 import python from "shiki/langs/python.mjs";
+import rust from "shiki/langs/rust.mjs";
 import sql from "shiki/langs/sql.mjs";
 import tsx from "shiki/langs/tsx.mjs";
 import typescript from "shiki/langs/typescript.mjs";
@@ -26,9 +30,12 @@ import vitesseLight from "shiki/themes/vitesse-light.mjs";
 
 export const LIMITED_SHIKI_LANGUAGES = [
   ...bash,
+  ...c,
+  ...cpp,
   ...css,
   ...diff,
   ...dockerfile,
+  ...go,
   ...html,
   ...ini,
   ...java,
@@ -38,6 +45,7 @@ export const LIMITED_SHIKI_LANGUAGES = [
   ...markdown,
   ...powershell,
   ...python,
+  ...rust,
   ...sql,
   ...tsx,
   ...typescript,
@@ -57,10 +65,16 @@ const BUILT_IN_LANGUAGES = ["text", "plaintext", "plain"];
 
 export const LIMITED_SHIKI_LANGUAGE_ALIASES = {
   bat: "powershell",
+  "c++": "cpp",
+  cc: "cpp",
   cjs: "javascript",
   console: "bash",
   cts: "typescript",
+  cxx: "cpp",
   docker: "dockerfile",
+  golang: "go",
+  h: "c",
+  hpp: "cpp",
   htm: "html",
   js: "javascript",
   md: "markdown",
@@ -71,6 +85,7 @@ export const LIMITED_SHIKI_LANGUAGE_ALIASES = {
   ps1: "powershell",
   pwsh: "powershell",
   py: "python",
+  rs: "rust",
   shell: "bash",
   shellscript: "bash",
   sh: "bash",

From 3ef08eaeb79992cb3a9f893d9d9daf09821dc069 Mon Sep 17 00:00:00 2001
From: elecvoid243 <67492363+elecvoid243@users.noreply.github.com>
Date: Fri, 29 May 2026 20:08:12 +0800
Subject: [PATCH 02/15] Update astrbot/core/tools/computer_tools/edit_engine.py

Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
---
 astrbot/core/tools/computer_tools/edit_engine.py | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/astrbot/core/tools/computer_tools/edit_engine.py b/astrbot/core/tools/computer_tools/edit_engine.py
index 539f13056e..47b5111bc8 100644
--- a/astrbot/core/tools/computer_tools/edit_engine.py
+++ b/astrbot/core/tools/computer_tools/edit_engine.py
@@ -240,14 +240,14 @@ def _block_anchor_replacer(content: str, find: str) -> Iterator[str]:
     search_block_size = len(search_lines)
 
     candidates: list[tuple[int, int]] = []
+    # Limit the search window for the last anchor to avoid matching unrelated lines far away
+    max_window = max(search_block_size * 2, search_block_size + 10)
     for i, line in enumerate(original_lines):
         if line.strip() != first_anchor:
             continue
-        for j in range(i + 2, len(original_lines)):
+        for j in range(i + 2, min(len(original_lines), i + max_window)):
             if original_lines[j].strip() == last_anchor:
                 candidates.append((i, j))
-                break
-
     if not candidates:
         return
 

From 7517a8a76161c951d70610cc018f5c28d08b52bc Mon Sep 17 00:00:00 2001
From: "DESKTOP-02FN3OU\\80423" <804235820@qq.com>
Date: Fri, 29 May 2026 20:16:40 +0800
Subject: [PATCH 03/15] =?UTF-8?q?fix(file=5Fedit=5Ftool):=20=E8=A1=A5?=
 =?UTF-8?q?=E5=85=A8=E6=B2=99=E7=9B=92=E6=A8=A1=E5=BC=8F=E7=BC=BA=E5=A4=B1?=
 =?UTF-8?q?=E7=9A=84=E6=96=87=E4=BB=B6=E9=94=81?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 .../core/tools/computer_tools/edit_engine.py  |  4 +-
 astrbot/core/tools/computer_tools/fs.py       | 90 +++++++++----------
 2 files changed, 42 insertions(+), 52 deletions(-)

diff --git a/astrbot/core/tools/computer_tools/edit_engine.py b/astrbot/core/tools/computer_tools/edit_engine.py
index 539f13056e..36f17e1012 100644
--- a/astrbot/core/tools/computer_tools/edit_engine.py
+++ b/astrbot/core/tools/computer_tools/edit_engine.py
@@ -29,7 +29,7 @@
 _locks_lock = asyncio.Lock()
 
 
-async def _get_lock(path: str) -> asyncio.Lock:
+async def get_file_lock(path: str) -> asyncio.Lock:
     """Get or create an asyncio.Lock for the given file path."""
     resolved = str(Path(path).resolve())
     async with _locks_lock:
@@ -553,7 +553,7 @@ async def edit_file(
     - Preserves BOM if present
     - Returns unified diff of changes
     """
-    lock = await _get_lock(path)
+    lock = await get_file_lock(path)
     async with lock:
         try:
             # Read file
diff --git a/astrbot/core/tools/computer_tools/fs.py b/astrbot/core/tools/computer_tools/fs.py
index 77a755c34f..c1018ae2d6 100644
--- a/astrbot/core/tools/computer_tools/fs.py
+++ b/astrbot/core/tools/computer_tools/fs.py
@@ -59,6 +59,7 @@
     EditResult,
     build_unified_diff,
     edit_file,
+    get_file_lock,
     robust_replace,
 )
 from .util import (
@@ -483,7 +484,6 @@ async def call(
         restricted = _is_restricted_env(context)
 
         try:
-            # ── path resolution ────────────────────────────────────
             if local_env:
                 normalized_path = _normalize_rw_path(
                     path,
@@ -498,13 +498,7 @@ async def call(
             if not normalized_path:
                 raise ValueError("`path` must be a non-empty string.")
 
-            # ── execute edit ───────────────────────────────────────
             if local_env:
-                # Local: full robust edit engine
-                #   BOM preservation  ✅
-                #   CRLF preservation ✅
-                #   file-level lock   ✅
-                #   diff output       ✅
                 result = await edit_file(
                     path=normalized_path,
                     old_string=old,
@@ -513,59 +507,55 @@ async def call(
                     encoding="utf-8",
                 )
             else:
-                # Sandbox: booter-mediated I/O + robust_replace
-                #   fuzzy matching   ✅
-                #   diff output      ✅
-                #   BOM/CRLF         handled by sandbox runtime
-                #   concurrency      handled by sandbox runtime
                 sb = await get_booter(
                     context.context.context,
                     umo,
                 )
-
-                read_result = await sb.fs.read_file(path=normalized_path)
-                if not read_result.get("success", False):
-                    error_detail = str(read_result.get("error", "") or "").strip()
-                    return (
-                        "Error editing file: "
-                        f"{error_detail or 'unknown filesystem read error'}"
+                lock = await get_file_lock(normalized_path)
+                async with lock:
+                    read_result = await sb.fs.read_file(path=normalized_path)
+                    if not read_result.get("success", False):
+                        error_detail = str(read_result.get("error", "") or "").strip()
+                        return (
+                            "Error editing file: "
+                            f"{error_detail or 'unknown filesystem read error'}"
+                        )
+
+                    old_content = read_result.get("content", "")
+
+                    try:
+                        new_content, replacements = robust_replace(
+                            old_content,
+                            old,
+                            new,
+                            replace_all=replace_all,
+                        )
+                    except ValueError as exc:
+                        return f"Error editing file: {exc}"
+
+                    write_result = await sb.fs.write_file(
+                        path=normalized_path,
+                        content=new_content,
                     )
-
-                old_content = read_result.get("content", "")
-
-                try:
-                    new_content, replacements = robust_replace(
+                    if not write_result.get("success", False):
+                        error_detail = str(write_result.get("error", "") or "").strip()
+                        return (
+                            "Error editing file: "
+                            f"{error_detail or 'unknown filesystem write error'}"
+                        )
+
+                    diff = build_unified_diff(
+                        normalized_path,
                         old_content,
-                        old,
-                        new,
-                        replace_all=replace_all,
+                        new_content,
                     )
-                except ValueError as exc:
-                    return f"Error editing file: {exc}"
 
-                write_result = await sb.fs.write_file(
-                    path=normalized_path,
-                    content=new_content,
-                )
-                if not write_result.get("success", False):
-                    error_detail = str(write_result.get("error", "") or "").strip()
-                    return (
-                        "Error editing file: "
-                        f"{error_detail or 'unknown filesystem write error'}"
+                    result = EditResult(
+                        success=True,
+                        replacements=replacements,
+                        diff=diff,
                     )
 
-                diff = build_unified_diff(
-                    normalized_path,
-                    old_content,
-                    new_content,
-                )
-
-                result = EditResult(
-                    success=True,
-                    replacements=replacements,
-                    diff=diff,
-                )
-
             return _format_result(
                 normalized_path,
                 result,

From 3168e36f4ed3c640df107e34128b9faad05b2ae0 Mon Sep 17 00:00:00 2001
From: "DESKTOP-02FN3OU\\80423" <804235820@qq.com>
Date: Fri, 29 May 2026 20:50:28 +0800
Subject: [PATCH 04/15] =?UTF-8?q?fix(file=5Fedit=5Ftool):=20=E5=A4=9A?=
 =?UTF-8?q?=E5=A4=84=E4=BF=AE=E5=A4=8D?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 astrbot/core/tools/computer_tools/edit_engine.py   | 14 ++++++--------
 astrbot/core/tools/computer_tools/fs.py            |  2 +-
 dashboard/src/components/chat/ReasoningSidebar.vue |  5 +----
 .../chat/message_list_comps/ToolCallCard.vue       |  9 +++++----
 .../chat/message_list_comps/ToolResultView.vue     |  8 ++------
 5 files changed, 15 insertions(+), 23 deletions(-)

diff --git a/astrbot/core/tools/computer_tools/edit_engine.py b/astrbot/core/tools/computer_tools/edit_engine.py
index e09f9af7fa..8710b32378 100644
--- a/astrbot/core/tools/computer_tools/edit_engine.py
+++ b/astrbot/core/tools/computer_tools/edit_engine.py
@@ -26,18 +26,16 @@
 # File-level locks to prevent concurrent edits on the same file.
 # Use WeakValueDictionary so locks for deleted files can be garbage-collected.
 _locks: weakref.WeakValueDictionary[str, asyncio.Lock] = weakref.WeakValueDictionary()
-_locks_lock = asyncio.Lock()
 
 
-async def get_file_lock(path: str) -> asyncio.Lock:
+def get_file_lock(path: str) -> asyncio.Lock:
     """Get or create an asyncio.Lock for the given file path."""
     resolved = str(Path(path).resolve())
-    async with _locks_lock:
-        lock = _locks.get(resolved)
-        if lock is None:
-            lock = asyncio.Lock()
-            _locks[resolved] = lock
-        return lock
+    lock = _locks.get(resolved)
+    if lock is None:
+        lock = asyncio.Lock()
+        _locks[resolved] = lock
+    return lock
 
 
 # ---------------------------------------------------------------------------
diff --git a/astrbot/core/tools/computer_tools/fs.py b/astrbot/core/tools/computer_tools/fs.py
index c1018ae2d6..d381ac78e1 100644
--- a/astrbot/core/tools/computer_tools/fs.py
+++ b/astrbot/core/tools/computer_tools/fs.py
@@ -511,7 +511,7 @@ async def call(
                     context.context.context,
                     umo,
                 )
-                lock = await get_file_lock(normalized_path)
+                lock = get_file_lock(normalized_path)
                 async with lock:
                     read_result = await sb.fs.read_file(path=normalized_path)
                     if not read_result.get("success", False):
diff --git a/dashboard/src/components/chat/ReasoningSidebar.vue b/dashboard/src/components/chat/ReasoningSidebar.vue
index 3486839b16..d15dc75bed 100644
--- a/dashboard/src/components/chat/ReasoningSidebar.vue
+++ b/dashboard/src/components/chat/ReasoningSidebar.vue
@@ -95,10 +95,7 @@ function onMouseUp() {
 }
 
 onBeforeUnmount(() => {
-  if (isResizing) {
-    document.removeEventListener("mousemove", onMouseMove);
-    document.removeEventListener("mouseup", onMouseUp);
-  }
+  onMouseUp();
 });
 </script>
 
diff --git a/dashboard/src/components/chat/message_list_comps/ToolCallCard.vue b/dashboard/src/components/chat/message_list_comps/ToolCallCard.vue
index 19296d303d..eeb8a45ba3 100644
--- a/dashboard/src/components/chat/message_list_comps/ToolCallCard.vue
+++ b/dashboard/src/components/chat/message_list_comps/ToolCallCard.vue
@@ -36,7 +36,7 @@
           >
             <span class="args-key">{{ entry.icon }}{{ entry.key }}</span>
             <span class="args-value">{{ entry.display }}</span>
-            <span v-if="entry.long && !expandedArgs.has(i)" class="args-expand-hint">…</span>
+            <span v-if="entry.long && !expandedArgs[i]" class="args-expand-hint">…</span>
           </div>
           <div
             v-if="argEntries.length > maxVisibleArgs"
@@ -139,20 +139,21 @@ const toolCallIcon = computed(() => {
 
 const maxVisibleArgs = 5;
 const showAllArgs = ref(false);
-const expandedArgs = reactive(new Set());
+const expandedArgs = reactive({});
 
 const argEntries = computed(() => {
   const args = props.toolCall.args;
   if (!args || typeof args !== "object") return [];
-  return Object.entries(args).map(([key, value]) => {
+  return Object.entries(args).map(([key, value], index) => {
     const raw = value === null || value === undefined ? "—" : String(value);
     const long = raw.length > 60;
+    const isExpanded = !!expandedArgs[index];
     return {
       key,
       raw,
       long,
       icon: argIcon(key),
-      display: long ? raw.slice(0, 60) : raw,
+      display: long && !isExpanded ? raw.slice(0, 60) : raw,
     };
   });
 });
diff --git a/dashboard/src/components/chat/message_list_comps/ToolResultView.vue b/dashboard/src/components/chat/message_list_comps/ToolResultView.vue
index ff66bc49b7..903c4adf37 100644
--- a/dashboard/src/components/chat/message_list_comps/ToolResultView.vue
+++ b/dashboard/src/components/chat/message_list_comps/ToolResultView.vue
@@ -53,15 +53,11 @@
 
     <!-- ── fallback ────────────────────────────────────────────── -->
     <template v-else>
-      <div class="result-status" :class="resultOk ? 'success' : 'error'">
+      <div v-if="resultText.length < 120 && !resultText.includes('\n')" class="result-status" :class="resultOk ? 'success' : 'error'">
         <v-icon size="16">{{ resultOk ? 'mdi-check-circle' : 'mdi-alert-circle' }}</v-icon>
         <span>{{ resultText }}</span>
       </div>
-    </template>
-
-    <!-- ── fallback ────────────────────────────────────────────── -->
-    <template v-else>
-      <pre class="result-raw">{{ formattedResult }}</pre>
+      <pre v-else class="result-raw">{{ formattedResult }}</pre>
     </template>
   </div>
 </template>

From 7b669af819b624a4a7e6ef7f45fd36f15cadaa0b Mon Sep 17 00:00:00 2001
From: "DESKTOP-02FN3OU\\80423" <804235820@qq.com>
Date: Fri, 29 May 2026 21:19:20 +0800
Subject: [PATCH 05/15] =?UTF-8?q?fix(file=5Fedit=5Ftool):=20=E5=A4=9A?=
 =?UTF-8?q?=E5=A4=84=E4=BF=AE=E5=A4=8D?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 .../core/tools/computer_tools/edit_engine.py  |  2 +-
 dashboard/src/components/chat/ThreadPanel.vue | 83 +------------------
 .../chat/message_list_comps/DiffPreview.vue   |  4 +-
 3 files changed, 5 insertions(+), 84 deletions(-)

diff --git a/astrbot/core/tools/computer_tools/edit_engine.py b/astrbot/core/tools/computer_tools/edit_engine.py
index 8710b32378..8b013376b6 100644
--- a/astrbot/core/tools/computer_tools/edit_engine.py
+++ b/astrbot/core/tools/computer_tools/edit_engine.py
@@ -551,7 +551,7 @@ async def edit_file(
     - Preserves BOM if present
     - Returns unified diff of changes
     """
-    lock = await get_file_lock(path)
+    lock = get_file_lock(path)
     async with lock:
         try:
             # Read file
diff --git a/dashboard/src/components/chat/ThreadPanel.vue b/dashboard/src/components/chat/ThreadPanel.vue
index e2bfc41320..bab1fe8d18 100644
--- a/dashboard/src/components/chat/ThreadPanel.vue
+++ b/dashboard/src/components/chat/ThreadPanel.vue
@@ -1,17 +1,6 @@
 <template>
   <transition name="slide-left">
-    <aside
-      v-if="modelValue && thread"
-      ref="panelRef"
-      class="thread-panel"
-      :style="{ width: panelWidth + 'px' }"
-    >
-      <!-- Drag handle -->
-      <div
-        class="thread-panel-resizer"
-        @mousedown="startResize"
-      />
-
+    <aside v-if="modelValue && thread" class="thread-panel">
       <div class="thread-panel-header">
         <div class="thread-panel-title">{{ tm("thread.title") }}</div>
         <div class="thread-panel-actions">
@@ -66,7 +55,7 @@
 </template>
 
 <script setup lang="ts">
-import { nextTick, onBeforeUnmount, ref, watch } from "vue";
+import { nextTick, ref, watch } from "vue";
 import axios from "axios";
 import {
   appendPlain,
@@ -119,49 +108,6 @@ function close() {
   emit("update:modelValue", false);
 }
 
-// ── Drag resize ────────────────────────────────────────────────────
-
-const MIN_WIDTH = 280;
-const MAX_WIDTH = 800;
-const DEFAULT_WIDTH = 380;
-
-const panelWidth = ref(DEFAULT_WIDTH);
-const panelRef = ref<HTMLElement | null>(null);
-let isResizing = false;
-
-function startResize(e: MouseEvent) {
-  e.preventDefault();
-  isResizing = true;
-  document.body.style.cursor = "ew-resize";
-  document.body.style.userSelect = "none";
-  document.addEventListener("mousemove", onMouseMove);
-  document.addEventListener("mouseup", onMouseUp);
-}
-
-function onMouseMove(e: MouseEvent) {
-  if (!isResizing || !panelRef.value) return;
-  const rect = panelRef.value.parentElement?.getBoundingClientRect();
-  if (!rect) return;
-  const newWidth = rect.right - e.clientX;
-  panelWidth.value = Math.min(MAX_WIDTH, Math.max(MIN_WIDTH, newWidth));
-}
-
-function onMouseUp() {
-  if (!isResizing) return;
-  isResizing = false;
-  document.body.style.cursor = "";
-  document.body.style.userSelect = "";
-  document.removeEventListener("mousemove", onMouseMove);
-  document.removeEventListener("mouseup", onMouseUp);
-}
-
-onBeforeUnmount(() => {
-  if (isResizing) {
-    document.removeEventListener("mousemove", onMouseMove);
-    document.removeEventListener("mouseup", onMouseUp);
-  }
-});
-
 async function loadThread(threadId: string) {
   try {
     const response = await axios.get("/api/chat/thread/get", {
@@ -391,29 +337,8 @@ function scrollToBottom() {
   display: flex;
   flex-direction: column;
   flex-shrink: 0;
-  position: relative;
 }
 
-/* ── Drag handle ──────────────────────────────────────────────── */
-
-.thread-panel-resizer {
-  position: absolute;
-  left: 0;
-  top: 0;
-  bottom: 0;
-  width: 6px;
-  cursor: ew-resize;
-  z-index: 10;
-  transition: background 0.15s ease;
-}
-
-.thread-panel-resizer:hover,
-.thread-panel-resizer:active {
-  background: rgba(var(--v-theme-primary), 0.2);
-}
-
-/* ── Transition ───────────────────────────────────────────────── */
-
 .slide-left-enter-active,
 .slide-left-leave-active {
   transition: all 0.2s ease;
@@ -521,10 +446,6 @@ function scrollToBottom() {
     border-left: 0;
   }
 
-  .thread-panel-resizer {
-    display: none;
-  }
-
   .thread-panel-header {
     min-height: 52px;
     padding: calc(10px + env(safe-area-inset-top)) 12px 8px;
diff --git a/dashboard/src/components/chat/message_list_comps/DiffPreview.vue b/dashboard/src/components/chat/message_list_comps/DiffPreview.vue
index e16a7025a0..edd0ed4905 100644
--- a/dashboard/src/components/chat/message_list_comps/DiffPreview.vue
+++ b/dashboard/src/components/chat/message_list_comps/DiffPreview.vue
@@ -44,7 +44,7 @@
         class="diff-hunk"
       >
         <div class="hunk-header">
-          @@ {{ hunk.header }} @@
+          {{ hunk.header }}
         </div>
         <div
           v-for="(line, li) in hunk.lines"
@@ -201,7 +201,7 @@ function parseUnifiedDiff(text: string, maxLines: number): DiffHunk[] {
       newNo = parseInt(hunkMatch[3], 10);
 
       currentHunk = {
-        header: rawLine.replace(/^@@\s+/, "").replace(/\s+@@.*$/, "").trim(),
+        header: rawLine,
         lines: [],
       };
       continue;

From 329f17ec2d40938266fe7862072cb59a914e9c87 Mon Sep 17 00:00:00 2001
From: "DESKTOP-02FN3OU\\80423" <804235820@qq.com>
Date: Fri, 29 May 2026 22:42:59 +0800
Subject: [PATCH 06/15] =?UTF-8?q?fix(file=5Fedit=5Ftool):=20=E6=B7=BB?=
 =?UTF-8?q?=E5=8A=A0=E5=AF=B9[SYSTEM=20NOTICE]=E7=9A=84=E5=8D=95=E7=8B=AC?=
 =?UTF-8?q?=E6=98=BE=E7=A4=BA?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 .../chat/message_list_comps/DiffPreview.vue   |  40 +++-
 .../chat/message_list_comps/ToolCallCard.vue  |  33 ++-
 .../message_list_comps/ToolResultView.vue     | 207 ++++++++++++++++--
 3 files changed, 246 insertions(+), 34 deletions(-)

diff --git a/dashboard/src/components/chat/message_list_comps/DiffPreview.vue b/dashboard/src/components/chat/message_list_comps/DiffPreview.vue
index edd0ed4905..e3465c3e0e 100644
--- a/dashboard/src/components/chat/message_list_comps/DiffPreview.vue
+++ b/dashboard/src/components/chat/message_list_comps/DiffPreview.vue
@@ -5,7 +5,7 @@
   >
     <!-- Summary header — always visible, clickable to toggle -->
     <button
-      v-if="summary || filePath || statsText"
+      v-if="summary || filePath || statsAdds || statsDels"
       type="button"
       class="diff-header"
       @click="toggleCollapsed"
@@ -15,7 +15,10 @@
         <span v-if="filePath" class="diff-file-path">{{ filePath }}</span>
       </div>
       <div class="diff-header-right">
-        <span v-if="statsText" class="diff-stats">{{ statsText }}</span>
+        <template v-if="statsAdds || statsDels">
+          <span v-if="statsAdds" class="diff-stats diff-stats-add">+{{ statsAdds }}</span>
+          <span v-if="statsDels" class="diff-stats diff-stats-del">−{{ statsDels }}</span>
+        </template>
         <v-icon
           v-if="collapsible"
           size="18"
@@ -151,17 +154,24 @@ const collapsedOverflow = computed(() => {
 
 // ── Stats ──────────────────────────────────────────────────────────
 
-const statsText = computed(() => {
+const statsAdds = computed(() => {
   let adds = 0;
-  let dels = 0;
   for (const hunk of parsedHunks.value) {
     for (const line of hunk.lines) {
       if (line.type === "add") adds++;
+    }
+  }
+  return adds || null;
+});
+
+const statsDels = computed(() => {
+  let dels = 0;
+  for (const hunk of parsedHunks.value) {
+    for (const line of hunk.lines) {
       if (line.type === "del") dels++;
     }
   }
-  if (adds === 0 && dels === 0) return "";
-  return `+${adds} −${dels}`;
+  return dels || null;
 });
 
 // ── Helpers ────────────────────────────────────────────────────────
@@ -349,10 +359,26 @@ function parseUnifiedDiff(text: string, maxLines: number): DiffHunk[] {
   font-size: 11px;
   font-weight: 600;
   font-family: ui-monospace, SFMono-Regular, Menlo, Monaco, Consolas, monospace;
-  color: rgba(var(--v-theme-on-surface), 0.55);
   white-space: nowrap;
 }
 
+.diff-stats-add {
+  color: #2da44e;
+  margin-right: 4px;
+}
+
+.diff-preview.is-dark .diff-stats-add {
+  color: #57ab5a;
+}
+
+.diff-stats-del {
+  color: #cf222e;
+}
+
+.diff-preview.is-dark .diff-stats-del {
+  color: #f47067;
+}
+
 .diff-chevron {
   color: rgba(var(--v-theme-on-surface), 0.45);
   transition: transform 0.2s ease;
diff --git a/dashboard/src/components/chat/message_list_comps/ToolCallCard.vue b/dashboard/src/components/chat/message_list_comps/ToolCallCard.vue
index eeb8a45ba3..78eccd8b99 100644
--- a/dashboard/src/components/chat/message_list_comps/ToolCallCard.vue
+++ b/dashboard/src/components/chat/message_list_comps/ToolCallCard.vue
@@ -67,6 +67,7 @@
           :max-lines="25"
           :collapsible="false"
         />
+        <div v-if="editToolNotice" class="edit-tool-notice">{{ editToolNotice }}</div>
       </div>
 
       <div
@@ -200,23 +201,36 @@ const isEditTool = computed(
   () => props.toolCall.name === "astrbot_file_edit_tool",
 );
 
+// Strip [SYSTEM NOTICE] suffix from the raw result for all edit tool computed properties.
+const editToolCleanResult = computed(() => {
+  const raw = props.toolCall.result ?? "";
+  const idx = raw.search(/\[SYSTEM NOTICE\]/i);
+  return idx < 0 ? raw : raw.slice(0, idx).trim();
+});
+
+const editToolNotice = computed(() => {
+  const raw = props.toolCall.result ?? "";
+  const idx = raw.search(/\[SYSTEM NOTICE\]/i);
+  return idx < 0 ? null : raw.slice(idx).trim();
+});
+
 const editToolDiff = computed(() => {
   if (!isEditTool.value) return "";
-  const raw = props.toolCall.result ?? "";
+  const raw = editToolCleanResult.value;
   const match = raw.match(/```diff\s*\n?([\s\S]*?)```/);
   return match ? match[1] : raw;
 });
 
 const editToolFilePath = computed(() => {
   if (!isEditTool.value) return "";
-  const raw = props.toolCall.result ?? "";
+  const raw = editToolCleanResult.value;
   const match = raw.match(/^Edited\s+(.+?)\./m);
   return match ? match[1] : "";
 });
 
 const editToolSummary = computed(() => {
   if (!isEditTool.value) return "";
-  const raw = props.toolCall.result ?? "";
+  const raw = editToolCleanResult.value;
   const lines = raw.split("\n");
   const statusParts = [];
   for (const line of lines) {
@@ -336,6 +350,19 @@ onUnmounted(() => {
   animation: fadeIn 0.2s ease-in-out;
 }
 
+.edit-tool-notice {
+  margin-top: 6px;
+  padding: 4px 8px;
+  border-radius: 4px;
+  background: rgba(var(--v-theme-on-surface), 0.03);
+  font-family: ui-monospace, SFMono-Regular, Menlo, Monaco, Consolas, monospace;
+  font-size: 11px;
+  line-height: 1.55;
+  color: rgba(var(--v-theme-on-surface), 0.55);
+  white-space: pre-wrap;
+  word-break: break-word;
+}
+
 .tool-call-detail-row {
   display: flex;
   flex-direction: column;
diff --git a/dashboard/src/components/chat/message_list_comps/ToolResultView.vue b/dashboard/src/components/chat/message_list_comps/ToolResultView.vue
index 903c4adf37..a94224a6c6 100644
--- a/dashboard/src/components/chat/message_list_comps/ToolResultView.vue
+++ b/dashboard/src/components/chat/message_list_comps/ToolResultView.vue
@@ -45,20 +45,30 @@
 
     <!-- ── execute_shell ───────────────────────────────────────── -->
     <template v-else-if="toolName === 'astrbot_execute_shell'">
-      <pre class="result-terminal" v-text="resultText"></pre>
-      <div v-if="shellExitCode != null" class="result-exit-code" :class="shellExitCode === 0 ? 'success' : 'error'">
-        Exit code: {{ shellExitCode }}
+      <div class="shell-result">
+        <div class="shell-row">
+          <span class="shell-label">Stdout</span>
+          <pre class="shell-value" v-text="shellStdout"></pre>
+        </div>
+        <div v-if="shellStderr" class="shell-row shell-stderr">
+          <span class="shell-label">Stderr</span>
+          <pre class="shell-value shell-stderr-text" v-text="shellStderr"></pre>
+        </div>
+        <div class="shell-row">
+          <span class="shell-label">Exit code</span>
+          <span class="shell-exit-code" :class="shellExitCodeVal === 0 ? 'success' : 'error'">{{ shellExitCodeVal }}</span>
+        </div>
       </div>
+      <div v-if="shellExtra" class="shell-extra-text">{{ shellExtra }}</div>
     </template>
 
     <!-- ── fallback ────────────────────────────────────────────── -->
     <template v-else>
-      <div v-if="resultText.length < 120 && !resultText.includes('\n')" class="result-status" :class="resultOk ? 'success' : 'error'">
-        <v-icon size="16">{{ resultOk ? 'mdi-check-circle' : 'mdi-alert-circle' }}</v-icon>
-        <span>{{ resultText }}</span>
-      </div>
-      <pre v-else class="result-raw">{{ formattedResult }}</pre>
+      <pre class="result-raw">{{ formattedResult }}</pre>
     </template>
+
+    <!-- ── shared [SYSTEM NOTICE] suffix (exclude shell which handles it separately) ── -->
+    <div v-if="resultSuffix && toolName !== 'astrbot_execute_shell'" class="result-suffix">{{ resultSuffix }}</div>
   </div>
 </template>
 
@@ -131,7 +141,23 @@ const shikiReady = ref(false);
 
 // ── helpers ──────────────────────────────────────────────────────
 
-const resultText = computed(() => (props.result ?? "").trim());
+const rawResult = computed(() => (props.result ?? "").trim());
+
+// Strip [SYSTEM NOTICE] suffix for all non-shell templates.
+// Shell uses rawResult directly via brace-tracking in shellParsed.
+const resultText = computed(() => {
+  const text = rawResult.value;
+  const idx = text.search(/\[SYSTEM NOTICE\]/i);
+  if (idx < 0) return text;
+  return text.slice(0, idx).trim();
+});
+
+const resultSuffix = computed(() => {
+  const text = rawResult.value;
+  const idx = text.search(/\[SYSTEM NOTICE\]/i);
+  if (idx < 0) return null;
+  return text.slice(idx).trim();
+});
 
 const resultOk = computed(() => {
   const t = resultText.value.toLowerCase();
@@ -139,11 +165,11 @@ const resultOk = computed(() => {
 });
 
 const formattedResult = computed(() => {
-  if (!props.result) return "";
+  if (!resultText.value) return "";
   try {
-    return JSON.stringify(JSON.parse(props.result), null, 2);
+    return JSON.stringify(JSON.parse(resultText.value), null, 2);
   } catch {
-    return props.result;
+    return resultText.value;
   }
 });
 
@@ -269,9 +295,62 @@ const grepTruncated = computed(() => {
 
 // ── execute_shell ──────────────────────────────────────────────
 
-const shellExitCode = computed(() => {
-  return null; // Shell output doesn't include exit code in result string
-  // We could try to parse it but it's not reliable
+const shellParsed = computed(() => {
+  // Extract JSON from text that may have trailing non-JSON content (e.g. [SYSTEM NOTICE]).
+  const text = rawResult.value;
+  const start = text.indexOf("{");
+  if (start < 0) {
+    return { json: null, extra: text };
+  }
+  // Track brace depth to find the matching closing brace
+  let depth = 0;
+  let end = -1;
+  for (let i = start; i < text.length; i++) {
+    if (text[i] === "{") depth++;
+    else if (text[i] === "}") {
+      depth--;
+      if (depth === 0) {
+        end = i + 1;
+        break;
+      }
+    }
+  }
+  if (end < 0) {
+    return { json: null, extra: text };
+  }
+  const jsonStr = text.slice(start, end);
+  const extraStr = text.slice(end).trim();
+  try {
+    const parsed = JSON.parse(jsonStr);
+    if (parsed && typeof parsed === "object") {
+      return { json: parsed, extra: extraStr || null };
+    }
+  } catch {
+    // not valid JSON
+  }
+  return { json: null, extra: text };
+});
+
+const shellStdout = computed(() => {
+  if (shellParsed.value?.json && "stdout" in shellParsed.value.json) {
+    return shellParsed.value.json.stdout;
+  }
+  return resultText.value;
+});
+
+const shellStderr = computed(() => {
+  return shellParsed.value?.json?.stderr || "";
+});
+
+const shellExitCodeVal = computed(() => {
+  if (shellParsed.value?.json && "exit_code" in shellParsed.value.json) {
+    return shellParsed.value.json.exit_code;
+  }
+  return null;
+});
+
+const shellExtra = computed(() => {
+  return shellParsed.value?.extra || null;
 });
 
 </script>
@@ -358,23 +437,103 @@ const shellExitCode = computed(() => {
   line-height: inherit;
 }
 
-/* ── Terminal block ──────────────────────────────────────────── */
+/* ── Shell result ─────────────────────────────────────────── */
 
-.result-terminal {
-  margin: 0;
-  padding: 8px 10px;
+.shell-result {
+  border: 1px solid rgba(var(--v-theme-on-surface), 0.1);
   border-radius: 4px;
-  background: rgba(0, 0, 0, 0.85);
-  color: #e0e0e0;
-  font-family: ui-monospace, SFMono-Regular, Menlo, Monaco, Consolas, monospace;
-  font-size: 11.5px;
+  overflow: hidden;
+}
+
+.shell-row {
+  display: flex;
+  align-items: flex-start;
+  padding: 3px 8px;
+  font-size: 11px;
   line-height: 1.55;
+}
+
+.shell-row + .shell-row {
+  border-top: 1px solid rgba(var(--v-theme-on-surface), 0.06);
+}
+
+.shell-label {
+  flex-shrink: 0;
+  width: 64px;
+  font-family: ui-monospace, SFMono-Regular, Menlo, Monaco, Consolas, monospace;
+  font-size: 11px;
+  font-weight: 600;
+  color: rgba(var(--v-theme-on-surface), 0.5);
+  padding-right: 8px;
+}
+
+.shell-value {
+  flex: 1;
+  font-family: ui-monospace, SFMono-Regular, Menlo, Monaco, Consolas, monospace;
+  font-size: 11px;
+  color: rgba(var(--v-theme-on-surface), 0.8);
   white-space: pre-wrap;
   word-break: break-all;
-  max-height: 300px;
+  margin: 0;
+  padding: 0;
+  max-height: 200px;
   overflow-y: auto;
 }
 
+.shell-stderr {
+  background: rgba(207, 34, 46, 0.04);
+}
+
+.shell-stderr-text {
+  color: #cf222e;
+}
+
+.shell-exit-code {
+  font-family: ui-monospace, SFMono-Regular, Menlo, Monaco, Consolas, monospace;
+  font-size: 11px;
+  font-weight: 600;
+}
+
+.shell-exit-code.success {
+  color: #2da44e;
+}
+
+.shell-exit-code.error {
+  color: #cf222e;
+}
+
+/* ── Terminal block (deprecated, shell now uses .shell-result) ── */
+.result-terminal-deprecated {}
+
+/* ── Shell extra text (e.g. [SYSTEM NOTICE]) ────────────── */
+
+.shell-extra-text {
+  margin-top: 6px;
+  padding: 4px 8px;
+  border-radius: 4px;
+  background: rgba(var(--v-theme-on-surface), 0.03);
+  font-family: ui-monospace, SFMono-Regular, Menlo, Monaco, Consolas, monospace;
+  font-size: 11px;
+  line-height: 1.55;
+  color: rgba(var(--v-theme-on-surface), 0.55);
+  white-space: pre-wrap;
+  word-break: break-word;
+}
+
+/* Shared [SYSTEM NOTICE] suffix for non-shell tools */
+.result-suffix {
+  margin-top: 6px;
+  padding: 4px 8px;
+  border-radius: 4px;
+  background: rgba(var(--v-theme-on-surface), 0.03);
+  font-family: ui-monospace, SFMono-Regular, Menlo, Monaco, Consolas, monospace;
+  font-size: 11px;
+  line-height: 1.55;
+  color: rgba(var(--v-theme-on-surface), 0.55);
+  white-space: pre-wrap;
+  word-break: break-word;
+}
+
 /* ── Status badge ────────────────────────────────────────────── */
 
 .result-status {

From 6ea7cb838187b27209ee7f04bb9ba7c2bbb7d69c Mon Sep 17 00:00:00 2001
From: "DESKTOP-02FN3OU\\80423" <804235820@qq.com>
Date: Sat, 30 May 2026 09:42:42 +0800
Subject: [PATCH 07/15] no message

---
 .../chat/message_list_comps/ToolCallCard.vue     | 16 +---------------
 1 file changed, 1 insertion(+), 15 deletions(-)

diff --git a/dashboard/src/components/chat/message_list_comps/ToolCallCard.vue b/dashboard/src/components/chat/message_list_comps/ToolCallCard.vue
index 78eccd8b99..1caf60660d 100644
--- a/dashboard/src/components/chat/message_list_comps/ToolCallCard.vue
+++ b/dashboard/src/components/chat/message_list_comps/ToolCallCard.vue
@@ -178,21 +178,7 @@ function argIcon(key) {
 }
 
 function toggleArgExpand(index) {
-  const entry = argEntries.value[index];
-  if (!entry || !entry.long) return;
-  if (expandedArgs.has(index)) {
-    expandedArgs.delete(index);
-  } else {
-    expandedArgs.add(index);
-  }
-  // Update display
-  const displayed = displayedArgEntries.value;
-  for (let i = 0; i < displayed.length; i++) {
-    const e = argEntries.value.find((x) => x.key === displayed[i].key);
-    if (e && e.long) {
-      displayed[i].display = expandedArgs.has(i) ? e.raw : e.raw.slice(0, 60);
-    }
-  }
+  expandedArgs[index] = !expandedArgs[index];
 }
 
 // ── file_edit_tool diff rendering ─────────────────────────────

From e2d348a399c862e6ad6e734531e109ea1181bf4b Mon Sep 17 00:00:00 2001
From: "DESKTOP-02FN3OU\\80423" <804235820@qq.com>
Date: Sat, 30 May 2026 14:01:42 +0800
Subject: [PATCH 08/15] =?UTF-8?q?fix(chatui):=20=E4=BF=AE=E5=A4=8D?=
 =?UTF-8?q?=E6=96=87=E4=BB=B6=E8=AF=BB=E5=8F=96=E6=97=B6=E6=98=BE=E7=A4=BA?=
 =?UTF-8?q?[SYSTEM=20NOTICE]=E7=9A=84=E8=BE=B9=E7=95=8C=E6=83=85=E5=86=B5?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 .../chat/message_list_comps/ToolCallCard.vue  |   7 +-
 .../message_list_comps/ToolResultView.vue     |  22 ++++
 dashboard/src/utils/systemNotice.ts           | 118 ++++++++++++++++++
 3 files changed, 145 insertions(+), 2 deletions(-)
 create mode 100644 dashboard/src/utils/systemNotice.ts

diff --git a/dashboard/src/components/chat/message_list_comps/ToolCallCard.vue b/dashboard/src/components/chat/message_list_comps/ToolCallCard.vue
index 1caf60660d..532481f864 100644
--- a/dashboard/src/components/chat/message_list_comps/ToolCallCard.vue
+++ b/dashboard/src/components/chat/message_list_comps/ToolCallCard.vue
@@ -88,6 +88,7 @@
 <script setup>
 import { computed, onMounted, onUnmounted, reactive, ref } from "vue";
 import { useModuleI18n } from "@/i18n/composables";
+import { findRealSystemNoticeIndex } from "@/utils/systemNotice";
 import DiffPreview from "./DiffPreview.vue";
 import ToolResultView from "./ToolResultView.vue";
 
@@ -188,15 +189,17 @@ const isEditTool = computed(
 );
 
 // Strip [SYSTEM NOTICE] suffix from the raw result for all edit tool computed properties.
+// Uses the shared findRealSystemNoticeIndex() to correctly distinguish genuine system
+// notices from "[SYSTEM NOTICE]" text that may appear inside the edited file's diff content.
 const editToolCleanResult = computed(() => {
   const raw = props.toolCall.result ?? "";
-  const idx = raw.search(/\[SYSTEM NOTICE\]/i);
+  const idx = findRealSystemNoticeIndex(raw);
   return idx < 0 ? raw : raw.slice(0, idx).trim();
 });
 
 const editToolNotice = computed(() => {
   const raw = props.toolCall.result ?? "";
-  const idx = raw.search(/\[SYSTEM NOTICE\]/i);
+  const idx = findRealSystemNoticeIndex(raw);
   return idx < 0 ? null : raw.slice(idx).trim();
 });
 
diff --git a/dashboard/src/components/chat/message_list_comps/ToolResultView.vue b/dashboard/src/components/chat/message_list_comps/ToolResultView.vue
index a94224a6c6..28063951e0 100644
--- a/dashboard/src/components/chat/message_list_comps/ToolResultView.vue
+++ b/dashboard/src/components/chat/message_list_comps/ToolResultView.vue
@@ -62,6 +62,11 @@
       <div v-if="shellExtra" class="shell-extra-text">{{ shellExtra }}</div>
     </template>
 
+    <!-- ── execute_python ──────────────────────────────────────── -->
+    <template v-else-if="toolName === 'astrbot_execute_python' || toolName === 'astrbot_execute_ipython'">
+      <pre class="result-terminal" v-text="resultText"></pre>
+    </template>
+
     <!-- ── fallback ────────────────────────────────────────────── -->
     <template v-else>
       <pre class="result-raw">{{ formattedResult }}</pre>
@@ -79,6 +84,7 @@ import {
   escapeHtml,
   renderShikiCode,
 } from "@/utils/shiki";
+import { findRealSystemNoticeIndex } from "@/utils/systemNotice";
 
 const props = defineProps<{
   toolName: string;
@@ -145,8 +151,17 @@ const rawResult = computed(() => (props.result ?? "").trim());
 
 // Strip [SYSTEM NOTICE] suffix for all non-shell templates.
 // Shell uses rawResult directly via brace-tracking in shellParsed.
+// file_read_tool uses smart detection to avoid false positives from
+// file content that coincidentally contains "[SYSTEM NOTICE]".
 const resultText = computed(() => {
   const text = rawResult.value;
+
+  if (props.toolName === "astrbot_file_read_tool") {
+    const idx = findRealSystemNoticeIndex(text);
+    if (idx < 0) return text;
+    return text.slice(0, idx).trim();
+  }
+
   const idx = text.search(/\[SYSTEM NOTICE\]/i);
   if (idx < 0) return text;
   return text.slice(0, idx).trim();
@@ -154,6 +169,13 @@ const resultText = computed(() => {
 
 const resultSuffix = computed(() => {
   const text = rawResult.value;
+
+  if (props.toolName === "astrbot_file_read_tool") {
+    const idx = findRealSystemNoticeIndex(text);
+    if (idx < 0) return null;
+    return text.slice(idx).trim();
+  }
+
   const idx = text.search(/\[SYSTEM NOTICE\]/i);
   if (idx < 0) return null;
   return text.slice(idx).trim();
diff --git a/dashboard/src/utils/systemNotice.ts b/dashboard/src/utils/systemNotice.ts
new file mode 100644
index 0000000000..d682767342
--- /dev/null
+++ b/dashboard/src/utils/systemNotice.ts
@@ -0,0 +1,118 @@
+/**
+ * Utilities for detecting and extracting the genuine AstrBot
+ * [SYSTEM NOTICE] suffix from tool call results.
+ *
+ * @author AstrBot Agent Harness
+ * @since 2026-05-30
+ */
+
+/**
+ * Known message prefixes produced by the AstrBot system (not file content).
+ * Used by findRealSystemNoticeIndex() to distinguish genuine system notices
+ * from [SYSTEM NOTICE] text that happens to appear inside file content
+ * (e.g. when reading AstrBot log files or editing files that contain the
+ * literal string "[SYSTEM NOTICE]").
+ */
+export const SYSTEM_NOTICE_PREFIXES = [
+  "By the way,",
+  "Important:",
+  "This is a",
+];
+
+const MARKER = "[SYSTEM NOTICE]";
+
+/**
+ * Find the index of the genuine AstrBot system notice in the text.
+ *
+ * For tools whose result may contain file content (e.g. astrbot_file_read_tool,
+ * astrbot_file_edit_tool), the raw result can include literal "[SYSTEM NOTICE]"
+ * strings that are part of the file being read/edited, not a system message.
+ *
+ * Key invariant: the REAL [SYSTEM NOTICE] is ALWAYS the very last thing in
+ * the tool result — there is NO content after it.  Any "[SYSTEM NOTICE]"
+ * that has non-empty lines following it is necessarily part of the file
+ * content, not a system message.
+ *
+ * Strategy:
+ *   1. Find all occurrences of [SYSTEM NOTICE] in the text.
+ *   2. Scan from the LAST occurrence backward (most likely to be real).
+ *   3. For each candidate:
+ *      a. Check it matches a known system prefix.
+ *      b. Check there are NO non-empty lines after the notice line.
+ *         If there are, this [SYSTEM NOTICE] is embedded in file
+ *         content — skip it and try the previous occurrence.
+ *   4. If no candidate passes both checks, there is no real notice.
+ *
+ * @returns The character index of the real [SYSTEM NOTICE], or -1 if none.
+ */
+export function findRealSystemNoticeIndex(text: string): number {
+  // Collect all occurrences so we can scan from last to first.
+  const occurrences: number[] = [];
+  let searchFrom = 0;
+  while (searchFrom < text.length) {
+    const idx = text.indexOf(MARKER, searchFrom);
+    if (idx < 0) break;
+    occurrences.push(idx);
+    searchFrom = idx + MARKER.length;
+  }
+  if (occurrences.length === 0) return -1;
+
+  // Scan from the last occurrence backward.
+  for (let i = occurrences.length - 1; i >= 0; i--) {
+    const idx = occurrences[i];
+    const suffix = text.slice(idx).trim();
+
+    // Must match a known system prefix pattern.
+    if (
+      !SYSTEM_NOTICE_PREFIXES.some((p) =>
+        suffix.startsWith(`${MARKER} ${p}`),
+      )
+    ) {
+      continue;
+    }
+
+    // The real system notice is always at the very end of the tool
+    // result with NO content after it.  Check if there are any
+    // non-empty lines following the notice line — if so, this is
+    // file content that happens to contain [SYSTEM NOTICE], not a
+    // genuine system message.
+    const afterText = text.slice(idx);
+    const lines = afterText.split("\n");
+    let hasContentAfter = false;
+    for (let j = 1; j < lines.length; j++) {
+      if (lines[j].trim() !== "") {
+        hasContentAfter = true;
+        break;
+      }
+    }
+    if (hasContentAfter) {
+      // This [SYSTEM NOTICE] has file content after it — it's fake.
+      // Continue checking earlier occurrences.
+      continue;
+    }
+
+    // No content after this notice — it's the real one.
+    return idx;
+  }
+
+  return -1;
+}
+
+/**
+ * Split tool result text into the main content and the optional
+ * [SYSTEM NOTICE] suffix.
+ *
+ * @param text  The raw tool result text.
+ * @returns An object with `content` (the main result) and `notice`
+ *          (the system notice string, or null if none).
+ */
+export function splitSystemNotice(
+  text: string,
+): { content: string; notice: string | null } {
+  const idx = findRealSystemNoticeIndex(text);
+  if (idx < 0) return { content: text, notice: null };
+  return {
+    content: text.slice(0, idx).trim(),
+    notice: text.slice(idx).trim(),
+  };
+}

From daa52149a2808e3946a30ed77b3b477482f13f1f Mon Sep 17 00:00:00 2001
From: "DESKTOP-02FN3OU\\80423" <804235820@qq.com>
Date: Sat, 30 May 2026 15:35:06 +0800
Subject: [PATCH 09/15] =?UTF-8?q?fix(chatui):=20=E4=BF=AE=E5=A4=8D?=
 =?UTF-8?q?=E6=9B=B4=E5=A4=9A=E6=9C=89=E5=85=B3[SYSTEM=20NOTICE]=E6=98=BE?=
 =?UTF-8?q?=E7=A4=BA=E7=9A=84=E8=BE=B9=E7=95=8C=E6=9D=A1=E4=BB=B6?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 .../src/components/chat/ReasoningSidebar.vue  |   2 +-
 .../message_list_comps/IPythonToolBlock.vue   | 142 ++++++-----
 .../chat/message_list_comps/ToolCallCard.vue  |   8 +-
 .../message_list_comps/ToolResultView.vue     |  30 +--
 dashboard/src/utils/systemNotice.ts           | 225 ++++++++++++------
 5 files changed, 252 insertions(+), 155 deletions(-)

diff --git a/dashboard/src/components/chat/ReasoningSidebar.vue b/dashboard/src/components/chat/ReasoningSidebar.vue
index d15dc75bed..9fdb3c47d1 100644
--- a/dashboard/src/components/chat/ReasoningSidebar.vue
+++ b/dashboard/src/components/chat/ReasoningSidebar.vue
@@ -58,7 +58,7 @@ function close() {
 // ── Drag resize ────────────────────────────────────────────────────
 
 const MIN_WIDTH = 280;
-const MAX_WIDTH = 800;
+const MAX_WIDTH = 1200;
 const DEFAULT_WIDTH = 380;
 
 const sidebarWidth = ref(DEFAULT_WIDTH);
diff --git a/dashboard/src/components/chat/message_list_comps/IPythonToolBlock.vue b/dashboard/src/components/chat/message_list_comps/IPythonToolBlock.vue
index 716d8d1b0b..edb72a4f9d 100644
--- a/dashboard/src/components/chat/message_list_comps/IPythonToolBlock.vue
+++ b/dashboard/src/components/chat/message_list_comps/IPythonToolBlock.vue
@@ -1,21 +1,23 @@
 <template>
     <div class="ipython-tool-block" :class="{ compact: !showHeader }">
         <div v-if="displayExpanded" class="py-3 animate-fade-in">
-            <!-- Code Section -->
+            <!-- Code Section (matches ToolResultView result-code style) -->
             <div class="code-section">
-                <div v-if="shikiReady && code" class="code-highlighted"
-                    v-html="highlightedCode"></div>
-                <pre v-else class="code-fallback"
-                    :class="{ 'dark-theme': isDark }">{{ code || 'No code available' }}</pre>
+                <div
+                    v-if="shikiReady && code"
+                    class="code-highlighted code-result-shiki"
+                    v-html="highlightedCode"
+                ></div>
+                <pre v-else class="code-fallback">{{ code || 'No code available' }}</pre>
             </div>
 
-            <!-- Result Section -->
+            <!-- Result Section (matches ToolResultView result-code style) -->
             <div v-if="result" class="result-section">
                 <div class="result-label">
                     {{ tm('ipython.output') }}:
                 </div>
-                <pre class="result-content"
-                    :class="{ 'dark-theme': isDark }">{{ formattedResult }}</pre>
+                <pre class="result-content">{{ formattedResult }}</pre>
+                <div v-if="resultNotice" class="result-suffix">{{ resultNotice }}</div>
             </div>
         </div>
     </div>
@@ -25,6 +27,7 @@
 import { ref, computed, onMounted } from 'vue';
 import { useModuleI18n } from '@/i18n/composables';
 import { ensureShikiLanguages, escapeHtml, renderShikiCode } from '@/utils/shiki';
+import { findSystemNoticeIndex } from '@/utils/systemNotice';
 
 const props = defineProps({
     toolCall: {
@@ -69,14 +72,26 @@ const result = computed(() => props.toolCall.result);
 
 const formattedResult = computed(() => {
     if (!result.value) return '';
+    let text = result.value;
+    const idx = findSystemNoticeIndex(text);
+    if (idx >= 0) {
+        text = text.slice(0, idx).trim();
+    }
     try {
-        const parsed = JSON.parse(result.value);
+        const parsed = JSON.parse(text);
         return JSON.stringify(parsed, null, 2);
     } catch {
-        return result.value;
+        return text;
     }
 });
 
+const resultNotice = computed(() => {
+    if (!result.value) return null;
+    const idx = findSystemNoticeIndex(result.value);
+    if (idx < 0) return null;
+    return result.value.slice(idx).trim();
+});
+
 const highlightedCode = computed(() => {
     if (!shikiReady.value || !shikiHighlighter.value || !code.value) {
         return '';
@@ -128,89 +143,104 @@ onMounted(async () => {
     padding-bottom: 12px;
 }
 
+/* ── Code section (matches ToolResultView .result-code) ──────── */
+
 .code-section {
-    margin-bottom: 12px;
+    margin-bottom: 8px;
 }
 
-.code-highlighted {
-    border-radius: 6px;
-    overflow: hidden;
-    font-size: 12px;
-    line-height: 1.5;
+/* Fallback (non-Shiki) code block */
+.code-fallback {
+    margin: 0;
+    padding: 8px 10px;
+    border-radius: 4px;
     overflow-x: auto;
+    font-family: ui-monospace, SFMono-Regular, Menlo, Monaco, Consolas, monospace;
+    font-size: 11.5px;
+    line-height: 1.55;
+    background: rgba(var(--v-theme-on-surface), 0.04);
+    white-space: pre-wrap;
+    word-break: break-all;
+    max-height: 300px;
+    overflow-y: auto;
 }
 
-:deep(.code-highlighted pre.shiki) {
+/* Shiki highlighted code — mirrors ToolResultView .result-code-shiki */
+.code-result-shiki {
+    padding: 0;
+    border-radius: 4px;
+    overflow: hidden;
+}
+
+:deep(.code-result-shiki pre.shiki) {
     margin: 0;
-    padding: 16px;
-    border-radius: 6px;
+    padding: 8px 10px;
+    border-radius: 4px;
     overflow: auto;
+    max-height: 300px;
+    font-size: 11.5px;
+    line-height: 1.55;
+    tab-size: 4;
 }
 
-:deep(.code-highlighted pre.shiki code) {
+:deep(.code-result-shiki pre.shiki code) {
     display: block;
     padding: 0;
     background: transparent;
-    border-radius: 0;
-}
-
-.code-fallback {
-    margin: 0;
-    padding: 12px;
-    border-radius: 6px;
-    overflow-x: auto;
-    font-size: 12px;
-    line-height: 1.5;
-    background-color: #f5f5f5;
+    font-family: inherit;
+    font-size: inherit;
+    line-height: inherit;
 }
 
-.code-fallback.dark-theme {
-    background-color: rgb(var(--v-theme-codeBg));
-}
+/* ── Result section ─────────────────────────────────────────── */
 
 .result-section {
-    margin-top: 12px;
+    margin-top: 8px;
 }
 
 .result-label {
-    font-size: 12px;
+    font-size: 11px;
     font-weight: 600;
-    color: var(--v-theme-secondaryText);
-    margin-bottom: 6px;
+    color: rgba(var(--v-theme-on-surface), 0.55);
+    margin-bottom: 4px;
     opacity: 0.8;
 }
 
 .result-content {
     margin: 0;
-    padding: 12px;
-    border-radius: 6px;
+    padding: 8px 10px;
+    border-radius: 4px;
     overflow-x: auto;
-    font-size: 12px;
-    line-height: 1.5;
-    background-color: #f5f5f5;
+    font-family: ui-monospace, SFMono-Regular, Menlo, Monaco, Consolas, monospace;
+    font-size: 11.5px;
+    line-height: 1.55;
+    background: rgba(var(--v-theme-on-surface), 0.04);
+    white-space: pre-wrap;
+    word-break: break-all;
     max-height: 300px;
     overflow-y: auto;
 }
 
-.result-content.dark-theme {
-    background-color: rgb(var(--v-theme-codeBg));
+/* ── System notice suffix ───────────────────────────────────── */
+.result-suffix {
+    margin-top: 6px;
+    padding: 4px 8px;
+    border-radius: 4px;
+    background: rgba(var(--v-theme-on-surface), 0.03);
+    font-family: ui-monospace, SFMono-Regular, Menlo, Monaco, Consolas, monospace;
+    font-size: 11px;
+    line-height: 1.55;
+    color: rgba(var(--v-theme-on-surface), 0.55);
+    white-space: pre-wrap;
+    word-break: break-word;
 }
 
 .animate-fade-in {
     animation: fadeIn 0.2s ease-in-out;
 }
 
-:deep(.code-highlighted pre) {
-    background-color: transparent !important;
-}
-
 @keyframes fadeIn {
-    from {
-        opacity: 0;
-    }
-
-    to {
-        opacity: 1;
-    }
+    from { opacity: 0; }
+    to { opacity: 1; }
 }
 </style>
diff --git a/dashboard/src/components/chat/message_list_comps/ToolCallCard.vue b/dashboard/src/components/chat/message_list_comps/ToolCallCard.vue
index 532481f864..5e76d38456 100644
--- a/dashboard/src/components/chat/message_list_comps/ToolCallCard.vue
+++ b/dashboard/src/components/chat/message_list_comps/ToolCallCard.vue
@@ -88,7 +88,7 @@
 <script setup>
 import { computed, onMounted, onUnmounted, reactive, ref } from "vue";
 import { useModuleI18n } from "@/i18n/composables";
-import { findRealSystemNoticeIndex } from "@/utils/systemNotice";
+import { findSystemNoticeIndex } from "@/utils/systemNotice";
 import DiffPreview from "./DiffPreview.vue";
 import ToolResultView from "./ToolResultView.vue";
 
@@ -189,17 +189,17 @@ const isEditTool = computed(
 );
 
 // Strip [SYSTEM NOTICE] suffix from the raw result for all edit tool computed properties.
-// Uses the shared findRealSystemNoticeIndex() to correctly distinguish genuine system
+// Uses the shared findSystemNoticeIndex() to correctly distinguish genuine system
 // notices from "[SYSTEM NOTICE]" text that may appear inside the edited file's diff content.
 const editToolCleanResult = computed(() => {
   const raw = props.toolCall.result ?? "";
-  const idx = findRealSystemNoticeIndex(raw);
+  const idx = findSystemNoticeIndex(raw);
   return idx < 0 ? raw : raw.slice(0, idx).trim();
 });
 
 const editToolNotice = computed(() => {
   const raw = props.toolCall.result ?? "";
-  const idx = findRealSystemNoticeIndex(raw);
+  const idx = findSystemNoticeIndex(raw);
   return idx < 0 ? null : raw.slice(idx).trim();
 });
 
diff --git a/dashboard/src/components/chat/message_list_comps/ToolResultView.vue b/dashboard/src/components/chat/message_list_comps/ToolResultView.vue
index 28063951e0..ce6407f9bc 100644
--- a/dashboard/src/components/chat/message_list_comps/ToolResultView.vue
+++ b/dashboard/src/components/chat/message_list_comps/ToolResultView.vue
@@ -72,7 +72,7 @@
       <pre class="result-raw">{{ formattedResult }}</pre>
     </template>
 
-    <!-- ── shared [SYSTEM NOTICE] suffix (exclude shell which handles it separately) ── -->
+    <!-- ── shared system suffix ([SYSTEM NOTICE] + overflow notice; exclude shell which handles it separately) ── -->
     <div v-if="resultSuffix && toolName !== 'astrbot_execute_shell'" class="result-suffix">{{ resultSuffix }}</div>
   </div>
 </template>
@@ -84,7 +84,7 @@ import {
   escapeHtml,
   renderShikiCode,
 } from "@/utils/shiki";
-import { findRealSystemNoticeIndex } from "@/utils/systemNotice";
+import { findSystemNoticeIndex } from "@/utils/systemNotice";
 
 const props = defineProps<{
   toolName: string;
@@ -149,34 +149,20 @@ const shikiReady = ref(false);
 
 const rawResult = computed(() => (props.result ?? "").trim());
 
-// Strip [SYSTEM NOTICE] suffix for all non-shell templates.
-// Shell uses rawResult directly via brace-tracking in shellParsed.
-// file_read_tool uses smart detection to avoid false positives from
-// file content that coincidentally contains "[SYSTEM NOTICE]".
+// Strip system-injected suffixes ([SYSTEM NOTICE] + overflow notice) for all
+// non-shell templates.  Shell uses rawResult directly via brace-tracking in
+// shellParsed.  All other tools now use the smart findSystemNoticeIndex()
+// which handles both [SYSTEM NOTICE] markers and the overflow notice pattern.
 const resultText = computed(() => {
   const text = rawResult.value;
-
-  if (props.toolName === "astrbot_file_read_tool") {
-    const idx = findRealSystemNoticeIndex(text);
-    if (idx < 0) return text;
-    return text.slice(0, idx).trim();
-  }
-
-  const idx = text.search(/\[SYSTEM NOTICE\]/i);
+  const idx = findSystemNoticeIndex(text);
   if (idx < 0) return text;
   return text.slice(0, idx).trim();
 });
 
 const resultSuffix = computed(() => {
   const text = rawResult.value;
-
-  if (props.toolName === "astrbot_file_read_tool") {
-    const idx = findRealSystemNoticeIndex(text);
-    if (idx < 0) return null;
-    return text.slice(idx).trim();
-  }
-
-  const idx = text.search(/\[SYSTEM NOTICE\]/i);
+  const idx = findSystemNoticeIndex(text);
   if (idx < 0) return null;
   return text.slice(idx).trim();
 });
diff --git a/dashboard/src/utils/systemNotice.ts b/dashboard/src/utils/systemNotice.ts
index d682767342..b0497e76ef 100644
--- a/dashboard/src/utils/systemNotice.ts
+++ b/dashboard/src/utils/systemNotice.ts
@@ -1,98 +1,179 @@
 /**
- * Utilities for detecting and extracting the genuine AstrBot
- * [SYSTEM NOTICE] suffix from tool call results.
+ * Utilities for detecting and extracting system-injected suffixes from
+ * tool call results.  Two kinds of suffixes are recognised:
+ *
+ * 1. **[SYSTEM NOTICE]** — repeated-tool warnings, follow-up notices, etc.
+ *    Always prefixed with the literal marker `[SYSTEM NOTICE]` followed by
+ *    one of the known SYSTEM_NOTICE_PREFIXES.
+ *
+ * 2. **Tool-result overflow notice** — emitted when a tool output is too
+ *    large and gets spilled to a file.  The notice text starts with
+ *    `"Truncated tool output preview shown above."` and does NOT carry the
+ *    `[SYSTEM NOTICE]` marker (it is a plain-text suffix).
+ *
+ * Both suffixes are always appended at the very end of the tool result by
+ * the backend, possibly as a chain of multiple notices (e.g. repeated-tool
+ * warning + follow-up notice).  When the tool output itself contains text
+ * that looks like a system notice (e.g. reading a log file), the algorithm
+ * must correctly ignore those false positives.
+ *
+ * **Key invariant**: the real system notice chain starts at the EARLIEST
+ * valid candidate where everything from that position to end-of-text is
+ * exclusively notice-chain content (no file content mixed in).
  *
- * @author AstrBot Agent Harness
- * @since 2026-05-30
- */
-
-/**
  * Known message prefixes produced by the AstrBot system (not file content).
- * Used by findRealSystemNoticeIndex() to distinguish genuine system notices
- * from [SYSTEM NOTICE] text that happens to appear inside file content
- * (e.g. when reading AstrBot log files or editing files that contain the
- * literal string "[SYSTEM NOTICE]").
+ * Used to distinguish genuine system notices from text that happens to
+ * contain the literal string "[SYSTEM NOTICE]".
  */
 export const SYSTEM_NOTICE_PREFIXES = [
   "By the way,",
   "Important:",
-  "This is a",
+  "User sent",
 ];
 
 const MARKER = "[SYSTEM NOTICE]";
 
+/** Distinctive opening of the tool-result overflow notice. */
+const OVERFLOW_NOTICE_PREFIX = "Truncated tool output preview shown above.";
+
+// ── Helpers ─────────────────────────────────────────────────────────────
+
+/**
+ * Check whether the text starting at `idx` begins with a genuine
+ * `[SYSTEM NOTICE]` marker followed by one of the known prefixes.
+ */
+function isGenuineSystemNotice(text: string, idx: number): boolean {
+  const suffix = text.slice(idx).trim();
+  return SYSTEM_NOTICE_PREFIXES.some((p) =>
+    suffix.startsWith(`${MARKER} ${p}`),
+  );
+}
+
 /**
- * Find the index of the genuine AstrBot system notice in the text.
+ * Check whether the entire region from `fromPos` to end-of-text contains
+ * ONLY notice-chain content (no real file content).
  *
- * For tools whose result may contain file content (e.g. astrbot_file_read_tool,
- * astrbot_file_edit_tool), the raw result can include literal "[SYSTEM NOTICE]"
- * strings that are part of the file being read/edited, not a system message.
+ * Allowed line types within the notice chain:
+ *   - Blank lines (separators between notices)
+ *   - Lines containing `[SYSTEM NOTICE]` (notice markers and their bodies)
+ *   - Lines starting with the overflow notice prefix
+ *   - Numbered list items (`N. …`) — but ONLY when preceded by a
+ *     `[SYSTEM NOTICE] User sent` (follow-up notice) in the chain.
+ *     This prevents file content like `1. item` from being mistaken for
+ *     notice content.
+ *   - Continuation lines of a notice paragraph (non-blank, directly after
+ *     a notice marker line, before any blank line gap).
+ */
+function isNoticeChainRegion(text: string, fromPos: number): boolean {
+  const region = text.slice(fromPos);
+  const lines = region.split("\n");
+  let inFollowUp = false;
+  let sawBlankAfterNotice = false;
+
+  for (const rawLine of lines) {
+    const trimmed = rawLine.trim();
+
+    // Blank lines are separators between notices — always allowed.
+    if (trimmed === "") {
+      if (!inFollowUp) sawBlankAfterNotice = true;
+      continue;
+    }
+
+    // [SYSTEM NOTICE] marker — starts a new notice in the chain.
+    if (trimmed.includes(MARKER)) {
+      // Track whether this is a follow-up notice (allows numbered items).
+      const markerIdx = trimmed.indexOf(MARKER);
+      const afterMarker = trimmed.slice(markerIdx).trim();
+      if (afterMarker.startsWith(`${MARKER} User sent`)) {
+        inFollowUp = true;
+      }
+      sawBlankAfterNotice = false;
+      continue;
+    }
+
+    // Overflow notice prefix — part of the chain.
+    if (trimmed.startsWith(OVERFLOW_NOTICE_PREFIX)) {
+      inFollowUp = false;
+      sawBlankAfterNotice = false;
+      continue;
+    }
+
+    // Numbered list items — only valid after a "User sent" follow-up notice.
+    if (inFollowUp && /^\d+\.\s/.test(trimmed)) continue;
+
+    // Continuation lines:
+    // - In follow-up mode: free-form instruction text is part of the notice body.
+    // - Outside follow-up: continuation of an overflow notice paragraph
+    //   (before any blank line gap).
+    if (inFollowUp || !sawBlankAfterNotice) {
+      continue;
+    }
+
+    // Anything else is real file content — not a valid notice chain.
+    return false;
+  }
+
+  return true;
+}
+
+// ── Public API ──────────────────────────────────────────────────────────
+
+/**
+ * Find the character index where the system notice suffix begins.
+ *
+ * Returns the EARLIEST position where a genuine notice candidate starts
+ * and everything from that position to end-of-text is exclusively
+ * notice-chain content.  This ensures that when multiple notices are
+ * chained (e.g. repeated-tool warning + follow-up notice), ALL of them
+ * are captured in the suffix.
  *
- * Key invariant: the REAL [SYSTEM NOTICE] is ALWAYS the very last thing in
- * the tool result — there is NO content after it.  Any "[SYSTEM NOTICE]"
- * that has non-empty lines following it is necessarily part of the file
- * content, not a system message.
+ * For tools whose result may contain file content (e.g. astrbot_file_read_tool,
+ * astrbot_file_edit_tool), the raw result can include literal
+ * `[SYSTEM NOTICE]` strings or overflow-notice-like text that are part of
+ * the file being read/edited, not a system message.  These are rejected
+ * because the region from the fake notice to end-of-text contains real
+ * file content.
  *
- * Strategy:
- *   1. Find all occurrences of [SYSTEM NOTICE] in the text.
- *   2. Scan from the LAST occurrence backward (most likely to be real).
- *   3. For each candidate:
- *      a. Check it matches a known system prefix.
- *      b. Check there are NO non-empty lines after the notice line.
- *         If there are, this [SYSTEM NOTICE] is embedded in file
- *         content — skip it and try the previous occurrence.
- *   4. If no candidate passes both checks, there is no real notice.
+ * **Strategy** (first valid candidate wins):
+ *   1. Find all `[SYSTEM NOTICE]` markers in the text (in order).
+ *   2. For each genuine marker (earliest first), check if the region from
+ *      that position to end-of-text is a valid notice chain.
+ *   3. Return the first position that passes.
+ *   4. If no `[SYSTEM NOTICE]` passes, check for overflow notice.
+ *   5. If nothing passes, return -1.
  *
- * @returns The character index of the real [SYSTEM NOTICE], or -1 if none.
+ * @returns The character index where the notice suffix begins, or -1.
  */
-export function findRealSystemNoticeIndex(text: string): number {
-  // Collect all occurrences so we can scan from last to first.
-  const occurrences: number[] = [];
+export function findSystemNoticeIndex(text: string): number {
+  // ── Collect all candidates and sort by position ascending ─────────
+  const positions: { pos: number; type: "system" | "overflow" }[] = [];
+
   let searchFrom = 0;
   while (searchFrom < text.length) {
     const idx = text.indexOf(MARKER, searchFrom);
     if (idx < 0) break;
-    occurrences.push(idx);
+    positions.push({ pos: idx, type: "system" });
     searchFrom = idx + MARKER.length;
   }
-  if (occurrences.length === 0) return -1;
-
-  // Scan from the last occurrence backward.
-  for (let i = occurrences.length - 1; i >= 0; i--) {
-    const idx = occurrences[i];
-    const suffix = text.slice(idx).trim();
-
-    // Must match a known system prefix pattern.
-    if (
-      !SYSTEM_NOTICE_PREFIXES.some((p) =>
-        suffix.startsWith(`${MARKER} ${p}`),
-      )
-    ) {
-      continue;
-    }
 
-    // The real system notice is always at the very end of the tool
-    // result with NO content after it.  Check if there are any
-    // non-empty lines following the notice line — if so, this is
-    // file content that happens to contain [SYSTEM NOTICE], not a
-    // genuine system message.
-    const afterText = text.slice(idx);
-    const lines = afterText.split("\n");
-    let hasContentAfter = false;
-    for (let j = 1; j < lines.length; j++) {
-      if (lines[j].trim() !== "") {
-        hasContentAfter = true;
-        break;
-      }
-    }
-    if (hasContentAfter) {
-      // This [SYSTEM NOTICE] has file content after it — it's fake.
-      // Continue checking earlier occurrences.
-      continue;
-    }
+  searchFrom = 0;
+  while (searchFrom < text.length) {
+    const idx = text.indexOf(OVERFLOW_NOTICE_PREFIX, searchFrom);
+    if (idx < 0) break;
+    positions.push({ pos: idx, type: "overflow" });
+    searchFrom = idx + OVERFLOW_NOTICE_PREFIX.length;
+  }
 
-    // No content after this notice — it's the real one.
-    return idx;
+  positions.sort((a, b) => a.pos - b.pos);
+
+  // ── Check each candidate (earliest first) ────────────────────────
+  for (const { pos, type } of positions) {
+    // Quick prefix check — skip non-genuine candidates.
+    if (type === "system" && !isGenuineSystemNotice(text, pos)) continue;
+
+    if (isNoticeChainRegion(text, pos)) {
+      return pos;
+    }
   }
 
   return -1;
@@ -100,7 +181,7 @@ export function findRealSystemNoticeIndex(text: string): number {
 
 /**
  * Split tool result text into the main content and the optional
- * [SYSTEM NOTICE] suffix.
+ * system notice suffix.
  *
  * @param text  The raw tool result text.
  * @returns An object with `content` (the main result) and `notice`
@@ -109,7 +190,7 @@ export function findRealSystemNoticeIndex(text: string): number {
 export function splitSystemNotice(
   text: string,
 ): { content: string; notice: string | null } {
-  const idx = findRealSystemNoticeIndex(text);
+  const idx = findSystemNoticeIndex(text);
   if (idx < 0) return { content: text, notice: null };
   return {
     content: text.slice(0, idx).trim(),

From 5328bea3d19a6b6ccb543321d76317a7c724a175 Mon Sep 17 00:00:00 2001
From: "DESKTOP-02FN3OU\\80423" <804235820@qq.com>
Date: Sat, 30 May 2026 16:10:21 +0800
Subject: [PATCH 10/15] no message

---
 dashboard/src/components/shared/ThemeAwareMarkdownCodeBlock.vue | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/dashboard/src/components/shared/ThemeAwareMarkdownCodeBlock.vue b/dashboard/src/components/shared/ThemeAwareMarkdownCodeBlock.vue
index 608edae579..fc0b4dc4b7 100644
--- a/dashboard/src/components/shared/ThemeAwareMarkdownCodeBlock.vue
+++ b/dashboard/src/components/shared/ThemeAwareMarkdownCodeBlock.vue
@@ -3,7 +3,7 @@
     v-if="node.lang === 'diff'"
     :content="String((node as Record<string, unknown>).content ?? (node as Record<string, unknown>).code ?? '')"
     :is-dark="effectiveIsDark"
-    :max-lines="30"
+    :max-lines="100"
   />
   <MarkdownCodeBlockNode
     v-else

From 323d8c97ef31f43e467df9c82431c8f59672cfaa Mon Sep 17 00:00:00 2001
From: "DESKTOP-02FN3OU\\80423" <804235820@qq.com>
Date: Sat, 30 May 2026 16:11:49 +0800
Subject: [PATCH 11/15] Revert "no message"

This reverts commit 5328bea3d19a6b6ccb543321d76317a7c724a175.
---
 dashboard/src/components/shared/ThemeAwareMarkdownCodeBlock.vue | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/dashboard/src/components/shared/ThemeAwareMarkdownCodeBlock.vue b/dashboard/src/components/shared/ThemeAwareMarkdownCodeBlock.vue
index fc0b4dc4b7..608edae579 100644
--- a/dashboard/src/components/shared/ThemeAwareMarkdownCodeBlock.vue
+++ b/dashboard/src/components/shared/ThemeAwareMarkdownCodeBlock.vue
@@ -3,7 +3,7 @@
     v-if="node.lang === 'diff'"
     :content="String((node as Record<string, unknown>).content ?? (node as Record<string, unknown>).code ?? '')"
     :is-dark="effectiveIsDark"
-    :max-lines="100"
+    :max-lines="30"
   />
   <MarkdownCodeBlockNode
     v-else

From 6818a88243e97bf9920e38c57916d25f6aea3683 Mon Sep 17 00:00:00 2001
From: "DESKTOP-02FN3OU\\80423" <804235820@qq.com>
Date: Sat, 30 May 2026 18:24:15 +0800
Subject: [PATCH 12/15] =?UTF-8?q?fix(edit=5Fengine):=20=E8=A7=A3=E5=86=B3?=
 =?UTF-8?q?=E6=A8=A1=E7=B3=8A=E6=9B=BF=E6=8D=A2=E6=97=B6=E7=9A=84=E7=BC=A9?=
 =?UTF-8?q?=E8=BF=9B=E9=97=AE=E9=A2=98?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 .../core/tools/computer_tools/edit_engine.py  | 138 +++++++++++++++++-
 1 file changed, 136 insertions(+), 2 deletions(-)

diff --git a/astrbot/core/tools/computer_tools/edit_engine.py b/astrbot/core/tools/computer_tools/edit_engine.py
index 8b013376b6..66f7182e0b 100644
--- a/astrbot/core/tools/computer_tools/edit_engine.py
+++ b/astrbot/core/tools/computer_tools/edit_engine.py
@@ -430,6 +430,126 @@ def _multi_occurrence_replacer(content: str, find: str) -> Iterator[str]:
 ]
 
 
+# ---------------------------------------------------------------------------
+# Indent adjustment helper
+# ---------------------------------------------------------------------------
+
+
+def _get_leading_whitespace(line: str) -> str:
+    """Return the leading whitespace of a line."""
+    return line[: len(line) - len(line.lstrip())]
+
+
+def _first_nonempty_line_indent(text: str) -> str:
+    """Get the leading whitespace of the first non-empty line in text."""
+    for line in text.split("\n"):
+        if line.strip():
+            return _get_leading_whitespace(line)
+    return ""
+
+
+def _adjust_replacement_indent(
+    new_string: str,
+    old_string: str,
+    matched_block: str,
+    content: str = "",
+    match_idx: int = -1,
+) -> str:
+    """
+    Adjust new_string's indentation to preserve the file's indent level.
+
+    When a fuzzy replacer (e.g. LineTrimmedReplacer, IndentationFlexibleReplacer)
+    matches a block whose indentation differs from old_string, this function
+    computes the indent delta and applies it to every non-empty line of
+    new_string so the replacement preserves the file's original indentation.
+
+    When *content* and *match_idx* are provided, the effective indent is
+    computed from the actual line in the file rather than from matched_block
+    alone. This correctly handles substring matches (e.g. SimpleReplacer
+    matching ``"print(x)"`` inside ``"                print(x)"``).
+
+    Indent adjustment rules:
+    - If the match starts at the beginning of a line (match_idx == line_start),
+      all lines of new_string get the indent delta applied.
+    - If the match starts after a whitespace-only prefix (at the content
+      boundary of an indented line), the first line is NOT adjusted —
+      the content prefix already provides the indent. Lines 2+ get adjusted.
+    - If the match starts mid-line (after non-whitespace content), NO lines
+      are adjusted — the user wrote absolute indentation and the first line
+      inherits the prefix context.
+
+    For exact matches (old_string == matched_block and both span the full
+    line), delta is zero and this is a no-op.
+    """
+    old_indent = _first_nonempty_line_indent(old_string)
+
+    # Compute effective file indent from the actual line context when available.
+    # This handles substring matches where matched_block doesn't include
+    # the line's leading whitespace.
+    match_starts_at_line_beginning = True
+    adjust_subsequent_lines = True
+    if content and match_idx >= 0:
+        line_start = content.rfind("\n", 0, match_idx) + 1
+        effective_line = content[line_start:]
+        file_indent = _get_leading_whitespace(effective_line)
+        if match_idx == line_start:
+            # Match starts at the very beginning of the line
+            match_starts_at_line_beginning = True
+            adjust_subsequent_lines = True
+        else:
+            prefix = content[line_start:match_idx]
+            match_starts_at_line_beginning = False
+            if prefix.strip() == "":
+                # Match starts after whitespace-only prefix (content boundary)
+                # First line inherits prefix; subsequent lines get adjusted
+                adjust_subsequent_lines = True
+            else:
+                # Match starts after non-whitespace (truly mid-line)
+                # User wrote absolute indentation; no adjustment needed
+                adjust_subsequent_lines = False
+    else:
+        file_indent = _first_nonempty_line_indent(matched_block)
+
+    if file_indent == old_indent:
+        return new_string
+
+    # Determine indent character from the file's indentation
+    indent_char = " "
+    if file_indent:
+        indent_char = file_indent[0]
+    elif old_indent:
+        indent_char = old_indent[0]
+
+    delta = len(file_indent) - len(old_indent)
+
+    new_lines = new_string.split("\n")
+    adjusted: list[str] = []
+    for i, line in enumerate(new_lines):
+        if not line.strip():
+            # Empty or whitespace-only line: keep as-is
+            adjusted.append(line)
+        elif i == 0 and not match_starts_at_line_beginning:
+            # First line and match is not at line start: don't adjust.
+            # The content prefix already provides the indent context.
+            adjusted.append(line)
+        elif i > 0 and not adjust_subsequent_lines:
+            # Subsequent lines in a truly mid-line match: don't adjust.
+            # User wrote absolute indentation.
+            adjusted.append(line)
+        elif delta > 0:
+            # Need to add indentation to match file's deeper indent level
+            adjusted.append(indent_char * delta + line)
+        elif delta < 0:
+            # Need to remove indentation (file is less indented than old_string)
+            current_indent = _get_leading_whitespace(line)
+            remove = min(-delta, len(current_indent))
+            adjusted.append(line[remove:])
+        else:
+            adjusted.append(line)
+
+    return "\n".join(adjusted)
+
+
 # ---------------------------------------------------------------------------
 # Core replace function
 # ---------------------------------------------------------------------------
@@ -487,8 +607,15 @@ def robust_replace(
             new_content = content
             replacements = 0
             for idx, match in sorted(match_positions, key=lambda x: x[0], reverse=True):
+                adjusted_new = _adjust_replacement_indent(
+                    new_string,
+                    old_string,
+                    match,
+                    content,
+                    idx,
+                )
                 new_content = (
-                    new_content[:idx] + new_string + new_content[idx + len(match) :]
+                    new_content[:idx] + adjusted_new + new_content[idx + len(match) :]
                 )
                 replacements += 1
             return new_content, replacements
@@ -496,7 +623,14 @@ def robust_replace(
         # Single replacement mode: require exactly one match
         if len(match_positions) == 1:
             idx, match = match_positions[0]
-            return content[:idx] + new_string + content[idx + len(match) :], 1
+            adjusted_new = _adjust_replacement_indent(
+                new_string,
+                old_string,
+                match,
+                content,
+                idx,
+            )
+            return content[:idx] + adjusted_new + content[idx + len(match) :], 1
 
         # Multiple matches found in single-replacement mode: continue to next replacer
         # to try a more specific strategy

From 567d9ac4097e9448bd0001bed818381f549df907 Mon Sep 17 00:00:00 2001
From: "DESKTOP-02FN3OU\\80423" <804235820@qq.com>
Date: Sat, 30 May 2026 18:39:33 +0800
Subject: [PATCH 13/15] =?UTF-8?q?fix(edit=5Fengine):=20=E8=A7=A3=E5=86=B3?=
 =?UTF-8?q?=E6=B2=99=E7=9B=92=E4=B8=AD=E7=9A=84CRLF=E5=8C=B9=E9=85=8D?=
 =?UTF-8?q?=E5=92=8C=E5=86=99=E5=85=A5=E9=97=AE=E9=A2=98?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 .../core/tools/computer_tools/edit_engine.py  | 130 +++++++++++-------
 astrbot/core/tools/computer_tools/fs.py       | 118 +++++++++++-----
 2 files changed, 166 insertions(+), 82 deletions(-)

diff --git a/astrbot/core/tools/computer_tools/edit_engine.py b/astrbot/core/tools/computer_tools/edit_engine.py
index 66f7182e0b..24c222d8fa 100644
--- a/astrbot/core/tools/computer_tools/edit_engine.py
+++ b/astrbot/core/tools/computer_tools/edit_engine.py
@@ -668,6 +668,79 @@ class EditResult:
 # ---------------------------------------------------------------------------
 
 
+def bytes_edit_file(
+    raw_bytes: bytes,
+    old_string: str,
+    new_string: str,
+    *,
+    replace_all: bool = False,
+    encoding: str = "utf-8",
+) -> tuple[bytes, EditResult]:
+    """
+    Core line-ending-aware edit logic operating on raw bytes.
+
+    Performs:
+    1. BOM detection and preservation
+    2. Line-ending detection (CRLF vs LF)
+    3. Normalize to LF for matching (replacer chain works on LF)
+    4. Execute robust_replace()
+    5. Convert back to original line endings
+    6. Re-attach BOM if present
+
+    Returns:
+        (output_bytes, EditResult) — the caller is responsible for writing
+        *output_bytes* back to the file.
+
+    Raises:
+        ValueError: If old_string cannot be found or is not unique.
+    """
+    has_bom = raw_bytes.startswith(b"\xef\xbb\xbf")
+    if has_bom:
+        raw_bytes = raw_bytes[3:]
+
+    old_content = raw_bytes.decode(encoding)
+    original_ending = _detect_line_ending(old_content)
+
+    # Normalize for matching: ONLY normalize actual CRLF line endings.
+    # Escape sequence handling (\n vs actual newline, \t vs tab, etc.)
+    # is deferred to the _escape_normalized_replacer in the replacer chain.
+    normalized_old = _normalize_line_endings(old_string)
+    normalized_new = _normalize_line_endings(new_string)
+
+    # Normalize file content to LF for matching (replacers work on LF)
+    normalized_content = _normalize_line_endings(old_content)
+
+    # Perform replacement
+    new_content, replacements = robust_replace(
+        normalized_content,
+        normalized_old,
+        normalized_new,
+        replace_all=replace_all,
+    )
+
+    # Convert back to original line endings
+    if original_ending == "\r\n":
+        new_content = _convert_to_line_ending(new_content, "\r\n")
+
+    # Re-add BOM if present
+    write_bytes = b""
+    if has_bom:
+        write_bytes += b"\xef\xbb\xbf"
+    write_bytes += new_content.encode(encoding)
+
+    # Generate unified diff
+    diff = build_unified_diff("", old_content, new_content)
+
+    result = EditResult(
+        success=True,
+        replacements=replacements,
+        diff=diff,
+        old_content=old_content,
+        new_content=new_content,
+    )
+    return write_bytes, result
+
+
 async def edit_file(
     path: str,
     old_string: str,
@@ -688,59 +761,18 @@ async def edit_file(
     lock = get_file_lock(path)
     async with lock:
         try:
-            # Read file
+            # Read file in binary mode to preserve original line endings
             raw_bytes = await asyncio.to_thread(_read_file_bytes, path)
-            has_bom = raw_bytes.startswith(b"\xef\xbb\xbf")
-            if has_bom:
-                raw_bytes = raw_bytes[3:]
-
-            old_content = raw_bytes.decode(encoding)
-            original_ending = _detect_line_ending(old_content)
-
-            # Normalize for matching: ONLY normalize actual CRLF line endings.
-            # Escape sequence handling (\n vs actual newline, \t vs tab, etc.)
-            # is deferred to the _escape_normalized_replacer in the replacer chain.
-            normalized_old = _normalize_line_endings(old_string)
-            normalized_new = _normalize_line_endings(new_string)
-
-            # Normalize file content to LF for matching (replacers work on LF)
-            normalized_content = _normalize_line_endings(old_content)
-            # Perform replacement
-            new_content, replacements = robust_replace(
-                normalized_content,
-                normalized_old,
-                normalized_new,
+            write_bytes, result = bytes_edit_file(
+                raw_bytes,
+                old_string,
+                new_string,
                 replace_all=replace_all,
+                encoding=encoding,
             )
-
-            # Convert back to original line endings
-            if original_ending == "\r\n":
-                new_content = _convert_to_line_ending(new_content, "\r\n")
-
-            # Re-add BOM if present
-            write_bytes = b""
-            if has_bom:
-                write_bytes += b"\xef\xbb\xbf"
-            write_bytes += new_content.encode(encoding)
-
-            # Write file
+            # Write file in binary mode to preserve restored line endings
             await asyncio.to_thread(_write_file_bytes, path, write_bytes)
-
-            # Generate unified diff
-            diff = build_unified_diff(
-                path,
-                old_content,
-                new_content,
-            )
-
-            return EditResult(
-                success=True,
-                replacements=replacements,
-                diff=diff,
-                old_content=old_content,
-                new_content=new_content,
-            )
-
+            return result
         except Exception as exc:
             return EditResult(
                 success=False,
diff --git a/astrbot/core/tools/computer_tools/fs.py b/astrbot/core/tools/computer_tools/fs.py
index d381ac78e1..b4eaadc414 100644
--- a/astrbot/core/tools/computer_tools/fs.py
+++ b/astrbot/core/tools/computer_tools/fs.py
@@ -33,6 +33,7 @@
 - In sandbox runtime, relative paths are passed through unchanged.
 """
 
+import base64
 import os
 import uuid
 from dataclasses import dataclass, field
@@ -57,10 +58,9 @@
 from . import util as computer_util
 from .edit_engine import (
     EditResult,
-    build_unified_diff,
+    bytes_edit_file,
     edit_file,
     get_file_lock,
-    robust_replace,
 )
 from .util import (
     check_admin_permission,
@@ -375,6 +375,73 @@ async def call(
             return f"Error writing file: {exc}"
 
 
+async def _sandbox_read_bytes(sb, path: str) -> bytes:
+    """Read a file in binary mode inside the sandbox via sb.python.exec.
+
+    Returns the raw bytes so that original line endings (CRLF/LF) are preserved.
+    Raises IOError on read failure.
+    """
+    code = (
+        "import base64 as _b64, sys\n"
+        f"path = {path!r}\n"
+        "try:\n"
+        "    with open(path, 'rb') as _f:\n"
+        "        _data = _f.read()\n"
+        "    print(_b64.b64encode(_data).decode(), end='')\n"
+        "except Exception as _e:\n"
+        "    print('ERROR:' + str(_e), end='', file=sys.stderr)\n"
+        "    sys.exit(1)\n"
+    )
+    result = await sb.python.exec(code, timeout=30)
+    if not result.get("success", False):
+        error = str(result.get("error", "") or "").strip()
+        if not error:
+            output = result.get("output", "")
+            if isinstance(output, dict):
+                error = str(output.get("error", "") or "").strip()
+        raise IOError(
+            f"Failed to read file in sandbox: "
+            f"{error or 'unknown read error'}"
+        )
+    output = result.get("output", "")
+    if isinstance(output, dict):
+        output = output.get("text", "")
+    b64_text = str(output).strip()
+    return base64.b64decode(b64_text)
+
+
+async def _sandbox_write_bytes(sb, path: str, data: bytes) -> None:
+    """Write raw bytes to a file inside the sandbox via sb.python.exec.
+
+    Raises IOError on write failure.
+    """
+    b64_data = base64.b64encode(data).decode()
+    code = (
+        "import base64 as _b64, sys\n"
+        f"path = {path!r}\n"
+        f"b64 = {b64_data!r}\n"
+        "try:\n"
+        "    _raw = _b64.b64decode(b64)\n"
+        "    with open(path, 'wb') as _f:\n"
+        "        _f.write(_raw)\n"
+        "    print('ok', end='')\n"
+        "except Exception as _e:\n"
+        "    print('ERROR:' + str(_e), end='', file=sys.stderr)\n"
+        "    sys.exit(1)\n"
+    )
+    result = await sb.python.exec(code, timeout=30)
+    if not result.get("success", False):
+        error = str(result.get("error", "") or "").strip()
+        if not error:
+            output = result.get("output", "")
+            if isinstance(output, dict):
+                error = str(output.get("error", "") or "").strip()
+        raise IOError(
+            f"Failed to write file in sandbox: "
+            f"{error or 'unknown write error'}"
+        )
+
+
 def _format_result(
     path: str,
     result: EditResult,
@@ -513,48 +580,33 @@ async def call(
                 )
                 lock = get_file_lock(normalized_path)
                 async with lock:
-                    read_result = await sb.fs.read_file(path=normalized_path)
-                    if not read_result.get("success", False):
-                        error_detail = str(read_result.get("error", "") or "").strip()
-                        return (
-                            "Error editing file: "
-                            f"{error_detail or 'unknown filesystem read error'}"
+                    # 1. Binary read — preserves original line endings (CRLF/LF)
+                    try:
+                        raw_bytes = await _sandbox_read_bytes(
+                            sb, normalized_path
                         )
+                    except IOError as exc:
+                        return f"Error editing file: {exc}"
 
-                    old_content = read_result.get("content", "")
-
+                    # 2. Line-ending-aware edit (reuses edit_engine core logic)
                     try:
-                        new_content, replacements = robust_replace(
-                            old_content,
+                        write_bytes, result = bytes_edit_file(
+                            raw_bytes,
                             old,
                             new,
                             replace_all=replace_all,
+                            encoding="utf-8",
                         )
                     except ValueError as exc:
                         return f"Error editing file: {exc}"
 
-                    write_result = await sb.fs.write_file(
-                        path=normalized_path,
-                        content=new_content,
-                    )
-                    if not write_result.get("success", False):
-                        error_detail = str(write_result.get("error", "") or "").strip()
-                        return (
-                            "Error editing file: "
-                            f"{error_detail or 'unknown filesystem write error'}"
+                    # 3. Binary write — preserves restored line endings
+                    try:
+                        await _sandbox_write_bytes(
+                            sb, normalized_path, write_bytes
                         )
-
-                    diff = build_unified_diff(
-                        normalized_path,
-                        old_content,
-                        new_content,
-                    )
-
-                    result = EditResult(
-                        success=True,
-                        replacements=replacements,
-                        diff=diff,
-                    )
+                    except IOError as exc:
+                        return f"Error editing file: {exc}"
 
             return _format_result(
                 normalized_path,

From 021c12c11eba89baa3574fe87e328a23e7ee02df Mon Sep 17 00:00:00 2001
From: "DESKTOP-02FN3OU\\80423" <804235820@qq.com>
Date: Sat, 30 May 2026 18:41:15 +0800
Subject: [PATCH 14/15] ruff

---
 astrbot/core/tools/computer_tools/fs.py | 22 ++++++++--------------
 1 file changed, 8 insertions(+), 14 deletions(-)

diff --git a/astrbot/core/tools/computer_tools/fs.py b/astrbot/core/tools/computer_tools/fs.py
index b4eaadc414..6b4a9b8709 100644
--- a/astrbot/core/tools/computer_tools/fs.py
+++ b/astrbot/core/tools/computer_tools/fs.py
@@ -399,9 +399,8 @@ async def _sandbox_read_bytes(sb, path: str) -> bytes:
             output = result.get("output", "")
             if isinstance(output, dict):
                 error = str(output.get("error", "") or "").strip()
-        raise IOError(
-            f"Failed to read file in sandbox: "
-            f"{error or 'unknown read error'}"
+        raise OSError(
+            f"Failed to read file in sandbox: {error or 'unknown read error'}"
         )
     output = result.get("output", "")
     if isinstance(output, dict):
@@ -436,9 +435,8 @@ async def _sandbox_write_bytes(sb, path: str, data: bytes) -> None:
             output = result.get("output", "")
             if isinstance(output, dict):
                 error = str(output.get("error", "") or "").strip()
-        raise IOError(
-            f"Failed to write file in sandbox: "
-            f"{error or 'unknown write error'}"
+        raise OSError(
+            f"Failed to write file in sandbox: {error or 'unknown write error'}"
         )
 
 
@@ -582,10 +580,8 @@ async def call(
                 async with lock:
                     # 1. Binary read — preserves original line endings (CRLF/LF)
                     try:
-                        raw_bytes = await _sandbox_read_bytes(
-                            sb, normalized_path
-                        )
-                    except IOError as exc:
+                        raw_bytes = await _sandbox_read_bytes(sb, normalized_path)
+                    except OSError as exc:
                         return f"Error editing file: {exc}"
 
                     # 2. Line-ending-aware edit (reuses edit_engine core logic)
@@ -602,10 +598,8 @@ async def call(
 
                     # 3. Binary write — preserves restored line endings
                     try:
-                        await _sandbox_write_bytes(
-                            sb, normalized_path, write_bytes
-                        )
-                    except IOError as exc:
+                        await _sandbox_write_bytes(sb, normalized_path, write_bytes)
+                    except OSError as exc:
                         return f"Error editing file: {exc}"
 
             return _format_result(

From 0357cfd60a5f59d619cbcae467e42af86f1578bd Mon Sep 17 00:00:00 2001
From: "DESKTOP-02FN3OU\\80423" <804235820@qq.com>
Date: Sat, 30 May 2026 23:04:50 +0800
Subject: [PATCH 15/15] Resolve conflicts:
 dashboard\src\components\chat\ReasoningSidebar.vue

---
 dashboard/src/components/chat/ReasoningSidebar.vue | 1 -
 1 file changed, 1 deletion(-)

diff --git a/dashboard/src/components/chat/ReasoningSidebar.vue b/dashboard/src/components/chat/ReasoningSidebar.vue
index 6920cffbc8..e2d0d1f240 100644
--- a/dashboard/src/components/chat/ReasoningSidebar.vue
+++ b/dashboard/src/components/chat/ReasoningSidebar.vue
@@ -34,7 +34,6 @@
 
 <script setup lang="ts">
 import { onBeforeUnmount, ref, computed } from "vue";
-import type { MessagePart } from "@/composables/useMessages";
 import {
   reasoningActivityCounts,
   reasoningActivityTitle,