Merge pull request #318 from constructive-io/feat/transform-pure-ast

refactor(transform): keep @pgsql/transform pure AST-level; move change-aware pieces up

Author: pyramation

AGENTS.md

@@ -17,7 +17,7 @@ A pnpm monorepo for PostgreSQL AST parsing, deparsing, and code generation. All
 | `@pgsql/utils` | `packages/utils` | Type-safe AST node creation utilities |
 | `@pgsql/traverse` | `packages/traverse` | Visitor-pattern AST traversal |
 | `@pgsql/transform-ast` | `packages/transform-ast` | Multi-version AST transformer (PG 13-17) |
-| `@pgsql/transform` | `packages/transform` | SQL schema transformation, statement classification, qualification, dependency closure |
+| `@pgsql/transform` | `packages/transform` | SQL schema transformation, statement classification (AST facts), qualification, round-trip validation |
 | `@pgsql/quotes` | `packages/quotes` | SQL identifier/string quoting and keyword classification |
 | `@pgsql/cli` | `packages/pgsql-cli` | CLI tool for parse/deparse operations |
 | `pg-proto-parser` | `packages/proto-parser` | Generate TypeScript from PostgreSQL protobuf definitions |

README.md

@@ -174,7 +174,7 @@ walk(ast, visitor);
 | [**@pgsql/utils**](./packages/utils) | Type-safe AST node creation utilities | • Programmatic AST construction<br>• Runtime Schema<br>• Seamless integration with types |
 | [**pg-proto-parser**](./packages/proto-parser) | PostgreSQL protobuf parser and code generator | • Generate TypeScript interfaces from protobuf<br>• Create enum mappings and utilities<br>• AST helper generation |
 | [**@pgsql/transform-ast**](./packages/transform-ast) | Multi-version PostgreSQL AST transformer | • Transform ASTs between PostgreSQL versions (13→17)<br>• Single source of truth deparser pipeline<br>• Backward compatibility for legacy SQL |
-| [**@pgsql/transform**](./packages/transform) | SQL transformation & classification | • Schema-name rewriting (incl. PL/pgSQL bodies)<br>• Per-statement AST facts (`classifyStatements`)<br>• Transitive dependency closure & qualification |
+| [**@pgsql/transform**](./packages/transform) | SQL transformation & classification | • Schema-name rewriting (incl. PL/pgSQL bodies)<br>• Per-statement AST facts (`classifyStatements`)<br>• Qualification & round-trip validation |
 | [**@pgsql/traverse**](./packages/traverse) | PostgreSQL AST traversal utilities | • Visitor pattern for traversing PostgreSQL AST nodes<br>• NodePath context with parent/path information<br>• Runtime schema-based precise traversal |
 
 ## 🛠️ Development

packages/transform/README.md

@@ -54,14 +54,6 @@ const facts = classifyStatements(sql);
 
 Qualify unqualified object references against an inventory of known objects, with multi-schema routing support.
 
-### `resolveFixtureClosure` — transitive dependency closure
-
-Given a set of changes (name + SQL + optional declared dependencies), compute the transitive closure of a selection: forward producers of every referenced object/schema/role, plus attached fixtures (policies/grants/RLS targeting closure members), with explicit unresolved-reference reporting.
-
-### `categorizeChange` / `buildCategoryOf` — change categorization
-
-Profile-driven categorization of migration changes (e.g. schema / functionality / security / fixtures) from their AST facts.
-
 ### Round-trip validation
 
 `normalizeTree` / `cleanTree` / `validateRoundTrip` — dependency-free AST normalization and mutation-aware parse→deparse→re-parse validation.