Add erroring for when variables are given for aBulkOps query

Author: ericlee878

packages/app/src/cli/services/bulk-operations/execute-bulk-operation.ts

@@ -4,6 +4,7 @@ import {AppLinkedInterface} from '../../models/app/app.js'
 import {renderSuccess, renderInfo, renderWarning} from '@shopify/cli-kit/node/ui'
 import {outputContent, outputToken} from '@shopify/cli-kit/node/output'
 import {ensureAuthenticatedAdmin} from '@shopify/cli-kit/node/session'
+import {AbortError} from '@shopify/cli-kit/node/error'
 import {parse} from 'graphql'
 
 interface ExecuteBulkOperationInput {
@@ -23,6 +24,12 @@ export async function executeBulkOperation(input: ExecuteBulkOperationInput): Pr
 
   const adminSession = await ensureAuthenticatedAdmin(storeFqdn)
 
+  if (!isMutation(query) && variables) {
+    throw new AbortError(
+      outputContent`The ${outputToken.yellow('--variables')} flag can only be used with mutations, not queries.`,
+    )
+  }
+
   const bulkOperationResponse = isMutation(query)
     ? await runBulkOperationMutation({adminSession, query, variables})
     : await runBulkOperationQuery({adminSession, query})

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')

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
@@ -14,6 +16,7 @@ export async function runBulkOperationQuery(
   options: BulkOperationRunQueryOptions,
 ): Promise<BulkOperationRunQueryMutation['bulkOperationRunQuery']> {
   const {adminSession, query} = options
+
   const response = await adminRequestDoc<BulkOperationRunQueryMutation, {query: string}>({
     query: BulkOperationRunQuery,
     session: adminSession,