PluginBench
Skill
Pass
Audit score 90

swift-concurrency

avdlee/swift-concurrency-agent-skill

Diagnose and fix Swift concurrency issues, refactor to async/await, and guide Swift 6 migration.

What is swift-concurrency?

Helps diagnose Swift concurrency problems including data races, actor isolation, Sendable conformance, and @MainActor issues. Use this when working with async/await, tasks, actors, or concurrency-related compiler warnings, or when migrating callback-based code to modern Swift concurrency patterns.

  • Analyze project settings (Swift language mode, strict concurrency level, default isolation) to determine concurrency behavior
  • Diagnose and fix main actor isolation, actor-isolated protocol conformance, and Sendable type violations
  • Refactor callback-based code to async/await with proper isolation boundaries
  • Guide Swift 6 migration including upcoming features and strict concurrency enablement
  • Optimize Task spawning: use @concurrent entry points when appropriate and MainActor.run for UI updates only when needed
  • Provide smallest safe fixes that preserve behavior while satisfying data-race safety

How to install swift-concurrency

npx skills add https://github.com/avdlee/swift-concurrency-agent-skill --skill swift-concurrency
Claude Code
Cursor
Windsurf
Cline

How to use swift-concurrency

  1. 1.Confirm project settings: check Package.swift or .pbxproj for Swift language mode, strict concurrency level, default isolation, and upcoming features
  2. 2.Capture the exact diagnostic message and identify the offending symbol
  3. 3.Determine the isolation boundary: @MainActor, custom actor, actor instance isolation, or nonisolated
  4. 4.For unstructured tasks, inspect the synchronous prefix before the first await to decide whether to start on @MainActor or use Task { @concurrent in ... }
  5. 5.Apply the smallest safe fix from the Common Diagnostics table, preserving behavior while satisfying data-race safety
  6. 6.For complex issues crossing module boundaries or requiring unsafe escape hatches, escalate to the matching reference file

Use cases

Good for
  • Fix 'Main actor-isolated cannot be used from nonisolated context' errors by isolating callers or using await MainActor.run
  • Convert callback-based async code to async/await with correct isolation boundaries
  • Resolve 'Sending value of non-Sendable type risks causing data races' by keeping access inside one actor or converting to immutable types
  • Migrate projects to Swift 6 strict concurrency by analyzing and updating build settings
  • Optimize unstructured Task spawning to avoid unnecessary main-actor entry points for background work
Who it's for
  • Swift developers working with async/await and concurrency
  • Teams migrating to Swift 6 strict concurrency
  • Developers fixing data-race safety warnings and Sendable conformance issues
  • iOS/macOS engineers managing UI-bound state and background tasks
  • Anyone refactoring legacy callback-based async code

swift-concurrency FAQ

When should I use @MainActor vs. Task { @concurrent in ... }?

Use @MainActor only when the code is truly UI-bound. For background work, use Task { @concurrent in ... } if the synchronous prefix (before the first await) does not need main-actor access. Only use inherited @MainActor if the prefix genuinely requires it; a trivial non-main statement followed by main-actor work is not sufficient reason.

How do I fix 'Sending value of non-Sendable type risks causing data races'?

First identify which isolation boundary is being crossed. Keep access inside one actor, or convert the transferred value to an immutable or value type. Avoid @unchecked Sendable unless you have a documented safety invariant and a plan to remove it.

What is the difference between structured and unstructured concurrency?

Structured concurrency (async/await, async let, withTaskGroup) is automatically cancelled and cleaned up when the scope exits. Unstructured tasks (Task { }, Task.detached) run independently; use Task.detached only with a clear documented reason, as it does not inherit actor context.

How do I migrate callback-based code to async/await?

Replace callback parameters with async/await syntax, ensuring the new function runs on the correct isolation boundary. Determine whether the work is UI-bound (@MainActor) or background (nonisolated or @concurrent), and use await to suspend instead of callbacks.

What should I check before applying a concurrency fix?

Always analyze Package.swift or .pbxproj to determine Swift language mode, strict concurrency level, default isolation, and upcoming features. Do not guess these settings, even for new Xcode 26 projects. If unknown, ask the developer to confirm before giving migration-sensitive guidance.

Full instructions (SKILL.md)

Source of truth, from avdlee/swift-concurrency-agent-skill.


name: swift-concurrency description: 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.

Swift Concurrency

Fast Path

