uql-codemod 0.31.0 → 0.33.0

package/README.md

@@ -39,6 +39,7 @@ It reports rather than guesses, and exits non-zero when anything is left for you
 - **`target: esnext`** is reported, not changed. Removing the line falls back to the compiler default (`es5` for `tsc`), and choosing a replacement means guessing which era the project targets. Any dated target works; `esnext` is the one where TypeScript emits decorator syntax untransformed.
 - **A value inherited through `extends`** cannot be edited here, so the base config is reported instead.
 - **`@Log()`, `@Serialized()`, `@Transactional()` and `@InjectQuerier()`** are reported and left in place. They no longer exist, and what to do instead is a judgement call - a `@Transactional()` method becomes a `pool.transaction(async (querier) => { ... })` around its body, and the codemod cannot know which pool.
+- **An export that no longer exists** (`setQuerierPool`, `getQuerier`, `QueryWhereMap`, `augmentWhere`, ...) is reported with what replaces it. A `$where` given an id, a list of them or a bare `raw()` needs no report: it is a compile error on that value, fixed by naming the key (`{ id: [1, 2] }`) or wrapping the raw in `$and`.
 - **A key written under a computed name** (`@Id() [KEY]?: string`) is reported: the brand has to spell the name out, and only you know what that expression evaluates to.
 - **Options it cannot read** (`@Field(sharedOptions)`, a spread, or a `@Field` never called at all) are left exactly as written. Replacing an argument it cannot parse would silently drop the options.
 - **A property whose type it cannot map to a column** (a `Json<T>`, a vector, a union that disagrees with itself) is reported as `needs a decision`. Those always had to declare `type` by hand anyway, because `design:type` reported the useless `Object`/`Array` for them.

package/dist/transform.js

@@ -16,12 +16,26 @@ const REMOVED_DECORATORS = new Map([
 ]);
 /**
  * Exports that no longer exist, mapped to what to do instead. Reported rather than rewritten, for the
- * same reason as the decorators: which pool belongs at a given call site is a judgement call.
+ * same reason as the decorators: what replaces each is decided at its call site.
  */
 const REMOVED_EXPORTS = new Map([
     ['setQuerierPool', 'pass the pool where it is used: `createFetchHandler({ pool })`, `querierMiddleware({ pool })`'],
     ['getQuerierPool', 'take the pool from the module that builds it, or from Nest DI'],
     ['getQuerier', 'use `pool.withQuerier(...)` / `pool.transaction(...)`, which release the connection'],
+    ['QueryWhereMap', 'it is `QueryWhere` now'],
+    ['QueryWhereFieldMap', 'use `QueryWhere`'],
+    ['augmentWhere', 'spread the two maps: `{ ...where, ...extra }`'],
+    ['buildQueryWhereAsMap', 'a `$where` is a map already; name the key for ids: `{ id: [1, 2] }`'],
+    ['PgDialect', 'the pools build `PostgresDialect` (`uql-orm/postgres`); name that where a dialect is typed'],
+    ['NeonDialect', 'the pools build `PostgresDialect` (`uql-orm/postgres`); name that where a dialect is typed'],
+    ['PgliteDialect', 'the pools build `PostgresDialect` (`uql-orm/postgres`); name that where a dialect is typed'],
+    ['MySql2Dialect', 'the pool builds `MySqlDialect` (`uql-orm/mysql`)'],
+    ['MongodbNativeDialect', 'the pool builds `MongoDialect` (`uql-orm/mongo`)'],
+    ['AbstractPgQuerier', 'every pg-compatible pool returns `PgQuerier` (`uql-orm/postgres`), which is concrete'],
+    ['CrdbQuerier', 'every pg-compatible pool returns `PgQuerier` (`uql-orm/postgres`)'],
+    ['NeonQuerier', 'every pg-compatible pool returns `PgQuerier` (`uql-orm/postgres`)'],
+    ['LibsqlQuerier', 'the libSQL and Turso pools return `HranaQuerier` (`uql-orm/sqlite`)'],
+    ['TursoQuerier', 'the libSQL and Turso pools return `HranaQuerier` (`uql-orm/sqlite`)'],
 ]);
 /** A node's decorators, or none where it is a kind that cannot carry them. */
 function decoratorsOf(node) {
@@ -327,9 +341,9 @@ function reportRemovedDecorators(node, ctx) {
         }
     }
 }
-/** Names an export that no longer exists, where it is imported from the package. */
+/** Names an export that no longer exists, where it is imported from the package or one of its entries. */
 function reportRemovedExports(source, ctx) {
-    for (const element of uqlImports(source)) {
+    for (const element of uqlImports(source, true)) {
         const name = importedName(element);
         const advice = REMOVED_EXPORTS.get(name);
         if (advice) {
@@ -337,13 +351,18 @@ function reportRemovedExports(source, ctx) {
         }
     }
 }
-/** What the file imports from `uql-orm` by name, which is every import this codemod reads or writes. */
-function uqlImports(source) {
+/**
+ * What the file imports from `uql-orm` by name, which is every import this codemod reads or writes.
+ * `entries` adds the driver entries (`uql-orm/postgres`, ...), for reporting only: an import written
+ * into one of those would land on an entry that may not export it.
+ */
+function uqlImports(source, entries = false) {
     return source.statements.flatMap((statement) => {
         if (!ts.isImportDeclaration(statement) || !ts.isStringLiteral(statement.moduleSpecifier)) {
             return [];
         }
-        if (statement.moduleSpecifier.text !== 'uql-orm') {
+        const specifier = statement.moduleSpecifier.text;
+        if (specifier !== 'uql-orm' && !(entries && specifier.startsWith('uql-orm/'))) {
             return [];
         }
         const bindings = statement.importClause?.namedBindings;

package/package.json

@@ -2,7 +2,7 @@
   "name": "uql-codemod",
   "description": "Upgrade uql-orm across versions that introduce breaking changes",
   "license": "MIT",
-  "version": "0.31.0",
+  "version": "0.33.0",
   "type": "module",
   "engines": {
     "node": ">=24"