fix(react-native): skipPackageJson was being ignored

docs/generated/packages/react-native.json

@@ -47,6 +47,11 @@
             "type": "boolean",
             "default": false,
             "description": "Use JavaScript instead of TypeScript"
+          },
+          "skipPackageJson": {
+            "description": "Do not add dependencies to `package.json`.",
+            "type": "boolean",
+            "default": false
           }
         },
         "required": [],
@@ -135,6 +140,11 @@
             "type": "boolean",
             "description": "Runs `pod install` for native modules before building iOS app.",
             "default": true
+          },
+          "skipPackageJson": {
+            "description": "Do not add dependencies to `package.json`.",
+            "type": "boolean",
+            "default": false
           }
         },
         "required": [],
@@ -240,6 +250,11 @@
             "type": "boolean",
             "description": "Whether or not to configure the ESLint `parserOptions.project` option. We do not do this by default for lint performance reasons.",
             "default": false
+          },
+          "skipPackageJson": {
+            "description": "Do not add dependencies to `package.json`.",
+            "type": "boolean",
+            "default": false
           }
         },
         "required": ["name"],

packages/detox/src/generators/init/init.ts

@@ -2,6 +2,7 @@ import {
   addDependenciesToPackageJson,
   convertNxGenerator,
   formatFiles,
+  GeneratorCallback,
   removeDependenciesFromPackageJson,
   Tree,
 } from '@nrwl/devkit';
@@ -15,7 +16,12 @@ import {
 } from '../../utils/versions';
 
 export async function detoxInitGenerator(host: Tree, schema: Schema) {
-  const tasks = [moveDependency(host), updateDependencies(host)];
+  const tasks: GeneratorCallback[] = [];
+
+  if (!schema.skipPackageJson) {
+    tasks.push(moveDependency(host));
+    tasks.push(updateDependencies(host));
+  }
 
   if (!schema.skipFormat) {
     await formatFiles(host);

packages/detox/src/generators/init/schema.d.ts

@@ -1,3 +1,4 @@
 export interface Schema {
   skipFormat?: boolean;
+  skipPackageJson?: boolean; //default is false
 }

packages/react-native/src/generators/application/application.spec.ts

@@ -189,4 +189,21 @@ describe('app', () => {
       });
     });
   });
+
+  describe('--skipPackageJson', () => {
+    it('should not add or update dependencies when true', async () => {
+      const packageJsonBefore = appTree.read('package.json', 'utf-8');
+
+      await reactNativeApplicationGenerator(appTree, {
+        name: 'myApp',
+        displayName: 'myApp',
+        linter: Linter.EsLint,
+        e2eTestRunner: 'none',
+        install: false,
+        skipPackageJson: true,
+      });
+
+      expect(appTree.read('package.json', 'utf-8')).toEqual(packageJsonBefore);
+    });
+  });
 });

packages/react-native/src/generators/application/application.ts

@@ -42,7 +42,8 @@ export async function reactNativeApplicationGenerator(
     options.unitTestRunner,
     options.projectName,
     options.appProjectRoot,
-    options.js
+    options.js,
+    options.skipPackageJson
   );
   const detoxTask = await addDetox(host, options);
   const symlinkTask = runSymlink(host.root, options.appProjectRoot);

packages/react-native/src/generators/application/schema.d.ts

@@ -15,4 +15,5 @@ export interface Schema {
   setParserOptionsProject?: boolean;
   e2eTestRunner?: 'detox' | 'none';
   install: boolean; // default is true
+  skipPackageJson?: boolean; //default is false
 }

packages/react-native/src/generators/application/schema.json

@@ -75,6 +75,11 @@
       "type": "boolean",
       "description": "Runs `pod install` for native modules before building iOS app.",
       "default": true
+    },
+    "skipPackageJson": {
+      "description": "Do not add dependencies to `package.json`.",
+      "type": "boolean",
+      "default": false
     }
   },
   "required": []

packages/react-native/src/generators/init/init.ts

@@ -3,6 +3,7 @@ import {
   convertNxGenerator,
   detectPackageManager,
   formatFiles,
+  GeneratorCallback,
   removeDependenciesFromPackageJson,
   Tree,
 } from '@nrwl/devkit';
@@ -42,7 +43,14 @@ export async function reactNativeInitGenerator(host: Tree, schema: Schema) {
   addGitIgnoreEntry(host);
   initRootBabelConfig(host);
 
-  const tasks = [moveDependency(host), updateDependencies(host)];
+  const tasks: GeneratorCallback[] = [];
+
+  if (!schema.skipPackageJson) {
+    const installTask = updateDependencies(host);
+
+    tasks.push(moveDependency(host));
+    tasks.push(installTask);
+  }
 
   if (!schema.unitTestRunner || schema.unitTestRunner === 'jest') {
     const jestTask = jestInitGenerator(host, schema);

packages/react-native/src/generators/init/schema.d.ts

@@ -2,5 +2,6 @@ export interface Schema {
   unitTestRunner?: 'jest' | 'none';
   skipFormat?: boolean;
   e2eTestRunner?: 'detox' | 'none';
+  skipPackageJson?: boolean; //default is false
   js?: boolean;
 }

packages/react-native/src/generators/init/schema.json

@@ -27,6 +27,11 @@
       "type": "boolean",
       "default": false,
       "description": "Use JavaScript instead of TypeScript"
+    },
+    "skipPackageJson": {
+      "description": "Do not add dependencies to `package.json`.",
+      "type": "boolean",
+      "default": false
     }
   },
   "required": []

