Make Integer transformation extremely conservative to prevent over-transformation

- Remove TypeName context transformation that was over-transforming arrayBounds
- Remove broad DefineStmt transformation conditions
- Only keep the most specific, documented transformation cases (initcond, sspace)
- Should resolve CI failures with 95 failed test suites by preventing widespread over-transformation

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

Author: devin-ai-integration[bot]

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

@@ -546,8 +546,8 @@ export class V15ToV16Transformer {
       // Handle empty Integer objects directly since transform() can't detect their type
       if (typeof result.ival === 'object' && Object.keys(result.ival).length === 0) {
         const parentTypes = childContext.parentNodeTypes || [];
-        if (parentTypes.includes('TypeName') || 
-            (parentTypes.includes('DefineStmt') && !(context as any).defElemName)) {
+        // Only transform in the most specific cases where absolutely necessary
+        if (parentTypes.includes('DefineStmt') && (context as any).defElemName === 'initcond') {
           result.ival = this.Integer(result.ival as any, childContext).Integer;
         }
       } else {
@@ -890,14 +890,11 @@ export class V15ToV16Transformer {
   Integer(node: PG15.Integer, context: TransformerContext): { Integer: PG16.Integer } {
     const result: any = { ...node };
 
+    // Only transform empty Integer objects in very specific, documented cases
     if (Object.keys(result).length === 0) {
       const parentTypes = context.parentNodeTypes || [];
 
-      if (parentTypes.includes('TypeName')) {
-        result.ival = -1;  // Based on alter_table test failure pattern
-      }
-      // DefineStmt context: Only very specific cases from v14-to-v15
-      else if (parentTypes.includes('DefineStmt')) {
+      if (parentTypes.includes('DefineStmt')) {
         const defElemName = (context as any).defElemName;
 
         // Only transform for very specific defElemName values that are documented in v14-to-v15
@@ -906,10 +903,6 @@ export class V15ToV16Transformer {
         } else if (defElemName === 'sspace') {
           result.ival = 0;     // v14-to-v15 line 468: ival === 0
         }
-        // DefineStmt args context: empty Integer objects should transform to ival: -1
-        else if (!defElemName) {
-          result.ival = -1;    // v14-to-v15 line 473: !defElemName && (ival === -1 || ival === 0), default to -1
-        }
       }
     }