fix: template render error [INS-4962]

[filfreire]

This PR closes INS-4962. Changes:

• A "dirty" fix to the issue - follows same approach as

  insomnia/packages/insomnia/src/ui/components/codemirror/extensions/nunjucks-tags.ts

  Lines 283 to 288 in b7e9597

  	const cleanedStr = str
  	.replace(/^{%/, '')
  	.replace(/%}$/, '')
  	.replace(/^{{/, '')
  	.replace(/}}$/, '')
  	.trim();

• Reproduction of the bug added to pre-request-script-window smoke test to help test a better a fix in the future.

context

The bug was introduced after 10.3.0 due to the sanitization code we added in order to solve another bug (see 90bd25c#diff-28ee43a50d28cabb98317967663a3abd59d3e11c9fafa39bca88eed9d290c0b6R173).

Something about the variables.replaceIn(sanitizedRawUrl) call that happens on the insomnia sdk does not work well with this particular format of template tags {{ _['host_environment'] }}.

notes

One important thing to note, is that the code-path to render template tags in insomnia-sdk is a different one than the one we use on the app (for example for rendering the templates in the preview of the URL )

• The app we use this render function:

  insomnia/packages/insomnia/src/common/render.ts

  Line 262 in b7e9597

  export async function render<T>(

• But in the SDK we use this one:

  insomnia/packages/insomnia-sdk/src/objects/interpolator.ts

  Line 11 in b7e9597

  render = (template: string, context: object) => {

We might want to consider merging both to make use of old edge case fixes we already have done before the sdk.

Another side-note is that to debug this issue is a bit tricky as it requires:

1. start hidden browser window
2. send a request
3. show hidden browser window
4. going to dev tools of hidden browser window

[gatzjames]

✅