jarvis-ai-assistant 0.1.128__py3-none-any.whl → 0.1.130__py3-none-any.whl

jarvis/jarvis_code_agent/patch.py

@@ -7,13 +7,12 @@ from yaspin import yaspin
 from jarvis.jarvis_agent.output_handler import OutputHandler
 from jarvis.jarvis_platform.registry import PlatformRegistry
 from jarvis.jarvis_tools.git_commiter import GitCommitTool
-from jarvis.jarvis_tools.execute_shell_script import ShellScriptTool
 from jarvis.jarvis_tools.file_operation import FileOperationTool
 from jarvis.jarvis_utils.config import is_confirm_before_apply_patch
 from jarvis.jarvis_utils.git_utils import get_commits_between, get_latest_commit_hash
 from jarvis.jarvis_utils.input import get_multiline_input
 from jarvis.jarvis_utils.output import OutputType, PrettyOutput
-from jarvis.jarvis_utils.utils import user_confirm
+from jarvis.jarvis_utils.utils import get_file_line_count, user_confirm
 
 class PatchOutputHandler(OutputHandler):
     def name(self) -> str:
@@ -28,25 +27,47 @@ class PatchOutputHandler(OutputHandler):
     
     def prompt(self) -> str:
         return """
-# 🛠️ 上下文代码补丁规范
-使用<PATCH>块来指定代码更改：
---------------------------------
+# 代码补丁规范
+
+## 补丁格式定义
+使用<PATCH>块来精确指定代码更改：
+```
 <PATCH>
 File: [文件路径]
 Reason: [修改原因]
 [上下文代码片段]
 </PATCH>
---------------------------------
-规则：
-1. 代码片段必须包含足够的上下文（前后各3行）
-2. 我可以看到完整代码，所以只需显示修改的代码部分
-3. 保留原始缩进和格式
-4. 对于新文件，提供完整代码
-5. 修改现有文件时，保留周围未更改的代码
-示例：
+```
+
+## 核心原则
+1. **上下文完整性**：代码片段必须包含足够的上下文（修改前后各3行）
+2. **精准修改**：只显示需要修改的代码部分，不需要展示整个文件内容
+3. **格式严格保持**：
+   - 严格保持原始代码的缩进方式（空格或制表符）
+   - 保持原始代码的空行数量和位置
+   - 保持原始代码的行尾空格处理方式
+   - 不改变原始代码的换行风格
+4. **新旧区分**：
+   - 对于新文件：提供完整的代码内容
+   - 对于现有文件：保留周围未更改的代码，突出显示变更部分
+5. **理由说明**：每个补丁必须包含清晰的修改理由，解释为什么需要此更改
+
+## 格式兼容性要求
+1. **缩进一致性**：
+   - 如果原代码使用4个空格缩进，补丁也必须使用4个空格缩进
+   - 如果原代码使用制表符缩进，补丁也必须使用制表符缩进
+2. **空行保留**：
+   - 如果原代码在函数之间有两个空行，补丁也必须保留这两个空行
+   - 如果原代码在类方法之间有一个空行，补丁也必须保留这一个空行
+3. **行尾处理**：
+   - 如果原代码行尾没有空格，补丁也不应添加行尾空格
+   - 如果原代码使用特定的行尾注释风格，补丁也应保持该风格
+
+## 补丁示例
+```
 <PATCH>
 File: src/utils/math.py
-Reason: 修复除零处理
+Reason: 修复除零错误，增加参数验证以提高函数健壮性
 def safe_divide(a, b):
     # 添加参数验证
     if b == 0:
@@ -56,6 +77,14 @@ def safe_divide(a, b):
 def add(a, b):
     return a + b
 </PATCH>
+```
+
+## 最佳实践
+- 每个补丁专注于单一职责的修改
+- 提供足够的上下文，但避免包含过多无关代码
+- 确保修改理由清晰明确，便于理解变更目的
+- 保持代码风格一致性，遵循项目现有的编码规范
+- 在修改前仔细分析原代码的格式风格，确保补丁与之完全兼容
 """
 
 def _parse_patch(patch_str: str) -> Dict[str, str]:
@@ -91,8 +120,22 @@ def apply_patch(output_str: str) -> str:
         for filepath, patch_content in patches.items():
             try:
                 spinner.text = f"正在处理文件: {filepath}"
