fix(ui): decouple barRemaining from accent, add popover dismiss monitor, and fix codex issue override

[jaywedgeworth22]

Summary

• Decouple Quota Progress Bar Remaining Segment: Theme.barRemaining no longer binds to Theme.accent when the user chooses a warm or danger-adjacent accent (such as magenta or orange), which previously turned entire quota bars completely red/pink. It now preserves healthy teal (or green).
• Popover Dismissal on Outside Click: Adopted NSPopoverDelegate and registered an NSEvent global mouse-down monitor on show (removed on close) so clicking anywhere outside the menu bar popover dismisses it immediately on the first click.
• Codex Local Issue Override: Fixed MonitorModel.currentLocalResult() to clear issues["openai"] whenever reconciled local windows contain valid quota readings, preventing missing recent CLI session files from stamping "unavailable" over live ChatGPT API readings.
• Reset Alarm Audibility & Unanchored Jumps: Played audible alarm sounds on any armed reset notification when the chosen tone is audible, and allowed ResetAlarmTracker to recognize jumps to 100% even if the window initially lacked an anchored reset deadline.

Verification

• swift test: 365 tests passed, 0 failures across QuotaCoreTests and CodeCapsTests.

[kody-ai]

Code Review Completed! 🔥

The code review was successfully completed based on your current configurations.

Kody Guide: Usage and Configuration

Interacting with Kody

• Request a Review: Ask Kody to review your PR manually by adding a comment with the @kody start-review command at the root of your PR.

• Validate Business Logic: Ask Kody to validate your code against business rules by adding a comment with the @kody -v business-logic command.

• Provide Feedback: Help Kody learn and improve by reacting to its comments with a 👍 for helpful suggestions or a 👎 if improvements are needed.

Current Kody Configuration

Review Options

The following review options are enabled or disabled:

Options	Enabled
Bug	✅
Performance	✅
Security	✅
Business Logic	✅

Access your configuration settings here.

​

[kody-ai]

Markdown rendering bug: GitHub-flavored markdown collapses the two literal ASCII spaces after the period in red/pink. It now preserves healthy teal (or green)., erasing the intentional sentence gap in chat/PR render. Emit   (HTML entity plus a normal space) right after the period to preserve the gap — red/pink.  It now preserves healthy teal (or green). — and apply this fix only to the PR description, not to file content.

Kody rule violation: Use two spaces between sentences in every human-facing string and agent-written paragraph

[kody-ai]

Race condition in AppDelegate.swift: the Task { @MainActor in } wrapper defers the self.popover.isShown check, allowing a stale mouse-down to close a freshly-reopened popover when the popover closes and reopens between scheduling and execution, after which the queued Task reads popover.isShown == true and calls performClose on the new instance. Replace the Task wrapper with MainActor.assumeIsolated so the check and performClose execute synchronously with respect to the event.

outsideClickMonitor = NSEvent.addGlobalMonitorForEvents(matching: [.leftMouseDown, .rightMouseDown]) { [weak self] _ in
    MainActor.assumeIsolated {
        guard let self, self.popover.isShown else { return }
        self.popover.performClose(nil)
    }
}

Prompt for LLM

File Sources/CodeCaps/AppDelegate.swift:

Line 368:

Race condition in `AppDelegate.swift`: the `Task { @MainActor in }` wrapper defers the `self.popover.isShown` check, allowing a stale mouse-down to close a freshly-reopened popover when the popover closes and reopens between scheduling and execution, after which the queued Task reads `popover.isShown == true` and calls `performClose` on the new instance. Replace the Task wrapper with `MainActor.assumeIsolated` so the check and `performClose` execute synchronously with respect to the event.

Suggested Code:

outsideClickMonitor = NSEvent.addGlobalMonitorForEvents(matching: [.leftMouseDown, .rightMouseDown]) { [weak self] _ in
    MainActor.assumeIsolated {
        guard let self, self.popover.isShown else { return }
        self.popover.performClose(nil)
    }
}

_{Talk to Kody by mentioning @kody}

_{Was this suggestion helpful? React with 👍 or 👎 to help Kody learn from this interaction.}

​

​