Update NightConfig and cleanup config code

[Technici4n]

Change list:

• Update to the latest NightConfig. Configs are now threadsafe, and reloading works correctly with Notepad++.
• Remove internal ConfigConfig tech debt.
• Redesign IConfigSpec a bit to have a better-defined contract.
• Inline the methods from ConfigFileTypeHandler into ConfigTracker.
• Use the correct mechanism to create default configurations. This suppresses the ugly Incorrect key xxx was corrected from xxx to its default, xxx. warnings on the first startup.
• Separate access to registered configs via the new ModConfigs from the internal ConfigTracker.
• Add a few basic config loading and reloading tests.
• Design the multithreading model for configs:
  • Some operations such as firing events now need to take a lock. This makes sure that reloading and loading cannot fire concurrently.
  • Config objects themselves are threadsafe, and may in fact get reloaded at any time from another thread.
  • If a Config changes, the values will eventually be updated, and an event will eventually be fired to notify the mod.
• Fire ModConfigEvent.Reloading on the client-side when a new config is received.

[neoforged-pr-publishing[bot]]

• [x] Publish PR to GitHub Packages

Last commit published: 2c5679cf0c3d1b92e3a6db6406af4285698c41d1.

PR Publishing

### The artifacts published by this PR: - :package: [`net.neoforged.fancymodloader:earlydisplay:4.0.35-pr-172-config-rework`](https://github.com/neoforged/FancyModLoader/packages/2197400) - :package: [`net.neoforged.fancymodloader:junit-fml:4.0.35-pr-172-config-rework`](https://github.com/neoforged/FancyModLoader/packages/2197401) - :package: [`net.neoforged.fancymodloader:loader:4.0.35-pr-172-config-rework`](https://github.com/neoforged/FancyModLoader/packages/2197402) ### Repository Declaration In order to use the artifacts published by the PR, add the following repository to your buildscript: ```gradle repositories { maven { name 'Maven for PR #172' // https://github.com/neoforged/FancyModLoader/pull/172 url 'https://prmaven.neoforged.net/FancyModLoader/pr172' content { includeModule('net.neoforged.fancymodloader', 'earlydisplay') includeModule('net.neoforged.fancymodloader', 'junit-fml') includeModule('net.neoforged.fancymodloader', 'loader') } } } ```

[neoforged-compatibility-checks[bot]]

@Technici4n, this PR introduces breaking changes. Fortunately, this project is currently accepting breaking changes, but if they are not intentional, please revert them. Last checked commit: 2c5679cf0c3d1b92e3a6db6406af4285698c41d1.

Compatibility checks

## `loader (:loader)` - `net/neoforged/fml/config/IConfigSpec` * ❗ Class missing interface: com/electronwill/nightconfig/core/UnmodifiableConfig * `self()Lnet/neoforged/fml/config/IConfigSpec;`: ❗ API method was removed * `isEmpty()Z`: ❗ Method was made abstract * `isCorrecting()Z`: ❗ API method was removed * `correct(Lcom/electronwill/nightconfig/core/CommentedConfig;)I`: ❗ API method was removed * `correct(Lcom/electronwill/nightconfig/core/CommentedConfig;)V`: ❗ Method was made abstract * `isCorrect(Lcom/electronwill/nightconfig/core/UnmodifiableCommentedConfig;)Z`: ❗ Method was made abstract * `afterReload()V`: ❗ API method was removed * `isCorrect(Lcom/electronwill/nightconfig/core/CommentedConfig;)Z`: ❗ API method was removed - `net/neoforged/fml/ModContainer` * `addConfig(Lnet/neoforged/fml/config/ModConfig;)V`: ❗ Method was lowered in visibility - `net/neoforged/fml/config/ModConfig` * ❗ Class was made final * `(Lnet/neoforged/fml/config/ModConfig$Type;Lnet/neoforged/fml/config/IConfigSpec;Lnet/neoforged/fml/ModContainer;)V`: ❗ API method was removed * `save()V`: ❗ API method was removed * `acceptSyncedConfig([B)V`: ❗ API method was removed * `(Lnet/neoforged/fml/config/ModConfig$Type;Lnet/neoforged/fml/config/IConfigSpec;Lnet/neoforged/fml/ModContainer;Ljava/lang/String;)V`: ❗ API method was removed * `getConfigData()Lcom/electronwill/nightconfig/core/CommentedConfig;`: ❗ API method was removed * `container:Lnet/neoforged/fml/ModContainer;`: ❗ Field was lowered in visibility - `net/neoforged/fml/config/ConfigFileTypeHandler` * ❗ API class no longer exists - `net/neoforged/fml/config/ConfigTracker` * `configSets()Ljava/util/Map;`: ❗ API method was removed * `fileMap()Ljava/util/concurrent/ConcurrentHashMap;`: ❗ API method was removed * `openConfig(Lnet/neoforged/fml/config/ModConfig;Ljava/nio/file/Path;Ljava/nio/file/Path;)V`: ❗ API method was removed * `getConfigFileName(Ljava/lang/String;Lnet/neoforged/fml/config/ModConfig$Type;)Ljava/lang/String;`: ❗ API method was removed