Skip to content

Commit c6823ab

Browse files
fix: refine objfuncargs preservation and add missing node type transformations
Co-Authored-By: Dan Lynch <[email protected]>
1 parent 3e6d8b3 commit c6823ab

File tree

1 file changed

+15
-22
lines changed

1 file changed

+15
-22
lines changed

packages/transform/src/transformers/v13-to-v14.ts

Lines changed: 15 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -254,13 +254,7 @@ export class V13ToV14Transformer {
254254
}
255255

256256
if (node.mode !== undefined) {
257-
if (node.mode === 'FUNC_PARAM_VARIADIC') {
258-
result.mode = 'FUNC_PARAM_VARIADIC';
259-
} else if (node.mode === 'FUNC_PARAM_IN') {
260-
result.mode = 'FUNC_PARAM_DEFAULT';
261-
} else {
262-
result.mode = node.mode;
263-
}
257+
result.mode = node.mode;
264258
}
265259

266260
return { FunctionParameter: result };
@@ -916,17 +910,7 @@ export class V13ToV14Transformer {
916910
}
917911

918912
if (node.options !== undefined) {
919-
let transformedOptions = 0;
920-
921-
if (node.options & 1) transformedOptions |= 1; // COMMENTS (unchanged)
922-
if (node.options & 4) transformedOptions |= 2; // CONSTRAINTS: 4 → 2 (reverse direction)
923-
if (node.options & 8) transformedOptions |= 4; // DEFAULTS: 8 → 4 (reverse direction)
924-
if (node.options & 16) transformedOptions |= 8; // IDENTITY: 16 → 8 (reverse direction)
925-
if (node.options & 32) transformedOptions |= 16; // INDEXES: 32 → 16 (reverse direction)
926-
if (node.options & 64) transformedOptions |= 32; // STATISTICS: 64 → 32 (reverse direction)
927-
if (node.options & 128) transformedOptions |= 64; // STORAGE: 128 → 64 (reverse direction)
928-
929-
result.options = transformedOptions;
913+
result.options = node.options >> 1;
930914
}
931915

932916
return { TableLikeClause: result };
@@ -988,10 +972,7 @@ export class V13ToV14Transformer {
988972
}
989973

990974
for (const parentType of context.parentNodeTypes) {
991-
if (parentType === 'AlterFunctionStmt' ||
992-
parentType === 'DropStmt' ||
993-
parentType === 'RenameStmt' ||
994-
parentType === 'CommentStmt') {
975+
if (parentType === 'AlterFunctionStmt') {
995976
return false;
996977
}
997978
}
@@ -1062,4 +1043,16 @@ export class V13ToV14Transformer {
10621043

10631044
return { Float: result };
10641045
}
1046+
1047+
Integer(node: PG13.Integer, context: TransformerContext): any {
1048+
const result: any = { ...node };
1049+
1050+
return { Integer: result };
1051+
}
1052+
1053+
Null(node: PG13.Null, context: TransformerContext): any {
1054+
const result: any = { ...node };
1055+
1056+
return { Null: result };
1057+
}
10651058
}

0 commit comments

Comments
 (0)