Add add-icon-data script

[LitoMore]

Resolves #7848

I used a simple localeCompare on titles. So it's valid when we add simple icons (the lowercase). I may need to check the duplication of their slugs. Seems our-lint has the same issue.

Overview

Command

npm run add-icon-data

Prompts

field	validation
title	Not empty The title should be unique Its slug should be unique
hex	/^#?[a-f0-9]{3,8}$/i
source	/^https:\/\/[^\s]+$/

The confirm prompt is inspired by npm init command.

Preview

[LitoMore]

I don't have a good idea to implement the UX for aliases. Any ideas?

[mondeja]

Thanks for working on this 👍🏼

I think that the current implementation solves mainly the problem of the ordering in the data file. It does not validate so many things. Additionally, add-icon is really confusing, should be renamed to add-icon-data as the script is not adding an icon, only their object to the data file?

Also, I think that we should standarize the assertions done to our data in tests and reuse them in other parts of the project like this script, but that could be in a separate PR.

[mondeja]

As you're adding a terminal color formatting library we can replace the unicode characters used in scripts/lint/ourlint.js:

simple-icons/scripts/lint/ourlint.js

Line 72 in d5444f6

errors.forEach((error) => console.error(`\u001b[31m${error}\u001b[0m`));

[mondeja]

Related: #7850 and #7851

[LitoMore]

Waiting for #7854 (the locale comparison) and #7855 (the URL regex) to merge.

[mondeja]

Are you planning to add the rest of the fields @LitoMore?

[LitoMore]

Are you planning to add the rest of the fields? (@mondeja)

Let me try.

[mondeja]

Done a couple of changes. Now looks good to me 🌠

Can be improved in the future though, feel free to merge it!