Skip to content

Improve test coverage for the summarizer module #106

Open

Description

@sepehr-safari

Good First Issue

The summarizer module (packages/toolkit/src/core/summarizer.ts) has only 5 tests. Improve coverage by adding edge case tests.

What to add

  • Test with empty sessions array
  • Test with session having null timestamps
  • Test with session having very large duration
  • Test with multiple sessions (different transaction IDs)
  • Test with session having zero events
  • Test summarizeSession() directly with various inputs

Files to modify

  • packages/toolkit/src/core/summarizer.test.ts

How to claim

Comment "I'd like to work on this" and it will be assigned to you.

Please hold one open claim at a time. Once the pull request for it is merged,
say which issue you want next and it will be assigned. That keeps issues from
sitting claimed while another is still in review, so other newcomers can see
what is genuinely free.

Activity

  1. YANGCHUNHONG3000 commented on Jul 26, 2026

    @YANGCHUNHONG3000
    Contributor

    I'd like to work on this test coverage improvement for summarizer.

  2. sepehr-safari commented on Jul 26, 2026

    @sepehr-safari
    MemberAuthor

    Thanks for the interest. I am assigning these one at a time, and #104 is with you
    now, so I am leaving this one open for the moment. Once #133 merges, say the word
    and it is yours.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions