abbyswag/qwen-code
1
0
Fork 0
mirror of https://github.com/QwenLM/qwen-code.git synced 2025-12-19 09:33:53 +00:00
Code Issues Packages Projects Releases Wiki Activity

refactor(vscode-ide-companion): simplify openDiff tool implementation

Browse Source 
- Remove redundant file reading logic from ide-server
- Leverage diffManager's new capability to read old content internally
- Simplify openDiff tool call site to pass only newContent
- Update comments to reflect the simplified implementation
This commit is contained in:
yiliang114
2025-12-09 15:53:41 +08:00
parent f2a74c74b6
commit e91ea3ac1a
1 changed files with 2 additions and 12 deletions
Download Patch File Download Diff File Expand all files Collapse all files

14
packages/vscode-ide-companion/src/ide-server.ts
View File

@@ -437,18 +437,8 @@ const createMcpServer = (diffManager: DiffManager) => {
inputSchema: OpenDiffRequestSchema.shape, inputSchema: OpenDiffRequestSchema.shape,
}, },
async ({ filePath, newContent }: z.infer<typeof OpenDiffRequestSchema>) => { async ({ filePath, newContent }: z.infer<typeof OpenDiffRequestSchema>) => {
// Read old content if file exists, otherwise use empty string // Minimal call site: only pass newContent; DiffManager reads old content itself
let oldContent = ''; await diffManager.showDiff(filePath, newContent);
try {
const fileUri = vscode.Uri.file(filePath);
const document = await vscode.workspace.openTextDocument(fileUri);
oldContent = document.getText();
} catch (_error) {
// File doesn't exist, use empty string (creating new file)
oldContent = '';
}
await diffManager.showDiff(filePath, oldContent, newContent);
return { content: [] }; return { content: [] };
}, },
); );