packages/react-native/src/generators/library/library.spec.ts

@@ -381,4 +381,17 @@ describe('lib', () => {
       ).not.toBeDefined();
     });
   });
+
+  describe('--skipPackageJson', () => {
+    it('should not add or update dependencies when true', async () => {
+      const packageJsonBefore = appTree.read('package.json', 'utf-8');
+
+      await libraryGenerator(appTree, {
+        ...defaultSchema,
+        skipPackageJson: true,
+      });
+
+      expect(appTree.read('package.json', 'utf-8')).toEqual(packageJsonBefore);
+    });
+  });
 });

packages/react-native/src/generators/library/library.ts

@@ -61,7 +61,8 @@ export async function reactNativeLibraryGenerator(
     options.unitTestRunner,
     options.name,
     options.projectRoot,
-    options.js
+    options.js,
+    options.skipPackageJson
   );
 
   if (options.publishable || options.buildable) {

packages/react-native/src/generators/library/schema.d.ts

@@ -19,4 +19,5 @@ export interface Schema {
   globalCss?: boolean;
   strict?: boolean;
   setParserOptionsProject?: boolean;
+  skipPackageJson?: boolean; //default is false
 }

packages/react-native/src/generators/library/schema.json

@@ -91,6 +91,11 @@
       "type": "boolean",
       "description": "Whether or not to configure the ESLint `parserOptions.project` option. We do not do this by default for lint performance reasons.",
       "default": false
+    },
+    "skipPackageJson": {
+      "description": "Do not add dependencies to `package.json`.",
+      "type": "boolean",
+      "default": false
     }
   },
   "required": ["name"]

packages/react-native/src/utils/add-jest.ts

@@ -6,7 +6,8 @@ export async function addJest(
   unitTestRunner: 'jest' | 'none',
   projectName: string,
   appProjectRoot: string,
-  js: boolean
+  js: boolean,
+  skipPackageJson: boolean
 ) {
   if (unitTestRunner !== 'jest') {
     return () => {};
@@ -19,6 +20,7 @@ export async function addJest(
     skipSerializers: true,
     setupFile: 'none',
     compiler: 'babel',
+    skipPackageJson,
   });
 
   // overwrite the jest.config.ts file because react native needs to have special transform property

packages/react-native/src/utils/add-linting.ts

@@ -2,6 +2,7 @@ import { runTasksInSerial } from '@nrwl/workspace/src/utilities/run-tasks-in-ser
 import { Linter, lintProjectGenerator } from '@nrwl/linter';
 import {
   addDependenciesToPackageJson,
+  GeneratorCallback,
   joinPathFragments,
   Tree,
   updateJson,
@@ -15,21 +16,26 @@ interface NormalizedSchema {
   projectRoot: string;
   setParserOptionsProject?: boolean;
   tsConfigPaths: string[];
+  skipPackageJson?: boolean;
 }
 
 export async function addLinting(host: Tree, options: NormalizedSchema) {
   if (options.linter === Linter.None) {
     return () => {};
   }
+  const tasks: GeneratorCallback[] = [];
 
   const lintTask = await lintProjectGenerator(host, {
     linter: options.linter,
     project: options.projectName,
     tsConfigPaths: options.tsConfigPaths,
     eslintFilePatterns: [`${options.projectRoot}/**/*.{ts,tsx,js,jsx}`],
     skipFormat: true,
+    skipPackageJson: options.skipPackageJson,
   });
 
+  tasks.push(lintTask);
+
   updateJson(
     host,
     joinPathFragments(options.projectRoot, '.eslintrc.json'),
@@ -51,11 +57,14 @@ export async function addLinting(host: Tree, options: NormalizedSchema) {
     }
   );
 
-  const installTask = await addDependenciesToPackageJson(
-    host,
-    extraEslintDependencies.dependencies,
-    extraEslintDependencies.devDependencies
-  );
+  if (!options.skipPackageJson) {
+    const installTask = await addDependenciesToPackageJson(
+      host,
+      extraEslintDependencies.dependencies,
+      extraEslintDependencies.devDependencies
+    );
+    tasks.push(installTask);
+  }
 
-  return runTasksInSerial(lintTask, installTask);
+  return runTasksInSerial(...tasks);
 }