@graphql-codegen/cli 6.0.0 → 6.0.1-alpha-20251020115833-8d7877f21dec8264954784745fe577eccf6bb9da

package/cjs/utils/watcher.js

@@ -48,7 +48,21 @@ const createWatcher = (initialContext, onNext) => {
         const debouncedExec = (0, debounce_1.default)(() => {
             if (!isShutdown) {
                 (0, codegen_js_1.executeCodegen)(initialContext)
-                    .then(({ result }) => onNext(result), () => Promise.resolve())
+                    .then(({ result, error }) => {
+                    // FIXME: this is a quick fix to stop `onNext` (writeOutput) from
+                    // removing all files when there is an error.
+                    //
+                    // This is because `removeStaleFiles()` will remove files if the
+                    // generated files are different between runs. And on error, it
+                    // returns an empty array i.e. will remove all generated files from
+                    // the previous run
+                    //
+                    // This also means we don't have config.allowPartialOutputs in watch mode
+                    if (error) {
+                        return;
+                    }
+                    onNext(result);
+                }, () => Promise.resolve())
                     .then(() => emitWatching(watchDirectory));
             }
         }, 100);
@@ -146,7 +160,14 @@ const createWatcher = (initialContext, onNext) => {
      */
     stopWatching.runningWatcher = new Promise((resolve, reject) => {
         (0, codegen_js_1.executeCodegen)(initialContext)
-            .then(({ result }) => onNext(result), () => Promise.resolve())
+            .then(({ result, error }) => {
+            // TODO: this is the initial run, the logic here mimics the above watcher logic.
+            // We need to check whether it's ok to deviate between these two.
+            if (error) {
+                return;
+            }
+            onNext(result);
+        }, () => Promise.resolve())
             .then(() => runWatcher(abortController.signal))
             .catch(err => {
             watcherSubscription.unsubscribe();

package/esm/utils/watcher.js

@@ -44,7 +44,21 @@ export const createWatcher = (initialContext, onNext) => {
         const debouncedExec = debounce(() => {
             if (!isShutdown) {
                 executeCodegen(initialContext)
-                    .then(({ result }) => onNext(result), () => Promise.resolve())
+                    .then(({ result, error }) => {
+                    // FIXME: this is a quick fix to stop `onNext` (writeOutput) from
+                    // removing all files when there is an error.
+                    //
+                    // This is because `removeStaleFiles()` will remove files if the
+                    // generated files are different between runs. And on error, it
+                    // returns an empty array i.e. will remove all generated files from
+                    // the previous run
+                    //
+                    // This also means we don't have config.allowPartialOutputs in watch mode
+                    if (error) {
+                        return;
+                    }
+                    onNext(result);
+                }, () => Promise.resolve())
                     .then(() => emitWatching(watchDirectory));
             }
         }, 100);
@@ -142,7 +156,14 @@ export const createWatcher = (initialContext, onNext) => {
      */
     stopWatching.runningWatcher = new Promise((resolve, reject) => {
         executeCodegen(initialContext)
-            .then(({ result }) => onNext(result), () => Promise.resolve())
+            .then(({ result, error }) => {
+            // TODO: this is the initial run, the logic here mimics the above watcher logic.
+            // We need to check whether it's ok to deviate between these two.
+            if (error) {
+                return;
+            }
+            onNext(result);
+        }, () => Promise.resolve())
             .then(() => runWatcher(abortController.signal))
             .catch(err => {
             watcherSubscription.unsubscribe();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@graphql-codegen/cli",
-  "version": "6.0.0",
+  "version": "6.0.1-alpha-20251020115833-8d7877f21dec8264954784745fe577eccf6bb9da",
   "peerDependenciesMeta": {
     "@parcel/watcher": {
       "optional": true

package/typings/generate-and-save.d.cts

@@ -2,4 +2,9 @@ import { Types } from '@graphql-codegen/plugin-helpers';
 import { CodegenContext } from './config.cjs';
 export declare function generate(input: CodegenContext | (Types.Config & {
     cwd?: string;
-}), saveToFile?: boolean): Promise<Types.FileOutput[] | any>;
+}), saveToFile?: boolean): Promise<Types.FileOutput[]
+/**
+ * When this function runs in watch mode, it'd return an empty promise that doesn't resolve until the watcher exits
+ * FIXME: this effectively makes the result `any`, which loses type-hints
+ */
+ | any>;

package/typings/generate-and-save.d.ts

@@ -2,4 +2,9 @@ import { Types } from '@graphql-codegen/plugin-helpers';
 import { CodegenContext } from './config.js';
 export declare function generate(input: CodegenContext | (Types.Config & {
     cwd?: string;
-}), saveToFile?: boolean): Promise<Types.FileOutput[] | any>;
+}), saveToFile?: boolean): Promise<Types.FileOutput[]
+/**
+ * When this function runs in watch mode, it'd return an empty promise that doesn't resolve until the watcher exits
+ * FIXME: this effectively makes the result `any`, which loses type-hints
+ */
+ | any>;