Yeah, you don't just get to say "I'm not doing that because it's hard" and call that a plan to implement resolution to this issue. I'm fine keeping focus narrow now and working on the localization…
Okay. No red flags from me. Do it. Let's see the result.
- There is no
resetState()at this time. You're referencing another unimplemented issue's proposed code. That is not guaranteed to exist. You should add this caveat if you're going to be that…
I'm not redundant here! I am the one who smelled something fishy in Denmark. Yes, I called in Claude for backup, but I knew something was off. DON'T JUDGE ME!!!!
Anyway. @vybe. When…
The general idea look okay. But check your detectClipboard.onExited behavior before implementing your plan. #12's plan is to completely refactor that behavior. #12's plan may be implemented first.
Good enough to give it a shot.
Looks okay enough. Give it a shot @vybe.
Not sold on step 3. This is a simple toggle. Its objective should be a dirt simple toggle. You're storing state all over the place. Why even bother telling the user what state the action failed to…
I'm not sold on this approach. It's better than what's there, but it feels overengineered and like we are papering over the problem.
The whitelist validation is only necessary because the…
Yes. Very much agree that the current tests are minimally useful. Needs dramatic expansion to cover edge cases and drastically widen coverage, or it should be removed as code maintenance overhead,…
See #33's discussion on the 5-second timer disaster design decision.
Yeah, this is the second worst decision in @vybe's vibe-coded solution in my opinion. We don't need to poll Tailscale very often at all. I'd up this to 5-minute polling even, with triggers on…
This issue should probably be looked at more generically as well. In the event of unexpected output or command failure, how does the user know what happened? Some guidance is needed. Either the…
Yes! The popout size and scroll behavior could use some refinement generally. For most users, it can probably dynamically size to fit the content. However, there should be some maximum size.…
Gemini also flagged a good point about the which command being non-POSIX compliant and not as universal as the POSIX command command -v. While fixing this issue, serious consideration to…
@vybe: Note, when you get to this one, let's not publish to DMS plugin registry. Yet. Just polish the packaging, include screenshots, et cetera.
The issue was resolved by adding a root-level MouseArea at the PluginComponent level (sibling to BasePill) with acceptedButtons: Qt.RightButton. This ensures right-click events are captured by our…