CHANGED: remove datagen artifacts, if they are empty

readme.md

@@ -132,6 +132,10 @@ meta, put the JSON files of these versions (e.g. 1_16_combat-0.json) into the
 "extra-versions" directory
 ```
 
+## Performance
+- Windows filesystem implementation degrades performance very much (especially the commit step is affected by this)
+- Windows defender will also intercept every file I/O, which causes additional slowdown
+
 ## Version Manifest Source
 - The manifest provider source is changeable, `ManifestProvider` needs to be extended.
 - By default, the version manifest information is fetched from [Mojang](https://piston-meta.mojang.com/mc/game/version_manifest_v2.json)

src/main/groovy/com/github/winplay02/gitcraft/pipeline/DatagenStep.java

@@ -135,7 +135,13 @@ public StepResult run(PipelineCache pipelineCache, OrderedVersion mcVersion, Map
 		}
 		Path artifactsRootPath = pipelineCache.getForKey(Step.STEP_FETCH_ARTIFACTS);
 		Path artifactSnbtArchive = getDatagenSNBTArchive(artifactsRootPath);
+		if (Files.exists(artifactSnbtArchive) && Files.size(artifactSnbtArchive) <= 22 /* empty jar */) {
+			Files.delete(artifactSnbtArchive);
+		}
 		Path artifactReportsArchive = getDatagenReportsArchive(artifactsRootPath);
+		if (Files.exists(artifactReportsArchive) && Files.size(artifactReportsArchive) <= 22 /* empty jar */) {
+			Files.delete(artifactReportsArchive);
+		}
 		if ((!GitCraft.config.readableNbt || Files.exists(artifactSnbtArchive)) &&
 				(!GitCraft.config.loadDatagenRegistry || Files.exists(artifactReportsArchive))) {
 			return StepResult.UP_TO_DATE;