v1.8.4: collapsible insights, star glow system, and menu bar polish (#103)

Redesigned Insights as a collapsible section with 5 rows (Today, All Time,
Longest, Tools, Period). Menu bar icon now uses severity-based glow: none
below 80%, static star glow at 80-95%, breathing at 95%+, starburst rays
when throttled. Text-first layout with 12pt font matches macOS battery.

Added two-tap logout confirmation, "Updated Xm ago" footer timestamp,
time-proximity boost for auto mode urgency, and shared CollapsibleSectionHeader/
FooterLink/RefreshButton components. CI skips doc-only and draft PR builds.

Co-authored-by: KyleNesium <22541778+KyleNesium@users.noreply.github.com>

Author: KyleNesium

.github/workflows/ci.yml

@@ -3,8 +3,20 @@ name: CI
 on:
   push:
     branches: [main]
+    paths-ignore:
+      - '**.md'
+      - 'spec/**'
+      - 'LICENSE'
+      - '.github/*.md'
   pull_request:
     branches: [main]
+    types: [opened, synchronize, ready_for_review]
+    paths-ignore:
+      - '**.md'
+      - 'spec/**'
+      - 'LICENSE'
+      - '.github/*.md'
+  workflow_dispatch:
 
 concurrency:
   group: ${{ github.workflow }}-${{ github.ref }}
@@ -13,6 +25,7 @@ concurrency:
 jobs:
   build-and-test:
     name: Build, Test & Verify
+    if: github.event_name == 'workflow_dispatch' || github.event.pull_request.draft != true
     runs-on: macos-15
     timeout-minutes: 10
     steps:

AIBattery/Info.plist

@@ -17,9 +17,9 @@
 	<key>CFBundlePackageType</key>
 	<string>APPL</string>
 	<key>CFBundleShortVersionString</key>
-	<string>1.8.3</string>
+	<string>1.8.4</string>
 	<key>CFBundleVersion</key>
-	<string>1.8.3</string>
+	<string>1.8.4</string>
 	<key>LSMinimumSystemVersion</key>
 	<string>13.0</string>
 	<key>CFBundleIconFile</key>

AIBattery/Models/UsageSnapshot.swift

@@ -67,16 +67,34 @@ struct UsageSnapshot {
         }
 
         // Tier 3: Urgency-normalized — each mode's thresholds map to a shared 0–1 scale.
+        // Rate limit modes get a time-proximity boost when estimated time to limit is short:
+        // a window hitting its cap in minutes is more urgent than one with hours left.
         // Ties broken by actionability: context > 5h > 7d.
         let scored: [(MetricMode, Double)] = MetricMode.allCases.map { mode in
-            (mode, Self.urgencyScore(percent: percent(for: mode), mode: mode))
+            var score = Self.urgencyScore(percent: percent(for: mode), mode: mode)
+            if let rl = rateLimits {
+                let window = mode == .sevenDay ? RateLimitUsage.sevenDayWindow : RateLimitUsage.fiveHourWindow
+                if mode == .fiveHour || mode == .sevenDay,
+                   let ttl = rl.estimatedTimeToLimit(for: window) {
+                    // Boost: <30min → +0.20, <2h → +0.10, <6h → +0.05
+                    let boost: Double
+                    switch ttl {
+                    case ..<(30 * 60):    boost = 0.20
+                    case ..<(2 * 3600):   boost = 0.10
+                    case ..<(6 * 3600):   boost = 0.05
+                    default:              boost = 0.0
+                    }
+                    score += boost
+                }
+            }
+            return (mode, score)
         }
         let maxUrgency = scored.map(\.1).max() ?? 0
         // Among tied modes, prefer context > 5h > 7d (most actionable first)
         let tiePriority: [MetricMode] = [.contextHealth, .fiveHour, .sevenDay]
         return scored
             .filter { $0.1 == maxUrgency }
-            .min { tiePriority.firstIndex(of: $0.0)! < tiePriority.firstIndex(of: $1.0)! }!
+            .min { (tiePriority.firstIndex(of: $0.0) ?? .max) < (tiePriority.firstIndex(of: $1.0) ?? .max) }!
             .0
     }
 

AIBattery/Utilities/DateFormatters.swift

@@ -33,4 +33,30 @@ enum DateFormatters {
         f.dateFormat = "MMM"
         return f
     }()
