Skip to content

Commit 943d27e

Browse files
committed
fix: fix build and test errors
1 parent bbfe6c5 commit 943d27e

File tree

3 files changed

+16
-9
lines changed

3 files changed

+16
-9
lines changed

apps/dashboard/src/main.tsx

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
1+
import {addStatusListener, getSdkWorker, type WorkerStatus} from '@sanity/sdk'
2+
import sdkWorker from '@sanity/sdk/worker?worker&url'
13
import {StrictMode} from 'react'
24
import {createRoot} from 'react-dom/client'
3-
import {getSdkWorker, addStatusListener, type WorkerStatus} from '@sanity/sdk'
4-
import sdkWorker from '@sanity/sdk/worker?worker&url'
55

66
import App from './App'
77

@@ -15,9 +15,11 @@ async function initializeSharedWorker() {
1515

1616
// Add status listener for debugging
1717
addStatusListener((status: WorkerStatus) => {
18+
// eslint-disable-next-line no-console
1819
console.log('[Dashboard] Worker status changed:', status)
1920
})
2021
} catch (error) {
22+
// eslint-disable-next-line no-console
2123
console.warn('Failed to initialize SharedWorker:', error)
2224
// Fallback to local subscription management
2325
}

packages/core/src/query/queryStore.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -106,7 +106,7 @@ function normalizeOptionsWithPerspective(
106106
*/
107107
function isInDashboardContext(): boolean {
108108
// For the POC, we'll forward queries when we're in an iframe
109-
return window.self !== window.top
109+
return globalThis?.window !== undefined && globalThis.window.self !== globalThis.window.top
110110
}
111111

112112
/**

packages/core/src/sharedWorkerStore/sharedWorkerStore.ts

Lines changed: 11 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -23,16 +23,20 @@ export const sharedWorkerStore = createStore<SharedWorkerStoreState & SharedWork
2323
// Actions
2424
registerSubscription: (subscription) => {
2525
const state = get()
26-
26+
2727
// Check if we already have an equivalent subscription
2828
const existingSubscriptions = Array.from(state.subscriptions.values())
29-
const equivalentSubscription = existingSubscriptions.find(existing =>
30-
areSubscriptionsEquivalent(existing, subscription)
29+
const equivalentSubscription = existingSubscriptions.find((existing) =>
30+
areSubscriptionsEquivalent(existing, subscription),
3131
)
3232

3333
if (equivalentSubscription) {
3434
// Return the existing subscription ID instead of creating a new one
35-
console.log('[SharedWorkerStore] Found equivalent subscription, reusing:', equivalentSubscription.subscriptionId)
35+
// eslint-disable-next-line no-console
36+
console.log(
37+
'[SharedWorkerStore] Found equivalent subscription, reusing:',
38+
equivalentSubscription.subscriptionId,
39+
)
3640
return equivalentSubscription.subscriptionId
3741
}
3842

@@ -43,15 +47,16 @@ export const sharedWorkerStore = createStore<SharedWorkerStoreState & SharedWork
4347
lastUpdate: Date.now(),
4448
}
4549

46-
set((state) => {
47-
const newSubscriptions = new Map(state.subscriptions)
50+
set((currentState) => {
51+
const newSubscriptions = new Map(currentState.subscriptions)
4852
newSubscriptions.set(subscription.subscriptionId, activeSubscription)
4953

5054
return {
5155
subscriptions: newSubscriptions,
5256
}
5357
})
5458

59+
// eslint-disable-next-line no-console
5560
console.log('[SharedWorkerStore] Created new subscription:', subscription.subscriptionId)
5661
return subscription.subscriptionId
5762
},

0 commit comments

Comments
 (0)