Skip to content

Commit 72939e8

Browse files
Revert INSERT VALUES transformation to restore 194 passing tests baseline
- Remove INSERT context from A_Const method (lines 556-558) - Remove INSERT context from Integer method (lines 907-908) - Restores stable baseline from 185 to 194 passing tests - Eliminates regression caused by over-transformation of zero values Co-Authored-By: Dan Lynch <[email protected]>
1 parent 81609cd commit 72939e8

File tree

1 file changed

+1
-5
lines changed

1 file changed

+1
-5
lines changed

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

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -554,8 +554,7 @@ export class V15ToV16Transformer {
554554
const parentTypes = childContext.parentNodeTypes || [];
555555

556556
if (parentTypes.includes('TypeName') ||
557-
(parentTypes.includes('DefineStmt') && !(context as any).defElemName) ||
558-
(parentTypes.includes('InsertStmt') && parentTypes.includes('SelectStmt') && parentTypes.includes('List'))) {
557+
(parentTypes.includes('DefineStmt') && !(context as any).defElemName)) {
559558
result.ival = this.Integer(result.ival as any, childContext).Integer;
560559
}
561560
} else {
@@ -904,9 +903,6 @@ export class V15ToV16Transformer {
904903
if (parentTypes.includes('TypeName')) {
905904
result.ival = -1; // Based on alter_table test failure pattern and rangetypes-289 arrayBounds
906905
}
907-
else if (parentTypes.includes('InsertStmt') && parentTypes.includes('SelectStmt') && parentTypes.includes('List') && parentTypes.includes('A_Const')) {
908-
result.ival = -3; // Based on alter_table test failure pattern for INSERT VALUES
909-
}
910906
// DefineStmt context: Only very specific cases from v14-to-v15
911907
else if (parentTypes.includes('DefineStmt')) {
912908
const defElemName = (context as any).defElemName;

0 commit comments

Comments
 (0)