Vm2: Добавить поддержку «правильных» трассировок стека для ошибок в песочнице

Это было бы здорово. Я пытаюсь отследить ошибку кодирования прямо сейчас, и мне чертовски трудно без номера строки

Если вы используете тестовые примеры, которые, как вам кажется, безопасны, вы можете временно заменить vm2 на собственный vm, чтобы отследить ошибку.

Мне тоже нужна эта функция. Я рад написать его, если кто-то укажет мне правильное направление.

Эта проблема была автоматически помечена как устаревшая, поскольку в последнее время не было активности. Он будет закрыт, если больше не будет активности. Спасибо за ваш вклад.

Оказалось, что заменить vm2 на Native vm нетривиально, так как есть несовместимости api, надеюсь, это останется открытым!

@patriksimek , я разработал простую реализацию на основе парсера / перезаписчика исключений. Думаю, стоит интегрировать его в vm2 как дополнительный обработчик ошибок.

const StackTracey = require ('stacktracey');
const { VM } = require('vm2');

const sourceCode = `function f() {
    throw Exception('e');
}
f();`;
const scriptName = "hello_world.js";

process.on("uncaughtException",  e => {
    if (!e.stack.includes("/node_modules/vm2/")) {
        // This is not a vm2 error, so print it normally
        console.log(e);
        return;
    }
    const oldStack = new StackTracey(e);
    const newStack = [];
    for (const line of oldStack) {
        // Discard internal code
        if (line.file.includes("/cjs"))
            continue;
        if (line.thirdParty && line.file.includes("/node_modules/vm2/"))
            continue;
        if (line.callee == "Script.runInContext")
            continue;
        // Replace the default filename with the user-provided one
        if (line.fileName == "vm.js")
            line.fileRelative = line.fileShort = line.fileName = scriptName;
        newStack.push(line);
    }
    console.log("[vm2] A clean stack trace follows:");
    console.log(new StackTracey(newStack).pretty);
});

const vm = new VM();
vm.run(sourceCode, scriptName);

Приведенный выше код напечатает:

ReferenceError: Exception is not defined
    at f (vm.js:2:2)
    at vm.js:4:1
    at Script.runInContext (vm.js:135:20)
    at VM.run (/tmp/vm2-test/node_modules/vm2/lib/main.js:210:72)
    at Object.<anonymous> (/tmp/vm2-test/index.js:33:4)
    at Module._compile (internal/modules/cjs/loader.js:805:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:816:10)
    at Module.load (internal/modules/cjs/loader.js:672:32)
    at tryModuleLoad (internal/modules/cjs/loader.js:612:12)
    at Function.Module._load (internal/modules/cjs/loader.js:604:3)
[vm2] A clean stack trace follows:
at f            hello_world.js:2
at              hello_world.js:4
at <anonymous>  index.js:33       vm.run(sourceCode, scriptName);

У меня не было возможности протестировать эту библиотеку, но я недавно использовал модуль vm и выполнил точные номера трассировки стека, установив соответствующее смещение строки при создании нового vm.Script.

Эта проблема была автоматически помечена как устаревшая, поскольку в последнее время не было активности. Он будет закрыт, если больше не будет активности. Спасибо за ваш вклад.

Подумал, что я прокомментирую, так как эта проблема + предложения все еще актуальны для меня!

Очень актуально и для меня.

@patriksimek может ли реализация
Спасибо.

Патрик, в целом набросок реализации выглядит нормально? Если так, я могу устроить пиар, и мы сможем работать оттуда.

@CapacitorSet Да, я буду рад просмотреть PR. Некоторые примечания:

• Метод vm.run не принимает второй параметр - поэтому имя скрипта игнорируется. В настоящее время вам необходимо создать VMScript с указанным первым именем. Но это можно легко исправить.
• Есть ли способ избежать использования внешней библиотеки? Было бы неплохо придерживаться нулевой глубины.

Просто оставьте комментарий, чтобы уведомить людей, которые подписаны на эту проблему - я открыл PR, так что вы можете отслеживать там прогресс.