refactor(terraform): consolidate output parsing logic into locals.tf …

…for improved maintainability and error handling (#11)

* refactor(terraform): consolidate output parsing logic into locals.tf for improved maintainability and error handling

* terraform-docs: automated action

---------

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

README.md

@@ -13,7 +13,6 @@
 
 | Name | Version |
 |------|---------|
-| <a name="provider_local"></a> [local](#provider\_local) | ~> 2.5 |
 | <a name="provider_null"></a> [null](#provider\_null) | ~> 3.0 |
 
 ## Modules
@@ -25,7 +24,6 @@ No modules.
 | Name | Type |
 |------|------|
 | [null_resource.create_cluster](https://registry.terraform.io/providers/hashicorp/null/latest/docs/resources/resource) | resource |
-| [local_file.cluster_output](https://registry.terraform.io/providers/hashicorp/local/latest/docs/data-sources/file) | data source |
 
 ## Inputs
 

locals.tf

@@ -73,5 +73,24 @@ locals {
     )
   )
 
+
   output_file = "${path.module}/cluster_output.txt"
+
+  # Safely read the output file if it exists
+  raw_output = try(
+    fileexists(local.output_file) ? file(local.output_file) : "",
+    ""
+  )
+
+  # Parse the output lines safely
+  output_lines = compact(split("\n", local.raw_output))
+
+  # Create the output map with proper error handling
+  output_map = {
+    for line in local.output_lines :
+    split("::", line)[0] => split("::", line)[1]
+    if length(split("::", line)) == 2
+  }
 }
+
+

main.tf

@@ -18,21 +18,3 @@ resource "null_resource" "create_cluster" {
   }
 }
 
-data "local_file" "cluster_output" {
-  filename   = local.output_file
-  depends_on = [null_resource.create_cluster]
-}
-
-locals {
-  output_lines = try(
-    compact(split("\n", data.local_file.cluster_output.content)),
-    []
-  )
-  output_map = try(
-    { for line in local.output_lines :
-      split("::", line)[0] => split("::", line)[1]
-      if length(split("::", line)) == 2
-    },
-    {}
-  )
-}