Add erroring for when variables are given for aBulkOps query

Author: ericlee878

packages/app/src/cli/services/bulk-operations/run-query.test.ts

@@ -1,5 +1,6 @@
 import {runBulkOperationQuery} from './run-query.js'
 import {adminRequestDoc} from '@shopify/cli-kit/node/api/admin'
+import {AbortError} from '@shopify/cli-kit/node/error'
 import {describe, test, expect, vi} from 'vitest'
 
 vi.mock('@shopify/cli-kit/node/api/admin')
@@ -33,4 +34,19 @@ describe('runBulkOperationQuery', () => {
     expect(bulkOperationResult?.bulkOperation).toEqual(successfulBulkOperation)
     expect(bulkOperationResult?.userErrors).toEqual([])
   })
+
+  test('throws AbortError when variables are provided with a query', async () => {
+    const query = '{ products { edges { node { id } } } }'
+    const variables = '[{"input":{"id":"gid://shopify/Product/123"}}]'
+
+    await expect(
+      runBulkOperationQuery({
+        adminSession: mockSession,
+        query,
+        variables,
+      }),
+    ).rejects.toThrow(AbortError)
+
+    expect(adminRequestDoc).not.toHaveBeenCalled()
+  })
 })

packages/app/src/cli/services/bulk-operations/run-query.ts

@@ -4,6 +4,8 @@ import {
 } from '../../api/graphql/bulk-operations/generated/bulk-operation-run-query.js'
 import {adminRequestDoc} from '@shopify/cli-kit/node/api/admin'
 import {AdminSession} from '@shopify/cli-kit/node/session'
+import {AbortError} from '@shopify/cli-kit/node/error'
+import {outputContent, outputToken} from '@shopify/cli-kit/node/output'
 
 interface BulkOperationRunQueryOptions {
   adminSession: AdminSession
@@ -13,7 +15,14 @@ interface BulkOperationRunQueryOptions {
 export async function runBulkOperationQuery(
   options: BulkOperationRunQueryOptions,
 ): Promise<BulkOperationRunQueryMutation['bulkOperationRunQuery']> {
-  const {adminSession, query} = options
+  const {adminSession, query, variables} = options
+
+  if (variables) {
+    throw new AbortError(
+      outputContent`The ${outputToken.yellow('--variables')} flag can only be used with mutations, not queries.`,
+    )
+  }
+
   const response = await adminRequestDoc<BulkOperationRunQueryMutation, {query: string}>({
     query: BulkOperationRunQuery,
     session: adminSession,