fix(bulk-model-sync): close input streams after reading and writing JSON files

Author: Oleksandr Dzhychko

bulk-model-sync-lib/src/jvmMain/kotlin/org/modelix/model/sync/bulk/ModelExporter.kt

@@ -35,6 +35,8 @@ actual class ModelExporter actual constructor(private val root: INode) {
         outputFile.parentFile.mkdirs()
 
         val modelData = ModelData(root = root.asExported())
-        Json.encodeToStream(modelData, outputFile.outputStream())
+        outputFile.outputStream().use { outputStream ->
+            Json.encodeToStream(modelData, outputStream)
+        }
     }
 }

bulk-model-sync-lib/src/jvmMain/kotlin/org/modelix/model/sync/bulk/PlatformSpecific.kt

@@ -40,14 +40,11 @@ fun ModelImporter.import(jsonFile: File) {
 
 @OptIn(ExperimentalSerializationApi::class)
 fun ModelImporter.importFilesAsRootChildren(files: Collection<File>) {
-    val models: List<ModelData> = files.map {
-        Json.decodeFromStream(it.inputStream())
+    val models: List<ModelData> = files.map { file ->
+        file.inputStream().use(Json::decodeFromStream)
     }
     import(mergeModelData(models))
 }
 
 @Deprecated("use collection parameter for better performance")
-fun ModelImporter.importFilesAsRootChildren(vararg files: File) {
-    val models = files.map { ModelData.fromJson(it.readText()) }
-    import(mergeModelData(*models.toTypedArray()))
-}
+fun ModelImporter.importFilesAsRootChildren(vararg files: File) = importFilesAsRootChildren(files.asList())