Skip to content

Commit 48744b6

Browse files
committed
removed changeset and changed record naming
1 parent 12ba0b8 commit 48744b6

File tree

4 files changed

+11
-16
lines changed

4 files changed

+11
-16
lines changed

.changeset/curvy-bears-raise.md

Lines changed: 0 additions & 5 deletions
This file was deleted.

packages/cli-kit/src/private/node/api.test.ts

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -416,7 +416,7 @@ describe('retryAwareRequest', () => {
416416
url: 'https://themes.example.com/api',
417417
useNetworkLevelRetry: true,
418418
maxRetryTimeMs: 10000,
419-
recordThemeCommandRetries: true,
419+
recordCommandRetries: true,
420420
},
421421
undefined,
422422
{
@@ -471,7 +471,7 @@ describe('retryAwareRequest', () => {
471471
url: 'https://app.example.com/api',
472472
useNetworkLevelRetry: true,
473473
maxRetryTimeMs: 10000,
474-
recordThemeCommandRetries: false,
474+
recordCommandRetries: false,
475475
},
476476
undefined,
477477
{
@@ -528,7 +528,7 @@ describe('retryAwareRequest', () => {
528528
url: 'https://themes.example.com/upload',
529529
useNetworkLevelRetry: true,
530530
maxRetryTimeMs: 10000,
531-
recordThemeCommandRetries: true,
531+
recordCommandRetries: true,
532532
},
533533
undefined,
534534
{
@@ -584,7 +584,7 @@ describe('retryAwareRequest', () => {
584584
url: 'https://themes.example.com/auth',
585585
useNetworkLevelRetry: true,
586586
maxRetryTimeMs: 10000,
587-
recordThemeCommandRetries: true,
587+
recordCommandRetries: true,
588588
},
589589
undefined,
590590
{

packages/cli-kit/src/private/node/api.ts

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -18,11 +18,11 @@ export type NetworkRetryBehaviour =
1818
| {
1919
useNetworkLevelRetry: true
2020
maxRetryTimeMs: number
21-
recordThemeCommandRetries?: boolean
21+
recordCommandRetries?: boolean
2222
}
2323
| {
2424
useNetworkLevelRetry: false
25-
recordThemeCommandRetries?: boolean
25+
recordCommandRetries?: boolean
2626
}
2727

2828
type RequestOptions<T> = {
@@ -153,8 +153,8 @@ async function runRequestWithNetworkLevelRetry<T extends {headers: Headers; stat
153153
throw err
154154
}
155155

156-
// Record theme command retries
157-
if (requestOptions.recordThemeCommandRetries) {
156+
// Record command retries
157+
if (requestOptions.recordCommandRetries) {
158158
recordRetry(requestOptions.url, `network-retry:${(err as Error).message}`)
159159
}
160160

@@ -380,8 +380,8 @@ ${result.sanitizedHeaders}
380380
}
381381
retriesUsed += 1
382382

383-
// Record theme command retries
384-
if (requestOptions.recordThemeCommandRetries) {
383+
// Record command retries
384+
if (requestOptions.recordCommandRetries) {
385385
recordRetry(requestOptions.url, `http-retry-${retriesUsed}:${result.status}:`)
386386
}
387387

packages/cli-kit/src/public/node/themes/api.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@ const THEME_API_NETWORK_BEHAVIOUR: RequestModeInput = {
3737
useNetworkLevelRetry: true,
3838
useAbortSignal: false,
3939
maxRetryTimeMs: 90 * 1000,
40-
recordThemeCommandRetries: true,
40+
recordCommandRetries: true,
4141
}
4242

4343
export async function fetchTheme(id: number, session: AdminSession): Promise<Theme | undefined> {

0 commit comments

Comments
 (0)