added prettier, fixed formatting and linting errors

Author: ppaska

.eslintrc

@@ -13,7 +13,7 @@
     "@typescript-eslint/no-explicit-any": "warn",
     "@typescript-eslint/explicit-module-boundary-types": "warn",
     "@typescript-eslint/explicit-function-return-type": "warn",
-    "@typescript-eslint/no-unused-vars": "off",
+    "@typescript-eslint/no-unused-vars": "warn",
     "@typescript-eslint/no-control-regex": "off",
     "@typescript-eslint/no-constant-condition": "off"
   }

.prettierrc

@@ -0,0 +1,8 @@
+{
+  "printWidth": 100,
+  "trailingComma": "none",
+  "arrowParens": "avoid",
+  "parser": "typescript",
+  "singleQuote": true,
+  "tabWidth": 2
+}

package.json

@@ -1,6 +1,6 @@
 {
   "name": "jspython-interpreter",
-  "version": "2.1.10",
+  "version": "2.1.11",
   "description": "JSPython is a javascript implementation of Python language that runs within web browser or NodeJS environment",
   "keywords": [
     "python",
@@ -43,6 +43,7 @@
     "@typescript-eslint/parser": "^4.8.2",
     "ace-builds": "^1.4.12",
     "eslint": "^7.14.0",
+    "husky": "^4.2.5",
     "jest": "^26.6.3",
     "rollup": "^2.52.7",
     "rollup-plugin-copy": "^3.4.0",
@@ -53,5 +54,10 @@
     "ts-jest": "^26.4.4",
     "tslib": "^2.3.0",
     "typescript": "^4.3.5"
-  }
+  },
+  "husky": {
+    "hooks": {
+        "pre-commit": "npm run lint && npm run test"
+    }
+}  
 }

src/common/ast-types.ts

@@ -1,35 +1,30 @@
-import {
-  ExpressionOperators,
-  LogicalOperators,
-  OperationTypes,
-  Operators,
-} from "./operators";
-import { getTokenLoc, getTokenValue, Token } from "./token-types";
+import { ExpressionOperators, LogicalOperators } from './operators';
+import { getTokenLoc, getTokenValue, Token } from './token-types';
 
 export type AstNodeType =
