feat(replays): Add Breadcrumb AI Summaries #93256
Conversation
github-actions
bot
added
the
Scope: Backend
Codecov ReportAttention: Patch coverage is ✅ All tests successful. No failed tests found.
Additional details and impacted files@@ Coverage Diff @@
## master #93256 +/- ##
===========================================
+ Coverage 61.52% 88.02% +26.50%
===========================================
Files 10298 10279 -19
Lines 593920 593696 -224
Branches 23101 22963 -138
===========================================
+ Hits 365389 522587 +157198
+ Misses 228082 70619 -157463
- Partials 449 490 +41 |
| | ------------------------ | --------------- | --------------------------------------------------------------------------------------------- | | ||
| | title | str | The main title of the user journey summary. | | ||
| | summary | str | A concise summary featuring the highlights of the user's journey while using the application. | | ||
| | time_ranges | list[TimeRange] | A list of TimeRange objects. | |
There was a problem hiding this comment.
| | time_ranges | list[TimeRange] | A list of TimeRange objects. | | |
| | time_ranges | list[TimeRange] | An ordered list of TimeRange objects. | |
| "period_end": 1749584992.912, | ||
| "period_title": "Second Replay Load Failure" | ||
| } | ||
| ] |
There was a problem hiding this comment.
Could add a second example to show the ordering and relationship of start/end (will all ranges be disjoint? no gaps in between?)
| return response.content | ||
|
|
||
|
|
||
| def get_request_data(iterator: Iterator[tuple[int, memoryview]]) -> list[str]: |
There was a problem hiding this comment.
| def get_request_data(iterator: Iterator[tuple[int, memoryview]]) -> list[str]: | |
| def get_request_data(segment_iterator: Iterator[tuple[int, memoryview]]) -> list[str]: |
src/sentry/replays/endpoints/project_replay_summarize_breadcrumbs.py
Outdated
Show resolved
Hide resolved
@aliu39 if you look at |
…mbs.py Co-authored-by: Andrew Liu <[email protected]>
| }, | ||
| ) | ||
| if response.status_code != 200: | ||
| raise ParseError("A non 200 HTTP status code was returned.") |
There was a problem hiding this comment.
could we make this error more detailed? when i was getting errors testing in devserver it was tough to figure out that it was coming from here, and what error was happening
There was a problem hiding this comment.
(it was a google cloud auth error but i didn't know until i looked at my seer terminal)
There was a problem hiding this comment.
+1, think it'd be better to raise a ValueError or Exception so the code is 500 instead of 400, since it's not a user triggered error right? And the error message can contain the specific Seer response code
Suspect IssuesThis pull request was deployed and Sentry observed the following issues:
Did you find this useful? React with a 👍 or 👎 |
Adds support for transforming RRWeb events into log messages and submitting them to Seer for AI analysis. Additionally exposes endpoints for querying and a blueprint for building clients against. --------- Co-authored-by: Andrew Liu <[email protected]>
Adds support for transforming RRWeb events into log messages and submitting them to Seer for AI analysis. Additionally exposes endpoints for querying and a blueprint for building clients against. --------- Co-authored-by: Andrew Liu <[email protected]>
Adds support for transforming RRWeb events into log messages and submitting them to Seer for AI analysis. Additionally exposes endpoints for querying and a blueprint for building clients against.