Skip to content

principal names ride every prompt — <@id> (name) from the adapter's roster - #13

Merged
Octember merged 1 commit into
mainfrom
claude/principal-names
Jul 31, 2026
Merged

principal names ride every prompt — <@id> (name) from the adapter's roster#13
Octember merged 1 commit into
mainfrom
claude/principal-names

Conversation

@Octember

Copy link
Copy Markdown
Owner

Follow-up to #11 (the blind-ear incident): retire the bare-id failure class entirely.

agent-tools 0.5.0 (c5f00fc) resolves user ids to display names at ingestion: a users.list roster prewarm at start, a synchronous cache lookup at dispatch (the ack-after-handling contract stays intact), and one background users.info per unknown id so the person's next message arrives named. Best-effort — without users:read everything ships bare, exactly as before.

This PR threads it through earshot: the router persists principalName in the event payload, and one who() renderer makes inbox delivery, ear batch lines, and the ear's thread tails all read <@U096GG0F354> (noah): …. Old events without a name render exactly as before.

⚠️ bun install from scratch needs agent-tools 0.5.0 on npm — publish pending operator OTP. Deploy goes via the usual scp hotfix path meanwhile.

bun run check green — 384 pass.

🤖 Generated with Claude Code

…oster

agent-tools 0.5.0 resolves user ids to display names at ingestion
(users.list prewarm + lazy users.info). The router persists the name in
the event payload, inbox delivery and the ear's thread tails render
who() = <@id> (name), so neither mind nor ear ever judges who-is-
talking-to-whom from a bare id again (2026-07-30: the ear attributed a
teammate's reply to her and recorded his question as her debt).

Events from before names existed render exactly as before — the name is
optional everywhere. NOTE: fresh bun install needs @bevyl-ai/agent-tools
0.5.0 on npm (publish pending operator OTP); the dep bump is committed
ahead of it.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@Octember
Octember merged commit e5bd83c into main Jul 31, 2026
1 check failed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant