H-5764: Setup Sentry UserFeedback + ErrorTracker #8241

kube · 2026-01-06T22:06:55Z

Demo

Kapture 2026-01-07 at 20.22.59.mp4 (uploaded via Graphite)

🌟 What is the purpose of this PR?

This PR adds Sentry User Feedback + Error Tracker inside Petrinaut demo website.

🔍 What does this change?

Error Tracker

Because we don't want Petrinaut to be directly coupled to Sentry:

Adds React Context is added ErrorTrackerContext
Adds ErrorTrackerContext Provider for Sentry <SentryErrorTrackerProvider> (only used in demo-website)

This SentryErrorTrackerProvider is still not used for now in the app, but will be extended later to add more granular tracking of user experience in the demo app.

User Feedback

For now the User Feedback widget is the default one, this keeps it simple and prevents having to deal with React component props, or with design.

If we get a good Figma design, we'll be able customize this feedback button to place it properly in the UI.

Pre-Merge Checklist 🚀

🚢 Has this modified a publishable library?

This PR:

does not modify any publishable blocks or libraries, or modifications do not need publishing

📜 Does this require a change to the docs?

The changes in this PR:

are internal and do not require a docs change

🕸️ Does this require a change to the Turbo Graph?

The changes in this PR:

do not affect the execution graph

⚠️ Known issues

Given the User Feedback widget is the default one, it's on top of the UI and could possibly hide some things.

🐾 Next steps

In the future, we'll do more granular tracking of what happens in the app.

❓ How to test this?

Use latest Petrinaut deployment

kube · 2026-01-06T22:07:25Z

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

This stack of pull requests is managed by Graphite. Learn more about stacking.

…ider to a new path and enhance error handling callbacks

…e; improve Sentry integration by injecting SENTRY_DSN directly into the application.

…bsite

This will need a better solution soon.

cursor · 2026-01-07T19:25:39Z

PR Summary

Introduces Sentry-based error tracking and feedback to the demo site with minimal coupling to the library.

Initialize Sentry in demo-site/sentry/instrument.ts with browser integrations and Feedback widget; enable only in production and wire DSN via import.meta.env.SENTRY_DSN
Wrap app in SentryErrorTrackerProvider implementing generic ErrorTrackerContext (src/error-tracker/error-tracker.context.ts); add React root error handlers (onUncaughtError, onCaughtError, onRecoverableError)
Vite: load .env, expose SENTRY_DSN at build time, add env typings, and include new sentry files in tsconfig
UX: keyboard shortcuts ignore Sentry feedback element to prevent conflicts
Add @sentry/react dependency and local .env with SENTRY_DSN

^{Written by Cursor Bugbot for commit 7a07628. This will update automatically on new commits. Configure here.}

graphite-app · 2026-01-07T19:29:52Z

libs/@hashintel/petrinaut/package.json

    "@hashintel/ds-helpers": "0.0.1-b",
    "@local/eslint": "0.0.0-private",
    "@pandacss/dev": "1.4.3",
+    "@sentry/react": "10.22.0",


Move '@sentry/react': '10.22.0' from devDependencies to dependencies since it's being used at runtime in the demo-site files. Dependencies used at runtime should not be in devDependencies.

Spotted by Graphite Agent (based on CI logs)

Is this helpful? React 👍 or 👎 to let us know.

augmentcode · 2026-01-07T19:30:05Z

🤖 Augment PR Summary

Summary: This PR wires Sentry error tracking + user feedback into the Petrinaut demo site.

Changes:

Initialize Sentry in the demo site with browser integrations, tracing, and the default Feedback widget.
Add a generic ErrorTrackerContext in the Petrinaut library and a Sentry-backed provider used by the demo site.
Wrap the demo app with SentryErrorTrackerProvider and register React root error handlers (onUncaughtError, onCaughtError, onRecoverableError).
Load SENTRY_DSN from .env via Vite and expose it at build time via import.meta.env.SENTRY_DSN typings.
Adjust editor keyboard shortcut handling to avoid capturing keystrokes while Sentry feedback UI is focused.
Add @sentry/react dependency for the Petrinaut package.

_{🤖 Was this summary useful? React with 👍 or 👎}

augmentcode

Review completed. 3 suggestions posted.

Comment augment review to trigger a new review at any time.

augmentcode · 2026-01-07T19:30:07Z

libs/@hashintel/petrinaut/demo-site/main.tsx

@@ -1,12 +1,30 @@
+import "./sentry/instrument.js";


./sentry/instrument.js doesn’t exist in this PR (only instrument.ts is added), so this import will likely fail at runtime/build time.

_{🤖 Was this useful? React with 👍 or 👎}

augmentcode · 2026-01-07T19:30:07Z

libs/@hashintel/petrinaut/demo-site/tsconfig.json

+    "./*.ts",
+    "./*.d.ts",
+    "main/app.tsx",
+    "sentry/init-sentry.ts"


This include entry points at sentry/init-sentry.ts, but the added Sentry initializer file is sentry/instrument.ts (so this looks like a stale/incorrect path).

_{🤖 Was this useful? React with 👍 or 👎}

augmentcode · 2026-01-07T19:30:07Z

libs/@hashintel/petrinaut/demo-site/sentry/sentry-error-tracker-provider.tsx

