fix(sell): keep multi-currency --accept whole (disable cli/v3 comma split)

[bussyjd]

Summary

Fixes #664. Multi-currency --accept offers could not be created from the CLI — every attempt failed with network is required even when network= was present.

--accept is a cli.StringSliceFlag, and cli/v3 splits slice-flag values on , by default. One --accept is meant to be a single comma-separated key=value list (token=X,network=Y,price=Z), and parseAcceptKV already splits on , itself — so cli/v3 shredded each option into fragments first:

--accept token=USDC,network=base,price=0.03  →  ["token=USDC", "network=base", "price=0.03"]

buildAcceptPayments then read the first fragment (token=USDC) as a whole option with no network → network is required. The grouping is lost in the split, so it must be prevented up front.

Fix

Set DisableSliceFlagSeparator: true on the three commands that carry --accept (sell http, sell agent, sell update). Each --accept now arrives whole; parseAcceptKV keeps doing the intra-option , split.

The repeatable --register-skills / --register-domains / --register-metadata flags on those commands are unaffected in practice: each value is a single token or key=value pair, and nothing in the repo passes them comma-joined (verified with a repo-wide grep). Repeating the flag still works exactly as before.

Test

accept_test.go's existing tests call parseAcceptOption / buildAcceptPayments directly and bypass cli/v3 argv parsing, which is why they never caught this. Added TestSellAccept_CommaSeparatorDisabled:

• Structural guard — asserts all three commands keep DisableSliceFlagSeparator: true.
• Behavioral — drives real cli/v3 argv parsing: with the separator disabled the two --accept options arrive whole and build two payments; the negative control (default separator) shreds them into 6 fragments and fails to build, proving the field is load-bearing.

go test ./cmd/obol/   # ok
go vet  ./cmd/obol/   # clean

Manual verification

obol sell http ... --accept token=USDC,network=base,price=0.03 --accept token=OBOL,network=ethereum,price=0.0001 now produces a ServiceOffer with both spec.payments[] entries directly — no kubectl patch workaround.