@viamrobotics/motion-tools 0.18.2 → 0.18.3

package/dist/__tests__/fixtures/MockCanvas.svelte

@@ -0,0 +1,11 @@
+<script>
+	import { Canvas } from '@threlte/core'
+	let { child, ...rest } = $props()
+</script>
+
+<Canvas>
+	<svelte:component
+		this={child}
+		{...rest}
+	/>
+</Canvas>

package/dist/__tests__/fixtures/MockCanvas.svelte.d.ts

@@ -0,0 +1,11 @@
+export default MockCanvas;
+type MockCanvas = {
+    $on?(type: string, callback: (e: any) => void): () => void;
+    $set?(props: Partial<$$ComponentProps>): void;
+};
+declare const MockCanvas: import("svelte").Component<{
+    child: any;
+} & Record<string, any>, {}, "">;
+type $$ComponentProps = {
+    child: any;
+} & Record<string, any>;

package/dist/hooks/useDrawAPI.svelte.js

@@ -10,17 +10,17 @@ import { createPose, createPoseFromFrame } from '../transform';
 import { useCameraControls } from './useControls.svelte';
 import { useThrelte } from '@threlte/core';
 import { OrientationVector } from '../three/OrientationVector';
+import { useLogs } from './useLogs.svelte';
 const axis = new Vector3();
 const quaternion = new Quaternion();
 const ov = new OrientationVector();
 const key = Symbol('draw-api-context-key');
 const tryParse = (json) => {
     try {
-        return JSON.parse(json);
+        return [null, JSON.parse(json)];
     }
     catch (error) {
-        console.warn('Failed to parse JSON:', error);
-        return;
+        return [error, null];
     }
 };
 /**
@@ -55,6 +55,7 @@ class Float32Reader {
     }
 }
 export const provideDrawAPI = () => {
+    const logs = useLogs();
     const cameraControls = useCameraControls();
     const { invalidate } = useThrelte();
     let pointsIndex = 0;
@@ -380,23 +381,27 @@ export const provideDrawAPI = () => {
     const scheduleReconnect = () => {
         setTimeout(() => {
             reconnectDelay = Math.min(reconnectDelay * 2, maxReconnectDelay);
-            console.log(`Reconnecting in ${reconnectDelay / 1000} seconds...`);
+            logs.add(`Reconnecting to drawing server in ${reconnectDelay / 1000} seconds...`, 'warn');
             connect();
         }, reconnectDelay);
     };
     const onOpen = () => {
         connectionStatus = 'open';
         reconnectDelay = 1000;
-        console.log(`Connected to websocket server at ${BACKEND_IP}:${BUN_SERVER_PORT}`);
+        logs.add(`Connected to drawing server at ${BACKEND_IP}:${BUN_SERVER_PORT}`);
     };
     const onClose = () => {
         connectionStatus = 'closed';
-        console.log('Disconnected from websocket server');
+        logs.add('Disconnected from drawing server', 'warn');
         scheduleReconnect();
     };
     const onError = (event) => {
-        console.log('Websocket error', JSON.stringify(event));
+        const stringified = JSON.stringify(event);
         ws.close();
+        if (stringified === '{"isTrusted":true}') {
+            return;
+        }
+        logs.add(`Drawing server error: ${JSON.stringify(event)}`, 'error');
     };
     const onMessage = async (event) => {
         if (typeof event.data === 'object' && 'arrayBuffer' in event.data) {
@@ -418,7 +423,10 @@ export const provideDrawAPI = () => {
                 return drawGLTF(reader.buffer);
             }
         }
-        const data = tryParse(event.data);
+        const [error, data] = tryParse(event.data);
+        if (error) {
+            logs.add(`Failed to parse JSON from drawing server: ${JSON.stringify(error)}`, 'error');
+        }
         if (!data)
             return;
         if ('setCameraPose' in data) {

package/dist/hooks/useFrames.svelte.js

@@ -24,7 +24,11 @@ export const provideFrames = (partID) => {
         if (revision) {
             untrack(() => query.current).refetch();
         }
-        logs.add('Fetching frames...');
+    });
+    $effect.pre(() => {
+        if (query.current.isFetching) {
+            logs.add('Fetching frames...');
+        }
     });
     $effect.pre(() => {
         if (partConfig.isDirty) {

package/dist/lib.js

@@ -1,4 +1,6 @@
 // Components
+// NOTE: These components should be pure and not use any hooks if you add a new component to export here
+// ensure you write a corresponding unit test to assert the component works in absence of parent providers in /src/lib/__tests__/PureComponents.svelte.spec.ts
 export { default as Geometry } from './components/Geometry.svelte';
 export { default as AxesHelper } from './components/AxesHelper.svelte';
 // Classes

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@viamrobotics/motion-tools",
-  "version": "0.18.2",
+  "version": "0.18.3",
   "description": "Motion visualization with Viam",
   "license": "Apache-2.0",
   "type": "module",
@@ -40,6 +40,7 @@
     "@viamrobotics/sdk": "0.55.0",
     "@viamrobotics/svelte-sdk": "0.7.1",
     "@vitejs/plugin-basic-ssl": "2.1.0",
+    "@vitest/coverage-v8": "^3.2.4",
     "@zag-js/svelte": "1.22.1",
     "@zag-js/tree-view": "1.22.1",
     "camera-controls": "3.1.0",
@@ -129,6 +130,7 @@
     "test:unit": "vitest",
     "test:client": "go test ./client/... -count=1",
     "test": "pnpm test:unit -- --run",
+    "test:coverage": "npx vitest run --coverage",
     "test:e2e": "playwright test",
     "model-pipeline:run": "node scripts/model-pipeline.js",
     "release": "changeset publish"