Merge branch 'main' into persist-query-logging-settings

Author: DTCurrie

CHANGELOG.md

@@ -1,5 +1,11 @@
 # @viamrobotics/svelte-sdk
 
+## 0.7.2
+
+### Patch Changes
+
+- 530af6c: do not create resource clients until connected
+
 ## 0.7.1
 
 ### Patch Changes

package.json

@@ -2,7 +2,7 @@
   "name": "@viamrobotics/svelte-sdk",
   "description": "Build Svelte apps with Viam",
   "license": "Apache-2.0",
-  "version": "0.7.1",
+  "version": "0.7.2",
   "scripts": {
     "dev": "vite dev",
     "build": "vite build && npm run prepack",

src/lib/hooks/create-resource-client.svelte.ts

@@ -1,6 +1,10 @@
-import type { Resource, RobotClient } from '@viamrobotics/sdk';
+import {
+  type Resource,
+  type RobotClient,
+  MachineConnectionEvent,
+} from '@viamrobotics/sdk';
 
-import { useRobotClient } from './robot-clients.svelte';
+import { useConnectionStatus, useRobotClient } from './robot-clients.svelte';
 
 export type Client<T> = new (part: RobotClient, name: string) => T;
 
@@ -10,12 +14,17 @@ export const createResourceClient = <T extends Resource>(
   resourceName: () => string
 ): { current: T | undefined } => {
   const robotClient = useRobotClient(partID);
+  const connectionStatus = useConnectionStatus(partID);
 
   const resourceClient = $derived.by<T | undefined>(() => {
     if (!robotClient.current) {
       return;
     }
 
+    if (connectionStatus.current !== MachineConnectionEvent.CONNECTED) {
+      return;
+    }
+
     const nextClient = new client(robotClient.current, resourceName());
 
     // PartIDs are used to invalidate queries for this client