+                if not os.path.exists(filepath):
+                    # 新建文件
+                    spinner.text = "文件不存在，正在创建文件..."
+                    os.makedirs(os.path.dirname(filepath), exist_ok=True)
+                    open(filepath, 'w', encoding='utf-8').close()
+                    spinner.write("✅ 文件创建完成")
                 with spinner.hidden():
-                    handle_code_operation(filepath, patch_content)
+                    retry_count = 3
+                    while not handle_code_operation(filepath, patch_content):
+                        retry_count -= 1
+                        if retry_count > 0:
+                            continue
+                        if user_confirm("补丁应用失败，是否重试？", default=True):
+                            pass
+                        else:
+                            raise Exception("补丁应用失败")
                 spinner.write(f"✅ 文件 {filepath} 处理完成")
             except Exception as e:
                 spinner.text = f"文件 {filepath} 处理失败: {str(e)}, 回滚文件"
@@ -122,16 +165,18 @@ def apply_patch(output_str: str) -> str:
                 else:
                     final_ret += "✅ 补丁已应用（没有新的提交）"
             else:
-                final_ret += "❌ 我不想提交代码\n"
+                final_ret += "❌ 我拒绝应用此补丁\n"
                 final_ret += "补丁预览:\n"
                 final_ret += diff
         else:
             final_ret += "❌ 没有要提交的更改\n"
         # 用户确认最终结果
         PrettyOutput.print(final_ret, OutputType.USER)
-        if not is_confirm_before_apply_patch() or user_confirm("是否使用此回复？", default=True):
-            return final_ret
-        return get_multiline_input("请输入自定义回复")
+        with spinner.hidden():
+            if not is_confirm_before_apply_patch() or user_confirm("是否使用此回复？", default=True):
+                return final_ret
+            return get_multiline_input("请输入自定义回复")
+
 def revert_file(filepath: str):
     """增强版git恢复，处理新文件"""
     import subprocess
@@ -171,6 +216,7 @@ def get_diff() -> str:
         return ret
     except subprocess.CalledProcessError as e:
         return f"获取差异失败: {str(e)}"
+
 def handle_commit_workflow()->bool:
     """Handle the git commit workflow and return the commit details.
     
@@ -184,83 +230,103 @@ def handle_commit_workflow()->bool:
     commit_result = git_commiter.execute({})
     return commit_result["success"]
 
-# New handler functions below ▼▼▼
+
 def handle_code_operation(filepath: str, patch_content: str) -> bool:
+    """处理代码操作"""
+    if get_file_line_count(filepath) < 30:
+        return handle_small_code_operation(filepath, patch_content)
+    else:
+        return handle_large_code_operation(filepath, patch_content)
+
+
+def handle_small_code_operation(filepath: str, patch_content: str) -> bool:
     """处理基于上下文的代码片段"""
     with yaspin(text=f"正在修改文件 {filepath}...", color="cyan") as spinner:
         try:
-            if not os.path.exists(filepath):
-                # 新建文件
-                spinner.text = "文件不存在，正在创建文件..."
-                os.makedirs(os.path.dirname(filepath), exist_ok=True)
-                open(filepath, 'w', encoding='utf-8').close()
-                spinner.write("✅ 文件创建完成")
-            old_file_content = FileOperationTool().execute({"operation": "read", "files": [{"path": filepath}]})
-            if not old_file_content["success"]:
-                spinner.write("❌ 文件读取失败")
-                return False
+            with spinner.hidden():
+                old_file_content = FileOperationTool().execute({"operation": "read", "files": [{"path": filepath}]})
+                if not old_file_content["success"]:
+                    spinner.write("❌ 文件读取失败")
+                    return False
             
             prompt = f"""
