Skip to content

Commit 52cd79d

Browse files
nickwesselmanclaude
andcommitted
Fix lint errors, import order, regenerate docs and snapshots
Fix prettier formatting in test files, import ordering in service files, regenerate dev docs and E2E snapshots. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent b2f1d4e commit 52cd79d

File tree

9 files changed

+473
-38
lines changed

9 files changed

+473
-38
lines changed

docs-shopify.dev/commands/interfaces/auth-login.interface.ts

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,4 +5,16 @@ export interface authlogin {
55
* @environment SHOPIFY_FLAG_AUTH_ALIAS
66
*/
77
'--alias <value>'?: string
8+
9+
/**
10+
* Start the login flow without polling. Prints the auth URL and exits immediately.
11+
* @environment SHOPIFY_FLAG_AUTH_NO_POLLING
12+
*/
13+
'--no-polling'?: ''
14+
15+
/**
16+
* Resume a previously started login flow.
17+
* @environment SHOPIFY_FLAG_AUTH_RESUME
18+
*/
19+
'--resume'?: ''
820
}

docs-shopify.dev/generated/generated_docs_data.json

Lines changed: 431 additions & 1 deletion
Large diffs are not rendered by default.

packages/app/src/cli/commands/store/bulk/cancel.test.ts

Lines changed: 3 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -6,28 +6,21 @@ vi.mock('../../../services/store-bulk-cancel-operation.js')
66

