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: Setup Spotlight integration for local dev #3787

Merged

Conversation

nicholas-codecov
Copy link
Contributor

Description

Did some exploring with @BYK utilizing Spotlight, and since it was an easy add I thought I'd commit it, with some other updates for Sentry as well.

Notable Changes

  • Conditionally add Spotlight integration into Sentry integrations list
  • Add document policy header for dev and preview modes in Vite
  • Update to latest version of Sentry JS SDK and Sentry Vite plugin

Copy link

codecov bot commented Mar 3, 2025

Bundle Report

Changes will increase total bundle size by 6.12kB (0.05%) ⬆️. This is within the configured threshold ✅

Detailed changes
Bundle name Size Change
gazebo-production-system 6.13MB 2.9kB (0.05%) ⬆️
gazebo-production-esm 6.2MB 3.22kB (0.05%) ⬆️

Affected Assets, Files, and Routes:

view changes for bundle: gazebo-production-esm

Assets Changed:

Asset Name Size Change Total Size Change (%)
assets/index.*.js 48 bytes 638.16kB 0.01%
assets/vendor_sentry.*.js 3.17kB 315.17kB 1.02%

Files in assets/index.*.js:

  • ./src/sentry.ts → Total Size: 5.4kB
view changes for bundle: gazebo-production-system

Assets Changed:

Asset Name Size Change Total Size Change (%)
assets/index-legacy.*.js 51 bytes 716.99kB 0.01%
assets/vendor_sentry-legacy.*.js 2.85kB 304.72kB 0.94%

Files in assets/index-legacy.*.js:

  • ./src/sentry.ts → Total Size: 5.41kB

@codecov-staging
Copy link

codecov-staging bot commented Mar 3, 2025

Bundle Report

Changes will increase total bundle size by 6.12kB (0.05%) ⬆️. This is within the configured threshold ✅

Detailed changes
Bundle name Size Change
gazebo-staging-system 6.13MB 2.9kB (0.05%) ⬆️
gazebo-staging-esm 6.2MB 3.22kB (0.05%) ⬆️

Affected Assets, Files, and Routes:

view changes for bundle: gazebo-staging-system

Assets Changed:

Asset Name Size Change Total Size Change (%)
assets/index-legacy.*.js 51 bytes 716.99kB 0.01%
assets/vendor_sentry-legacy.*.js 2.85kB 304.72kB 0.94%

Files in assets/index-legacy.*.js:

  • ./src/sentry.ts → Total Size: 5.41kB
view changes for bundle: gazebo-staging-esm

Assets Changed:

Asset Name Size Change Total Size Change (%)
assets/index.*.js 48 bytes 638.16kB 0.01%
assets/vendor_sentry.*.js 3.17kB 315.17kB 1.02%

Files in assets/index.*.js:

  • ./src/sentry.ts → Total Size: 5.4kB

@codecov-notifications
Copy link

codecov-notifications bot commented Mar 3, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

✅ All tests successful. No failed tests found.

@@           Coverage Diff           @@
##             main    #3787   +/-   ##
=======================================
  Coverage   98.74%   98.74%           
=======================================
  Files         826      826           
  Lines       14947    14947           
  Branches     4256     4264    +8     
=======================================
  Hits        14759    14759           
  Misses        181      181           
  Partials        7        7           
Components Coverage Δ
Assets 100.00% <ø> (ø)
Layouts 99.69% <ø> (ø)
Pages 98.31% <ø> (ø)
Services 99.38% <ø> (ø)
Shared 99.12% <ø> (ø)
UI 99.07% <ø> (ø)

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 39c5c50...2a61a2f. Read the comment docs.

Copy link

codecov bot commented Mar 3, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 98.74%. Comparing base (39c5c50) to head (2a61a2f).
Report is 1 commits behind head on main.

✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #3787   +/-   ##
=======================================
  Coverage   98.74%   98.74%           
