Loading source
Pulling the file list, source metadata, and syntax-aware rendering for this listing.
Source from repo
Diagnose and fix Swift Concurrency issues: async/await, actor isolation, Sendable, and Swift 6 migration.
Files
Skill
Size
Entrypoint
Format
Open file
Syntax-highlighted preview of this file as included in the skill package.
SKILL.md
1---2name: swift-concurrency3description: Diagnose Swift Concurrency issues, refactor callback-based code to async/await, and guide Swift 6 migration when working with tasks, actors, @MainActor, Sendable, data races, thread safety, or concurrency-related compiler and linter warnings.4---5# Swift Concurrency67## Fast Path89Before proposing a fix:10111. Analyze `Package.swift` or `.pbxproj` to determine Swift language mode, strict concurrency level, default isolation, and upcoming features. Do this always, not only for migration work.122. Capture the exact diagnostic and offending symbol.133. Determine the isolation boundary: `@MainActor`, custom actor, actor instance isolation, or `nonisolated`.144. Confirm whether the code is UI-bound or intended to run off the main actor. When spawning unstructured tasks, inspect the synchronous prefix (everything before the first `await`): start on `@MainActor` only when that prefix truly needs main-actor access; otherwise use `Task { @concurrent in ... }` and hop back with `MainActor.run` only after the suspension. A trivial non-main line (for example, `print`) followed by main-actor work in the same prefix is not a reason to use `@concurrent`. For delayed retries, timers, and backoff tasks, separate the waiting from the UI mutation. The sleep often belongs off the main actor even when the final state update belongs on it.1516Project settings that change concurrency behavior:1718| Setting | SwiftPM (`Package.swift`) | Xcode (`.pbxproj`) |19|---|---|---|20| Language mode | `swiftLanguageVersions` or `-swift-version` (`// swift-tools-version:` is not a reliable proxy) | Swift Language Version |21| Strict concurrency | `.enableExperimentalFeature("StrictConcurrency=targeted")` | `SWIFT_STRICT_CONCURRENCY` |22| Default isolation | `.defaultIsolation(MainActor.self)` | `SWIFT_DEFAULT_ACTOR_ISOLATION` |23| Upcoming features | `.enableUpcomingFeature("NonisolatedNonsendingByDefault")` | `SWIFT_UPCOMING_FEATURE_*` |2425If any of these are unknown, ask the developer to confirm them before giving migration-sensitive guidance. Do not guess.2627Guardrails:2829- Do not recommend `@MainActor` as a blanket fix. Justify why the code is truly UI-bound.30- Prefer structured concurrency over unstructured tasks. Use `Task.detached` only with a clear reason.31- If recommending `@preconcurrency`, `@unchecked Sendable`, or `nonisolated(unsafe)`, require a documented safety invariant and a follow-up removal plan.32- Optimize for the smallest safe change. Do not refactor unrelated architecture during migration.33- Course references are for deeper learning only. Use them sparingly and only when they clearly help answer the developer's question.3435## Quick Fix Mode3637Use Quick Fix Mode when all of these are true:3839- The issue is localized to one file or one type.40- The isolation boundary is clear.41- The fix can be explained in 1-2 behavior-preserving steps.4243Skip Quick Fix Mode when any of these are true:4445- Build settings or default isolation are unknown.46- The issue crosses module boundaries or changes public API behavior.47- The likely fix depends on unsafe escape hatches.4849## Common Diagnostics5051| Diagnostic | First check | Smallest safe fix | Escalate to |52|---|---|---|---|53| `Main actor-isolated ... cannot be used from a nonisolated context` | Is this truly UI-bound? | Isolate the caller to `@MainActor` or use `await MainActor.run { ... }` only when main-actor ownership is correct. | `references/actors.md`, `references/threading.md` |54| `Actor-isolated type does not conform to protocol` | Must the requirement run on the actor? | Prefer isolated conformance (e.g., `extension Foo: @MainActor SomeProtocol`); use `nonisolated` only for truly nonisolated requirements. | `references/actors.md` |55| `Sending value of non-Sendable type ... risks causing data races` | What isolation boundary is being crossed? | Keep access inside one actor, or convert the transferred value to an immutable/value type. | `references/sendable.md`, `references/threading.md` |56| `SwiftLint async_without_await` | Is `async` actually required by protocol, override, or `@concurrent`? | Remove `async`, or use a narrow suppression with rationale. Never add fake awaits. | `references/linting.md` |57| `wait(...) is unavailable from asynchronous contexts` | Is this legacy XCTest async waiting? | Replace with `await fulfillment(of:)` or Swift Testing equivalents. | `references/testing.md` |58| Core Data concurrency warnings | Are `NSManagedObject` instances crossing contexts or actors? | Pass `NSManagedObjectID` or map to a Sendable value type. | `references/core-data.md` |59| `Thread.current` unavailable from asynchronous contexts | Are you debugging by thread instead of isolation? | Reason in terms of isolation and use Instruments/debugger instead. | `references/threading.md` |60| SwiftLint concurrency-related warnings | Which specific lint rule triggered? | Use `references/linting.md` for rule intent and preferred fixes; avoid dummy awaits. | `references/linting.md` |6162## When Quick Fixes Fail63641. Gather project settings if not already confirmed.652. Re-evaluate which isolation boundaries the type crosses.663. Route to the matching reference file for a deeper fix.674. If the fix may change behavior, document the invariant and add verification steps.6869## Smallest Safe Fixes7071Prefer changes that preserve behavior while satisfying data-race safety:7273- **UI-bound state**: isolate the type or member to `@MainActor`.74- **Shared mutable state**: move it behind an `actor`, or use `@MainActor` only if the state is UI-owned.75- **Background work**: when work must hop off caller isolation, use an `async` API marked `@concurrent`; when work can safely inherit caller isolation, use `nonisolated` without `@concurrent`. When spawning a `Task`, match entry isolation to its synchronous prefix. If nothing before the first `await` needs the main actor, use `Task { @concurrent in ... }` and hop back via `await MainActor.run { ... }` for the UI update. If the prefix mixes a trivial non-main statement with main-actor work, keep the inherited `@MainActor` start—splitting the cheap line off-main is not worth an extra hop.76- **Sendability issues**: prefer immutable values and explicit boundaries over `@unchecked Sendable`.7778## Concurrency Tool Selection7980| Need | Tool | Key Guidance |81|---|---|---|82| Single async operation | `async/await` | Default choice for sequential async work |83| Fixed parallel operations | `async let` | Known count at compile time; auto-cancelled on throw |84| Dynamic parallel operations | `withTaskGroup` | Unknown count; structured — cancels children on scope exit |85| Sync → async bridge | `Task { }` | Inherits actor context; use `Task.detached` only with documented reason |86| Shared mutable state | `actor` | Prefer over locks/queues; keep isolated sections small |87| UI-bound state | `@MainActor` | Only for truly UI-related code; justify isolation |8889### Common Scenarios9091**Network request with UI update**92```swift93Task { @concurrent in94let data = try await fetchData()95await MainActor.run { self.updateUI(with: data) }96}97```9899**Processing array items in parallel**100```swift101await withTaskGroup(of: ProcessedItem.self) { group in102for item in items {103group.addTask { await process(item) }104}105for await result in group {106results.append(result)107}108}109```110111112## Task entry isolation113114Match a `Task`'s entry isolation to its synchronous prefix (everything from `{` to the first `await`).115116- If anything in that prefix needs `@MainActor`, keep the inherited `@MainActor` start.117- If nothing in that prefix needs `@MainActor`, prefer `Task { @concurrent in ... }` and hop back only for UI-owned mutation.118119```swift120// ❌ Synchronous prefix is empty; first work hops away121Task {122await hopToOtherIsolationDomain()123}124125// ❌ Synchronous prefix is only `print` (trivial, non-main); first await hops away126Task {127print("Also not main-thread-bound")128await hopToOtherIsolationDomain()129}130131// ✅ Start off the main actor, hop back only for UI work132Task { @concurrent in133await hopToOtherIsolationDomain()134await MainActor.run { updateUI() }135}136137// ✅ Synchronous prefix DOES contain main-actor work — keep inheritance138Task {139print("debug") // trivial, non-main — rides along140self.isLoading = true // needs @MainActor, before any await141await fetchData()142}143```144145## Swift 6 Migration Quick Guide146147Key changes in Swift 6:148- **Strict concurrency checking** enabled by default149- **Complete data-race safety** at compile time150- **Sendable requirements** enforced on boundaries151- **Isolation checking** for all async boundaries152153### Migration Validation Loop154155Apply this cycle for each migration change:1561571. **Build** — Run `swift build` or Xcode build to surface new diagnostics1582. **Fix** — Address one category of error at a time (e.g., all Sendable issues first)1593. **Rebuild** — Confirm the fix compiles cleanly before moving on1604. **Test** — Run the test suite to catch regressions (`swift test` or Cmd+U)1615. **Only proceed** to the next file/module when all diagnostics are resolved162163If a fix introduces new warnings, resolve them before continuing. Never batch multiple unrelated fixes — keep commits small and reviewable.164165For detailed migration steps, see `references/migration.md`.166167## Reference Router168169Open the smallest reference that matches the question:170171- Foundations172- `references/async-await-basics.md` — async/await syntax, execution order, async let, URLSession patterns173- `references/tasks.md` — Task lifecycle, cancellation, priorities, task groups, structured vs unstructured174- `references/actors.md` — Actor isolation, @MainActor, global actors, reentrancy, custom executors, Mutex175- `references/sendable.md` — Sendable conformance, value/reference types, @unchecked, region isolation176- `references/threading.md` — Execution model, suspension points, Swift 6.2 isolation behavior177- Streams178- `references/async-sequences.md` — AsyncSequence, AsyncStream, when to use vs regular async methods179- `references/async-algorithms.md` — Debounce, throttle, merge, combineLatest, channels, timers180- Applied topics181- `references/testing.md` — Swift Testing first, XCTest fallback, leak checks182- `references/performance.md` — Profiling with Instruments, reducing suspension points, execution strategies183- `references/memory-management.md` — Retain cycles in tasks, memory safety patterns184- `references/core-data.md` — NSManagedObject sendability, custom executors, isolation conflicts185- Migration and tooling186- `references/migration.md` — Swift 6 migration strategy, closure-to-async conversion, @preconcurrency, FRP migration187- `references/linting.md` — Concurrency-focused lint rules and SwiftLint `async_without_await`188- Glossary189- `references/glossary.md` — Quick definitions of core concurrency terms190191## Verification Checklist192193When changing concurrency code:1941951. Re-check build settings before interpreting diagnostics.1962. Build and clear one category of errors before moving on. Do not batch unrelated fixes into the same change.1973. Run tests, especially actor-, lifetime-, and cancellation-sensitive tests.1984. Use Instruments for performance claims instead of guessing.1995. Verify deallocation and cancellation behavior for long-lived tasks.2006. Check `Task.isCancelled` in long-running operations.2017. Never use semaphores or ad hoc locking in async contexts when actor isolation or `Mutex` would express ownership more safely.202203---204205**Note**: This skill is based on the comprehensive [Swift Concurrency Course](https://www.swiftconcurrencycourse.com?utm_source=github&utm_medium=agent-skill&utm_campaign=skill-footer) by Antoine van der Lee.206