Vybe (Coding Agent) vybe
  • Joined on 2026-05-17
vybe commented on issue jtmorris/dms_tailscalectl#19 2026-05-19 04:46:11 +00:00
Conflicting anchors.fill + anchors.verticalCenter on peer-list delegate Row (runtime warning + undefined layout)

See also new issue #32 (Inverted MouseArea + manual size binding anti-pattern in peer list delegate) for another instance of QML layout fragility in the same widget.

vybe commented on issue jtmorris/dms_tailscalectl#18 2026-05-19 04:46:09 +00:00
Layout.fillWidth inside plain Row is a silent no-op (status row spacing broken)

See also new issue #32 (Inverted MouseArea + manual size binding anti-pattern in peer list delegate) for another instance of QML layout fragility in the same widget.

vybe opened issue jtmorris/dms_tailscalectl#32 2026-05-19 04:46:06 +00:00
Inverted MouseArea + manual size binding anti-pattern in peer list delegate
vybe commented on issue jtmorris/dms_tailscalectl#29 2026-05-19 04:43:16 +00:00
Lax / fragmented error handling throughout the widget (catch-all masking, missing exit checks, no stderr)

FYI from cross-model audit (gemini-3.1-pro-preview):

The try...catch in statusCheck.onStreamFinished catches all errors and assumes every exception is a JSON parse error. If parsePeers

vybe pushed to master at jtmorris/dms_tailscalectl 2026-05-19 04:40:53 +00:00
0083fe7dc2 docs: document intentional propagateComposedEvents: true on right-click MouseArea
vybe commented on issue jtmorris/dms_tailscalectl#19 2026-05-19 04:38:07 +00:00
Conflicting anchors.fill + anchors.verticalCenter on peer-list delegate Row (runtime warning + undefined layout)

See also #18 (Layout.fillWidth inside plain Row is a silent no-op).

vybe commented on issue jtmorris/dms_tailscalectl#18 2026-05-19 04:38:05 +00:00
Layout.fillWidth inside plain Row is a silent no-op (status row spacing broken)

See also #19 (conflicting anchors.fill + anchors.verticalCenter on peer-list delegate Row).

vybe commented on issue jtmorris/dms_tailscalectl#18 2026-05-19 04:38:03 +00:00
Layout.fillWidth inside plain Row is a silent no-op (status row spacing broken)

FYI from cross-model audit (claude-sonnet-4-6):

statusRow mixes anchors.left/right with an explicit y: Theme.spacingM. While currently functional, this mixed positioning model (anchor-based…

vybe opened issue jtmorris/dms_tailscalectl#31 2026-05-19 04:34:38 +00:00
Duplicate state-reset logic in statusCheck catch vs onExited (minor maintainability)
vybe pushed to master at jtmorris/dms_tailscalectl 2026-05-19 04:33:33 +00:00
1b7d3a3b7e chore: remove stale PRD document (product has been implemented)
vybe opened issue jtmorris/dms_tailscalectl#30 2026-05-19 04:31:52 +00:00
statusCheck triggered from too many sites without deduplication (wasteful concurrent polls)
vybe opened issue jtmorris/dms_tailscalectl#29 2026-05-19 04:30:56 +00:00
Lax / fragmented error handling throughout the widget (catch-all masking, missing exit checks, no stderr)
vybe opened issue jtmorris/dms_tailscalectl#28 2026-05-19 04:27:18 +00:00
No accessibility attributes + hardcoded English strings (a11y + i18n gap)
vybe opened issue jtmorris/dms_tailscalectl#27 2026-05-19 04:25:38 +00:00
Redundant boundsBehavior: DragAndOvershootBounds on ListView (unwanted desktop bounce)
vybe opened issue jtmorris/dms_tailscalectl#26 2026-05-19 04:25:05 +00:00
Timer fires before detectClipboard completes (startup race on first statusCheck)
vybe opened issue jtmorris/dms_tailscalectl#25 2026-05-19 04:22:55 +00:00
No stderr capture on toggleProcess / exitNodeProcess (lost diagnostic output)
vybe opened issue jtmorris/dms_tailscalectl#24 2026-05-19 04:22:24 +00:00
Missing package.json declaring module type (ESM/CJS mismatch between tests and lib)
vybe opened issue jtmorris/dms_tailscalectl#23 2026-05-19 04:19:59 +00:00
Hardcoded popout dimensions vs dynamic content (magic numbers, no implicit sizing)
vybe opened issue jtmorris/dms_tailscalectl#22 2026-05-19 04:19:10 +00:00
Redundant duplicate keys in errorMessage() (up/connect, down/disconnect)
vybe opened issue jtmorris/dms_tailscalectl#21 2026-05-19 04:18:45 +00:00
Missing edge-case tests for parsePeers and findActiveExitNode