-  | "assign"
-  | "binOp"
-  | "const"
-  | "logicalOp"
-  | "getSingleVar"
-  | "setSingleVar"
-  | "dotObjectAccess"
-  | "bracketObjectAccess"
-  | "funcCall"
-  | "funcDef"
-  | "arrowFuncDef"
-  | "createObject"
-  | "createArray"
-  | "if"
-  | "for"
-  | "while"
-  | "tryExcept"
-  | "raise"
-  | "import"
-  | "comment"
-  | "return"
-  | "continue"
-  | "break";
+  | 'assign'
+  | 'binOp'
+  | 'const'
+  | 'logicalOp'
+  | 'getSingleVar'
+  | 'setSingleVar'
+  | 'dotObjectAccess'
+  | 'bracketObjectAccess'
+  | 'funcCall'
+  | 'funcDef'
+  | 'arrowFuncDef'
+  | 'createObject'
+  | 'createArray'
+  | 'if'
+  | 'for'
+  | 'while'
+  | 'tryExcept'
+  | 'raise'
+  | 'import'
+  | 'comment'
+  | 'return'
+  | 'continue'
+  | 'break';
 
 export interface NameAlias {
   name: string;
@@ -61,12 +56,8 @@ export abstract class AstNode {
 }
 
 export class AssignNode extends AstNode {
-  constructor(
-    public target: AstNode,
-    public source: AstNode,
-    public loc: Uint16Array
-  ) {
-    super("assign");
+  constructor(public target: AstNode, public source: AstNode, public loc: Uint16Array) {
+    super('assign');
     this.loc = loc;
   }
 }
@@ -75,56 +66,49 @@ export class ConstNode extends AstNode {
   public value: number | string | boolean | null;
 
   constructor(token: Token) {
-    super("const");
+    super('const');
     this.value = getTokenValue(token);
     this.loc = getTokenLoc(token);
   }
 }
 
 export class CommentNode extends AstNode {
   constructor(public comment: string, public loc: Uint16Array) {
-    super("comment");
+    super('comment');
     this.loc = loc;
   }
 }
 
 export class ReturnNode extends AstNode {
-  constructor(
-    public returnValue: AstNode | undefined = undefined,
-    public loc: Uint16Array
-  ) {
-    super("return");
+  constructor(public returnValue: AstNode | undefined = undefined, public loc: Uint16Array) {
+    super('return');
     this.loc = loc;
   }
 }
 
 export class RaiseNode extends AstNode {
-  constructor(
-    public errorName: string,
-    public errorMessageAst: AstNode,
-    public loc: Uint16Array
-  ) {
-    super("raise");
+  constructor(public errorName: string, public errorMessageAst: AstNode, public loc: Uint16Array) {
+    super('raise');
     this.loc = loc;
   }
 }
 
 export class ContinueNode extends AstNode {
   constructor() {
-    super("continue");
+    super('continue');
   }
 }
 
 export class BreakNode extends AstNode {
   constructor() {
-    super("break");
+    super('break');
   }
 }
 
 export class SetSingleVarNode extends AstNode {
   public name: string;
   constructor(token: Token) {
-    super("setSingleVar");
+    super('setSingleVar');
     this.name = token[0] as string;
     this.loc = getTokenLoc(token);
   }
@@ -133,12 +117,8 @@ export class SetSingleVarNode extends AstNode {
 export class FunctionCallNode extends AstNode implements IsNullCoelsing {
   public nullCoelsing: boolean | undefined = undefined;
 
-  constructor(
-    public name: string,
-    public paramNodes: AstNode[] | null,
-    public loc: Uint16Array
-  ) {
-    super("funcCall");
+  constructor(public name: string, public paramNodes: AstNode[] | null, public loc: Uint16Array) {
+    super('funcCall');
     this.loc = loc;
   }
 }
@@ -150,18 +130,14 @@ export class FunctionDefNode extends AstNode implements FuncDefNode {
     public isAsync: boolean,
     public loc: Uint16Array
   ) {
-    super("funcDef");
+    super('funcDef');
     this.loc = loc;
   }
 }
 
 export class ArrowFuncDefNode extends AstNode implements FuncDefNode {
-  constructor(
-    public funcAst: AstBlock,
-    public params: string[],
-    public loc: Uint16Array
-  ) {
-    super("arrowFuncDef");
+  constructor(public funcAst: AstBlock, public params: string[], public loc: Uint16Array) {
+    super('arrowFuncDef');
     this.loc = loc;
   }
 }
@@ -173,7 +149,7 @@ export class IfNode extends AstNode {
     public elseBody: AstNode[] | undefined = undefined,
     public loc: Uint16Array
   ) {
-    super("if");
+    super('if');
     this.loc = loc;
   }
 }
@@ -187,7 +163,7 @@ export class TryExceptNode extends AstNode {
 
     public loc: Uint16Array
   ) {
-    super("tryExcept");
+    super('tryExcept');
     this.loc = loc;
   }
 }
@@ -199,18 +175,14 @@ export class ForNode extends AstNode {
     public body: AstNode[],
     public loc: Uint16Array
   ) {
-    super("for");
+    super('for');
     this.loc = loc;
   }
 }
 
 export class WhileNode extends AstNode {
-  constructor(
-    public condition: AstNode,
-    public body: AstNode[],
-    public loc: Uint16Array
-  ) {
-    super("while");
+  constructor(public condition: AstNode, public body: AstNode[], public loc: Uint16Array) {
+    super('while');
     this.loc = loc;
   }
 }
@@ -222,7 +194,7 @@ export class ImportNode extends AstNode {
     public parts: NameAlias[] | undefined = undefined,
     public loc: Uint16Array
   ) {
-    super("import");
+    super('import');
     this.loc = loc;
   }
 }
@@ -232,7 +204,7 @@ export class GetSingleVarNode extends AstNode implements IsNullCoelsing {
   nullCoelsing: boolean | undefined = undefined;
 
   constructor(token: Token, nullCoelsing: boolean | undefined = undefined) {
-    super("getSingleVar");
+    super('getSingleVar');
     this.name = token[0] as string;
     this.nullCoelsing = nullCoelsing;
     this.loc = getTokenLoc(token);
@@ -241,21 +213,21 @@ export class GetSingleVarNode extends AstNode implements IsNullCoelsing {
 
 export class DotObjectAccessNode extends AstNode {
   constructor(public nestedProps: AstNode[], public loc: Uint16Array) {
-    super("dotObjectAccess");
+    super('dotObjectAccess');
     this.loc = loc;
   }
 }
 
 export class CreateObjectNode extends AstNode {
   constructor(public props: ObjectPropertyInfo[], public loc: Uint16Array) {
-    super("createObject");
+    super('createObject');
     this.loc = loc;
   }
 }
 
 export class CreateArrayNode extends AstNode {
   constructor(public items: AstNode[], public loc: Uint16Array) {
-    super("createArray");
+    super('createArray');
     this.loc = loc;
   }
 }
@@ -267,7 +239,7 @@ export class BracketObjectAccessNode extends AstNode {
     public nullCoalescing: boolean | undefined = undefined,
     public loc: Uint16Array
   ) {
-    super("bracketObjectAccess");
+    super('bracketObjectAccess');
     this.loc = loc;
   }
 }
@@ -279,7 +251,7 @@ export interface LogicalNodeItem {
 
 export class LogicalOpNode extends AstNode {
   constructor(public items: LogicalNodeItem[], public loc: Uint16Array) {
-    super("logicalOp");
+    super('logicalOp');
     this.loc = loc;
   }
 }
@@ -291,14 +263,14 @@ export class BinOpNode extends AstNode {
     public right: AstNode,
     public loc: Uint16Array
   ) {
-    super("binOp");
+    super('binOp');
     this.loc = loc;
   }
 }
 
 export interface AstBlock {
   name: string;
-  type: "module" | "func" | "if" | "for" | "while" | "trycatch";
+  type: 'module' | 'func' | 'if' | 'for' | 'while' | 'trycatch';
   funcs: FunctionDefNode[];
   body: AstNode[];
 }

src/common/index.ts

@@ -1,4 +1,4 @@
 export * from './ast-types';
 export * from './token-types';
 export * from './parser-types';
-export * from './operators';
+export * from './operators';