@viamrobotics/motion-tools 1.25.4 → 1.25.5

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/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.5",
   "description": "Motion visualization with Viam",
   "license": "Apache-2.0",
   "type": "module",