hacklab/strudel
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

use Function instead of eval

Browse source 
- fixes scope issues (e.g. seq duplication)
- should be much safer and much faster
This commit is contained in:
Felix Roos 2022-05-15 00:35:17 +02:00
parent 1cbf4c8c7e
commit 273cb81da0
1 changed files with 5 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

6
packages/eval/evaluate.mjs
View file

@ -15,9 +15,13 @@ export const extend = (...args) => {
Object.assign(globalThis, ...args); Object.assign(globalThis, ...args);
}; };
function safeEval(str) {
return Function('"use strict";return (' + str + ')')();
}
export const evaluate = async (code) => { export const evaluate = async (code) => {
const shapeshifted = shapeshifter(code); // transform syntactically correct js code to semantically usable code const shapeshifted = shapeshifter(code); // transform syntactically correct js code to semantically usable code
let evaluated = await eval(shapeshifted); let evaluated = await safeEval(shapeshifted);
if (!isPattern(evaluated)) { if (!isPattern(evaluated)) {
console.log('evaluated', evaluated); console.log('evaluated', evaluated);
const message = `got "${typeof evaluated}" instead of pattern`; const message = `got "${typeof evaluated}" instead of pattern`;