Skip to content

Emit quest completion SSE events#427

Open
Vinaystwt wants to merge 1 commit into
Bitcoindefi:mainfrom
Vinaystwt:vinay/quest-completion-sse-218
Open

Emit quest completion SSE events#427
Vinaystwt wants to merge 1 commit into
Bitcoindefi:mainfrom
Vinaystwt:vinay/quest-completion-sse-218

Conversation

@Vinaystwt

Copy link
Copy Markdown
Contributor

Summary

Adds quest completion events to the existing system event/SSE/activity feed pipeline.

Changes:

  • emit quest.completed after the first successful quest claim
  • keep replay/idempotent claims from emitting duplicate events
  • include agent ID, quest ID/title, and reward details in the event payload
  • expose quest completion events through /api/events and activity feed SSE
  • normalize quest completion into the activity feed
  • show a frontend success toast through the existing hub event listener

Verification

  • focused quest/SSE/feed tests passed
  • broader quest/SSE/notification tests passed
  • full npm test passed
  • focused ESLint passed
  • git diff --check passed

Typecheck is currently blocked by an existing repo dependency issue:

app/api/agents/[id]/capabilities/compat/route.ts(3,27): error TS7016:
Could not find a declaration file for module 'semver'.

@sonarqubecloud

sonarqubecloud Bot commented Jul 1, 2026

Copy link
Copy Markdown

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