fix: constructor validation on build (#226)

* fix: constructor validation on build

* Update unordered-map.js

* feat: constructor checker

* feat: constructor validation

* fix: resolve conflicts

* fix: resolve conflicts

* fix: code cleaning

* fix: code cleaning

* fix: code cleaning

* fix: lint

* fix: tests

* fix: empty state contract

* fix: update contract valdiation to handle clean-state example

* linter fix

* constructor validation moved to utils

* constructor test contracts renamed, typos fixed

* redandunt comment deleted

* Error messages changed

* log comments added to contract validation

* yarn build

* cli/cli.js deleted

* contract validation added to the main pipeline

* contract validation rewrittent with TS

* signal used in contract validation, tests rewritten

* contract validation doc added

Co-authored-by: Serhii Volovyk <[email protected]>

Author: osalkanovic

Diff for: examples/package.json

@@ -43,6 +43,7 @@
     "@types/lodash-es": "^4.17.6",
     "ava": "^4.2.0",
     "near-workspaces": "3.2.1",
-    "typescript": "^4.7.4"
+    "typescript": "^4.7.4",
+    "npm-run-all": "^4.1.5"
   }
 }

Diff for: examples/yarn.lock

@@ -61,6 +61,8 @@
     "@types/eslint": "^8.4.6",
     "@types/node": "^17.0.38",
     "@types/rollup": "^0.54.0",
+    "chalk": "^5.1.0",
+    "ts-morph": "^16.0.0",
     "@types/signale": "^1.4.4",
     "@typescript-eslint/eslint-plugin": "^5.37.0",
     "@typescript-eslint/parser": "^5.37.0",

Diff for: lib/cli/cli.js

@@ -10,7 +10,7 @@ import { rollup } from "rollup";
 import { Command } from "commander";
 import signal from "signale";
 
-import { executeCommand } from "./utils.js";
+import { executeCommand, validateContract } from "./utils.js";
 
 const { Signale } = signal;
 const PROJECT_DIR = process.cwd();
