@selvajs/ui 4.3.0 → 4.4.0

package/src/lib/schema/param-exporter.test.ts

@@ -0,0 +1,136 @@
+import { describe, expect, it } from 'vitest';
+import { validateSavedState, extractLoadableValues, loadPreset } from './param-exporter';
+import type { UISchema, ParameterPreset } from '@selvajs/schemas';
+
+// These tests cover the load path a user hits when restoring a preset against a
+// schema that has drifted. The severity of each mismatch decides whether the
+// preset can load at all (error blocks, warning allows), so those are the cases
+// worth locking in — not the happy path.
+
+function schema(over: Partial<UISchema> = {}): UISchema {
+	return {
+		id: 'schema-1',
+		documentId: 'doc-1',
+		inputs: [
+			{ id: 'a', nickname: 'A', paramType: 'number' },
+			{ id: 'b', nickname: 'B', paramType: 'number' }
+		],
+		...over
+	} as unknown as UISchema;
+}
+
+function preset(over: Partial<ParameterPreset> = {}): ParameterPreset {
+	return {
+		id: 'p1',
+		name: 'My Preset',
+		schemaId: 'schema-1',
+		documentId: 'doc-1',
+		parameters: [
+			{ paramId: 'a', nickname: 'A', value: 1 },
+			{ paramId: 'b', nickname: 'B', value: 2 }
+		],
+		...over
+	} as unknown as ParameterPreset;
+}
+
+describe('validateSavedState', () => {
+	it('document-ID mismatch is a blocking error', () => {
+		const result = validateSavedState(preset({ documentId: 'other-doc' }), schema());
+		expect(result.canLoad).toBe(false);
+		expect(result.issues.some((i) => i.severity === 'error' && i.paramId === '__document__')).toBe(
+			true
+		);
+	});
+
+	it('schema-ID change is a warning that still allows loading', () => {
+		const result = validateSavedState(preset({ schemaId: 'old-schema' }), schema());
+		expect(result.canLoad).toBe(true);
+		expect(result.isValid).toBe(false);
+		expect(result.issues.some((i) => i.severity === 'warning' && i.paramId === '__schema__')).toBe(
+			true
+		);
+	});
+
+	it('a parameter missing from the schema is a blocking error', () => {
+		const p = preset({
+			parameters: [
+				{ paramId: 'a', nickname: 'A', value: 1 },
+				{ paramId: 'gone', nickname: 'Gone', value: 9 }
+			] as ParameterPreset['parameters']
+		});
+		const result = validateSavedState(p, schema());
+		expect(result.canLoad).toBe(false);
+		expect(result.issues.some((i) => i.severity === 'error' && i.paramId === 'gone')).toBe(true);
+	});
+
+	it('a renamed nickname is a warning, not a block', () => {
+		const renamed = schema({
+			inputs: [
+				{ id: 'a', nickname: 'A renamed', paramType: 'number' },
+				{ id: 'b', nickname: 'B', paramType: 'number' }
+			]
+		} as unknown as Partial<UISchema>);
+		const result = validateSavedState(preset(), renamed);
+		expect(result.canLoad).toBe(true);
+		expect(result.issues.some((i) => i.severity === 'warning' && i.paramId === 'a')).toBe(true);
+	});
+
+	it('a fully matching preset is valid with no issues', () => {
+		const result = validateSavedState(preset(), schema());
+		expect(result).toEqual({ isValid: true, issues: [], canLoad: true });
+	});
+});
+
+describe('extractLoadableValues', () => {
+	it('drops params that no longer exist in the schema, keeps the rest', () => {
+		const p = preset({
+			parameters: [
+				{ paramId: 'a', nickname: 'A', value: 1 },
+				{ paramId: 'gone', nickname: 'Gone', value: 9 } // not in schema
+			] as ParameterPreset['parameters']
+		});
+		expect(extractLoadableValues(p, schema())).toEqual({ a: 1 });
+	});
+
+	it('keeps a warned-but-valid param (nickname drift) on load', () => {
+		const renamed = schema({
+			inputs: [
+				{ id: 'a', nickname: 'A renamed', paramType: 'number' },
+				{ id: 'b', nickname: 'B', paramType: 'number' }
+			]
+		} as unknown as Partial<UISchema>);
+		expect(extractLoadableValues(preset(), renamed)).toEqual({ a: 1, b: 2 });
+	});
+});
+
+describe('loadPreset', () => {
+	it('fuses validation and extraction: valid preset loads cleanly', () => {
+		const result = loadPreset(preset(), schema());
+		expect(result.isValid).toBe(true);
+		expect(result.canLoad).toBe(true);
+		expect(result.issues).toEqual([]);
+		expect(result.values).toEqual({ a: 1, b: 2 });
+	});
+
+	it('blocks on a document mismatch but still returns loadable values', () => {
+		const result = loadPreset(preset({ documentId: 'other-doc' }), schema());
+		expect(result.canLoad).toBe(false);
+		expect(result.issues.some((i) => i.severity === 'error')).toBe(true);
+		// values are still computed (the caller decides whether to apply them)
+		expect(result.values).toEqual({ a: 1, b: 2 });
+	});
+
+	it('allows load with warnings (schema drift) and drops missing params', () => {
+		const p = preset({
+			parameters: [
+				{ paramId: 'a', nickname: 'A', value: 1 },
+				{ paramId: 'gone', nickname: 'Gone', value: 9 }
+			] as ParameterPreset['parameters'],
+			schemaId: 'old-schema'
+		});
+		const result = loadPreset(p, schema());
+		expect(result.isValid).toBe(false);
+		expect(result.canLoad).toBe(false); // missing param is an error
+		expect(result.values).toEqual({ a: 1 });
+	});
+});

