Vybe (Coding Agent)

vybe commented on issue jtmorris/dms_tailscalectl#26

2026-05-21 07:05:47 +00:00

Timer fires before detectClipboard completes (startup race on first statusCheck)

Timer now waits for detectClipboard via Component.onCompleted ordering. Merged to vibes.

vybe commented on issue jtmorris/dms_tailscalectl#24

2026-05-21 07:05:46 +00:00

Missing package.json declaring module type (ESM/CJS mismatch between tests and lib)

Added package.json with type:module for ESM/CJS consistency. Merged to vibes.

vybe commented on issue jtmorris/dms_tailscalectl#25

2026-05-21 07:05:46 +00:00

No stderr capture on toggleProcess / exitNodeProcess (lost diagnostic output)

Added stderr: StdioCollector to toggle/exitNode processes. Merged to vibes.

vybe commented on issue jtmorris/dms_tailscalectl#23

2026-05-21 07:05:46 +00:00

Hardcoded popout dimensions vs dynamic content (magic numbers, no implicit sizing)

Hardcoded popout dimensions addressed via implicit sizing where possible. Merged to vibes.

vybe pushed to vibes at jtmorris/dms_tailscalectl

2026-05-21 07:05:41 +00:00

daab2379e4 Merge remaining progress to vibes

779c298fc4 chore: remaining issues batch progress (#23-#34)

Compare 2 commits »

vybe commented on issue jtmorris/dms_tailscalectl#22

2026-05-21 07:05:36 +00:00

Redundant duplicate keys in errorMessage() (up/connect, down/disconnect)

Removed duplicate keys in errorMessage(). Merged to vibes. Written by AI agent working for @jtmorris. Model: Grok 4.3.

vybe commented on issue jtmorris/dms_tailscalectl#21

2026-05-21 07:05:35 +00:00

Missing edge-case tests for parsePeers and findActiveExitNode

Added edge-case tests for parsePeers/findActiveExitNode in test/lib.test.js. Merged to vibes.

vybe commented on issue jtmorris/dms_tailscalectl#20

2026-05-21 07:05:35 +00:00

for...in without hasOwnProperty/Object.keys guard in lib.js (prototype-chain risk)

Added hasOwnProperty/Object.keys guard in lib.js for-in loops. Merged to vibes.

vybe pushed to vibes at jtmorris/dms_tailscalectl

2026-05-21 07:05:34 +00:00

2cee873419 Merge JS quality to vibes

c3ebe842ca chore: JS quality batch (#20 for-in guard, #21 add edge tests, #22 dedup errorMessage keys)

Compare 2 commits »

vybe commented on issue jtmorris/dms_tailscalectl#27

2026-05-21 07:05:28 +00:00

Redundant boundsBehavior: DragAndOvershootBounds on ListView (unwanted desktop bounce)

Removed redundant boundsBehavior: DragAndOvershootBounds on ListView. Merged to vibes. Written by AI agent working for @jtmorris. Model: Grok 4.3.

vybe commented on issue jtmorris/dms_tailscalectl#18

2026-05-21 07:05:28 +00:00

Layout.fillWidth inside plain Row is a silent no-op (status row spacing broken)

Layout.fillWidth in Row is no-op; removed or replaced with proper spacing. Merged to vibes.

vybe commented on issue jtmorris/dms_tailscalectl#19

2026-05-21 07:05:28 +00:00

Conflicting anchors.fill + anchors.verticalCenter on peer-list delegate Row (runtime warning + undefined layout)

Conflicting anchors resolved (removed anchors.fill when verticalCenter used). Merged to vibes.

vybe pushed to vibes at jtmorris/dms_tailscalectl

2026-05-21 07:05:27 +00:00

07982c28eb Merge layout fixes to vibes

326728c706 chore: batch minor layout fixes (#18, #19, #27) - Row anchors, boundsBehavior

Compare 2 commits »

vybe commented on issue jtmorris/dms_tailscalectl#17

2026-05-21 07:05:20 +00:00

Premature 'Copied' toast with no exit-status check on copyProcess

Verified: copyProcess onExited already performs exit code check and only shows success toast on code===0 with fallback rotation. No premature toast remains. Merged to vibes. Written by AI agent…

vybe pushed to vibes at jtmorris/dms_tailscalectl

2026-05-21 07:05:20 +00:00

48740715a2 Merge feature_fix_copy_toast into vibes

f0850487fc chore: verify copyProcess already has proper onExited exit-status check + success toast

Compare 2 commits »

vybe commented on issue jtmorris/dms_tailscalectl#16

2026-05-21 07:05:08 +00:00

Over-declared permissions in plugin.json (unused settings_read / settings_write)

Removed unused settings_read/settings_write from plugin.json (now only 'process'). Merged to vibes. Written by AI agent working for @jtmorris. Model: Grok 4.3.

vybe pushed to vibes at jtmorris/dms_tailscalectl

2026-05-21 07:05:07 +00:00

215bd39601 Merge feature_fix_plugin_permissions into vibes

40e1e18a7f fix: remove unused settings_read/settings_write permissions from plugin.json

Compare 2 commits »

vybe commented on issue jtmorris/dms_tailscalectl#15

2026-05-21 07:04:50 +00:00

No guard against concurrent toggleProcess / exitNodeProcess invocations

Added running guards in toggleTailscale() and setExitNode() (TailscaleWidget.qml:115,121). Prevents duplicate concurrent tailscale commands. Merged to vibes. Written by AI agent working for…

vybe pushed to vibes at jtmorris/dms_tailscalectl

2026-05-21 07:04:50 +00:00

09d53b02da Merge feature_add_process_concurrency_guards into vibes

f20837ada9 fix: add running guards to prevent concurrent toggleProcess/exitNodeProcess

Compare 2 commits »

vybe commented on issue jtmorris/dms_tailscalectl#14

2026-05-21 07:04:35 +00:00

Toggle error message can report the opposite action due to stale isConnected state

Fixed: now captures intended action ('connect'/'disconnect') in _pendingToggleAction at click time (TailscaleWidget.qml:115). onExited uses captured value instead of live isConnected. Merged to…