build: prep api, atom, bench, binary, checks, equiv, errors, paths fo…

…r multi outputs

- update tsconfig files for ES6 module output
- update build scripts in these 8 modules
- update atom: Cursor/View NEXT_ID, replace w/ nextID() helper fn
- update various tests
- update .*ignore files

.gitignore

@@ -1,4 +1,5 @@
 node_modules
+.cache
 .nyc_output
 coverage
 build

packages/api/.npmignore

@@ -1,9 +1,12 @@
+.cache
+.nyc_output
+*.gz
+*.html
+*.tgz
 build
 coverage
 dev
 doc
 src*
 test
-.nyc_output
-*.tgz
-*.html
+tsconfig.json

packages/api/package.json

@@ -2,7 +2,8 @@
     "name": "@thi.ng/api",
     "version": "4.2.4",
     "description": "Common, generic types & interfaces for thi.ng projects",
-    "main": "./index.js",
+    "main": "./lib/index.js",
+    "module": "./index.js",
     "typings": "./index.d.ts",
     "repository": {
         "type": "git",
@@ -12,12 +13,16 @@
     "author": "Karsten Schmidt <[email protected]>",
     "license": "Apache-2.0",
     "scripts": {
-        "build": "yarn clean && tsc --declaration",
-        "clean": "rimraf *.js *.d.ts .nyc_output build coverage doc decorators mixins",
+        "build": "yarn clean && yarn build:es6 && yarn build:parcel && yarn build:parcel:min",
+        "build:es6": "tsc --declaration",
+        "build:parcel:min": "parcel build --global thing_api -o index.min.js -d lib --experimental-scope-hoisting src/index.ts",
+        "build:parcel": "parcel build --global thing_api -d lib --no-minify --experimental-scope-hoisting src/index.ts && prettier --write lib/index.js",
+        "build:gzip": "gzip -c lib/index.min.js > lib/index.min.js.gz",
+        "clean": "rimraf *.js *.d.ts .cache .nyc_output build coverage doc lib",
+        "test": "rimraf build && parcel build -d build --no-cache --no-minify --no-source-maps test/index.ts && nyc mocha build/index.js",
         "cover": "yarn test && nyc report --reporter=lcov",
         "doc": "node_modules/.bin/typedoc --mode modules --out doc src",
-        "pub": "yarn build && yarn publish --access public",
-        "test": "rimraf build && tsc -p test && nyc mocha build/test/*.js"
+        "pub": "yarn build && yarn publish --access public"
     },
     "devDependencies": {
         "@types/mocha": "^5.2.5",
@@ -42,5 +47,12 @@
     ],
     "publishConfig": {
         "access": "public"
+    },
+    "browserslist": [
+        "since 2018-07"
+    ],
+    "browser": {
+        "process": false,
+        "setTimeout": false
     }
 }

packages/api/src/decorators/deprecated.ts

@@ -1,4 +1,4 @@
-import { illegalArgs } from "@thi.ng/errors";
+import { illegalArgs } from "@thi.ng/errors/illegal-arguments";
 
 /**
  * Method property decorator factory. Augments original method with

packages/api/test/index.ts

@@ -0,0 +1 @@
+export * from "./mixins";

packages/api/test/tsconfig.json

@@ -1,7 +1,9 @@
 {
     "extends": "../../../tsconfig.json",
     "compilerOptions": {
-        "outDir": "../build"
+        "outDir": "../build",
+        "module": "es6",
+        "target": "es6"
     },
     "include": [
         "./**/*.ts",

packages/api/tsconfig.json

