optionally load tracing, add remote functions

Author: elliott-with-the-longest-name-on-github

packages/kit/src/exports/vite/dev/index.js

@@ -1,4 +1,4 @@
-import fs from 'node:fs';
+import fs, { existsSync } from 'node:fs';
 import path from 'node:path';
 import process from 'node:process';
 import { URL } from 'node:url';
@@ -502,7 +502,9 @@ export async function dev(vite, vite_config, svelte_config) {
 					return;
 				}
 
-				await vite.ssrLoadModule(tracing.server);
+				if (existsSync(tracing.server)) {
+					await vite.ssrLoadModule(tracing.server);
+				}
 
 				// we have to import `Server` before calling `set_assets`
 				const { Server } = /** @type {import('types').ServerModule} */ (

packages/kit/src/runtime/server/remote.js

@@ -12,14 +12,32 @@ import { normalize_error } from '../../utils/error.js';
 import { check_incorrect_fail_use } from './page/actions.js';
 import { DEV } from 'esm-env';
 import { get_event_state } from './event-state.js';
+import { record_span } from '../telemetry/record_span.js';
+import { merge_tracing } from '../utils.js';
+
+/** @type {typeof handle_remote_call_internal} */
+export async function handle_remote_call(event, options, manifest, id) {
+	return record_span({
+		name: 'sveltekit.remote.call',
+		attributes: {
+			'sveltekit.remote.call.id': id
+		},
+		fn: async (current) => {
+			const traced_event = merge_tracing(event, current);
+			return with_event(traced_event, () =>
+				handle_remote_call_internal(event, options, manifest, id)
+			);
+		}
+	});
+}
 
 /**
  * @param {RequestEvent} event
  * @param {SSROptions} options
  * @param {SSRManifest} manifest
  * @param {string} id
  */
-export async function handle_remote_call(event, options, manifest, id) {
+export async function handle_remote_call_internal(event, options, manifest, id) {
 	const [hash, name, prerender_args] = id.split('/');
 	const remotes = manifest._.remotes;
 
@@ -155,13 +173,27 @@ export async function handle_remote_call(event, options, manifest, id) {
 	}
 }
 
+/** @type {typeof handle_remote_form_post_internal} */
+export async function handle_remote_form_post(event, manifest, id) {
+	return record_span({
+		name: 'sveltekit.remote.form.post',
+		attributes: {
+			'sveltekit.remote.form.post.id': id
+		},
+		fn: async (current) => {
+			const traced_event = merge_tracing(event, current);
+			return with_event(traced_event, () => handle_remote_form_post_internal(event, manifest, id));
+		}
+	});
+}
+
 /**
  * @param {RequestEvent} event
  * @param {SSRManifest} manifest
  * @param {string} id
  * @returns {Promise<ActionResult>}
  */
-export async function handle_remote_form_post(event, manifest, id) {
+async function handle_remote_form_post_internal(event, manifest, id) {
 	const [hash, name, action_id] = id.split('/');
 	const remotes = manifest._.remotes;
 	const module = await remotes[hash]?.();