Runtime: improve eval functions

- Use indirect eval (faster and avoid variable captures)
- Put expressions within parentheses to allow arbitrary expressions
  without confusion with statements.

Add a `"use strict"` directive since this is not inherited from the
enclosing function with indirect eval.

Author: vouillon

runtime/js/jslib.js

@@ -280,7 +280,7 @@ function caml_js_var(x) {
     //console.error("Js.Unsafe.eval_string")
   }
   // biome-ignore lint/security/noGlobalEval:
-  return eval(x);
+  return eval?.(x);
 }
 //Provides: caml_js_call (const, mutable, shallow)
 //Requires: caml_js_from_array
@@ -479,23 +479,23 @@ function caml_js_strict_equals(x, y) {
 //Requires: caml_jsstring_of_string
 function caml_js_eval_string(s) {
   // biome-ignore lint/security/noGlobalEval:
-  return eval(caml_jsstring_of_string(s));
+  return eval?.('"use strict";' + caml_jsstring_of_string(s));
 }
 
 //Provides: caml_js_expr (const)
 //Requires: caml_jsstring_of_string
 function caml_js_expr(s) {
   console.error("caml_js_expr: fallback to runtime evaluation\n");
   // biome-ignore lint/security/noGlobalEval:
-  return eval(caml_jsstring_of_string(s));
+  return eval?.('"use strict";(' + caml_jsstring_of_string(s) + ")");
 }
 
 //Provides: caml_pure_js_expr const (const)
 //Requires: caml_jsstring_of_string
 function caml_pure_js_expr(s) {
   console.error("caml_pure_js_expr: fallback to runtime evaluation\n");
   // biome-ignore lint/security/noGlobalEval:
-  return eval(caml_jsstring_of_string(s));
+  return eval?.('"use strict";(' + caml_jsstring_of_string(s) + ")");
 }
 
 //Provides: caml_js_object (object_literal)