Incorrect order of initialization variables

[sirenkovladd]

• Version (npm -v): 10.5.2
• Node Version node -v: v20.13.1
• OS (uname -a on Linux): Darwin arm64

Incorrect order of initialization variables

import { minify } from "@putout/minify";

const source = `
const c = (e) => e.w;
const w = (c) => c;
function f(e) {
  const { n } = e;
  e.w = n;
  const q = c(e);
  return q && w(q)
}
console.log(f({n:1}))`;

eval(source);
const result = minify(source);
console.log(result);
eval(result);

❯ node test.js
1
var c=e=>e.w,w=c=>c;function f(e){var{n}=e,q=c(e);e.w=n;return q&&w(q)}console.log(f({n:1}));
undefined

[coderaiser]

Fixed in #17 🎉. Is it works for you?

[sirenkovladd]

No, same result

"dependencies": {
    "@putout/minify": "^3.16.0"
  }

[coderaiser]

Try to re-install @putout/minify.

[sirenkovladd]

❯ node index.js
1
var c=e=>e.w,w=c=>c;function f(e){const{n}=e,q=c(e);e.w=n;return q&&w(q)}console.log(f({n:1}));
undefined

❯ cat node_modules/@putout/minify/package.json | jq '.version'
"3.17.1"

[coderaiser]

Added ability to disable mergeVariables.

[sirenkovladd]

But it's a useful feature I'd like to use
the problem is that it may not work correctly in such scenarios

[coderaiser]

Here is the code, you can play with it, if you get any ideas how to make this work, I’ll merge.

[coderaiser]

Fixed in latest version of @putout/minify 🎉