Before proposing a fix:

  1. 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.
  2. Capture the exact diagnostic and offending symbol.
  3. Determine the isolation boundary: @MainActor, custom actor, actor instance isolation, or nonisolated.
  4. 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.

Project settings that change concurrency behavior:

SettingSwiftPM (Package.swift)Xcode (.pbxproj)
Language modeswiftLanguageVersions or -swift-version (// swift-tools-version: is not a reliable proxy)Swift Language Version
Strict concurrency.enableExperimentalFeature("StrictConcurrency=targeted")SWIFT_STRICT_CONCURRENCY
Default isolation.defaultIsolation(MainActor.self)SWIFT_DEFAULT_ACTOR_ISOLATION
Upcoming features.enableUpcomingFeature("NonisolatedNonsendingByDefault")SWIFT_UPCOMING_FEATURE_*
Approachable ConcurrencyN/A (use individual upcoming features)SWIFT_APPROACHABLE_CONCURRENCY

Xcode 26 note: New projects created in Xcode 26 will often start with SWIFT_DEFAULT_ACTOR_ISOLATION = MainActor and SWIFT_APPROACHABLE_CONCURRENCY = YES enabled by default. Treat these as likely defaults for newly created projects, not as confirmed settings.

If any of these are unknown, ask the developer to confirm them before giving migration-sensitive guidance. Do not guess, even for new Xcode 26 projects.

Guardrails:

  • Do not recommend @MainActor as a blanket fix. Justify why the code is truly UI-bound.
  • Prefer structured concurrency over unstructured tasks. Use Task.detached only with a clear reason.
  • If recommending @preconcurrency, @unchecked Sendable, or nonisolated(unsafe), require a documented safety invariant and a follow-up removal plan.
  • Optimize for the smallest safe change. Do not refactor unrelated architecture during migration.
  • Course references are for deeper learning only. Use them sparingly and only when they clearly help answer the developer's question.

Quick Fix Mode

Use Quick Fix Mode when all of these are true:

  • The issue is localized to one file or one type.
  • The isolation boundary is clear.
  • The fix can be explained in 1-2 behavior-preserving steps.

Skip Quick Fix Mode when any of these are true:

  • Build settings or default isolation are unknown.
  • The issue crosses module boundaries or changes public API behavior.
  • The likely fix depends on unsafe escape hatches.

Common Diagnostics

DiagnosticFirst checkSmallest safe fixEscalate to
Main actor-isolated ... cannot be used from a nonisolated contextIs 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
Actor-isolated type does not conform to protocolMust 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
Sending value of non-Sendable type ... risks causing data racesWhat 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
SwiftLint async_without_awaitIs async actually required by protocol, override, or @concurrent?Remove async, or use a narrow suppression with rationale. Never add fake awaits.references/linting.md
wait(...) is unavailable from asynchronous contextsIs this legacy XCTest async waiting?Replace with await fulfillment(of:) or Swift Testing equivalents.references/testing.md
Core Data concurrency warningsAre NSManagedObject instances crossing contexts or actors?Pass NSManagedObjectID or map to a Sendable value type.references/core-data.md
Thread.current unavailable from asynchronous contextsAre you debugging by thread instead of isolation?Reason in terms of isolation and use Instruments/debugger instead.references/threading.md
SwiftLint concurrency-related warningsWhich specific lint rule triggered?Use references/linting.md for rule intent and preferred fixes; avoid dummy awaits.references/linting.md
... cannot satisfy conformance requirement for a 'Sendable' type parameter (SendableMetatype)Does the conformance carry global-actor isolation?Remove actor isolation from the conformance, or avoid passing the metatype across isolation boundaries. See SendableMetatype section in references/actors.md.references/actors.md

When Quick Fixes Fail

  1. Gather project settings if not already confirmed.
  2. Re-evaluate which isolation boundaries the type crosses.
  3. Route to the matching reference file for a deeper fix.
  4. If the fix may change behavior, document the invariant and add verification steps.

Smallest Safe Fixes

Prefer changes that preserve behavior while satisfying data-race safety:

  • UI-bound state: isolate the type or member to @MainActor.
  • Shared mutable state: move it behind an actor, or use @MainActor only if the state is UI-owned.
  • 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.
  • Sendability issues: prefer immutable values and explicit boundaries over @unchecked Sendable.

Concurrency Tool Selection

NeedToolKey Guidance
Single async operationasync/awaitDefault choice for sequential async work
Fixed parallel operationsasync letKnown count at compile time; auto-cancelled on throw
Dynamic parallel operationswithTaskGroupUnknown count; structured — cancels children on scope exit
Sync → async bridgeTask { }Inherits actor context; use Task.detached only with documented reason
Shared mutable stateactorPrefer over locks/queues; keep isolated sections small
UI-bound state@MainActorOnly for truly UI-related code; justify isolation

Common Scenarios

Network request with UI update

Task { @concurrent in
    let data = try await fetchData()
    await MainActor.run { self.updateUI(with: data) }
}

Processing array items in parallel

await withTaskGroup(of: ProcessedItem.self) { group in
    for item in items {
        group.addTask { await process(item) }
    }
    for await result in group {
        results.append(result)
    }
}

Task entry isolation

Match a Task's entry isolation to its synchronous prefix (everything from { to the first await).

  • If anything in that prefix needs @MainActor, keep the inherited @MainActor start.
  • If nothing in that prefix needs @MainActor, prefer Task { @concurrent in ... } and hop back only for UI-owned mutation.
// ❌ Synchronous prefix is empty; first work hops away
Task {
    await hopToOtherIsolationDomain()
}

// ❌ Synchronous prefix is only `print` (trivial, non-main); first await hops away
Task {
    print("Also not main-thread-bound")
    await hopToOtherIsolationDomain()
}

// ✅ Start off the main actor, hop back only for UI work
Task { @concurrent in
    await hopToOtherIsolationDomain()
    await MainActor.run { updateUI() }
}

// ✅ Synchronous prefix DOES contain main-actor work — keep inheritance
Task {
    print("debug")              // trivial, non-main — rides along
    self.isLoading = true       // needs @MainActor, before any await
    await fetchData()
}

Swift 6 Migration Quick Guide

Key changes in Swift 6:

  • Strict concurrency checking enabled by default
  • Complete data-race safety at compile time
  • Sendable requirements enforced on boundaries
  • Isolation checking for all async boundaries

Migration Validation Loop

Apply this cycle for each migration change:

  1. Build — Run swift build or Xcode build to surface new diagnostics
  2. Fix — Address one category of error at a time (e.g., all Sendable issues first)
  3. Rebuild — Confirm the fix compiles cleanly before moving on
  4. Test — Run the test suite to catch regressions (swift test or Cmd+U)
  5. Only proceed to the next file/module when all diagnostics are resolved

If a fix introduces new warnings, resolve them before continuing. Never batch multiple unrelated fixes — keep commits small and reviewable.

For detailed migration steps, see references/migration.md.

Reference Router

Open the smallest reference that matches the question:

  • Foundations
    • references/async-await-basics.md — async/await syntax, execution order, async let, URLSession patterns
    • references/tasks.md — Task lifecycle, cancellation, priorities, task groups, structured vs unstructured
    • references/actors.md — Actor isolation, @MainActor, global actors, reentrancy, custom executors, Mutex
    • references/sendable.md — Sendable conformance, value/reference types, @unchecked, region isolation
    • references/threading.md — Execution model, suspension points, Swift 6.2 isolation behavior
  • Streams
    • references/async-sequences.md — AsyncSequence, AsyncStream, when to use vs regular async methods
    • references/async-algorithms.md — Debounce, throttle, merge, combineLatest, channels, timers
  • Applied topics
    • references/testing.md — Swift Testing first, XCTest fallback, leak checks
    • references/performance.md — Profiling with Instruments, reducing suspension points, execution strategies
    • references/memory-management.md — Retain cycles in tasks, memory safety patterns
    • references/core-data.md — NSManagedObject sendability, custom executors, isolation conflicts
  • Migration and tooling
    • references/migration.md — Swift 6 migration strategy, closure-to-async conversion, @preconcurrency, FRP migration
    • references/linting.md — Concurrency-focused lint rules and SwiftLint async_without_await
  • Glossary
    • references/glossary.md — Quick definitions of core concurrency terms

Verification Checklist

When changing concurrency code:

  1. Re-check build settings before interpreting diagnostics.
  2. Build and clear one category of errors before moving on. Do not batch unrelated fixes into the same change.
  3. Run tests, especially actor-, lifetime-, and cancellation-sensitive tests.
  4. Use Instruments for performance claims instead of guessing.
  5. Verify deallocation and cancellation behavior for long-lived tasks.
  6. Check Task.isCancelled in long-running operations.
  7. Never use semaphores or ad hoc locking in async contexts when actor isolation or Mutex would express ownership more safely.

Note: This skill is based on the comprehensive Swift Concurrency Course by Antoine van der Lee.