`npm run lint` is broken

[EGJ-Moorington]

Running the command npm run lint returns the following error:

ESLint: 8.57.1

TypeError: Converting circular structure to JSON
--> starting at object with constructor 'Object'
| property 'configs' -> object with constructor 'Object'
| property 'flat/recommended-module' -> object with constructor 'Object'
| property 'plugins' -> object with constructor 'Object'
--- property 'n' closes the circle
Referenced from: C:\...\aside-test\.eslintrc.json

This can be fixed by pinning the package gts to a version older than v7.0.0, which switched to the new eslint config format. This is all probably fixed by #46, since pinning to that version works fine. Thanks to @ryohidaka for finding a fix for this issue!

[EGJ-Moorington]

I believe the fix mentioned above worked fine for me, but it broke again:

gts v6.0.2 uses eslint-plugin-n v15.7.0, which when running npm run lint throws the following exception:

(node:3864) [DEP0180] DeprecationWarning: fs.Stats constructor is deprecated.

This can be fixed by using a Node version older than v22 (v21.3.7, for example).

[dsvignacio]

Hi, just encountered this issue now, and maybe the ESLint config file should be updated to match the newer version instead of downgrading to the deprecated Node version.

The newer version should follow the newer configuration standards as specified here, instead of using the old .eslintrc.json?

However, I must admit, I have no idea how to update the config haha. Maybe an expert can chime in.

Thanks though!

[EGJ-Moorington]

[...] maybe the ESLint config file should be updated to match the newer version instead of downgrading to the deprecated Node version.

I agree! However, I wanted to propose these fixes since they should be easier/quicker to implement, and there was already a pull request with some of the fixes.

I believe ESLint provided some tools to make the migration process automatic or at least easier, maybe that could be looked into. I wouldn't mind working on it myself, but I wanted to hear from the repository's maintainers first.

[evil-shrike]

I've come up (thanks Gemini) with the following eslint.config.mjs and it seems to work:

import gts from 'gts';
import globals from 'globals';

export default [
  {
    ignores: [
      'template/**/*',
      'template-ui/**/*',
      '**/node_modules/*',
      'build/*',
      '**/dist/*',
      '**/testing',
    ],
  },
  ...gts,
  {
    languageOptions: {
      globals: {
        ...globals.browser,
        ...globals.node,
      },
    },
  },
];