v0.3.54: fix read_file line ranges, tool-name aliases, always force a final reply

- read_file now honors line_start/line_end (and offset/limit) instead of always
  returning the file head, so the model can navigate large files instead of
  re-reading the same chunk until it runs out of rounds. Params are advertised
  in both the builtin and coding tool defs.
- Map hallucinated tool names (search→grep_search, print_tree/tree→list_directory,
  find/glob→glob_search, cat→read_file) onto the real tools so the call works.
- Fixed the coding prompt to name the real tools (glob_search/grep_search).
- The cloud tool loop now always does a final tool-less completion after running
  out of rounds, even if some interim text leaked, so the turn never ends empty.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

Author: metiu1

vortelio-pip/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "vortelio"
-version = "0.3.53"
+version = "0.3.54"
 description = "Local-first AI platform. Run LLMs, generate images & video, transcribe audio, create 3D — on your own machine. OpenAI & Ollama API compatible. Apache 2.0."
 readme = "README.md"
 requires-python = ">=3.8"

vortelio-pip/src/vortelio_cli/__init__.py

@@ -1 +1 @@
-__version__ = "0.3.53"
+__version__ = "0.3.54"

vortelio-pip/src/vortelio_cli/__main__.py

@@ -10,7 +10,7 @@
 import urllib.request
 from pathlib import Path
 
-VERSION = "0.3.53"
+VERSION = "0.3.54"
 RELEASE_BASE = os.environ.get(
     "VORTELIO_RELEASE_BASE",
     f"https://github.com/metiu1/Vortelio/releases/download/v{VERSION}",

vortelio-pip/src/vortelio_cli/bin/vortelio-windows-amd64.exe

@@ -638,10 +638,17 @@ func chatOpenAIWithTools(p Provider, apiKey string, messages []Message, opts *To
 		}
 	}
 
