fix: CLI profile config view without driver

- `profile config view`: do not throw error if a driver is not set
- use `jsonEnabled` instead of `flags.json` to test for JSON output

packages/cli-common/src/commands/base-command.ts

@@ -119,7 +119,7 @@ abstract class BaseCommand<T extends typeof Command=typeof Command> extends Comm
       oclifSettings.debug = true
     }
     this.#rawArgs = raw
-    this.logger = commandLogger(this, this.flags.json ? 'stderr' : 'stdout')
+    this.logger = commandLogger(this, this.jsonEnabled() ? 'stderr' : 'stdout')
     this.stdErrLogger = commandLogger(this, 'stderr')
   }
 

packages/cli/src/commands/profile/config/update.ts

@@ -66,7 +66,7 @@ export default class UpdateProfileConfig extends ProfileCommand<typeof UpdatePro
     const profileDriver = this.profile.driver as DriverName | undefined
     const driver: DriverName | undefined = (this.flags.driver || profileDriver)
     if (!driver) {
-      ux.error('Missing driver configuration in profile, use the --driver flag to set the desired machine driver')
+      ux.error(`Missing driver configuration in profile, specify the ${text.code('--driver')} flag to set the driver`)
     }
     const { unset } = this.flags
     validateUnset(driver, unset)

packages/cli/src/commands/profile/config/view.ts

@@ -12,15 +12,18 @@ export default class ViewProfileConfig extends ProfileCommand<typeof ViewProfile
 
   async run(): Promise<unknown> {
     const pStore = profileStore(this.store).ref
-    const driver = this.profile.driver as DriverName
-    const config = await pStore.defaultDriverFlags(driver)
+    const driver = this.profile.driver as DriverName | undefined
     if (!driver) {
-      ux.error([
-        'Missing driver configuration in profile.',
-        `Run ${text.command(this.config, 'profile config update --driver <driver>')} to set the desired machine driver`,
-      ].join(EOL))
+      if (this.jsonEnabled()) {
+        return { driver: null, defaultFlags: {} }
+      }
+      ux.info('No driver specified in profile.')
+      ux.info(`Run ${text.command(this.config, 'profile config update --driver <driver>')} to set a driver`)
+      return undefined
     }
-    if (this.flags.json) {
+
+    const config = await pStore.defaultDriverFlags(driver)
+    if (this.jsonEnabled()) {
       return { driver, defaultFlags: config }
     }
     ux.info(`Current configuration for driver ${text.code(driver)}:`)

packages/cli/src/commands/profile/current.ts

@@ -15,6 +15,6 @@ export default class CurrentProfile extends ProfileCommand<typeof CurrentProfile
     }
     const { alias, id, location } = currentProfile
     const result = { alias, id, location }
-    return this.flags.json ? result : ux.styledObject(result)
+    return this.jsonEnabled() ? result : ux.styledObject(result)
   }
 }

packages/cli/src/commands/profile/key.ts

@@ -47,7 +47,7 @@ export default class Key extends ProfileCommand<typeof Key> {
       throw new Error('Could not find tunneling key in profile store')
     }
     const value = await extractKey(tunnelingKey, this.args.type as KeyType)
-    if (this.flags.json) {
+    if (this.jsonEnabled()) {
       return value
     }
     ux.log(value)

packages/cli/src/commands/profile/ls.ts

@@ -18,7 +18,7 @@ export default class ListProfile extends ProfileCommand<typeof ListProfile> {
   async run(): Promise<unknown> {
     const { profiles, current } = await this.profileConfig.list()
 
-    if (this.flags.json) {
+    if (this.jsonEnabled()) {
       return { profiles, current }
     }