Strict Concurrency Support (#15)

* Remove SwiftLint, enable strict concurrency

* Upgrade dependencies

* Fix errors from change to HTTPTypes

* Sendable  conformances

* Fix warnings

* Add Danger workflow

* Sort rules

* Disable optional_data_string_conversion

* SwiftLint fixes

* Replace @retroactive with fully qualified names

* generic -> regular

* Update toolchain version

Author: mgacy

.github/workflows/danger.yml

@@ -0,0 +1,26 @@
+name: Run Danger
+on:
+  workflow_dispatch:
+  pull_request:
+    types:
+      - opened
+      - reopened
+      - synchronize
+      - ready_for_review
+
+jobs:
+  build:
+    if: github.event.pull_request.draft == false
+    name: Run Danger
+    runs-on: ubuntu-latest
+    permissions:
+      contents: write
+      issues: write
+      pull-requests: write
+    steps:
+      - name: Git checkout
+        uses: actions/checkout@v4
+      - name: Danger
+        uses: 417-72KI/danger-swiftlint@v5.9
+        env:
+          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}

.swiftlint.yml

@@ -2,31 +2,32 @@ excluded:
   - .build
 
 opt_in_rules:
+  - array_init
+  - attributes
+  - closure_end_indentation
+  - closure_spacing
   - empty_count
   - explicit_init
-  - closure_spacing
-  - overridden_super_call
-  - redundant_nil_coalescing
-  - private_outlet
+  - extension_access_modifier
+  - fatal_error_message
+  - first_where
+  - let_var_whitespace
+  - literal_expression_end_indentation
   - nimble_operator
-  - attributes
   - operator_usage_whitespace
-  - closure_end_indentation
-  - first_where
+  - overridden_super_call
+  - pattern_matching_keywords
+  - private_outlet
   - prohibited_super_call
-  - fatal_error_message
-  - vertical_parameter_alignment_on_call
-  - let_var_whitespace
+  - redundant_nil_coalescing
   - unneeded_parentheses_in_closure_argument
-  - extension_access_modifier
-  - pattern_matching_keywords
-  - array_init
-  - literal_expression_end_indentation
+  - vertical_parameter_alignment_on_call
 
 disabled_rules:
-  - void_return
   - multiple_closures_with_trailing_closure
+  - optional_data_string_conversion
   - vertical_parameter_alignment_on_call
+  - void_return
 
 identifier_name:
   excluded:

Dangerfile.swift

@@ -0,0 +1,15 @@
+import Danger
+
+extension String: Error {}
+
+let danger = Danger()
+
+if danger.github.pullRequest.body == nil {
+    danger.fail("Please add a description to this Pull Request")
+}
+
+SwiftLint
+    .lint(
+        .all(directory: nil),
+        configFile: ".swiftlint.yml"
+    )

Package.resolved

@@ -1,4 +1,4 @@
-// swift-tools-version: 5.7
+// swift-tools-version: 5.9
 // The swift-tools-version declares the minimum version of Swift required to build this package.
 
 import PackageDescription
@@ -14,10 +14,10 @@ let package = Package(
         .library(name: "LambdaMocks", targets: ["LambdaMocks"])
     ],
     dependencies: [
-        .package(url: "https://github.com/apple/swift-log.git", .upToNextMajor(from: "1.4.2")),
-        .package(url: "https://github.com/apple/swift-nio.git", .upToNextMajor(from: "2.43.1")),
-        .package(url: "https://github.com/swift-server/swift-aws-lambda-events.git", .upToNextMajor(from: "0.2.0")),
-        .package(url: "https://github.com/swift-server/swift-aws-lambda-runtime.git", "1.0.0-alpha.1"..<"1.0.0-beta.999")
+        .package(url: "https://github.com/apple/swift-log.git", from: "1.4.2"),
+        .package(url: "https://github.com/apple/swift-nio.git", from: "2.43.1"),
+        .package(url: "https://github.com/swift-server/swift-aws-lambda-events", from: "0.2.0"),
+        .package(url: "https://github.com/swift-server/swift-aws-lambda-runtime", from: "1.0.0-alpha.3")
     ]
 )
 
@@ -33,7 +33,7 @@ let targets: [Target] = [
         name: "LambdaExtras",
         dependencies: [
             "LambdaExtrasCore",
-            .product(name: "AWSLambdaRuntime",package: "swift-aws-lambda-runtime"),
+            .product(name: "AWSLambdaRuntime", package: "swift-aws-lambda-runtime"),
             .product(name: "AWSLambdaEvents", package: "swift-aws-lambda-events")
         ]
     ),
