fix: correct SortBy node wrapping in v15-to-v16 transformer

devin-ai-integration[bot] · pyramation · devin-ai-integration[bot] · commit 30d894a55fad · 2025-06-29T03:37:59.000Z
- Change SortBy method to return wrapped { SortBy: node } instead of unwrapped node
- Fixes CI failures in 15-16 test suite caused by incorrect node wrapping
- Maintains consistency with expected PG16 AST format
- Resolves original-upstream-circle.test.ts and other SortBy-related test failures

Co-Authored-By: Dan Lynch &lt;pyramation@gmail.com&gt;
diff --git a/packages/transform/src/transformers/v15-to-v16.ts b/packages/transform/src/transformers/v15-to-v16.ts
@@ -880,7 +880,7 @@ export class V15ToV16Transformer {
   }
 
   SortBy(node: PG15.SortBy, context: TransformerContext): any {
-    return node;
+    return { SortBy: node };
   }
 
   GroupingSet(node: PG15.GroupingSet, context: TransformerContext): any {

Original file line number	Diff line number	Diff line change
`@@ -880,7 +880,7 @@ export class V15ToV16Transformer {`
`880`	`880`	`}`
`881`	`881`
`882`	`882`	`SortBy(node: PG15.SortBy, context: TransformerContext): any {`
`883`		`- return node;`
	`883`	`+ return { SortBy: node };`
`884`	`884`	`}`
`885`	`885`
`886`	`886`	`GroupingSet(node: PG15.GroupingSet, context: TransformerContext): any {`