chore: remove --json flag

Author: nrkruk

README.md

@@ -117,7 +117,7 @@ Preview a Lightning Experience app locally and in real-time, without deploying i
 
 ```
 USAGE
-  $ sf lightning dev app -o <value> [--json] [--flags-dir <value>] [-n <value>] [-t desktop|ios|android] [-i <value>]
+  $ sf lightning dev app -o <value> [--flags-dir <value>] [-n <value>] [-t desktop|ios|android] [-i <value>]
 
 FLAGS
   -i, --device-id=<value>     ID of the mobile device to display the preview if device type is set to `ios` or
@@ -130,7 +130,6 @@ FLAGS
 
 GLOBAL FLAGS
   --flags-dir=<value>  Import flag values from a directory.
-  --json               Format output as json.
 
 DESCRIPTION
   Preview a Lightning Experience app locally and in real-time, without deploying it.
@@ -178,7 +177,7 @@ Preview an Experience Builder site locally and in real-time, without deploying i
 
 ```
 USAGE
-  $ sf lightning dev site [--json] [--flags-dir <value>] [-n <value>] [--debug] [-o <value>]
+  $ sf lightning dev site [--flags-dir <value>] [-n <value>] [--debug] [-o <value>]
 
 FLAGS
   -n, --name=<value>        Name of the Experience Builder site to preview. It has to match a site name from the current
@@ -189,7 +188,6 @@ FLAGS
 
 GLOBAL FLAGS
   --flags-dir=<value>  Import flag values from a directory.
-  --json               Format output as json.
 
 DESCRIPTION
   Preview an Experience Builder site locally and in real-time, without deploying it.

command-snapshot.json

@@ -4,15 +4,15 @@
     "command": "lightning:dev:app",
     "flagAliases": [],
     "flagChars": ["i", "n", "o", "t"],
-    "flags": ["device-id", "device-type", "flags-dir", "json", "name", "target-org"],
+    "flags": ["device-id", "device-type", "flags-dir", "name", "target-org"],
     "plugin": "@salesforce/plugin-lightning-dev"
   },
   {
     "alias": [],
     "command": "lightning:dev:site",
     "flagAliases": [],
     "flagChars": ["n", "o"],
-    "flags": ["flags-dir", "json", "name", "target-org"],
+    "flags": ["flags-dir", "name", "target-org"],
     "plugin": "@salesforce/plugin-lightning-dev"
   }
 ]

messages/lightning.dev.app.md

@@ -23,10 +23,6 @@ To learn more about Local Dev enablement, considerations, and limitations, see t
 
 Name of the Lightning Experience app to preview.
 
-# flags.target-org.summary
-
-Username or alias of the target org. Not required if the `target-org` configuration variable is already set.
-
 # flags.device-type.summary
 
 Type of device to display the app preview.

messages/lightning.dev.site.md

@@ -21,10 +21,6 @@ For more considerations and limitations, see the Lightning Web Components Develo
 
 Name of the Experience Builder site to preview. It has to match a site name from the current org.
 
-# flags.target-org.summary
-
-Username or alias of the target org. Not required if the `target-org` configuration variable is already set.
-
 # examples
 
 - Preview the site "Partner Central" from the org "myOrg":

src/commands/lightning/dev/app.ts

@@ -59,15 +59,14 @@ export default class LightningDevApp extends SfCommand<void> {
   public static readonly summary = messages.getMessage('summary');
   public static readonly description = messages.getMessage('description');
   public static readonly examples = messages.getMessages('examples');
+  public static readonly enableJsonFlag = false; // Disable json flag since we don't return anything
 
   public static readonly flags = {
     name: Flags.string({
       summary: messages.getMessage('flags.name.summary'),
       char: 'n',
     }),
-    'target-org': Flags.requiredOrg({
-      summary: messages.getMessage('flags.target-org.summary'),
-    }),
+    'target-org': Flags.requiredOrg(),
     'device-type': Flags.option({
       summary: messages.getMessage('flags.device-type.summary'),
       char: 't',

src/commands/lightning/dev/site.ts

@@ -18,13 +18,14 @@ export default class LightningDevSite extends SfCommand<void> {
   public static readonly summary = messages.getMessage('summary');
   public static readonly description = messages.getMessage('description');
   public static readonly examples = messages.getMessages('examples');
+  public static readonly enableJsonFlag = false; // Disable json flag since we don't return anything
 
   public static readonly flags = {
     name: Flags.string({
       summary: messages.getMessage('flags.name.summary'),
       char: 'n',
     }),
-    'target-org': Flags.optionalOrg({ summary: messages.getMessage('flags.target-org.summary') }),
+    'target-org': Flags.requiredOrg(),
   };
 
   public async run(): Promise<void> {

test/commands/lightning/dev/site.test.ts

@@ -31,11 +31,6 @@ describe('lightning dev site', () => {
     // expect(output).to.include('hello world');
   });
 
-  it('runs hello with --json and no provided name', async () => {
-    // const result = await LightningDevSite.run([]);
-    // expect(result.path).to.equal('/Users/nkruk/git/plugin-lightning-dev/src/commands/lightning/dev/site.ts');
-  });
-
   it('runs hello world --name Astro', async () => {
     // await LightningDevSite.run(['--name', 'Astro']);
     // const output = sfCommandStubs.log