build(ui): Add rspack as webpack alternative - #77077

Merged
scttcper merged 54 commits into
masterfrom
scttcper/rspack
May 15, 2025
Merged

build(ui): Add rspack as webpack alternative#77077
scttcper merged 54 commits into
masterfrom
scttcper/rspack

Conversation

@scttcper

@scttcperscttcper commented Sep 6, 2024•
edited
Loading

Copy link
Copy Markdown
Member

Building off the work done in #61475 I think the plan would be to use this as a dev-ui replacement until we're more confident and can setup the getsentry repo as well (if you aren't familiar we have another repo with billing pages etc).

Potential adoption roadmap

  1. merge rspack as optional dev only command yarn dev-ui-rspack
  2. make rspack default for dev-ui
  3. use rspack for acceptance tests
  4. build production with rspack
  5. remove webpack

Summary

WebpackRspack
HMR Time4s607ms
Dev Memory Usage3.85 GB2.8 GB
Build Production131.53s39.42s

RSPACK + SENTRY TODO (updated Oct 7th, 2024)

  • Investigate emotion autoLabel in development mode. Currently set to autoLabel: 'always', Could just set label setting based on node_env.
  • Add back webpack in order to merge (getsentry repo will block until it can accept a rspack file instead of webpack)
  • Figure out a path forward for translation string extraction that use webpack + babel's AST make build-js-po
  • Chartcuterie is failing for some reason

Thanks @hardfist for helping with this PR!

anonrigand others added 3 commits December 11, 2023 20:03
@anonrig
build: replace webpack with rspack
6e51e59
@scttcper
Merge branch 'master' into scttcper/rspack
d266cfd
# Conflicts:
#	build-utils/last-built-plugin.ts
#	config/rspack.chartcuterie.config.ts
#	package.json
#	rspack.config.ts
#	yarn.lock
@scttcper
mostly working
859e9fc
@github-actionsgithub-actionsBot added Scope: Frontend Automatically applied to PRs that change frontend components Scope: Backend Automatically applied to PRs that change backend components labels Sep 6, 2024
@github-actions

github-actionsBot commented Sep 6, 2024

Copy link
Copy Markdown
Contributor

🚨 Warning: This pull request contains Frontend and Backend changes!

It's discouraged to make changes to Sentry's Frontend and Backend in a single pull request. The Frontend and Backend are not atomically deployed. If the changes are interdependent of each other, they must be separated into two pull requests and be made forward or backwards compatible, such that the Backend or Frontend can be safely deployed independently.

Have questions? Please ask in the #discuss-dev-infra channel.

@scttcperscttcper added the WIP label Sep 6, 2024
@scttcperscttcper mentioned this pull request Sep 6, 2024
Closed
6 tasks
@scttcper

scttcper commented Sep 6, 2024•
edited
Loading

Copy link
Copy Markdown
MemberAuthor

Just realized its working for me locally because i fixed platformicons locally will need to publish a fix to the svg paths there

@hardfisthardfist mentioned this pull request Sep 9, 2024
Closed
@scttcper
Merge branch 'master' into scttcper/rspack
e92e6d8
# Conflicts:
#	package.json
#	rspack.config.ts
#	yarn.lock
@scttcper
enable ContextReplacementPlugin
2068906
@codecov

codecovBot commented Sep 26, 2024•
edited
Loading

Copy link
Copy Markdown

❌ 1 Tests Failed:

Tests completedFailedPassedSkipped
102901102899
View the full list of 1 ❄️ flaky tests
SentryApplicationDetailsEditinganexistingpublicSentryAppwithascopeerrorhandlesclientsecretrotation

Flake rate in main: 18.18% (Passed 9 times, Failed 2 times)

Stack Traces | 0.405s run time
TestingLibraryElementError: Unabletofindanelementwiththetext: Thiswillbetheonlytimeyourclientsecretisvisible!. Thiscouldbebecausethetextisbrokenupbymultipleelements. Inthiscase, youcanprovideafunctionforyourtextmatchertomakeyourmatchermoreflexible.
Ignorednodes: comments, script, style
...
atObject.getElementError (.../sentry/sentry/node_modules/@.../dom/dist/config.js:37:19)
at .../sentry/sentry/node_modules/@.../dom/dist/query-helpers.js:76:38at .../sentry/sentry/node_modules/@.../dom/dist/query-helpers.js:52:17at .../sentry/sentry/node_modules/@.../dom/dist/query-helpers.js:95:19atObject.<anonymous> (.../settings/organizationDeveloperSettings/sentryApplicationDetails.spec.tsx:591:42)

To view more test analytics, go to the Test Analytics Dashboard
📋 Got 3 mins? Take this short survey to help us improve Test Analytics.

@scttcperscttcper mentioned this pull request Sep 27, 2024
Merged
@scttcper
scttcper marked this pull request as ready for review May 9, 2025 20:12
@scttcper
scttcper requested review from a team as code ownersMay 9, 2025 20:12
@scttcper
scttcper requested a review from a teamMay 9, 2025 20:12
evanpurkhiser
evanpurkhiser approved these changes May 9, 2025
View reviewed changes

@evanpurkhiserevanpurkhiser left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

slam it

JonasBa
JonasBa approved these changes May 12, 2025
View reviewed changes

@JonasBaJonasBa left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Those HMR numbers look 🔥

Comment threadrspack.config.ts
supported_locales: string[];
};

