[order] pathGroup doesn't appear to work

Problem

I am configuring pathGroups, but they do not appear to be working. No matter the group I set for the pattern, it always requires that it appear at the top of the external imports. I’ve manually verified that minimatch is returning the correct output for some of the test paths that I’m working with.

tsconfig.json

"compilerOptions": {
   ...
   "paths": {
     "@/*": ["./*"]
   }
}

.eslintrc.js

  'import/order': [
    'error',
    {
      groups: ['builtin', 'external', ['sibling', 'parent'], 'index', 'internal', 'object'],
      pathGroups: [
        {
          pattern: '@/**',
          group: 'internal',
        },
      ],
      'newlines-between': 'always',
      alphabetize: {
        order: 'asc',
        caseInsensitive: true,
      },
    },
  ],

Expected

import { StatusCodes } from 'http-status-codes';
import { Route, useRouter } from 'next-router-provider';
import React, { useCallback, useEffect, useRef, useState } from 'react';
import styled from 'styled-components';

import { useSession } from '@/client/providers/session';
import { pageRoutes } from '@/client/routes';

Actual

import { useSession } from '@/client/providers/session';
import { pageRoutes } from '@/client/routes';
import { StatusCodes } from 'http-status-codes';
import { Route, useRouter } from 'next-router-provider';
import React, { useCallback, useEffect, useRef, useState } from 'react';
import styled from 'styled-components';

versions

eslint 7.22.0
eslint-plugin-import 2.22.1
typescript 4.2.3

minimatch outputs (both of which are correct)

> minimatch('@/client/types', '@/**')
true
> minimatch('@external-package/foo-package', '@/**')
false

[nickzelei]

@Jack-Barry I’ve been testing and it looks like the config can actually be a bit simpler. The external pathGroup isn’t necessary.

      {
        pathGroups: [
          {
            pattern: '@/**',
            group: 'internal',
          },
        ],
        pathGroupsExcludedImportTypes: ['builtin', 'object'],
      },

gets the job done.

[johnloven]

You can also use an eslint setting in order to interpret specific path patterns as internal:

# .eslintrc.yml
settings:
  import/internal-regex: ^@/

https://github.com/benmosher/eslint-plugin-import/blob/master/README.md#importinternal-regex