feat: ⚙️ Refactor pattern code structure

[PKief]

Reduce amount of code and amount of logic to apply the patterns to the file icons.

[github-actions]

Merge Successful

Thanks for your contribution! 🎉

The changes will be part of the upcoming update on the marketplace.

[Airkro]

Looks like the cosmiconfig pattern lose loss many extensions, it's in the plan?

[PKief]

Looks like the cosmiconfig pattern lose loss many extensions, it's in the plan?

Which extensions are missing?

[Airkro]

Run:

import { fileIcons } from './fileIcons';

const io = fileIcons.icons[220].fileNames?.join('\n');

console.log(io);

Get:

Before--------------------------------------

After-------------------------------

.stylelintrc .stylelintrc.json .stylelintrc.jsonc .stylelintrc.json5 .stylelintrc.yaml .stylelintrc.yml .stylelintrc.toml .stylelintrc.js .stylelintrc.mjs .stylelintrc.cjs .stylelintrc.ts .stylelintrc.mts .stylelintrc.cts .config/stylelintrc .config/stylelintrc.json .config/stylelintrc.jsonc .config/stylelintrc.json5 .config/stylelintrc.yaml .config/stylelintrc.yml .config/stylelintrc.toml .config/stylelintrc.js .config/stylelintrc.mjs .config/stylelintrc.cjs .config/stylelintrc.ts .config/stylelintrc.mts .config/stylelintrc.cts stylelint.config.json stylelint.config.jsonc stylelint.config.json5 stylelint.config.yaml stylelint.config.yml stylelint.config.toml stylelint.config.js stylelint.config.mjs stylelint.config.cjs stylelint.config.ts stylelint.config.mts stylelint.config.cts .stylelintignore .stylelintcache

.stylelintrc .config/stylelintrc stylelint.config stylelint.js stylelint.mjs stylelint.cjs stylelint.ts stylelint.mts stylelint.cts stylelint.json stylelint.jsonc stylelint.json5 stylelint.yaml stylelint.yml stylelint.toml .stylelintignore .stylelintcache

[PKief]

@Airkro thanks a lot, no it was not on purpose to remove some patterns. I've restored it, so that it should cover all patterns now: fe98e7c. The reason for the refactoring was to make the code more self-explanatory and more explicit what kind of file names are used for each pattern. I know that the code looks a bit more comprehensive but I prefer it that way.

[Airkro]

These lines should be removed, they are not part of Cosmiconfig.
Sorry to show them in the debug code before.

vscode-material-icon-theme/src/icons/patterns/patterns.ts

Lines 94 to 95 in bcc8288

	`.${fileName}ignore`,
	`.${fileName}cache`,

[Airkro]

Looks like the cosmiconfig pattern lose loss many extensions, it's in the plan?

We should add some test cases for function parseByPattern, to avoid bugs like this.

[PKief]

@Airkro thanks for the hint. I've updated the code and also added a unit test for the parseByPattern function :)

See commit: d8bdd33