fix(kit): avoid @observable macro bug with package-access stored vars

Swift 5.10's @observable macro emits a duplicate access modifier for
'package' tracked stored properties. Those properties (and the enums in
their signatures) go public instead — HalenKit is never importable by
plugin targets, so nothing leaks into the plugin surface. Also isolate
NotchPanelManager's nested PanelController to the main actor.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01LTBT2mWSqwRv3mRnMhJ2qS

Author: claude

Sources/HalenKit/Accessibility/SystemPermissions.swift

@@ -14,7 +14,7 @@ import HalenPluginAPI
 @MainActor
 @Observable
 package final class SystemPermissionsModel {
-    package private(set) var grants: [SystemPermission: PermissionGrant] = [:]
+    public private(set) var grants: [SystemPermission: PermissionGrant] = [:]
 
     package init() {
         // Populate synchronously where we can so the Settings view doesn't

Sources/HalenKit/Hotkeys/HotkeyRegistrar.swift

@@ -18,14 +18,14 @@ package enum HotkeyID: UInt32 {
 /// were when the conflict was detected. The label is the *attempted* owner
 /// — the one we rejected; `existingOwner` is the registration that keeps
 /// the chord. Surfaced in Settings so the user can disable or rebind.
-package struct HotkeyConflict: Equatable, Identifiable, Sendable {
-    package let id = UUID()
-    package let keyCode: UInt32
-    package let modifiers: UInt32
-    package let existingOwner: String
-    package let attemptedOwner: String
+public struct HotkeyConflict: Equatable, Identifiable, Sendable {
+    public let id = UUID()
+    public let keyCode: UInt32
+    public let modifiers: UInt32
+    public let existingOwner: String
+    public let attemptedOwner: String
 
-    package static func == (lhs: HotkeyConflict, rhs: HotkeyConflict) -> Bool {
+    public static func == (lhs: HotkeyConflict, rhs: HotkeyConflict) -> Bool {
         lhs.keyCode == rhs.keyCode
             && lhs.modifiers == rhs.modifiers
             && lhs.existingOwner == rhs.existingOwner
@@ -36,7 +36,7 @@ package struct HotkeyConflict: Equatable, Identifiable, Sendable {
     /// bitmask values are stable across macOS versions, so this is a pure
     /// lookup. Unknown key codes fall back to their numeric form so the
     /// row stays informative even for obscure keys.
-    package var displayChord: String {
+    public var displayChord: String {
         var s = ""
         if modifiers & UInt32(controlKey) != 0 { s += "⌃" }
         if modifiers & UInt32(optionKey)  != 0 { s += "⌥" }
@@ -106,7 +106,7 @@ package final class HotkeyConflictRegistry {
     /// Conflicts collected since launch. Two plugins attempting the same
     /// chord at startup append once; the UI uses identity (`UUID`) to
     /// stably render rows even if the array is mutated.
-    package private(set) var conflicts: [HotkeyConflict] = []
+    public private(set) var conflicts: [HotkeyConflict] = []
 
     private init() {}
 

Sources/HalenKit/Inference/InferenceBackend.swift

@@ -41,12 +41,12 @@ extension InferenceBackend {
     }
 }
 
-package enum BackendKind: String, Sendable, Codable, CaseIterable {
+public enum BackendKind: String, Sendable, Codable, CaseIterable {
     case bundledLlama          = "bundled-llama"
     case appleFoundationModels = "apple-fm"
     case ollama                = "ollama"
 
-    package var displayName: String {
+    public var displayName: String {
         switch self {
         case .bundledLlama:          return "Built-in (Gemma 4 E4B)"
         case .appleFoundationModels: return "Apple Intelligence"
@@ -72,12 +72,12 @@ package struct BackendCapability: Sendable {
     }
 }
 
-package enum BackendAvailability: Sendable, Equatable {
+public enum BackendAvailability: Sendable, Equatable {
     case available
     /// Human-readable reason, surfaced in Settings (e.g. "Ollama not reachable").
     case unavailable(reason: String)
 
-    package var isAvailable: Bool {
+    public var isAvailable: Bool {
         if case .available = self { return true }
         return false
     }

Sources/HalenKit/Inference/InferenceSettings.swift

@@ -10,13 +10,13 @@ package final class InferenceSettings {
     package static let preferenceOrderKey = "halen.inference.backendOrder"
 
     /// Backend priority, highest first. Persisted as raw-value strings.
-    package var preferenceOrder: [BackendKind] {
+    public var preferenceOrder: [BackendKind] {
         didSet { persist() }
     }
 
     /// Last probed availability per backend — drives the Settings status dots.
     /// Live state, not persisted.
-    package var availability: [BackendKind: BackendAvailability] = [:]
+    public var availability: [BackendKind: BackendAvailability] = [:]
 
     package init() {
         if let raw = UserDefaults.standard.array(forKey: Self.preferenceOrderKey) as? [String] {

Sources/HalenKit/Inference/LlamaCpp/ModelDownloader.swift

@@ -39,7 +39,7 @@ package final class ModelDownloader {
     /// on even a slow residential link — beyond that we'd rather fail loudly.
     private static let resourceTimeout: TimeInterval = 60 * 60
 
-    package enum State: Equatable {
+    public enum State: Equatable {
         case notDownloaded
         case downloading(fraction: Double, bytes: Int64, total: Int64)
         case verifying
@@ -48,7 +48,7 @@ package final class ModelDownloader {
         case failed(message: String)
     }
 
-    package private(set) var state: State
+    public private(set) var state: State
 
     private var downloadTask: Task<Void, Never>?
 

Sources/HalenKit/Notch/NotchPanelManager.swift

@@ -13,6 +13,7 @@ import HalenPluginAPI
 /// throws if another plugin already holds it.
 @MainActor
 package final class NotchPanelManager {
+    @MainActor
     private final class PanelController {
         let panel: NSPanel
 

Sources/HalenKit/PluginHosting/PermissionBroker.swift

@@ -27,7 +27,7 @@ package final class PermissionBroker {
     package let system = SystemPermissionsModel()
 
     /// Bumped on every grant change so SwiftUI dependents refresh.
-    package private(set) var revision = 0
+    public private(set) var revision = 0
 
     private let defaults: UserDefaults
 

Sources/HalenKit/PluginHosting/PluginRegistry.swift

@@ -8,7 +8,7 @@ import HalenPluginAPI
 @Observable
 @MainActor
 package final class PluginRegistry {
-    package private(set) var plugins: [any HalenPlugin] = []
+    public private(set) var plugins: [any HalenPlugin] = []
     private var enabledStates: [String: Bool] = [:]
 
     private let defaults = UserDefaults.standard