fix: resolve all Copilot review comments from PR #25

- config/loader.go: remove misleading force_reindex mention, keep only recreate: true
- tools/index_workspace.go: fix tool name (search_code -> rag_search/rag_search_code); gate runtime info behind include_runtime_info flag
- tools/doc_find_usages.md: fix JSON filter key casing Relations[] -> relations[]
- tools/README.md: fix Determinist -> Deterministic; Relations[] casing; file_path MANDATORY -> PREFER
- tools/find_usages.go: deduplicate matchedRelations
- tools/list_package_exports.go: is_public fallback Go-only; filePath -> payloadFilePath
- engine/index_progress.go: deep-copy Languages map in disk-load branch
- logger/logger.go: Warn() writes to log file; shouldLog() defaults unknown level to info
- .goreleaser.yaml: remove deleted config.yaml from archives
- .github/workflows/build-binaries.yml: fix path ./cmd/install -> ./cmd/rag-code-install
- README.md: fix broken links to deleted docs files
- QUICKSTART.md, config/README.md, search/README.md, cmd/sse-client-test: doc fixes

Author: razvan

.github/workflows/build-binaries.yml

@@ -45,7 +45,7 @@ jobs:
           go build -o bin/index-all ./cmd/index-all
           
           echo "Building rag-code-install..."
