The user's code should be transformed alongside the injected code

[ojeytonwilliams]

Since the head, contents and tail are evaluated together, they should be Babelified together.

Mostly this is not a problem, but there are edge cases where it breaks. For example https://github.com/freeCodeCamp/freeCodeCamp/blob/237f11be8b777be582e9cfc8d73a3fef907aefa2/curriculum/challenges/english/02-javascript-algorithms-and-data-structures/intermediate-algorithm-scripting/make-a-person.md tries to reassign a const. This should not be allowed, but no errors are thrown because Babel only adds an error when it detects reassignment, but it cannot because Babel evaluates the tail separately from the contents.

[raisedadead]

Anyone willing to work on this should:

• look into transpiling the user code alongside the seed and any other setup code we have
• look into getting the tests to evaluate this final transpiled code

As a helpful note babel is already available in the browser. See us in the contributors' chat if you need more help or guidance.

[raisedadead]

@ojeytonwilliams could you help us with a sample challenge to help test the solution when its ready?

[shuaib128]

@raisedadead is the bug still open? Or the bug has been closed?

[ojeytonwilliams]

It's still an issue, though it's not a serious one. The problem is that https://www.freecodecamp.org/learn/javascript-algorithms-and-data-structures/intermediate-algorithm-scripting/make-a-person isn't entirely valid JavaScript, when you take into account the code we append to the learner's. It tries to reassign a const, which should fail, but does not.

In this particular challenge, it's fine, it just illustrates that the code is not be evaluated as you would expect.