getsentry / getsentry/sentry-javascript

False positive pushstate warnings with SvelteKit

Abierto
#13,823 8 comentarios 0 reacciones 0 asignados Ver en GitHub
Improvement SvelteKit
Lenguaje dominante
TypeScript
Estrellas
8.7k
Forks
1.8k
Merge medio
1 d 17 h
PR fusionados (30 d)
515

Descripción

### Is there an existing issue for this?

- [x] I have checked for existing issues https://github.com/getsentry/sentry-javascript/issues
- [x] I have reviewed the documentation https://docs.sentry.io/
- [x] I am using the latest SDK release https://github.com/getsentry/sentry-javascript/releases

### How do you use Sentry?

Sentry Saas (sentry.io)

### Which SDK are you using?

@sentry/sveltekit

### SDK Version

8.32.0

### Framework Version

sveltekit 2.5.28

### Link to Sentry event

_No response_

### Reproduction Example/SDK Setup

https://github.com/sveltejs/kit/issues/12177

### Steps to Reproduce

Just start a sveltekit app with sentry and on first page load you will see the warning.

seems like initiating here `// node_modules/@sentry-internal/browser-utils/build/esm/instrument/history.js`

`[Warning] Avoid using `history.pushState(...)` and `history.replaceState(...)` as these will conflict with SvelteKit's router. Use the `pushState` and `replaceState` imports from `$app/navigation` instead. (client.js, line xxx)`

### Expected Result

No wanning while using svelte app navigation.

### Actual Result

![Image](https://github.com/user-attachments/assets/1fe4c303-3e7b-4619-8caf-dd9b7a92d43b)

`[Warning] Avoid using `history.pushState(...)` and `history.replaceState(...)` as these will conflict with SvelteKit's router. Use the `pushState` and `replaceState` imports from `$app/navigation` instead. (client.js, line xxx)`

Guía de contribución

Abrir la guía de contribución

Evaluación

Este issue todavía no se ha evaluado.

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.