fix(core): fix migration that enables source analysis

nrwl · Oct 14, 2022 · 900711e · 900711e
1 parent 59db301
commit 900711e
Showing 1 changed file with 18 additions and 17 deletions.
diff --git a/packages/workspace/src/migrations/update-14-2-0/enable-source-analysis.ts b/packages/workspace/src/migrations/update-14-2-0/enable-source-analysis.ts
@@ -1,28 +1,29 @@
 import {
   formatFiles,
-  readWorkspaceConfiguration,
+  NxJsonConfiguration,
   Tree,
-  updateWorkspaceConfiguration,
+  updateJson,
 } from '@nrwl/devkit';
 
 export async function enableSourceAnalysis(tree: Tree) {
-  const config = readWorkspaceConfiguration(tree);
-  if (
-    config.extends === 'nx/presets/core.json' ||
-    config.extends === 'nx/presets/npm.json'
-  ) {
-    const explicitlyDisabled =
-      config.pluginsConfig &&
-      config.pluginsConfig['@nrwl/js'] &&
-      (config.pluginsConfig['@nrwl/js'] as any).analyzeSourceFiles === false;
+  updateJson<NxJsonConfiguration>(tree, 'nx.json', (config) => {
+    if (
+      config.extends === 'nx/presets/core.json' ||
+      config.extends === 'nx/presets/npm.json'
+    ) {
+      const explicitlyDisabled =
+        config.pluginsConfig &&
+        config.pluginsConfig['@nrwl/js'] &&
+        (config.pluginsConfig['@nrwl/js'] as any).analyzeSourceFiles === false;
 
-    if (!explicitlyDisabled) {
-      config.pluginsConfig ||= {};
-      config.pluginsConfig['@nrwl/js'] ||= {};
-      (config.pluginsConfig['@nrwl/js'] as any).analyzeSourceFiles = true;
+      if (!explicitlyDisabled) {
+        config.pluginsConfig ||= {};
+        config.pluginsConfig['@nrwl/js'] ||= {};
+        (config.pluginsConfig['@nrwl/js'] as any).analyzeSourceFiles = true;
+      }
     }
-  }
-  updateWorkspaceConfiguration(tree, config);
+    return config;
+  });
   await formatFiles(tree);
 }