Incorrect behavior of import/order in a monorepo setup - using internal package in workspace about eslint-plugin-import

[kynesis-root]

pathGroupsExcludedImportTypes: ["builtin", "object"]

thanks it worked

[mpellegrini]

The plugin doesn't know that '@workspace/*` is internal unless you tell it to treat as internal.

can solve one of two ways:

1. via eslint settings config

  settings: {
    'import/internal-regex': '^@workspace/',
  },

or via path group in this rules config see: https://github.com/import-js/eslint-plugin-import/blob/main/docs/rules/order.md#pathgroups-array-of-objects

[kynesis-root]

The plugin doesn't know that '@workspace/*` is internal unless you tell it to treat as internal.

can solve one of two ways:

1. via eslint settings config

  settings: {
    'import/internal-regex': '^@workspace/',
  },

or via path group in this rules config see: https://github.com/import-js/eslint-plugin-import/blob/main/docs/rules/order.md#pathgroups-array-of-objects

But I already did the second option,

{
						pattern: '@workspace/**',
						group: 'external',
						position: 'after',
					},

You can see it in the main post

[mpellegrini]

Apologies, I missed that in your config. You are using group: 'external' and using the default set of values for pathGroupsExcludedImportTypes. The default values are ["builtin", "external", "object"]. so if you add the following config it will group as you are expecting. I normally treat these workspace dependencies as internal so normally don't have an issue.

'import/order': [
      'error',
      {
        groups: ['builtin', 'external', 'internal', 'parent', 'sibling', 'index', 'object'],
        pathGroupsExcludedImportTypes: ["builtin", "object"]
        pathGroups: [
          {
            pattern: '@workspace/**',
            group: 'external',
            position: 'after',
          },
          {
            pattern: '@/**',
            group: 'external',
            position: 'after',
          },
        ],
        distinctGroup: true,
        'newlines-between': 'always',
      },
    ],