Skip to content

Commit bf2f1b3

Browse files
Remove forbidden location-based extraction infrastructure
- Remove additionalContext parameter from ASTTransformer.transform method - Remove originalSql context creation from test utils - Clean up all location/SQL string dependencies as required - Maintains pure AST structure transformation approach Co-Authored-By: Dan Lynch <[email protected]>
1 parent ff926a8 commit bf2f1b3

File tree

2 files changed

+4
-9
lines changed

2 files changed

+4
-9
lines changed

packages/transform/src/transformer.ts

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ export class ASTTransformer {
1111
'16-17': new V16ToV17Transformer(),
1212
};
1313

14-
transform(ast: any, fromVersion: number, toVersion: number, additionalContext?: any): any {
14+
transform(ast: any, fromVersion: number, toVersion: number): any {
1515
if (fromVersion === toVersion) {
1616
return ast;
1717
}
@@ -36,11 +36,7 @@ export class ASTTransformer {
3636
currentAst = this.transformers['14-15'].transform(currentAst, { parentNodeTypes: [] });
3737
break;
3838
case '15-16':
39-
const context15to16 = {
40-
parentNodeTypes: [],
41-
...(arguments[3] || {}) // Pass through any additional context from the caller
42-
};
43-
currentAst = this.transformers['15-16'].transform(currentAst, context15to16);
39+
currentAst = this.transformers['15-16'].transform(currentAst, { parentNodeTypes: [] });
4440
break;
4541
case '16-17':
4642
currentAst = this.transformers['16-17'].transform(currentAst, { parentNodeTypes: [] });

packages/transform/test-utils/index.ts

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -78,9 +78,8 @@ export async function expectTransformedAstToEqualParsedAst(
7878
astToTransform.stmts = astToTransform.stmts.map((stmtWrapper: any, index: number) => {
7979
if (stmtWrapper.stmt) {
8080
try {
81-
// Transform the actual statement using the ASTTransformer with SQL context
82-
const contextWithSql = { originalSql: sql };
83-
const transformedStmt = transformer.transform(stmtWrapper.stmt, versionPrevious, versionNext, contextWithSql);
81+
// Transform the actual statement using the ASTTransformer
82+
const transformedStmt = transformer.transform(stmtWrapper.stmt, versionPrevious, versionNext);
8483
return { ...stmtWrapper, stmt: transformedStmt };
8584
} catch (error: any) {
8685
const errorMessage = [

0 commit comments

Comments
 (0)