fix: Convert destructuring aliases to import aliases in CommonJS->ESM

[mohammedahmed18]

Problem

When converting CommonJS require statements with destructuring aliases to ESM imports, the code was preserving the JavaScript object destructuring syntax a: b which is invalid in ES6 import statements. This caused syntax errors when running Jest tests:

x Expected ',', got ':'
import { db: dbCore } from '@budibase/backend-core';
             ^

Solution

Added a helper function _convert_destructuring_to_imports() that converts destructuring alias syntax to import alias syntax:

• { a: b } → { a as b }
• { foo, bar: aliasB } → { foo, bar as aliasB }

Changes

• Modified codeflash/languages/javascript/module_system.py:
  • Added _convert_destructuring_to_imports() helper function
  • Updated replace_destructured() to use the helper
  • Updated docstring for convert_commonjs_to_esm()
• Added comprehensive unit tests in tests/test_languages/test_javascript_module_system.py:
  • Test simple destructuring
  • Test destructuring with aliases
  • Test mixed destructuring (some aliased, some not)
  • Test real-world Budibase case that was failing

Testing

• ✅ All 26 unit tests passing
• ✅ Tested the real-world Budibase case that was failing

Fixes the "Expected ',', got ':'" syntax error in generated Jest tests for TypeScript projects.