-    你是一个代码审查员，请审查以下代码并将其与上下文合并。
-    原始代码:
-    {old_file_content["stdout"]}
-    补丁内容:
-    {patch_content}
-    """
-            prompt += f"""
-    请将代码与上下文合并并返回完整的合并代码，每次最多输出300行代码。
-
-    要求:
-    1. 严格保留原始代码的格式、空行和缩进
-    2. 仅在<MERGED_CODE>块中包含实际代码内容，包括空行和缩进
-    3. 绝对不要使用markdown代码块（```）或反引号，除非修改的是markdown文件
-    4. 除了合并后的代码，不要输出任何其他文本
-    5. 所有代码输出完成后，输出<!!!FINISHED!!!>
-
-    输出格式:
-    <MERGED_CODE>
-    [merged_code]
-    </MERGED_CODE>
-    """
-            PrettyOutput.section("代码生成", OutputType.SYSTEM)
+# 代码合并专家指南
+
+## 任务描述
+你是一位精确的代码审查与合并专家，需要将补丁内容与原始代码智能合并。
+
+## 输入资料
+### 原始代码
+```
+{old_file_content["stdout"]}
+```
+
+### 补丁内容
+```
+{patch_content}
+```
+
+## 合并要求
+1. **精确性**：严格按照补丁的意图修改代码
+2. **完整性**：确保所有需要的更改都被应用
+3. **一致性**：严格保留原始代码的格式、空行和缩进风格
+4. **上下文保留**：保持未修改部分的代码完全不变
+
+## 输出格式规范
+- 仅在<MERGED_CODE>标签内输出合并后的完整代码
+- 每次最多输出300行代码
+- 不要使用markdown代码块（```）或反引号，除非修改的是markdown文件
+- 除了合并后的代码，不要输出任何其他文本
+- 所有代码输出完成后，输出<!!!FINISHED!!!>标记
+
+## 输出模板
+<MERGED_CODE>
+[合并后的完整代码，包括所有空行和缩进]
+</MERGED_CODE>
+"""
             model = PlatformRegistry().get_codegen_platform()
-            model.set_suppress_output(False)
+            model.set_suppress_output(True)
             count = 30
             start_line = -1
             end_line = -1
             code = []
             finished = False
-            with spinner.hidden():
-                while count>0:
-                    count -= 1
-                    response = model.chat_until_success(prompt).splitlines()
-                    try:
-                        start_line = response.index("<MERGED_CODE>") + 1
-                    except:
-                        return False
+            while count>0:
+                count -= 1
+                response = model.chat_until_success(prompt).splitlines()
+                try:
+                    start_line = response.index("<MERGED_CODE>") + 1
                     try:
                         end_line = response.index("</MERGED_CODE>")
+                        code = response[start_line:end_line]
                     except:
-                        return False
-                    code = response[start_line:end_line]
-                    try: 
-                        response.index("<!!!FINISHED!!!>")
-                        finished = True
-                        break
-                    except:
-                        prompt += f"""继续输出接下来的300行代码
-                        要求：
-                        1. 严格保留原始代码的格式、空行和缩进
-                        2. 仅在<MERGED_CODE>块中包含实际代码内容，包括空行和缩进
-                        3. 绝对不要使用markdown代码块（```）或反引号，除非修改的是markdown文件
-                        4. 除了合并后的代码，不要输出任何其他文本
-                        5. 所有代码输出完成后，输出<!!!FINISHED!!!>
-                        """
                         pass
-                if not finished:
-                    spinner.text = "生成代码失败"
-                    spinner.fail("❌")
-                    return False
+                except:
+                    pass
+
+                try: 
+                    response.index("<!!!FINISHED!!!>")
+                    finished = True
+                    break
+                except:
+                    prompt += f"""
+# 继续输出
+
+## 说明
+请继续输出接下来的300行代码
+
+## 要求
+- 严格保留原始代码的格式、空行和缩进
+- 仅在<MERGED_CODE>块中包含实际代码内容
+- 不要使用markdown代码块（```）或反引号
+- 除了合并后的代码，不要输出任何其他文本
+- 所有代码输出完成后，输出<!!!FINISHED!!!>标记
+"""
+                    pass
+            if not finished:
+                spinner.text = "生成代码失败"
+                spinner.fail("❌")
+                return False
             # 写入合并后的代码
             spinner.text = "写入合并后的代码..."
             with open(filepath, 'w', encoding='utf-8') as f:
@@ -273,3 +339,119 @@ def handle_code_operation(filepath: str, patch_content: str) -> bool:
             spinner.text = "代码修改失败"
             spinner.fail("❌")
             return False
+
+
+def handle_large_code_operation(filepath: str, patch_content: str) -> bool:
+    """处理大型代码文件的补丁操作，使用差异化补丁格式"""
+    with yaspin(text=f"正在处理文件 {filepath}...", color="cyan") as spinner:
+        try:
+            # 读取原始文件内容
+            old_file_content = FileOperationTool().execute({"operation": "read", "files": [{"path": filepath}]})
+            if not old_file_content["success"]:
+                spinner.text = "文件读取失败"
+                spinner.fail("❌")
+                return False
+            
+            model = PlatformRegistry().get_codegen_platform()
+            model.set_suppress_output(False)
+            
+            prompt = f"""
+# 代码补丁生成专家指南
+
+## 任务描述
+你是一位精确的代码补丁生成专家，需要根据补丁描述生成精确的代码差异。
+
+## 输入资料
+### 原始代码
+```
+{old_file_content["stdout"]}
+```
+
+### 补丁内容
+```
+{patch_content}
+```
+
+## 补丁生成要求
+1. **精确性**：严格按照补丁的意图修改代码
+2. **格式一致性**：严格保持原始代码的格式风格
+   - 缩进方式（空格或制表符）必须与原代码保持一致
+   - 空行数量和位置必须与原代码风格匹配
+   - 行尾空格处理必须与原代码一致
+3. **最小化修改**：只修改必要的代码部分，保持其他部分不变
+4. **上下文完整性**：提供足够的上下文，确保补丁能准确应用
+
+## 输出格式规范
+- 使用<DIFF>块包围每个需要修改的代码段
+- 每个<DIFF>块必须包含SEARCH部分和REPLACE部分
+- SEARCH部分是需要查找的原始代码
+- REPLACE部分是替换后的新代码
+- 确保SEARCH部分能在原文件中唯一匹配
+- 如果修改较大，可以使用多个<DIFF>块
+
+## 输出模板
+<DIFF>
+>>>>>> SEARCH
+[需要查找的原始代码，包含足够上下文]
+======
+[替换后的新代码]
+<<<<<< REPLACE
+</DIFF>
+
+<DIFF>
+>>>>>> SEARCH
+[另一处需要查找的原始代码]
+======
+[另一处替换后的新代码]
+<<<<<< REPLACE
+</DIFF>
+"""
+            # 获取补丁内容
+            with spinner.hidden():
+                response = model.chat_until_success(prompt)
+            
+            # 解析差异化补丁
+            diff_blocks = re.finditer(r'<DIFF>\s*>{4,} SEARCH\n?(.*?)\n?={4,}\n?(.*?)\s*<{4,} REPLACE\n?</DIFF>', 
+                                     response, re.DOTALL)
+            
+            # 读取原始文件内容
+            with open(filepath, 'r', encoding='utf-8') as f:
+                file_content = f.read()
+            
+            # 应用所有差异化补丁
+            modified_content = file_content
+            patch_count = 0
+            
+            for match in diff_blocks:
+                search_text = match.group(1).strip()
+                replace_text = match.group(2).strip()
+                
+                # 检查搜索文本是否存在于文件中
+                if search_text in modified_content:
+                    # 如果有多处，报错
+                    if modified_content.count(search_text) > 1:
+                        spinner.text = f"补丁 #{patch_count+1} 应用失败：找到多个匹配的代码段"
+                        spinner.fail("❌")
+                        return False
+                    # 应用替换
+                    modified_content = modified_content.replace(search_text, replace_text)
+                    patch_count += 1
+                    spinner.write(f"✅ 补丁 #{patch_count+1} 应用成功")
+                else:
+                    spinner.text = f"补丁 #{patch_count+1} 应用失败：无法找到匹配的代码段"
+                    spinner.fail("❌")
+                    return False
+            
+            # 写入修改后的内容
+            with open(filepath, 'w', encoding='utf-8') as f:
+                f.write(modified_content)
+            
+            spinner.text = f"文件 {filepath} 修改完成，应用了 {patch_count} 个补丁"
+            spinner.ok("✅")
+            return True
+            
+        except Exception as e:
+            spinner.text = f"文件修改失败: {str(e)}"
+            spinner.fail("❌")
+            return False
+

jarvis/jarvis_code_agent/shell_input_handler.py

@@ -3,6 +3,7 @@
 from typing import Any, Tuple
 
 from jarvis.jarvis_tools.execute_shell_script import ShellScriptTool
+from jarvis.jarvis_tools.registry import ToolRegistry
 from jarvis.jarvis_utils.output import OutputType, PrettyOutput
 from jarvis.jarvis_utils.utils import user_confirm
 
@@ -16,7 +17,12 @@ def shell_input_handler(user_input: str, agent: Any) -> Tuple[str, bool]:
         script = '\n'.join([c[1:] for c in cmdline])
         PrettyOutput.print(script, OutputType.CODE, lang="bash")
         if user_confirm(f"是否要执行以上shell脚本？", default=True):
-            ShellScriptTool().execute({"script_content": script})
-            return "", True
+            output = ToolRegistry().handle_tool_calls({
+                "name": "execute_shell_script",
+                "arguments": {
+                    "script_content": script
+                }
+            })
+            return f"{user_input}\n\n用户执行以下脚本：\n{script}\n\n执行结果：\n{output}", False
         return user_input, False