Repository navigation
fix: repair service, TUI and desktop interface regressions - #105
Merged
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Fix reproduced regressions found while auditing Driga's CLI, TUI, desktop, MCP, chat and service interfaces.
Also restore table k navigation, retain dashboards on transient errors, and show resume commands when clipboard access fails. Update provider privacy wording and the header metric limit.
Evidence
After: regression tests exercise repaired shipping components and fixture subprocesses. Independent Standards and Spec reviews exposed pending-mode and wheel-cancellation gaps; Astra independently reproduced the deleted-dashboard history trap. Fixes and final results are recorded in the audit.
Merge Danger
Door: two-way
Blast Radius: interfaces
No schema, wire, configuration or assistant-history changes. Service status executes a bounded read-only launchctl query; owned fixture/process cleanup never targets user workloads. TUI stop execution retains existing confirmation and identity checks. Chat now requires a successful Confirm reset before hiding a known conversation.
Risk and rollout
Reviewable bug fixes; no migration. Installed release remains unchanged. Native windows/notifications, real vendor authentication/model turns, live fleets and real destructive reclamation remain outside fixture coverage.
Contract changes
None.
Verified
go test ./... -count=1,go test -race ./..., final affected TUI/service tests and TUI race tests.See the audit for browser-matrix results and explicit live-platform limits. No real assistant CLIs or production data were used in tests.