+
+    /// "Nov 6" — month + day, no year. Pinned to en_US_POSIX for deterministic output.
+    static let rangeShort: DateFormatter = {
+        let f = DateFormatter()
+        f.locale = Locale(identifier: "en_US_POSIX")
+        f.dateFormat = "MMM d"
+        return f
+    }()
+
+    /// "Nov 6, 2025" — month + day + year. Pinned to en_US_POSIX for deterministic output.
+    static let rangeWithYear: DateFormatter = {
+        let f = DateFormatter()
+        f.locale = Locale(identifier: "en_US_POSIX")
+        f.dateFormat = "MMM d, yyyy"
+        return f
+    }()
+
+    /// Formats a date range: same year → "Nov 6 – Mar 10, 2026", cross-year → "Dec 15, 2025 – Mar 10, 2026".
+    static func formatDateRange(from start: Date, to end: Date) -> String {
+        let cal = Calendar.current
+        let sameYear = cal.component(.year, from: start) == cal.component(.year, from: end)
+        if sameYear {
+            return "\(rangeShort.string(from: start)) – \(rangeWithYear.string(from: end))"
+        }
+        return "\(rangeWithYear.string(from: start)) – \(rangeWithYear.string(from: end))"
+    }
 }

AIBattery/Utilities/ThemeColors.swift

