Order bug: pathGroup overrides type group

[atav32]

Given the imports

import c from 'Bar';
import a from 'foo';

import b from 'dirA/bar';

import index from './';

import type { C } from 'dirA/Bar';
import type { A } from 'foo';

and ESLint config

'import/order': [
  error: {
    alphabetize: { order: 'asc' },
    groups: ['external', 'internal', 'index', 'type'],
    pathGroups: [
      {
        pattern: 'dirA/**',
        group: 'internal',
      },
    ],
    'newlines-between': 'always',
    pathGroupsExcludedImportTypes: [],
  },
],

The rules engine currently requires grouping the paths that match the pathGroup pattern together, even if one of the imports is a type import, creating this unintuitive order:

import c from 'Bar';
import a from 'foo';

import type { C } from 'dirA/Bar';
import b from 'dirA/bar';

import index from './';

import type { A } from 'foo';

[atav32]

Ah nvm, even though I read #2088, I made a typo and had pathGroupsExcludedImportTypes: ['types'], which is why I though it was broken.

I didn't understand how pathGroupsExcludedImportTypes worked until I played with the computeRank().