String Interpolation

[helloanoop]

String Interpolation

Goal

Today our interpolation logic is duplicated across multiple packages (App, Electron, CLI)
The goal is to centralize a single source of truth for all interpolation logic.

Considerations

• We want to be flexible in terms of key naming conventions
  • Allow variable names to begin with a digit #931
  • Can't create environment variable with a "." in the name (Postman import issue) #964
• We plan to support Nested environments in the future
  • Nested Environments? #292
  • Nested Variables #1098

Moving away from handlebars

I think its time to move away from handlebars.
We don't need the full power of handlebars and we want write a custom interpolation function that meets our needs instead of shoehorning handlebars to match our use case.

Related Tickets

• [BUG] Random ] get added to GraphQL mutation input #1143
• 1st occurrence of process.env.{{VARIABLE}} is not handled when used in Body #1148
• [bug] Scripts are being send it with extra square bracket #1150
• [bug] Escaping double curly bracers creates brackets: \{{foobar}} > {{[foobar]}} #1367
• [Bug] Unexpected Prepended Character Found in JSON String Sent in the Request Body #1321
• How to escape {{...}} ? #1410

[helloanoop]

I have a working PR.

@nelup20 @martinsefcik @Its-treason @mirkogolze Requesting Review.

[helloanoop]

I have integrated the new common interpolation librray with App, Electron and CLI.

Have also started maintain a test suite via a collection that is also run as a part of github action
The test suite should give us more confidence in refactoring code in the future

Also happy to bid adieu to Handlebars. We were clearly shoehorning our use case into it. The new interpolation function has extensive tests cases matching our requirements and we can keep adding more cases in the future.