How to get an exception on undefined variable when using Context?

[ilg-ul]

Hi,

I have a small issues when using a Context, I can't make parseAndRender() trigger an exception when a variable is not defined:

import { Context, Liquid } from "liquidjs";

const engine = new Liquid({
  strictFilters: true,
  strictVariables: true,
  trimTagLeft: false,
  trimTagRight: false,
  trimOutputLeft: false,
  trimOutputRight: false,
  greedy: false,
  lenientIf: true,
});

const map = {
  s: "a",
};

console.log(await engine.parseAndRender("{{ s }}", map));
try {
  console.log(await engine.parseAndRender("({{ x }})", map));
} catch (e) {
  console.log("Caught error:", e.message);
}

// So far, so good, without context, undefined variables throw.

const context = new Context({
  s: "b",
});

console.log(await engine.parseAndRender("{{ s }}", context));
try {
  console.log(
    await engine.parseAndRender("({{ y }})", context, {
      strictVariables: true,
    }),
  );
} catch (e) {
  console.log("Caught error:", e.message);
}

The result I get when running this code is:

% node ./index.js
a
Caught error: undefined variable: x, line:1, col:5
b
()

I also tried without the second strictVariables: true, with the same result. Is this really needed? I expected it to be propagated from the engine.

Is there anything else I have to configure to make this work?

[jg-rp]

Try passing engine.options as the second argument to the Context constructor.

This is roughly what Liquid.parseAndRender does when creating an implicit context for you.

const context = new Context({ s: "b" }, engine.options, { sync: false });

liquidjs/src/context/context.ts

Line 41 in 02403a1

public constructor (env: object = {}, opts: NormalizedFullOptions = defaultOptions, renderOptions: RenderOptions = {}, { memoryLimit, renderLimit }: { [key: string]: Limiter } = {}) {

[ilg-ul]

That was useful, thank you.

It would be useful to have it explained explicitly in the documentation.