Fix create result handling

Author: jahooma

.agents/types/tools.ts

@@ -181,10 +181,10 @@ export interface ProposeStrReplaceParams {
   /** Array of replacements to make. */
   replacements: {
     /** The string to replace. This must be an *exact match* of the string you want to replace, including whitespace and punctuation. */
-    old: string
-    /** The string to replace the corresponding old string with. Can be empty to delete. */
-    new: string
-    /** Whether to allow multiple replacements of old string. */
+    oldString: string
+    /** The string to replace the corresponding oldString with. Can be empty to delete. */
+    newString: string
+    /** Whether to allow multiple replacements of oldString. */
     allowMultiple?: boolean
   }[]
 }
@@ -305,10 +305,10 @@ export interface StrReplaceParams {
   /** Array of replacements to make. */
   replacements: {
     /** The string to replace. This must be an *exact match* of the string you want to replace, including whitespace and punctuation. */
-    old: string
-    /** The string to replace the corresponding old string with. Can be empty to delete. */
-    new: string
-    /** Whether to allow multiple replacements of old string. */
+    oldString: string
+    /** The string to replace the corresponding oldString with. Can be empty to delete. */
+    newString: string
+    /** Whether to allow multiple replacements of oldString. */
     allowMultiple?: boolean
   }[]
 }

cli/src/components/tools/str-replace.tsx

@@ -40,6 +40,10 @@ function extractValueForKey(output: string, key: string): string | null {
   return null
 }
 
+function isCreatedFileMessage(message: string | null): boolean {
+  return message === 'Created file successfully.' || message === 'Created new file'
+}
+
 interface EditHeaderProps {
   name: string
   filePath: string | null
@@ -97,7 +101,7 @@ export const StrReplaceComponent = defineToolComponent({
         ? (toolBlock.input as any).path
         : null)
     const message = extractValueForKey(outputStr, 'message')
-    const isCreate = message === 'Created new file'
+    const isCreate = isCreatedFileMessage(message)
 
     return {
       content: (

cli/src/utils/__tests__/implementor-helpers.test.ts

@@ -215,6 +215,17 @@ describe('getFileChangeType', () => {
     expect(getFileChangeType(block)).toBe('A')
   })
 
+  test('returns A for successful file creation', () => {
+    const block: ToolContentBlock = {
+      type: 'tool',
+      toolCallId: 'test-1',
+      toolName: 'write_file',
+      input: {},
+      output: 'message: Created file successfully.',
+    }
+    expect(getFileChangeType(block)).toBe('A')
+  })
+
   test('returns M for write_file modification', () => {
     const block: ToolContentBlock = {
       type: 'tool',

cli/src/utils/implementor-helpers.ts

@@ -325,7 +325,8 @@ export function isCreateFile(toolBlock: ToolContentBlock): boolean {
   const message = extractValueForKey(outputStr, 'message')
   return (
     typeof message === 'string' &&
-    (message.startsWith('Created new file') ||
+    (message.startsWith('Created file successfully') ||
+      message.startsWith('Created new file') ||
       message.startsWith('Proposed new file'))
   )
 }