@@ -49,21 +49,28 @@ enum ThemeColors {
     /// the opaque light-mode background provides enough contrast.
     private static let deepOrange = Color(nsColor: .systemOrange)
 
+    /// System colors wrapped as SwiftUI Color — matches barNSColor exactly.
+    private static let sysGreen = Color(nsColor: .systemGreen)
+    private static let sysRed = Color(nsColor: .systemRed)
+    private static let sysBlue = Color(nsColor: .systemBlue)
+    private static let sysCyan = Color(nsColor: .systemTeal)
+    private static let sysPurple = Color(nsColor: .systemPurple)
+
     /// Color for a usage percentage (0–100).
     static func barColor(percent: Double) -> Color {
         if isColorblind {
             switch percent {
-            case 0..<50: return .blue
-            case 50..<80: return .cyan
+            case 0..<50: return sysBlue
+            case 50..<80: return sysCyan
             case 80..<95: return amber
-            default: return .purple
+            default: return sysPurple
             }
         }
         switch percent {
-        case 0..<50: return .green
+        case 0..<50: return sysGreen
         case 50..<80: return gold
         case 80..<95: return deepOrange
-        default: return .red
+        default: return sysRed
         }
     }
 

AIBattery/Utilities/UserDefaultsKeys.swift

@@ -24,6 +24,7 @@ enum UserDefaultsKeys {
     static let contextCollapsed = "aibattery_contextCollapsed"
     static let tokensCollapsed = "aibattery_tokensCollapsed"
     static let activityCollapsed = "aibattery_activityCollapsed"
+    static let insightsCollapsed = "aibattery_insightsCollapsed"
     /// Prefix for per-account token expiry timestamps (append account ID).
     static let tokenExpiresAtPrefix = "aibattery_expiresAt_"
 }

AIBattery/Views/ActivityChartView.swift

@@ -65,24 +65,11 @@ struct ActivityChartView: View {
         VStack(alignment: .leading, spacing: 6) {
             // Header with toggle
             HStack {
-                Button {
-                    withAnimation(.easeInOut(duration: 0.2)) { collapsed.toggle() }
-                } label: {
-                    HStack(spacing: 4) {
-                        Image(systemName: "chevron.right")
-                            .font(.system(size: 8, weight: .bold))
-                            .rotationEffect(.degrees(collapsed ? 0 : 90))
-                            .foregroundStyle(ThemeColors.tertiaryLabel)
-                        Text("Activity")
-                            .font(.subheadline.bold())
-                            .foregroundStyle(.primary)
-                    }
-                    .contentShape(Rectangle())
-                }
-                .buttonStyle(.plain)
-                .accessibilityLabel("Activity, \(collapsed ? "collapsed" : "expanded")")
-                .accessibilityHint(collapsed ? "Double-tap to expand" : "Double-tap to collapse")
-                .help("Message activity over time")
+                CollapsibleSectionHeader(
+                    title: "Activity",
+                    collapsed: $collapsed,
+                    tooltip: "Message activity over time"
+                )
                 Spacer()
                 if collapsed, let snapshot, let change = changeVsYesterday(snapshot) {
                     Text(change.symbol)
@@ -114,7 +101,7 @@ struct ActivityChartView: View {
                     Image(systemName: "chart.line.flattrend.xyaxis")
                         .font(.system(size: 14))
                         .foregroundStyle(ThemeColors.tertiaryLabel)
-                    Text("No activity data")
+                    Text("No activity in \(mode.rawValue) window")
                         .font(.caption2)
                         .foregroundStyle(ThemeColors.tertiaryLabel)
                 }
@@ -134,6 +121,7 @@ struct ActivityChartView: View {
             // Trend summary
             if !collapsed, let snapshot {
                 trendSummary(snapshot)
+                    .accessibilityElement(children: .combine)
             }
         }
         .padding(.horizontal, 16)
@@ -438,7 +426,7 @@ struct ActivityChartView: View {
                     .font(.system(.caption, design: .monospaced))
                     .foregroundStyle(ThemeColors.caution)
             } else {
-                Text("Throttled: 0")
+                Text("Throttled: 0×")
                     .font(.system(.caption, design: .monospaced))
                     .foregroundStyle(ThemeColors.secondaryLabel)
             }

AIBattery/Views/AuthView.swift

@@ -97,9 +97,10 @@ public struct AuthView: View {
                     }
                     .frame(maxWidth: .infinity, alignment: .leading)
 
-                    TextField("Paste authorization code...", text: $authCode)
+                    TextField("Paste code...", text: $authCode)
                         .textFieldStyle(.roundedBorder)
                         .font(.system(.caption, design: .monospaced))
+                        .onSubmit(submitCode)
                         .accessibilityLabel("Authorization code")
                         .accessibilityHint("Paste the code from the browser")
 
@@ -160,6 +161,7 @@ public struct AuthView: View {
                         .buttonStyle(.plain)
                         .font(.caption2)
                         .foregroundStyle(.secondary)
+                        .keyboardShortcut("q", modifiers: .command)
                 }
                 Spacer()
             }

AIBattery/Views/CollapsibleSectionHeader.swift

@@ -0,0 +1,31 @@
+import SwiftUI
+
+/// Shared collapsible section header with rotating chevron and bold title.
+/// Used by Context Health, Tokens, Activity, and Insights sections.
+struct CollapsibleSectionHeader: View {
+    let title: String
+    @Binding var collapsed: Bool
+    var tooltip: String = ""
+
+    var body: some View {
+        Button {
+            withAnimation(.easeInOut(duration: 0.2)) { collapsed.toggle() }
+        } label: {
+            HStack(spacing: 4) {
+                Image(systemName: "chevron.right")
+                    .font(.system(size: 8, weight: .bold))
+                    .rotationEffect(.degrees(collapsed ? 0 : 90))
+                    .foregroundStyle(ThemeColors.tertiaryLabel)
+                Text(title)
+                    .font(.subheadline.bold())
+                    .foregroundStyle(.primary)
+            }
+            .contentShape(Rectangle())
+        }
+        .buttonStyle(.plain)
+        .help(tooltip)
+        .accessibilityAddTraits(.isHeader)
+        .accessibilityLabel("\(title), \(collapsed ? "collapsed" : "expanded")")
+        .accessibilityHint(collapsed ? "Double-tap to expand" : "Double-tap to collapse")
+    }
+}

AIBattery/Views/CopyableText.swift

@@ -58,13 +58,13 @@ struct CopyableModifier: ViewModifier {
                 // Cancel any previous feedback timer
                 feedbackTask?.cancel()
 
-                withAnimation(.easeOut(duration: 0.12)) {
+                withAnimation(.easeOut(duration: 0.15)) {
                     copied = true
                 }
                 feedbackTask = Task {
-                    try? await Task.sleep(nanoseconds: 1_200_000_000)
+                    try? await Task.sleep(nanoseconds: 1_500_000_000)
                     guard !Task.isCancelled else { return }
-                    withAnimation(.easeIn(duration: 0.2)) {
+                    withAnimation(.easeIn(duration: 0.15)) {
                         copied = false
                     }
                 }