Support Swift6 #83
Support Swift6 #83
Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR updates the AnthropicSwiftSDK to support Swift 6 by adding Sendable conformance to types used in concurrent contexts and fixing compilation issues. The changes include adding Sendable conformance to various data structures, protocol constraints, and error types while also correcting spelling errors in test files.
- Adds
Sendableconformance to numerous data structures and enums throughout the SDK - Updates protocol constraints and generic parameters to support Swift 6 concurrency
- Fixes spelling errors in test files from "succeeeded" to "succeeded"
Reviewed Changes
Copilot reviewed 31 out of 32 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| Package.swift | Updates Swift tools version to 6.0 and minimum platform versions |
| Multiple entity files | Adds Sendable conformance to data structures and enums |
| StreamingResponse.swift | Adds Sendable conformance to protocol |
| AnthropicStreamingParser.swift | Adds Sendable constraint to generic parameter |
| InputJSONDeltaAccumulator.swift | Adds @unchecked Sendable conformance and improves error handling |
| StreamingResponse+Extension.swift | Adds @Sendable annotation to Task closure |
| Multiple test files | Corrects spelling from "succeeeded" to "succeeded" |
| BatchRequestCounts.swift | Fixes property name spelling |
| HTTPMock.swift | Adds nonisolated(unsafe) annotation |
| Example.swiftpm files | Updates iOS version requirement and dependencies |
| @@ -19,7 +19,7 @@ final class ObjectListResponseTests: XCTestCase { | |||
| "processing_status": "ended", | |||
| "request_counts": { | |||
| "processing": 0, | |||
| "succeeeded": 95, | |||
| "succeeded": 95, | |||
There was a problem hiding this comment.
Fixed spelling correction from "succeeeded" to "succeeded".
Copilot uses AI. Check for mistakes.
| @@ -17,7 +17,7 @@ final class BatchResponseTests: XCTestCase { | |||
| "processing_status": "ended", | |||
| "request_counts": { | |||
| "processing": 0, | |||
| "succeeeded": 95, | |||
| "succeeded": 95, | |||
There was a problem hiding this comment.
Fixed spelling correction from "succeeeded" to "succeeded".
Copilot uses AI. Check for mistakes.
| @@ -34,7 +34,7 @@ final class MessageBatchesTests: XCTestCase { | |||
| "processing_status": "ended", | |||
| "request_counts": { | |||
| "processing": 0, | |||
| "succeeeded": 95, | |||
| "succeeded": 95, | |||
There was a problem hiding this comment.
Fixed spelling correction from "succeeeded" to "succeeded".
Copilot uses AI. Check for mistakes.
| @@ -12,7 +12,7 @@ public struct BatchRequestCounts: Decodable { | |||
| /// Number of requests in the Message Batch that have completed successfully. | |||
| /// | |||
| /// This is zero until processing of the entire Message Batch has ended. | |||
| let succeeeded: Int | |||
| let succeeded: Int | |||
There was a problem hiding this comment.
Fixed spelling correction from "succeeeded" to "succeeded".
Copilot uses AI. Check for mistakes.
No description provided.