=======================================
  Files         826      826           
  Lines       14947    14947           
  Branches     4256     4256           
=======================================
  Hits        14759    14759           
  Misses        181      181           
  Partials        7        7           
Components Coverage Δ
Assets 100.00% <ø> (ø)
Layouts 99.69% <ø> (ø)
Pages 98.31% <ø> (ø)
Services 99.38% <ø> (ø)
Shared 99.12% <ø> (ø)
UI 99.07% <ø> (ø)

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 39c5c50...2a61a2f. Read the comment docs.

Copy link

codecov-public-qa bot commented Mar 3, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 98.74%. Comparing base (39c5c50) to head (2a61a2f).
Report is 1 commits behind head on main.

✅ All tests successful. No failed tests found.

@@           Coverage Diff           @@
##             main    #3787   +/-   ##
=======================================
  Coverage   98.74%   98.74%           
=======================================
  Files         826      826           
  Lines       14947    14947           
  Branches     4264     4264           
=======================================
  Hits        14759    14759           
  Misses        181      181           
  Partials        7        7           
Components Coverage Δ
Assets 100.00% <ø> (ø)
Layouts 99.69% <ø> (ø)
Pages 98.31% <ø> (ø)
Services 99.38% <ø> (ø)
Shared 99.12% <ø> (ø)
UI 99.07% <ø> (ø)

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 39c5c50...2a61a2f. Read the comment docs.

@codecov-releaser
Copy link
Contributor

✅ Deploy preview for gazebo ready!

Previews expire after 1 month automatically.

Storybook

Commit Created Cloud Enterprise
2a61a2f Mon, 03 Mar 2025 15:11:08 GMT Cloud Enterprise

@codecov-qa
Copy link

codecov-qa bot commented Mar 3, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 98.74%. Comparing base (39c5c50) to head (2a61a2f).
Report is 1 commits behind head on main.

✅ All tests successful. No failed tests found.

@@           Coverage Diff           @@
##             main    #3787   +/-   ##
=======================================
  Coverage   98.74%   98.74%           
=======================================
  Files         826      826           
  Lines       14947    14947           
  Branches     4256     4256           
=======================================
  Hits        14759    14759           
  Misses        181      181           
  Partials        7        7           
Components Coverage Δ
Assets 100.00% <ø> (ø)
Layouts 99.69% <ø> (ø)
Pages 98.31% <ø> (ø)
Services 99.38% <ø> (ø)
Shared 99.12% <ø> (ø)
UI 99.07% <ø> (ø)

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 39c5c50...2a61a2f. Read the comment docs.

@@ -59,9 +59,15 @@ export default defineConfig((config) => {
return {
server: {
port: 3000,
headers: {
Copy link
Contributor

Choose a reason for hiding this comment

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

haven't worked with vite config before so just sanity checking: this doesn't end up in prod OR it does and we want it to, right?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

So this isn't a worry cause we don't use Vite, to serve anything in prod

Copy link
Contributor Author

Choose a reason for hiding this comment

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

And if we did ... we'd want this value being set 😄

Copy link
Contributor

Choose a reason for hiding this comment

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

Oh good point, a file being served can't add headers

Copy link
Contributor

Choose a reason for hiding this comment

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

Based on your second comment, should we add this header to our file server config?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

That's been done for a hot minute, this was just something extra so we could collect profiling info in spotlight from a local instance

Copy link
Contributor

Choose a reason for hiding this comment

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

Sick okie doke

Copy link
Contributor

@spalmurray-codecov spalmurray-codecov left a comment

Choose a reason for hiding this comment

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

one q. lgtm

@nicholas-codecov nicholas-codecov added this pull request to the merge queue Mar 3, 2025
Merged via the queue into main with commit 51de4f0 Mar 3, 2025
62 checks passed
@nicholas-codecov nicholas-codecov deleted the feat-setup-spotlight-integration-for-local-dev branch March 3, 2025 15:45
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.

3 participants