* turn on a couple of type-aware lint rules

* turn on recommended-type-checked except for all the ones I hate

* hm we actually sort of caught a bug and I'm not sure why eslint made this possible

* turn on react/button-has-type lint rule and fix all

* add MiniTable.RemoveCell to reduce fiddly logic

* Add step to attach floating IPs to instance creation form

* Slight refactoring of const names

* Smoother handling of unchecking Floating IP box

* Add noItemsPlaceholder to Listbox

* add type guard to clean up code

* Remove checkbox; button-only to open modal

* only show message when IPs are available to attach

* Simplify floating IP label, sans-pool

* revert removal of api call

* Add header to new file

* Merge main; disable modal trigger button if no floating IPs available

* Add empty state for when no floatingIPs exist

* Update app/forms/instance-create.tsx

Co-authored-by: David Crespo <[email protected]>

* Refactor deeply-nested JSX

* Extract FloatingIpLabel, update TipIcon copy

* Extract Listbox labels

* Revert "Extract Listbox labels"

This reverts commit 59b27d1.

* Refactor to use useState instead of weird form entry for state management

* small refactor on onAction

* another small refactor

* refactor to use FloatingIp in useState, rather than a string

* simplify and future-proof logic

Co-authored-by: David Crespo <[email protected]>

* Add test for attaching a floating IP to instance

* include assertion regarding IP address in modal message

* Add test for empty floating IPs state

* Update mock serviceworker to attach IPs to instance; update tests

* Move ipFromPool to util function

* refactor

* fix the button!

* use RemoveCell in floating IPs MiniTable

* Add test to remove row from form

---------

Co-authored-by: David Crespo <[email protected]>
Co-authored-by: David Crespo <[email protected]>

* replace implementations of array utils with remeda calls to prove identical

* get rid of mapValues, lowestBy, sumBy, isTruthy (going away in TS 5.5 anyway)

* get rid of pick

* get rid of sortBy and splitOnceBy

upgrade xterm packages to new @xterm namespace version

* the event is 'close', not 'closed'

* plumb instance state into serial console skeleton

* make a dent in a more robust can't connect message

* handle error case in mediocre fashion, aria-label on scroll buttons