Merge branch 'main' into testu

Author: jdalton

src/commands/repos/list-repos.ts

@@ -48,6 +48,20 @@ export async function listRepos({
     return
   }
 
+  spinner.stop()
+
+  if (outputJson) {
+    const data = result.data.results.map(o => ({
+      id: o.id,
+      name: o.name,
+      visibility: o.visibility,
+      defaultBranch: o.default_branch,
+      archived: o.archived
+    }))
+    console.log(JSON.stringify(data, null, 2))
+    return
+  }
+
   const options = {
     columns: [
       { field: 'id', name: colors.magenta('ID') },
@@ -58,5 +72,5 @@ export async function listRepos({
     ]
   }
 
-  spinner.stop(chalkTable(options, result.data.results))
+  console.log(chalkTable(options, result.data.results))
 }

src/utils/api.ts

@@ -28,7 +28,7 @@ export function handleUnsuccessfulApiResponse<T extends SocketSdkOperations>(
     spinner.stop()
     throw new AuthError(message)
   }
-  spinner.error(
+  spinner.errorAndStop(
     `${colors.bgRed(colors.white('API returned an error:'))} ${message}`
   )
   process.exit(1)