app bulk execute should require --watch when using --output-file

* Requires removing the default flag value for `--watch`

Author: nickwesselman

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

@@ -29,7 +29,7 @@ export default class BulkExecute extends AppLinkedCommand {
       query,
       variables: flags.variables,
       variableFile: flags['variable-file'],
-      watch: flags.watch,
+      watch: flags.watch ?? false,
       outputFile: flags['output-file'],
       ...(flags.version && {version: flags.version}),
     })

packages/app/src/cli/flags.ts

@@ -64,13 +64,13 @@ export const bulkOperationFlags = {
     parse: async (input) => normalizeStoreFqdn(input),
   }),
   watch: Flags.boolean({
-    description: 'Wait for bulk operation results before exiting.',
+    description: 'Wait for bulk operation results before exiting. Defaults to false.',
     env: 'SHOPIFY_FLAG_WATCH',
-    default: false,
   }),
   'output-file': Flags.string({
     description: 'The file path where results should be written. If not specified, results will be written to STDOUT.',
     env: 'SHOPIFY_FLAG_OUTPUT_FILE',
+    dependsOn: ['watch'],
   }),
   version: Flags.string({
     description: 'The API version to use for the bulk operation. If not specified, uses the latest stable version.',

packages/cli/oclif.manifest.json

@@ -125,6 +125,9 @@
           "type": "boolean"
         },
         "output-file": {
+          "dependsOn": [
+            "watch"
+          ],
           "description": "The file path where results should be written. If not specified, results will be written to STDOUT.",
           "env": "SHOPIFY_FLAG_OUTPUT_FILE",
           "hasDynamicHelp": false,
@@ -212,7 +215,7 @@
         },
         "watch": {
           "allowNo": false,
-          "description": "Wait for bulk operation results before exiting.",
+          "description": "Wait for bulk operation results before exiting. Defaults to false.",
           "env": "SHOPIFY_FLAG_WATCH",
           "name": "watch",
           "type": "boolean"