-          go build -o bin/rag-code-install ./cmd/install
+          go build -o bin/rag-code-install ./cmd/rag-code-install
           
           chmod +x bin/*
 

.goreleaser.yaml

@@ -111,7 +111,6 @@ archives:
       - LICENSE
       - README.md
       - llms.txt
-      - config.yaml
       - .env.example
 
 checksum:

README.md

@@ -104,8 +104,8 @@ First query triggers background indexing. Subsequent queries are instant.
 | [🛠️ MCP Tools](#️-11-powerful-mcp-tools) | All 11 tools explained |
 | [🌐 Supported Languages](#-multi-language-code-intelligence) | Go, PHP, Python support |
 | [💻 IDE Integration](#-ide-integration) | Windsurf, Cursor, VS Code, Claude |
-| [⚙️ Configuration](./docs/configuration.md) | Advanced settings, models, env vars |
-| [🐛 Troubleshooting](./docs/troubleshooting.md) | Common issues and solutions |
+| [⚙️ Configuration](#-configuration) | Advanced settings, models, env vars |
+| [🐛 Troubleshooting](#-troubleshooting) | Common issues and solutions |
 | [📚 Documentation](#-documentation) | All guides and references |
 
 ---
@@ -264,9 +264,8 @@ RagCode works with all major AI-powered IDEs:
 - **[IDE Setup](./docs/IDE-SETUP.md)** - Manual IDE configuration
 
 ### Configuration & Operations
-- **[Configuration Guide](./docs/configuration.md)** - Models, env vars, advanced settings
-- **[Troubleshooting](./docs/troubleshooting.md)** - Common issues and solutions
-- **[Docker Setup](./docs/docker-setup.md)** - Docker configuration details
+- **[Configuration Guide](#-configuration)** - Models, env vars, advanced settings
+- **[Troubleshooting](#-troubleshooting)** - Common issues and solutions
 
 ### Language Analyzers
 - **[Go Analyzer](./internal/ragcode/analyzers/golang/README.md)** - Functions, types, interfaces, GoDoc

internal/logger/logger.go

@@ -50,7 +50,15 @@ func (l *SimpleLogger) shouldLog(msgLevel string) bool {
 	if logLevel == "" {
 		logLevel = "info"
 	}
-	return levels[msgLevel] >= levels[logLevel]
+	configuredLevel, ok := levels[logLevel]
+	if !ok {
+		configuredLevel = levels["info"]
+	}
+	msgLevelValue, ok := levels[msgLevel]
+	if !ok {
+		return false
+	}
+	return msgLevelValue >= configuredLevel
 }
 
 // Debug logs a debug message (only if MCP_LOG_LEVEL is set to 'debug').
@@ -87,6 +95,9 @@ func (l *SimpleLogger) Error(format string, args ...interface{}) {
 func (l *SimpleLogger) Warn(format string, args ...interface{}) {
 	if l.shouldLog("warn") {
 		fmt.Fprintf(os.Stderr, "[WARN] "+format+"\n", args...)
+		if l.writer != nil {
+			fmt.Fprintf(l.writer, "[WARN] "+format+"\n", args...)
+		}
 	}
 }
 

internal/service/engine/index_progress.go

@@ -57,6 +57,12 @@ func (s *progressStore) get(workspaceID string, workspaceRoot string) *IndexProg
 		if p := loadIndexStatus(workspaceRoot); p != nil && p.WorkspaceID == workspaceID {
 			s.jobs[workspaceID] = p // cache in memory for subsequent calls
 			cp := *p
+			if p.Languages != nil {
+				cp.Languages = make(map[string]IndexLanguageProgress, len(p.Languages))
+				for k, v := range p.Languages {
+					cp.Languages[k] = v
+				}
+			}
 			return &cp
 		}
 	}

internal/service/tools/doc_find_usages.md

@@ -19,7 +19,7 @@ It completely **bypasses the Semantic Embedder (LLM)** and issues an Exact Match
 }
 ```
 
-By querying the exact path of the payload JSON, the database operates efficiently and precisely, scanning the `Relations` arrays of every indexed AST node (both methods, classes or standalone functions) to identify callers, implementers, and dependent types referencing the `SymbolName`.
+By querying the exact path of the payload JSON, the database operates efficiently and precisely, scanning the `relations` arrays of every indexed AST node (both methods, classes or standalone functions) to identify callers, implementers, and dependent types referencing the `SymbolName`.
 
 ## Features
 - **Deterministic and Robust**: Resolves complex usage paths correctly regardless of lexical variations or generic terms used by developers. Zero semantic hallucinations!

internal/service/tools/find_usages.go

@@ -150,15 +150,19 @@ func (t *FindUsagesTool) Execute(ctx context.Context, args map[string]interface{
 			startLine = v
 		}
 
-		// Extract exactly which relation(s) matched to show reasoning
+		// Extract exactly which relation type(s) matched to show reasoning (deduplicated)
+		seenRelTypes := make(map[string]struct{})
 		var matchedRelations []string
 		if relationsRaw, hasRel := result.Point.Payload["relations"]; hasRel {
 			if relList, ok := relationsRaw.([]interface{}); ok {
 				for _, relItem := range relList {
 					if rMap, ok := relItem.(map[string]interface{}); ok {
 						if target, _ := rMap["target_name"].(string); target == symbolName {
 							relType, _ := rMap["type"].(string)
-							matchedRelations = append(matchedRelations, relType)
+							if _, seen := seenRelTypes[relType]; !seen {
+								seenRelTypes[relType] = struct{}{}
+								matchedRelations = append(matchedRelations, relType)
+							}
 						}
 					}
 				}

internal/service/tools/list_package_exports.go

@@ -135,13 +135,19 @@ func (t *ListPackageExportsTool) Execute(ctx context.Context, args map[string]in
 	actualBytes := 0
 
 	for _, result := range allResults {
-		// is_public filter: use payload field when present, fallback to naming convention
-		isPublic, _ := result.Point.Payload["is_public"].(bool)
-		if !isPublic {
-			name, _ := result.Point.Payload["name"].(string)
-			if !isExported(name) {
+		// is_public filter: prefer explicit payload field; fallback to naming convention only for Go
+		if isPublicVal, ok := result.Point.Payload["is_public"]; ok {
+			if isPublic, okBool := isPublicVal.(bool); okBool && !isPublic {
 				continue
 			}
+		} else {
+			lang, _ := result.Point.Payload["language"].(string)
+			if strings.EqualFold(lang, "go") {
+				name, _ := result.Point.Payload["name"].(string)
+				if !isExported(name) {
+					continue
+				}
+			}
 		}
 
 		name, _ := result.Point.Payload["name"].(string)
@@ -160,7 +166,7 @@ func (t *ListPackageExportsTool) Execute(ctx context.Context, args map[string]in
 		docstring, _ := result.Point.Payload["docstring"].(string)
 
 		descLine := strings.Split(docstring, "\n")[0]
-		filePath, _ := result.Point.Payload["file_path"].(string)
+		payloadFilePath, _ := result.Point.Payload["file_path"].(string)
 		startLineVal := result.Point.Payload["start_line"]
 		startLine := 0
 		switch v := startLineVal.(type) {
@@ -173,9 +179,9 @@ func (t *ListPackageExportsTool) Execute(ctx context.Context, args map[string]in
 		relsRaw, _ := result.Point.Payload["relations"].([]interface{})
 
 		actualBytes += len(name) + len(signature) + len(descLine)
-		if filePath != "" && !seenFiles[filePath] {
-			seenFiles[filePath] = true
-			if info, statErr := os.Stat(filePath); statErr == nil {
+		if payloadFilePath != "" && !seenFiles[payloadFilePath] {
+			seenFiles[payloadFilePath] = true
+			if info, statErr := os.Stat(payloadFilePath); statErr == nil {
 				baselineBytes += int(info.Size())
 			}
 		}
@@ -185,7 +191,7 @@ func (t *ListPackageExportsTool) Execute(ctx context.Context, args map[string]in
 			Type:           symType,
 			Signature:      signature,
 			Description:    descLine,
-			FilePath:       filePath,
+			FilePath:       payloadFilePath,
 			StartLine:      startLine,
 			Package:        packageName,
 			RelationsCount: len(relsRaw),