feat(associative): enable TS strict compiler flags (refactor)

packages/associative/src/api.ts

@@ -17,7 +17,7 @@ export interface IEquivSet<T>
         IGet<T, T>,
         IInto<T, IEquivSet<T>> {
     disj(xs: Iterable<T>): this;
-    first(): T;
+    first(): T | undefined;
 }
 
 export interface EquivSetConstructor<T> {

packages/associative/src/array-set.ts

@@ -30,7 +30,7 @@ export class ArraySet<T> extends Set<T> implements IEquivSet<T> {
         vals && this.into(vals);
     }
 
-    *[Symbol.iterator]() {
+    *[Symbol.iterator](): IterableIterator<T> {
         yield* __private.get(this).vals;
     }
 
@@ -42,7 +42,7 @@ export class ArraySet<T> extends Set<T> implements IEquivSet<T> {
         return "ArraySet";
     }
 
-    get size() {
+    get size(): number {
         return __private.get(this).vals.length;
     }
 
@@ -61,7 +61,7 @@ export class ArraySet<T> extends Set<T> implements IEquivSet<T> {
         __private.get(this).vals.length = 0;
     }
 
-    first() {
+    first(): T | undefined {
         if (this.size) {
             return __private.get(this).vals[0];
         }

packages/associative/src/equiv-map.ts

@@ -81,7 +81,7 @@ export class EquivMap<K, V> extends Map<K, V>
         return "EquivMap";
     }
 
-    get size() {
+    get size(): number {
         return __private.get(this).keys.size;
     }
 
@@ -91,7 +91,7 @@ export class EquivMap<K, V> extends Map<K, V>
         $this.map.clear();
     }
 
-    empty() {
+    empty(): EquivMap<K, V> {
         return new EquivMap<K, V>(null, __private.get(this).opts);
     }
 
@@ -157,7 +157,7 @@ export class EquivMap<K, V> extends Map<K, V>
         return notFound;
     }
 
-    has(key: K) {
+    has(key: K): boolean {
         return __private.get(this).keys.has(key);
     }
 

packages/associative/src/hash-map.ts

@@ -73,7 +73,7 @@ export class HashMap<K, V> extends Map<K, V>
         return "HashMap";
     }
 
-    get size() {
+    get size(): number {
         return __private.get(this).size;
     }
 
@@ -151,7 +151,7 @@ export class HashMap<K, V> extends Map<K, V>
         return i >= 0 && $this.bins[i] != undefined;
     }
 
-    get(key: K, notFound?: V) {
+    get(key: K, notFound?: V): V | undefined {
         const $this = __private.get(this);
         const i = this.find(key, $this);
         return i >= 0 && $this.bins[i] ? $this.bins[i][1] : notFound;

packages/associative/src/ll-set.ts

@@ -46,7 +46,7 @@ export class LLSet<T> extends Set<T> implements IEquivSet<T> {
         return "LLSet";
     }
 
-    get size() {
+    get size(): number {
         return __private.get(this).vals.length;
     }
 
@@ -65,7 +65,7 @@ export class LLSet<T> extends Set<T> implements IEquivSet<T> {
         __private.get(this).vals.clear();
     }
 
-    first() {
+    first(): T | undefined {
         if (this.size) {
             return __private.get(this).vals.head.value;
         }

packages/associative/src/merge-deep.ts

@@ -2,7 +2,10 @@ import { IObjectOf } from "@thi.ng/api";
 import { isPlainObject } from "@thi.ng/checks";
 import { mergeObjWith } from "./merge-with";
 
-export const mergeDeepObj = (dest: IObjectOf<any>, ...xs: IObjectOf<any>[]) =>
+export const mergeDeepObj = (
+    dest: IObjectOf<any>,
+    ...xs: IObjectOf<any>[]
+): any =>
     mergeObjWith(
         (a, b) =>
             isPlainObject(a) && isPlainObject(b) ? mergeDeepObj(a, b) : b,

packages/associative/src/select-keys.ts

@@ -31,7 +31,7 @@ export const selectKeysObj = <T>(
 ): { [id in keyof T]?: T[id] } => {
     const dest: any = {};
     for (let k of ks) {
-        src.hasOwnProperty(k) && (dest[k] = src[<any>k]);
+        src.hasOwnProperty(k) && (dest[k] = (<any>src)[<any>k]);
     }
     return dest;
 };

packages/associative/src/sorted-map.ts

@@ -129,7 +129,7 @@ export class SortedMap<K, V> extends Map<K, V> {
         return map((p) => p[1], this.entries(key, max));
     }
 
-    get size() {
+    get size(): number {
         return __private.get(this).length;
     }
 
@@ -189,7 +189,7 @@ export class SortedMap<K, V> extends Map<K, V> {
         return true;
     }
 
-    first(): Pair<K, V> {
+    first(): Pair<K, V> | undefined {
         const node = __private.get(this).head.next[0];
         return node ? [node.k, node.v] : undefined;
     }

packages/associative/src/sorted-set.ts

@@ -54,7 +54,7 @@ export class SortedSet<T> extends Set<T>
         return "SortedSet";
     }
 
-    get size() {
+    get size(): number {
         return __private.get(this).size;
     }
 
@@ -104,7 +104,7 @@ export class SortedSet<T> extends Set<T>
         return true;
     }
 
-    $reduce(rfn: ReductionFn<any, T>, acc: any) {
+    $reduce(rfn: ReductionFn<any, T>, acc: any): any {
         return __private.get(this).$reduce((_acc, x) => rfn(_acc, x[0]), acc);
     }
 
@@ -157,7 +157,7 @@ export class SortedSet<T> extends Set<T>
         thisArg?: any
     ): void {
         for (let p of this) {
-            fn.call(thisArg, p[0], p[0], this);
+            fn.call(thisArg, p, p, this);
         }
     }
 

packages/associative/src/sparse-set.ts

@@ -37,7 +37,7 @@ export abstract class ASparseSet<T extends UIntArray> extends Set<number>
         return this.keys();
     }
 
-    get size() {
+    get size(): number {
         return __private.get(this).n;
     }
 

packages/associative/src/utils.ts

@@ -1,12 +1,12 @@
 import { Pair } from "@thi.ng/api";
 import { implementsFunction, isMap, isSet } from "@thi.ng/checks";
 
-export const empty = (x, ctor) =>
+export const empty = (x: any, ctor: Function) =>
     implementsFunction(x, "empty")
         ? x.empty()
         : new (x[Symbol.species] || ctor)();
 
-export const copy = (x, ctor) =>
+export const copy = (x: any, ctor: Function) =>
     implementsFunction(x, "copy")
         ? x.copy()
         : new (x[Symbol.species] || ctor)(x);