Skip to content

feat(auth): add wallet auth telemetry events#448

Merged
xeeenab merged 1 commit into
InChordSync:mainfrom
sorinc03:codex/wallet-auth-telemetry-424
May 31, 2026
Merged

feat(auth): add wallet auth telemetry events#448
xeeenab merged 1 commit into
InChordSync:mainfrom
sorinc03:codex/wallet-auth-telemetry-424

Conversation

@sorinc03
Copy link
Copy Markdown
Contributor

Summary

  • Define shared wallet-link and wallet-auth telemetry event names, outcomes, boundaries, and payload shape in @chordially/types
  • Emit sanitized server-side events from API login/logout boundaries and Stellar starter-intent boundary
  • Add focused tests proving success, failure, revocation, and starter-intent events exclude emails, passwords, tokens, user-agent strings, and signing artifacts

Testing

  • pnpm --filter @chordially/api exec tsx --test tests/wallet-auth-telemetry-424.test.ts
  • pnpm --filter @chordially/stellar-service exec tsx --test tests/wallet-auth-telemetry-424.test.ts
  • pnpm --filter @chordially/types typecheck
  • pnpm --filter @chordially/api typecheck
  • pnpm --filter @chordially/stellar-service typecheck
  • pnpm --filter @chordially/api lint (passes with existing warnings)
  • pnpm --filter @chordially/stellar-service lint
  • pnpm --filter @chordially/types lint
  • git diff --check

Note: full pnpm --filter @chordially/api test currently has 4 pre-existing lockout expectation failures in auth-330-lockout.test.ts around ACCOUNT_LOCKED / LOGIN_FAILED; the new telemetry tests pass.

Fixes #424

@xeeenab xeeenab merged commit 8cda7e7 into InChordSync:main May 31, 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.

Create shared telemetry events for wallet-link and wallet-auth milestones

2 participants