File_System tool is easier for smaller models to understand and 1-shot

Modules/File_System.py

@@ -18,17 +18,18 @@ TOOL_SUMMARY = (
     "Browse, search, and manage files within a safe root. "
     "Actions: list, read, write, append, mkdir, move, copy, delete, info, search, help. "
     "Fill other fields as needed. "
-    "Use paths like `.` because all paths are relative to the root (`/`). "
+    "Use paths like `/` or `/notes/todo.txt` because all paths are relative to the root (`/`). "
     "Use 'help' to see action-specific required fields and examples."
 )
 
 HELP_TEXT = (
     "File System — actions and usage\n\n"
     "Root: paths resolve under Nymbo-Tools/Filesystem by default (or NYMBO_TOOLS_ROOT if set). "
+    "Start paths with '/' to refer to the tool root (e.g., /notes). "
     "Absolute paths are disabled unless UNSAFE_ALLOW_ABS_PATHS=1.\n\n"
     "Actions and fields:\n"
-    "- list: path='.' (default), recursive=false, show_hidden=false, max_entries=20\n"
-    "- read: path, offset=0, max_chars=4000 (shows next_cursor when truncated)\n"
+    "- list: path='/' (default), recursive=false, show_hidden=false, max_entries=20\n"
+    "- read: path (e.g., /notes/todo.txt), offset=0, max_chars=4000 (shows next_cursor when truncated)\n"
     "- write: path, content (UTF-8), create_dirs=true\n"
     "- append: path, content (UTF-8), create_dirs=true\n"
     "- mkdir: path (directory), exist_ok=true\n"
@@ -40,13 +41,13 @@ HELP_TEXT = (
     "- help: show this guide\n\n"
     "Errors are returned as JSON with fields: {status:'error', code, message, path?, hint?, data?}.\n\n"
     "Examples:\n"
-    "- list current: action=list, path='.'\n"
-    "- make folder: action=mkdir, path='notes'\n"
-    "- write file: action=write, path='notes/todo.txt', content='hello'\n"
-    "- read file: action=read, path='notes/todo.txt', max_chars=200\n"
-    "- move file: action=move, path='notes/todo.txt', dest_path='notes/todo-old.txt', overwrite=true\n"
-    "- delete dir: action=delete, path='notes', recursive=true\n"
-    "- search text: action=search, path='notes', content='TODO', recursive=true, max_entries=50\n"
+    "- list current: action=list, path='/'\n"
+    "- make folder: action=mkdir, path='/notes'\n"
+    "- write file: action=write, path='/notes/todo.txt', content='hello'\n"
+    "- read file: action=read, path='/notes/todo.txt', max_chars=200\n"
+    "- move file: action=move, path='/notes/todo.txt', dest_path='/notes/todo-old.txt', overwrite=true\n"
+    "- delete dir: action=delete, path='/notes', recursive=true\n"
+    "- search text: action=search, path='/notes', content='TODO', recursive=true, max_entries=50\n"
     "- page search results: action=search, content='TODO', offset=10\n"
     "- case-sensitive search: action=search, content='TODO', case_sensitive=true\n"
 )
@@ -119,16 +120,24 @@ def _resolve_path(path: str) -> tuple[str, str]:
     (unless UNSAFE_ALLOW_ABS_PATHS=1). Returns (abs_path, error_message). error_message is empty when ok.
     """
     try:
-        user_input = (path or ".").strip()
-        raw = os.path.expanduser(user_input)
-        if os.path.isabs(raw):
+        user_input = (path or "/").strip() or "/"
+        if user_input.startswith("/"):
+            # Treat leading '/' as the virtual root for the tool.
+            rel_part = user_input.lstrip("/") or "."
+            raw = os.path.expanduser(rel_part)
+            treat_as_relative = True
+        else:
+            raw = os.path.expanduser(user_input)
+            treat_as_relative = False
+
+        if not treat_as_relative and os.path.isabs(raw):
             if not ALLOW_ABS:
                 # Absolute paths are not allowed in safe mode
                 return "", _err(
                     "absolute_path_disabled",
                     "Absolute paths are disabled in safe mode.",
                     path=raw.replace("\\", "/"),
-                    hint="Use a path relative to / (e.g., notes/todo.txt)."
+                    hint="Use a path relative to / (e.g., /notes/todo.txt)."
                 )
             abs_path = os.path.abspath(raw)
         else:
@@ -499,7 +508,7 @@ def _info(abs_path: str) -> str:
 @autodoc(summary=TOOL_SUMMARY)
 def File_System(
     action: Annotated[str, "Operation to perform: 'list', 'read', 'write', 'append', 'mkdir', 'move', 'copy', 'delete', 'info', 'search'."],
-    path: Annotated[str, "Target path, relative to root unless UNSAFE_ALLOW_ABS_PATHS=1."] = ".",
+    path: Annotated[str, "Target path, relative to root unless UNSAFE_ALLOW_ABS_PATHS=1."] = "/",
     content: Annotated[Optional[str], "Content for write/append actions or search query (UTF-8)."] = None,
     dest_path: Annotated[Optional[str], "Destination for move/copy (relative to root unless unsafe absolute allowed)."] = None,
     recursive: Annotated[bool, "For list/search (recurse into subfolders) and delete (required for directories)."] = False,
@@ -610,7 +619,7 @@ def build_interface() -> gr.Interface:
                 choices=["list", "read", "write", "append", "mkdir", "move", "copy", "delete", "info", "search", "help"],
                 value="help",
             ),
-            gr.Textbox(label="Path", placeholder=". or src/file.txt", max_lines=1, value="."),
+            gr.Textbox(label="Path", placeholder="/ or /src/file.txt", max_lines=1, value="/"),
             gr.Textbox(label="Content (write/append/search)", lines=6, placeholder="Text to write or search for..."),
             gr.Textbox(label="Destination (for move/copy)", max_lines=1),
             gr.Checkbox(label="Recursive (list/delete/search)", value=False),