H-6327: Integrate Sentry feedback button into ViewportControls by kube · Pull Request #8537 · hashintel/hash

kube · 2026-03-12T23:49:33Z

🌟 What is the purpose of this PR?

The Sentry "Give feedback" button was floating on top of the UI and hiding elements underneath it. This PR moves it into the ViewportControls panel by introducing a viewportActions prop on the Petrinaut component, allowing consumers to inject custom action buttons.

🔗 Related links

H-6327: Integrate Sentry feedback button into Petrinaut ViewportControls
Follows up on H-5764

🔍 What does this change?

Adds a ViewportAction type (key, icon, label, tooltip, onClick, style, className, ref) exported from @hashintel/petrinaut
Adds an optional viewportActions?: ViewportAction[] prop to PetrinautProps, threaded through to ViewportControls
ViewportControls renders each action as an IconButton after the built-in buttons
Sets autoInject: false on Sentry's feedbackIntegration to remove the floating button
Adds a useSentryFeedbackAction() hook in the website that returns a ViewportAction using Sentry's getFeedback().attachTo() API with a React 19 ref cleanup pattern
Purple-styled button with MdBugReport icon

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

❓ How to test this?

Checkout the branch
Run yarn dev in apps/petrinaut-website
Confirm the floating Sentry feedback button is gone
Confirm a purple feedback button appears at the bottom of the viewport controls (bottom-right)
Click it and confirm the Sentry feedback dialog opens

Add a `viewportActions` prop to Petrinaut that allows consumers to inject custom action buttons into the viewport controls panel. Use this to move the Sentry feedback trigger from a floating overlay into the controls, with a purple button style and bug report icon. - Add `ViewportAction` type with key, icon, label, tooltip, onClick, style, className, and ref fields - Thread `viewportActions` through Petrinaut → EditorView → SDCPNView → ViewportControls - Set `autoInject: false` on Sentry feedbackIntegration - Use React 19 ref cleanup to attach/detach Sentry feedback listener Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

vercel · 2026-03-12T23:49:39Z

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
hash	Ready	Preview, Comment	Mar 13, 2026 0:03am
petrinaut	Error		Mar 13, 2026 0:03am

2 Skipped Deployments

Project	Deployment	Actions	Updated (UTC)
hashdotdesign	Ignored	Preview	Mar 13, 2026 0:03am
hashdotdesign-tokens	Ignored	Preview	Mar 13, 2026 0:03am

cursor · 2026-03-12T23:49:49Z

PR Summary

Medium Risk
Introduces a new viewportActions extension point that changes the public Petrinaut API and affects core editor rendering; misuse (refs/callbacks) could impact viewport controls behavior, but changes are additive and UI-scoped.

Overview
Adds an extension point for custom buttons in the editor viewport controls by introducing a new ViewportAction type and a viewportActions prop that is threaded through Petrinaut → EditorView → SDCPNView → ViewportControls and rendered after the built-in controls.

Updates petrinaut-website to use this hook-based action to attach Sentry’s feedback widget to a new “Give feedback” viewport button (using react-icons), and disables Sentry feedback’s default auto-injected trigger via autoInject: false.

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

kube · 2026-03-12T23:49:50Z

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

H-6327: Integrate Sentry feedback button into ViewportControls #8537 👈 (View in Graphite)
FE-515: Extract petrinaut demo-site into @apps/petrinaut-website #8528
H-5655: Refactor selection logic and migrate to @xyflow/react v12 #8523 : 1 other dependent PR (#8532 )
FE-225: Petrinaut: Setup rolldown-plugin-dts and update @typeScript/native-preview + vite #8509
FE-225, H-6116, H-6286: Update Petrinaut UI #8472 : 1 other dependent PR (#8463 )
H-5839, FE-488: Petrinaut: LSP-like interface for Diagnostics and Code Editor #8300
FE-223, H-5780, H-6227: Setup Petrinaut inside HASH + Build with Vite 8 and TypeScript 7 #8369
main

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

augmentcode · 2026-03-12T23:52:35Z

🤖 Augment PR Summary

Summary: This PR adds an extension point for extra viewport control buttons and uses it to expose a Sentry “Give feedback” button in Petrinaut.

Changes:

Introduces a shared ViewportAction type and a new viewportActions prop on Petrinaut, threading it through EditorView and SDCPNView into ViewportControls.
Updates ViewportControls to render the provided actions as additional IconButtons after the built-in controls.
Adds a Petrinaut-website hook that attaches Sentry Feedback to a viewport control button (using a bug-report icon).
Adjusts Sentry Feedback integration config to disable auto-injection so the UI button is only rendered via the new viewport action.

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

augmentcode

Review completed. 2 suggestions posted.

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

augmentcode · 2026-03-12T23:52:36Z

apps/petrinaut-website/src/main/app/sentry-feedback-button.tsx

