Fix DefineStmt args Integer transformation

- Remove overly broad TypeName arrayBounds transformation that was causing over-transformation
- Keep targeted DefineStmt args context transformation for CREATE AGGREGATE statements
- Transform empty Integer objects to {ival: -1} when in DefineStmt context but not DefElem context
- Resolves original-define.test.ts failure while reducing over-transformation issues
- Local test results: 75 failed tests (improved from 98 initial failures)

Co-Authored-By: Dan Lynch <[email protected]>

Author: devin-ai-integration[bot]

packages/transform/src/transformers/v15-to-v16.ts

@@ -892,13 +892,8 @@ export class V15ToV16Transformer {
       const parentTypes = context.parentNodeTypes || [];
       const contextData = context as any;
 
-      // TypeName arrayBounds context: Transform empty objects to ival: -1
-      if (parentTypes.includes('TypeName')) {
-        result.ival = -1;
-      }
-      
       // DefineStmt context: Only very specific cases from v14-to-v15
-      else if (parentTypes.includes('DefineStmt')) {
+      if (parentTypes.includes('DefineStmt')) {
         const defElemName = contextData.defElemName;
 
         // Only transform for very specific defElemName values that are documented in v14-to-v15
@@ -907,6 +902,10 @@ export class V15ToV16Transformer {
         } else if (defElemName === 'sspace') {
           result.ival = 0;     // v14-to-v15 line 468: ival === 0
         }
+        // DefineStmt args context: Only for CREATE AGGREGATE statements (v14-to-v15 line 473)
+        else if (!defElemName && parentTypes.includes('DefineStmt') && !parentTypes.includes('DefElem')) {
+          result.ival = -1;    // v14-to-v15 line 473: !defElemName && (ival === -1 || ival === 0), default to -1
+        }
       }
 
       // AlterTableCmd context: Only for SET STATISTICS operations (specific case from v14-to-v15 line 456)