feat: Optimize CI/CD workflows with conditional job execution and file change detection (#10)

## Summary

This PR significantly improves CI/CD efficiency by implementing
conditional job execution based on file changes. Jobs now only run when
relevant files are modified, reducing unnecessary workflow runs and
improving build times.

## Key Improvements

### 🚀 Performance Optimization
- **Smart File Change Detection**: Added `Detect Changes` job using
`dorny/paths-filter` to identify modified file types
- **Conditional Job Execution**: Jobs only run when their relevant files
change (e.g., Go tests run only when `.go` files change)
- **Resource Efficiency**: Reduces CI minutes usage by skipping
irrelevant jobs

### 📋 Workflow Changes

#### CI Workflow (`ci.yml`)
- Added `changes` job that detects modifications in:
  - Go files (`*.go`, `go.mod`, `go.sum`, `.golangci.yml`)
  - Scripts and workflow files
- Made all jobs conditional based on detected changes:
  - `lint`: Runs only when Go files change
  - `test`: Runs only when Go files change
  - `acceptance-test`: Runs only when Go files change
  - `build-provider`: Runs only when Go files change
  - `terraform-compatibility`: Runs only when Go files change
- Updated `ci-status` job to handle skipped jobs correctly

#### Validation Workflow (`validation.yml`)
- Added file change detection for:
  - Markdown files (`*.md`)
  - YAML files (`*.yml`, `*.yaml`)
  - JSON files (`*.json`, `*.jsonc`)
  - Terraform files (`*.tf`, `*.tfvars`)
  - Scripts
- Made validation jobs conditional:
- `file-validation`: Runs specific checks only when relevant files
change
  - `markdown-lint`: Runs only when Markdown files change
  - `terraform-fmt`: Runs only when Terraform files change

#### Pre-release Workflow
- **Separated Concerns**: Extracted pre-release job into dedicated
workflow (`.github/workflows/pre-release.yml`)
- **Simplified Triggers**: Removed `push` triggers from `ci.yml` and
`pre-commit.yml`
- **Better Organization**: Pre-release now triggers only on pushes to
`main` branch

### 🔧 Technical Details

#### Status Check Script Enhancement
Updated `scripts/ci-status-check.sh` to properly handle job statuses:
- `skipped` jobs are now considered acceptable (when no relevant files
changed)
- Better status reporting with counts of successful vs skipped jobs
- Improved error handling for cancelled jobs

## Benefits

1. **Faster PR Feedback**: Developers get quicker CI results by running
only relevant checks
2. **Cost Savings**: Reduced GitHub Actions minutes usage
3. **Better Maintainability**: Clearer separation of concerns with
dedicated workflows
4. **Improved Developer Experience**: Less waiting for irrelevant jobs
to complete

## Testing

- All conditional logic has been tested with various file change
scenarios
- Status checks properly handle skipped jobs
- Pre-release workflow functions independently on main branch

## Migration Notes

No action required. These changes are backward compatible and will
automatically optimize future CI runs.

Author: Trozz

.github/workflows/ci.yml

@@ -1,10 +1,6 @@
 name: CI
 
 on:
-  push:
-    branches:
-      - main
-      - develop
   pull_request:
     branches:
       - main
@@ -21,9 +17,35 @@ permissions:
   pull-requests: write
 
 jobs:
+  changes:
+    name: Detect Changes
+    runs-on: ubuntu-latest
+    outputs:
+      go: ${{ steps.filter.outputs.go }}
+      scripts: ${{ steps.filter.outputs.scripts }}
+    steps:
+      - name: Checkout code
+        uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4
+
+      - name: Check for file changes
+        uses: dorny/paths-filter@v3
+        id: filter
+        with:
+          filters: |
+            go:
+              - '**/*.go'
+              - 'go.mod'
+              - 'go.sum'
+              - '.golangci.yml'
+            scripts:
+              - 'scripts/**'
+              - '.github/workflows/ci.yml'
+
   lint:
     name: Lint
     runs-on: ubuntu-latest
+    needs: changes
+    if: needs.changes.outputs.go == 'true'
     steps:
       - name: Checkout code
         uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4
@@ -37,12 +59,25 @@ jobs:
       - name: Run go mod tidy
         run: go mod tidy
 
+      - name: Check for Go file changes
+        uses: dorny/paths-filter@v3
+        id: filter
+        with:
+          filters: |
+            go:
+              - '**/*.go'
+              - 'go.mod'
+              - 'go.sum'
+              - '.golangci.yml'
+
       - name: Run golangci-lint
+        if: steps.filter.outputs.go == 'true'
         uses: golangci/golangci-lint-action@4afd733a84b1f43292c63897423277bb7f4313a9 # v8
         with:
           version: ${{ env.GOLANGCI_LINT_VERSION }}
 
       - name: Check code formatting
+        if: steps.filter.outputs.go == 'true'
         run: |
           if [ -n "$(gofmt -l .)" ]; then
             echo "The following files need formatting:"
@@ -53,6 +88,8 @@ jobs:
   test:
     name: Test
     runs-on: ubuntu-latest
+    needs: changes
+    if: needs.changes.outputs.go == 'true'
     steps:
       - name: Checkout code
         uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4
@@ -95,7 +132,8 @@ jobs:
   acceptance-test:
     name: Acceptance Test
     runs-on: ubuntu-latest
-    needs: test
+    needs: [changes, test]
+    if: needs.changes.outputs.go == 'true'
     steps:
       - name: Checkout code
         uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4
@@ -149,7 +187,8 @@ jobs:
   build-provider:
     name: Build Provider
     runs-on: ubuntu-latest
-    needs: test
+    needs: [changes, test]
+    if: needs.changes.outputs.go == 'true'
     steps:
       - name: Checkout code
         uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4
@@ -184,7 +223,8 @@ jobs:
   terraform-compatibility:
     name: Terraform Compatibility Test
     runs-on: ubuntu-latest
-    needs: build-provider
+    needs: [changes, build-provider]
+    if: needs.changes.outputs.go == 'true'
     strategy:
       fail-fast: false
       matrix:
@@ -260,140 +300,13 @@ jobs:
           # Clean up test data
           rm -rf test-data
 
-  pre-release:
-    name: Pre-release
-    runs-on: ubuntu-latest
-    needs: [lint, test, acceptance-test, terraform-compatibility]
-    if: github.ref == 'refs/heads/main' && github.event_name != 'pull_request' && github.actor != 'dependabot[bot]'
-    permissions:
-      contents: write
-      id-token: write
-      attestations: write
-    steps:
-      - name: Checkout code
-        uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4
-        with:
-          fetch-depth: 0
-
-      - name: Check for Go file changes
-        id: go_changes
-        run: |
-          # Get the latest tag
-          LATEST_TAG=$(git describe --tags --abbrev=0 2>/dev/null || echo "")
-
-          if [ -z "$LATEST_TAG" ]; then
-            echo "No previous tags found, assuming Go files changed"
-            echo "changed=true" >> $GITHUB_OUTPUT
-          else
-            # Check if any Go files changed since the last tag
-            GO_CHANGES=$(git diff --name-only ${LATEST_TAG}..HEAD -- '*.go' 'go.mod' 'go.sum' | wc -l)
-
-            if [ "$GO_CHANGES" -gt 0 ]; then
-              echo "Go files changed since ${LATEST_TAG}"
-              echo "changed=true" >> $GITHUB_OUTPUT
-              git diff --name-only ${LATEST_TAG}..HEAD -- '*.go' 'go.mod' 'go.sum'
-            else
-              echo "No Go files changed since ${LATEST_TAG}"
-              echo "changed=false" >> $GITHUB_OUTPUT
-            fi
-          fi
-
-      - name: Set up Go
-        if: steps.go_changes.outputs.changed == 'true'
-        uses: actions/setup-go@d35c59abb061a4a6fb18e82ac0862c26744d6ab5 # v5
-        with:
-          go-version: ${{ env.GO_VERSION }}
-          cache: true
-
-      - name: Import GPG key
-        if: steps.go_changes.outputs.changed == 'true'
-        id: import_gpg
-        uses: crazy-max/ghaction-import-gpg@e89d40939c28e39f97cf32126055eeae86ba74ec # v6
-        with:
-          gpg_private_key: ${{ secrets.GPG_PRIVATE_KEY }}
-          passphrase: ${{ secrets.GPG_PASSPHRASE }}
-
-      - name: Generate snapshot version
-        if: steps.go_changes.outputs.changed == 'true'
-        id: version
-        run: |
-          # Get the latest release tag (exclude pre-release tags)
-          LATEST_TAG=$(git tag -l 'v*' | grep -E '^v[0-9]+\.[0-9]+\.[0-9]+$' | sort -V | tail -n1 || echo "v0.0.0")
-          if [ -z "$LATEST_TAG" ]; then
-            LATEST_TAG="v0.0.0"
-          fi
-          echo "Latest release tag: ${LATEST_TAG}"
-
-          # Remove 'v' prefix
-          VERSION=${LATEST_TAG#v}
-          echo "Version without prefix: ${VERSION}"
-
-          # Parse version components
-          IFS='.' read -r major minor patch <<< "$VERSION"
-          echo "Version components: major=${major}, minor=${minor}, patch=${patch}"
-
-          # Generate new version
-          NEW_VERSION="${major}.${minor}.$((patch + 1))-dev.$(date +%Y%m%d%H%M%S)+$(git rev-parse --short HEAD)"
-          echo "Pre-release version: ${NEW_VERSION}"
-          echo "version=${NEW_VERSION}" >> $GITHUB_OUTPUT
-
-      - name: Run GoReleaser (snapshot)
-        if: steps.go_changes.outputs.changed == 'true'
-        uses: goreleaser/goreleaser-action@9c156ee8a17a598857849441385a2041ef570552 # v6
-        with:
-          version: latest
-          args: release --snapshot --skip=sign --clean --skip=validate
-        env:
-          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
-
-      - name: Generate pre-release attestations
-        if: steps.go_changes.outputs.changed == 'true'
-        uses: actions/attest-build-provenance@96b4a1ef7235a096b17240c259729fdd70c83d45 # v2
-        with:
-          subject-path: |
-            dist/*.zip
-            dist/*_checksums.txt
-
-      - name: Upload artifacts
-        if: steps.go_changes.outputs.changed == 'true'
-        uses: actions/upload-artifact@ea165f8d65b6e75b540449e92b4886f43607fa02 # v4
-        with:
-          name: pre-release-artifacts
-          path: dist/*
-
-      - name: Create GitHub pre-release
-        if: steps.go_changes.outputs.changed == 'true'
-        uses: softprops/action-gh-release@72f2c25fcb47643c292f7107632f7a47c1df5cd8 # v2
-        with:
-          name: "Development Build v${{ steps.version.outputs.version }}"
-          tag_name: "v${{ steps.version.outputs.version }}"
-          prerelease: true
-          draft: false
-          files: |
-            dist/*.zip
-            dist/*_checksums.txt
-          body: |
-            ## Development Build
-
-            This is an automated development build from commit ${{ github.sha }}.
-
-            **⚠️ This is a pre-release version and should not be used in production.**
-
-            ### Commit Information
-            - SHA: ${{ github.sha }}
-            - Author: ${{ github.actor }}
-            - Message: ${{ github.event.head_commit.message }}
-
-            ### Installation
-
-            Download the appropriate archive for your platform and extract the provider binary.
 
 
   # Job to consolidate status for PR checks
   ci-status:
     name: CI Status
     runs-on: ubuntu-latest
-    needs: [lint, test, acceptance-test, terraform-compatibility]
+    needs: [changes, lint, test, acceptance-test, terraform-compatibility]
     if: always()
     steps:
       - name: Checkout code

.github/workflows/pre-commit.yml

@@ -7,8 +7,6 @@ on:
       - edited
       - synchronize
     branches: [ main ]
-  push:
-    branches: [ main ]
 
 permissions:
   contents: read