@@ -1,7 +1,9 @@
 {
     "extends": "../../tsconfig.json",
     "compilerOptions": {
-        "outDir": "."
+        "outDir": ".",
+        "module": "es6",
+        "target": "es6"
     },
     "include": [
         "./src/**/*.ts"

packages/atom/.npmignore

@@ -1,10 +1,12 @@
+.cache
+.nyc_output
+*.gz
+*.html
+*.tgz
 build
 coverage
 dev
 doc
 src*
 test
-.nyc_output
 tsconfig.json
-*.tgz
-*.html

packages/atom/package.json

@@ -1,8 +1,9 @@
 {
     "name": "@thi.ng/atom",
     "version": "1.5.8",
-    "description": "Mutable wrapper for immutable values",
-    "main": "./index.js",
+    "description": "Mutable wrappers for nested immutable values w/ optional undo/redo history",
+    "module": "./index.js",
+    "main": "./lib/index.js",
     "typings": "./index.d.ts",
     "repository": {
         "type": "git",
@@ -12,12 +13,16 @@
     "author": "Karsten Schmidt <[email protected]>",
     "license": "Apache-2.0",
     "scripts": {
-        "build": "yarn clean && tsc --declaration",
-        "clean": "rimraf *.js *.d.ts .nyc_output build coverage doc",
+        "build": "yarn clean && yarn build:es6 && yarn build:parcel && yarn build:parcel:min",
+        "build:es6": "tsc --declaration",
+        "build:parcel:min": "parcel build --global thing_atom -o index.min.js -d lib --experimental-scope-hoisting src/index.ts",
+        "build:parcel": "parcel build --global thing_atom -d lib --no-minify --experimental-scope-hoisting src/index.ts && prettier --write lib/index.js",
+        "build:gzip": "gzip -c lib/index.min.js > lib/index.min.js.gz",
+        "clean": "rimraf *.js *.d.ts .cache .nyc_output build coverage doc lib",
+        "test": "rimraf build && parcel build -d build --no-cache --no-minify --no-source-maps test/index.ts && nyc mocha build/index.js",
         "cover": "yarn test && nyc report --reporter=lcov",
         "doc": "typedoc --mode modules --out doc src",
-        "pub": "yarn build && yarn publish --access public",
-        "test": "rimraf build && tsc -p test && nyc mocha build/test/*.js"
+        "pub": "yarn build && yarn publish --access public"
     },
     "devDependencies": {
         "@types/mocha": "^5.2.5",
@@ -35,15 +40,24 @@
         "@thi.ng/paths": "^1.6.6"
     },
     "keywords": [
+        "derived views",
         "cursor",
-        "datastructure",
+        "data structure",
         "ES6",
         "history",
         "immutable",
+        "redo",
         "typescript",
         "undo"
     ],
     "publishConfig": {
         "access": "public"
+    },
+    "browserslist": [
+        "since 2018-07"
+    ],
+    "browser": {
+        "process": false,
+        "setTimeout": false
     }
 }

packages/atom/src/atom.ts

@@ -72,19 +72,19 @@ export class Atom<T> implements
 
     // mixin stub
     /* istanbul ignore next */
-    addWatch(id: string, fn: Watch<T>) {
+    addWatch(_: string, __: Watch<T>) {
         return false;
     }
 
     // mixin stub
     /* istanbul ignore next */
-    removeWatch(id: string) {
+    removeWatch(_: string) {
         return false;
     }
 
     // mixin stub
     /* istanbul ignore next */
-    notifyWatches(oldState: T, newState: T) { }
+    notifyWatches(_: T, __: T) { }
 
     addView<V>(path: Path, tx?: ViewTransform<V>, lazy = true): IView<V> {
         return new View<V>(this, path, tx, lazy);

packages/atom/src/cursor.ts

@@ -4,7 +4,6 @@ import { isFunction } from "@thi.ng/checks/is-function";
 import { illegalArgs } from "@thi.ng/errors/illegal-arguments";
 import { illegalArity } from "@thi.ng/errors/illegal-arity";
 import { getter, Path, setter } from "@thi.ng/paths";
-
 import {
     CursorOpts,
     IAtom,
@@ -13,9 +12,9 @@ import {
     ViewTransform
 } from "./api";
 import { Atom } from "./atom";
+import { nextID } from "./idgen";
 import { View } from "./view";
 
-
 /**
  * A cursor provides read/write access to a path location within a
  * nested parent state (Atom or another Cursor). Cursors behave like
@@ -46,8 +45,6 @@ export class Cursor<T> implements
     IID<string>,
     IRelease {
 
-    static NEXT_ID = 0;
-
     readonly id: string;
     parent: IAtom<any>;
 
@@ -89,7 +86,7 @@ export class Cursor<T> implements
                 illegalArity(args.length);
         }
         this.parent = parent;
-        this.id = id || `cursor-${Cursor.NEXT_ID++}`;
+        this.id = id || `cursor-${nextID()}`;
         this.selfUpdate = false;
         if (!lookup || !update) {
             illegalArgs();

packages/atom/src/history.ts

@@ -255,14 +255,14 @@ export class History<T> implements
         return true;
     }
 
-    addListener(id: string, fn: (e: Event) => void, scope?: any): boolean {
+    addListener(_: string, __: (e: Event) => void, ___?: any): boolean {
         return false;
     }
 
-    removeListener(id: string, fn: (e: Event) => void, scope?: any): boolean {
+    removeListener(_: string, __: (e: Event) => void, ___?: any): boolean {
         return false;
     }
 
-    notify(event: Event): void {
+    notify(_: Event): void {
     }
 }

packages/atom/src/idgen.ts

@@ -0,0 +1,3 @@
+let NEXT_ID = 0;
+
+export const nextID = () => NEXT_ID++;

packages/atom/src/view.ts

@@ -1,7 +1,7 @@
 import { equiv as _equiv } from "@thi.ng/equiv";
 import { getter, Path, toPath } from "@thi.ng/paths";
-
 import { IView, ReadonlyAtom, ViewTransform } from "./api";
+import { nextID } from "./idgen";
 
 /**
  * This class implements readonly access to a deeply nested value with
@@ -47,8 +47,6 @@ import { IView, ReadonlyAtom, ViewTransform } from "./api";
 export class View<T> implements
     IView<T> {
 
-    protected static NEXT_ID = 0;
-
     readonly id: string;
 
     readonly parent: ReadonlyAtom<any>;
@@ -62,7 +60,7 @@ export class View<T> implements
 
     constructor(parent: ReadonlyAtom<any>, path: Path, tx?: ViewTransform<T>, lazy = true, equiv = _equiv) {
         this.parent = parent;
-        this.id = `view-${View.NEXT_ID++}`;
+        this.id = `view-${nextID()}`;
         this.tx = tx || ((x: any) => x);
         this.path = toPath(path);
         this.isDirty = true;

packages/atom/test/atom.ts

@@ -15,8 +15,8 @@ describe("atom", function () {
     });
 
     it("can be equiv'd", () => {
-        assert(a.equiv(a));
-        assert(!a.equiv(new Atom(23)));
+        assert.ok(a.equiv(a));
+        assert.ok(!a.equiv(new Atom(23)));
     });
 
     it("can be reset", () => {
@@ -30,10 +30,10 @@ describe("atom", function () {
     });
 
     it("can add & remove watch", () => {
-        assert(a.addWatch("foo", () => { }), "can't add watch");
-        assert((<any>a)._watches && (<any>a)._watches.foo, "watch missing");
-        assert(a.removeWatch("foo"), "can't remove watch");
-        assert(!a.removeWatch("foo"), "should fail to remove invalid watch id");
+        assert.ok(a.addWatch("foo", () => { }), "can't add watch");
+        assert.ok((<any>a)._watches && (<any>a)._watches.foo, "watch missing");
+        assert.ok(a.removeWatch("foo"), "can't remove watch");
+        assert.ok(!a.removeWatch("foo"), "should fail to remove invalid watch id");
     });
 
     it("can be watched", () => {

packages/atom/test/cursor.ts

@@ -126,17 +126,17 @@ describe("cursor", function () {
         c = new Cursor(a, "a");
         let id = c.id;
         assert.notEqual((<any>a)._watches[id], null);
-        assert(c.release());
+        assert.ok(c.release());
         assert.strictEqual(c.parent, undefined);
         assert.strictEqual((<any>a)._watches[id], undefined);
     });
 
     it("can add & remove watch", () => {
         c = new Cursor(a, "a.b.c");
-        assert(c.addWatch("foo", () => { }), "can't add watch");
-        assert((<any>c).local._watches && (<any>c).local._watches.foo, "watch missing");
-        assert(c.removeWatch("foo"), "can't remove watch");
-        assert(!c.removeWatch("foo"), "should fail to remove invalid watch id");
+        assert.ok(c.addWatch("foo", () => { }), "can't add watch");
+        assert.ok((<any>c).local._watches && (<any>c).local._watches.foo, "watch missing");
+        assert.ok(c.removeWatch("foo"), "can't remove watch");
+        assert.ok(!c.removeWatch("foo"), "should fail to remove invalid watch id");
     });
 
     it("can be watched", () => {

packages/atom/test/index.ts

@@ -0,0 +1,4 @@
+export * from "./atom";
+export * from "./cursor";
+export * from "./history";
+export * from "./view";

packages/atom/test/tsconfig.json

@@ -2,7 +2,8 @@
     "extends": "../../../tsconfig.json",
     "compilerOptions": {
         "outDir": "../build",
-        "noUnusedParameters": false
+        "module": "es6",
+        "target": "es6"
     },
     "include": [
         "./**/*.ts",