@sriinnu/runic 1.0.0 → 2.0.0

package/Sources/Runic/Views/Preferences/PreferencesIntegrationsPane.swift

@@ -1,5 +1,6 @@
 import AppKit
 import RunicCore
+import Security
 import SwiftUI
 
 @MainActor
@@ -9,7 +10,7 @@ struct IntegrationsPane: View {
 
     @AppStorage("vaayuAPIServerEnabled") private var vaayuAPIServerEnabled = false
     @AppStorage("vaayuAPIServerPort") private var vaayuAPIServerPort = 3000
-    @AppStorage("vaayuAPIKey") private var vaayuAPIKey = ""
+    @State private var vaayuAPIKey = ""
     @AppStorage("defaultWebhookURL") private var defaultWebhookURL = ""
     @AppStorage("webhookFormat") private var webhookFormat = "slack"
     @AppStorage("githubIntegrationEnabled") private var githubIntegrationEnabled = false
@@ -28,15 +29,20 @@ struct IntegrationsPane: View {
         var isRunning: Bool
         var port: Int
 
-        var statusIndicator: String { "●" }
-        var statusColor: Color { self.isRunning ? .green : .red }
+        var statusIndicator: String {
+            "●"
+        }
+
+        var statusColor: Color {
+            self.isRunning ? .green : .red
+        }
     }
 
     var body: some View {
         PreferencesPane {
             SettingsSection(contentSpacing: PreferencesLayoutMetrics.sectionSpacing) {
                 Text("vaayu Integration")
-                    .font(.caption)
+                    .font(RunicFont.caption)
                     .foregroundStyle(.secondary)
                     .textCase(.uppercase)
 
@@ -49,11 +55,11 @@ struct IntegrationsPane: View {
 
                         if self.vaayuAPIServerEnabled {
                             Text("Running on port \(self.vaayuAPIServerPort)")
-                                .font(.footnote)
+                                .font(RunicFont.footnote)
                                 .foregroundStyle(.green)
                         } else {
                             Text("Stopped")
-                                .font(.footnote)
+                                .font(RunicFont.footnote)
                                 .foregroundStyle(.secondary)
                         }
                     }
@@ -61,21 +67,21 @@ struct IntegrationsPane: View {
                     if self.vaayuAPIServerEnabled {
                         VStack(alignment: .leading, spacing: 4) {
                             Text("Port number")
-                                .font(.body)
+                                .font(RunicFont.body)
 
                             TextField("Port", value: self.$vaayuAPIServerPort, format: .number)
                                 .textFieldStyle(.roundedBorder)
                                 .frame(maxWidth: 150)
 
                             Text("Default: 3000. Restart required after changing port.")
-                                .font(.footnote)
+                                .font(RunicFont.footnote)
                                 .foregroundStyle(.tertiary)
                         }
 
                         VStack(alignment: .leading, spacing: 4) {
                             HStack {
                                 Text("API Key")
-                                    .font(.body)
+                                    .font(RunicFont.body)
 
                                 Spacer()
 
@@ -98,7 +104,7 @@ struct IntegrationsPane: View {
 
                             if !self.vaayuAPIKey.isEmpty {
                                 Text(self.maskedAPIKey)
-                                    .font(.footnote.monospaced())
+                                    .font(RunicFont.footnote.monospaced())
                                     .foregroundStyle(.secondary)
                                     .padding(RunicSpacing.xs)
                                     .background(Color(nsColor: .controlBackgroundColor))
@@ -107,14 +113,14 @@ struct IntegrationsPane: View {
 
                             if let generatedKey = self.generatedAPIKey {
                                 Text("New key generated: \(generatedKey)")
-                                    .font(.footnote)
+                                    .font(RunicFont.footnote)
                                     .foregroundStyle(.green)
                             }
                         }
                     }
 
                     Text("Exposes Runic usage data via REST API for external integrations.")
-                        .font(.footnote)
+                        .font(RunicFont.footnote)
                         .foregroundStyle(.tertiary)
                 }
             }
@@ -123,14 +129,14 @@ struct IntegrationsPane: View {
 
             SettingsSection(contentSpacing: PreferencesLayoutMetrics.sectionSpacing) {
                 Text("MCP Servers")
-                    .font(.caption)
+                    .font(RunicFont.caption)
                     .foregroundStyle(.secondary)
                     .textCase(.uppercase)
 
                 VStack(spacing: RunicSpacing.xs) {
                     HStack {
                         Text("Configured servers")
-                            .font(.body)
+                            .font(RunicFont.body)
                         Spacer()
                         Button {
                             self.showingAddServerSheet = true
@@ -147,10 +153,10 @@ struct IntegrationsPane: View {
                                 .font(.system(size: 32))
                                 .foregroundStyle(.tertiary)
                             Text("No MCP servers yet")
-                                .font(.body.weight(.semibold))
+                                .font(RunicFont.body.weight(.semibold))
                                 .foregroundStyle(.secondary)
                             Text("Add a server to manage connection details")
-                                .font(.footnote)
+                                .font(RunicFont.footnote)
                                 .foregroundStyle(.tertiary)
                         }
                         .frame(maxWidth: .infinity)
@@ -160,13 +166,13 @@ struct IntegrationsPane: View {
                             HStack(spacing: RunicSpacing.sm) {
                                 Text(server.statusIndicator)
                                     .foregroundStyle(server.statusColor)
-                                    .font(.title3)
+                                    .font(RunicFont.title3)
 
                                 VStack(alignment: .leading, spacing: RunicSpacing.xxs) {
                                     Text(server.name)
-                                        .font(.body)
+                                        .font(RunicFont.body)
                                     Text("Port \(server.port)")
-                                        .font(.footnote)
+                                        .font(RunicFont.footnote)
                                         .foregroundStyle(.secondary)
                                 }
 
@@ -205,7 +211,7 @@ struct IntegrationsPane: View {
                 }
 
                 Text("MCP servers enable Claude Desktop and other tools to access Runic data.")
-                    .font(.footnote)
+                    .font(RunicFont.footnote)
                     .foregroundStyle(.tertiary)
             }
 
@@ -213,26 +219,26 @@ struct IntegrationsPane: View {
 
             SettingsSection(contentSpacing: PreferencesLayoutMetrics.sectionSpacing) {
                 Text("Webhooks")
-                    .font(.caption)
+                    .font(RunicFont.caption)
                     .foregroundStyle(.secondary)
                     .textCase(.uppercase)
 
                 VStack(alignment: .leading, spacing: RunicSpacing.sm) {
                     VStack(alignment: .leading, spacing: 4) {
                         Text("Default webhook URL")
-                            .font(.body)
+                            .font(RunicFont.body)
 
                         TextField("https://hooks.slack.com/services/...", text: self.$defaultWebhookURL)
                             .textFieldStyle(.roundedBorder)
 
                         Text("Used for all alerts unless overridden in alert rule settings.")
-                            .font(.footnote)
+                            .font(RunicFont.footnote)
                             .foregroundStyle(.tertiary)
                     }
 
                     VStack(alignment: .leading, spacing: 4) {
                         Text("Webhook format")
-                            .font(.body)
+                            .font(RunicFont.body)
 
                         Picker("", selection: self.$webhookFormat) {
                             Text("Slack").tag("slack")
@@ -243,7 +249,7 @@ struct IntegrationsPane: View {
                         .frame(maxWidth: 300)
 
                         Text("Formats webhook payloads for different platforms.")
-                            .font(.footnote)
+                            .font(RunicFont.footnote)
                             .foregroundStyle(.tertiary)
                     }
 
@@ -256,7 +262,7 @@ struct IntegrationsPane: View {
 
                         if let result = self.testWebhookResult {
                             Text(result)
-                                .font(.footnote)
+                                .font(RunicFont.footnote)
                                 .foregroundStyle(result.contains("Success") ? .green : .red)
                         }
                     }
@@ -267,7 +273,7 @@ struct IntegrationsPane: View {
 
             SettingsSection(contentSpacing: PreferencesLayoutMetrics.sectionSpacing) {
                 Text("GitHub Integration")
-                    .font(.caption)
+                    .font(RunicFont.caption)
                     .foregroundStyle(.secondary)
                     .textCase(.uppercase)
 
@@ -280,7 +286,7 @@ struct IntegrationsPane: View {
                     if self.githubIntegrationEnabled {
                         VStack(alignment: .leading, spacing: 4) {
                             Text("Repository path")
-                                .font(.body)
+                                .font(RunicFont.body)
 
                             HStack {
                                 TextField("/path/to/repo", text: self.$githubRepositoryPath)
@@ -299,7 +305,7 @@ struct IntegrationsPane: View {
                                         Image(systemName: "checkmark.circle.fill")
                                             .foregroundStyle(.green)
                                         Text("Valid Git repository")
-                                            .font(.footnote)
+                                            .font(RunicFont.footnote)
                                             .foregroundStyle(.green)
                                     }
                                 } else {
@@ -307,7 +313,7 @@ struct IntegrationsPane: View {
                                         Image(systemName: "xmark.circle.fill")
                                             .foregroundStyle(.red)
                                         Text("Not a valid Git repository")
-                                            .font(.footnote)
+                                            .font(RunicFont.footnote)
                                             .foregroundStyle(.red)
                                     }
                                 }
@@ -319,6 +325,22 @@ struct IntegrationsPane: View {
         }
         .onAppear {
             self.loadMCPServers()
+            self.vaayuAPIKey = VaayuKeychainHelper.load() ?? ""
+            // Migrate any existing value out of UserDefaults
+            let defaults = UserDefaults.standard
+            if let legacyKey = defaults.string(forKey: "vaayuAPIKey"), !legacyKey.isEmpty {
+                if VaayuKeychainHelper.save(legacyKey) {
+                    self.vaayuAPIKey = legacyKey
+                    defaults.removeObject(forKey: "vaayuAPIKey")
+                }
+            }
+        }
+        .onChange(of: self.vaayuAPIKey) { _, newValue in
+            if newValue.isEmpty {
+                VaayuKeychainHelper.delete()
+            } else {
+                VaayuKeychainHelper.save(newValue)
+            }
         }
         .sheet(isPresented: self.$showingAddServerSheet) {
             AddMCPServerSheet(
@@ -462,11 +484,11 @@ private struct AddMCPServerSheet: View {
     var body: some View {
         VStack(alignment: .leading, spacing: RunicSpacing.lg) {
             Text("Add MCP Server")
-                .font(.title2.weight(.semibold))
+                .font(RunicFont.title2.weight(.semibold))
 
             VStack(alignment: .leading, spacing: RunicSpacing.xs) {
                 Text("Name")
-                    .font(.subheadline.weight(.medium))
+                    .font(RunicFont.subheadline.weight(.medium))
                 TextField("Local MCP Server", text: self.$name)
                     .textFieldStyle(.roundedBorder)
                     .focused(self.$isNameFocused)
@@ -474,7 +496,7 @@ private struct AddMCPServerSheet: View {
 
             VStack(alignment: .leading, spacing: RunicSpacing.xs) {
                 Text("Port")
-                    .font(.subheadline.weight(.medium))
+                    .font(RunicFont.subheadline.weight(.medium))
                 TextField("", value: self.$port, format: .number)
                     .textFieldStyle(.roundedBorder)
                     .frame(width: 140)
@@ -495,3 +517,56 @@ private struct AddMCPServerSheet: View {
         .onAppear { self.isNameFocused = true }
     }
 }
+
+// MARK: - Vaayu API Key Keychain Helper
+
+private enum VaayuKeychainHelper {
+    private static let service = "com.sriinnu.athena.Runic"
+    private static let account = "vaayu-api-key"
+
+    static func load() -> String? {
+        var result: CFTypeRef?
+        let query: [String: Any] = [
+            kSecClass as String: kSecClassGenericPassword,
+            kSecAttrService as String: self.service,
+            kSecAttrAccount as String: self.account,
+            kSecMatchLimit as String: kSecMatchLimitOne,
+            kSecReturnData as String: true,
+            kSecUseAuthenticationUI as String: kSecUseAuthenticationUIFail,
+        ]
+        let status = SecItemCopyMatching(query as CFDictionary, &result)
+        guard status == errSecSuccess,
+              let data = result as? Data,
+              let token = String(data: data, encoding: .utf8)?.trimmingCharacters(in: .whitespacesAndNewlines),
+              !token.isEmpty
+        else {
+            return nil
+        }
+        return token
+    }
+
+    @discardableResult
+    static func save(_ value: String) -> Bool {
+        self.delete()
+        let data = Data(value.utf8)
+        let query: [String: Any] = [
+            kSecClass as String: kSecClassGenericPassword,
+            kSecAttrService as String: self.service,
+            kSecAttrAccount as String: self.account,
+            kSecValueData as String: data,
+            kSecAttrAccessible as String: kSecAttrAccessibleAfterFirstUnlock,
+        ]
+        return SecItemAdd(query as CFDictionary, nil) == errSecSuccess
+    }
+
+    @discardableResult
+    static func delete() -> Bool {
+        let query: [String: Any] = [
+            kSecClass as String: kSecClassGenericPassword,
+            kSecAttrService as String: self.service,
+            kSecAttrAccount as String: self.account,
+        ]
+        let status = SecItemDelete(query as CFDictionary)
+        return status == errSecSuccess || status == errSecItemNotFound
+    }
+}

package/Sources/Runic/Views/Preferences/PreferencesPerformancePane.swift

@@ -23,17 +23,19 @@ struct PerformancePane: View {
     @State private var appeared = false
 
     enum QualityRatingFrequency: String, CaseIterable, Identifiable {
-        case every = "every"
+        case every
         case over1000 = "over_1000"
         case over5000 = "over_5000"
 
-        var id: String { self.rawValue }
+        var id: String {
+            self.rawValue
+        }
 
         var label: String {
             switch self {
-            case .every: return "After every response"
-            case .over1000: return "Only >1000 tokens"
-            case .over5000: return "Only >5000 tokens"
+            case .every: "After every response"
+            case .over1000: "Only >1000 tokens"
+            case .over5000: "Only >5000 tokens"
             }
         }
     }
@@ -47,8 +49,9 @@ struct PerformancePane: View {
                     binding: self.$performanceTrackingEnabled)
 
                 if !self.performanceTrackingEnabled {
-                    Text("Performance tracking is disabled. Historical data is preserved but new metrics won't be collected.")
-                        .font(.footnote)
+                    Text(
+                        "Performance tracking is disabled. Historical data is preserved but new metrics won't be collected.")
+                        .font(RunicFont.footnote)
                         .foregroundStyle(.orange)
                         .padding(.vertical, RunicSpacing.xs)
                 }
@@ -59,7 +62,7 @@ struct PerformancePane: View {
                 VStack(alignment: .leading, spacing: RunicSpacing.sm) {
                     VStack(alignment: .leading, spacing: 4) {
                         Text("Raw metrics retention")
-                            .font(.body)
+                            .font(RunicFont.body)
 
                         Picker("", selection: self.$rawMetricsRetentionDays) {
                             Text("30 days").tag(30)
@@ -70,13 +73,13 @@ struct PerformancePane: View {
                         .frame(maxWidth: 300)
 
                         Text("How long to keep detailed latency and error records.")
-                            .font(.footnote)
+                            .font(RunicFont.footnote)
                             .foregroundStyle(.tertiary)
                     }
 
                     VStack(alignment: .leading, spacing: 4) {
                         Text("Aggregated stats retention")
-                            .font(.body)
+                            .font(RunicFont.body)
 
                         Picker("", selection: self.$aggregatedStatsRetentionYears) {
                             Text("1 year").tag(1)
@@ -87,7 +90,7 @@ struct PerformancePane: View {
                         .frame(maxWidth: 300)
 
                         Text("How long to keep daily performance summaries.")
-                            .font(.footnote)
+                            .font(RunicFont.footnote)
                             .foregroundStyle(.tertiary)
                     }
                 }
@@ -104,7 +107,7 @@ struct PerformancePane: View {
                     if self.qualityRatingPromptsEnabled {
                         VStack(alignment: .leading, spacing: 4) {
                             Text("Prompt frequency")
-                                .font(.body)
+                                .font(RunicFont.body)
 
                             Picker("", selection: self.$qualityRatingFrequency) {
                                 ForEach(QualityRatingFrequency.allCases) { freq in
@@ -115,7 +118,7 @@ struct PerformancePane: View {
                             .frame(maxWidth: 400)
 
                             Text("When to show rating prompts based on response size.")
-                                .font(.footnote)
+                                .font(RunicFont.footnote)
                                 .foregroundStyle(.tertiary)
                         }
 
@@ -135,9 +138,9 @@ struct PerformancePane: View {
                 VStack(alignment: .leading, spacing: RunicSpacing.sm) {
                     HStack {
                         Text("Current database size:")
-                            .font(.body)
+                            .font(RunicFont.body)
                         Text(self.databaseSize)
-                            .font(.body.weight(.semibold))
+                            .font(RunicFont.body.weight(.semibold))
                             .foregroundStyle(.secondary)
                     }
 
@@ -157,13 +160,13 @@ struct PerformancePane: View {
 
                         if let status = self.vacuumStatus {
                             Text(status)
-                                .font(.footnote)
+                                .font(RunicFont.footnote)
                                 .foregroundStyle(status.contains("Success") ? .green : .red)
                         }
                     }
 
                     Text("Optimizes database file by reclaiming unused space.")
-                        .font(.footnote)
+                        .font(RunicFont.footnote)
                         .foregroundStyle(.tertiary)
 
                     Divider()
@@ -185,13 +188,13 @@ struct PerformancePane: View {
 
                         if let status = self.clearDataStatus {
                             Text(status)
-                                .font(.footnote)
+                                .font(RunicFont.footnote)
                                 .foregroundStyle(status.contains("Success") ? .green : .red)
                         }
                     }
 
                     Text("Removes metrics older than configured retention periods.")
-                        .font(.footnote)
+                        .font(RunicFont.footnote)
                         .foregroundStyle(.tertiary)
                 }
             }
@@ -204,8 +207,9 @@ struct PerformancePane: View {
                     binding: self.$anonymousUsageStatsEnabled)
 
                 if self.anonymousUsageStatsEnabled {
-                    Text("Only aggregate statistics are shared. Request IDs, prompts, and responses are never included.")
-                        .font(.footnote)
+                    Text(
+                        "Only aggregate statistics are shared. Request IDs, prompts, and responses are never included.")
+                        .font(RunicFont.footnote)
                         .foregroundStyle(.tertiary)
                         .padding(.vertical, RunicSpacing.xs)
                 }
@@ -223,8 +227,7 @@ struct PerformancePane: View {
         Task {
             let appSupport = FileManager.default.urls(
                 for: .applicationSupportDirectory,
-                in: .userDomainMask
-            ).first!
+                in: .userDomainMask).first!
             let dbPath = appSupport.appendingPathComponent("Runic/performance.db")
 
             if let attributes = try? FileManager.default.attributesOfItem(atPath: dbPath.path),