package/src/lib/schema/param-exporter.ts

@@ -1,5 +1,5 @@
-import { APP_DEFAULTS } from '../constants';
 import { randomId } from '../utils/randomId';
+import { getGroups } from '@selvajs/schemas';
 import type {
 	UISchema,
 	ParameterPreset,
@@ -7,13 +7,6 @@ import type {
 	ValidationIssueMessage
 } from '@selvajs/schemas';
 
-function getGroups(schema: UISchema) {
-	if (!schema.layout) return [];
-	if (schema.layout.type === 'tabbed') return schema.layout.tabs.flatMap((t) => t.groups ?? []);
-	if (schema.layout.type === 'flat') return schema.layout.groups ?? [];
-	return [];
-}
-
 export function createSavedState(
 	schema: UISchema,
 	currentValues: Record<string, unknown>,
@@ -113,43 +106,64 @@ export function validateSavedState(
 	return { isValid: issues.length === 0, issues, canLoad: !hasErrors };
 }
 
+/**
+ * The subset of a preset's parameters that can safely be applied to the current schema:
+ * every param that still exists as an input. A param that no longer exists is dropped
+ * (it's the same condition `validateSavedState` flags as an error), so this derives its
+ * own "loadable" rule from the schema rather than re-reading validation issues — no
+ * coupling to how sentinel ids are encoded.
+ */
 export function extractLoadableValues(
 	savedState: ParameterPreset,
-	currentSchema: UISchema,
-	validation: ReturnType<typeof validateSavedState>
+	currentSchema: UISchema
 ): Record<string, unknown> {
-	const errorParamIds = new Set(
-		validation.issues
-			.filter((i) => i.severity === 'error' && !i.paramId.startsWith('__'))
-			.map((i) => i.paramId)
-	);
-
 	const values: Record<string, unknown> = {};
 	for (const paramState of savedState.parameters) {
 		const exists = currentSchema.inputs.some((i) => i.id === paramState.paramId);
-		if (!errorParamIds.has(paramState.paramId) && exists) {
+		if (exists) {
 			values[paramState.paramId] = paramState.value;
 		}
 	}
 	return values;
 }
 
+/** The full outcome of loading a preset: what to apply, plus the diagnostics to surface. */
+export interface PresetLoadResult {
+	/** Values safe to apply now (params that still exist). */
+	values: Record<string, unknown>;
+	/** All validation issues (errors + warnings) for the load dialog. */
+	issues: ValidationIssueMessage[];
+	/** No issues at all — load silently. */
+	isValid: boolean;
+	/** No blocking errors — load is allowed (warnings are fine). */
+	canLoad: boolean;
+}
+
+/**
+ * Single entry point for applying a preset: validates against the current schema and, in
+ * the same pass, computes the loadable values. Callers get one object instead of
+ * threading a validation result back into a separate extraction step.
+ */
+export function loadPreset(savedState: ParameterPreset, currentSchema: UISchema): PresetLoadResult {
+	const validation = validateSavedState(savedState, currentSchema);
+	return {
+		values: extractLoadableValues(savedState, currentSchema),
+		issues: validation.issues,
+		isValid: validation.isValid,
+		canLoad: validation.canLoad
+	};
+}
+
 export function exportStateAsJson(savedState: ParameterPreset): void {
 	const blob = new Blob([JSON.stringify(savedState, null, 2)], { type: 'application/json' });
-	const url = URL.createObjectURL(blob);
+	const safeName = savedState.name.replace(/[^a-z0-9]/gi, '_');
+	const date = savedState.timestamp.split('T')[0].replace(/-/g, '_');
+
 	const link = document.createElement('a');
-	link.href = url;
-	link.download = `${savedState.name.replace(/[^a-z0-9]/gi, '_')}_${savedState.timestamp.split('T')[0].replace(/-/g, '_')}.sps`;
-	link.style.display = 'none';
-	document.body.appendChild(link);
-
-	setTimeout(() => {
-		link.click();
-		setTimeout(() => {
-			document.body.removeChild(link);
-			URL.revokeObjectURL(url);
-		}, APP_DEFAULTS.TIMEOUTS.PARAM_EXPORT_DELAY);
-	}, 0);
+	link.href = URL.createObjectURL(blob);
+	link.download = `${safeName}_${date}.sps`;
+	link.click();
+	URL.revokeObjectURL(link.href);
 }
 
 export async function importStateFromJson(file: File): Promise<ParameterPreset> {

package/src/lib/schema/traversal.test.ts

@@ -0,0 +1,92 @@
+import { describe, expect, it } from 'vitest';
+import { getGroups, getLayoutItems, getInputItems } from '@selvajs/schemas';
+import type { UISchema } from '@selvajs/schemas';
+
+// These pin the layout-union discrimination (tabbed vs flat) and the defensive contract
+// (missing layout / groups / items yield empty, never throw), since this module is the
+// single place every caller relies on to walk a schema.
+
+const input = (id: string, source?: { kind: string }) =>
+	({ type: 'input', paramId: id, displayName: id, ...(source ? { source } : {}) }) as never;
+const linebreak = (id: string) => ({ type: 'linebreak', id }) as never;
+const group = (id: string, items: unknown[]) => ({ id, label: id, items }) as never;
+
+function schema(layout: unknown): UISchema {
+	return { layout } as unknown as UISchema;
+}
+
+describe('getGroups — layout discrimination', () => {
+	it('flattens tabs into a single group list, tab order preserved', () => {
+		const s = schema({
+			type: 'tabbed',
+			tabs: [
+				{ id: 't1', label: 't1', groups: [group('g1', []), group('g2', [])] },
+				{ id: 't2', label: 't2', groups: [group('g3', [])] }
+			]
+		});
+		expect(getGroups(s).map((g) => g.id)).toEqual(['g1', 'g2', 'g3']);
+	});
+
+	it('returns flat groups directly', () => {
+		const s = schema({ type: 'flat', groups: [group('g1', []), group('g2', [])] });
+		expect(getGroups(s).map((g) => g.id)).toEqual(['g1', 'g2']);
+	});
+});
+
+describe('getGroups — defensive contract', () => {
+	it('returns [] when layout is missing', () => {
+		expect(getGroups({} as UISchema)).toEqual([]);
+	});
+
+	it('returns [] for an unknown layout type', () => {
+		expect(getGroups(schema({ type: 'mystery' }))).toEqual([]);
+	});
+
+	it('tolerates missing tabs / groups arrays', () => {
+		expect(getGroups(schema({ type: 'tabbed' }))).toEqual([]);
+		expect(getGroups(schema({ type: 'flat' }))).toEqual([]);
+	});
+
+	it('tolerates a tab with no groups', () => {
+		const s = schema({ type: 'tabbed', tabs: [{ id: 't1', label: 't1' }] });
+		expect(getGroups(s)).toEqual([]);
+	});
+});
+
+describe('getLayoutItems', () => {
+	it('collects items across all groups', () => {
+		const s = schema({
+			type: 'flat',
+			groups: [group('g1', [input('a'), linebreak('lb')]), group('g2', [input('b')])]
+		});
+		expect(getLayoutItems(s).map((i) => (i.type === 'linebreak' ? i.id : i.paramId))).toEqual([
+			'a',
+			'lb',
+			'b'
+		]);
+	});
+
+	it('tolerates a group with no items', () => {
+		const s = schema({ type: 'flat', groups: [{ id: 'g1', label: 'g1' }] });
+		expect(getLayoutItems(s)).toEqual([]);
+	});
+});
+
+describe('getInputItems', () => {
+	it('keeps only input items, dropping outputs and linebreaks', () => {
+		const s = schema({
+			type: 'flat',
+			groups: [group('g1', [input('a'), linebreak('lb'), { type: 'output', paramId: 'out' }])]
+		});
+		expect(getInputItems(s).map((i) => i.paramId)).toEqual(['a']);
+	});
+
+	it('preserves the source field so client-input filters work', () => {
+		const s = schema({
+			type: 'flat',
+			groups: [group('g1', [input('a', { kind: 'client' }), input('b')])]
+		});
+		const clients = getInputItems(s).filter((i) => i.source?.kind === 'client');
+		expect(clients.map((i) => i.paramId)).toEqual(['a']);
+	});
+});

package/src/lib/schema/traversal.ts

@@ -0,0 +1,5 @@
+// Schema layout traversal now lives in @selvajs/schemas (next to the types it walks) so
+// any package depending on the schema can traverse it without pulling in @selvajs/ui.
+// Re-exported here to keep it part of @selvajs/ui's published surface for existing
+// consumers.
+export { getGroups, getLayoutItems, getInputItems, type InputLayoutItem } from '@selvajs/schemas';

package/src/lib/schema/visibility-rules.test.ts

@@ -0,0 +1,173 @@
+import { describe, expect, it } from 'vitest';
+import {
+	evaluateRule,
+	evaluateVisibility,
+	evaluateGroupVisibility,
+	buildVisibilityMap,
+	itemKey
+} from './visibility-rules';
+import type { GroupVisibilityCondition, LayoutItem, VisibilityRule } from '@selvajs/schemas';
+
+// These tests pin the non-obvious branches: operator edge cases that are easy to
+// break in a refactor, and the action/short-circuit precedence in evaluateVisibility.
+// Trivial passthroughs (equals true === true) are omitted on purpose.
+
+// `operator` is widened to string so tests can exercise the unknown-operator path.
+function rule(partial: {
+	operator: string;
+	paramId: string;
+	value?: unknown;
+	values?: unknown[];
+}): VisibilityRule {
+	return partial as unknown as VisibilityRule;
+}
+
+describe('evaluateRule — operator edge cases', () => {
+	it('between requires exactly two values and is inclusive', () => {
+		const r = rule({ operator: 'between', paramId: 'x', values: [10, 20] });
+		expect(evaluateRule(r, { x: 10 })).toBe(true); // lower bound inclusive
+		expect(evaluateRule(r, { x: 20 })).toBe(true); // upper bound inclusive
+		expect(evaluateRule(r, { x: 21 })).toBe(false);
+		// Wrong arity must fail closed, not throw or pass.
+		expect(evaluateRule(rule({ operator: 'between', paramId: 'x', values: [10] }), { x: 15 })).toBe(
+			false
+		);
+	});
+
+	it('matches returns false for an invalid regex instead of throwing', () => {
+		const r = rule({ operator: 'matches', paramId: 'x', value: '(' });
+		expect(evaluateRule(r, { x: 'anything' })).toBe(false);
+	});
+
+	it('contains / containsAny coerce scalars and arrays through toStringArray', () => {
+		// scalar value is treated as a single-element array
+		expect(evaluateRule(rule({ operator: 'contains', paramId: 'x', value: '5' }), { x: 5 })).toBe(
+			true
+		);
+		expect(
+			evaluateRule(rule({ operator: 'containsAny', paramId: 'x', values: ['a', 'b'] }), {
+				x: ['b', 'c']
+			})
+		).toBe(true);
+	});
+
+	it('isEmpty treats empty string, null, and [] as empty but not a non-empty value', () => {
+		const empty = rule({ operator: 'isEmpty', paramId: 'x' });
+		expect(evaluateRule(empty, { x: '' })).toBe(true);
+		expect(evaluateRule(empty, { x: null })).toBe(true);
+		expect(evaluateRule(empty, { x: [] })).toBe(true);
+		expect(evaluateRule(empty, { x: '0' })).toBe(false);
+	});
+
+	it('unknown operator fails closed', () => {
+		expect(evaluateRule(rule({ operator: 'definitelyNotAnOp', paramId: 'x' }), { x: 1 })).toBe(
+			false
+		);
+	});
+});
+
+describe('evaluateVisibility — action & short-circuit precedence', () => {
+	const condition = {
+		rules: [{ operator: 'equals', paramId: 'mode', value: 'advanced' }],
+		mode: 'all'
+	};
+
+	it('explicit visible:false beats any visibility condition', () => {
+		const item = {
+			type: 'input',
+			paramId: 'p',
+			visible: false,
+			visibilityCondition: { ...condition, action: 'show' }
+		} as unknown as LayoutItem;
+		// Condition is met, but visible:false wins.
+		expect(evaluateVisibility(item, { mode: 'advanced' })).toEqual({
+			visible: false,
+			disabled: false
+		});
+	});
+
+	it('hide action inverts visibility and only carries defaultValue', () => {
+		const item = {
+			type: 'input',
+			paramId: 'p',
+			visibilityCondition: { ...condition, action: 'hide', defaultValue: 7 }
+		} as unknown as LayoutItem;
+		expect(evaluateVisibility(item, { mode: 'advanced' })).toEqual({
+			visible: false,
+			disabled: false,
+			defaultValue: 7
+		});
+		expect(evaluateVisibility(item, { mode: 'basic' }).visible).toBe(true);
+	});
+
+	it('disable action keeps item visible and only applies defaultValue when the condition is met', () => {
+		const item = {
+			type: 'input',
+			paramId: 'p',
+			visibilityCondition: { ...condition, action: 'disable', defaultValue: 'X' }
+		} as unknown as LayoutItem;
+		expect(evaluateVisibility(item, { mode: 'advanced' })).toEqual({
+			visible: true,
+			disabled: true,
+			defaultValue: 'X'
+		});
+		expect(evaluateVisibility(item, { mode: 'basic' })).toEqual({
+			visible: true,
+			disabled: false,
+			defaultValue: undefined
+		});
+	});
+
+	it('item without a condition is visible and enabled', () => {
+		const item = { type: 'input', paramId: 'p' } as unknown as LayoutItem;
+		expect(evaluateVisibility(item, {})).toEqual({ visible: true, disabled: false });
+	});
+});
+
+describe('buildVisibilityMap', () => {
+	const cond = {
+		rules: [{ operator: 'equals', paramId: 'mode', value: 'advanced' }],
+		mode: 'all'
+	};
+
+	it('keys each item by paramId (inputs/outputs) or id (linebreak) and matches evaluateVisibility', () => {
+		const items = [
+			{ type: 'input', paramId: 'a' },
+			{ type: 'linebreak', id: 'lb1' },
+			{
+				type: 'input',
+				paramId: 'b',
+				visibilityCondition: { ...cond, action: 'show' }
+			}
+		] as unknown as LayoutItem[];
+		const values = { mode: 'basic' };
+
+		const map = buildVisibilityMap(items, values);
+		expect(map.a).toEqual(evaluateVisibility(items[0], values));
+		expect(map.lb1).toEqual(evaluateVisibility(items[1], values));
+		// 'b' shows only when mode === 'advanced'; here it's hidden.
+		expect(map.b.visible).toBe(false);
+		expect(map.b).toEqual(evaluateVisibility(items[2], values));
+	});
+
+	it('itemKey returns paramId for controls and id for linebreaks', () => {
+		expect(itemKey({ type: 'input', paramId: 'p1' } as unknown as LayoutItem)).toBe('p1');
+		expect(itemKey({ type: 'linebreak', id: 'lb' } as unknown as LayoutItem)).toBe('lb');
+	});
+});
+
+describe('evaluateGroupVisibility', () => {
+	it('hide action inverts, show is the default, no condition means visible', () => {
+		const cond = (action?: string): { visibilityCondition: GroupVisibilityCondition } => ({
+			visibilityCondition: {
+				action,
+				mode: 'all',
+				rules: [{ operator: 'equals', paramId: 'show', value: true }]
+			} as unknown as GroupVisibilityCondition
+		});
+		expect(evaluateGroupVisibility(cond('hide'), { show: true })).toBe(false);
+		expect(evaluateGroupVisibility(cond('show'), { show: true })).toBe(true);
+		expect(evaluateGroupVisibility(cond(), { show: true })).toBe(true);
+		expect(evaluateGroupVisibility({}, {})).toBe(true);
+	});
+});

package/src/lib/schema/visibility-rules.ts

@@ -84,6 +84,31 @@ export function evaluateVisibility(
 	return actionFn(met, defaultValue);
 }
 
+/**
+ * Stable key for a layout item: `paramId` for inputs/outputs, `id` for linebreaks.
+ * Mirrors the key the renderers use in their `{#each}` blocks.
+ */
+export function itemKey(item: LayoutItem): string {
+	return item.type === 'linebreak' ? item.id : item.paramId;
+}
+
+/**
+ * Evaluates every item's visibility once against the current values, keyed by itemKey.
+ * Callers that touch the same items more than once per render (column layout + cell
+ * render, plus the default-value sweep) read from this single map instead of
+ * re-evaluating per access — one source of truth for "what's visible right now".
+ */
+export function buildVisibilityMap(
+	items: LayoutItem[],
+	values: Record<string, unknown>
+): Record<string, VisibilityResult> {
+	const map: Record<string, VisibilityResult> = {};
+	for (const item of items) {
+		map[itemKey(item)] = evaluateVisibility(item, values);
+	}
+	return map;
+}
+
 export function evaluateGroupVisibility(
 	group: { visibilityCondition?: GroupVisibilityCondition },
 	values: Record<string, unknown>

package/src/lib/types/solveFn.ts

@@ -6,7 +6,7 @@
  * @property errors - Optional array of error messages that occurred
  * @property warnings - Optional array of warning messages from the computation
  */
-interface SolveResult {
+export interface SolveResult {
 	outputs: Record<string, unknown>;
 	meshes?: any[];
 	errors?: string[];

package/src/lib/utils/file-download.ts

@@ -1,12 +1,8 @@
-/**
- * Wrapper utilities for file downloads from Grasshopper outputs
- * Uses the core package implementation for file handling
- */
-
 import { downloadFileData, type FileData } from '@selvajs/compute';
+import { SvelteMap } from 'svelte/reactivity';
 import { APP_DEFAULTS } from '../constants';
 
-const MIME_BY_EXT: Record<string, string> = {
+export const MIME_BY_EXT: Record<string, string> = {
 	'.png': 'image/png',
 	'.jpg': 'image/jpeg',
 	'.jpeg': 'image/jpeg',
@@ -45,10 +41,6 @@ function saveSingleFile(file: FileData): void {
 	URL.revokeObjectURL(a.href);
 }
 
-/**
- * Download file(s) from Grasshopper outputs
- * Single files are downloaded directly, multiple files are packaged as ZIP
- */
 export async function downloadFiles(
 	fileData: FileData | FileData[],
 	fileName: string = 'grasshopper-output'
@@ -73,9 +65,6 @@ export async function downloadFiles(
 	}
 }
 
-/**
- * Check if data is FileData
- */
 export function isFileData(data: unknown): data is FileData {
 	return (
 		typeof data === 'object' &&
@@ -86,9 +75,17 @@ export function isFileData(data: unknown): data is FileData {
 	);
 }
 
-/**
- * Format file size for display
- */
+export function groupMessages(messages: string[]): Array<{ message: string; count: number }> {
+	const grouped = new SvelteMap<string, number>();
+	for (const msg of messages) {
+		const baseMsg = msg.replace(/\([a-f0-9-]{36}\)$/i, '(...)').trim();
+		grouped.set(baseMsg, (grouped.get(baseMsg) ?? 0) + 1);
+	}
+	return Array.from(grouped.entries())
+		.map(([message, count]) => ({ message, count }))
+		.sort((a, b) => b.count - a.count);
+}
+
 export function formatFileSize(bytes: number): string {
 	if (bytes === 0) return '0 Bytes';
 
@@ -99,10 +96,6 @@ export function formatFileSize(bytes: number): string {
 	return Math.round((bytes / Math.pow(k, i)) * 100) / 100 + ' ' + sizes[i];
 }
 
-/**
- * Get approximate file size from base64 string
- */
 export function getBase64FileSize(base64: string): number {
-	// Base64 encoded string is approximately 33% larger than original
 	return Math.ceil((base64.length * 3) / 4);
 }

package/src/lib/utils/randomId.ts

@@ -1,22 +1,6 @@
-// Generate a UUIDv4-shaped string in any browser context.
-//
-// `crypto.randomUUID()` requires a secure context (HTTPS or localhost).
-// Plain-HTTP deployments — common for first-time self-hosters trying things
-// out — get `crypto.randomUUID is not a function`, which is a confusing
-// browser-level error nobody can diagnose without devtools open.
-//
-// We try in order:
-//   1. crypto.randomUUID()         — fastest path, available in secure contexts
-//   2. crypto.getRandomValues()    — Web Crypto's lower-level entropy, works
-//                                    in any context that ships Web Crypto
-//   3. Math.random() fallback      — last resort; non-cryptographic, but the
-//                                    IDs we generate here are only used as
-//                                    collision-resistant keys for client-side
-//                                    form rows, not as security tokens.
-//
-// Callers that need cryptographic randomness (tokens, secrets) must NOT use
-// this — they should fail loudly in insecure contexts instead of silently
-// downgrading.
+// Falls back through crypto.randomUUID → crypto.getRandomValues → Math.random so
+// plain-HTTP deployments don't crash. Math.random IDs are non-cryptographic —
+// fine for collision-resistant UI keys, not for tokens or secrets.
 export function randomId(): string {
 	const c = typeof crypto !== 'undefined' ? crypto : undefined;
 	if (c?.randomUUID) return c.randomUUID();