From 4e998ac083aee759beeea8304570bac3af560bf3 Mon Sep 17 00:00:00 2001
From: Lorenzo Blasa <lblasa@meta.com>
Date: Wed, 2 Aug 2023 07:56:17 -0700
Subject: [PATCH] Get OS by inspecting command

Summary: Commands can be idb or adb. If no ClientQuery is provided, set the OS accordingly.

Reviewed By: antonk52

Differential Revision: D47990745

fbshipit-source-id: 3e38b6ecb0a8ea21e584c1d6eff6feb53dd2cfab
---
 desktop/flipper-server-core/src/recorder.tsx | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/desktop/flipper-server-core/src/recorder.tsx b/desktop/flipper-server-core/src/recorder.tsx
index 3e39da624..9725f35e6 100644
--- a/desktop/flipper-server-core/src/recorder.tsx
+++ b/desktop/flipper-server-core/src/recorder.tsx
@@ -47,7 +47,8 @@ class Recorder {
         const device = clientQuery?.device ?? 'NONE';
         const app = clientQuery?.app ?? 'NONE';
         const medium = clientQuery?.medium ?? 'NONE';
-        const os = clientQuery?.os ?? 'Browser';
+        const os =
+          clientQuery?.os ?? (payload.cmd.includes('idb') ? 'iOS' : 'Android');
 
         const entry: CommandRecordEntry = {
           time: new Date(),