Skip to content

feat: add sourceType to event tracking - #27

Merged
gunoooo merged 2 commits into
mainfrom
add-source-type
Feb 13, 2026
Merged

gunoooo merged 2 commits into
mainfrom
add-source-type

Conversation

@gunoooo

@gunoooo gunoooo commented Feb 13, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • SESSION_START, PUSH_NOTIFICATION_RECEIVED, PUSH_NOTIFICATION_TAPPED 이벤트에 sourceType: "CLIX" 추가
  • EventService, EventAPIService에 sourceType 파라미터 추가 (기본값 undefined, 브레이킹 체인지 없음)

Summary by CodeRabbit

  • New Features

    • Event tracking now accepts an optional source type to record event origins.
    • Push notification and session events now include a CLIX source attribution for clearer source identification.
  • Chores

    • Package version bumped (patch release).

@coderabbitai

coderabbitai Bot commented Feb 13, 2026 •

Copy link
Copy Markdown
Contributor

Walkthrough

Adds an optional sourceType parameter to event tracking. EventAPIService.trackEvent and EventService.trackEvent signatures now accept sourceType?: string; several call sites (NotificationService, SessionService) were updated to pass 'CLIX'. Package version incremented to 1.4.1.

Changes

Cohort / File(s) Summary
Event API & Service
src/services/EventAPIService.ts, src/services/EventService.ts
Added optional sourceType?: string to trackEvent signatures and propagated the parameter from EventService to EventAPIService (emitted payload includes source_type when provided).
Event Tracking Call Sites
src/services/NotificationService.ts, src/services/SessionService.ts
Updated trackEvent invocations to include an extra sourceType argument ('CLIX'), adjusting call arity where these events are emitted.
Package Manifest
package.json
Bumped package version from 1.4.0 to 1.4.1.

Possibly related PRs

  • feat: add session tracking #25: Touches the same event-tracking flows and call sites (NotificationService, SessionService), indicating a strong code-level relationship.

Suggested reviewers

  • pitzcarraldo
  • nyanxyz
🚥 Pre-merge checks | ✅ 4
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'feat: add sourceType to event tracking' accurately and concisely summarizes the main change of adding a sourceType parameter to event tracking services.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Merge Conflict Detection ✅ Passed ✅ No merge conflicts detected when merging into main

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch add-source-type

No actionable comments were generated in the recent review. 🎉


Comment @coderabbitai help to get the list of available commands and usage tips.

@gunoooo gunoooo changed the title Add sourceType to event tracking feat: add sourceType to event tracking Feb 13, 2026
@gunoooo
gunoooo merged commit 1ea5062 into main Feb 13, 2026
3 checks passed
@gunoooo
gunoooo deleted the add-source-type branch February 13, 2026 08:34
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