feat(client): scope the securities trade counterparty picker to securities

[gilgardosh]

Final step of the per-security businesses plan. Stacked on #4257.

Closes the loop on the screenshot that started this: on a foreign-securities charge, the main
transaction's counterparty should be the specific security, the fee row stays Poalim, and the
ledger is untouched.

When the main transaction has no counterparty yet, its picker now lists the tenant's security
businesses plus the general foreign-securities business (the fallback for a trade whose security
cannot be told) instead of the whole business directory — where the right answer is a needle in a
haystack and a wrong one is one click away. Each option carries its ISIN, which is what tells two
share classes of one issuer apart. The fee row keeps the full list, matching the server's suggestion
rule (#4255).

The rule is decided client-side from what the charge already knows, so chargeType is threaded from
charge-extended-info → ChargeTransactionsTable → TransactionsTable onto the row, exactly like
enableEdit/enableChargeLink. The other TransactionsTable callers (business transactions tab,
balance report) pass no charge type and are unaffected.

The suggestion itself needed no client change — the cell already pre-seeds the select from
missingInfoSuggestions, which now resolves the security named in the description.

Adds useGetSecurityBusinesses (the securities-scoped counterpart of useGetAdminBusinesses) and
surfaces UserContext.foreignSecuritiesBusinessId through the user provider for the fallback option.

Verification

• yarn generate, yarn lint, yarn prettier clean; client typechecks and yarn build succeeds.
• yarn test — 3439 passed.

🤖 Generated with Claude Code

[Copilot]

Pull request overview

This PR narrows the counterparty picker for the main transaction row of a ForeignSecuritiesCharge so that users choose from security businesses (plus a general fallback business) instead of the full business directory, while keeping the fee row behavior unchanged.

Changes:

• Thread chargeType from ChargeExtendedInfo → ChargeTransactionsTable → TransactionsTable → row cells to enable charge-type-specific UI behavior.
• Add useGetSecurityBusinesses and update the counterparty cell to use security-scoped options (with ISIN in labels) plus UserContext.foreignSecuritiesBusinessId as a fallback.
• Extend UserContext and update client tests/changeset to reflect the new context field and behavior.

Reviewed changes

Copilot reviewed 9 out of 9 changed files in this pull request and generated 5 comments.

Show a summary per file

File	Description
packages/client/src/providers/user-provider.tsx	Exposes foreignSecuritiesBusinessId via the user context query + mapping.
packages/client/src/hooks/use-get-security-businesses.ts	Adds a new hook to fetch and format selectable security businesses (with ISIN).
packages/client/src/components/transactions-table/index.tsx	Adds optional chargeType prop and threads it into row data.
packages/client/src/components/transactions-table/columns.tsx	Extends row type to include chargeType for use in cells.
packages/client/src/components/transactions-table/cells/counterparty.tsx	Scopes counterparty picker options to securities for foreign-securities main rows, with fallback.
packages/client/src/components/charges/charge-transactions-table.tsx	Passes chargeType down into TransactionsTable.
packages/client/src/components/charges/charge-extended-info.tsx	Provides chargeType into the charge transactions table when rendered in charge view.
packages/client/src/components/tests/user-menu.test.tsx	Updates test fixture to include foreignSecuritiesBusinessId.
.changeset/security-counterparty-picker.md	Documents the client-facing behavior change for release notes.

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.