chore: re-export shared errors (#16356)

Rich-Harris · web-flow · commit 61f75651d6a4 · 2025-07-13T12:48:08.000-04:00
diff --git a/packages/svelte/scripts/process-messages/templates/client-errors.js b/packages/svelte/scripts/process-messages/templates/client-errors.js
@@ -1,5 +1,7 @@
 import { DEV } from 'esm-env';
 
+export * from '../shared/errors.js';
+
 /**
  * MESSAGE
  * @param {string} PARAMETER
diff --git a/packages/svelte/scripts/process-messages/templates/server-errors.js b/packages/svelte/scripts/process-messages/templates/server-errors.js
@@ -1,3 +1,5 @@
+export * from '../shared/errors.js';
+
 /**
  * MESSAGE
  * @param {string} PARAMETER
diff --git a/packages/svelte/src/index-client.js b/packages/svelte/src/index-client.js
@@ -5,7 +5,6 @@ import { active_reaction, untrack } from './internal/client/runtime.js';
 import { is_array } from './internal/shared/utils.js';
 import { user_effect } from './internal/client/index.js';
 import * as e from './internal/client/errors.js';
-import { lifecycle_outside_component } from './internal/shared/errors.js';
 import { legacy_mode_flag } from './internal/flags/index.js';
 import { component_context } from './internal/client/context.js';
 import { DEV } from 'esm-env';
@@ -91,7 +90,7 @@ export function getAbortSignal() {
  */
 export function onMount(fn) {
 	if (component_context === null) {
-		lifecycle_outside_component('onMount');
+		e.lifecycle_outside_component('onMount');
 	}
 
 	if (legacy_mode_flag && component_context.l !== null) {
@@ -115,7 +114,7 @@ export function onMount(fn) {
  */
 export function onDestroy(fn) {
 	if (component_context === null) {
-		lifecycle_outside_component('onDestroy');
+		e.lifecycle_outside_component('onDestroy');
 	}
 
 	onMount(() => () => untrack(fn));
@@ -158,7 +157,7 @@ function create_custom_event(type, detail, { bubbles = false, cancelable = false
 export function createEventDispatcher() {
 	const active_component_context = component_context;
 	if (active_component_context === null) {
-		lifecycle_outside_component('createEventDispatcher');
+		e.lifecycle_outside_component('createEventDispatcher');
 	}
 
 	return (type, detail, options) => {
@@ -196,7 +195,7 @@ export function createEventDispatcher() {
  */
 export function beforeUpdate(fn) {
 	if (component_context === null) {
-		lifecycle_outside_component('beforeUpdate');
+		e.lifecycle_outside_component('beforeUpdate');
 	}
 
 	if (component_context.l === null) {
@@ -219,7 +218,7 @@ export function beforeUpdate(fn) {
  */
 export function afterUpdate(fn) {
 	if (component_context === null) {
-		lifecycle_outside_component('afterUpdate');
+		e.lifecycle_outside_component('afterUpdate');
 	}
 
 	if (component_context.l === null) {
diff --git a/packages/svelte/src/internal/client/context.js b/packages/svelte/src/internal/client/context.js
@@ -1,7 +1,7 @@
 /** @import { ComponentContext, DevStackEntry } from '#client' */
 
 import { DEV } from 'esm-env';
-import { lifecycle_outside_component } from '../shared/errors.js';
+import * as e from './errors.js';
 import { source } from './reactivity/sources.js';
 import {
 	active_effect,
@@ -205,7 +205,7 @@ export function is_runes() {
  */
 function get_or_init_context_map(name) {
 	if (component_context === null) {
-		lifecycle_outside_component(name);
+		e.lifecycle_outside_component(name);
 	}
 
 	return (component_context.c ??= new Map(get_parent_context(component_context) || undefined));
diff --git a/packages/svelte/src/internal/client/dev/validation.js b/packages/svelte/src/internal/client/dev/validation.js
@@ -1,15 +1,16 @@
-import { invalid_snippet_arguments } from '../../shared/errors.js';
+import * as e from '../errors.js';
 /**
  * @param {Node} anchor
  * @param {...(()=>any)[]} args
  */
 export function validate_snippet_args(anchor, ...args) {
 	if (typeof anchor !== 'object' || !(anchor instanceof Node)) {
-		invalid_snippet_arguments();
+		e.invalid_snippet_arguments();
 	}
+
 	for (let arg of args) {
 		if (typeof arg !== 'function') {
-			invalid_snippet_arguments();
+			e.invalid_snippet_arguments();
 		}
 	}
 }
diff --git a/packages/svelte/src/internal/client/errors.js b/packages/svelte/src/internal/client/errors.js
@@ -2,6 +2,8 @@
 
 import { DEV } from 'esm-env';
 
+export *  from '../shared/errors.js';
+
 /**
  * Using `bind:value` together with a checkbox input is not allowed. Use `bind:checked` instead
  * @returns {never}
diff --git a/packages/svelte/src/internal/server/context.js b/packages/svelte/src/internal/server/context.js
@@ -1,7 +1,7 @@
 /** @import { Component } from '#server' */
 import { DEV } from 'esm-env';
 import { on_destroy } from './index.js';
-import * as e from '../shared/errors.js';
+import * as e from './errors.js';
 
 /** @type {Component | null} */
 export var current_component = null;
diff --git a/packages/svelte/src/internal/server/dev.js b/packages/svelte/src/internal/server/dev.js
@@ -5,7 +5,7 @@ import {
 	is_tag_valid_with_parent
 } from '../../html-tree-validation.js';
 import { current_component } from './context.js';
-import { invalid_snippet_arguments } from '../shared/errors.js';
+import * as e from './errors.js';
 import { HeadPayload, Payload } from './payload.js';
 
 /**
@@ -102,6 +102,6 @@ export function validate_snippet_args(payload) {
 		// for some reason typescript consider the type of payload as never after the first instanceof
 		!(payload instanceof Payload || /** @type {any} */ (payload) instanceof HeadPayload)
 	) {
-		invalid_snippet_arguments();
+		e.invalid_snippet_arguments();
 	}
 }
diff --git a/packages/svelte/src/internal/server/errors.js b/packages/svelte/src/internal/server/errors.js
@@ -1,6 +1,6 @@
 /* This file is generated by scripts/process-messages/index.js. Do not edit! */
 
-
+export *  from '../shared/errors.js';
 
 /**
  * `%name%(...)` is not available on the server
diff --git a/packages/svelte/src/legacy/legacy-client.js b/packages/svelte/src/legacy/legacy-client.js
@@ -4,8 +4,8 @@ import { user_pre_effect } from '../internal/client/reactivity/effects.js';
 import { mutable_source, set } from '../internal/client/reactivity/sources.js';
 import { hydrate, mount, unmount } from '../internal/client/render.js';
 import { active_effect, flushSync, get, set_signal_status } from '../internal/client/runtime.js';
-import { lifecycle_outside_component } from '../internal/shared/errors.js';
 import { define_property, is_array } from '../internal/shared/utils.js';
+import * as e from '../internal/client/errors.js';
 import * as w from '../internal/client/warnings.js';
 import { DEV } from 'esm-env';
 import { FILENAME } from '../constants.js';
@@ -245,7 +245,7 @@ export function handlers(...handlers) {
 export function createBubbler() {
 	const active_component_context = component_context;
 	if (active_component_context === null) {
-		lifecycle_outside_component('createBubbler');
+		e.lifecycle_outside_component('createBubbler');
 	}
 
 	return (/**@type {string}*/ type) => (/**@type {Event}*/ event) => {

Original file line number	Diff line number	Diff line change
`@@ -1,3 +1,5 @@`
	`1`	`+export * from '../shared/errors.js';`
	`2`	`+`
`1`	`3`	`/**`
`2`	`4`	`* MESSAGE`
`3`	`5`	`* @param {string} PARAMETER`
Original file line number	Diff line number	Diff line change
`@@ -1,15 +1,16 @@`
`1`		`-import { invalid_snippet_arguments } from '../../shared/errors.js';`
	`1`	`+import * as e from '../errors.js';`
`2`	`2`	`/**`
`3`	`3`	`* @param {Node} anchor`
`4`	`4`	`* @param {...(()=>any)[]} args`
`5`	`5`	`*/`
`6`	`6`	`export function validate_snippet_args(anchor, ...args) {`
`7`	`7`	`if (typeof anchor !== 'object' \|\| !(anchor instanceof Node)) {`
`8`		`- invalid_snippet_arguments();`
	`8`	`+ e.invalid_snippet_arguments();`
`9`	`9`	`}`
	`10`	`+`
`10`	`11`	`for (let arg of args) {`
`11`	`12`	`if (typeof arg !== 'function') {`
`12`		`- invalid_snippet_arguments();`
	`13`	`+ e.invalid_snippet_arguments();`
`13`	`14`	`}`
`14`	`15`	`}`
`15`	`16`	`}`