add strict mode for js scripts

Author: bobzhang

Diff for: jscomp/test/test_promise_bind.js

@@ -1,8 +1,8 @@
 'use strict';
 
-var Promise = require("./promise.js");
+var $$Promise = require("./promise.js");
 
-var v = Promise.f;
+var v = $$Promise.f;
 
 exports.v = v;
 /* Promise Not a pure module */

Diff for: lib/bsb

@@ -16,9 +16,15 @@ var bsb_exe = __filename + ".exe"
 var LAST_SUCCESS_BUILD_STAMP = 0
 
 // All clients of type MiniWebSocket
+/**
+ * @type {any[]}
+ */
 var wsClients = []
 var watch_mode = false
 var verbose = false
+/**
+ * @type {string | undefined}
+ */
 var postBuild = undefined
 var useWebSocket = false
 var webSocketPort = 9999

Diff for: lib/jsconfig.json

@@ -0,0 +1,17 @@
+{
+    "compilerOptions": {
+        "alwaysStrict": true
+        // , "noImplicitAny": true
+        // , "strictNullChecks": true
+        , "target": "es2018"
+        // target need be set up
+        // otherwise some method is missing
+        , "module": "commonjs"
+        // setup to understand commonjs module
+    }   
+
+    
+    , "files": [
+        "./bsb"
+    ]
+}

Diff for: scripts/jsconfig.json

@@ -0,0 +1,12 @@
+{
+    "compilerOptions": {
+        "alwaysStrict": true
+        , "noImplicitAny": true
+        // , "strictNullChecks": true
+        , "target": "es2018"
+        // target need be set up
+        // otherwise some method is missing
+        , "module": "commonjs"
+        // setup to understand commonjs module
+    }   
+}

Diff for: scripts/runtimeDeps.js

@@ -282,16 +282,27 @@ async function stdlibNinja(){
         }
     )    
 }
-
+/**
+ * 
+ * @param {string} text 
+ */
 function getDeps(text) {
+    /**
+     * @type {string[]}
+     */
     var deps = []
     text.replace(/(\/\*[\w\W]*?\*\/|\/\/[^\n]*|[.$]r)|\brequire\s*\(\s*["']([^"']*)["']\s*\)/g, function(_, ignore, id) {
         if (!ignore)
             deps.push(id);
+        return "" // TODO: examine the regex
     });
     return deps;
 }
 
+/**
+ * 
+ * @param {string} x 
+ */
 function baseName(x) {
     return x.substr(0, x.indexOf('.'))
 }
@@ -361,7 +372,7 @@ function runJSCheckAsync(depsMap){
  * @param {string[]} output 
  * @param {string[]} inputs 
  * @param {DepsMap} depsMap 
- * @param {any[]} overrides
+ * @param {Override[]} overrides
  */
 function build_stmt(output, inputs,depsMap, overrides = [] ){
     var stmt =  `build ${output.join(' ')} : cc ${inputs.join(' ')}` 
@@ -386,7 +397,11 @@ function build_stmt(output, inputs,depsMap, overrides = [] ){
     return stmt
 }
 
-
+/**
+ * 
+ * @typedef {{key : string, value : string}} Override
+ * 
+ */
 
 
 /**
@@ -406,6 +421,9 @@ function generateNinja(depsMap,allTargets){
         var output_cmi = mod + ".cmi"
         var input_ml = mod + ".ml"
         var input_mli = mod + ".mli"
+        /**
+         * @type {Override[]}
+         */
         var overrides = []
         if(mod.endsWith('Labels')){
             overrides.push({key:'bsc_flags',value : '$bsc_flags -nolabels'})