+    Sentry.withScope((scope) => {
+      if (context) {
+        for (const [key, value] of Object.entries(context)) {
+          scope.setContext(key, value as Record<string, unknown>);


scope.setContext() expects an object, but ErrorTrackerContextData allows any unknown value; passing primitives here can lead to dropped context or unexpected serialization.

_{🤖 Was this useful? React with 👍 or 👎}

cursor · 2026-01-07T19:31:33Z

libs/@hashintel/petrinaut/demo-site/tsconfig.json

+    "./*.ts",
+    "./*.d.ts",
+    "main/app.tsx",
+    "sentry/init-sentry.ts"


tsconfig.json references non-existent file name

Medium Severity

The tsconfig include array references sentry/init-sentry.ts but the actual file is named sentry/instrument.ts. This mismatch means the TypeScript configuration explicitly includes a non-existent file while not explicitly including the actual sentry files in the sentry/ directory. While the files may still be type-checked through imports from main.tsx, this configuration is incorrect and could cause issues with TypeScript project references or direct tsc invocations.

kube added 5 commits January 6, 2026 22:58

Add @sentry/react dependency

43b7cc0

Add ErrorTracker interface/context

c294c43

Simplify interface

8978810

Add SentryErrorTrackerProvider in demo-site

3e41f4f

Remove changeset

44589ec

kube mentioned this pull request Jan 6, 2026

H-5691: Simulation Timeline Visualizer #8211

Open

3 tasks

github-actions bot added area/deps Relates to third-party dependencies (area) area/infra Relates to version control, CI, CD or IaC (area) area/libs Relates to first-party libraries/crates/packages (area) type/eng > frontend Owned by the @frontend team labels Jan 6, 2026

kube changed the title ~~Add @sentry/react dependency~~ Setup Sentry UserFeedback + ErrorTracker Jan 6, 2026

vercel bot deployed to Preview – petrinaut January 6, 2026 22:12 View deployment

vercel bot deployed to Preview – hash January 6, 2026 22:20 View deployment

kube added 3 commits January 7, 2026 19:16

Refactor Sentry integration in demo-site: move SentryErrorTrackerProv…

3a130b0

…ider to a new path and enhance error handling callbacks

Load SENTRY_DSN from environment variables and expose it at build tim…

a71c4ee

…e; improve Sentry integration by injecting SENTRY_DSN directly into the application.

Add User Feedback

2b838e0

kube changed the title ~~Setup Sentry UserFeedback + ErrorTracker~~ H-5764: Setup Sentry UserFeedback + ErrorTracker Jan 7, 2026

vercel bot deployed to Preview – petrinaut January 7, 2026 19:01 View deployment

kube added 3 commits January 7, 2026 20:08

Move @sentry/react to devDependencies because only needed for demo we…

b5cc3f6

…bsite

Exclude Sentry Feedback from Keyboard Shortcuts.

d522574

This will need a better solution soon.

Only enable in production

7a07628

kube marked this pull request as ready for review January 7, 2026 19:25

graphite-app bot assigned kube Jan 7, 2026

graphite-app bot reviewed Jan 7, 2026

View reviewed changes

vercel bot deployed to Preview – petrinaut January 7, 2026 19:30 View deployment

augmentcode bot reviewed Jan 7, 2026

View reviewed changes

cursor bot reviewed Jan 7, 2026

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

H-5764: Setup Sentry UserFeedback + ErrorTracker #8241

H-5764: Setup Sentry UserFeedback + ErrorTracker #8241

Uh oh!

kube commented Jan 6, 2026 •

edited

Loading

Uh oh!

kube commented Jan 6, 2026

Uh oh!

cursor bot commented Jan 7, 2026 •

edited

Loading

Uh oh!

graphite-app bot Jan 7, 2026

Uh oh!

augmentcode bot commented Jan 7, 2026

Uh oh!

augmentcode bot left a comment

Uh oh!

augmentcode bot Jan 7, 2026

Uh oh!

augmentcode bot Jan 7, 2026

Uh oh!

augmentcode bot Jan 7, 2026

Uh oh!

cursor bot Jan 7, 2026

Uh oh!

Reviewers

Assignees

Labels

Milestone

Development

Uh oh!

2 participants

H-5764: Setup Sentry UserFeedback + ErrorTracker #8241

Are you sure you want to change the base?

H-5764: Setup Sentry UserFeedback + ErrorTracker #8241

Uh oh!

Conversation

kube commented Jan 6, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Demo

🌟 What is the purpose of this PR?

🔍 What does this change?

Error Tracker

User Feedback

Pre-Merge Checklist 🚀

🚢 Has this modified a publishable library?

📜 Does this require a change to the docs?

🕸️ Does this require a change to the Turbo Graph?

⚠️ Known issues

🐾 Next steps

❓ How to test this?

Uh oh!

kube commented Jan 6, 2026

Uh oh!

cursor bot commented Jan 7, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

PR Summary

Uh oh!

graphite-app bot Jan 7, 2026

Choose a reason for hiding this comment

Uh oh!

augmentcode bot commented Jan 7, 2026

Uh oh!

augmentcode bot left a comment

Choose a reason for hiding this comment

Uh oh!

augmentcode bot Jan 7, 2026

Choose a reason for hiding this comment

Uh oh!

augmentcode bot Jan 7, 2026

Choose a reason for hiding this comment

Uh oh!

augmentcode bot Jan 7, 2026

Choose a reason for hiding this comment

Uh oh!

cursor bot Jan 7, 2026

Choose a reason for hiding this comment

tsconfig.json references non-existent file name

Uh oh!

Reviewers

Assignees

Labels

Milestone

Development

Uh oh!

2 participants

kube commented Jan 6, 2026 •

edited

Loading

cursor bot commented Jan 7, 2026 •

edited

Loading