Address PR review feedback and update swift-build to v1.5.3

- Remove unnecessary checkout steps from cleanup workflows
- Remove template echo from codeql.yml build step
- Add comments to lint/docs jobs explaining !failure() intent
- Clarify empty matrix.type in build-macos
- Improve Settings.swift comments for Strategy 1b and os(Android)
- Update all swift-build references from branch ref to v1.5.3

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: leogdion

.github/workflows/SyntaxKit.yml

@@ -74,7 +74,7 @@ jobs:
             type: wasm-embedded
     steps:
       - uses: actions/checkout@v6
-      - uses: brightdigit/swift-build@90-windows-cache-symlinks
+      - uses: brightdigit/swift-build@v1.5.3
         id: build
         with:
           type: ${{ matrix.type }}
@@ -120,7 +120,7 @@ jobs:
       - name: Enable git symlinks
         shell: pwsh
         run: git config --global core.symlinks true
-      - uses: brightdigit/swift-build@90-windows-cache-symlinks
+      - uses: brightdigit/swift-build@v1.5.3
         id: build
         with:
           windows-swift-version: ${{ matrix.swift.version }}
@@ -147,7 +147,7 @@ jobs:
       fail-fast: false
       matrix:
         include:
-          # SPM Build
+          # SPM Build — no platform type; matrix.type evaluates to '' by design
           - runs-on: macos-26
             xcode: "/Applications/Xcode_26.4.app"
 
@@ -156,7 +156,7 @@ jobs:
 
       - name: Build and Test
         id: build
-        uses: brightdigit/swift-build@90-windows-cache-symlinks
+        uses: brightdigit/swift-build@v1.5.3
         with:
           scheme: ${{ env.PACKAGE_NAME }}-Package
           type: ${{ matrix.type }}
@@ -239,7 +239,7 @@ jobs:
 
       - name: Build and Test
         id: build
-        uses: brightdigit/swift-build@90-windows-cache-symlinks
+        uses: brightdigit/swift-build@v1.5.3
         with:
           scheme: ${{ env.PACKAGE_NAME }}-Package
           type: ${{ matrix.type }}
@@ -284,7 +284,7 @@ jobs:
           large-packages: true
           docker-images: true
           swap-storage: true
-      - uses: brightdigit/swift-build@90-windows-cache-symlinks
+      - uses: brightdigit/swift-build@v1.5.3
         id: build
         with:
           type: android
@@ -303,6 +303,8 @@ jobs:
 
   lint:
     name: Linting
+    # build-windows, build-macos-full, and build-android are skipped on feature branches;
+    # !failure() allows this job to proceed past skipped (not failed) dependencies.
     if: ${{ !cancelled() && !failure() && (github.event_name == 'pull_request' || !contains(github.event.head_commit.message, 'ci skip')) }}
     runs-on: ubuntu-latest
     needs: [build-ubuntu, build-macos, build-windows, build-macos-full, build-android]
@@ -334,6 +336,7 @@ jobs:
 
   docs:
     name: Documentation Validation
+    # Same rationale as lint: !failure() lets this proceed when platform-specific jobs are skipped.
     if: ${{ !cancelled() && !failure() && (github.event_name == 'pull_request' || !contains(github.event.head_commit.message, 'ci skip')) }}
     needs: [build-ubuntu, build-macos, build-windows, build-macos-full, build-android]
     runs-on: ubuntu-latest

.github/workflows/cleanup-caches.yml

@@ -9,7 +9,6 @@ jobs:
     permissions:
       actions: write
     steps:
-      - uses: actions/checkout@v6
       - name: Delete branch caches
         uses: actions/github-script@v8
         with:

.github/workflows/cleanup-pr-caches.yml

@@ -10,7 +10,6 @@ jobs:
     permissions:
       actions: write
     steps:
-      - uses: actions/checkout@v6
       - name: Delete PR branch caches
         uses: actions/github-script@v8
         with:

.github/workflows/codeql.yml

@@ -82,9 +82,8 @@ jobs:
 
     # Autobuild attempts to build any compiled languages (C/C++, C#, Go, Java, or Swift).
     # If this step fails, then you should remove it and run the build manually (see below)
-    - run: |
-        echo "Run, Build Application using script"
-        swift build
+    - name: Build
+      run: swift build
 
     - name: Perform CodeQL Analysis
       uses: github/codeql-action/analyze@v4

Tests/SyntaxDocTests/Settings.swift

@@ -17,7 +17,10 @@ internal enum Settings {
       return workingDir
     }
 
-    // Strategy 1b: Documentation.docc copied as last component (Android flat copy via android-copy-files)
+    // Strategy 1b: Documentation.docc present in working directory.
+    // The swift-build action's android-copy-files parameter copies Documentation.docc/ as
+    // a flat sibling of the test binary. Strategy 1a always runs first, so a macOS project
+    // root containing Sources/ will never reach this check.
     if FileManager.default.fileExists(
       atPath: workingDir.appendingPathComponent("Documentation.docc").path)
     {
@@ -81,7 +84,7 @@ internal enum Settings {
         return .init(fileURLWithPath: filePath)
       }
     } else {
-      #if os(Android)
+      #if os(Android)  // os(Android) is a valid Swift platform condition since Swift 5.9
       let resolvedPath = filePath
       #else
       let resolvedPath = "Sources/SyntaxKit/" + filePath