Add a format-check script for non-code formatting validations

This brings our files into compliance with our existing .editorconfig configuration.

Author: jakepetroules

.github/scripts/format-check.sh

@@ -0,0 +1,48 @@
+#!/bin/bash
+##===----------------------------------------------------------------------===##
+##
+## This source file is part of the Swift open source project
+##
+## Copyright (c) 2025 Apple Inc. and the Swift project authors
+## Licensed under Apache License v2.0 with Runtime Library Exception
+##
+## See http://swift.org/LICENSE.txt for license information
+## See http://swift.org/CONTRIBUTORS.txt for the list of Swift project authors
+##
+##===----------------------------------------------------------------------===##
+
+set -euo pipefail
+
+fail=0
+
+while IFS= read -r file; do
+  # Skip empty or binary files
+  if [ ! -s "$file" ] || [[ "$file" == *png ]] || [[ "$file" == *mlmodel ]] || [[ "$file" == *mlpackage* ]] ; then
+    continue
+  fi
+
+  # --- Check for trailing whitespace (spaces or tabs before end of line) ---
+  # Using POSIX-compatible regex; no -P flag needed.
+  if grep -nE '[[:space:]]+$' "$file" >/dev/null; then
+    echo "❌ Trailing whitespace in: $file"
+    # Print offending lines (indent for readability)
+    grep -nE '[[:space:]]+$' "$file" | sed 's/^/    /'
+    fail=1
+  fi
+
+  # --- Check for final newline ---
+  # tail -c handles both GNU and BSD variants
+  lastchar=$(tail -c 1 "$file" | od -An -tx1 | tr -d ' \n')
+  if [[ "$lastchar" != "0a" ]]; then
+    echo "❌ Missing final newline: $file"
+    fail=1
+  fi
+
+done < <(git ls-files)
+
+if [[ $fail -eq 0 ]]; then
+  echo "✅ All tracked files are clean (no trailing whitespace, final newline present)."
+else
+  echo "⚠️  Some files failed checks."
+  exit 1
+fi

.github/workflows/pull_request.yml

@@ -54,3 +54,13 @@ jobs:
       license_header_check_project_name: "Swift"
       api_breakage_check_enabled: false
       format_check_enabled: false
+
+  space-format-check:
+    name: Space format check
+    runs-on: ubuntu-latest
+    timeout-minutes: 10
+    steps:
+      - name: Checkout repository
+        uses: actions/checkout@v4
+      - name: Run space format check
+        run: cat .github/scripts/format-check.sh | bash

.swift-version

@@ -1 +1 @@
-6.2.0
+6.2.0

Plugins/generate-windows-installer-component-groups/generate-windows-installer-component-groups.swift

@@ -28,7 +28,7 @@ struct GenerateWindowsInstallerComponentGroups: CommandPlugin {
                   <Component>
                     <File Source="$(ToolchainRoot)\usr\bin\\#(sourceModule.name).dll" />
                   </Component>
-            
+
             """#
 
             let resources = sourceModule.sourceFiles.filter { resource in resource.type == .resource && ["xcspec", "xcbuildrules"].contains(resource.url.pathExtension) }
@@ -41,7 +41,7 @@ struct GenerateWindowsInstallerComponentGroups: CommandPlugin {
                       <Component>
                         <File Source="$(ToolchainRoot)\usr\share\pm\SwiftBuild_\#(sourceModule.name).resources\\#(resource.url.lastPathComponent)" />
                       </Component>
-                
+
                 """#
                 }
                 resourcesComponents += "    </ComponentGroup>\n"

Sources/SWBAndroidPlatform/AndroidSDK.swift

