registered? never returns nil, no need to handle it

Use `(reg || nil)&.phone` instead of `reg && reg.phone` to satisfy
Style/SafeNavigation while still handling false (not just nil).

Assisted-by: Claude Opus 4.5 via Crush

Display customer ID and contact information (Jabber ID, phone number)
when selecting actions to help admins identify which customer they're
working with.

Implements: https://todo.sr.ht/~singpolyma/soprani.ca/465
Assisted-by: MiniMax M2 via Crush <crush@charm.land>

We have a loader, let's use it

Probably we want to load list of command as part of the sgx repo? But
maybe that's too heavy and we don't always need it?

* '380-set-port-out-pin' of https://git.secluded.site/sgx-jmp:
  feat: add command to set port-out PIN

Currently, port-out PINs require manual intervention from support. This
command allows customers to self-manage their port-out PINs. Actual
functionality is delegated to the user's backend. For now, that's just
sgx-bwmsgsv2.

References: https://todo.sr.ht/~singpolyma/soprani.ca/380

- uses solargraph sourcemap, which requires a patch for Ruby 3.4
that is not available (patched solargraph version is incompatible
with pinned rubocop)

- conditionally checks for two different kinds of ivar nodes
using the same check that solargraph uses internally to decide between
current and legacy parsers

SIMAction mixin referenced @customer, but PleaseTopUp
didn't have such a field, which caused "undefined
method `jid’ for nil:NilClass" when checking customer onboarding

* 'fix/change-number-for-unregistered' of git.sr.ht:~phdavis1027/sgx-jmp:
  NumberChange can now handle unregistered cust

- old_tel is nil if customer unregistered
- test unregistered for blank num
  ^ involved fixing a race between check_noop and check_orphan

act on feedback

i apparently thought this was broken earlier, but it seems to just
work in test cases i have tried, so let's see if this version just does
what it should

for API call

gives us "step" and "next" commands

- gotta GRANT CREATE ON PUBLIC
- postgis
- redis_fdw
  + commands on setting up the foreign server and user mapping

To figure out why we're saving non-+1 stuff

* premium:
  Factor out BuyNumber
  Inject price
  Bill after charge
  Inline useless methods
  Remove unused method
  Fix test
  charge for premium nums
  refactor registration_test to use real Tn's
  two changes related to registration forms
  price functionality on Tns
  Remove legacy card charge behaviour
  two fixes related to eSIM nicks:
  Factor out reload customer, check balance, maybe bill
  Unify more invite code handling
  Remove mostly-unused final_message
  Remove legacy card charge behaviour

Since that's what we actually care about.

Also makes JustCharge more generic for future use

instead of this blind retry

And use a better transaction id

And move price method where it actually goes