chore: optimize CI pipeline and fix JSONB NULL scan error

- Add Go module caching (cache: true) to all workflows
- Add Docker layer caching via GitHub Actions cache in main workflow
- Add Elasticsearch health check to avoid test flakiness
- Skip CI for docs-only changes (paths-ignore for *.md and docs/**)
- Remove unnecessary fetch-depth: 0 from lint and main workflows
- Remove duplicate go mod tidy (Makefile clean no longer depends on tidy)
- Update Dockerfile.dev to use parameterized Go version (1.25) and alpine 3.21
- Fix JSONMap.Scan() to handle NULL JSONB values from PostgreSQL

Author: ravisuhag

.github/workflows/lint.yml

@@ -1,17 +1,20 @@
 name: Lint
 
-on: [push]
+on:
+  push:
+    paths-ignore:
+      - "**.md"
+      - "docs/**"
 
 jobs:
   golangci:
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v6
-        with:
-          fetch-depth: 0
       - uses: actions/setup-go@v6
         with:
           go-version-file: "go.mod"
+          cache: true
       - name: golangci-lint
         uses: golangci/golangci-lint-action@v9
         with:

.github/workflows/main.yml

@@ -3,19 +3,23 @@ on:
   push:
     branches:
       - main
+    paths-ignore:
+      - "**.md"
+      - "docs/**"
 
 jobs:
   dev:
     runs-on: ubuntu-latest
     steps:
       - name: Checkout code
         uses: actions/checkout@v6
-        with:
-          fetch-depth: 0
       - name: Set up Go
         uses: actions/setup-go@v6
         with:
           go-version-file: "go.mod"
+          cache: true
+      - name: Set up Docker Buildx
+        uses: docker/setup-buildx-action@v3
       - name: Login to DockerHub
         uses: docker/login-action@v4
         with:
@@ -29,3 +33,5 @@ jobs:
           push: true
           file: "./Dockerfile.dev"
           tags: raystack/compass:dev
+          cache-from: type=gha
+          cache-to: type=gha,mode=max

.github/workflows/test.yml

@@ -1,5 +1,13 @@
 name: Test
-on: [push, pull_request]
+on:
+  push:
+    paths-ignore:
+      - "**.md"
+      - "docs/**"
+  pull_request:
+    paths-ignore:
+      - "**.md"
+      - "docs/**"
 
 jobs:
   test:
@@ -13,17 +21,21 @@ jobs:
           discovery.type: single-node
           xpack.security.enabled: "false"
           ES_JAVA_OPTS: "-Xms128m -Xmx128m"
+        options: >-
+          --health-cmd "curl -f http://localhost:9200/_cluster/health"
+          --health-interval 10s
+          --health-timeout 5s
+          --health-retries 10
     env:
       ES_TEST_SERVER_URL: "http://localhost:9200"
     steps:
       - uses: actions/checkout@v6
       - uses: actions/setup-go@v6
         with:
           go-version-file: "go.mod"
+          cache: true
       - name: Install dependencies
         run: sudo apt-get install build-essential
-      - name: Install packages
-        run: go mod tidy
       - name: Run Test
         run: make test
       - name: Install goveralls and send coverage

Dockerfile.dev

@@ -1,10 +1,11 @@
-FROM golang:1.20-alpine3.17 as builder
+ARG GO_VERSION=1.25
+FROM golang:${GO_VERSION}-alpine3.21 as builder
 RUN apk add make
 WORKDIR /build/
 COPY . .
 RUN make build
 
-FROM alpine:3.17
+FROM alpine:3.21
 
 COPY --from=builder /build/compass /usr/bin/compass
 RUN apk update

Makefile

@@ -25,7 +25,7 @@ lint: ## Lint checker
 	@echo "Running lint checks using golangci-lint..."
 	@golangci-lint run
 
-clean: tidy ## Clean the build artifacts
+clean: ## Clean the build artifacts
 	@echo "Cleaning up build directories..."
 	@rm -rf $coverage.out ${BUILD_DIR}
 

internal/store/postgres/asset_model.go

@@ -156,6 +156,10 @@ func (m JSONMap) Value() (driver.Value, error) {
 }
 
 func (m *JSONMap) Scan(value interface{}) error {
+	if value == nil {
+		*m = JSONMap{}
+		return nil
+	}
 	var ba []byte
 	switch v := value.(type) {
 	case []byte: