Skip to content

feat: add breadcrumb support for crash reporting#307

Draft
t-jimbo wants to merge 9 commits intomainfrom
feat/breadcrumb-support
Draft

feat: add breadcrumb support for crash reporting#307
t-jimbo wants to merge 9 commits intomainfrom
feat/breadcrumb-support

Conversation

@t-jimbo
Copy link
Collaborator

@t-jimbo t-jimbo commented Jan 6, 2026

Summary

  • Add Breadcrumb struct with message, category, level, data, timestamp
  • Add recordBreadcrumb() and getBreadcrumbs() to TrackRepository2 protocol
  • Implement breadcrumb buffering (max 50 entries) in TrackRespositoryImpl
  • Include breadcrumbs when sending Flutter crash reports via sendFlutterCrash()
  • Expose recordBreadcrumb() via @_spi(FlutterBridge) in NubrickExperiment

Usage (from Flutter Bridge)

@_spi(FlutterBridge)
import Nubrick

let breadcrumb = Breadcrumb(
    message: "Button tapped",
    category: "ui",
    level: "info",
    data: ["button_id": "submit"],
    timestamp: Int64(Date().timeIntervalSince1970 * 1000)
)
nubrickClient.experiment.recordBreadcrumb(breadcrumb)

Related PRs

Test plan

  • Verify iOS SDK compiles without errors
  • Test recordBreadcrumb() stores breadcrumbs correctly
  • Verify breadcrumbs are included in crash reports
  • Test breadcrumb buffer limits to 50 entries

🤖 Generated with Claude Code

@t-jimbo @claude
feat: add breadcrumb support for crash reporting
ee61e23 
- Add Breadcrumb struct with message, category, level, data, timestamp
- Add recordBreadcrumb and getBreadcrumbs to TrackRepository2 protocol
- Implement breadcrumb buffering (max 50 entries) in TrackRespositoryImpl
- Include breadcrumbs when sending Flutter crash reports
- Add recordBreadcrumb to Container protocol and implementations
- Expose recordBreadcrumb via @_spi(FlutterBridge) in NubrickExperiment

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@t-jimbo t-jimbo mentioned this pull request Jan 6, 2026
Draft
4 tasks
t-jimbo and others added 8 commits January 6, 2026 17:15
@t-jimbo @claude
fix: add BreadcrumbCategory and BreadcrumbLevel enums for type safety
8662bfc 
- Add BreadcrumbCategory enum (navigation, ui, http, console, custom)
- Add BreadcrumbLevel enum (debug, info, warning, error, fatal)
- Update Breadcrumb struct to use enums instead of strings
- Add Flutter Bridge compatible recordBreadcrumb overload for string-based data

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@t-jimbo @claude
docs: add detailed comments for Flutter Bridge recordBreadcrumb method
6d538d3 
Explain expected data structure from Flutter's method channel with inline comments for each type cast.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@t-jimbo @claude
feat: persist breadcrumbs for MetricKit crash reports
0025c60 
- Change Breadcrumb, BreadcrumbCategory, BreadcrumbLevel to Codable
- Persist breadcrumbs to UserDefaults on each record
- Load persisted breadcrumbs in processMetricKitCrash
- Clear persisted breadcrumbs after loading (one-time use)

This enables breadcrumbs to be included in native iOS crash reports
delivered by MetricKit in the next app session.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@t-jimbo @claude
refactor: make breadcrumb API public for iOS SDK users
9ba746a 
- Remove @_spi(FlutterBridge) from BreadcrumbCategory, BreadcrumbLevel, Breadcrumb
- Remove Flutter Bridge recordBreadcrumb(Dictionary) overload
- Keep only typed recordBreadcrumb(Breadcrumb) for iOS SDK users

Flutter Bridge support will be added in a separate PR.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@t-jimbo @claude
docs: add comments explaining breadcrumb retrieval strategy
b852d54 
- MetricKit crashes: use persisted breadcrumbs (previous session)
- Flutter crashes: use in-memory breadcrumbs (same session, immediate send)

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@t-jimbo
fix
10df73b
@t-jimbo @claude
feat: record breadcrumb when experiment is displayed
8e21acb 
- Add default timestamp to Breadcrumb initializer
- Record navigation breadcrumb in trackExperimentEvent
- Add doc comment explaining trackExperimentEvent purpose

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@t-jimbo @claude
feat: add breadcrumb recording for page navigation
484d523 
Record breadcrumbs when navigating between pages within experiments:
- Modal page navigation (with from/to page IDs)
- Embedded page navigation (with from/to page IDs)
- Tooltip page navigation (with from/to page IDs)
- WebView modal page navigation (with from/to page IDs)
- Modal/Embedded dismissed (with from page ID)

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
t-jimbo
t-jimbo commented Jan 9, 2026
View reviewed changes
Copy link
Collaborator Author

@t-jimbo t-jimbo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • currentPageのreset
  • .navigation or .ui
  • flutterとiosのbreadcrumbsの使い分け
  • 非同期処理にする？

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

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@t-jimbo