chore(ci): sync improvements from HoldFast (#19)

* chore(ci): sync improvements from HoldFast

- cliff.toml: add breaking changes rule; skip feat/fix with non-user-visible scopes
- nexus-upload: allow hyphens in semver pre-release; smarter ref on workflow_dispatch; better input description
- ci: replace native paths trigger filter with dorny/paths-filter for per-job skipping

* fix(ci): address review issues

- nexus-upload: revert ref to v$version (dispatch implies tag exists)
- ci: skip dorny/paths-filter on workflow_dispatch (jobs already short-circuit via event_name condition)

* fix(ci): remove unnecessary checkout from changes job

Author: codepuncher

.github/workflows/ci.yml

@@ -3,16 +3,6 @@ name: CI
 on:
   pull_request:
     branches: [main]
-    paths:
-      - "src/**"
-      - "test/**"
-      - ".clang-format"
-      - ".clang-tidy"
-      - "CMakeLists.txt"
-      - "CMakePresets.json"
-      - "cmake/**"
-      - "vcpkg.json"
-      - ".github/workflows/ci.yml"
   workflow_dispatch:
 
 concurrency:
@@ -22,10 +12,56 @@ concurrency:
 permissions: {}
 
 jobs:
+  changes:
+    runs-on: ubuntu-latest
+
+    permissions:
+      pull-requests: read
+
+    outputs:
+      src:    ${{ steps.filter.outputs.src }}
+      tests:  ${{ steps.filter.outputs.tests }}
+      format: ${{ steps.filter.outputs.format }}
+      tidy:   ${{ steps.filter.outputs.tidy }}
+      cmake:  ${{ steps.filter.outputs.cmake }}
+      ci:     ${{ steps.filter.outputs.ci }}
+
+    steps:
+      - uses: dorny/paths-filter@v4
+        id: filter
+        if: github.event_name == 'pull_request'
+        with:
+          filters: |
+            src:
+              - 'src/**'
+            tests:
+              - 'test/**'
+            format:
+              - '.clang-format'
+            tidy:
+              - '.clang-tidy'
+            cmake:
+              - 'CMakeLists.txt'
+              - 'CMakePresets.json'
+              - 'cmake/**'
+              - 'vcpkg.json'
+            ci:
+              - '.github/workflows/ci.yml'
+
   clang-format:
+    needs: [changes]
     runs-on: ubuntu-latest
+
+    if: >-
+      github.event_name == 'workflow_dispatch' ||
+      needs.changes.outputs.src == 'true' ||
+      needs.changes.outputs.tests == 'true' ||
+      needs.changes.outputs.format == 'true' ||
+      needs.changes.outputs.ci == 'true'
+
     permissions:
       contents: read
+
     steps:
       - uses: actions/checkout@v6
 
@@ -40,8 +76,20 @@ jobs:
           check-path: test
 
   test:
-    needs: clang-format
+    needs: [changes, clang-format]
     runs-on: windows-latest
+
+    if: >-
+      always() &&
+      (needs.clang-format.result == 'success' || needs.clang-format.result == 'skipped') &&
+      (
+        github.event_name == 'workflow_dispatch' ||
+        needs.changes.outputs.src == 'true' ||
+        needs.changes.outputs.tests == 'true' ||
+        needs.changes.outputs.cmake == 'true' ||
+        needs.changes.outputs.ci == 'true'
+      )
+
     permissions:
       contents: read
       actions: write
@@ -81,8 +129,19 @@ jobs:
         run: ctest --preset test-windows
 
   build:
-    needs: clang-format
+    needs: [changes, clang-format]
     runs-on: windows-latest
+
+    if: >-
+      always() &&
+      (needs.clang-format.result == 'success' || needs.clang-format.result == 'skipped') &&
+      (
+        github.event_name == 'workflow_dispatch' ||
+        needs.changes.outputs.src == 'true' ||
+        needs.changes.outputs.cmake == 'true' ||
+        needs.changes.outputs.ci == 'true'
+      )
+
     permissions:
       contents: read
       actions: write
@@ -127,11 +186,26 @@ jobs:
             build/msvc/Release/*.pdb
 
   clang-tidy:
-    needs: [build, test]
+    needs: [changes, clang-format, build, test]
     runs-on: windows-latest
+
+    if: >-
+      always() &&
+      (needs.clang-format.result == 'success' || needs.clang-format.result == 'skipped') &&
+      (needs.build.result == 'success' || needs.build.result == 'skipped') &&
+      (needs.test.result == 'success' || needs.test.result == 'skipped') &&
+      (
+        github.event_name == 'workflow_dispatch' ||
+        needs.changes.outputs.src == 'true' ||
+        needs.changes.outputs.tidy == 'true' ||
+        needs.changes.outputs.cmake == 'true' ||
+        needs.changes.outputs.ci == 'true'
+      )
+
     permissions:
       contents: read
       actions: write
+
     env:
       FORCE_JAVASCRIPT_ACTIONS_TO_NODE24: "true"
 

.github/workflows/nexus-upload.yml

@@ -6,7 +6,7 @@ on:
   workflow_dispatch:
     inputs:
       version:
-        description: "Version without 'v' prefix (e.g. 1.0.0)"
+        description: "Version without 'v' prefix (e.g. 1.0.0, 1.0.0-rc.1, 1.0.0+build.1)"
         type: string
         required: true
 
@@ -29,8 +29,8 @@ jobs:
         run: |
           VERSION="${RELEASE_TAG#v}"
           VERSION="${VERSION:-$INPUT_VERSION}"
-          if [[ ! "$VERSION" =~ ^[0-9]+\.[0-9]+\.[0-9]+(-[a-zA-Z0-9.]+)?(\+[a-zA-Z0-9.]+)?$ ]]; then
-            echo "Error: version must be in format X.Y.Z without 'v' prefix (e.g. 1.0.0)" >&2
+          if [[ ! "$VERSION" =~ ^[0-9]+\.[0-9]+\.[0-9]+(-[a-zA-Z0-9.-]+)?(\+[a-zA-Z0-9.-]+)?$ ]]; then
+            echo "Error: version must be in format X.Y.Z (optionally with prerelease/build metadata, e.g. 1.0.0-rc.1+build.1) without 'v' prefix" >&2
             exit 1
           fi
           echo "value=$VERSION" >> "$GITHUB_OUTPUT"

cliff.toml

@@ -17,6 +17,9 @@ commit_preprocessors = [
     { pattern = ' \(#\d+\)$', replace = "" },
 ]
 commit_parsers = [
+    { message = "^[a-z]+(\\([^)]+\\))?!:", group = "Breaking Changes" },
+    { message = "^feat\\((ci|build|docs|test|style)\\)", skip = true },
+    { message = "^fix\\((ci|build|docs|test|style)\\)", skip = true },
     { message = "^feat", group = "New Features" },
     { message = "^fix", group = "Bug Fixes" },
     { message = "^chore", skip = true },