@@ -108,11 +108,11 @@ public import Foundation
             init(from decoder: any Decoder, configuration: Version) throws {
                 struct DynamicCodingKey: CodingKey {
                     var stringValue: String
-                    
+
                     init?(stringValue: String) {
                         self.stringValue = stringValue
                     }
-                    
+
                     let intValue: Int? = nil
 
                     init?(intValue: Int) {

Sources/SWBApplePlatform/EXUtil.swift

@@ -46,7 +46,7 @@ final class ExtensionPointExtractorSpec: GenericCommandLineToolSpec, SpecIdentif
         let inputs = cbc.inputs.map { input in
             return delegate.createNode(input.absolutePath)
         }.filter { node in
-            node.path.fileExtension == "swiftconstvalues" 
+            node.path.fileExtension == "swiftconstvalues"
         }
         var outputs = [any PlannedNode]()
 

Sources/SWBApplePlatform/ExtensionPointExtractorTaskProducer.swift

@@ -126,7 +126,7 @@ final class AppExtensionInfoPlistGeneratorTaskProducer: PhasedTaskProducer, Task
         let isApplePlatform = context.isApplePlatform
         guard AppExtensionPlistGeneratorSpec.shouldConstructTask(scope: scope, productType: productType, isApplePlatform: isApplePlatform) else {
             return []
-        }        
+        }
 
         let tasks: [any PlannedTask] = []
         let buildFilesProcessingContext = BuildFilesProcessingContext(scope)

Sources/SWBBuildService/Messages.swift

@@ -1622,7 +1622,7 @@ package struct ServiceMessageHandlers: ServiceExtension {
         service.registerMessageHandler(ComputeDependencyClosureMsg.self)
         service.registerMessageHandler(ComputeDependencyGraphMsg.self)
         service.registerMessageHandler(DumpBuildDependencyInfoMsg.self)
-        
+
         service.registerMessageHandler(BuildDescriptionConfiguredTargetsMsg.self)
         service.registerMessageHandler(BuildDescriptionSelectConfiguredTargetsForIndexMsg.self)
         service.registerMessageHandler(BuildDescriptionConfiguredTargetSourcesMsg.self)

Sources/SWBBuildSystem/BuildOperation.swift

@@ -474,7 +474,7 @@ package final class BuildOperation: BuildSystemOperation {
         // Create the low-level build system.
         let adaptor: OperationSystemAdaptor
         let system: BuildSystem
-        
+
         let llbQoS: SWBLLBuild.BuildSystem.QualityOfService?
         switch request.qos {
         case .default: llbQoS = .default

Sources/SWBCore/CustomTaskTypeDescription.swift

@@ -13,48 +13,48 @@
 public import SWBUtil
 
 public final class CustomTaskTypeDescription: TaskTypeDescription {
-    
+
     private init() {
     }
-    
+
     public static let only: any TaskTypeDescription = CustomTaskTypeDescription()
-    
+
     public var payloadType: (any TaskPayload.Type)? {
         nil
     }
-    
+
     public var isUnsafeToInterrupt: Bool {
         false
     }
-    
+
     public var toolBasenameAliases: [String] {
         []
     }
-    
+
     public func commandLineForSignature(for task: any ExecutableTask) -> [ByteString]? {
         return nil
     }
-    
+
     public func serializedDiagnosticsPaths(_ task: any ExecutableTask, _ fs: any FSProxy) -> [Path] {
         []
     }
-    
+
     public func generateIndexingInfo(for task: any ExecutableTask, input: TaskGenerateIndexingInfoInput) -> [TaskGenerateIndexingInfoOutput] {
         []
     }
-    
+
     public func generatePreviewInfo(for task: any ExecutableTask, input: TaskGeneratePreviewInfoInput, fs: any FSProxy) -> [TaskGeneratePreviewInfoOutput] {
         []
     }
-    
+
     public func generateDocumentationInfo(for task: any ExecutableTask, input: TaskGenerateDocumentationInfoInput) -> [TaskGenerateDocumentationInfoOutput] {
         []
     }
-    
+
     public func customOutputParserType(for task: any ExecutableTask) -> (any TaskOutputParser.Type)? {
         ShellScriptOutputParser.self
     }
-    
+
     public func interestingPath(for task: any ExecutableTask) -> Path? {
         nil
     }