Agent batch system - #453
Conversation
|
Hi @Jumongweb 👋 — two things before this can be reviewed/merged:
Happy to merge once both are green. 🙏 |
|
Hi @Jumongweb 👋 — quick update: video/screenshots are not required to merge anymore, just a bonus — if you add one it'll be considered for GrantFox rewards on this issue. So I'll review this on code + checks alone now. 🙏 |
|
I have added the screenshot. |
|
Please merge |
Needs a rebase — and that is our fault, not yoursWe just merged a batch of PRs that had been waiting far too long for review. Yours was in that batch and was green, but the merges ahead of it touched overlapping files, so this branch now conflicts with To get it in: git fetch origin
git rebase origin/main
# resolve conflicts, then
git push --force-with-leaseOnce CI goes green again, comment here and it gets merged — no second review round needed for the parts that were already approved. Sorry for the churn. The delay that caused this was on our side: this PR sat for weeks while other work landed on top of it. |
|



Summary
Closes #223
Completes the agent badge system by wiring badge awards into the live XP, quest, and task-completion flows, persisting earned badges per agent, and rendering badges on the agent detail page with hover tooltips.
What changed
lib/agents/badges.tsfirst_taskquest_masterlevel_10veterantop_earner.data/badges/[agentId].jsonlib/gamification/quests.tsrecordTaskCompletion()inlib/reputation/reputation-store.tslib/agent-runtime/agent.tsapp/agent-functions/[id]/route.tsapp/api/agents/[id]/tasks/drain/route.tsfirst_taskcondition so it depends on actual completed tasks rather than any XP gain0completed tasksGET /api/agents/[id]/badgesas:{ badges: Badge[], count: number }Testing
Added/updated tests covering:
first_taskawarding on task completionquest_masterawarding after the fifth completed questfirst_taskfrom being awarded by quest XP aloneProof
This issue requires visual proof for review.
Please attach a short demo video or screenshots showing:
/api/agents/[id]/badges