Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: add sentry support #644

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Conversation

agallardol
Copy link
Collaborator

@agallardol agallardol commented Feb 7, 2025

  • logs file is always stored under the OS logs directory and named shinkai-logs.logs
  • app logs sent to 3 targets:
    • Logs file
    • Std
    • Webview Console
  • Added sentry enabled by default in prod
    • Crash native crashes and errors are always sent to sentry
    • If users opt-out/opt-in analytics we don't send browser logs to sentry
  • App error logs are sent to sentry (currently we don't have too much log:error but from now they could be useful)

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