Merge pull request #346 from near/fix-contract-method-conflicting

Error and stop build a contract that has method name confliction

Author: ailisp

Diff for: packages/near-sdk-js/lib/cli/cli.js

@@ -342,6 +342,12 @@ async function createMethodsHeaderFile(rollupTarget: string, verbose = false) {
 
   const mod = await import(`${PROJECT_DIR}/${rollupTarget}`);
   const exportNames = Object.keys(mod);
+  if (exportNames.includes('panic')) {
+    signal.error(
+      "'panic' is a reserved word, please use another name for contract method"
+    );
+    process.exit(1);
+  }
   const methods = exportNames.reduce(
     (result, key) => `${result}DEFINE_NEAR_METHOD(${key})\n`,
     ""

Diff for: packages/near-sdk-js/lib/cli/utils.js

@@ -28,6 +28,11 @@ export async function executeCommand(
   }
   if (code != 0) {
     signale.error(`Command failed: ${command}`);
+
+    const failDueToNameConflict = stderr.match(/conflicting types for '([a-zA-Z0-9_]+)'/);
+    if (failDueToNameConflict.length > 1) {
+      signale.error(`'${failDueToNameConflict[1]}' is a reserved word, please use another name for contract method"`);
+    }
   }
   if (stderr && verbose) {
     signale.error(`Command stderr: ${stderr}`);