@shipfox/client-agent 6.0.1 → 8.0.0

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@shipfox/client-agent",
   "license": "MIT",
-  "version": "6.0.1",
+  "version": "8.0.0",
   "repository": {
     "type": "git",
     "url": "git+https://github.com/ShipfoxHQ/shipfox.git",
@@ -28,11 +28,11 @@
     "@swc/helpers": "^0.5.17",
     "@tanstack/react-form": "^1.32.0",
     "@tanstack/react-query": "^5.101.0",
-    "@shipfox/api-agent-dto": "9.0.1",
+    "@shipfox/api-agent-dto": "9.0.2",
     "@shipfox/client-api": "6.0.1",
-    "@shipfox/client-shell": "6.0.1",
-    "@shipfox/react-ui": "0.3.6",
-    "@shipfox/client-ui": "6.0.1"
+    "@shipfox/client-shell": "8.0.0",
+    "@shipfox/client-ui": "6.0.2",
+    "@shipfox/react-ui": "0.3.7"
   },
   "peerDependencies": {
     "@tanstack/react-query": "^5.101.0",

package/src/state/model-provider-onboarding.test.ts

@@ -29,11 +29,11 @@ describe('model provider onboarding dismissed state', () => {
   });
 
   test('keeps dismissed state isolated per workspace', () => {
-    const localStorage = createStorage({
-      'shipfox.modelProviderOnboardingDismissed': JSON.stringify({'workspace-1': true}),
-    });
+    const localStorage = createStorage();
     vi.stubGlobal('window', {localStorage});
 
+    dismissModelProviderOnboarding('workspace-1');
+
     expect(isModelProviderOnboardingDismissed('workspace-1')).toBe(true);
     expect(isModelProviderOnboardingDismissed('workspace-2')).toBe(false);
   });
@@ -46,7 +46,7 @@ describe('model provider onboarding dismissed state', () => {
 
   test('treats corrupt JSON as empty state', () => {
     const localStorage = createStorage({
-      'shipfox.modelProviderOnboardingDismissed': '{',
+      'shipfox.modelProviderOnboardingDismissed.workspace.workspace-1': '{',
     });
     vi.stubGlobal('window', {localStorage});
 

package/src/state/model-provider-onboarding.ts

@@ -1,37 +1,27 @@
-import {createTypedBrowserStorage, localStorageOrUndefined} from '@shipfox/client-ui';
+import {
+  type BrowserStorageKey,
+  createTypedBrowserStorage,
+  localStorageOrUndefined,
+} from '@shipfox/client-ui';
 
-type DismissedMap = Record<string, true>;
-
-const dismissedStorage = createTypedBrowserStorage(localStorageOrUndefined, {
+const dismissedStorageKey = {
   key: 'shipfox.modelProviderOnboardingDismissed',
   lifetime: 'persistent',
   principalScope: 'workspace',
-  serialize: (dismissed: DismissedMap) => JSON.stringify(dismissed),
+  serialize: (dismissed: boolean) => JSON.stringify(dismissed),
   parse: (raw) => {
-    try {
-      const parsed: unknown = JSON.parse(raw);
-      return isDismissedMap(parsed) ? parsed : undefined;
-    } catch {
-      return undefined;
-    }
+    return raw === 'true' ? true : raw === 'false' ? false : undefined;
   },
-});
+} satisfies BrowserStorageKey<boolean>;
 
 export function isModelProviderOnboardingDismissed(workspaceId: string): boolean {
-  return Boolean(readDismissedMap()[workspaceId]);
+  return dismissedStorage(workspaceId).read() === true;
 }
 
 export function dismissModelProviderOnboarding(workspaceId: string): void {
-  const dismissed = readDismissedMap();
-  dismissed[workspaceId] = true;
-  dismissedStorage.write(dismissed);
-}
-
-function readDismissedMap(): DismissedMap {
-  return dismissedStorage.read() ?? {};
+  dismissedStorage(workspaceId).write(true);
 }
 
-function isDismissedMap(value: unknown): value is DismissedMap {
-  if (typeof value !== 'object' || value === null || Array.isArray(value)) return false;
-  return Object.values(value).every((entry) => entry === true);
+function dismissedStorage(workspaceId: string) {
+  return createTypedBrowserStorage(localStorageOrUndefined, dismissedStorageKey, {workspaceId});
 }