chore: update eslint-plugin-import-x

[JounQin]

Related un-ts/eslint-plugin-import-x#266

cc @mshima

---

Please make sure the below checklist is followed for Pull Requests.

• All continuous integration tests are green
• Tests are added where necessary
• The JDL part is updated if necessary
• jhipster-online is updated if necessary
• Documentation is added/updated where necessary
• Coding Rules & Commit Guidelines as per our CONTRIBUTING.md document are followed

When you are still working on the PR, consider converting it to Draft (below reviewers) and adding skip-ci label, you can still see CI build result at your branch.

[JounQin]

Why ubuntu-20.04 is used instead of ubuntu-latest?

[mshima]

Thanks @JounQin.
The issue is happening here:

generator-jhipster/lib/eslint/base.js

Line 13 in fa9eb9b

...imports.flatConfigs.recommended.plugins,

We apply selected rules that have auto fix to generated code.

Eslint logic is here:
https://github.com/jhipster/generator-jhipster/blob/fa9eb9b71c502d0710a10eb172cdd394b84d34f5/generators/bootstrap/support/eslint-worker.js

[mshima]

@JounQin I've picked the config change here #29089.
I hope that's ok.

[JounQin]

@JounQin I've picked the config change here #29089. I hope that's ok.

Not sure to understand the downgrade commit.

[mshima]

The segmentation fault happens on the thread teardown (piscina pool destroy):

generator-jhipster/generators/bootstrap/support/eslint-transform.ts

Line 79 in fa9eb9b

await pool.destroy();

[JounQin]

Why ubuntu-20.04 is used instead of ubuntu-latest?

I think it could be related?

[mshima]

Why ubuntu-20.04 is used instead of ubuntu-latest?

I think it could be related?

No, #29070

[mshima]

@JounQin I've picked the config change here #29089. I hope that's ok.

Not sure to understand the downgrade commit.

New version doesn’t pass CI.

[JounQin]

@mshima By the way, why eslint-plugin-import-x is in dependencies, eslint-import-resolver-typescript and other eslint plugins are in devDependencies?

[mshima]

@mshima By the way, why eslint-plugin-import-x is in dependencies, eslint-import-resolver-typescript and other eslint plugins are in devDependencies?

eslint-plugin-import-x is applied to projects generated by JHipster at generation time.
Some rules only.

[JounQin]

I believe this should be related to napi-rs

https://github.com/unrs/unrs-resolver/pull/32/files#diff-7ae45ad102eab3b6d7e7896acd08c427a9b25b346470d7bc6507b6481575d519R13-R14

cc @Brooooooklyn Does --strip change anything?

[JounQin]

@mshima I just upgraded unrs-resolver by removing --strip flag, can you help to rerun the workflows?

[JounQin]

@mshima Wow, seems passing...

[mshima]

@JounQin thanks for your help.