ssh-licco 2.5.0__tar.gz → 2.5.2__tar.gz

{ssh_licco-2.5.0/ssh_licco.egg-info → ssh_licco-2.5.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: ssh-licco
-Version: 2.5.0
+Version: 2.5.2
 Summary: SSH Model Context Protocol Server - Enable SSH functionality for AI models
 Author-email: SSH LICCO Team <contact@ssh-licco.dev>
 License: MIT
@@ -622,7 +622,7 @@ ssh-licco 的核心依赖是：
 
 | 指标 | 状态 |
 |------|------|
-| **测试用例** | 418 passed, 0 skipped |
+| **测试用例** | 424 passed, 0 skipped |
 | **覆盖率** | 16 个源模块全覆盖 |
 | **测试框架** | pytest + pytest-asyncio |
 

{ssh_licco-2.5.0 → ssh_licco-2.5.2}/README.md

@@ -589,7 +589,7 @@ ssh-licco 的核心依赖是：
 
 | 指标 | 状态 |
 |------|------|
-| **测试用例** | 418 passed, 0 skipped |
+| **测试用例** | 424 passed, 0 skipped |
 | **覆盖率** | 16 个源模块全覆盖 |
 | **测试框架** | pytest + pytest-asyncio |
 

{ssh_licco-2.5.0 → ssh_licco-2.5.2}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "ssh-licco"
-version = "2.5.0"
+version = "2.5.2"
 description = "SSH Model Context Protocol Server - Enable SSH functionality for AI models"
 readme = "README.md"
 requires-python = ">=3.10"

{ssh_licco-2.5.0 → ssh_licco-2.5.2/ssh_licco.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: ssh-licco
-Version: 2.5.0
+Version: 2.5.2
 Summary: SSH Model Context Protocol Server - Enable SSH functionality for AI models
 Author-email: SSH LICCO Team <contact@ssh-licco.dev>
 License: MIT
@@ -622,7 +622,7 @@ ssh-licco 的核心依赖是：
 
 | 指标 | 状态 |
 |------|------|
-| **测试用例** | 418 passed, 0 skipped |
+| **测试用例** | 424 passed, 0 skipped |
 | **覆盖率** | 16 个源模块全覆盖 |
 | **测试框架** | pytest + pytest-asyncio |
 

{ssh_licco-2.5.0 → ssh_licco-2.5.2}/ssh_mcp/__init__.py

@@ -1,8 +1,8 @@
 # SSH-LICCO 版本信息
 # 所有代码中的版本信息都从这里读取
-# 版本号：2.5.0
+# 版本号：2.5.2
 
-__version__ = "2.5.0"
+__version__ = "2.5.2"
 __author__ = "Li Qi"
 __email__ = "1985694657@qq.com"
 __license__ = "MIT"

{ssh_licco-2.5.0 → ssh_licco-2.5.2}/ssh_mcp/security.py

@@ -859,15 +859,17 @@ class CommandValidator:
 # -----------------------------------------------------------------------------
 
 def parse_deletion_command(command: str) -> tuple[bool, list[str]]:
-    """解析命令是否为需要备份确认的递归删除操作，并返回目标路径列表。
+    """解析命令是否为需要备份确认的删除操作，并返回目标路径列表。
 
     支持的形式（含 sudo 前缀）：
+      - rm <path>
+      - rm -f <path>
       - rm -rf <path>
       - rm -r <path>
-      - rm -fr <path>
+      - rmdir <path>
       - rm --recursive --force <path>
 
-    仅当检测到递归删除（含 -r/--recursive）且有明确目标时返回 True。
+    只要有明确删除目标（排除 --help/--version 等无目标用法）就返回 True。
     绝对路径的 rm -rf 会在硬拦截阶段被阻止，因此此处主要针对相对路径或
     已通过安全策略显式允许的删除操作。
 
@@ -875,7 +877,7 @@ def parse_deletion_command(command: str) -> tuple[bool, list[str]]:
         command: 原始 shell 命令
 
     Returns:
-        tuple[bool, list[str]]: (是否为递归删除, 目标路径列表)
+        tuple[bool, list[str]]: (是否为删除操作, 目标路径列表)
     """
     try:
         parts = shlex.split(command)
@@ -892,27 +894,25 @@ def parse_deletion_command(command: str) -> tuple[bool, list[str]]:
         return False, []
 
     base = parts[0]
-    flags: set[str] = set()
     targets: list[str] = []
 
     for part in parts[1:]:
+        # 忽略帮助/版本信息类选项（无实际删除目标）
+        if part in ("--help", "--version", "-V"):
+            return False, []
+        # 普通短选项（非 -- 长选项）不收集为 target
         if part.startswith("-") and len(part) > 1 and not part.startswith("--"):
-            flags.update(part[1:])
-        elif part == "--recursive":
-            flags.add("r")
-        elif part == "--force":
-            flags.add("f")
-        elif part == "--dir":
-            flags.add("d")
-        elif not part.startswith("--"):
-            targets.append(part)
-
-    # rmdir 只删除空目录，风险较低，不强制要求备份
-    if base == "rmdir":
-        return False, []
-
-    # 仅对递归删除要求备份确认
-    if "r" in flags and targets:
+            continue
+        # 显式终止选项 `--` 本身不是 target
+        if part == "--":
+            continue
+        # 长选项（如 --recursive）不是 target
+        if part.startswith("--"):
+            continue
+        targets.append(part)
+
+    # 有明确删除目标才触发备份确认
+    if targets:
         return True, targets
 
     return False, []
@@ -949,7 +949,9 @@ def generate_backup_command(targets: list[str]) -> str:
 def format_backup_prompt(command: str, targets: list[str]) -> str:
     """生成备份确认提示文案。"""
     targets_str = ", ".join(targets)
-    return f"""⚠️ 检测到递归删除操作
+    # rmdir 删除的是目录但非递归；rm -r/rf 属于递归删除。
+    operation_type = "目录删除操作" if command.lstrip().startswith("rmdir") else "递归删除操作"
+    return f"""⚠️ 检测到{operation_type}
 
 📋 将要执行的命令：
    {command}

{ssh_licco-2.5.0 → ssh_licco-2.5.2}/tests/test_security.py

@@ -219,9 +219,13 @@ def test_hard_block_patterns_cover_documented_categories() -> None:
     ("rm --recursive --force ./data", ["./data"]),
     ("sudo rm -rf /home/user/tmp", ["/home/user/tmp"]),
     ("rm -rf -- ./data", ["./data"]),
+    ("rm /tmp/single.log", ["/tmp/single.log"]),
+    ("rm -f /tmp/single.log", ["/tmp/single.log"]),
+    ("rmdir /tmp/empty_dir", ["/tmp/empty_dir"]),
+    ("sudo rmdir /tmp/empty_dir", ["/tmp/empty_dir"]),
 ])
-def test_parse_deletion_command_detects_recursive_rm(cmd: str, expected_targets: list[str]) -> None:
-    """递归 rm 命令应被识别为需要备份确认的删除操作。"""
+def test_parse_deletion_command_detects_deletion(cmd: str, expected_targets: list[str]) -> None:
+    """普通 rm / rmdir / 递归 rm 都应被识别为需要备份确认的删除操作。"""
     from ssh_mcp.security import parse_deletion_command
     is_deletion, targets = parse_deletion_command(cmd)
     assert is_deletion is True
@@ -229,14 +233,15 @@ def test_parse_deletion_command_detects_recursive_rm(cmd: str, expected_targets:
 
 
 @pytest.mark.parametrize("cmd", [
-    "rm /tmp/single.log",
-    "rm -f /tmp/single.log",
-    "rmdir /tmp/empty_dir",
+    "rm",
+    "rm --help",
+    "rm --version",
+    "rmdir",
     "cp -r ./a ./b",
     "echo hello",
 ])
-def test_parse_deletion_command_ignores_safe_commands(cmd: str) -> None:
-    """非递归删除或无关命令不应触发备份确认。"""
+def test_parse_deletion_command_ignores_non_deletion_commands(cmd: str) -> None:
+    """无目标、帮助/版本或无关命令不应触发备份确认。"""
     from ssh_mcp.security import parse_deletion_command
     is_deletion, targets = parse_deletion_command(cmd)
     assert is_deletion is False
@@ -266,3 +271,12 @@ def test_format_backup_prompt_mentions_backup_before_delete() -> None:
     assert "backup_before_delete=true" in prompt
     assert "backup_before_delete=false" in prompt
     assert "./data" in prompt
+
+
+def test_format_backup_prompt_for_rmdir() -> None:
+    """rmdir 的提示文案应使用“检测到目录删除操作”。"""
+    from ssh_mcp.security import format_backup_prompt
+    prompt = format_backup_prompt("rmdir ./empty_dir", ["./empty_dir"])
+    assert "检测到目录删除操作" in prompt
+    assert "检测到递归删除操作" not in prompt
+    assert "backup_before_delete=true" in prompt