refactor: add skip prop

Author: gabitoesmiapodo

source/app.tsx

@@ -1,5 +1,5 @@
-import { Box, Text } from 'ink'
-import React, { useState, type ReactNode, useMemo, useCallback } from 'react'
+import { Box } from 'ink'
+import { type ReactNode, useCallback, useMemo, useState } from 'react'
 import MainTitle from './components/MainTitle.js'
 import CloneRepo from './components/steps/CloneRepo/CloneRepo.js'
 import FileCleanup from './components/steps/FileCleanup.js'
@@ -42,10 +42,12 @@ const App = () => {
         onSelect={onSelectSetupType}
         key={3}
       />,
+      // TODO: add a skip parameter to all (or most) steps
+      // to allow skipping when testing, etc.
       <OptionalPackages
-        installation={setupType?.value}
         onCompletion={finishStep}
         onSubmit={onSelectSelectedFeatures}
+        skip={setupType?.value === 'full'}
         key={4}
       />,
       <Install

source/components/steps/OptionalPackages.tsx

@@ -1,14 +1,8 @@
 import { Text } from 'ink'
-import React, { useState, type FC, useEffect } from 'react'
-import type { InstallationType, MultiSelectItem } from '../../types/types.js'
+import { type FC, useState } from 'react'
+import type { MultiSelectItem } from '../../types/types.js'
 import MultiSelect from '../Multiselect/index.js'
 
-interface Props {
-  installation: InstallationType | undefined
-  onCompletion: () => void
-  onSubmit: (selectedItems: Array<MultiSelectItem>) => void
-}
-
 const customPackages: Array<MultiSelectItem> = [
   {
     label: 'Component Demos',
@@ -32,23 +26,26 @@ const customPackages: Array<MultiSelectItem> = [
   },
 ]
 
+interface Props {
+  onCompletion: () => void
+  onSubmit: (selectedItems: Array<MultiSelectItem>) => void
+  skip?: boolean
+}
+
 /**
  * Step for selecting optional packages. Skipped if installation type is 'full'.
  * @param onCompletion
  * @param onSubmit
  * @param installation
+ * @param skip
  */
-const OptionalPackages: FC<Props> = ({ onCompletion, onSubmit, installation }) => {
+const OptionalPackages: FC<Props> = ({ onCompletion, onSubmit, skip = false }) => {
   const [isFocused, setIsFocused] = useState(true)
-  const skip = installation === 'full'
 
-  // biome-ignore lint/correctness/useExhaustiveDependencies: Run this only once
-  useEffect(() => {
-    // full installation, do nothing
-    if (skip) {
-      onCompletion()
-    }
-  }, [])
+  // full installation, do nothing
+  if (skip) {
+    onCompletion()
+  }
 
   const onHandleSubmit = (selectedItems: Array<MultiSelectItem>) => {
     onSubmit(selectedItems)