feat(nest): Add strict option for nest applications and default stric…

…t option for libraries closes: #12407
nrwl · Apr 18, 2023 · d20d548 · d20d548
1 parent 5769256
commit d20d548
Show file tree

Hide file tree

Showing 11 changed files with 51 additions and 15 deletions.
diff --git a/docs/generated/packages/nest/generators/application.json b/docs/generated/packages/nest/generators/application.json
@@ -68,6 +68,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
+      },
+      "strict": {
+        "type": "boolean",
+        "description": "Adds strictNullChecks, noImplicitAny, strictBindCallApply, forceConsistentCasingInFileNames and noFallthroughCasesInSwitch to tsconfig.",
+        "default": false
       }
     },
     "additionalProperties": false,

diff --git a/docs/generated/packages/nest/generators/library.json b/docs/generated/packages/nest/generators/library.json
@@ -110,7 +110,7 @@
       "strict": {
         "description": "Whether to enable tsconfig strict mode or not.",
         "type": "boolean",
-        "default": false
+        "default": true
       },
       "standaloneConfig": {
         "description": "Split the project configuration into <projectRoot>/project.json rather than including it inside workspace.json",

diff --git a/packages/nest/src/generators/application/application.spec.ts b/packages/nest/src/generators/application/application.spec.ts
@@ -59,6 +59,22 @@ describe('application generator', () => {
     ]);
   });
 
+  it('should add strict checks with --strict', async () => {
+    await applicationGenerator(tree, { name: appName, strict: true });
+    const tsConfig = devkit.readJson(
+      tree,
+      `apps/${appDirectory}/tsconfig.app.json`
+    );
+
+    expect(tsConfig.compilerOptions.strictNullChecks).toBeTruthy();
+    expect(tsConfig.compilerOptions.noImplicitAny).toBeTruthy();
+    expect(tsConfig.compilerOptions.strictBindCallApply).toBeTruthy();
+    expect(
+      tsConfig.compilerOptions.forceConsistentCasingInFileNames
+    ).toBeTruthy();
+    expect(tsConfig.compilerOptions.noFallthroughCasesInSwitch).toBeTruthy();
+  });
+
   describe('--skipFormat', () => {
     it('should format files', async () => {
       jest.spyOn(devkit, 'formatFiles');

diff --git a/packages/nest/src/generators/application/lib/normalize-options.ts b/packages/nest/src/generators/application/lib/normalize-options.ts
@@ -25,6 +25,7 @@ export function normalizeOptions(
 
   return {
     ...options,
+    strict: options.strict ?? false,
     appProjectRoot,
     linter: options.linter ?? Linter.EsLint,
     unitTestRunner: options.unitTestRunner ?? 'jest',

diff --git a/packages/nest/src/generators/application/lib/update-tsconfig.ts b/packages/nest/src/generators/application/lib/update-tsconfig.ts
@@ -9,6 +9,16 @@ export function updateTsConfig(tree: Tree, options: NormalizedOptions): void {
     (json) => {
       json.compilerOptions.emitDecoratorMetadata = true;
       json.compilerOptions.target = 'es2015';
+      if (options.strict) {
+        json.compilerOptions = {
+          ...json.compilerOptions,
+          strictNullChecks: true,
+          noImplicitAny: true,
+          strictBindCallApply: true,
+          forceConsistentCasingInFileNames: true,
+          noFallthroughCasesInSwitch: true,
+        };
+      }
       return json;
     }
   );

diff --git a/packages/nest/src/generators/application/schema.d.ts b/packages/nest/src/generators/application/schema.d.ts
@@ -13,6 +13,7 @@ export interface ApplicationGeneratorOptions {
   e2eTestRunner?: 'jest' | 'none';
   setParserOptionsProject?: boolean;
   rootProject?: boolean;
+  strict?: boolean;
 }
 
 interface NormalizedOptions extends ApplicationGeneratorOptions {

diff --git a/packages/nest/src/generators/application/schema.json b/packages/nest/src/generators/application/schema.json
@@ -68,6 +68,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
+    },
+    "strict": {
+      "type": "boolean",
+      "description": "Adds strictNullChecks, noImplicitAny, strictBindCallApply, forceConsistentCasingInFileNames and noFallthroughCasesInSwitch to tsconfig.",
+      "default": false
     }
   },
   "additionalProperties": false,

diff --git a/packages/nest/src/generators/library/lib/normalize-options.ts b/packages/nest/src/generators/library/lib/normalize-options.ts
@@ -28,6 +28,7 @@ export function normalizeOptions(
 
   const normalized: NormalizedOptions = {
     ...options,
+    strict: options.strict ?? true,
     controller: options.controller ?? false,
     fileName,
     global: options.global ?? false,

diff --git a/packages/nest/src/generators/library/lib/update-tsconfig.ts b/packages/nest/src/generators/library/lib/update-tsconfig.ts
@@ -10,9 +10,10 @@ export function updateTsConfig(tree: Tree, options: NormalizedOptions): void {
     if (options.strict) {
       json.compilerOptions = {
         ...json.compilerOptions,
+        strictNullChecks: true,
+        noImplicitAny: true,
+        strictBindCallApply: true,
         forceConsistentCasingInFileNames: true,
-        strict: true,
-        noImplicitReturns: true,
         noFallthroughCasesInSwitch: true,
       };
     }

diff --git a/packages/nest/src/generators/library/library.spec.ts b/packages/nest/src/generators/library/library.spec.ts
@@ -254,18 +254,14 @@ describe('lib', () => {
     it('should update the projects tsconfig with strict true', async () => {
       await libraryGenerator(tree, { name: libName, strict: true });
 
-      const tsconfigJson = readJson(
-        tree,
-        `/libs/${libFileName}/tsconfig.lib.json`
-      );
-      expect(tsconfigJson.compilerOptions.strict).toBe(true);
+      const tsConfig = readJson(tree, `/libs/${libFileName}/tsconfig.lib.json`);
+      expect(tsConfig.compilerOptions.strictNullChecks).toBeTruthy();
+      expect(tsConfig.compilerOptions.noImplicitAny).toBeTruthy();
+      expect(tsConfig.compilerOptions.strictBindCallApply).toBeTruthy();
       expect(
-        tsconfigJson.compilerOptions.forceConsistentCasingInFileNames
-      ).toBe(true);
-      expect(tsconfigJson.compilerOptions.noImplicitReturns).toBe(true);
-      expect(tsconfigJson.compilerOptions.noFallthroughCasesInSwitch).toBe(
-        true
-      );
+        tsConfig.compilerOptions.forceConsistentCasingInFileNames
+      ).toBeTruthy();
+      expect(tsConfig.compilerOptions.noFallthroughCasesInSwitch).toBeTruthy();
     });
 
     it('should default to strict false', async () => {

diff --git a/packages/nest/src/generators/library/schema.json b/packages/nest/src/generators/library/schema.json
@@ -110,7 +110,7 @@
     "strict": {
       "description": "Whether to enable tsconfig strict mode or not.",
       "type": "boolean",
-      "default": false
+      "default": true
     },
     "standaloneConfig": {
       "description": "Split the project configuration into <projectRoot>/project.json rather than including it inside workspace.json",