nx 19.7.2 → 19.7.3

package/src/command-line/show/command-object.js

@@ -3,7 +3,7 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.yargsShowCommand = void 0;
 const yargs_1 = require("yargs");
 const shared_options_1 = require("../yargs-utils/shared-options");
-const params_1 = require("../../utils/params");
+const handle_errors_1 = require("../../utils/handle-errors");
 exports.yargsShowCommand = {
     command: 'show',
     describe: 'Show information about the workspace (e.g., list of projects).',
@@ -65,7 +65,7 @@ const showProjectsCommand = {
         .example('$0 show projects --type app --affected', 'Show affected apps in the workspace')
         .example('$0 show projects --affected --exclude=*-e2e', 'Show affected projects in the workspace, excluding end-to-end projects'),
     handler: async (args) => {
-        const exitCode = await (0, params_1.handleErrors)(args.verbose, async () => {
+        const exitCode = await (0, handle_errors_1.handleErrors)(args.verbose, async () => {
             const { showProjectsHandler } = await Promise.resolve().then(() => require('./projects'));
             await showProjectsHandler(args);
         });
@@ -106,7 +106,7 @@ const showProjectCommand = {
         .example('$0 show project my-app', 'View project information for my-app in JSON format')
         .example('$0 show project my-app --web', 'View project information for my-app in the browser'),
     handler: async (args) => {
-        const exitCode = await (0, params_1.handleErrors)(args.verbose, async () => {
+        const exitCode = await (0, handle_errors_1.handleErrors)(args.verbose, async () => {
             const { showProjectHandler } = await Promise.resolve().then(() => require('./project'));
             await showProjectHandler(args);
         });

package/src/command-line/sync/sync.js

@@ -5,11 +5,11 @@ const ora = require("ora");
 const nx_json_1 = require("../../config/nx-json");
 const project_graph_1 = require("../../project-graph/project-graph");
 const output_1 = require("../../utils/output");
-const params_1 = require("../../utils/params");
+const handle_errors_1 = require("../../utils/handle-errors");
 const sync_generators_1 = require("../../utils/sync-generators");
 const chalk = require("chalk");
 function syncHandler(options) {
-    return (0, params_1.handleErrors)(options.verbose, async () => {
+    return (0, handle_errors_1.handleErrors)(options.verbose, async () => {
         const projectGraph = await (0, project_graph_1.createProjectGraphAsync)();
         const nxJson = (0, nx_json_1.readNxJson)();
         const syncGenerators = await (0, sync_generators_1.collectAllRegisteredSyncGenerators)(projectGraph, nxJson);
@@ -32,24 +32,82 @@ function syncHandler(options) {
             });
             return 0;
         }
+        const { failedGeneratorsCount, areAllResultsFailures, anySyncGeneratorsFailed, } = (0, sync_generators_1.processSyncGeneratorResultErrors)(results);
+        const failedSyncGeneratorsFixMessageLines = (0, sync_generators_1.getFailedSyncGeneratorsFixMessageLines)(results, options.verbose);
+        if (areAllResultsFailures) {
+            output_1.output.error({
+                title: `The workspace is probably out of sync because ${failedGeneratorsCount === 1
+                    ? 'a sync generator'
+                    : 'some sync generators'} failed to run`,
+                bodyLines: failedSyncGeneratorsFixMessageLines,
+            });
+            return 1;
+        }
+        const resultBodyLines = (0, sync_generators_1.getSyncGeneratorSuccessResultsMessageLines)(results);
         if (options.check) {
             output_1.output.error({
-                title: `The workspace is out of sync`,
-                bodyLines: (0, sync_generators_1.syncGeneratorResultsToMessageLines)(results),
+                title: 'The workspace is out of sync',
+                bodyLines: resultBodyLines,
             });
+            if (anySyncGeneratorsFailed) {
+                output_1.output.error({
+                    title: failedGeneratorsCount === 1
+                        ? 'A sync generator failed to run'
+                        : 'Some sync generators failed to run',
+                    bodyLines: failedSyncGeneratorsFixMessageLines,
+                });
+            }
             return 1;
         }
         output_1.output.warn({
-            title: `The workspace is out of sync`,
-            bodyLines: (0, sync_generators_1.syncGeneratorResultsToMessageLines)(results),
+            title: 'The workspace is out of sync',
+            bodyLines: resultBodyLines,
         });
         const spinner = ora('Syncing the workspace...');
         spinner.start();
-        await (0, sync_generators_1.flushSyncGeneratorChanges)(results);
-        spinner.succeed(`The workspace was synced successfully!
-
-Please make sure to commit the changes to your repository.
-`);
+        try {
+            const flushResult = await (0, sync_generators_1.flushSyncGeneratorChanges)(results);
+            if ('generatorFailures' in flushResult) {
+                spinner.fail();
+                output_1.output.error({
+                    title: 'Failed to sync the workspace',
+                    bodyLines: (0, sync_generators_1.getFlushFailureMessageLines)(flushResult, options.verbose),
+                });
+                return 1;
+            }
+        }
+        catch (e) {
+            spinner.fail();
+            output_1.output.error({
+                title: 'Failed to sync the workspace',
+                bodyLines: [
+                    'Syncing the workspace failed with the following error:',
+                    '',
+                    e.message,
+                    ...(options.verbose && !!e.stack ? [`\n${e.stack}`] : []),
+                    '',
+                    'Please rerun with `--verbose` and report the error at: https://github.com/nrwl/nx/issues/new/choose',
+                ],
+            });
+            return 1;
+        }
+        const successTitle = anySyncGeneratorsFailed
+            ? // the identified changes were synced successfully, but the workspace
+                // is still not up to date, which we'll mention next
+                'The identified changes were synced successfully!'
+            : // the workspace is fully up to date
+                'The workspace was synced successfully!';
+        const successSubtitle = 'Please make sure to commit the changes to your repository.';
+        spinner.succeed(`${successTitle}\n\n${successSubtitle}`);
+        if (anySyncGeneratorsFailed) {
+            output_1.output.error({
+                title: `The workspace is probably still out of sync because ${failedGeneratorsCount === 1
+                    ? 'a sync generator'
+                    : 'some sync generators'} failed to run`,
+                bodyLines: failedSyncGeneratorsFixMessageLines,
+            });
+            return 1;
+        }
         return 0;
     });
 }

package/src/daemon/client/client.d.ts

@@ -4,7 +4,7 @@ import { Hash } from '../../hasher/task-hasher';
 import { Task, TaskGraph } from '../../config/task-graph';
 import { ConfigurationSourceMaps } from '../../project-graph/utils/project-configuration-utils';
 import { NxWorkspaceFiles, TaskRun } from '../../native';
-import type { SyncGeneratorChangesResult } from '../../utils/sync-generators';
+import type { FlushSyncGeneratorChangesResult, SyncGeneratorRunResult } from '../../utils/sync-generators';
 export type UnregisterCallback = () => void;
 export type ChangedFile = {
     path: string;
@@ -52,8 +52,8 @@ export declare class DaemonClient {
     hashGlob(globs: string[], exclude?: string[]): Promise<string>;
     getFlakyTasks(hashes: string[]): Promise<string[]>;
     recordTaskRuns(taskRuns: TaskRun[]): Promise<void>;
-    getSyncGeneratorChanges(generators: string[]): Promise<SyncGeneratorChangesResult[]>;
-    flushSyncGeneratorChangesToDisk(generators: string[]): Promise<void>;
+    getSyncGeneratorChanges(generators: string[]): Promise<SyncGeneratorRunResult[]>;
+    flushSyncGeneratorChangesToDisk(generators: string[]): Promise<FlushSyncGeneratorChangesResult>;
     getRegisteredSyncGenerators(): Promise<string[]>;
     updateWorkspaceContext(createdFiles: string[], updatedFiles: string[], deletedFiles: string[]): Promise<void>;
     isServerAvailable(): Promise<boolean>;

package/src/daemon/server/handle-flush-sync-generator-changes-to-disk.js

@@ -3,9 +3,9 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.handleFlushSyncGeneratorChangesToDisk = handleFlushSyncGeneratorChangesToDisk;
 const sync_generators_1 = require("./sync-generators");
 async function handleFlushSyncGeneratorChangesToDisk(generators) {
-    await (0, sync_generators_1.flushSyncGeneratorChangesToDisk)(generators);
+    const result = await (0, sync_generators_1.flushSyncGeneratorChangesToDisk)(generators);
     return {
-        response: '{}',
+        response: JSON.stringify(result),
         description: 'handleFlushSyncGeneratorChangesToDisk',
     };
 }

package/src/daemon/server/handle-get-sync-generator-changes.js

@@ -4,12 +4,14 @@ exports.handleGetSyncGeneratorChanges = handleGetSyncGeneratorChanges;
 const sync_generators_1 = require("./sync-generators");
 async function handleGetSyncGeneratorChanges(generators) {
     const changes = await (0, sync_generators_1.getCachedSyncGeneratorChanges)(generators);
-    // strip out the content of the changes and any potential callback
-    const result = changes.map((change) => ({
-        generatorName: change.generatorName,
-        changes: change.changes.map((c) => ({ ...c, content: null })),
-        outOfSyncMessage: change.outOfSyncMessage,
-    }));
+    const result = changes.map((change) => 'error' in change
+        ? change
+        : // strip out the content of the changes and any potential callback
+            {
+                generatorName: change.generatorName,
+                changes: change.changes.map((c) => ({ ...c, content: null })),
+                outOfSyncMessage: change.outOfSyncMessage,
+            });
     return {
         response: JSON.stringify(result),
         description: 'handleGetSyncGeneratorChanges',

package/src/daemon/server/sync-generators.d.ts

@@ -1,10 +1,10 @@
 import type { ProjectGraph } from '../../config/project-graph';
-import { type SyncGeneratorChangesResult } from '../../utils/sync-generators';
-export declare function getCachedSyncGeneratorChanges(generators: string[]): Promise<SyncGeneratorChangesResult[]>;
-export declare function flushSyncGeneratorChangesToDisk(generators: string[]): Promise<void>;
+import { type FlushSyncGeneratorChangesResult, type SyncGeneratorRunResult } from '../../utils/sync-generators';
+export declare function getCachedSyncGeneratorChanges(generators: string[]): Promise<SyncGeneratorRunResult[]>;
+export declare function flushSyncGeneratorChangesToDisk(generators: string[]): Promise<FlushSyncGeneratorChangesResult>;
 export declare function collectAndScheduleSyncGenerators(projectGraph: ProjectGraph): void;
 export declare function getCachedRegisteredSyncGenerators(): Promise<string[]>;
 /**
  * @internal
  */
-export declare function _getConflictingGeneratorGroups(results: SyncGeneratorChangesResult[]): string[][];
+export declare function _getConflictingGeneratorGroups(results: SyncGeneratorRunResult[]): string[][];

package/src/daemon/server/sync-generators.js

@@ -59,7 +59,7 @@ async function flushSyncGeneratorChangesToDisk(generators) {
     for (const generator of generators) {
         syncGeneratorsCacheResultPromises.delete(generator);
     }
-    await (0, sync_generators_1.flushSyncGeneratorChanges)(results);
+    return await (0, sync_generators_1.flushSyncGeneratorChanges)(results);
 }
 function collectAndScheduleSyncGenerators(projectGraph) {
     if (!projectGraph) {
@@ -191,6 +191,7 @@ async function processConflictingGenerators(conflicts, initialResults) {
     const conflictRunResults = (await Promise.all(conflicts.map((generators) => {
         const [firstGenerator, ...generatorsToRun] = generators;
         // it must exists because the conflicts were identified from the initial results
+        // and it's guaranteed to be a success result
         const firstGeneratorResult = initialResults.find((r) => r.generatorName === firstGenerator);
         const tree = new tree_1.FsTree(workspace_root_1.workspaceRoot, false, `running sync generators ${generators.join(',')}`);
         // pre-apply the changes from the first generator to avoid running it
@@ -234,6 +235,9 @@ async function processConflictingGenerators(conflicts, initialResults) {
 function _getConflictingGeneratorGroups(results) {
     const changedFileToGeneratorMap = new Map();
     for (const result of results) {
+        if ('error' in result) {
+            continue;
+        }
         for (const change of result.changes) {
             if (!changedFileToGeneratorMap.has(change.path)) {
                 changedFileToGeneratorMap.set(change.path, new Set());
@@ -318,7 +322,12 @@ function runGenerator(generator, projects, tree) {
     scheduledGenerators.delete(generator);
     tree ??= new tree_1.FsTree(workspace_root_1.workspaceRoot, false, `running sync generator ${generator}`);
     return (0, sync_generators_1.runSyncGenerator)(tree, generator, projects).then((result) => {
-        log(generator, 'changes:', result.changes.map((c) => c.path).join(', '));
+        if ('error' in result) {
+            log(generator, 'error:', result.error.message);
+        }
+        else {
+            log(generator, 'changes:', result.changes.map((c) => c.path).join(', '));
+        }
         return result;
     });
 }

package/src/nx-cloud/generators/connect-to-nx-cloud/connect-to-nx-cloud.js

@@ -89,7 +89,6 @@ async function connectToNxCloud(tree, schema, nxJson = (0, nx_json_1.readNxJson)
         return null;
     }
     const isGitHubDetected = schema.github ?? (await (0, url_shorten_1.repoUsesGithub)(schema.github));
-    let responseFromCreateNxCloudWorkspaceV1;
     let responseFromCreateNxCloudWorkspaceV2;
     /**
      * Do not create an Nx Cloud token if the user is using GitHub and

package/src/nx-cloud/utilities/get-cloud-options.d.ts

@@ -2,3 +2,4 @@ import { CloudTaskRunnerOptions } from '../nx-cloud-tasks-runner-shell';
 export declare function getCloudOptions(directory?: string): CloudTaskRunnerOptions;
 export declare function getCloudUrl(): string;
 export declare function removeTrailingSlash(apiUrl: string): string;
+export declare function isNxCloudId(token: string): boolean;

package/src/nx-cloud/utilities/get-cloud-options.js

@@ -3,6 +3,7 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.getCloudOptions = getCloudOptions;
 exports.getCloudUrl = getCloudUrl;
 exports.removeTrailingSlash = removeTrailingSlash;
+exports.isNxCloudId = isNxCloudId;
 const nx_json_1 = require("../../config/nx-json");
 const run_command_1 = require("../../tasks-runner/run-command");
 const workspace_root_1 = require("../../utils/workspace-root");
@@ -17,3 +18,6 @@ function getCloudUrl() {
 function removeTrailingSlash(apiUrl) {
     return apiUrl[apiUrl.length - 1] === '/' ? apiUrl.slice(0, -1) : apiUrl;
 }
+function isNxCloudId(token) {
+    return token.length === 24;
+}

package/src/nx-cloud/utilities/is-workspace-claimed.d.ts

@@ -1 +1 @@
-export declare function isWorkspaceClaimed(nxCloudAccessToken: any): Promise<any>;
+export declare function isWorkspaceClaimed(accessToken: string): Promise<any>;

package/src/nx-cloud/utilities/is-workspace-claimed.js

@@ -2,14 +2,15 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.isWorkspaceClaimed = isWorkspaceClaimed;
 const get_cloud_options_1 = require("./get-cloud-options");
-async function isWorkspaceClaimed(nxCloudAccessToken) {
-    if (!nxCloudAccessToken)
+async function isWorkspaceClaimed(accessToken) {
+    if (!accessToken)
         return false;
     const apiUrl = (0, get_cloud_options_1.getCloudUrl)();
     try {
-        const response = await require('axios').post(`${apiUrl}/nx-cloud/is-workspace-claimed`, {
-            nxCloudAccessToken,
-        });
+        const requestData = (0, get_cloud_options_1.isNxCloudId)(accessToken)
+            ? { nxCloudId: accessToken }
+            : { nxCloudAccessToken: accessToken };
+        const response = await require('axios').post(`${apiUrl}/nx-cloud/is-workspace-claimed`, requestData);
         if (response.data.message) {
             return false;
         }

package/src/nx-cloud/utilities/onboarding.js

@@ -23,6 +23,6 @@ async function getNxCloudAppOnBoardingUrl(token) {
 }
 function readNxCloudToken(tree) {
     const nxJson = (0, devkit_exports_1.readNxJson)(tree);
-    const { accessToken } = (0, run_command_1.getRunnerOptions)('default', nxJson, {}, true);
-    return accessToken;
+    const { accessToken, nxCloudId } = (0, run_command_1.getRunnerOptions)('default', nxJson, {}, true);
+    return accessToken || nxCloudId;
 }

package/src/project-graph/error-types.d.ts

@@ -18,7 +18,7 @@ export declare class ProjectGraphError extends Error {
      */
     getPartialProjectGraph(): ProjectGraph;
     getPartialSourcemaps(): ConfigurationSourceMaps;
-    getErrors(): (AggregateCreateNodesError | MergeNodesError | CreateMetadataError | ProjectsWithNoNameError | MultipleProjectsWithSameNameError | ProcessDependenciesError | ProcessProjectGraphError | WorkspaceValidityError)[];
+    getErrors(): (AggregateCreateNodesError | MergeNodesError | ProjectsWithNoNameError | MultipleProjectsWithSameNameError | CreateMetadataError | ProcessDependenciesError | ProcessProjectGraphError | WorkspaceValidityError)[];
 }
 export declare class MultipleProjectsWithSameNameError extends Error {
     conflicts: Map<string, string[]>;

package/src/project-graph/error-types.js

@@ -24,7 +24,7 @@ class ProjectGraphError extends Error {
         tslib_1.__classPrivateFieldSet(this, _ProjectGraphError_partialProjectGraph, partialProjectGraph, "f");
         tslib_1.__classPrivateFieldSet(this, _ProjectGraphError_partialSourceMaps, partialSourceMaps, "f");
         this.stack = `${this.message}\n  ${errors
-            .map((error) => error.stack.split('\n').join('\n  '))
+            .map((error) => indentString(formatErrorStackAndCause(error), 2))
             .join('\n')}`;
     }
     /**
@@ -186,13 +186,13 @@ class MergeNodesError extends Error {
         this.name = this.constructor.name;
         this.file = file;
         this.pluginName = pluginName;
-        this.stack = `${this.message}\n  ${error.stack.split('\n').join('\n  ')}`;
+        this.stack = `${this.message}\n${indentString(formatErrorStackAndCause(error), 2)}`;
     }
 }
 exports.MergeNodesError = MergeNodesError;
 class CreateMetadataError extends Error {
     constructor(error, plugin) {
-        super(`The "${plugin}" plugin threw an error while creating metadata:`, {
+        super(`The "${plugin}" plugin threw an error while creating metadata: ${error.message}`, {
             cause: error,
         });
         this.error = error;
@@ -203,7 +203,7 @@ class CreateMetadataError extends Error {
 exports.CreateMetadataError = CreateMetadataError;
 class ProcessDependenciesError extends Error {
     constructor(pluginName, { cause }) {
-        super(`The "${pluginName}" plugin threw an error while creating dependencies:`, {
+        super(`The "${pluginName}" plugin threw an error while creating dependencies: ${cause.message}`, {
             cause,
         });
         this.pluginName = pluginName;
@@ -234,7 +234,7 @@ class ProcessProjectGraphError extends Error {
         });
         this.pluginName = pluginName;
         this.name = this.constructor.name;
-        this.stack = `${this.message}\n  ${cause.stack.split('\n').join('\n  ')}`;
+        this.stack = `${this.message}\n${indentString(cause, 2)}`;
     }
 }
 exports.ProcessProjectGraphError = ProcessProjectGraphError;
@@ -289,3 +289,17 @@ class LoadPluginError extends Error {
     }
 }
 exports.LoadPluginError = LoadPluginError;
+function indentString(str, indent) {
+    return (' '.repeat(indent) +
+        str
+            .split('\n')
+            .map((line) => ' '.repeat(indent) + line)
+            .join('\n'));
+}
+function formatErrorStackAndCause(error) {
+    const cause = error.cause && error.cause instanceof Error ? error.cause : null;
+    return (error.stack +
+        (cause
+            ? `\nCaused by: \n${indentString(cause.stack ?? cause.message, 2)}`
+            : ''));
+}

package/src/tasks-runner/run-command.js

@@ -11,12 +11,13 @@ const nx_json_1 = require("../config/nx-json");
 const client_1 = require("../daemon/client/client");
 const create_task_hasher_1 = require("../hasher/create-task-hasher");
 const hash_task_1 = require("../hasher/hash-task");
+const native_1 = require("../native");
 const project_graph_1 = require("../project-graph/project-graph");
 const fileutils_1 = require("../utils/fileutils");
 const is_ci_1 = require("../utils/is-ci");
 const nx_cloud_utils_1 = require("../utils/nx-cloud-utils");
 const output_1 = require("../utils/output");
-const params_1 = require("../utils/params");
+const handle_errors_1 = require("../utils/handle-errors");
 const sync_generators_1 = require("../utils/sync-generators");
 const workspace_root_1 = require("../utils/workspace-root");
 const create_task_graph_1 = require("./create-task-graph");
@@ -33,7 +34,6 @@ const task_timings_life_cycle_1 = require("./life-cycles/task-timings-life-cycle
 const task_graph_utils_1 = require("./task-graph-utils");
 const utils_1 = require("./utils");
 const chalk = require("chalk");
-const native_1 = require("../native");
 async function getTerminalOutputLifeCycle(initiatingProject, projectNames, tasks, nxArgs, nxJson, overrides) {
     const { runnerOptions } = getRunner(nxArgs, nxJson);
     const isRunOne = initiatingProject != null;
@@ -98,7 +98,7 @@ function createTaskGraphAndRunValidations(projectGraph, extraTargetDependencies,
     return taskGraph;
 }
 async function runCommand(projectsToRun, currentProjectGraph, { nxJson }, nxArgs, overrides, initiatingProject, extraTargetDependencies, extraOptions) {
-    const status = await (0, params_1.handleErrors)(process.env.NX_VERBOSE_LOGGING === 'true', async () => {
+    const status = await (0, handle_errors_1.handleErrors)(process.env.NX_VERBOSE_LOGGING === 'true', async () => {
         const projectNames = projectsToRun.map((t) => t.name);
         const { projectGraph, taskGraph } = await ensureWorkspaceIsInSyncAndGetGraphs(currentProjectGraph, nxJson, projectNames, nxArgs, overrides, extraTargetDependencies, extraOptions);
         const tasks = Object.values(taskGraph.tasks);
@@ -135,14 +135,49 @@ async function ensureWorkspaceIsInSyncAndGetGraphs(projectGraph, nxJson, project
         // There are no changes to sync, workspace is up to date
         return { projectGraph, taskGraph };
     }
+    const { failedGeneratorsCount, areAllResultsFailures, anySyncGeneratorsFailed, } = (0, sync_generators_1.processSyncGeneratorResultErrors)(results);
+    const failedSyncGeneratorsFixMessageLines = (0, sync_generators_1.getFailedSyncGeneratorsFixMessageLines)(results, nxArgs.verbose);
     const outOfSyncTitle = 'The workspace is out of sync';
-    const resultBodyLines = [...(0, sync_generators_1.syncGeneratorResultsToMessageLines)(results), ''];
-    const fixMessage = 'You can manually run `nx sync` to update your workspace or you can set `sync.applyChanges` to `true` in your `nx.json` to apply the changes automatically when running tasks in interactive environments.';
-    const willErrorOnCiMessage = 'Please note that this will be an error on CI.';
+    const resultBodyLines = (0, sync_generators_1.getSyncGeneratorSuccessResultsMessageLines)(results);
+    const fixMessage = 'You can manually run `nx sync` to update your workspace with the identified changes or you can set `sync.applyChanges` to `true` in your `nx.json` to apply the changes automatically when running tasks in interactive environments.';
+    const willErrorOnCiMessage = 'Please note that having the workspace out of sync will result in an error in CI.';
     if ((0, is_ci_1.isCI)() || !process.stdout.isTTY) {
         // If the user is running in CI or is running in a non-TTY environment we
         // throw an error to stop the execution of the tasks.
-        throw new Error(`${outOfSyncTitle}\n${resultBodyLines.join('\n')}\n${fixMessage}`);
+        if (areAllResultsFailures) {
+            output_1.output.error({
+                title: `The workspace is probably out of sync because ${failedGeneratorsCount === 1
+                    ? 'a sync generator'
+                    : 'some sync generators'} failed to run`,
+                bodyLines: failedSyncGeneratorsFixMessageLines,
+            });
+        }
+        else {
+            output_1.output.error({
+                title: outOfSyncTitle,
+                bodyLines: [...resultBodyLines, '', fixMessage],
+            });
+            if (anySyncGeneratorsFailed) {
+                output_1.output.error({
+                    title: failedGeneratorsCount === 1
+                        ? 'A sync generator failed to run'
+                        : 'Some sync generators failed to run',
+                    bodyLines: failedSyncGeneratorsFixMessageLines,
+                });
+            }
+        }
+        process.exit(1);
+    }
+    if (areAllResultsFailures) {
+        output_1.output.warn({
+            title: `The workspace is probably out of sync because ${failedGeneratorsCount === 1
+                ? 'a sync generator'
+                : 'some sync generators'} failed to run`,
+            bodyLines: failedSyncGeneratorsFixMessageLines,
+        });
+        await confirmRunningTasksWithSyncFailures();
+        // if all sync generators failed to run there's nothing to sync, we just let the tasks run
+        return { projectGraph, taskGraph };
     }
     if (nxJson.sync?.applyChanges === false) {
         // If the user has set `sync.applyChanges` to `false` in their `nx.json`
@@ -153,19 +188,30 @@ async function ensureWorkspaceIsInSyncAndGetGraphs(projectGraph, nxJson, project
             title: outOfSyncTitle,
             bodyLines: [
                 ...resultBodyLines,
-                'Your workspace is set to not apply changes automatically (`sync.applyChanges` is set to `false` in your `nx.json`).',
+                '',
+                'Your workspace is set to not apply the identified changes automatically (`sync.applyChanges` is set to `false` in your `nx.json`).',
                 willErrorOnCiMessage,
                 fixMessage,
             ],
         });
+        if (anySyncGeneratorsFailed) {
+            output_1.output.warn({
+                title: failedGeneratorsCount === 1
+                    ? 'A sync generator failed to run'
+                    : 'Some sync generators failed to run',
+                bodyLines: failedSyncGeneratorsFixMessageLines,
+            });
+            await confirmRunningTasksWithSyncFailures();
+        }
         return { projectGraph, taskGraph };
     }
     output_1.output.warn({
         title: outOfSyncTitle,
         bodyLines: [
             ...resultBodyLines,
+            '',
             nxJson.sync?.applyChanges === true
-                ? 'Proceeding to sync the changes automatically (`sync.applyChanges` is set to `true` in your `nx.json`).'
+                ? 'Proceeding to sync the identified changes automatically (`sync.applyChanges` is set to `true` in your `nx.json`).'
                 : willErrorOnCiMessage,
         ],
     });
@@ -175,31 +221,66 @@ async function ensureWorkspaceIsInSyncAndGetGraphs(projectGraph, nxJson, project
         const spinner = ora('Syncing the workspace...');
         spinner.start();
         // Flush sync generator changes to disk
-        await (0, sync_generators_1.flushSyncGeneratorChanges)(results);
+        const flushResult = await (0, sync_generators_1.flushSyncGeneratorChanges)(results);
+        if ('generatorFailures' in flushResult) {
+            spinner.fail();
+            output_1.output.error({
+                title: 'Failed to sync the workspace',
+                bodyLines: [
+                    ...(0, sync_generators_1.getFlushFailureMessageLines)(flushResult, nxArgs.verbose),
+                    ...(flushResult.generalFailure
+                        ? [
+                            'If needed, you can run the tasks with the `--skip-sync` flag to disable syncing.',
+                        ]
+                        : []),
+                ],
+            });
+            await confirmRunningTasksWithSyncFailures();
+        }
         // Re-create project graph and task graph
         projectGraph = await (0, project_graph_1.createProjectGraphAsync)();
         taskGraph = createTaskGraphAndRunValidations(projectGraph, extraTargetDependencies ?? {}, projectNames, nxArgs, overrides, extraOptions);
-        if (nxJson.sync?.applyChanges === true) {
-            spinner.succeed(`The workspace was synced successfully!
-
-Please make sure to commit the changes to your repository or this will error on CI.`);
-        }
-        else {
-            // The user was prompted and we already logged a message about erroring on CI
-            // so here we just tell them to commit the changes.
-            spinner.succeed(`The workspace was synced successfully!
-
-Please make sure to commit the changes to your repository.`);
+        const successTitle = anySyncGeneratorsFailed
+            ? // the identified changes were synced successfully, but the workspace
+                // is still not up to date, which we'll mention next
+                'The identified changes were synced successfully!'
+            : // the workspace is fully up to date
+                'The workspace was synced successfully!';
+        const successSubtitle = nxJson.sync?.applyChanges === true
+            ? 'Please make sure to commit the changes to your repository or this will error in CI.'
+            : // The user was prompted and we already logged a message about erroring in CI
+                // so here we just tell them to commit the changes.
+                'Please make sure to commit the changes to your repository.';
+        spinner.succeed(`${successTitle}\n\n${successSubtitle}`);
+        if (anySyncGeneratorsFailed) {
+            output_1.output.warn({
+                title: `The workspace is probably still out of sync because ${failedGeneratorsCount === 1
+                    ? 'a sync generator'
+                    : 'some sync generators'} failed to run`,
+                bodyLines: failedSyncGeneratorsFixMessageLines,
+            });
+            await confirmRunningTasksWithSyncFailures();
         }
     }
     else {
-        output_1.output.warn({
-            title: 'Syncing the workspace was skipped',
-            bodyLines: [
-                'This could lead to unexpected results or errors when running tasks.',
-                fixMessage,
-            ],
-        });
+        if (anySyncGeneratorsFailed) {
+            output_1.output.warn({
+                title: failedGeneratorsCount === 1
+                    ? 'A sync generator failed to report the sync status'
+                    : 'Some sync generators failed to report the sync status',
+                bodyLines: failedSyncGeneratorsFixMessageLines,
+            });
+            await confirmRunningTasksWithSyncFailures();
+        }
+        else {
+            output_1.output.warn({
+                title: 'Syncing the workspace was skipped',
+                bodyLines: [
+                    'This could lead to unexpected results or errors when running tasks.',
+                    fixMessage,
+                ],
+            });
+        }
     }
     return { projectGraph, taskGraph };
 }
@@ -208,7 +289,7 @@ async function promptForApplyingSyncGeneratorChanges() {
         const promptConfig = {
             name: 'applyChanges',
             type: 'select',
-            message: 'Would you like to sync the changes to get your worskpace up to date?',
+            message: 'Would you like to sync the identified changes to get your worskpace up to date?',
             choices: [
                 {
                     name: 'yes',
@@ -227,6 +308,35 @@ async function promptForApplyingSyncGeneratorChanges() {
         process.exit(1);
     }
 }
+async function confirmRunningTasksWithSyncFailures() {
+    try {
+        const promptConfig = {
+            name: 'runTasks',
+            type: 'select',
+            message: 'Would you like to ignore the sync failures and continue running the tasks?',
+            choices: [
+                {
+                    name: 'yes',
+                    message: 'Yes, ignore the failures and run the tasks',
+                },
+                {
+                    name: 'no',
+                    message: `No, don't run the tasks`,
+                },
+            ],
+            footer: () => chalk.dim(`\nWhen running in CI and there are sync failures, the tasks won't run. Addressing the errors above is highly recommended to prevent failures in CI.`),
+        };
+        const runTasks = await (0, enquirer_1.prompt)([
+            promptConfig,
+        ]).then(({ runTasks }) => runTasks === 'yes');
+        if (!runTasks) {
+            process.exit(1);
+        }
+    }
+    catch {
+        process.exit(1);
+    }
+}
 function setEnvVarsBasedOnArgs(nxArgs, loadDotEnvFiles) {
     if (nxArgs.outputStyle == 'stream' ||
         process.env.NX_BATCH_MODE === 'true' ||

package/src/utils/handle-errors.d.ts

@@ -0,0 +1 @@
+export declare function handleErrors(isVerbose: boolean, fn: Function): Promise<number>;