Added air live-reload on dev (#453)

Author: mishushakov

.gitignore

@@ -9,3 +9,4 @@ terraform.tfstate
 terraform.tfstate.backup
 **/.tfplan.*
 tests/periodic-test/build-template/e2b.toml
+.air

packages/api/.air.toml

@@ -0,0 +1,19 @@
+root = "."
+tmp_dir = ".air"
+
+[build]
+cmd = "CGO_ENABLED=1 go build -race -gcflags=all=\"-N -l\" -o bin/api ."
+bin = "bin/api"
+full_bin = "ENV=$(cat ../../.last_used_env || echo \"not-set\") && . ../../.env.${ENV} && POSTGRES_CONNECTION_STRING=${POSTGRES_CONNECTION_STRING} SUPABASE_JWT_SECRETS=${SUPABASE_JWT_SECRETS} GOTRACEBACK=crash GODEBUG=madvdontneed=1 TEMPLATE_BUCKET_NAME=${TEMPLATE_BUCKET_NAME} ENVIRONMENT=${ENVIRONMENT} ORCHESTRATOR_PORT=5008 bin/api --port 3000"
+include_ext = ["go", "yml", "yaml"]
+exclude_dir = ["assets", ".air", "vendor", "bin"]
+exclude_regex = ["_test\\.go"]
+exclude_unchanged = true
+delay = 500
+send_interrupt = true # not sure this is actually needed
+
+[log]
+time = true
+
+[screen]
+clear_on_rebuild = false

packages/api/Makefile

@@ -35,6 +35,18 @@ run:
 	ORCHESTRATOR_PORT=5008 \
 	./bin/api --port 3000
 
+# Run the API using air
+.PHONY: dev-init
+dev-init:
+	go install github.com/air-verse/air@latest
+
+.PHONY: dev
+dev:
+	@if [ ! -f "${GOPATH}/bin/air" ]; then \
+		echo "air not found in ${GOPATH}/bin/air, please run 'make dev-init' first"; \
+		exit 1; \
+	fi
+	${GOPATH}/bin/air
 
 # You run the parametrized command like this:
 # make metric=heap interval=90 profiler

packages/api/main.go

@@ -325,7 +325,11 @@ func run() int {
 		// This is a bit of a hack, but this way we can properly propagate
 		// the health status to the load balancer.
 		apiStore.Healthy = false
-		time.Sleep(15 * time.Second)
+
+		// Skip the delay in local environment for instant shutdown
+		if !env.IsLocal() {
+			time.Sleep(15 * time.Second)
+		}
 
 		// if the parent context `ctx` is canceled the
 		// shutdown will return early. This should only happen