Merge pull request #9127 from LedgerHQ/fix/no-issue-listApps-printed-…

…twice 🐛 (cli): print `listApps` output only once
LedgerHQ · Feb 10, 2025 · 1bef14e · 1bef14e
2 parents 87279d6 + f502aa5
commit 1bef14e
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 9 deletions.
diff --git a/.changeset/metal-poets-serve.md b/.changeset/metal-poets-serve.md
@@ -0,0 +1,5 @@
+---
+"@ledgerhq/live-cli": patch
+---
+
+fix(cli): print `listApps` output only once
diff --git a/apps/cli/src/commands/device/listApps.ts b/apps/cli/src/commands/device/listApps.ts
@@ -1,4 +1,4 @@
-import { from, tap } from "rxjs";
+import { from } from "rxjs";
 import { withDevice } from "@ledgerhq/live-common/hw/deviceAccess";
 import listApps from "@ledgerhq/live-common/hw/listApps";
 import { DeviceCommonOpts, deviceOpt } from "../../scan";
@@ -8,12 +8,5 @@ export type ListAppsJobOpts = DeviceCommonOpts;
 export default {
   description: "list all installed apps on the device",
   args: [deviceOpt],
-  job: ({ device }: ListAppsJobOpts) =>
-    withDevice(device || "")(t =>
-      from(listApps(t)).pipe(
-        tap(res => {
-          console.log(res);
-        }),
-      ),
-    ),
+  job: ({ device }: ListAppsJobOpts) => withDevice(device || "")(t => from(listApps(t))),
 };