chore: [ML-1392]:Fixed Download Log Files from MCP are inaccessible to Claude Code (#212)

* 3678fd chore: [ML-1392-new]: resolved comments

* 253fa3 chore: [ML-1392-new]: resolved comments

* a86f92 Apply suggestion from code review

* edb34c Apply suggestion from code review

* d72698 Apply suggestion from code review

* ac9e0c chore: [ML-1392]: resolved comments

* 9279ab chore: [ML-1392]: minor fixes

* 066d83 chore: [ML-1392]:Fixed  Download Log Files from MCP are inaccessible to Claude Code

Author: Saranya-jena

README.md

@@ -514,6 +514,104 @@ To use the Harness MCP Server with Amazon Q Developer CLI:
 
 [VS Code MCP Guide](https://code.visualstudio.com/docs/copilot/chat/mcp-servers)
 
+## Tool Usage Guide
+
+### Download Execution Logs
+
+#### Using Docker:
+
+We need to mount the logs directory to the container to download the logs.
+
+```bash
+docker run -d --name mcp-server -p 8080:8080 -v /path/to/logs/in/host:/path/in/container harness/mcp-server --output-dir=/path/in/container
+```
+This ensures that the logs downloaded to container are accessible in the host.
+
+Example:
+
+```bash
+docker run -d --name mcp-server -p 8080:8080 -v /Users/testuser/logs:/logs harness/mcp-server --output-dir=/logs
+```
+
+Sample MCP Configuration:
+
+```bash
+{
+  "mcpServers": {
+    "harness": {
+      "command": "docker",
+      "args": [
+        "run",
+        "-i",
+        "--rm",
+        "-v",
+        "/Users/testuser/logs:/logs",
+        "-e",
+        "HARNESS_MCP_USER_PAT",
+        "-e",
+        "HARNESS_DEFAULT_ORG_ID",
+        "-e",
+        "HARNESS_DEFAULT_PROJECT_ID",
+        "-e",
+        "HARNESS_MCP_BASE_URL",
+        "harness/mcp-server",
+        "stdio",
+        "--output-dir=/logs", #/path/in/container
+        "--toolsets=logs",
+        "--api-key="
+      ],
+      "env": {
+        "HARNESS_MCP_USER_PAT": "<YOUR_API_KEY>",
+        "HARNESS_DEFAULT_ORG_ID": "<YOUR_ORG_ID>",
+        "HARNESS_DEFAULT_PROJECT_ID": "<YOUR_PROJECT_ID>",
+        "HARNESS_MCP_BASE_URL": "<YOUR_BASE_URL>"
+      }
+    }
+  }
+}
+```
+
+Example Tool Input:
+
+```json
+{
+  "logs_directory": "pipeline-logs",
+  "org_id": "<YOUR_ORG_ID>",
+  "plan_execution_id": "<YOUR_PLAN_EXECUTION_ID>",
+  "project_id": "<YOUR_PROJECT_ID>"
+}
+```
+
+Sample Response:
+
+```json
+Files downloaded to : /Users/testuser/logs/pipeline-logs/logs-<YOUR_PLAN_EXECUTION_ID>/logs.zip
+```
+
+### Using Local Binary:
+
+Example configuration:
+
+```json
+"args": ["stdio", "--toolsets=logs", "--api-key=", "--output-dir=/Users/testuser/log-files"]
+```
+
+Example Tool Input:
+
+```json
+{
+  "logs_directory": "logs1",
+  "project_id": "<YOUR_PROJECT_ID>",
+  "plan_execution_id": "<YOUR_PLAN_EXECUTION_ID>",
+  "org_id": "<YOUR_ORG_ID>"
+}
+```
+
+Sample Response:
+
+```json
+Files downloaded to : /Users/testuser/log-files/logs1/logs-<YOUR_PLAN_EXECUTION_ID>/logs.zip
+```
 
 ## Development
 
@@ -530,6 +628,7 @@ The Harness MCP Server supports the following command line arguments:
 - `--version`: Show version information
 - `--help`: Show help message
 - `--base-url`: Base URL for Harness (default: "https://app.harness.io")
+- `--output-dir`: Directory where the tool writes output files (e.g., pipeline logs)
 
 
 ### Environment Variables

cmd/harness-mcp-server/config/config.go

@@ -11,6 +11,7 @@ type Config struct {
 	LogFilePath   string
 	Debug         bool
 	EnableLicense bool
+	OutputDir     string
 
 	// Server configuration
 	Transport enum.TransportType

cmd/harness-mcp-server/main.go

@@ -82,6 +82,7 @@ var (
 				},
 				Toolsets:      toolsets,
 				EnableModules: enableModules,
+				OutputDir:     viper.GetString("output_dir"),
 			}
 
 			return runHTTPServer(ctx, cfg)
@@ -132,6 +133,7 @@ var (
 				Debug:            viper.GetBool("debug"),
 				EnableModules:    enableModules,
 				EnableLicense:    viper.GetBool("enable_license"),
+				OutputDir:        viper.GetString("output_dir"),
 			}
 
 			if err := runStdioServer(ctx, cfg); err != nil {
@@ -192,6 +194,7 @@ var (
 				Debug:         viper.GetBool("debug"),
 				EnableLicense: viper.GetBool("enable_license"),
 				Internal:      true,
+				OutputDir:     viper.GetString("output_dir"),
 				AccountID:     session.Principal.AccountID,
 				// Internal mode specific fields
 				BearerToken:             viper.GetString("bearer_token"),
@@ -257,6 +260,7 @@ func init() {
 	rootCmd.PersistentFlags().Bool("read-only", false, "Restrict the server to read-only operations")
 	rootCmd.PersistentFlags().String("log-file", "", "Path to log file")
 	rootCmd.PersistentFlags().Bool("debug", false, "Enable debug logging")
+	rootCmd.PersistentFlags().String("output-dir", "", "Directory where the tool writes output files (e.g., pipeline logs)")
 
 	httpServerCmd.PersistentFlags().Int("http-port", 8080, "HTTP server port (when transport is 'http')")
 	httpServerCmd.PersistentFlags().String("http-path", "/mcp", "HTTP server path (when transport is 'http')")
@@ -312,6 +316,7 @@ func init() {
 	_ = viper.BindPFlag("read_only", rootCmd.PersistentFlags().Lookup("read-only"))
 	_ = viper.BindPFlag("log_file", rootCmd.PersistentFlags().Lookup("log-file"))
 	_ = viper.BindPFlag("debug", rootCmd.PersistentFlags().Lookup("debug"))
+	_ = viper.BindPFlag("output_dir", rootCmd.PersistentFlags().Lookup("output-dir"))
 
 	// Bind transport configuration flags to viper
 	_ = viper.BindPFlag("http_port", httpServerCmd.PersistentFlags().Lookup("http-port"))

pkg/harness/tools/logs.go

@@ -4,9 +4,11 @@ import (
 	"context"
 	"fmt"
 	"io"
+	"log/slog"
 	"net/http"
 	"os"
 	"path/filepath"
+	"strings"
 
 	"github.com/harness/harness-mcp/client"
 	"github.com/harness/harness-mcp/cmd/harness-mcp-server/config"
@@ -26,7 +28,7 @@ func DownloadExecutionLogsTool(config *config.Config, client *client.LogService)
 			),
 			mcp.WithString("logs_directory",
 				mcp.Required(),
-				mcp.Description("The absolute path to the directory where the logs should get downloaded"),
+				mcp.Description("The absolute path to the directory where the logs should get downloaded."),
 			),
 			mcp.WithString("log_key",
 				mcp.Description("Optional log key to be used for downloading logs directly"),
@@ -57,6 +59,22 @@ func DownloadExecutionLogsTool(config *config.Config, client *client.LogService)
 				return mcp.NewToolResultError(err.Error()), nil
 			}
 
+			// If OutputDir is configured, use it as the base directory for logs
+			if config.OutputDir != "" {
+
+				if !strings.HasPrefix(logsDirectory, config.OutputDir) {
+					// Create a subdirectory within the output directory for logs
+					oldLogsDirectory := logsDirectory
+					logsDirectoryName := filepath.Base(logsDirectory)
+					// If the logs directory is just a root path like /tmp, use a more descriptive name
+					if logsDirectoryName == "/" || logsDirectoryName == "" {
+						logsDirectoryName = "pipeline-logs"
+					}
+					logsDirectory = filepath.Join(config.OutputDir, logsDirectoryName)
+					slog.Info("Redirecting logs from %s to %s to ensure access", oldLogsDirectory, logsDirectory)
+				}
+			}
+
 			// Create the logs folder path (creates all parent directories if needed)
 			logsFolderName := fmt.Sprintf("logs-%s", planExecutionID)
 			logsFolderPath := filepath.Join(logsDirectory, logsFolderName)