Skip to content

fix(addie): route engagement ranking queries to admin tool set#1309

Merged
bokelley merged 1 commit intomainfrom
bokelley/addie-engagement-tools
Mar 2, 2026
Merged

fix(addie): route engagement ranking queries to admin tool set#1309
bokelley merged 1 commit intomainfrom
bokelley/addie-engagement-tools

Conversation

@bokelley
Copy link
Copy Markdown
Contributor

@bokelley bokelley commented Mar 2, 2026

Summary

  • Addie wasn't calling list_users_by_engagement because the router had no rule directing engagement/analytics questions to the admin tool set
  • The only admin triggers were escalations, org merges, and similar operations
  • Added an explicit routing rule for community-wide engagement ranking questions
  • Expanded the admin tool set description to include engagement analytics so both signals align

Changes

  • server/src/addie/router.ts: Adds routing rule — "Community-wide engagement ranking, most engaged members overall, top contributors, who to invite to events, lifecycle stage analytics → ['admin']"
  • server/src/addie/tool-sets.ts: Updates admin tool set description to mention "member insights and engagement analytics, community-wide engagement ranking"

Test plan

  • Ask Addie (as an admin) "Who are our most engaged members?" — should now call list_users_by_engagement
  • Ask Addie "Who should we invite to the upcoming summit?" — should route to admin and call list_users_by_engagement
  • Ask Addie "How engaged is [specific company]?" — should still route to get_account (not affected by this change)
  • All existing tests pass (pre-commit hook verifies)

🤖 Generated with Claude Code

Addie wasn't calling list_users_by_engagement because her router had no
rule pointing engagement/analytics questions to the admin tool set. The
only admin triggers were escalations, org merges, and similar operations.

Adds an explicit routing rule for community-wide engagement ranking and
expands the admin tool set description to mention engagement analytics,
so both the routing prompt and the tool set context window align.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@bokelley bokelley merged commit fa96a92 into main Mar 2, 2026
6 checks passed
@aao-release-bot aao-release-bot Bot mentioned this pull request May 3, 2026
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