-
-
Couldn't load subscription status.
- Fork 4.7k
fix: less confusing names for inspect errors #17026
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
Conversation
🦋 Changeset detectedLatest commit: 145bbad The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
|
Is it possible to also get some config flag to turn stack traces off with regular Writing |
|
Tested in Chrome and Firefox on windows, works. This must be new. |
Please, Svelte team, look at this. |
|
Could we do something with console.group to appease the "I don't want stack traces by default" crowd? |

We use error names like
UpdatedAtErrorwhen inspecting things, apparently because you need anErrorsuffix for the stack trace to be printed sensibly. But I'm trying this locally and... it seems like that's not the case? This works fine:Firefox also has the same behaviour with or without the suffix. Safari does its own thing, naturally — formats the stack traces totally differently, and doesn't respect
stackTraceLimitso the result would be pretty useless anyway. So I also added a check to prevent us from just loggingnull. (Have I ever mentioned that Safari is bad?)Caveat: it's possible that something different happens on Windows? idk man.
Closes #17014.
Before submitting the PR, please make sure you do the following
feat:,fix:,chore:, ordocs:.packages/svelte/src, add a changeset (npx changeset).Tests and linting
pnpm testand lint the project withpnpm lint