@effection/main - npm Package Compare versions

Comparing version 2.0.0-preview.2 to 2.0.0-preview.3

CHANGELOG.md

 # @effection/main
 
+## 2.0.0-preview.3
+
+### Minor Changes
+
+- 88dc59a: Remove `verbose` option from MainError
+- 88dc59a: Improve error output by including an Effection trace
+
+### Patch Changes
+
+- Updated dependencies [9998088]
+- Updated dependencies [2bce454]
+- Updated dependencies [1981b35]
+- Updated dependencies [88dc59a]
+  - @effection/core@2.0.0-preview.12
+
 ## 2.0.0-preview.2
+
 ### Patch Changes
@@ -5,0 +21,0 @@

dist/browser.js

@@ -15,2 +15,3 @@ "use strict";
 const core_1 = require("@effection/core");
+const error_1 = require("./error");
 __exportStar(require("./error"), exports);
@@ -24,14 +25,11 @@ function main(operation) {
         }
-        catch (e) {
-            if (e.name === 'EffectionMainError') {
-                if (e.options.message) {
-                    console.error(e.options.message);
+        catch (error) {
+            if (error_1.isMainError(error)) {
+                if (error.message) {
+                    console.error(error.message);
                 }
-                if (e.options.verbose) {
-                    console.error(e.stack);
-                }
             }
             else {
-                console.error(e);
-                throw e;
+                console.error(error);
+                throw error;
             }
@@ -38,0 +36,0 @@ }

dist/error.d.ts

 export interface MainErrorOptions {
     exitCode?: number;
-    verbose?: boolean;
     message?: string;
 }
 export declare class MainError extends Error {
-    options: MainErrorOptions;
     name: string;
+    exitCode: number;
     constructor(options?: MainErrorOptions);
 }
+export declare function isMainError(error: Error): error is MainError;

dist/error.js

 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.MainError = void 0;
+exports.isMainError = exports.MainError = void 0;
 class MainError extends Error {
     constructor(options = {}) {
-        super(options.message || "error");
-        this.options = options;
+        super(options.message || "");
         this.name = "EffectionMainError";
+        this.exitCode = options.exitCode || -1;
     }
 }
 exports.MainError = MainError;
+function isMainError(error) {
+    return error.name === 'EffectionMainError';
+}
+exports.isMainError = isMainError;
 //# sourceMappingURL=error.js.map

dist/node.js

@@ -15,2 +15,4 @@ "use strict";
 const core_1 = require("@effection/core");
+const format_error_node_1 = require("./format-error-node");
+const error_1 = require("./error");
 __exportStar(require("./error"), exports);
@@ -25,14 +27,8 @@ function main(operation) {
         }
-        catch (e) {
-            if (e.name === 'EffectionMainError') {
-                if (e.options.message) {
-                    console.error(e.options.message);
-                }
-                if (e.options.verbose) {
-                    console.error(e.stack);
-                }
-                process.exit(e.options.exitCode || -1);
+        catch (error) {
+            console.error(format_error_node_1.formatError(error));
+            if (error_1.isMainError(error)) {
+                process.exit(error.exitCode || -1);
             }
             else {
-                console.error(e);
                 process.exit(1);
@@ -39,0 +35,0 @@ }

package.json

 {
   "name": "@effection/main",
-  "version": "2.0.0-preview.2",
+  "version": "2.0.0-preview.3",
   "description": "Main entry point for Effection applications",
@@ -23,7 +23,9 @@ "main": "dist/node.js",
   "dependencies": {
-    "@effection/core": "2.0.0-preview.11"
+    "@effection/core": "2.0.0-preview.12",
+    "chalk": "^4.1.1",
+    "stacktrace-parser": "^0.1.10"
   },
   "devDependencies": {
-    "@effection/node": "2.0.0-preview.15",
-    "@effection/mocha": "2.0.0-preview.12",
+    "@effection/mocha": "2.0.0-preview.13",
+    "@effection/node": "2.0.0-preview.16",
     "@frontside/tsconfig": "0.0.1",
@@ -30,0 +32,0 @@ "@types/node": "^13.13.5",

src/browser.ts

 import { run, Task, Operation } from '@effection/core';
+import { isMainError } from './error';
 
@@ -11,13 +12,10 @@ export * from './error';
       return yield operation;
-    } catch(e) {
-      if(e.name === 'EffectionMainError') {
-        if(e.options.message) {
-          console.error(e.options.message);
+    } catch(error) {
+      if(isMainError(error)) {
+        if(error.message) {
+          console.error(error.message);
         }
-        if(e.options.verbose) {
-          console.error(e.stack);
-        }
       } else {
-        console.error(e);
-        throw e;
+        console.error(error);
+        throw error;
       }
@@ -24,0 +22,0 @@ } finally {

src/error.ts

 export interface MainErrorOptions {
   exitCode?: number;
-  verbose?: boolean;
   message?: string;
@@ -10,5 +9,12 @@ }
 
-  constructor(public options: MainErrorOptions = {}) {
-    super(options.message || "error");
+  public exitCode: number;
+
+  constructor(options: MainErrorOptions = {}) {
+    super(options.message || "");
+    this.exitCode = options.exitCode || -1;
   }
 }
+
+export function isMainError(error: Error): error is MainError {
+  return error.name === 'EffectionMainError';
+}

src/node.ts

 import { run, Task, Operation } from '@effection/core';
+import { formatError } from './format-error-node';
+import { isMainError } from './error';
 
@@ -12,13 +14,7 @@ export * from './error';
       return yield operation;
-    } catch(e) {
-      if(e.name === 'EffectionMainError') {
-        if(e.options.message) {
-          console.error(e.options.message);
-        }
-        if(e.options.verbose) {
-          console.error(e.stack);
-        }
-        process.exit(e.options.exitCode || -1);
+    } catch(error) {
+      console.error(formatError(error));
+      if(isMainError(error)) {
+        process.exit(error.exitCode || -1);
       } else {
-        console.error(e);
         process.exit(1);
@@ -25,0 +21,0 @@ }

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

16729

increased by69.18%

Number of package files

19

increased by26.67%

Lines of code

294

increased by65.17%

Worsened metrics

Dependency count

3

increased by200%

Dependency changes

• + Addedchalk@^4.1.1

• + Addedstacktrace-parser@^0.1.10

• + Added@effection/core@2.0.0-preview.12(transitive)

• + Addedansi-styles@4.3.0(transitive)

• + Addedchalk@4.1.2(transitive)

• + Addedcolor-convert@2.0.1(transitive)

• + Addedcolor-name@1.1.4(transitive)

• + Addedhas-flag@4.0.0(transitive)

• + Addedstacktrace-parser@0.1.10(transitive)

• + Addedsupports-color@7.2.0(transitive)

• + Addedtype-fest@0.7.1(transitive)

• - Removed@effection/core@2.0.0-preview.11(transitive)

• Updated@effection/core@2.0.0-preview.12