Skip to content

feat(issues): New stack trace on shared issues#111686

Merged
scttcper merged 7 commits into
masterfrom
scttcper/new-stack-trace-issue-preview
Apr 23, 2026
Merged

feat(issues): New stack trace on shared issues#111686
scttcper merged 7 commits into
masterfrom
scttcper/new-stack-trace-issue-preview

Conversation

@scttcper

@scttcper scttcper commented Mar 26, 2026

Copy link
Copy Markdown
Member

use the new stack trace on shared issues, no longer makes any requests. Not behind a flag since flags aren't possible on this page.

before

image

after

image

use the new stack trace on shared issues, no longer makes any requests
@github-actions github-actions Bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Mar 26, 2026
Re-apply stack trace changes on top of master refactor.

The sharedGroupDetails view was refactored in master to extract a
dedicated SharedEventContent component. Re-apply the SharedIssueStackTrace
integration into that component, replacing the legacy Exception and
StackTrace entry renderers with the new shared-safe stack trace component.

Also removes the eventEntries.tsx dependency (deleted in master) and
cleans up the inline SharedEventContent that was added to index.tsx.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
scttcper and others added 2 commits April 8, 2026 15:45
Extract duplicated `IndexedExceptionValue` type and
`resolveExceptionFields` helper into a shared utils module. Fix missing
`exception` and `isMinified` params in the copy-as and raw view handlers
so the exception header line shows up in copied text. Add the `Flex
gap="lg"` wrapper to the single-exception branch for consistent spacing.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
The shared stack trace component duplicated most of the render setup
from the authenticated one. Extracted the bits most likely to drift
(ordering, entry meta lookup, raw-text formatting used by both the raw
view and copy-as) into utils.ts so they stay in sync.

Also renames utils.tsx to utils.ts since it has no JSX, swaps a Flex
wrapper back to Fragment in the single-exception branch to match the
original, and adds a basic render test for the shared variant.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@scttcper scttcper marked this pull request as ready for review April 23, 2026 22:24
@scttcper scttcper requested a review from a team as a code owner April 23, 2026 22:24
@scttcper scttcper requested a review from a team April 23, 2026 22:24
@scttcper scttcper merged commit abdb163 into master Apr 23, 2026
65 checks passed
@scttcper scttcper deleted the scttcper/new-stack-trace-issue-preview branch April 23, 2026 22:38
@github-actions github-actions Bot locked and limited conversation to collaborators May 9, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants