fix(frontend): return early from in-flight actions so a double click sends one request #1

Merged

thisilike merged 1 commit from fix/double-submit into main

2026-09-23 15:42:56 +02:00

thisilike commented

2026-09-23 15:42:22 +02:00

Owner

Every panel set busy and relied on the disabled button, which lands a frame after a fast double click.

discard-latest pops the newest edit on each call, so a double click discarded two edits.
A double Enter in the command bar sent the console command twice.

The fix adds an early return to every panel, the command bar, player actions and the directory picker.

NewServerPage needed more than a guard: choose() re-entered itself through loadPackVersions(), so a plain guard would have skipped the inner call. The pack resolve is now a separate, unguarded function, and the Use button keeps busy on across both steps.

Checks: 400 tests pass, svelte-check has 0 errors, and deno fmt --check is clean.

🤖 Generated with Claude Code

Every panel set `busy` and relied on the disabled button, which lands a frame after a fast double click. - `discard-latest` pops the newest edit on each call, so a double click discarded two edits. - A double Enter in the command bar sent the console command twice. The fix adds an early return to every panel, the command bar, player actions and the directory picker. NewServerPage needed more than a guard: `choose()` re-entered itself through `loadPackVersions()`, so a plain guard would have skipped the inner call. The pack resolve is now a separate, unguarded function, and the Use button keeps `busy` on across both steps. Checks: 400 tests pass, svelte-check has 0 errors, and `deno fmt --check` is clean. 🤖 Generated with [Claude Code](https://claude.com/claude-code)

thisilike added 1 commit

2026-09-23 15:42:22 +02:00

fix(frontend): return early from in-flight actions so a double click sends one request 5377171ff2

Every panel set busy and relied on the disabled button, which lands a
frame after a fast double click. discard-latest pops the newest edit per
call, so a double click discarded two; a double Enter in the command bar
sent the console command twice.

NewServerPage's choose() re-entered itself through loadPackVersions(), so
the pack resolve is split out unguarded and the Use button keeps busy
across both steps.

thisilike merged commit bb60b55f4a into main

2026-09-23 15:42:56 +02:00

thisilike deleted branch fix/double-submit

2026-09-23 15:42:56 +02:00

thisilike referenced this pull request from a commit

2026-09-23 15:42:56 +02:00

Merge pull request 'fix(frontend): return early from in-flight actions so a double click sends one request' (#1) from fix/double-submit into main