fix: @typescript-eslint/no-non-null-assertion warnings (#746)

cybozu · Aug 3, 2022 · 9c055d9 · 9c055d9
1 parent 00f33c6
commit 9c055d9
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 5 deletions.
diff --git a/src/compiler.ts b/src/compiler.ts
@@ -256,7 +256,7 @@ export async function createCompilerOptionsForChunks(
     .map((inputs) => [...inputs])
     .flat();
   compilerOptions.chunk = sortedChunkIds.map((id) => {
-    const numOfInputs = chunkToInputPathSet.get(id)!.size;
+    const numOfInputs = assertNonNullable(chunkToInputPathSet.get(id)).size;
     return `${id}:${numOfInputs}:${chunks[id].deps.join(",")}`;
   });
   compilerOptions.chunk_wrapper = createChunkWrapper(

diff --git a/src/dag.ts b/src/dag.ts
@@ -1,4 +1,5 @@
 import Zet from "zet";
+import { assertNonNullable } from "./assert";
 
 export class Node {
   id: string;
@@ -53,7 +54,7 @@ export class Dag {
     // populate children (inverting deps)
     nodes.forEach((node) => {
       node.deps.forEach((dep) => {
-        this.idToNode.get(dep)!.children.add(node);
+        assertNonNullable(this.idToNode.get(dep)).children.add(node);
       });
     });
     this.populateDepth();
@@ -74,7 +75,10 @@ export class Dag {
     }
     ancestors.push(node.id);
     node.deps.forEach((dep) => {
-      this.populateAncestors(this.idToNode.get(dep)!, ancestors);
+      this.populateAncestors(
+        assertNonNullable(this.idToNode.get(dep)),
+        ancestors
+      );
     });
     return ancestors;
   }
@@ -150,15 +154,16 @@ export class Dag {
     for (const ancestor of commonAncestors) {
       if (
         least === null ||
-        this.idToDepth.get(least)! < this.idToDepth.get(ancestor)!
+        assertNonNullable(this.idToDepth.get(least)) <
+          assertNonNullable(this.idToDepth.get(ancestor))
       ) {
         least = ancestor;
       }
     }
     if (!least) {
       throw new Error("LCA not found");
     }
-    result = this.idToNode.get(least)!;
+    result = assertNonNullable(this.idToNode.get(least));
     this.setLcaCache(u, v, result);
     return result;
   }