Markdown data retrieval returns error text instead of failure

Author: radare

.gitignore

@@ -5,3 +5,4 @@ src/mcps/wttr/mai-mcp-wttr
 src/mcps/pipe/mai-mcp-pipe
 src/mcps/gemcode/mai-mcp-gemcode
 src/mcps/code/mai-mcp-code
+src/mcps/markdown/mai-mcp-markdown

prompts/tool.md

@@ -54,7 +54,7 @@ When it is required to call a tool, respond only in JSON without any explanation
   "next_step": "What should happen next.",
   "action": "Solve | Think | Iterate | Error",
   "tool_required": true,
-  "tool": "tool_provider/tool_name",
+  "tool": "tool_name",
   "tool_params": {
     "param1": "value1",
     "param2": "value2"

src/mcps/markdown/README.md

@@ -35,6 +35,7 @@ Below is a detailed list of tools designed to facilitate interaction with markdo
 5. **search(query or queries)**:
    - **Function**: Locates and returns sections corresponding to a search query or a list of search queries.
    - **Use Case**: Effective for finding multiple terms or keywords without scanning the entire document.
+   - **Notes**: Search is case-insensitive and supports multi-word phrases as a single query (e.g., `{"query": "multi-word phrase"}`).
 
 6. **summarize(section)**: 
    - **Function**: Generates a concise summary of a specified section.

src/mcps/markdown/mdown.go

@@ -7,6 +7,7 @@ import (
 	"os"
 	"regexp"
 	"strings"
+	"unicode"
 
 	"mcplib"
 )
@@ -119,7 +120,7 @@ func (s *MDownService) GetTools() []mcplib.Tool {
 		},
 		{
 			Name:        "search",
-			Description: "Locates and returns sections that correspond to a search query or list of search queries.",
+			Description: "Locates and returns sections that correspond to a search query or list of search queries. If there are no results try with less words",
 			InputSchema: map[string]any{
 				"type": "object",
 				"properties": map[string]any{
@@ -156,16 +157,54 @@ func (s *MDownService) handleListSections(args map[string]any) (any, error) {
 	return map[string]any{"sections": out}, nil
 }
 
+// normalizeSectionName trims surrounding whitespace and punctuation and lowercases the name
+func normalizeSectionName(s string) string {
+	s = strings.TrimSpace(s)
+	s = strings.TrimFunc(s, func(r rune) bool {
+		return unicode.IsSpace(r) || unicode.IsPunct(r)
+	})
+	return strings.ToLower(s)
+}
+
+// handleGetContents retrieves the full content of a given section,
+// matching names case-insensitively and ignoring surrounding punctuation.
 func (s *MDownService) handleGetContents(args map[string]any) (any, error) {
 	name, ok := args["section"].(string)
 	if !ok || name == "" {
 		return nil, fmt.Errorf("section is required")
 	}
 	content, ok := s.sectionContent[name]
 	if !ok {
+		norm := normalizeSectionName(name)
+		// try normalized exact match
+		for secName, secContent := range s.sectionContent {
+			if normalizeSectionName(secName) == norm {
+				content = secContent
+				ok = true
+				break
+			}
+		}
+	}
+	if !ok {
+		norm := normalizeSectionName(name)
+		// try normalized substring match
+		for secName, secContent := range s.sectionContent {
+			if strings.Contains(normalizeSectionName(secName), norm) {
+				content = secContent
+				ok = true
+				break
+			}
+		}
+	}
+	if !ok {
+		return "Section not found", nil
 		return nil, fmt.Errorf("section %q not found", name)
 	}
-	return map[string]any{"content": content}, nil
+	res := map[string]any{"content": content}
+	if res != nil {
+		return res, nil
+	}
+	return "", fmt.Errorf("Cannot find contents")
 }
 
 func (s *MDownService) handleShowContents(args map[string]any) (any, error) {
@@ -174,8 +213,10 @@ func (s *MDownService) handleShowContents(args map[string]any) (any, error) {
 		return nil, err
 	}
 	content := res.(map[string]any)["content"].(string)
-	fmt.Println(content)
-	return nil, nil
+	if content != "" {
+		return content, nil
+	}
+	return nil, fmt.Errorf("Cannot find contents")
 }
 
 func (s *MDownService) handleShowTree(args map[string]any) (any, error) {
@@ -204,8 +245,12 @@ func (s *MDownService) handleSearch(args map[string]any) (any, error) {
 	}
 	var matches []string
 	for name, content := range s.sectionContent {
+		// Perform case-insensitive matching for search terms, supporting multi-word phrases.
+		lowerName := strings.ToLower(name)
+		lowerContent := strings.ToLower(content)
 		for _, term := range terms {
-			if strings.Contains(name, term) || strings.Contains(content, term) {
+			lowerTerm := strings.ToLower(term)
+			if strings.Contains(lowerName, lowerTerm) || strings.Contains(lowerContent, lowerTerm) {
 				matches = append(matches, name)
 				break
 			}

src/repl/conf.go

@@ -56,7 +56,7 @@ func NewConfigOptions() *ConfigOptions {
 	co.RegisterOption("stream", BooleanOption, "Enable streaming mode", "true")
 	co.RegisterOption("include_replies", BooleanOption, "Include assistant replies in context", "true")
 	co.RegisterOption("logging", BooleanOption, "Enable conversation logging", "true")
-	co.RegisterOption("reasoning", BooleanOption, "Enable AI reasoning", "true")
+	co.RegisterOption("reasoning", BooleanOption, "Enable AI reasoning", "false")
 	co.RegisterOption("markdown", BooleanOption, "Enable markdown rendering with colors", "false")
 	co.RegisterOption("max_tokens", NumberOption, "Maximum tokens for AI response", "5128")
 	co.RegisterOption("temperature", NumberOption, "Temperature for AI response (0.0-1.0)", "0.7")