fix: remove derived state

Author: Elessar1802

src/Shared/Components/CICDHistory/LogsRenderer.tsx

@@ -184,15 +184,11 @@ export const LogsRenderer = ({
         triggerDetails.podStatus && triggerDetails.podStatus !== POD_STATUS.PENDING && logsURL,
     )
     const [stageList, setStageList] = useState<StageDetailType[]>([])
-    const [openAllStages, setOpenAllStages] = useState(false)
     // State for logs list in case no stages are available
     const [logsList, setLogsList] = useState<string[]>([])
     const { searchKey, handleSearch } = useUrlFilters()
 
-    const handleSetStageList = (list: StageDetailType[]) => {
-        setOpenAllStages(list.every((item) => item.isOpen))
-        setStageList(list)
-    }
+    const areAllStagesExpanded = stageList.every((item) => item.isOpen)
 
     const areStagesAvailable =
         (window._env_.FEATURE_STEP_WISE_LOGS_ENABLE && streamDataList[0]?.startsWith(LOGS_STAGE_IDENTIFIER)) || false
@@ -382,34 +378,34 @@ export const LogsRenderer = ({
         }
 
         const newStageList = getStageListFromStreamData()
-        handleSetStageList(newStageList)
+        setStageList(newStageList)
         // NOTE: Not adding searchKey as dependency since on mount we would already have searchKey
         // And for other cases we would use handleSearchEnter
     }, [streamDataList, areEventsProgressing])
 
     const handleSearchEnter = (searchText: string) => {
         handleSearch(searchText)
         const newStageList = getStageListFromStreamData(searchText)
-        handleSetStageList(newStageList)
+        setStageList(newStageList)
     }
 
     const handleStageClose = (index: number) => {
         const newLogs = structuredClone(stageList)
         newLogs[index].isOpen = false
-        handleSetStageList(newLogs)
+        setStageList(newLogs)
     }
 
     const handleStageOpen = (index: number) => {
         const newLogs = structuredClone(stageList)
         newLogs[index].isOpen = true
-        handleSetStageList(newLogs)
+        setStageList(newLogs)
     }
 
     const handleToggleOpenAllStages = () => {
-        handleSetStageList(
+        setStageList(
             stageList.map((stage) => ({
                 ...stage,
-                isOpen: !openAllStages,
+                isOpen: !areAllStagesExpanded,
             })),
         )
     }
@@ -451,7 +447,7 @@ export const LogsRenderer = ({
                                 onClick={handleToggleOpenAllStages}
                                 aria-label="Expand all stages"
                             >
-                                {openAllStages ? (
+                                {areAllStagesExpanded ? (
                                     <ICExpandAll className="icon-dim-16 dc__no-shrink dc__transition--transform scn-0" />
                                 ) : (
                                     <ICCollapseAll className="icon-dim-16 dc__no-shrink dc__transition--transform scn-0" />