fix: terraform templatefile syntax issues and validation

Author: toti85

registry/toti85/modules/auto-dev-server/main.tf

@@ -24,10 +24,10 @@ variable "enabled_frameworks" {
   description = "List of frameworks to detect and auto-start. Available: nodejs, rails, django, flask, fastapi, spring, go, rust, php"
   type        = list(string)
   default     = ["nodejs", "rails", "django", "flask", "fastapi", "spring", "go", "rust", "php"]
-  
+
   validation {
     condition = alltrue([
-      for framework in var.enabled_frameworks : 
+      for framework in var.enabled_frameworks :
       contains(["nodejs", "rails", "django", "flask", "fastapi", "spring", "go", "rust", "php"], framework)
     ])
     error_message = "Invalid framework. Allowed values: nodejs, rails, django, flask, fastapi, spring, go, rust, php"
@@ -44,7 +44,7 @@ variable "log_level" {
   description = "Log level for auto-dev-server. Options: DEBUG, INFO, WARN, ERROR"
   type        = string
   default     = "INFO"
-  
+
   validation {
     condition     = contains(["DEBUG", "INFO", "WARN", "ERROR"], var.log_level)
     error_message = "Log level must be one of: DEBUG, INFO, WARN, ERROR"
@@ -66,50 +66,63 @@ variable "custom_commands" {
 locals {
   # Default commands for each framework
   default_commands = {
-    nodejs   = "npm start"
-    rails    = "rails server"
-    django   = "python manage.py runserver 0.0.0.0:8000"
-    flask    = "flask run --host=0.0.0.0"
-    fastapi  = "uvicorn main:app --host 0.0.0.0 --port 8000"
-    spring   = "./mvnw spring-boot:run"
-    go       = "go run ."
-    rust     = "cargo run"
-    php      = "php -S 0.0.0.0:8000"
+    nodejs  = "npm start"
+    rails   = "rails server"
+    django  = "python manage.py runserver 0.0.0.0:8000"
+    flask   = "flask run --host=0.0.0.0"
+    fastapi = "uvicorn main:app --host 0.0.0.0 --port 8000"
+    spring  = "./mvnw spring-boot:run"
+    go      = "go run ."
+    rust    = "cargo run"
+    php     = "php -S 0.0.0.0:8000"
   }
-  
+
   # Merge custom commands with defaults
   framework_commands = merge(local.default_commands, var.custom_commands)
-  
+
   # Detection patterns for each framework
   detection_patterns = {
-    nodejs   = "package.json"
-    rails    = "Gemfile|config.ru|app/controllers"
-    django   = "manage.py|settings.py"
-    flask    = "app.py|application.py|wsgi.py"
-    fastapi  = "main.py|app.py"
-    spring   = "pom.xml|build.gradle|src/main/java"
-    go       = "go.mod|main.go"
-    rust     = "Cargo.toml|src/main.rs"
-    php      = "index.php|composer.json"
+    nodejs  = "package.json"
+    rails   = "Gemfile|config.ru|app/controllers"
+    django  = "manage.py|settings.py"
+    flask   = "app.py|application.py|wsgi.py"
+    fastapi = "main.py|app.py"
+    spring  = "pom.xml|build.gradle|src/main/java"
+    go      = "go.mod|main.go"
+    rust    = "Cargo.toml|src/main.rs"
+    php     = "index.php|composer.json"
   }
 }
 
 resource "coder_script" "auto_dev_server" {
-  agent_id           = var.agent_id
-  display_name       = "Auto Development Server"
-  icon               = "/icon/play.svg"
-  script             = templatefile("${path.module}/scripts/auto-dev-server.sh", {
-    project_dir        = var.project_dir
-    enabled_frameworks = jsonencode(var.enabled_frameworks)
-    framework_commands = jsonencode(local.framework_commands)
-    detection_patterns = jsonencode(local.detection_patterns)
-    start_delay        = var.start_delay
-    log_level         = var.log_level
-    use_devcontainer  = var.use_devcontainer
-  })
-  run_on_start       = true
-  run_on_stop        = false
-  timeout            = 300
+  agent_id     = var.agent_id
+  display_name = "Auto Development Server"
+  icon         = "/icon/play.svg"
+  script = replace(
+    replace(
+      replace(
+        replace(
+          replace(
+            replace(
+              replace(
+                file("${path.module}/scripts/auto-dev-server.sh"),
+                "PROJECT_DIR_PLACEHOLDER", var.project_dir
+              ),
+              "ENABLED_FRAMEWORKS_PLACEHOLDER", jsonencode(var.enabled_frameworks)
+            ),
+            "FRAMEWORK_COMMANDS_PLACEHOLDER", jsonencode(local.framework_commands)
+          ),
+          "DETECTION_PATTERNS_PLACEHOLDER", jsonencode(local.detection_patterns)
+        ),
+        "START_DELAY_PLACEHOLDER", tostring(var.start_delay)
+      ),
+      "LOG_LEVEL_PLACEHOLDER", var.log_level
+    ),
+    "USE_DEVCONTAINER_PLACEHOLDER", tostring(var.use_devcontainer)
+  )
+  run_on_start = true
+  run_on_stop  = false
+  timeout      = 300
 }
 
 # Output useful information

registry/toti85/modules/auto-dev-server/scripts/auto-dev-server.sh

@@ -6,13 +6,13 @@
 set -euo pipefail
 
 # Configuration from Terraform variables
-PROJECT_DIR="${project_dir}"
-ENABLED_FRAMEWORKS='${enabled_frameworks}'
-FRAMEWORK_COMMANDS='${framework_commands}'
-DETECTION_PATTERNS='${detection_patterns}'
-START_DELAY=${start_delay}
-LOG_LEVEL="${log_level}"
-USE_DEVCONTAINER=${use_devcontainer}
+PROJECT_DIR="PROJECT_DIR_PLACEHOLDER"
+ENABLED_FRAMEWORKS='ENABLED_FRAMEWORKS_PLACEHOLDER'
+FRAMEWORK_COMMANDS='FRAMEWORK_COMMANDS_PLACEHOLDER'
+DETECTION_PATTERNS='DETECTION_PATTERNS_PLACEHOLDER'
+START_DELAY=START_DELAY_PLACEHOLDER
+LOG_LEVEL="LOG_LEVEL_PLACEHOLDER"
+USE_DEVCONTAINER=USE_DEVCONTAINER_PLACEHOLDER
 
 # Expand variables like $HOME
 PROJECT_DIR=$(eval echo "$PROJECT_DIR")