use mcp.DefaultString, revert description, update unit test

LuluBeatson · LuluBeatson · commit 3bf01124924d · 2025-07-11T17:13:35.000+01:00
diff --git a/README.md b/README.md
@@ -899,7 +899,7 @@ export GITHUB_MCP_TOOL_ADD_ISSUE_COMMENT_DESCRIPTION="an alternative description
 
 - **get_file_contents** - Get file or directory contents
   - `owner`: Repository owner (username or organization) (string, required)
-  - `path`: Path to file/directory. Directories, including root, must end with '/' (string, required)
+  - `path`: Path to file/directory (directories must end with a slash '/') (string, optional)
   - `ref`: Accepts optional git refs such as `refs/tags/{tag}`, `refs/heads/{branch}` or `refs/pull/{pr_number}/head` (string, optional)
   - `repo`: Repository name (string, required)
   - `sha`: Accepts optional commit SHA. If specified, it will be used instead of ref (string, optional)
diff --git a/pkg/github/__toolsnaps__/get_file_contents.snap b/pkg/github/__toolsnaps__/get_file_contents.snap
@@ -11,7 +11,8 @@
         "type": "string"
       },
       "path": {
-        "description": "Path to file/directory. Directories, including root, must end with '/'",
+        "default": "/",
+        "description": "Path to file/directory (directories must end with a slash '/')",
         "type": "string"
       },
       "ref": {
@@ -29,8 +30,7 @@
     },
     "required": [
       "owner",
-      "repo",
-      "path"
+      "repo"
     ],
     "type": "object"
   },
diff --git a/pkg/github/repositories.go b/pkg/github/repositories.go
@@ -463,8 +463,8 @@ func GetFileContents(getClient GetClientFn, getRawClient raw.GetRawClientFn, t t
 				mcp.Description("Repository name"),
 			),
 			mcp.WithString("path",
-				mcp.Required(),
-				mcp.Description("Path to file/directory. Directories, including root, must end with '/'"),
+				mcp.Description("Path to file/directory (directories must end with a slash '/')"),
+				mcp.DefaultString("/"),
 			),
 			mcp.WithString("ref",
 				mcp.Description("Accepts optional git refs such as `refs/tags/{tag}`, `refs/heads/{branch}` or `refs/pull/{pr_number}/head`"),
diff --git a/pkg/github/repositories_test.go b/pkg/github/repositories_test.go
@@ -33,7 +33,7 @@ func Test_GetFileContents(t *testing.T) {
 	assert.Contains(t, tool.InputSchema.Properties, "path")
 	assert.Contains(t, tool.InputSchema.Properties, "ref")
 	assert.Contains(t, tool.InputSchema.Properties, "sha")
-	assert.ElementsMatch(t, tool.InputSchema.Required, []string{"owner", "repo", "path"})
+	assert.ElementsMatch(t, tool.InputSchema.Required, []string{"owner", "repo"})
 
 	// Mock response for raw content
 	mockRawContent := []byte("# Test Repository\n\nThis is a test repository.")