+const icon = <MdBugReport size={14} />;
+
+export function useSentryFeedbackAction(): ViewportAction {
+  return {


useSentryFeedbackAction() returns a new ViewportAction object (and callback ref) on every render, which can cause React to detach/reattach the ref and repeatedly run feedback.attachTo. That can lead to duplicate handlers or a subtle flicker if the host component re-renders frequently.

Severity: medium

_{🤖 Was this useful? React with 👍 or 👎, or 🚀 if it prevented an incident/outage.}

augmentcode · 2026-03-12T23:52:36Z

apps/petrinaut-website/src/main/app.tsx

        setTitle={setTitle}
        title={currentNet.title}
        undoRedo={undoRedo}
+        viewportActions={[sentryFeedbackAction]}


viewportActions={[sentryFeedbackAction]} creates a new array each render, so downstream components will always see this prop change. If the action/ref isn’t stable, this can amplify unnecessary ref detach/attach work for the Sentry feedback button.

Severity: low

_{🤖 Was this useful? React with 👍 or 👎, or 🚀 if it prevented an incident/outage.}

cursor

Cursor Bugbot has reviewed your changes and found 1 potential issue.

^{Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.}

cursor · 2026-03-12T23:54:20Z

apps/petrinaut-website/src/main/app/sentry-feedback-button.tsx

+      };
+    },
+  };
+}


Unstable ref callback causes repeated Sentry attach/detach cycles

Medium Severity

useSentryFeedbackAction returns a new object (with a new ref callback) on every render without any memoization. Since DevApp re-renders on every net edit or state change, React 19 will invoke the old ref's cleanup (calling unsubscribe) and then call the new ref (calling attachTo again) on each render. This causes the Sentry feedback integration to be repeatedly detached and re-attached, which could cause brief unresponsiveness of the feedback button and unnecessary overhead. The return value — or at minimum the ref callback — needs to be memoized (e.g. via useMemo/useCallback).

Additional Locations (1)

apps/petrinaut-website/src/main/app.tsx#L14-L15

kube and others added 2 commits March 13, 2026 00:49

Update yarn.lock for react-icons dependency

70d198a

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

github-actions bot added area/deps Relates to third-party dependencies (area) area/libs Relates to first-party libraries/crates/packages (area) type/eng > frontend Owned by the @frontend team area/apps labels Mar 12, 2026

kube marked this pull request as ready for review March 12, 2026 23:49

kube mentioned this pull request Mar 12, 2026

FE-515: Extract petrinaut demo-site into @apps/petrinaut-website #8528

Open

3 tasks

vercel bot had a problem deploying to Preview – petrinaut March 12, 2026 23:49 Failure

graphite-app bot assigned kube Mar 12, 2026

augmentcode bot reviewed Mar 12, 2026

View reviewed changes

cursor bot reviewed Mar 12, 2026

View reviewed changes

kube changed the title ~~Integrate Sentry feedback button into ViewportControls~~ H-6327: Integrate Sentry feedback button into ViewportControls Mar 12, 2026

vercel bot deployed to Preview – hash March 13, 2026 00:03 View deployment

graphite-app bot requested a review from a team March 13, 2026 00:08

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

H-6327: Integrate Sentry feedback button into ViewportControls#8537

H-6327: Integrate Sentry feedback button into ViewportControls#8537
kube wants to merge 2 commits intocf/petrinaut-extract-websitefrom
cf/petrinaut-sentry-integrated-button-in-ui

kube commented Mar 12, 2026 •

edited

Loading

Uh oh!

vercel bot commented Mar 12, 2026 •

edited

Loading

Uh oh!

cursor bot commented Mar 12, 2026 •

edited

Loading

Uh oh!

kube commented Mar 12, 2026

Uh oh!

augmentcode bot commented Mar 12, 2026

Uh oh!

augmentcode bot left a comment

Uh oh!

augmentcode bot Mar 12, 2026

Uh oh!

augmentcode bot Mar 12, 2026

Uh oh!

cursor bot left a comment

Uh oh!

cursor bot Mar 12, 2026

Uh oh!

Reviewers

Assignees

Labels

Milestone

Development

Uh oh!

1 participant

Conversation

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

Uh oh!

🌟 What is the purpose of this PR?

🔗 Related links

🔍 What does this change?

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?

❓ How to test this?

Uh oh!

vercel bot commented Mar 12, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

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

Uh oh!

PR Summary

Uh oh!

kube commented Mar 12, 2026

Uh oh!

augmentcode bot commented Mar 12, 2026

Uh oh!

augmentcode bot left a comment

Choose a reason for hiding this comment

Uh oh!

augmentcode bot Mar 12, 2026

Choose a reason for hiding this comment

Uh oh!

augmentcode bot Mar 12, 2026

Choose a reason for hiding this comment

Uh oh!

cursor bot left a comment

Choose a reason for hiding this comment

Uh oh!

cursor bot Mar 12, 2026

Choose a reason for hiding this comment

Unstable ref callback causes repeated Sentry attach/detach cycles

Uh oh!

Reviewers

Assignees

Labels

Milestone

Development

Uh oh!

1 participant

kube commented Mar 12, 2026 •

edited

Loading

vercel bot commented Mar 12, 2026 •

edited

Loading

cursor bot commented Mar 12, 2026 •

edited

Loading