Remove VSCode mock and refactor diff processing by moving file operations to handler layer

Author: robertpiosik

apps/editor/__mocks__/vscode.js

@@ -2,29 +2,10 @@
 module.exports = {
   preset: 'ts-jest',
   testEnvironment: 'node',
-  // Automatically clear mock calls, instances, contexts and results before every test
   clearMocks: true,
-  // Indicates whether the coverage information should be collected while executing the test
   testMatch: ['**/src/**/*.@(spec|test).ts?(x)'],
-  // Module name mapper for path aliases from tsconfig.json
   moduleNameMapper: {
     '^@/(.*)$': '<rootDir>/src/$1',
-    '^@shared/(.*)$': '<rootDir>/../../packages/shared/src/$1',
-    '^vscode$': '<rootDir>/__mocks__/vscode.js'
-  },
-  // Transform files with ts-jest
-  transform: {
-    '^.+\\.tsx?$': [
-      'ts-jest',
-      {
-        tsconfig: 'tsconfig.json' // Ensure it uses the correct tsconfig
-      }
-    ]
-  },
-  // Ignore transforms for node_modules
-  transformIgnorePatterns: ['/node_modules/']
-  // Setup files can be added here if needed, e.g., for global mocks or environment setup
-  // setupFilesAfterEnv: ['<rootDir>/jest.setup.js'],
-  // Note: For tests involving actual VS Code APIs, a different setup might be needed
-  // using vscode-test or similar, but for unit tests like clipboard-parser, 'node' is fine.
-}
+    '^@shared/(.*)$': '<rootDir>/../../packages/shared/src/$1'
+  }
+};

apps/editor/jest.config.js

@@ -6,7 +6,7 @@ import { Logger } from '@shared/utils/logger'
 import { promisify } from 'util'
 import { OriginalFileState } from '../types/original-file-state'
 import { create_safe_path } from '@/utils/path-sanitizer'
-import { apply_diff, process_diff } from '../utils/diff-processor'
+import { apply_diff } from '../utils/diff-processor'
 import { remove_directory_if_empty } from '../utils/file-operations'
 
 const execAsync = promisify(exec)
@@ -398,6 +398,36 @@ const handle_deleted_file_patch = async (
   }
 }
 
+const process_diff = async (params: {
+  file_path: string
+  diff_patch_path: string
+  use_strict_whitespace?: boolean
+}): Promise<string> => {
+  const file_content = fs.readFileSync(params.file_path, 'utf8')
+  const diff_patch_content = fs.readFileSync(params.diff_patch_path, 'utf8')
+
+  const result = apply_diff({
+    original_code: file_content,
+    diff_patch: diff_patch_content,
+    use_strict_whitespace: params.use_strict_whitespace
+  })
+
+  try {
+    await vscode.workspace.fs.writeFile(
+      vscode.Uri.file(params.file_path),
+      Buffer.from(result, 'utf8')
+    )
+    return result
+  } catch (error) {
+    Logger.error({
+      function_name: 'process_diff',
+      message: 'Failed to save file after applying diff patch',
+      data: { error, file_path: params.file_path }
+    })
+    throw new Error('Failed to save file after applying diff patch')
+  }
+}
+
 export const apply_git_patch = async (
   patch_content: string,
   workspace_path: string

apps/editor/src/commands/apply-chat-response-command/handlers/diff-handler.ts

@@ -1,6 +1,3 @@
-import * as vscode from 'vscode'
-import * as fs from 'fs'
-
 type ReplaceLine = {
   content: string
   search_index: number | null
@@ -14,31 +11,6 @@ type SearchBlock = {
   search_to_original_map: Map<number, number>
 }
 
-export const process_diff = async (params: {
-  file_path: string
-  diff_patch_path: string
-  use_strict_whitespace?: boolean
-}): Promise<string> => {
-  const file_content = fs.readFileSync(params.file_path, 'utf8')
-  const diff_patch_content = fs.readFileSync(params.diff_patch_path, 'utf8')
-
-  const result = apply_diff({
-    original_code: file_content,
-    diff_patch: diff_patch_content,
-    use_strict_whitespace: params.use_strict_whitespace
-  })
-
-  try {
-    await vscode.workspace.fs.writeFile(
-      vscode.Uri.file(params.file_path),
-      Buffer.from(result, 'utf8')
-    )
-    return result
-  } catch (error) {
-    throw new Error('Failed to save file after applying diff patch')
-  }
-}
-
 export const apply_diff = (params: {
   original_code: string
   diff_patch: string