summaryrefslogtreecommitdiff
path: root/packages/cli/src/ui/hooks
diff options
context:
space:
mode:
authorJaana Dogan <[email protected]>2025-04-21 12:15:47 -0700
committerGitHub <[email protected]>2025-04-21 12:15:47 -0700
commit53a5728009adf1e91115ee0eb839eafa00262adf (patch)
tree2ad9b1a643f55ea22cc19946213f5790a9f2c8ce /packages/cli/src/ui/hooks
parentdea0782c891b2d849cc31c1364b32a9fb9a7a312 (diff)
Remove redundant else branches (#86)
Else branches are an anti pattern especially if you can easily return from the previous branch. Over time, else branches cause deep nesting and make code unreadable and unmaintainable. Remove elses where possible.
Diffstat (limited to 'packages/cli/src/ui/hooks')
-rw-r--r--packages/cli/src/ui/hooks/useStdin.ts10
1 files changed, 5 insertions, 5 deletions
diff --git a/packages/cli/src/ui/hooks/useStdin.ts b/packages/cli/src/ui/hooks/useStdin.ts
index e91b90e6..dc245254 100644
--- a/packages/cli/src/ui/hooks/useStdin.ts
+++ b/packages/cli/src/ui/hooks/useStdin.ts
@@ -70,13 +70,13 @@ export function usePipedInput(): PipedInputState {
stdin.removeListener('error', handleError);
stdin.removeListener('end', handleEnd);
};
- } else {
- // No piped input (running interactively)
- setIsLoading(false);
- // Optionally set an 'info' state or just let isLoading=false & isPiped=false suffice
- // setError('No piped input detected.'); // Maybe don't treat this as an 'error'
}
+ // No piped input (running interactively)
+ setIsLoading(false);
+ // Optionally set an 'info' state or just let isLoading=false & isPiped=false suffice
+ // setError('No piped input detected.'); // Maybe don't treat this as an 'error'
+
// Intentionally run only once on mount or when stdin theoretically changes
}, [stdin, isRawModeSupported, setRawMode /*, exit */]);