@dudousxd/nestjs-codegen 0.22.0 → 0.22.1

package/CHANGELOG.md

@@ -1,5 +1,29 @@
 # @dudousxd/nestjs-codegen
 
+## 0.22.1
+
+### Patch Changes
+
+- da8ec39: Order routes by file path, so an unchanged source tree always generates the same client
+
+  Route order decided the order of everything emitted from it — the groups in
+  `api.ts`, the entries in `routes.ts` — and it was whatever order discovery
+  happened to reach the files in. The cold path adds controllers in `fast-glob`'s
+  directory-walk order, which is I/O-completion order from a concurrent walk: it
+  holds while the FS cache is warm and can differ on a cold one, so regenerating
+  an untouched project could move a controller group to a different position. The
+  watcher appended each newly-created controller to the end of its set, so from
+  the moment a file was added its output no longer matched a cold run's, for the
+  rest of the session.
+
+  Both extraction entry points now sort their roots by path. `discoverPages` has
+  sorted its glob from the start, which is why only the controller-derived
+  artifacts ever moved.
+
+  Nothing about the generated client changes except the order of its blocks —
+  same routes, same types, same members. Consumers who commit their generated
+  directory should expect one reorder-only diff on the first run after upgrading.
+
 ## 0.22.0
 
 ### Minor Changes

package/dist/cli/main.cjs

@@ -2754,7 +2754,7 @@ function bindDiscoveryContext(project, cwd, tsconfigPath, tsconfig = loadDiscove
 }
 function extractRoutesFrom(project, controllerPaths) {
   const routes = [];
-  for (const path of controllerPaths) {
+  for (const path of byPath([...controllerPaths])) {
     const sourceFile = project.getSourceFile(path);
     if (sourceFile) routes.push(...extractFromSourceFile(sourceFile, project));
   }
@@ -2762,11 +2762,17 @@ function extractRoutesFrom(project, controllerPaths) {
 }
 function extractAllRoutes(project) {
   const routes = [];
-  for (const sourceFile of project.getSourceFiles()) {
+  for (const sourceFile of byPath(project.getSourceFiles(), (f) => f.getFilePath())) {
     routes.push(...extractFromSourceFile(sourceFile, project));
   }
   return routes;
 }
+function byPath(items, path = String) {
+  return [...items].sort((a, b) => {
+    const [x, y] = [path(a), path(b)];
+    return x < y ? -1 : x > y ? 1 : 0;
+  });
+}
 var PersistentDiscovery = class _PersistentDiscovery {
   project;
   cwd;
@@ -5247,7 +5253,7 @@ async function watch(config, onChange, options = {}) {
 }
 
 // src/index.ts
-var VERSION = "0.22.0";
+var VERSION = "0.22.1";
 
 // src/cli/codegen.ts
 async function runCodegen(opts = {}) {