Skip to content

fix(desktop): lagging status indicator#3779

Open
elijah-potter wants to merge 11 commits into
masterfrom
fix-desktop-status
Open

fix(desktop): lagging status indicator#3779
elijah-potter wants to merge 11 commits into
masterfrom
fix-desktop-status

Conversation

@elijah-potter

Copy link
Copy Markdown
Collaborator

Issues

N/A

Description

I've noticed that much of the time, the desktop's status indicator was simply wrong. Out of the gate, I assume this was a side effect of the fact that the indicator itself was total slop.

So, I went ahead and rewrote that entire chunk of the program, making it far more readable and maintainable in the process. It now maintains a correct reflection of the status of the local Harper service. In addition, I found and resolved an issue that would result in the Harper service running after the parent process had ended.

How Has This Been Tested?

Manually.

AI Disclosure

  • I am a human and didn't use any AI.f
  • I used LLM features of my editor, but not an agent.
  • I used an AI agent interactively.
  • I am an agent or I got an agent to do the work autonomously.

Checklist

  • I have performed a self-review of my own code
  • I have added tests to cover my changes
  • I have considered splitting this into smaller pull requests.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant