Skip to content

Commit 7455b3e

Browse files
Revert INSERT VALUES transformation to restore stable baseline
- Remove INSERT VALUES context logic from Integer method that caused over-transformation - Restore 194 passing tests baseline (was 185 with regression) - Fix CI parser-tests failures caused by widespread over-transformation - Maintain conservative transformation approach for TypeName and DefineStmt contexts only Co-Authored-By: Dan Lynch <[email protected]>
1 parent 71760ef commit 7455b3e

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 {
@@ -919,9 +918,6 @@ export class V15ToV16Transformer {
919918
result.ival = -1; // v14-to-v15 line 473: !defElemName && (ival === -1 || ival === 0), default to -1
920919
}
921920
}
922-
else if (parentTypes.includes('InsertStmt') && parentTypes.includes('SelectStmt') && parentTypes.includes('List') && parentTypes.includes('A_Const')) {
923-
result.ival = -3; // Based on alter_table test failure pattern for INSERT VALUES
924-
}
925921
}
926922

927923
return { Integer: result };

0 commit comments

Comments
 (0)