const localeCatalog: LocaleCatalog = JSON.parse(

@JonasBaJonasBaMay 12, 2025

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Outside of the scope of this PR, but it would be interesting if we could time this locale procedure to see how much it contributes to the startup cost.

@scttcper
scttcper merged commit 9168894 into masterMay 15, 2025
@scttcper
scttcper deleted the scttcper/rspack branch May 15, 2025 16:57
@github-actionsgithub-actionsBot locked and limited conversation to collaborators May 31, 2025
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@evanpurkhiserevanpurkhiserevanpurkhiser approved these changes
+1 more reviewer
@JonasBaJonasBaJonasBa approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

Scope: BackendAutomatically applied to PRs that change backend componentsScope: FrontendAutomatically applied to PRs that change frontend componentsWIP

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@scttcper@evanpurkhiser@JonasBa@anonrig
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Add copy buttons to all 
 blocks\n(function() {\n function addCopyButtons() {\n document.querySelectorAll('pre code').forEach(function(codeBlock) {\n if (codeBlock.parentElement.hasAttribute('data-copy-added')) return;\n codeBlock.parentElement.setAttribute('data-copy-added', 'true');\n \n var btn = document.createElement('button');\n btn.textContent = 'Copy';\n btn.style.cssText = 'position:absolute;top:4px;right:4px;padding:2px 8px;font-size:11px;background:#4ecdc4;border:none;border-radius:4px;color:#1a1a2e;cursor:pointer;opacity:0.7;transition:opacity 0.2s;';\n btn.onmouseover = function() { this.style.opacity = '1'; };\n btn.onmouseout = function() { this.style.opacity = '0.7'; };\n btn.onclick = function() {\n navigator.clipboard.writeText(codeBlock.textContent).then(function() {\n btn.textContent = 'Copied!';\n setTimeout(function() { btn.textContent = 'Copy'; }, 1500);\n });\n };\n codeBlock.parentElement.style.position = 'relative';\n codeBlock.parentElement.appendChild(btn);\n });\n }\n \n addCopyButtons();\n \n // Re-run on dynamic content\n var observer = new MutationObserver(addCopyButtons);\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Add Copy Buttons to Code Blocks");
}
} catch(__e) { console.warn('[Userscript:Add Copy Buttons to Code Blocks]', __e); }
})();
(function(){
try {
var __m = "github.com";
var __re = new RegExp('^' + "github\\.com" + '

Skip to content
build(ui): Add rspack as webpack alternative - #77077
Merged
scttcper merged 54 commits into
masterfrom 
scttcper/rspack
May 15, 2025
Merged
build(ui): Add rspack as webpack alternative#77077
scttcper merged 54 commits into
masterfrom 
scttcper/rspack
Conversation
@scttcper
@scttcperscttcper
commented
Sep 6, 2024•

edited

Loading

Copy link


Copy Markdown

Member
Building off the work done in #61475 I think the plan would be to use this as a dev-ui replacement until we're more confident and can setup the getsentry repo as well (if you aren't familiar we have another repo with billing pages etc).
Potential adoption roadmap
  1. merge rspack as optional dev only command yarn dev-ui-rspack
  2. make rspack default for dev-ui
  3. use rspack for acceptance tests
  4. build production with rspack
  5. remove webpack
Summary
WebpackRspack
HMR Time4s607ms
Dev Memory Usage3.85 GB2.8 GB
Build Production131.53s39.42s
RSPACK + SENTRY TODO (updated Oct 7th, 2024)
  •  Investigate emotion autoLabel in development mode. Currently set to autoLabel: 'always', Could just set label setting based on node_env.
  •  Add back webpack in order to merge (getsentry repo will block until it can accept a rspack file instead of webpack)
  •  Figure out a path forward for translation string extraction that use webpack + babel's AST make build-js-po
  •  Chartcuterie is failing for some reason
Thanks @hardfist for helping with this PR!
anonrigand others
added 3 commits
December 11, 2023 20:03
@anonrig
build: replace webpack with rspack
6e51e59
@scttcper
Merge branch 'master' into scttcper/rspack
d266cfd
# Conflicts:
#	build-utils/last-built-plugin.ts
#	config/rspack.chartcuterie.config.ts
#	package.json
#	rspack.config.ts
#	yarn.lock
@scttcper
mostly working
859e9fc
@github-actionsgithub-actionsBot
added

Scope: Frontend
Automatically applied to PRs that change frontend components
Scope: Backend
Automatically applied to PRs that change backend components
labels
Sep 6, 2024
@github-actions
github-actionsBot
commented
Sep 6, 2024

Copy link


Copy Markdown

Contributor
🚨 Warning: This pull request contains Frontend and Backend changes!
It's discouraged to make changes to Sentry's Frontend and Backend in a single pull request. The Frontend and Backend are not atomically deployed. If the changes are interdependent of each other, they must be separated into two pull requests and be made forward or backwards compatible, such that the Backend or Frontend can be safely deployed independently.
Have questions? Please ask in the #discuss-dev-infra channel.
@scttcperscttcper
added
the 
WIP

label
Sep 6, 2024
@scttcperscttcper
mentioned this pull request
Sep 6, 2024
 Closed

6 tasks
@scttcper
scttcper
commented
Sep 6, 2024•

edited

Loading

Copy link


Copy Markdown

MemberAuthor
Just realized its working for me locally because i fixed platformicons locally will need to publish a fix to the svg paths there
@hardfisthardfist
mentioned this pull request
Sep 9, 2024

Closed

@scttcper
Merge branch 'master' into scttcper/rspack
e92e6d8
# Conflicts:
#	package.json
#	rspack.config.ts
#	yarn.lock
@scttcper
enable ContextReplacementPlugin
2068906
@codecov
codecovBot
commented
Sep 26, 2024•

edited

Loading

Copy link


Copy Markdown

❌ 1 Tests Failed:
Tests completedFailedPassedSkipped
102901102899
View the full list of 1 ❄️ flaky tests
SentryApplicationDetailsEditinganexistingpublicSentryAppwithascopeerrorhandlesclientsecretrotation
Flake rate in main: 18.18% (Passed 9 times, Failed 2 times)
Stack Traces | 0.405s run time
TestingLibraryElementError: Unabletofindanelementwiththetext: Thiswillbetheonlytimeyourclientsecretisvisible!. Thiscouldbebecausethetextisbrokenupbymultipleelements. Inthiscase, youcanprovideafunctionforyourtextmatchertomakeyourmatchermoreflexible.
Ignorednodes: comments, script, style
...
atObject.getElementError (.../sentry/sentry/node_modules/@.../dom/dist/config.js:37:19)
at .../sentry/sentry/node_modules/@.../dom/dist/query-helpers.js:76:38at .../sentry/sentry/node_modules/@.../dom/dist/query-helpers.js:52:17at .../sentry/sentry/node_modules/@.../dom/dist/query-helpers.js:95:19atObject.<anonymous> (.../settings/organizationDeveloperSettings/sentryApplicationDetails.spec.tsx:591:42)
To view more test analytics, go to the Test Analytics Dashboard
📋 Got 3 mins? Take this short survey to help us improve Test Analytics.
@scttcperscttcper
mentioned this pull request
Sep 27, 2024
 Merged

@scttcper
scttcper
marked this pull request as ready for review
May 9, 2025 20:12
@scttcper
scttcper
requested review from
a team
as code ownersMay 9, 2025 20:12
@scttcper
scttcper
requested a review
from a teamMay 9, 2025 20:12
evanpurkhiser
evanpurkhiser
approved these changes
May 9, 2025
View reviewed changes
@evanpurkhiserevanpurkhiser
left a comment


Copy link


Copy Markdown

Member
Choose a reason for hiding this comment

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

slam it
JonasBa
JonasBa
approved these changes
May 12, 2025
View reviewed changes
@JonasBaJonasBa
left a comment


Copy link


Copy Markdown

Contributor
Choose a reason for hiding this comment

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

Those HMR numbers look 🔥
Comment threadrspack.config.ts
 supported_locales: string[];
};

const localeCatalog: LocaleCatalog = JSON.parse(
@JonasBaJonasBaMay 12, 2025

Copy link


Copy Markdown

Contributor
Choose a reason for hiding this comment

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

Outside of the scope of this PR, but it would be interesting if we could time this locale procedure to see how much it contributes to the startup cost.
@scttcper
scttcper
merged commit 9168894
into
masterMay 15, 2025
@scttcper
scttcper
deleted the

scttcper/rspack

branch
May 15, 2025 16:57
@github-actionsgithub-actionsBot
locked and limited conversation to collaborators
May 31, 2025
Sign up for freeto subscribe to this conversation on GitHub.
Already have an account?
Sign in.


Reviewers

@evanpurkhiserevanpurkhiserevanpurkhiser approved these changes

+1 more reviewer

@JonasBaJonasBaJonasBa approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees


No one assigned


Labels

Scope: BackendAutomatically applied to PRs that change backend componentsScope: FrontendAutomatically applied to PRs that change frontend componentsWIP

Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

4 participants

@scttcper@evanpurkhiser@JonasBa@anonrig

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Force GitHub README to respect dark mode\n(function() {\n var style = document.createElement('style');\n style.textContent = '\n .markdown-body {\n color-scheme: dark light;\n }\n .markdown-body pre { background: #161b22 !important; }\n .markdown-body code { background: rgba(110, 118, 129, 0.4) !important; }\n .markdown-body table th, .markdown-body table td { border-color: #30363d !important; }\n .markdown-body img { background: #0d1117; }\n .markdown-body blockquote { border-left-color: #8b949e; }\n .markdown-body hr { border-color: #30363d; }\n ';\n document.head.appendChild(style);\n})();", "GitHub Dark Mode README Fix");
}
} catch(__e) { console.warn('[Userscript:GitHub Dark Mode README Fix]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '

Skip to content
build(ui): Add rspack as webpack alternative - #77077
Merged
scttcper merged 54 commits into
masterfrom 
scttcper/rspack
May 15, 2025
Merged
build(ui): Add rspack as webpack alternative#77077
scttcper merged 54 commits into
masterfrom 
scttcper/rspack
Conversation
@scttcper
@scttcperscttcper
commented
Sep 6, 2024•

edited

Loading

Copy link


Copy Markdown

Member
Building off the work done in #61475 I think the plan would be to use this as a dev-ui replacement until we're more confident and can setup the getsentry repo as well (if you aren't familiar we have another repo with billing pages etc).
Potential adoption roadmap
  1. merge rspack as optional dev only command yarn dev-ui-rspack
  2. make rspack default for dev-ui
  3. use rspack for acceptance tests
  4. build production with rspack
  5. remove webpack
Summary
WebpackRspack
HMR Time4s607ms
Dev Memory Usage3.85 GB2.8 GB
Build Production131.53s39.42s
RSPACK + SENTRY TODO (updated Oct 7th, 2024)
  •  Investigate emotion autoLabel in development mode. Currently set to autoLabel: 'always', Could just set label setting based on node_env.
  •  Add back webpack in order to merge (getsentry repo will block until it can accept a rspack file instead of webpack)
  •  Figure out a path forward for translation string extraction that use webpack + babel's AST make build-js-po
  •  Chartcuterie is failing for some reason
Thanks @hardfist for helping with this PR!
anonrigand others
added 3 commits
December 11, 2023 20:03
@anonrig
build: replace webpack with rspack
6e51e59
@scttcper
Merge branch 'master' into scttcper/rspack
d266cfd
# Conflicts:
#	build-utils/last-built-plugin.ts
#	config/rspack.chartcuterie.config.ts
#	package.json
#	rspack.config.ts
#	yarn.lock
@scttcper
mostly working
859e9fc
@github-actionsgithub-actionsBot
added

Scope: Frontend
Automatically applied to PRs that change frontend components
Scope: Backend
Automatically applied to PRs that change backend components
labels
Sep 6, 2024
@github-actions
github-actionsBot
commented
Sep 6, 2024

Copy link


Copy Markdown

Contributor
🚨 Warning: This pull request contains Frontend and Backend changes!
It's discouraged to make changes to Sentry's Frontend and Backend in a single pull request. The Frontend and Backend are not atomically deployed. If the changes are interdependent of each other, they must be separated into two pull requests and be made forward or backwards compatible, such that the Backend or Frontend can be safely deployed independently.
Have questions? Please ask in the #discuss-dev-infra channel.
@scttcperscttcper
added
the 
WIP

label
Sep 6, 2024
@scttcperscttcper
mentioned this pull request
Sep 6, 2024
 Closed

6 tasks
@scttcper
scttcper
commented
Sep 6, 2024•

edited

Loading

Copy link


Copy Markdown

MemberAuthor
Just realized its working for me locally because i fixed platformicons locally will need to publish a fix to the svg paths there
@hardfisthardfist
mentioned this pull request
Sep 9, 2024

Closed

@scttcper
Merge branch 'master' into scttcper/rspack
e92e6d8
# Conflicts:
#	package.json
#	rspack.config.ts
#	yarn.lock
@scttcper
enable ContextReplacementPlugin
2068906
@codecov
codecovBot
commented
Sep 26, 2024•

edited

Loading

Copy link


Copy Markdown

❌ 1 Tests Failed:
Tests completedFailedPassedSkipped
102901102899
View the full list of 1 ❄️ flaky tests
SentryApplicationDetailsEditinganexistingpublicSentryAppwithascopeerrorhandlesclientsecretrotation
Flake rate in main: 18.18% (Passed 9 times, Failed 2 times)
Stack Traces | 0.405s run time
TestingLibraryElementError: Unabletofindanelementwiththetext: Thiswillbetheonlytimeyourclientsecretisvisible!. Thiscouldbebecausethetextisbrokenupbymultipleelements. Inthiscase, youcanprovideafunctionforyourtextmatchertomakeyourmatchermoreflexible.
Ignorednodes: comments, script, style
...
atObject.getElementError (.../sentry/sentry/node_modules/@.../dom/dist/config.js:37:19)
at .../sentry/sentry/node_modules/@.../dom/dist/query-helpers.js:76:38at .../sentry/sentry/node_modules/@.../dom/dist/query-helpers.js:52:17at .../sentry/sentry/node_modules/@.../dom/dist/query-helpers.js:95:19atObject.<anonymous> (.../settings/organizationDeveloperSettings/sentryApplicationDetails.spec.tsx:591:42)
To view more test analytics, go to the Test Analytics Dashboard
📋 Got 3 mins? Take this short survey to help us improve Test Analytics.
@scttcperscttcper
mentioned this pull request
Sep 27, 2024
 Merged

@scttcper
scttcper
marked this pull request as ready for review
May 9, 2025 20:12
@scttcper
scttcper
requested review from
a team
as code ownersMay 9, 2025 20:12
@scttcper
scttcper
requested a review
from a teamMay 9, 2025 20:12
evanpurkhiser
evanpurkhiser
approved these changes
May 9, 2025
View reviewed changes
@evanpurkhiserevanpurkhiser
left a comment


Copy link


Copy Markdown

Member
Choose a reason for hiding this comment

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

slam it
JonasBa
JonasBa
approved these changes
May 12, 2025
View reviewed changes
@JonasBaJonasBa
left a comment


Copy link


Copy Markdown

Contributor
Choose a reason for hiding this comment

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

Those HMR numbers look 🔥
Comment threadrspack.config.ts
 supported_locales: string[];
};

const localeCatalog: LocaleCatalog = JSON.parse(
@JonasBaJonasBaMay 12, 2025

Copy link


Copy Markdown

Contributor
Choose a reason for hiding this comment

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

Outside of the scope of this PR, but it would be interesting if we could time this locale procedure to see how much it contributes to the startup cost.
@scttcper
scttcper
merged commit 9168894
into
masterMay 15, 2025
@scttcper
scttcper
deleted the

scttcper/rspack

branch
May 15, 2025 16:57
@github-actionsgithub-actionsBot
locked and limited conversation to collaborators
May 31, 2025
Sign up for freeto subscribe to this conversation on GitHub.
Already have an account?
Sign in.


Reviewers

@evanpurkhiserevanpurkhiserevanpurkhiser approved these changes

+1 more reviewer

@JonasBaJonasBaJonasBa approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees


No one assigned


Labels

Scope: BackendAutomatically applied to PRs that change backend componentsScope: FrontendAutomatically applied to PRs that change frontend componentsWIP

Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

4 participants

@scttcper@evanpurkhiser@JonasBa@anonrig

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Highlight search terms from Google/DuckDuckGo/Bing referrer\n(function() {\n var ref = document.referrer;\n var terms = [];\n \n if (ref.includes('google.com') || ref.includes('duckduckgo.com') || ref.includes('bing.com')) {\n var url = new URL(ref);\n var q = url.searchParams.get('q') || url.searchParams.get('p');\n if (q) {\n terms = q.split(/\\s+/).filter(function(t) { return t.length > 2; });\n }\n }\n \n if (terms.length === 0) return;\n \n var style = document.createElement('style');\n style.textContent = '.userscript-highlight { background: #fbbf24; color: #1a1a2e; padding: 1px 3px; border-radius: 2px; }';\n document.head.appendChild(style);\n \n function highlight(node) {\n if (node.nodeType === 3) { // text node\n var text = node.textContent;\n var found = false;\n terms.forEach(function(term) {\n var regex = new RegExp('(' + term.replace(/[.*+?^${}()|[\\]\\\\]/g, '\\\\') + ')', 'gi');\n if (regex.test(text)) {\n found = true;\n var frag = document.createDocumentFragment();\n var parts = text.split(regex);\n parts.forEach(function(part, i) {\n if (i % 2 === 0) {\n frag.appendChild(document.createTextNode(part));\n } else {\n var span = document.createElement('span');\n span.className = 'userscript-highlight';\n span.textContent = part;\n frag.appendChild(span);\n }\n });\n node.parentNode.replaceChild(frag, node);\n }\n });\n } else if (node.nodeType === 1 && node.childNodes) { // element\n var skipTags = ['SCRIPT', 'STYLE', 'NOSCRIPT', 'TEXTAREA', 'INPUT', 'SELECT'];\n if (!skipTags.includes(node.tagName)) {\n Array.from(node.childNodes).forEach(highlight);\n }\n }\n }\n \n highlight(document.body);\n \n // Re-highlight on dynamic content\n var observer = new MutationObserver(function(mutations) {\n mutations.forEach(function(m) {\n m.addedNodes.forEach(function(node) {\n if (node.nodeType === 1 || node.nodeType === 3) highlight(node);\n });\n });\n });\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Highlight Search Terms");
}
} catch(__e) { console.warn('[Userscript:Highlight Search Terms]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '

Skip to content
build(ui): Add rspack as webpack alternative - #77077
Merged
scttcper merged 54 commits into
masterfrom 
scttcper/rspack
May 15, 2025
Merged
build(ui): Add rspack as webpack alternative#77077
scttcper merged 54 commits into
masterfrom 
scttcper/rspack
Conversation
@scttcper
@scttcperscttcper
commented
Sep 6, 2024•

edited

Loading

Copy link


Copy Markdown

Member
Building off the work done in #61475 I think the plan would be to use this as a dev-ui replacement until we're more confident and can setup the getsentry repo as well (if you aren't familiar we have another repo with billing pages etc).
Potential adoption roadmap
  1. merge rspack as optional dev only command yarn dev-ui-rspack
  2. make rspack default for dev-ui
  3. use rspack for acceptance tests
  4. build production with rspack
  5. remove webpack
Summary
WebpackRspack
HMR Time4s607ms
Dev Memory Usage3.85 GB2.8 GB
Build Production131.53s39.42s
RSPACK + SENTRY TODO (updated Oct 7th, 2024)
  •  Investigate emotion autoLabel in development mode. Currently set to autoLabel: 'always', Could just set label setting based on node_env.
  •  Add back webpack in order to merge (getsentry repo will block until it can accept a rspack file instead of webpack)
  •  Figure out a path forward for translation string extraction that use webpack + babel's AST make build-js-po
  •  Chartcuterie is failing for some reason
Thanks @hardfist for helping with this PR!
anonrigand others
added 3 commits
December 11, 2023 20:03
@anonrig
build: replace webpack with rspack
6e51e59
@scttcper
Merge branch 'master' into scttcper/rspack
d266cfd
# Conflicts:
#	build-utils/last-built-plugin.ts
#	config/rspack.chartcuterie.config.ts
#	package.json
#	rspack.config.ts
#	yarn.lock
@scttcper
mostly working
859e9fc
@github-actionsgithub-actionsBot
added

Scope: Frontend
Automatically applied to PRs that change frontend components
Scope: Backend
Automatically applied to PRs that change backend components
labels
Sep 6, 2024
@github-actions
github-actionsBot
commented
Sep 6, 2024

Copy link


Copy Markdown

Contributor
🚨 Warning: This pull request contains Frontend and Backend changes!
It's discouraged to make changes to Sentry's Frontend and Backend in a single pull request. The Frontend and Backend are not atomically deployed. If the changes are interdependent of each other, they must be separated into two pull requests and be made forward or backwards compatible, such that the Backend or Frontend can be safely deployed independently.
Have questions? Please ask in the #discuss-dev-infra channel.
@scttcperscttcper
added
the 
WIP

label
Sep 6, 2024
@scttcperscttcper
mentioned this pull request
Sep 6, 2024
 Closed

6 tasks
@scttcper
scttcper
commented
Sep 6, 2024•

edited

Loading

Copy link


Copy Markdown

MemberAuthor
Just realized its working for me locally because i fixed platformicons locally will need to publish a fix to the svg paths there
@hardfisthardfist
mentioned this pull request
Sep 9, 2024

Closed

@scttcper
Merge branch 'master' into scttcper/rspack
e92e6d8
# Conflicts:
#	package.json
#	rspack.config.ts
#	yarn.lock
@scttcper
enable ContextReplacementPlugin
2068906
@codecov
codecovBot
commented
Sep 26, 2024•

edited

Loading

Copy link


Copy Markdown

❌ 1 Tests Failed:
Tests completedFailedPassedSkipped
102901102899
View the full list of 1 ❄️ flaky tests
SentryApplicationDetailsEditinganexistingpublicSentryAppwithascopeerrorhandlesclientsecretrotation
Flake rate in main: 18.18% (Passed 9 times, Failed 2 times)
Stack Traces | 0.405s run time
TestingLibraryElementError: Unabletofindanelementwiththetext: Thiswillbetheonlytimeyourclientsecretisvisible!. Thiscouldbebecausethetextisbrokenupbymultipleelements. Inthiscase, youcanprovideafunctionforyourtextmatchertomakeyourmatchermoreflexible.
Ignorednodes: comments, script, style
...
atObject.getElementError (.../sentry/sentry/node_modules/@.../dom/dist/config.js:37:19)
at .../sentry/sentry/node_modules/@.../dom/dist/query-helpers.js:76:38at .../sentry/sentry/node_modules/@.../dom/dist/query-helpers.js:52:17at .../sentry/sentry/node_modules/@.../dom/dist/query-helpers.js:95:19atObject.<anonymous> (.../settings/organizationDeveloperSettings/sentryApplicationDetails.spec.tsx:591:42)
To view more test analytics, go to the Test Analytics Dashboard
📋 Got 3 mins? Take this short survey to help us improve Test Analytics.
@scttcperscttcper
mentioned this pull request
Sep 27, 2024
 Merged

@scttcper
scttcper
marked this pull request as ready for review
May 9, 2025 20:12
@scttcper
scttcper
requested review from
a team
as code ownersMay 9, 2025 20:12
@scttcper
scttcper
requested a review
from a teamMay 9, 2025 20:12
evanpurkhiser
evanpurkhiser
approved these changes
May 9, 2025
View reviewed changes
@evanpurkhiserevanpurkhiser
left a comment


Copy link


Copy Markdown

Member
Choose a reason for hiding this comment

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

slam it
JonasBa
JonasBa
approved these changes
May 12, 2025
View reviewed changes
@JonasBaJonasBa
left a comment


Copy link


Copy Markdown

Contributor
Choose a reason for hiding this comment

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

Those HMR numbers look 🔥
Comment threadrspack.config.ts
 supported_locales: string[];
};

const localeCatalog: LocaleCatalog = JSON.parse(
@JonasBaJonasBaMay 12, 2025

Copy link


Copy Markdown

Contributor
Choose a reason for hiding this comment

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

Outside of the scope of this PR, but it would be interesting if we could time this locale procedure to see how much it contributes to the startup cost.
@scttcper
scttcper
merged commit 9168894
into
masterMay 15, 2025
@scttcper
scttcper
deleted the

scttcper/rspack

branch
May 15, 2025 16:57
@github-actionsgithub-actionsBot
locked and limited conversation to collaborators
May 31, 2025
Sign up for freeto subscribe to this conversation on GitHub.
Already have an account?
Sign in.


Reviewers

@evanpurkhiserevanpurkhiserevanpurkhiser approved these changes

+1 more reviewer

@JonasBaJonasBaJonasBa approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees


No one assigned


Labels

Scope: BackendAutomatically applied to PRs that change backend componentsScope: FrontendAutomatically applied to PRs that change frontend componentsWIP

Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

4 participants

@scttcper@evanpurkhiser@JonasBa@anonrig

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Strip utm_, fbclid, gclid, etc. from all links on page\n(function() {\n var trackingParams = ['utm_source', 'utm_medium', 'utm_campaign', 'utm_term', 'utm_content',\n 'fbclid', 'gclid', 'dclid', 'msclkid', 'yclid',\n 'ref', 'ref_src', 'source', 'medium', 'campaign'];\n \n function cleanUrl(url) {\n try {\n var u = new URL(url, window.location.origin);\n var changed = false;\n trackingParams.forEach(function(p) {\n if (u.searchParams.has(p)) {\n u.searchParams.delete(p);\n changed = true;\n }\n });\n return changed ? u.toString() : url;\n } catch (e) {\n return url;\n }\n }\n \n function cleanLinks() {\n document.querySelectorAll('a[href]').forEach(function(a) {\n var clean = cleanUrl(a.href);\n if (clean !== a.href) a.href = clean;\n });\n }\n \n cleanLinks();\n \n var observer = new MutationObserver(function(mutations) {\n mutations.forEach(function(m) {\n m.addedNodes.forEach(function(node) {\n if (node.nodeType === 1) {\n if (node.tagName === 'A') cleanLinks();\n node.querySelectorAll('a[href]').forEach(function(a) {\n var clean = cleanUrl(a.href);\n if (clean !== a.href) a.href = clean;\n });\n }\n });\n });\n });\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Remove Tracking Parameters from Links");
}
} catch(__e) { console.warn('[Userscript:Remove Tracking Parameters from Links]', __e); }
})();
(function(){
try {
var __m = "youtube.com";
var __re = new RegExp('^' + "youtube\\.com" + '

Skip to content
build(ui): Add rspack as webpack alternative - #77077
Merged
scttcper merged 54 commits into
masterfrom 
scttcper/rspack
May 15, 2025
Merged
build(ui): Add rspack as webpack alternative#77077
scttcper merged 54 commits into
masterfrom 
scttcper/rspack
Conversation
@scttcper
@scttcperscttcper
commented
Sep 6, 2024•

edited

Loading

Copy link


Copy Markdown

Member
Building off the work done in #61475 I think the plan would be to use this as a dev-ui replacement until we're more confident and can setup the getsentry repo as well (if you aren't familiar we have another repo with billing pages etc).
Potential adoption roadmap
  1. merge rspack as optional dev only command yarn dev-ui-rspack
  2. make rspack default for dev-ui
  3. use rspack for acceptance tests
  4. build production with rspack
  5. remove webpack
Summary
WebpackRspack
HMR Time4s607ms
Dev Memory Usage3.85 GB2.8 GB
Build Production131.53s39.42s
RSPACK + SENTRY TODO (updated Oct 7th, 2024)
  •  Investigate emotion autoLabel in development mode. Currently set to autoLabel: 'always', Could just set label setting based on node_env.
  •  Add back webpack in order to merge (getsentry repo will block until it can accept a rspack file instead of webpack)
  •  Figure out a path forward for translation string extraction that use webpack + babel's AST make build-js-po
  •  Chartcuterie is failing for some reason
Thanks @hardfist for helping with this PR!
anonrigand others
added 3 commits
December 11, 2023 20:03
@anonrig
build: replace webpack with rspack
6e51e59
@scttcper
Merge branch 'master' into scttcper/rspack
d266cfd
# Conflicts:
#	build-utils/last-built-plugin.ts
#	config/rspack.chartcuterie.config.ts
#	package.json
#	rspack.config.ts
#	yarn.lock
@scttcper
mostly working
859e9fc
@github-actionsgithub-actionsBot
added

Scope: Frontend
Automatically applied to PRs that change frontend components
Scope: Backend
Automatically applied to PRs that change backend components
labels
Sep 6, 2024
@github-actions
github-actionsBot
commented
Sep 6, 2024

Copy link


Copy Markdown

Contributor
🚨 Warning: This pull request contains Frontend and Backend changes!
It's discouraged to make changes to Sentry's Frontend and Backend in a single pull request. The Frontend and Backend are not atomically deployed. If the changes are interdependent of each other, they must be separated into two pull requests and be made forward or backwards compatible, such that the Backend or Frontend can be safely deployed independently.
Have questions? Please ask in the #discuss-dev-infra channel.
@scttcperscttcper
added
the 
WIP

label
Sep 6, 2024
@scttcperscttcper
mentioned this pull request
Sep 6, 2024
 Closed

6 tasks
@scttcper
scttcper
commented
Sep 6, 2024•

edited

Loading

Copy link


Copy Markdown

MemberAuthor
Just realized its working for me locally because i fixed platformicons locally will need to publish a fix to the svg paths there
@hardfisthardfist
mentioned this pull request
Sep 9, 2024

Closed

@scttcper
Merge branch 'master' into scttcper/rspack
e92e6d8
# Conflicts:
#	package.json
#	rspack.config.ts
#	yarn.lock
@scttcper
enable ContextReplacementPlugin
2068906
@codecov
codecovBot
commented
Sep 26, 2024•

edited

Loading

Copy link


Copy Markdown

❌ 1 Tests Failed:
Tests completedFailedPassedSkipped
102901102899
View the full list of 1 ❄️ flaky tests
SentryApplicationDetailsEditinganexistingpublicSentryAppwithascopeerrorhandlesclientsecretrotation
Flake rate in main: 18.18% (Passed 9 times, Failed 2 times)
Stack Traces | 0.405s run time
TestingLibraryElementError: Unabletofindanelementwiththetext: Thiswillbetheonlytimeyourclientsecretisvisible!. Thiscouldbebecausethetextisbrokenupbymultipleelements. Inthiscase, youcanprovideafunctionforyourtextmatchertomakeyourmatchermoreflexible.
Ignorednodes: comments, script, style
...
atObject.getElementError (.../sentry/sentry/node_modules/@.../dom/dist/config.js:37:19)
at .../sentry/sentry/node_modules/@.../dom/dist/query-helpers.js:76:38at .../sentry/sentry/node_modules/@.../dom/dist/query-helpers.js:52:17at .../sentry/sentry/node_modules/@.../dom/dist/query-helpers.js:95:19atObject.<anonymous> (.../settings/organizationDeveloperSettings/sentryApplicationDetails.spec.tsx:591:42)
To view more test analytics, go to the Test Analytics Dashboard
📋 Got 3 mins? Take this short survey to help us improve Test Analytics.
@scttcperscttcper
mentioned this pull request
Sep 27, 2024
 Merged

@scttcper
scttcper
marked this pull request as ready for review
May 9, 2025 20:12
@scttcper
scttcper
requested review from
a team
as code ownersMay 9, 2025 20:12
@scttcper
scttcper
requested a review
from a teamMay 9, 2025 20:12
evanpurkhiser
evanpurkhiser
approved these changes
May 9, 2025
View reviewed changes
@evanpurkhiserevanpurkhiser
left a comment


Copy link


Copy Markdown

Member
Choose a reason for hiding this comment

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

slam it
JonasBa
JonasBa
approved these changes
May 12, 2025
View reviewed changes
@JonasBaJonasBa
left a comment


Copy link


Copy Markdown

Contributor
Choose a reason for hiding this comment

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

Those HMR numbers look 🔥
Comment threadrspack.config.ts
 supported_locales: string[];
};

const localeCatalog: LocaleCatalog = JSON.parse(
@JonasBaJonasBaMay 12, 2025

Copy link


Copy Markdown

Contributor
Choose a reason for hiding this comment

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

Outside of the scope of this PR, but it would be interesting if we could time this locale procedure to see how much it contributes to the startup cost.
@scttcper
scttcper
merged commit 9168894
into
masterMay 15, 2025
@scttcper
scttcper
deleted the

scttcper/rspack

branch
May 15, 2025 16:57
@github-actionsgithub-actionsBot
locked and limited conversation to collaborators
May 31, 2025
Sign up for freeto subscribe to this conversation on GitHub.
Already have an account?
Sign in.


Reviewers

@evanpurkhiserevanpurkhiserevanpurkhiser approved these changes

+1 more reviewer

@JonasBaJonasBaJonasBa approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees


No one assigned


Labels

Scope: BackendAutomatically applied to PRs that change backend componentsScope: FrontendAutomatically applied to PRs that change frontend componentsWIP

Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

4 participants

@scttcper@evanpurkhiser@JonasBa@anonrig

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Auto-enable theater mode on YouTube\n(function() {\n function tryTheater() {\n var btn = document.querySelector('button[aria-label=\"Theater mode\"], ytd-player #player button[title=\"Theater mode\"]');\n if (btn && !btn.classList.contains('activated')) {\n btn.click();\n }\n }\n \n // Try immediately\n tryTheater();\n \n // Try after navigation (SPA)\n var lastUrl = location.href;\n setInterval(function() {\n if (location.href !== lastUrl) {\n lastUrl = location.href;\n setTimeout(tryTheater, 500);\n }\n }, 1000);\n \n // Also try on player load\n var observer = new MutationObserver(tryTheater);\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "YouTube Theater Mode Default");
}
} catch(__e) { console.warn('[Userscript:YouTube Theater Mode Default]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '

Skip to content
build(ui): Add rspack as webpack alternative - #77077
Merged
scttcper merged 54 commits into
masterfrom 
scttcper/rspack
May 15, 2025
Merged
build(ui): Add rspack as webpack alternative#77077
scttcper merged 54 commits into
masterfrom 
scttcper/rspack
Conversation
@scttcper
@scttcperscttcper
commented
Sep 6, 2024•

edited

Loading

Copy link


Copy Markdown

Member
Building off the work done in #61475 I think the plan would be to use this as a dev-ui replacement until we're more confident and can setup the getsentry repo as well (if you aren't familiar we have another repo with billing pages etc).
Potential adoption roadmap
  1. merge rspack as optional dev only command yarn dev-ui-rspack
  2. make rspack default for dev-ui
  3. use rspack for acceptance tests
  4. build production with rspack
  5. remove webpack
Summary
WebpackRspack
HMR Time4s607ms
Dev Memory Usage3.85 GB2.8 GB
Build Production131.53s39.42s
RSPACK + SENTRY TODO (updated Oct 7th, 2024)
  •  Investigate emotion autoLabel in development mode. Currently set to autoLabel: 'always', Could just set label setting based on node_env.
  •  Add back webpack in order to merge (getsentry repo will block until it can accept a rspack file instead of webpack)
  •  Figure out a path forward for translation string extraction that use webpack + babel's AST make build-js-po
  •  Chartcuterie is failing for some reason
Thanks @hardfist for helping with this PR!
anonrigand others
added 3 commits
December 11, 2023 20:03
@anonrig
build: replace webpack with rspack
6e51e59
@scttcper
Merge branch 'master' into scttcper/rspack
d266cfd
# Conflicts:
#	build-utils/last-built-plugin.ts
#	config/rspack.chartcuterie.config.ts
#	package.json
#	rspack.config.ts
#	yarn.lock
@scttcper
mostly working
859e9fc
@github-actionsgithub-actionsBot
added

Scope: Frontend
Automatically applied to PRs that change frontend components
Scope: Backend
Automatically applied to PRs that change backend components
labels
Sep 6, 2024
@github-actions
github-actionsBot
commented
Sep 6, 2024

Copy link


Copy Markdown

Contributor
🚨 Warning: This pull request contains Frontend and Backend changes!
It's discouraged to make changes to Sentry's Frontend and Backend in a single pull request. The Frontend and Backend are not atomically deployed. If the changes are interdependent of each other, they must be separated into two pull requests and be made forward or backwards compatible, such that the Backend or Frontend can be safely deployed independently.
Have questions? Please ask in the #discuss-dev-infra channel.
@scttcperscttcper
added
the 
WIP

label
Sep 6, 2024
@scttcperscttcper
mentioned this pull request
Sep 6, 2024
 Closed

6 tasks
@scttcper
scttcper
commented
Sep 6, 2024•

edited

Loading

Copy link


Copy Markdown

MemberAuthor
Just realized its working for me locally because i fixed platformicons locally will need to publish a fix to the svg paths there
@hardfisthardfist
mentioned this pull request
Sep 9, 2024

Closed

@scttcper
Merge branch 'master' into scttcper/rspack
e92e6d8
# Conflicts:
#	package.json
#	rspack.config.ts
#	yarn.lock
@scttcper
enable ContextReplacementPlugin
2068906
@codecov
codecovBot
commented
Sep 26, 2024•

edited

Loading

Copy link


Copy Markdown

❌ 1 Tests Failed:
Tests completedFailedPassedSkipped
102901102899
View the full list of 1 ❄️ flaky tests
SentryApplicationDetailsEditinganexistingpublicSentryAppwithascopeerrorhandlesclientsecretrotation
Flake rate in main: 18.18% (Passed 9 times, Failed 2 times)
Stack Traces | 0.405s run time
TestingLibraryElementError: Unabletofindanelementwiththetext: Thiswillbetheonlytimeyourclientsecretisvisible!. Thiscouldbebecausethetextisbrokenupbymultipleelements. Inthiscase, youcanprovideafunctionforyourtextmatchertomakeyourmatchermoreflexible.
Ignorednodes: comments, script, style
...
atObject.getElementError (.../sentry/sentry/node_modules/@.../dom/dist/config.js:37:19)
at .../sentry/sentry/node_modules/@.../dom/dist/query-helpers.js:76:38at .../sentry/sentry/node_modules/@.../dom/dist/query-helpers.js:52:17at .../sentry/sentry/node_modules/@.../dom/dist/query-helpers.js:95:19atObject.<anonymous> (.../settings/organizationDeveloperSettings/sentryApplicationDetails.spec.tsx:591:42)
To view more test analytics, go to the Test Analytics Dashboard
📋 Got 3 mins? Take this short survey to help us improve Test Analytics.
@scttcperscttcper
mentioned this pull request
Sep 27, 2024
 Merged

@scttcper
scttcper
marked this pull request as ready for review
May 9, 2025 20:12
@scttcper
scttcper
requested review from
a team
as code ownersMay 9, 2025 20:12
@scttcper
scttcper
requested a review
from a teamMay 9, 2025 20:12
evanpurkhiser
evanpurkhiser
approved these changes
May 9, 2025
View reviewed changes
@evanpurkhiserevanpurkhiser
left a comment


Copy link


Copy Markdown

Member
Choose a reason for hiding this comment

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

slam it
JonasBa
JonasBa
approved these changes
May 12, 2025
View reviewed changes
@JonasBaJonasBa
left a comment


Copy link


Copy Markdown

Contributor
Choose a reason for hiding this comment

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

Those HMR numbers look 🔥
Comment threadrspack.config.ts
 supported_locales: string[];
};

const localeCatalog: LocaleCatalog = JSON.parse(
@JonasBaJonasBaMay 12, 2025

Copy link


Copy Markdown

Contributor
Choose a reason for hiding this comment

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

Outside of the scope of this PR, but it would be interesting if we could time this locale procedure to see how much it contributes to the startup cost.
@scttcper
scttcper
merged commit 9168894
into
masterMay 15, 2025
@scttcper
scttcper
deleted the

scttcper/rspack

branch
May 15, 2025 16:57
@github-actionsgithub-actionsBot
locked and limited conversation to collaborators
May 31, 2025
Sign up for freeto subscribe to this conversation on GitHub.
Already have an account?
Sign in.


Reviewers

@evanpurkhiserevanpurkhiserevanpurkhiser approved these changes

+1 more reviewer

@JonasBaJonasBaJonasBa approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees


No one assigned


Labels

Scope: BackendAutomatically applied to PRs that change backend componentsScope: FrontendAutomatically applied to PRs that change frontend componentsWIP

Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

4 participants

@scttcper@evanpurkhiser@JonasBa@anonrig

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Remove or un-stick sticky/fixed headers that block content\n(function() {\n function unstick() {\n document.querySelectorAll('header, nav, [role=\"banner\"], .header, .navbar, .sticky, .fixed-top, [style*=\"position: fixed\"], [style*=\"position:sticky\"]').forEach(function(el) {\n if (el.style.position === 'fixed' || el.style.position === 'sticky' || \n getComputedStyle(el).position === 'fixed' || getComputedStyle(el).position === 'sticky') {\n el.style.position = 'static';\n el.style.top = 'auto';\n el.style.zIndex = 'auto';\n }\n });\n }\n \n unstick();\n \n var observer = new MutationObserver(unstick);\n observer.observe(document.body, { childList: true, subtree: true, attributes: true, attributeFilter: ['style', 'class'] });\n})();", "Kill Sticky Headers");
}
} catch(__e) { console.warn('[Userscript:Kill Sticky Headers]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '

Skip to content
build(ui): Add rspack as webpack alternative - #77077
Merged
scttcper merged 54 commits into
masterfrom 
scttcper/rspack
May 15, 2025
Merged
build(ui): Add rspack as webpack alternative#77077
scttcper merged 54 commits into
masterfrom 
scttcper/rspack
Conversation
@scttcper
@scttcperscttcper
commented
Sep 6, 2024•

edited

Loading

Copy link


Copy Markdown

Member
Building off the work done in #61475 I think the plan would be to use this as a dev-ui replacement until we're more confident and can setup the getsentry repo as well (if you aren't familiar we have another repo with billing pages etc).
Potential adoption roadmap
  1. merge rspack as optional dev only command yarn dev-ui-rspack
  2. make rspack default for dev-ui
  3. use rspack for acceptance tests
  4. build production with rspack
  5. remove webpack
Summary
WebpackRspack
HMR Time4s607ms
Dev Memory Usage3.85 GB2.8 GB
Build Production131.53s39.42s
RSPACK + SENTRY TODO (updated Oct 7th, 2024)
  •  Investigate emotion autoLabel in development mode. Currently set to autoLabel: 'always', Could just set label setting based on node_env.
  •  Add back webpack in order to merge (getsentry repo will block until it can accept a rspack file instead of webpack)
  •  Figure out a path forward for translation string extraction that use webpack + babel's AST make build-js-po
  •  Chartcuterie is failing for some reason
Thanks @hardfist for helping with this PR!
anonrigand others
added 3 commits
December 11, 2023 20:03
@anonrig
build: replace webpack with rspack
6e51e59
@scttcper
Merge branch 'master' into scttcper/rspack
d266cfd
# Conflicts:
#	build-utils/last-built-plugin.ts
#	config/rspack.chartcuterie.config.ts
#	package.json
#	rspack.config.ts
#	yarn.lock
@scttcper
mostly working
859e9fc
@github-actionsgithub-actionsBot
added

Scope: Frontend
Automatically applied to PRs that change frontend components
Scope: Backend
Automatically applied to PRs that change backend components
labels
Sep 6, 2024
@github-actions
github-actionsBot
commented
Sep 6, 2024

Copy link


Copy Markdown

Contributor
🚨 Warning: This pull request contains Frontend and Backend changes!
It's discouraged to make changes to Sentry's Frontend and Backend in a single pull request. The Frontend and Backend are not atomically deployed. If the changes are interdependent of each other, they must be separated into two pull requests and be made forward or backwards compatible, such that the Backend or Frontend can be safely deployed independently.
Have questions? Please ask in the #discuss-dev-infra channel.
@scttcperscttcper
added
the 
WIP

label
Sep 6, 2024
@scttcperscttcper
mentioned this pull request
Sep 6, 2024
 Closed

6 tasks
@scttcper
scttcper
commented
Sep 6, 2024•

edited

Loading

Copy link


Copy Markdown

MemberAuthor
Just realized its working for me locally because i fixed platformicons locally will need to publish a fix to the svg paths there
@hardfisthardfist
mentioned this pull request
Sep 9, 2024

Closed

@scttcper
Merge branch 'master' into scttcper/rspack
e92e6d8
# Conflicts:
#	package.json
#	rspack.config.ts
#	yarn.lock
@scttcper
enable ContextReplacementPlugin
2068906
@codecov
codecovBot
commented
Sep 26, 2024•

edited

Loading

Copy link


Copy Markdown

❌ 1 Tests Failed:
Tests completedFailedPassedSkipped
102901102899
View the full list of 1 ❄️ flaky tests
SentryApplicationDetailsEditinganexistingpublicSentryAppwithascopeerrorhandlesclientsecretrotation
Flake rate in main: 18.18% (Passed 9 times, Failed 2 times)
Stack Traces | 0.405s run time
TestingLibraryElementError: Unabletofindanelementwiththetext: Thiswillbetheonlytimeyourclientsecretisvisible!. Thiscouldbebecausethetextisbrokenupbymultipleelements. Inthiscase, youcanprovideafunctionforyourtextmatchertomakeyourmatchermoreflexible.
Ignorednodes: comments, script, style
...
atObject.getElementError (.../sentry/sentry/node_modules/@.../dom/dist/config.js:37:19)
at .../sentry/sentry/node_modules/@.../dom/dist/query-helpers.js:76:38at .../sentry/sentry/node_modules/@.../dom/dist/query-helpers.js:52:17at .../sentry/sentry/node_modules/@.../dom/dist/query-helpers.js:95:19atObject.<anonymous> (.../settings/organizationDeveloperSettings/sentryApplicationDetails.spec.tsx:591:42)
To view more test analytics, go to the Test Analytics Dashboard
📋 Got 3 mins? Take this short survey to help us improve Test Analytics.
@scttcperscttcper
mentioned this pull request
Sep 27, 2024
 Merged

@scttcper
scttcper
marked this pull request as ready for review
May 9, 2025 20:12
@scttcper
scttcper
requested review from
a team
as code ownersMay 9, 2025 20:12
@scttcper
scttcper
requested a review
from a teamMay 9, 2025 20:12
evanpurkhiser
evanpurkhiser
approved these changes
May 9, 2025
View reviewed changes
@evanpurkhiserevanpurkhiser
left a comment


Copy link


Copy Markdown

Member
Choose a reason for hiding this comment

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

slam it
JonasBa
JonasBa
approved these changes
May 12, 2025
View reviewed changes
@JonasBaJonasBa
left a comment


Copy link


Copy Markdown

Contributor
Choose a reason for hiding this comment

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

Those HMR numbers look 🔥
Comment threadrspack.config.ts
 supported_locales: string[];
};

const localeCatalog: LocaleCatalog = JSON.parse(
@JonasBaJonasBaMay 12, 2025

Copy link


Copy Markdown

Contributor
Choose a reason for hiding this comment

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

Outside of the scope of this PR, but it would be interesting if we could time this locale procedure to see how much it contributes to the startup cost.
@scttcper
scttcper
merged commit 9168894
into
masterMay 15, 2025
@scttcper
scttcper
deleted the

scttcper/rspack

branch
May 15, 2025 16:57
@github-actionsgithub-actionsBot
locked and limited conversation to collaborators
May 31, 2025
Sign up for freeto subscribe to this conversation on GitHub.
Already have an account?
Sign in.


Reviewers

@evanpurkhiserevanpurkhiserevanpurkhiser approved these changes

+1 more reviewer

@JonasBaJonasBaJonasBa approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees


No one assigned


Labels

Scope: BackendAutomatically applied to PRs that change backend componentsScope: FrontendAutomatically applied to PRs that change frontend componentsWIP

Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

4 participants

@scttcper@evanpurkhiser@JonasBa@anonrig

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Universal Dark Mode - works on any site\n(function() {\n var enabled = true;\n \n function applyDarkMode() {\n if (!enabled) return;\n \n // Create style element if it doesn't exist\n var style = document.getElementById('universal-dark-mode-style');\n if (!style) {\n style = document.createElement('style');\n style.id = 'universal-dark-mode-style';\n document.head.appendChild(style);\n }\n \n // Dark mode CSS - inverts colors but preserves images/video\n style.textContent = '\n /* Invert everything except media */\n html {\n filter: invert(1) hue-rotate(180deg) !important;\n background: #1a1a2e !important;\n }\n \n /* Restore images, videos, iframes, canvas */\n img, video, iframe, canvas, svg, picture, [style*=\"background-image\"] {\n filter: invert(1) hue-rotate(180deg) !important;\n }\n \n /* Preserve specific elements that should not be inverted */\n .no-dark-mode, .no-dark-mode *,\n [data-theme=\"light\"], [data-theme=\"light\"],\n .ace_editor, .ace_editor *,\n .CodeMirror, .CodeMirror *,\n .monaco-editor, .monaco-editor *,\n .markdown-body pre, .markdown-body pre *,\n .highlight, .highlight *,\n pre code, pre code * {\n filter: none !important;\n }\n \n /* Fix common UI elements */\n .modal, .popup, .dropdown-menu, .tooltip, .popover {\n filter: invert(1) hue-rotate(180deg) !important;\n background: #2d2d44 !important;\n border-color: #444 !important;\n }\n \n /* Scrollbars */\n ::-webkit-scrollbar { background: #1a1a2e !important; }\n ::-webkit-scrollbar-thumb { background: #444 !important; }\n ::-webkit-scrollbar-thumb:hover { background: #555 !important; }\n \n /* Selection */\n ::selection { background: #4ecdc4 !important; color: #1a1a2e !important; }\n ::-moz-selection { background: #4ecdc4 !important; color: #1a1a2e !important; }\n ';\n }\n \n function removeDarkMode() {\n var style = document.getElementById('universal-dark-mode-style');\n if (style) style.remove();\n }\n \n // Toggle with Alt+Shift+D\n document.addEventListener('keydown', function(e) {\n if (e.altKey && e.shiftKey && e.key === 'D') {\n e.preventDefault();\n enabled = !enabled;\n if (enabled) {\n applyDarkMode();\n console.log('[Universal Dark Mode] Enabled');\n } else {\n removeDarkMode();\n console.log('[Universal Dark Mode] Disabled');\n }\n }\n });\n \n // Apply on load\n applyDarkMode();\n \n // Re-apply on dynamic content\n var observer = new MutationObserver(function(mutations) {\n if (enabled && !document.getElementById('universal-dark-mode-style')) {\n applyDarkMode();\n }\n });\n observer.observe(document.head, { childList: true });\n \n console.log('[Universal Dark Mode] Loaded - Press Alt+Shift+D to toggle');\n})();", "Universal Dark Mode");
}
} catch(__e) { console.warn('[Userscript:Universal Dark Mode]', __e); }
})();
})();

Skip to content
build(ui): Add rspack as webpack alternative - #77077
Merged
scttcper merged 54 commits into
masterfrom 
scttcper/rspack
May 15, 2025
Merged
build(ui): Add rspack as webpack alternative#77077
scttcper merged 54 commits into
masterfrom 
scttcper/rspack
Conversation
@scttcper
@scttcperscttcper
commented
Sep 6, 2024•

edited

Loading

Copy link


Copy Markdown

Member
Building off the work done in #61475 I think the plan would be to use this as a dev-ui replacement until we're more confident and can setup the getsentry repo as well (if you aren't familiar we have another repo with billing pages etc).
Potential adoption roadmap
  1. merge rspack as optional dev only command yarn dev-ui-rspack
  2. make rspack default for dev-ui
  3. use rspack for acceptance tests
  4. build production with rspack
  5. remove webpack
Summary
WebpackRspack
HMR Time4s607ms
Dev Memory Usage3.85 GB2.8 GB
Build Production131.53s39.42s
RSPACK + SENTRY TODO (updated Oct 7th, 2024)
  •  Investigate emotion autoLabel in development mode. Currently set to autoLabel: 'always', Could just set label setting based on node_env.
  •  Add back webpack in order to merge (getsentry repo will block until it can accept a rspack file instead of webpack)
  •  Figure out a path forward for translation string extraction that use webpack + babel's AST make build-js-po
  •  Chartcuterie is failing for some reason
Thanks @hardfist for helping with this PR!
anonrigand others
added 3 commits
December 11, 2023 20:03
@anonrig
build: replace webpack with rspack
6e51e59
@scttcper
Merge branch 'master' into scttcper/rspack
d266cfd
# Conflicts:
#	build-utils/last-built-plugin.ts
#	config/rspack.chartcuterie.config.ts
#	package.json
#	rspack.config.ts
#	yarn.lock
@scttcper
mostly working
859e9fc
@github-actionsgithub-actionsBot
added

Scope: Frontend
Automatically applied to PRs that change frontend components
Scope: Backend
Automatically applied to PRs that change backend components
labels
Sep 6, 2024
@github-actions
github-actionsBot
commented
Sep 6, 2024

Copy link


Copy Markdown

Contributor
🚨 Warning: This pull request contains Frontend and Backend changes!
It's discouraged to make changes to Sentry's Frontend and Backend in a single pull request. The Frontend and Backend are not atomically deployed. If the changes are interdependent of each other, they must be separated into two pull requests and be made forward or backwards compatible, such that the Backend or Frontend can be safely deployed independently.
Have questions? Please ask in the #discuss-dev-infra channel.
@scttcperscttcper
added
the 
WIP

label
Sep 6, 2024
@scttcperscttcper
mentioned this pull request
Sep 6, 2024
 Closed

6 tasks
@scttcper
scttcper
commented
Sep 6, 2024•

edited

Loading

Copy link


Copy Markdown

MemberAuthor
Just realized its working for me locally because i fixed platformicons locally will need to publish a fix to the svg paths there
@hardfisthardfist
mentioned this pull request
Sep 9, 2024

Closed

@scttcper
Merge branch 'master' into scttcper/rspack
e92e6d8
# Conflicts:
#	package.json
#	rspack.config.ts
#	yarn.lock
@scttcper
enable ContextReplacementPlugin
2068906
@codecov
codecovBot
commented
Sep 26, 2024•

edited

Loading

Copy link


Copy Markdown

❌ 1 Tests Failed:
Tests completedFailedPassedSkipped
102901102899
View the full list of 1 ❄️ flaky tests
SentryApplicationDetailsEditinganexistingpublicSentryAppwithascopeerrorhandlesclientsecretrotation
Flake rate in main: 18.18% (Passed 9 times, Failed 2 times)
Stack Traces | 0.405s run time
TestingLibraryElementError: Unabletofindanelementwiththetext: Thiswillbetheonlytimeyourclientsecretisvisible!. Thiscouldbebecausethetextisbrokenupbymultipleelements. Inthiscase, youcanprovideafunctionforyourtextmatchertomakeyourmatchermoreflexible.
Ignorednodes: comments, script, style
...
atObject.getElementError (.../sentry/sentry/node_modules/@.../dom/dist/config.js:37:19)
at .../sentry/sentry/node_modules/@.../dom/dist/query-helpers.js:76:38at .../sentry/sentry/node_modules/@.../dom/dist/query-helpers.js:52:17at .../sentry/sentry/node_modules/@.../dom/dist/query-helpers.js:95:19atObject.<anonymous> (.../settings/organizationDeveloperSettings/sentryApplicationDetails.spec.tsx:591:42)
To view more test analytics, go to the Test Analytics Dashboard
📋 Got 3 mins? Take this short survey to help us improve Test Analytics.
@scttcperscttcper
mentioned this pull request
Sep 27, 2024
 Merged

@scttcper
scttcper
marked this pull request as ready for review
May 9, 2025 20:12
@scttcper
scttcper
requested review from
a team
as code ownersMay 9, 2025 20:12
@scttcper
scttcper
requested a review
from a teamMay 9, 2025 20:12
evanpurkhiser
evanpurkhiser
approved these changes
May 9, 2025
View reviewed changes
@evanpurkhiserevanpurkhiser
left a comment


Copy link


Copy Markdown

Member
Choose a reason for hiding this comment

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

slam it
JonasBa
JonasBa
approved these changes
May 12, 2025
View reviewed changes
@JonasBaJonasBa
left a comment


Copy link


Copy Markdown

Contributor
Choose a reason for hiding this comment

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

Those HMR numbers look 🔥
Comment threadrspack.config.ts
 supported_locales: string[];
};

const localeCatalog: LocaleCatalog = JSON.parse(
@JonasBaJonasBaMay 12, 2025

Copy link


Copy Markdown

Contributor
Choose a reason for hiding this comment

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

Outside of the scope of this PR, but it would be interesting if we could time this locale procedure to see how much it contributes to the startup cost.
@scttcper
scttcper
merged commit 9168894
into
masterMay 15, 2025
@scttcper
scttcper
deleted the

scttcper/rspack

branch
May 15, 2025 16:57
@github-actionsgithub-actionsBot
locked and limited conversation to collaborators
May 31, 2025
Sign up for freeto subscribe to this conversation on GitHub.
Already have an account?
Sign in.


Reviewers

@evanpurkhiserevanpurkhiserevanpurkhiser approved these changes

+1 more reviewer

@JonasBaJonasBaJonasBa approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees


No one assigned


Labels

Scope: BackendAutomatically applied to PRs that change backend componentsScope: FrontendAutomatically applied to PRs that change frontend componentsWIP

Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

4 participants

@scttcper@evanpurkhiser@JonasBa@anonrig