mirror of
https://github.com/block/buzz.git
synced 2026-08-18 06:50:31 +02:00
npub1qyvc0c5kl4gqv2fd97fsk46tu378sqgy35vc83rvgfwne90sel7s0ed67dandTyler Longwell
1e6bc6d584
Addresses Wren's review of the card-mint key UX (buzz-agent-trading-cards): 1. Cost disclosure + free alternative now persist in the normal mint form (pre-spend), not only the no-key setup panel: a compact cost note above the Mint action plus a shared FreeSharePathRow. 2. Both directions are actionable: 'Get a key at platform.openai.com' is a real external-link button (openUrl), and the free path is a 'Share without card art' action that closes the mint dialog and opens the existing snapshot export flow (wired at the UserProfilePanel callsite; optional prop so the dialog degrades gracefully elsewhere). 3. Key save no longer restarts running agents: replaced the general set_global_agent_config call (which stops/restarts every running local agent whose effective env changes, and whose restart counts the dialog discarded) with a narrow card_mint_save_openai_key command — validated single-key read-modify-write of the latest on-disk config under the managed-agents store lock, no restarts. The mint re-reads config per call, so no restart is needed; the toast says running agents pick the key up on next restart, and the Agent Defaults query cache is invalidated so the settings editor stays fresh. 4. Azure claim narrowed in the responses_url doc comment: OPENAI_BASE_URL covers Bearer-auth Responses-speaking endpoints/proxies; Azure's URL scheme + api-key header would need a real driver. 5. Owner-copy rule acknowledges the 220-character text-box contract: over-limit owner text is condensed minimally, keeping words and intent — no mutually impossible verbatim-vs-limit instructions. Prompt-contract test updated. Co-authored-by: Tyler Longwell <tlongwell@block.xyz> Signed-off-by: Tyler Longwell <tlongwell@block.xyz>