@@ -66,6 +66,9 @@ export async function buildCom(
   signale.await(`Creating ${TARGET_DIR} directory...`);
   await executeCommand(`mkdir -p ${TARGET_DIR}`, verbose);
 
+  signal.await(`Validatig ${source} contract...`);
+  await validateContract(source);
+
   signale.await(`Creating ${source} file with Rollup...`);
   await createJsFileWithRullup(source, ROLLUP_TARGET, verbose);
 

Diff for: lib/cli/utils.d.ts

@@ -1,6 +1,9 @@
 import childProcess from "child_process";
 import { promisify } from "util";
 import signal from "signale"
+import { ClassDeclaration, ClassDeclarationStructure, ConstructorDeclaration, OptionalKind, Project, PropertyDeclarationStructure, SourceFile } from "ts-morph";
+import chalk from "chalk";
+import signale from "signale";
 
 const {Signale} = signal;
 
@@ -37,3 +40,48 @@ export async function executeCommand(
 export async function download(url: string, verbose = false) {
   await executeCommand(`curl -LOf ${url}`, verbose);
 }
+
+const UNINITIALIZED_PARAMETERS_ERROR = "All parameters must be initialized in the constructor. Uninitialized parameters:";
+
+/**
+ * Validates the contract by checking that all parameters are initialized in the constructor. Works only for contracts written in TypeScript.
+ * @param contractPath Path to the contract.
+ **/
+export async function validateContract(contractPath: string): Promise<boolean> {
+  const project: Project = new Project();
+  project.addSourceFilesAtPaths(contractPath);
+  const sourceFile: SourceFile = project.getSourceFile(contractPath);
+  const classDeclarations: ClassDeclaration[] = sourceFile.getClasses();
+  for (const classDeclaration of classDeclarations) {
+    const classStructure: ClassDeclarationStructure = classDeclaration.getStructure();
+    const { decorators, properties } = classStructure;
+    const hasNearBindgen: boolean = decorators.find(
+      (decorator) => decorator.name === "NearBindgen"
+    ) ? true : false;
+    if (hasNearBindgen) {
+      const constructors: ConstructorDeclaration[] = classDeclaration.getConstructors();
+      const hasConstructor = constructors.length > 0;
+      const propertiesToBeInited: OptionalKind<PropertyDeclarationStructure>[] = properties.filter((p) => !p.initializer);
+      if (!hasConstructor && propertiesToBeInited.length === 0) {
+        return true;
+      }
+      if (!hasConstructor && propertiesToBeInited.length > 0) {
+        signale.error(chalk.redBright(`${UNINITIALIZED_PARAMETERS_ERROR} ${propertiesToBeInited.map((p) => p.name)}`));
+        process.exit(1);
+      }
+      const constructor: ConstructorDeclaration = constructors[0];
+      const constructorContent: string = constructor.getText();
+      const nonInitedProperties: string[] = [];
+      for (const property of propertiesToBeInited) {
+        if (!constructorContent.includes(`this.${property.name}`)) {
+          nonInitedProperties.push(property.name);
+        }
+      }
+      if (nonInitedProperties.length > 0) {
+        signale.error(chalk.redBright(`${UNINITIALIZED_PARAMETERS_ERROR} ${nonInitedProperties.join(", ")}`));
+        process.exit(1);
+      }
+    }
+  }
+  return true;
+}

Diff for: lib/cli/utils.js

@@ -0,0 +1,39 @@
+import test from "ava";
+import { execSync } from "child_process";
+
+const BUILD_FAILURE_ERROR_CODE = 1;
+
+test("should not throw error, constructor is correctly initialized", async (t) => {
+  t.notThrows(() => {
+    execSync(
+      "near-sdk-js build src/constructor-validation/all-parameters-set-in-constructor.ts build/all-parameters-set-in-constructor.wasm"
+    );
+  });
+});
+
+test("should throw error, name is not inited", async (t) => {
+  const error = t.throws(() => {
+    execSync(
+      "near-sdk-js build src/constructor-validation/1-parameter-not-set-in-constructor.ts build/1-parameter-not-set-in-constructor.wasm"
+    );
+  });
+  t.is(error.status, BUILD_FAILURE_ERROR_CODE);
+});
+
+test("should throw error, construcor is empty", async (t) => {
+  const error = t.throws(() => {
+    execSync(
+      "near-sdk-js build src/constructor-validation/no-parameters-set-in-constructor.ts build/no-parameters-set-in-constructor.wasm"
+    );
+  });
+  t.is(error.status, BUILD_FAILURE_ERROR_CODE);
+});
+
+test("should throw error, construcor is not declared", async (t) => {
+  const error = t.throws(() => {
+    execSync(
+      "near-sdk-js build src/constructor-validation/no-constructor.ts build/no-constructor.wasm"
+    );
+  });
+  t.is(error.status, BUILD_FAILURE_ERROR_CODE);
+});

Diff for: package.json

@@ -51,13 +51,15 @@
     "test:bigint-serialization": "ava __tests__/test-bigint-serialization.ava.js",
     "test:date-serialization": "ava __tests__/test-date-serialization.ava.js",
     "test:serialization": "ava __tests__/test-serialization.ava.js",
+    "test:constructor-validation": "ava __tests__/constructor_validation.ava.js",
     "test:middlewares": "ava __tests__/test-middlewares.ava.js"
   },
   "author": "Near Inc <[email protected]>",
   "license": "Apache-2.0",
   "devDependencies": {
     "ava": "^4.2.0",
     "near-workspaces": "3.2.1",
-    "typescript": "^4.7.4"
+    "typescript": "^4.7.4",
+    "npm-run-all": "^4.1.5"
   }
 }

Diff for: src/cli/cli.ts

@@ -0,0 +1,10 @@
+import { NearBindgen, LookupMap } from "near-sdk-js";
+
+@NearBindgen({})
+export class ConstructorValidation {
+  map: LookupMap<string>;
+  name: string;
+  constructor() {
+    this.map = new LookupMap<string>("a");
+  }
+}

Diff for: src/cli/utils.ts

@@ -0,0 +1,16 @@
+import { NearBindgen, LookupMap, call } from "near-sdk-js";
+
+@NearBindgen({})
+export class ConstructorValidation {
+  map: LookupMap<string>;
+  name: string;
+  constructor() {
+    this.map = new LookupMap<string>("a");
+    this.name = "";
+  }
+
+  @call({})
+  get() {
+    return { status: "ok" };
+  }
+}

Diff for: tests/__tests__/constructor_validation.ava.js

@@ -0,0 +1,7 @@
+import { NearBindgen, LookupMap } from "near-sdk-js";
+
+@NearBindgen({})
+export class ConstructorValidation {
+  map: LookupMap<string>;
+  name: string;
+}

Diff for: tests/package.json

@@ -0,0 +1,10 @@
+import { NearBindgen, LookupMap } from "near-sdk-js";
+
+@NearBindgen({})
+export class ConstructorValidation {
+  map: LookupMap<string>;
+  name: string;
+  constructor() {
+    //
+  }
+}