-	// Rounds exhausted while the model was still calling tools and never wrote a
-	// textual answer → the user would otherwise see silence. Force one final
-	// completion with tools disabled so there is always a reply.
-	if finalContent.Len() == 0 {
+	// Reaching here means every round ended in tool calls and we ran out of
+	// rounds without a concluding answer (a natural finish returns inside the
+	// loop). Force one final completion with tools disabled so the user always
+	// gets a real reply instead of silence — even if a little interim text leaked.
+	{
+		if finalContent.Len() > 0 {
+			finalContent.WriteString("\n\n")
+			if onToken != nil {
+				onToken("\n\n")
+			}
+		}
 		body := map[string]interface{}{"model": p.DefaultModel, "messages": msgs, "stream": true}
 		data, _ := json.Marshal(body)
 		if req, err := http.NewRequest("POST", p.BaseURL, bytes.NewReader(data)); err == nil {

vortelio/internal/cloud/cloud.go

@@ -97,7 +97,31 @@ func (c *CompositeProvider) Tools() []ToolDef {
 	return all
 }
 
+// toolAliases maps common names models hallucinate (used to other agents) onto
+// the real tool names, so the call succeeds instead of wasting a round.
+var toolAliases = map[string]string{
+	"search":     "grep_search",
+	"grep":       "grep_search",
+	"find":       "glob_search",
+	"glob":       "glob_search",
+	"tree":       "list_directory",
+	"print_tree": "list_directory",
+	"ls":         "list_directory",
+	"cat":        "read_file",
+	"read":       "read_file",
+}
+
 func (c *CompositeProvider) Execute(name, args string) (string, error) {
+	if alias, ok := toolAliases[name]; ok {
+		// Only remap if a provider actually offers the canonical tool.
+		for _, p := range c.providers {
+			for _, t := range p.Tools() {
+				if t.Function.Name == alias {
+					name = alias
+				}
+			}
+		}
+	}
 	for _, p := range c.providers {
 		for _, t := range p.Tools() {
 			if t.Function.Name == name {
@@ -150,8 +174,8 @@ func BuiltinTools() []ToolDef {
 			Type: "function",
 			Function: ToolFuncDef{
 				Name:        "read_file",
-				Description: "Reads the contents of a file from the local filesystem.",
-				Parameters:  json.RawMessage(`{"type":"object","properties":{"path":{"type":"string","description":"Absolute or relative path to the file"}},"required":["path"]}`),
+				Description: "Reads a text file. Optionally read only a line range with line_start/line_end (1-based, inclusive) to navigate large files.",
+				Parameters:  json.RawMessage(`{"type":"object","properties":{"path":{"type":"string","description":"Absolute or relative path to the file"},"line_start":{"type":"integer","description":"First line to return (1-based, inclusive). Optional."},"line_end":{"type":"integer","description":"Last line to return (1-based, inclusive). Optional."}},"required":["path"]}`),
 			},
 		},
 		{
@@ -620,6 +644,12 @@ func evalFunc(name string, args []float64) (float64, error) {
 func toolReadFile(argsJSON string) (string, error) {
 	var args struct {
 		Path string `json:"path"`
+		// Accept several common aliases models use for line ranges so navigation
+		// works instead of silently re-reading the file head.
+		LineStart *int `json:"line_start"`
+		LineEnd   *int `json:"line_end"`
+		Offset    *int `json:"offset"`
+		Limit     *int `json:"limit"`
 	}
 	if err := json.Unmarshal([]byte(argsJSON), &args); err != nil || args.Path == "" {
 		return "", fmt.Errorf("path is required")
@@ -629,16 +659,46 @@ func toolReadFile(argsJSON string) (string, error) {
 		return "", fmt.Errorf("cannot read file %q: %w", args.Path, err)
 	}
 	content := string(data)
+	lines := strings.Split(content, "\n")
+	total := len(lines)
+
+	// Resolve a 1-based inclusive [start,end] window from whichever params were
+	// given (line_start/line_end, or offset+limit).
+	start, end := 1, total
+	if args.LineStart != nil {
+		start = *args.LineStart
+	} else if args.Offset != nil {
+		start = *args.Offset + 1
+	}
+	if args.LineEnd != nil {
+		end = *args.LineEnd
+	} else if args.Limit != nil {
+		end = start + *args.Limit - 1
+	}
+	ranged := args.LineStart != nil || args.LineEnd != nil || args.Offset != nil || args.Limit != nil
+	if start < 1 {
+		start = 1
+	}
+	if end > total {
+		end = total
+	}
+	if ranged && start <= end {
+		content = strings.Join(lines[start-1:end], "\n")
+	}
+
 	const maxLen = 32000
 	truncated := false
 	if len(content) > maxLen {
 		content = content[:maxLen]
 		truncated = true
 	}
-	out := map[string]interface{}{"path": args.Path, "content": content}
+	out := map[string]interface{}{"path": args.Path, "content": content, "total_lines": total}
+	if ranged {
+		out["line_start"], out["line_end"] = start, end
+	}
 	if truncated {
 		out["truncated"] = true
-		out["note"] = fmt.Sprintf("file truncated to %d chars", maxLen)
+		out["note"] = fmt.Sprintf("content truncated to %d chars; request a smaller line range", maxLen)
 	}
 	b, _ := json.Marshal(out)
 	return string(b), nil

vortelio/internal/runtime/tools.go

@@ -198,7 +198,7 @@ func CodingSystemPrompt(autonomous bool) string {
 		"Sei orientato al progetto corrente: le richieste dell'utente riguardano quasi sempre i file e il codice " +
 		"di QUESTA cartella, non attività generiche. " +
 		"Prima di rispondere su \"il progetto\", \"questo\", \"qui\" o un file citato, USA gli strumenti " +
-		"(list_directory, read_file, glob, grep) per guardare i file reali: non indovinare e non inventare contenuti. " +
+		"(list_directory, read_file, glob_search, grep_search) per guardare i file reali: non indovinare e non inventare contenuti. " +
 		"Per modificare il progetto usa write_file / edit_file con percorsi relativi alla cartella di lavoro e " +
 		"riferisci sempre il percorso esatto. Non affermare di aver creato o cambiato un file se non hai chiamato lo strumento. " +
 		"Hai anche strumenti web (web_search) e di generazione media (immagini/audio/video/3D): usali solo quando " +
@@ -583,8 +583,8 @@ func newCodingProvider(cfg *AgenticConfig, emit rt.ToolEventEmitter) *codingProv
 
 func (c *codingProvider) Tools() []rt.ToolDef {
 	defs := []rt.ToolDef{
-		toolDef("read_file", "Read a UTF-8 text file from the workspace.",
-			`{"type":"object","properties":{"path":{"type":"string","description":"File path, relative to the workspace root or absolute."}},"required":["path"]}`),
+		toolDef("read_file", "Read a UTF-8 text file from the workspace. Use line_start/line_end (1-based, inclusive) to read only part of a large file.",
+			`{"type":"object","properties":{"path":{"type":"string","description":"File path, relative to the workspace root or absolute."},"line_start":{"type":"integer","description":"First line (1-based, inclusive). Optional."},"line_end":{"type":"integer","description":"Last line (1-based, inclusive). Optional."}},"required":["path"]}`),
 		toolDef("list_directory", "List files and folders at a path in the workspace.",
 			`{"type":"object","properties":{"path":{"type":"string","description":"Directory path. Defaults to workspace root."}},"required":[]}`),
 		toolDef("glob_search", "Find files matching a glob pattern (e.g. **/*.go).",
@@ -720,7 +720,9 @@ func (c *codingProvider) resolvePath(p string) (string, error) {
 
 func (c *codingProvider) readFile(argsJSON string) (string, error) {
 	var a struct {
-		Path string `json:"path"`
+		Path      string `json:"path"`
+		LineStart *int   `json:"line_start"`
+		LineEnd   *int   `json:"line_end"`
 	}
 	json.Unmarshal([]byte(argsJSON), &a)
 	full, err := c.resolvePath(a.Path)
@@ -734,6 +736,27 @@ func (c *codingProvider) readFile(argsJSON string) (string, error) {
 	if len(data) > 200*1024 {
 		data = data[:200*1024]
 	}
+	// Honor an optional 1-based inclusive line range so the model can navigate
+	// large files instead of re-reading the head.
+	if a.LineStart != nil || a.LineEnd != nil {
+		lines := strings.Split(string(data), "\n")
+		start, end := 1, len(lines)
+		if a.LineStart != nil {
+			start = *a.LineStart
+		}
+		if a.LineEnd != nil {
+			end = *a.LineEnd
+		}
+		if start < 1 {
+			start = 1
+		}
+		if end > len(lines) {
+			end = len(lines)
+		}
+		if start <= end {
+			return strings.Join(lines[start-1:end], "\n"), nil
+		}
+	}
 	return string(data), nil
 }
 

vortelio/internal/server/server_agentic.go

@@ -3,4 +3,4 @@ package version
 // Version è impostabile a build time via:
 //
 //	-ldflags "-X github.com/vortelio/vortelio/internal/version.Version=X.Y.Z"
-var Version = "0.3.53"
+var Version = "0.3.54"