[CSG]: Recent changes broke patch's colorizePolygons

[Cloud7050]

When attempting to run the snowglobe sample, the following error is shown in the REPL:

Line 11: ReferenceError: colorizePolygons is not defined

That name occurs in the JSCAD modelling library's patch file. It would appear that recent changes in master may have broken the CSG module.

Line 11 calls intersect() from the CSG module. A trivial example such as render(cube(silver)) still works without error.

[Cloud7050]

@leeyi45 would you happen to know of anything related to the build system that may have changed recently such that this would be impacted?

[martin-henz]

I can confirm that the problem exists as of today, in sourceacademy.org. Here is the direct share link: https://share.sourceacademy.org/ab906

[leeyi45]

I bumped the version of esbuild, that is the only possible breaking change that comes to mind

[joeng03]

Update: render(cube(silver)) also does not currently work at https://share.sourceacademy.org/ab906

[joeng03]

@leeyi45 at which commit was the version on esbuild bumped?

[Cloud7050]

Update: render(cube(silver)) also does not currently work at https://share.sourceacademy.org/ab906

I am currently able to run it:

[Cloud7050]

@leeyi45 at which commit was the version on esbuild bumped?

Likely in cad882252cfd1defc53c78fa4367cef9ef395877.

https://github.com/source-academy/modules/pull/244/files#diff-7ae45ad102eab3b6d7e7896acd08c427a9b25b346470d7bc6507b6481575d519L62-R73

[joeng03]

I am currently able to run it:

Yep, could also run it locally, but not on the website

[Cloud7050]

Yep, could also run it locally, but not on the website

I followed the share link and edited it there. So it was actually running on sourceacademy.org.

Testing on both deployments, neither org nor NUS can run the snowglobe snippet, but they can run the cube.

If the snowglobe works locally, perhaps that's a clue?