Skip to content

fix: block mutating actions on locked servers client-side#81

Merged
larkly merged 1 commit intomainfrom
worktree-quizzical-rolling-wadler
Mar 29, 2026
Merged

fix: block mutating actions on locked servers client-side#81
larkly merged 1 commit intomainfrom
worktree-quizzical-rolling-wadler

Conversation

@larkly
Copy link
Copy Markdown
Owner

@larkly larkly commented Mar 29, 2026

Summary

  • Pre-check server lock status before dispatching any mutating action (delete, reboot, resize, rename, etc.)
  • Show a clear sticky hint ("Server is locked. Unlock it first with Ctrl+L.") instead of letting the API reject with a confusing error
  • Read-only actions (console log, console URL, action history, copy SSH) and lock/unlock itself remain unguarded

Closes #55

Test plan

  • Select a locked server, press any mutating key (e.g., Ctrl+D for delete) — verify sticky hint appears and no confirmation modal opens
  • Select a locked server, press Ctrl+L — verify lock/unlock toggle still works
  • Select a locked server, press read-only keys (L, a, V, y) — verify they still work
  • Bulk-select servers including a locked one, press a mutating key — verify it's blocked
  • Select an unlocked server, press mutating keys — verify normal behavior unchanged

Pre-check lock status before dispatching any server action and show
a clear hint instead of letting the API reject with a confusing error.

Closes #55
@larkly larkly merged commit beba823 into main Mar 29, 2026
@larkly larkly deleted the worktree-quizzical-rolling-wadler branch April 17, 2026 14:57
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.

Block actions on locked servers client-side

1 participant