@@ -54,11 +54,8 @@ let targets: [Target] = [
     )
 ]
 
-#if os(macOS)
-package.dependencies.append(.package(url: "https://github.com/realm/SwiftLint.git", from: "0.54.0"))
 for target in targets {
-    target.plugins = [.plugin(name: "SwiftLintPlugin", package: "SwiftLint")]
+    target.swiftSettings = [.enableExperimentalFeature("StrictConcurrency")]
 }
-#endif
 
 package.targets = targets

Package.swift

@@ -4,7 +4,7 @@ Swifty helpers for working with AWS Lambda.
 
 ## 📱 Requirements
 
-Swift 5.7 toolchain with Swift Package Manager.
+Swift 5.9 toolchain with Swift Package Manager.
 
 ## 🖥 Installation
 
@@ -32,8 +32,8 @@ Where `<product>` is one of the following:
 
 This package is intended to support the creation of lambdas composed of 2 parts:
 
-- a generic target with a handler implementing the core logic without AWS dependencies
-- an executable target using that generic one
+- a regular target with a handler implementing the core logic without AWS dependencies
+- an executable target using that regular one
 
 ### Handler
 

README.md

@@ -8,10 +8,11 @@
 import AWSLambdaEvents
 import AWSLambdaRuntime
 import Foundation
+import HTTPTypes
 import LambdaExtrasCore
 
 /// A coder for APIGateway events.
-public struct APIGatewayCoder<E, O>: LambdaCoding where E: Codable, E: Sendable, O: Sendable {
+public struct APIGatewayCoder<E, O>: LambdaCoding, Sendable where E: Codable, E: Sendable, O: Sendable {
     /// A JSON decoder.
     let decoder: JSONDecoder
 
@@ -52,16 +53,16 @@ public struct APIGatewayCoder<E, O>: LambdaCoding where E: Codable, E: Sendable,
     }
 
     public func encode(error: Error) throws -> APIGatewayV2Response {
-        let statusCode: HTTPResponseStatus
+        let status: HTTPResponse.Status
         switch error {
         case HandlerError.emptyBody:
-            statusCode = .badRequest
+            status = .badRequest
         default:
-            statusCode = .internalServerError
+            status = .internalServerError
         }
 
         return APIGatewayV2Response(
-            statusCode: statusCode,
+            statusCode: status,
             body: try errorBodyProvider(error.localizedDescription))
     }
 }

Sources/LambdaExtras/APIGatewayCoder.swift

@@ -79,7 +79,7 @@ public extension Optional {
     }
 }
 
-extension String: LocalizedError {
+extension Swift.String: Foundation.LocalizedError {
     public var errorDescription: String? { self }
     public var failureReason: String? { self }
 }

Sources/LambdaExtrasCore/Extensions.swift

@@ -8,7 +8,7 @@
 import Foundation
 
 /// An error that occurs during lambda handler execution.
-public enum HandlerError: Error, Equatable, LocalizedError {
+public enum HandlerError: Error, Equatable, LocalizedError, Sendable {
     /// The request is missing a body.
     case emptyBody
     /// The lambda context is missing an expected environment variable.

Sources/LambdaExtrasCore/HandlerError.swift

@@ -7,6 +7,7 @@
 
 import AWSLambdaEvents
 import Foundation
+import HTTPTypes
 
 // MARK: - Helpers
 
@@ -53,7 +54,7 @@ public extension APIGatewayV2Request {
     static func mock<T: Encodable>(
         _ bodyValue: T,
         encodedWith encoder: JSONEncoder = .init(),
-        method: HTTPMethod = .POST,
+        method: HTTPRequest.Method = .post,
         rawPath: String = "/endpoint",
         cookies: [String]? = nil,
         headers: [String: String] = [:],
@@ -87,7 +88,7 @@ public extension APIGatewayV2Request {
     /// - Returns: The mock request.
     static func mock(
         // swiftlint:disable:previous function_body_length
-        method: HTTPMethod = .POST,
+        method: HTTPRequest.Method = .post,
         rawPath: String = "/endpoint",
         cookies: [String]? = nil,
         headers: [String: String] = [:],
@@ -206,7 +207,7 @@ public extension APIGatewayV2Response {
     static func mock<T: Encodable>(
         _ bodyValue: T,
         encodedWith encoder: JSONEncoder = .init(),
-        statusCode: HTTPResponseStatus = .ok,
+        statusCode: HTTPResponse.Status = .ok,
         headers: [String: String]? = nil,
         isBase64Encoded: Bool = false,
         cookies: [String]? = nil