feat(nest): Add strict option

closes: #12407
nrwl · Apr 18, 2023 · 5a78206 · 5a78206
1 parent 5769256
commit 5a78206
Show file tree

Hide file tree

Showing 6 changed files with 38 additions and 0 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/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,