src/patches/userMessageDisplay.ts (1)

193-217: New and old format branches generate identical content.

The conditional on isNewFormat (lines 193-217) produces the exact same newContent for both branches. The comment on line 194 states "preserve the pointer icon structure," but the generated content doesn't include the pointer at all.

This appears to be incomplete implementation. The new format should likely include the pointer using the captured icons object:

🐛 Potential fix to preserve pointer for new format

  if (isNewFormat) {
    // New format: preserve the pointer icon structure but apply customizations
+   const iconsObject = location.identifiers![4];
    newContent = `
return ${reactVar}.createElement(
  ${boxComponent},
  ${boxAttrsObjStr},
+  ${reactVar}.createElement(
+    ${textComponent},
+    {color:"subtle"},
+    ${iconsObject}.pointer," "
+  ),
  ${reactVar}.createElement(
    ${textComponent},
    ${textAttrsObjStr},
    ${needsChalk ? chalkChain + '(' : ''}${formattedMessage}${needsChalk ? ')' : ''}
  )
);`;
  } else {

Alternatively, if both formats intentionally produce the same output, remove the dead conditional and the unused icons capture.

src/patches/userMessageDisplay.ts (1)

193-217: Critical: New format generation missing pointer icon and both branches are identical.

Two critical issues:

1. Missing pointer icon in new format: The detection pattern at lines 15-16 correctly identifies a three-level nested structure with a pointer icon createElement (with color:"subtle"), but the generated code at lines 195-204 only produces a two-level structure. The middle layer containing the pointer icon is completely omitted.

2. Identical code in both branches: Lines 195-204 (new format) and lines 207-216 (old format) generate exactly the same code. This defeats the purpose of having two separate format branches. If the formats are truly different (as indicated by the detection patterns), the generated code should reflect those differences.

The new format generation should include all three createElement calls to match the detected structure.

🐛 Suggested fix for new format generation

The new format should preserve the three-level structure with the pointer icon. However, I need to verify what variables are available for the pointer icon. Consider capturing the pointer icon variable in the detection pattern and using it in generation:

  const [reactVar, textComponent, messageVar, formatType] =
    location.identifiers!;
  const isNewFormat = formatType === 'new_format';

  // Replace {} in format string with the message variable
  const formattedMessage =
    '"' + format.replace(/\{\}/g, `"+${messageVar}+"`) + '"';

  let newContent: string;

  if (isNewFormat) {
-    // New format: preserve the pointer icon structure but apply customizations
+    // New format: preserve the three-level structure with pointer icon
    newContent = `
return ${reactVar}.createElement(
  ${boxComponent},
  ${boxAttrsObjStr},
+  ${reactVar}.createElement(
+    ${textComponent},
+    {color:"subtle"},
+    /* TODO: Add pointer icon variable here - needs to be captured in detection pattern */
+    " "
+  ),
  ${reactVar}.createElement(
    ${textComponent},
    ${textAttrsObjStr},
    ${needsChalk ? chalkChain + '(' : ''}${formattedMessage}${needsChalk ? ')' : ''}
  )
);`;
  } else {
    // Old format
    newContent = `
return ${reactVar}.createElement(
  ${boxComponent},
  ${boxAttrsObjStr},
  ${reactVar}.createElement(
    ${textComponent},
    ${textAttrsObjStr},
    ${needsChalk ? chalkChain + '(' : ''}${formattedMessage}${needsChalk ? ')' : ''}
  )
);`;
  }

Note: You'll also need to update the detection pattern to capture and return the pointer icon variable (group 4 from the regex at line 16), and adjust the identifiers array to include it.