Skip to content

Conversation

logaretm
Copy link
Collaborator

@logaretm logaretm commented Sep 30, 2025

What

Fixes #17781 by adding the sentry.config.client.ts to the auto generated tsconfig by extending the types via the prepare:types hook.

This allows useRuntimeConfig to be properly typed in the root sentry.client.config.ts, or where ever the client file is found.

Not sure how this could be tested tho, since it is purely types.


Note

Adds a prepare:types hook to include the client Sentry config in Nuxt’s generated tsconfig for proper typing.

  • Nuxt module (packages/nuxt/src/module.ts):
    • Types setup: Add prepare:types hook to ensure tsconfig.include contains the relative path to the detected client Sentry config (sentry.client.config.ts), enabling proper typing (e.g., useRuntimeConfig) in root config files for Nuxt v4.

Written by Cursor Bugbot for commit 4de6d43. This will update automatically on new commits. Configure here.

Copy link

linear bot commented Sep 30, 2025

@logaretm logaretm marked this pull request as ready for review September 30, 2025 12:29
@logaretm logaretm requested a review from s1gr1d September 30, 2025 12:30
cursor[bot]

This comment was marked as outdated.

cursor[bot]

This comment was marked as outdated.

Copy link
Member

@s1gr1d s1gr1d left a comment

Choose a reason for hiding this comment

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

Nice 🎉

@logaretm logaretm merged commit 64ef553 into develop Sep 30, 2025
38 checks passed
@logaretm logaretm deleted the awad/js-988-nuxt-4-sentryclientconfigts-types branch September 30, 2025 14:01
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.

[Nuxt 4] sentry.client.config.ts should be app/sentry[.client].config.ts
2 participants