[import/order] "import type" always should be recognized as 'type' group

[Bardiamist]

Problem

import type {
  RouteName,
} from '~/enums';

should be recognized as type.

If import type then don't need to use pathGroups. Becuase sometimes I can use RouteName as value, sometomes only as type. Then import type keywords must take precedence on group recognition.

Full rules

'import/order': ['error', {
      'newlines-between': 'always',
      'groups': [
        'type',
        'external',
        'builtin',
        'parent',
        'object',
        'sibling',
      ],
      'pathGroups': [
        {
          'pattern': '~/types/**',
          'group': 'type',
        },
        {
          'pattern': '**/types',
          'group': 'type',
        },
        {
          'pattern': '~/**',
          'group': 'builtin',
        },
      ],
    }],

[ljharb]

Please check in the newly released v2.23.3 (#2092).

[Bardiamist]

@ljharb not helped

[geraintwhite]

@Bardiamist using the following option would prevent type imports being handled by pathGroups:

"pathGroupsExcludedImportTypes": ["type"],

[Bardiamist]

@grit96 no, that not helped

[geraintwhite]

The following test passes successfully with the latest plugin version using the config and code you provided plus the option I mentioned above:

test(
  {
    code: `
      import type { ViewStyle, TextStyle } from 'react-native';
      import type { Color } from '~/enums';
    `,
    parser,
    options: [
      {
        'newlines-between': 'always',
        'groups': [
          'type',
          'external',
          'builtin',
          'parent',
          'object',
          'sibling',
        ],
        'pathGroups': [
          {
            'pattern': '~/types/**',
            'group': 'type',
          },
          {
            'pattern': '**/types',
            'group': 'type',
          },
          {
            'pattern': '~/**',
            'group': 'builtin',
          },
        ],
        'pathGroupsExcludedImportTypes': ['type'],
      },
    ],
  },
  parserConfig,
),

[Bardiamist]

I'm not sure why that not applied at first time (tried twice). It realy working nice now. Thank you