admin chat: render accept_pay / decline_pay / counter_pay tool chips#186
Merged
Merged
Conversation
Companion to salem-engine ZBBS-127 (held-transaction LLM tick on pay). The engine's pay-deliberation path emits three new tools as part of the recipient's deliberation reply: - accept_pay — no args, the tool itself is the meaning - decline_pay(reason) — recipient's spoken refusal - counter_pay(new_amount, message) — recipient's spoken counter These already write to chat_message_texts via the deliberation's chat/send call, but ToolCallDisplay's v-else fallback rendered them as bare [accept_pay] / [decline_pay] / [counter_pay] chips with the input dropped — so the admin couldn't see what the recipient said without clicking into the detail dialog and reading raw JSON. Render shapes mirror the existing pay tool conventions: [accept_pay] [decline_pay] "I'd rather not sell at that price." [counter_pay] 5c — "Make it five and we have a deal." The reason / message strings are also broadcast as synthetic npc_spoke events by the engine, so they appear in the room's speech bubbles too — the admin chip just lets reviewers correlate the deliberation row with the speech bubble at a glance. — Home Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Companion to salem-engine ZBBS-127. Adds ToolCallDisplay rendering for the three new pay-deliberation tools so the admin chat log shows the recipient's spoken decline / counter inline, matching the existing pay / consume / attend_to conventions.
Render shapes
```
[accept_pay]
[decline_pay] "I'd rather not sell at that price."
[counter_pay] 5c — "Make it five and we have a deal."
```
The reason / message strings are also broadcast as synthetic `npc_spoke` events by the engine — these chips just give the admin a per-row anchor to correlate the deliberation chat row with the speech bubble in the talk log.
Test plan
— Home
🤖 Generated with Claude Code