Skip to content

Commit b2f0ae1

Browse files
committed
Trigger full reload when error overlay needs to be updated
1 parent 45d29fc commit b2f0ae1

File tree

2 files changed

+5
-0
lines changed

2 files changed

+5
-0
lines changed

packages/theme/src/cli/utilities/theme-fs.ts

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@ import {calculateChecksum} from './asset-checksum.js'
22
import {applyIgnoreFilters, getPatternsFromShopifyIgnore} from './asset-ignore.js'
33
import {Notifier} from './notifier.js'
44
import {createSyncingCatchError} from './errors.js'
5+
import {emitHotReloadEvent} from './theme-environment/hot-reload/server.js'
56
import {DEFAULT_IGNORE_PATTERNS, timestampDateFormat} from '../constants.js'
67
import {glob, readFile, ReadOptions, fileExists, mkdir, writeFile, removeFile} from '@shopify/cli-kit/node/fs'
78
import {joinPath, basename, relativePath} from '@shopify/cli-kit/node/path'
@@ -150,9 +151,11 @@ export function mountThemeFileSystem(root: string, options?: ThemeFileSystemOpti
150151

151152
if (result?.success) {
152153
uploadErrors.delete(fileKey)
154+
emitHotReloadEvent({type: 'full', key: fileKey})
153155
} else {
154156
const errors = result?.errors?.asset ?? ['Response was not successful.']
155157
uploadErrors.set(fileKey, errors)
158+
emitHotReloadEvent({type: 'full', key: fileKey})
156159
throw new Error(errors.join('\n'))
157160
}
158161

packages/theme/src/cli/utilities/theme-uploader.ts

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@ import {partitionThemeFiles} from './theme-fs.js'
22
import {rejectGeneratedStaticAssets} from './asset-checksum.js'
33
import {renderTasksToStdErr} from './theme-ui.js'
44
import {createSyncingCatchError, renderThrownError} from './errors.js'
5+
import {emitHotReloadEvent} from './theme-environment/hot-reload/server.js'
56
import {AdminSession} from '@shopify/cli-kit/node/session'
67
import {Result, Checksum, Theme, ThemeFileSystem} from '@shopify/cli-kit/node/themes/types'
78
import {AssetParams, bulkUploadThemeAssets, deleteThemeAssets} from '@shopify/cli-kit/node/themes/api'
@@ -392,6 +393,7 @@ export function updateUploadErrors(result: Result, localThemeFileSystem: ThemeFi
392393
} else {
393394
const errors = result.errors?.asset ?? ['Response was not successful.']
394395
localThemeFileSystem.uploadErrors.set(result.key, errors)
396+
emitHotReloadEvent({type: 'full', key: result.key})
395397
}
396398
}
397399

0 commit comments

Comments
 (0)