✨Please check .gitignore file existing or not in work_dir and only return ToolError Unsafe pattern if can not finding the .gitignore file.

Author: Eric-Guo

src/kimi_cli/tools/file/glob.py

@@ -45,11 +45,16 @@ def __init__(self, builtin_args: BuiltinSystemPromptArgs) -> None:
     async def _validate_pattern(self, pattern: str) -> ToolError | None:
         """Validate that the pattern is safe to use."""
         if pattern.startswith("**"):
+            gitignore_path = self._work_dir / ".gitignore"
+            if await gitignore_path.is_file():
+                return None
+
             ls_result = await list_directory(self._work_dir)
             return ToolError(
                 output=ls_result,
                 message=(
-                    f"Pattern `{pattern}` starts with '**' which is not allowed. "
+                    f"Pattern `{pattern}` starts with '**' which is not allowed because "
+                    "the working directory does not contain a .gitignore to constrain the search. "
                     "This would recursively search all directories and may include large "
                     "directories like `node_modules`. Use more specific patterns instead. "
                     "For your convenience, a list of all files and directories in the "

tests/test_glob.py

@@ -67,6 +67,20 @@ async def test_glob_recursive_pattern_prohibited(glob_tool: Glob, test_files: Ka
     assert "Unsafe pattern" in result.brief
 
 
+@pytest.mark.asyncio
+async def test_glob_recursive_pattern_allowed_with_gitignore(glob_tool: Glob, test_files: KaosPath):
+    """Allow recursive ** pattern when a .gitignore exists in work dir."""
+    await (test_files / ".gitignore").write_text("node_modules/\n")
+
+    result = await glob_tool(Params(pattern="**/*.py", directory=str(test_files)))
+
+    assert isinstance(result, ToolOk)
+    assert isinstance(result.output, str)
+    output = result.output.replace("\\", "/")
+    assert "setup.py" in output
+    assert "src/main/app.py" in output
+
+
 @pytest.mark.asyncio
 async def test_glob_safe_recursive_pattern(glob_tool: Glob, test_files: KaosPath):
     """Test safe recursive glob pattern that doesn't start with **/."""