@viamrobotics/motion-tools 1.25.4 → 1.25.6

package/dist/components/Scene.svelte

@@ -1,7 +1,7 @@
 <script lang="ts">
 	import type { Snippet } from 'svelte'
 
-	import { T } from '@threlte/core'
+	import { T, useThrelte } from '@threlte/core'
 	import { Environment, Grid, interactivity, PerfMonitor, PortalTarget } from '@threlte/extras'
 	import { useXR } from '@threlte/xr'
 	import { ShaderMaterial, Vector3 } from 'three'
@@ -29,10 +29,14 @@
 
 	let { children }: Props = $props()
 
+	const threlte = useThrelte()
 	const settings = useSettings()
 	const focusedObject3d = useFocusedObject3d()
 	const origin = useOrigin()
 
+	// @ts-expect-error This is for debugging
+	globalThis.__threlte__ = threlte
+
 	const { raycaster, enabled } = interactivity({
 		filter: (intersections) => {
 			const match = intersections.find((intersection) => {

package/dist/components/overlay/Details.svelte

@@ -477,13 +477,24 @@
 			<div>
 				<strong class="font-semibold">parent frame</strong>
 				{#if showEditFrameOptions}
-					<div aria-label="mutable parent frame">
-						<List
-							options={configFrames.getParentFrameOptions(name.current ?? '') ?? []}
-							value={parent.current ?? 'world'}
-							on:change={handleParentChange}
-						/>
-					</div>
+					<!--
+						Remount on entity change. svelte-tweakpane-ui's List runs
+						`listBlade.value = value` on the still-mounted blade before its
+						`options` prop has propagated, so the new entity's parent name
+						(absent from the previous entity's option set) hits Tweakpane's
+						ListConstraint, snaps to the first option, and fires a change
+						event that handleParentChange interprets as a user pick — silently
+						reparenting the clicked frame.
+					-->
+					{#key entity}
+						<div aria-label="mutable parent frame">
+							<List
+								options={configFrames.getParentFrameOptions(name.current ?? '') ?? []}
+								value={parent.current ?? 'world'}
+								on:change={handleParentChange}
+							/>
+						</div>
+					{/key}
 				{:else}
 					<div class="mt-0.5 flex gap-3">
 						{@render ImmutableField({

package/dist/components/overlay/FloatingPanel.svelte

@@ -89,11 +89,19 @@
 			</div>
 		</div>
 
+		<!--
+			Skip rendering the body subtree while collapsed. zag-js controls
+			visibility via attributes (the panel chrome stays mounted), but the
+			children don't need to react to upstream state when the user can't see them.
+			Children mount fresh on open.
+		-->
 		<div
 			{...api.getBodyProps()}
 			class="relative h-[calc(100%-33px)]"
 		>
-			{@render children()}
+			{#if isOpen}
+				{@render children()}
+			{/if}
 		</div>
 
 		{#if resizable}

package/dist/ecs/hierarchy.d.ts

@@ -10,6 +10,12 @@ export declare const parentTraits: (name: string | undefined) => ConfigurableTra
  * Set or clear an entity's parent. Strips any existing `ChildOf` or `Orphan`,
  * then writes `Orphan(name)` (the resolver converts it to `ChildOf` on the
  * next reactive flush). Pass `undefined` or `'world'` to detach to root.
+ *
+ * Short-circuits when the effective parent name (via resolved `ChildOf` or
+ * pending `Orphan`) already matches `name`. Network-backed reconcilers call
+ * this every refetch tick on stable entities; the demote-then-re-resolve
+ * dance otherwise flips `useParentName` to `undefined` and back, remounting
+ * every `<Portal id={parent.current}>` subtree per tick.
  */
 export declare const setParent: (entity: Entity, name: string | undefined) => void;
 /** The parent entity, or `undefined` at the world root or while orphaned. */

package/dist/ecs/hierarchy.js

@@ -16,15 +16,25 @@ export const parentTraits = (name) => {
  * Set or clear an entity's parent. Strips any existing `ChildOf` or `Orphan`,
  * then writes `Orphan(name)` (the resolver converts it to `ChildOf` on the
  * next reactive flush). Pass `undefined` or `'world'` to detach to root.
+ *
+ * Short-circuits when the effective parent name (via resolved `ChildOf` or
+ * pending `Orphan`) already matches `name`. Network-backed reconcilers call
+ * this every refetch tick on stable entities; the demote-then-re-resolve
+ * dance otherwise flips `useParentName` to `undefined` and back, remounting
+ * every `<Portal id={parent.current}>` subtree per tick.
  */
 export const setParent = (entity, name) => {
+    const desired = !name || name === 'world' ? undefined : name;
     const target = entity.targetFor(ChildOf);
+    const current = (target?.isAlive() ? target.get(Name) : undefined) ?? entity.get(Orphan);
+    if (current === desired)
+        return;
     if (target)
         entity.remove(ChildOf(target));
     entity.remove(Orphan);
-    if (!name || name === 'world')
+    if (desired === undefined)
         return;
-    entity.add(Orphan(name));
+    entity.add(Orphan(desired));
 };
 /** The parent entity, or `undefined` at the world root or while orphaned. */
 export const getParentEntity = (entity) => entity.targetFor(ChildOf);

package/dist/ecs/useWorld.js

@@ -1,8 +1,16 @@
 import { createWorld } from 'koota';
 import { getContext, setContext } from 'svelte';
+import * as relations from './relations';
+import * as traits from './traits';
 export const WORLD_CONTEXT_KEY = Symbol('koota-context');
 export function provideWorld() {
     const world = createWorld();
+    // @ts-expect-error This is for debugging.
+    globalThis.__koota__ = {
+        world,
+        traits,
+        relations,
+    };
     setContext(WORLD_CONTEXT_KEY, world);
 }
 export function useWorld() {

package/dist/hooks/useLogs.svelte.js

@@ -1,17 +1,31 @@
 import { getContext, setContext, untrack } from 'svelte';
 import { MathUtils } from 'three';
 const key = Symbol('logs-context');
+const MAX_LOGS = 200;
 export const provideLogs = () => {
-    const logs = $state([]);
-    const warnings = $state([]);
-    const errors = $state([]);
+    // Plain insertion-ordered Map keyed by `${level}|${timestamp}|${message}`
+    // drives storage; a single `$state` version counter drives reactivity.
+    // Hot path is `Map.set` (or in-place `count++`) plus `version++` — no
+    // array allocation per add. The display arrays are materialized lazily
+    // in `$derived.by`, so a closed logs panel costs nothing.
+    const entries = new Map();
+    let version = $state(0);
     const intl = new Intl.DateTimeFormat('en-US', {
         dateStyle: 'short',
         timeStyle: 'short',
     });
+    const dedupKey = (timestamp, level, message) => `${level}\0${timestamp}\0${message}`;
+    const all = $derived.by(() => {
+        void version;
+        const out = [...entries.values()];
+        out.reverse();
+        return out;
+    });
+    const errors = $derived(all.filter((l) => l.level === 'error'));
+    const warnings = $derived(all.filter((l) => l.level === 'warn'));
     setContext(key, {
         get current() {
-            return logs;
+            return all;
         },
         get errors() {
             return errors;
@@ -22,35 +36,26 @@ export const provideLogs = () => {
         add(message, level = 'info') {
             untrack(() => {
                 const timestamp = intl.format(Date.now());
-                const match = logs.find((log) => log.message === message && log.level === level && log.timestamp === timestamp);
+                const k = dedupKey(timestamp, level, message);
+                const match = entries.get(k);
                 if (match) {
                     match.count += 1;
                 }
                 else {
-                    const log = {
-                        timestamp,
+                    entries.set(k, {
+                        uuid: MathUtils.generateUUID(),
                         message,
                         count: 1,
                         level,
-                        uuid: MathUtils.generateUUID(),
-                    };
-                    logs.unshift(log);
-                    if (level === 'error') {
-                        errors.unshift(log);
-                    }
-                    else if (level === 'warn') {
-                        warnings.unshift(log);
-                    }
-                }
-                if (logs.length > 200) {
-                    const log = logs.pop();
-                    if (log?.level === 'error') {
-                        errors.splice(errors.indexOf(log), 1);
-                    }
-                    else if (log?.level === 'warn') {
-                        warnings.splice(warnings.indexOf(log), 1);
+                        timestamp,
+                    });
+                    if (entries.size > MAX_LOGS) {
+                        const oldestKey = entries.keys().next().value;
+                        if (oldestKey !== undefined)
+                            entries.delete(oldestKey);
                     }
                 }
+                version++;
             });
         },
     });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@viamrobotics/motion-tools",
-  "version": "1.25.4",
+  "version": "1.25.6",
   "description": "Motion visualization with Viam",
   "license": "Apache-2.0",
   "type": "module",