77
describe('store bulk cancel command', () => {
88
test('requires --store flag', async () => {
9-
await expect(
10-
StoreBulkCancel.run(['--id', '123'], import.meta.url),
11-
).rejects.toThrow()
9+
await expect(StoreBulkCancel.run(['--id', '123'], import.meta.url)).rejects.toThrow()
1210

1311
expect(storeCancelBulkOperation).not.toHaveBeenCalled()
1412
})
1513

1614
test('requires --id flag', async () => {
17-
await expect(
18-
StoreBulkCancel.run(['--store', 'test-store.myshopify.com'], import.meta.url),
19-
).rejects.toThrow()
15+
await expect(StoreBulkCancel.run(['--store', 'test-store.myshopify.com'], import.meta.url)).rejects.toThrow()
2016

2117
expect(storeCancelBulkOperation).not.toHaveBeenCalled()
2218
})
2319

2420
test('calls storeCancelBulkOperation with correct arguments', async () => {
2521
vi.mocked(storeCancelBulkOperation).mockResolvedValue()
2622

27-
await StoreBulkCancel.run(
28-
['--store', 'test-store.myshopify.com', '--id', '123'],
29-
import.meta.url,
30-
)
23+
await StoreBulkCancel.run(['--store', 'test-store.myshopify.com', '--id', '123'], import.meta.url)
3124

3225
expect(storeCancelBulkOperation).toHaveBeenCalledWith({
3326
storeFqdn: 'test-store.myshopify.com',

packages/app/src/cli/commands/store/bulk/execute.test.ts

Lines changed: 14 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -11,9 +11,7 @@ describe('store bulk execute command', () => {
1111
vi.mocked(loadQuery).mockResolvedValue('query { shop { name } }')
1212
vi.mocked(storeExecuteBulkOperation).mockResolvedValue()
1313

14-
await expect(
15-
StoreBulkExecute.run(['--query', 'query { shop { name } }'], import.meta.url),
16-
).rejects.toThrow()
14+
await expect(StoreBulkExecute.run(['--query', 'query { shop { name } }'], import.meta.url)).rejects.toThrow()
1715

1816
expect(storeExecuteBulkOperation).not.toHaveBeenCalled()
1917
})
@@ -27,9 +25,7 @@ describe('store bulk execute command', () => {
2725
import.meta.url,
2826
)
2927

30-
expect(loadQuery).toHaveBeenCalledWith(
31-
expect.objectContaining({query: 'query { shop { name } }'}),
32-
)
28+
expect(loadQuery).toHaveBeenCalledWith(expect.objectContaining({query: 'query { shop { name } }'}))
3329
expect(storeExecuteBulkOperation).toHaveBeenCalledWith(
3430
expect.objectContaining({
3531
storeFqdn: 'test-store.myshopify.com',
@@ -61,10 +57,13 @@ describe('store bulk execute command', () => {
6157

6258
await StoreBulkExecute.run(
6359
[
64-
'--store', 'test-store.myshopify.com',
65-
'--query', 'query { shop { name } }',
60+
'--store',
61+
'test-store.myshopify.com',
62+
'--query',
63+
'query { shop { name } }',
6664
'--watch',
67-
'--output-file', '/tmp/out.jsonl',
65+
'--output-file',
66+
'/tmp/out.jsonl',
6867
],
6968
import.meta.url,
7069
)
@@ -83,9 +82,12 @@ describe('store bulk execute command', () => {
8382

8483
await StoreBulkExecute.run(
8584
[
86-
'--store', 'test-store.myshopify.com',
87-
'--query', 'mutation { productCreate { product { id } } }',
88-
'--variables', '{"input": {"title": "test"}}',
85+
'--store',
86+
'test-store.myshopify.com',
87+
'--query',
88+
'mutation { productCreate { product { id } } }',
89+
'--variables',
90+
'{"input": {"title": "test"}}',
8991
],
9092
import.meta.url,
9193
)

packages/app/src/cli/commands/store/bulk/status.test.ts

Lines changed: 2 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -15,10 +15,7 @@ describe('store bulk status command', () => {
1515
test('calls storeGetBulkOperationStatus when --id is provided', async () => {
1616
vi.mocked(storeGetBulkOperationStatus).mockResolvedValue()
1717

18-
await StoreBulkStatus.run(
19-
['--store', 'test-store.myshopify.com', '--id', '123'],
20-
import.meta.url,
21-
)
18+
await StoreBulkStatus.run(['--store', 'test-store.myshopify.com', '--id', '123'], import.meta.url)
2219

2320
expect(storeGetBulkOperationStatus).toHaveBeenCalledWith({
2421
storeFqdn: 'test-store.myshopify.com',
@@ -29,10 +26,7 @@ describe('store bulk status command', () => {
2926
test('calls storeListBulkOperations when --id is not provided', async () => {
3027
vi.mocked(storeListBulkOperations).mockResolvedValue()
3128

32-
await StoreBulkStatus.run(
33-
['--store', 'test-store.myshopify.com'],
34-
import.meta.url,
35-
)
29+
await StoreBulkStatus.run(['--store', 'test-store.myshopify.com'], import.meta.url)
3630

3731
expect(storeListBulkOperations).toHaveBeenCalledWith({
3832
storeFqdn: 'test-store.myshopify.com',

packages/app/src/cli/commands/store/bulk/status.ts

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,4 @@
1-
import {
2-
storeGetBulkOperationStatus,
3-
storeListBulkOperations,
4-
} from '../../../services/store-bulk-operation-status.js'
1+
import {storeGetBulkOperationStatus, storeListBulkOperations} from '../../../services/store-bulk-operation-status.js'
52
import {normalizeBulkOperationId} from '../../../services/bulk-operations/bulk-operation-status.js'
63
import {Flags} from '@oclif/core'
74
import {globalFlags} from '@shopify/cli-kit/node/cli'

packages/app/src/cli/services/store-bulk-cancel-operation.ts

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,13 @@
1-
import {renderBulkOperationUserErrors, formatBulkOperationCancellationResult} from './bulk-operations/format-bulk-operation-status.js'
1+
import {
2+
renderBulkOperationUserErrors,
3+
formatBulkOperationCancellationResult,
4+
} from './bulk-operations/format-bulk-operation-status.js'
5+
import {formatStoreOperationInfo} from './graphql/common.js'
26
import {
37
BulkOperationCancel,
48
BulkOperationCancelMutation,
59
BulkOperationCancelMutationVariables,
610
} from '../api/graphql/bulk-operations/generated/bulk-operation-cancel.js'
7-
import {formatStoreOperationInfo} from './graphql/common.js'
811
import {renderInfo, renderError, renderSuccess, renderWarning} from '@shopify/cli-kit/node/ui'
912
import {outputContent, outputToken} from '@shopify/cli-kit/node/output'
1013
import {ensureAuthenticatedAdmin} from '@shopify/cli-kit/node/session'

packages/app/src/cli/services/store-bulk-operation-status.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,11 +2,11 @@ import {BulkOperation} from './bulk-operations/watch-bulk-operation.js'
22
import {formatBulkOperationStatus} from './bulk-operations/format-bulk-operation-status.js'
33
import {BULK_OPERATIONS_MIN_API_VERSION} from './bulk-operations/constants.js'
44
import {extractBulkOperationId} from './bulk-operations/bulk-operation-status.js'
5+
import {formatStoreOperationInfo, resolveApiVersion} from './graphql/common.js'
56
import {
67
GetBulkOperationById,
78
GetBulkOperationByIdQuery,
89
} from '../api/graphql/bulk-operations/generated/get-bulk-operation-by-id.js'
9-
import {formatStoreOperationInfo, resolveApiVersion} from './graphql/common.js'
1010
import {
1111
ListBulkOperations,
1212
ListBulkOperationsQuery,

packages/e2e/data/snapshots/commands.txt

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -90,6 +90,10 @@
9090
│ └─ update
9191
├─ search
9292
├─ store
93+
│ ├─ bulk
94+
│ │ ├─ cancel
95+
│ │ ├─ execute
96+
│ │ └─ status
9397
│ └─ execute
9498
├─ theme
9599
│ ├─ check

0 commit comments

Comments
 (0)