Convert params to redux

[NexInfinite]

Moved params page and its components into redux

[Copilot]

Pull Request Overview

This PR migrates the parameters page and its components to use Redux for state management, replacing local React state with centralized Redux slices. This improves state consistency and enables better data flow throughout the application.

• Converts local state management to Redux with a new paramsSlice
• Refactors socket event handling to use Redux middleware
• Simplifies component prop passing by utilizing Redux selectors

Reviewed Changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated no comments.

Show a summary per file

File	Description
gcs/src/redux/store.js	Adds paramsSlice to store configuration and improves code formatting
gcs/src/redux/middleware/socketMiddleware.js	Integrates parameter-specific socket event handlers and cleanup
gcs/src/redux/middleware/emitters.js	Adds parameter-related socket emitters
gcs/src/params.jsx	Converts from local state to Redux selectors and actions
gcs/src/components/realtimeGraph.jsx	Minor optimization in time formatting logic
gcs/src/components/params/paramsToolbar.jsx	Migrates to Redux state management
gcs/src/components/params/autopilotRebootModal.jsx	Converts to Redux-based modal state

Comments suppressed due to low confidence (1)

gcs/src/components/params/autopilotRebootModal.jsx:1

• The close function is undefined. It should be () => dispatch(setAutoPilotRebootModalOpen(false)) to match the onClose handler.

/*

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

[1Blademaster]

Have you forgotten to commit the paramSlice.js file?

[NexInfinite]

Somehow yes what the helly

[1Blademaster]

LGTM, haven't tested locally though

[NexInfinite]

Can you please test this one locally as it may have breaking changes

Found bugs..

[Copilot]

Pull Request Overview

Copilot reviewed 12 out of 12 changed files in this pull request and generated 2 comments.

Comments suppressed due to low confidence (2)

gcs/src/components/params/autopilotRebootModal.jsx:54

• The close function is not defined. It should be replaced with () => dispatch(setAutoPilotRebootModalOpen(false)) to close the modal.

                <Button
                  onClick={close}
                  color={tailwindColors.red[600]}
                  className="mt-4"
                >

gcs/src/components/params/valueInput.jsx:80

• For consistency with the Select component above, this should also sanitize the input: onChange={(value) => onChange(sanitiseInput(value), param)}

      onChange={(value) => onChange(value, param)}

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

[1Blademaster]

Parameters with both a range and values key in the param def should use range and be a number input, not a select. E.g. ACRO_RP_RATE_TC should be a number input, even though it gives values. The values can be displayed on the side or something to help with selection. The best solution would be to have an input where you can EITHER input your number directly OR select one from the dropdown (but no point spending more time on it if it's not easily done via Mantine).

[Copilot]

Pull Request Overview

Copilot reviewed 12 out of 12 changed files in this pull request and generated 1 comment.

Comments suppressed due to low confidence (3)

gcs/src/redux/slices/paramsSlice.js:1

• The property name is inconsistent. Line 108 references paramDef?.Bitmask but line 101 checks for paramDef?.BitMask. This will cause the bitmask conditional to never execute properly.

import { createSlice } from "@reduxjs/toolkit"

gcs/src/components/params/autopilotRebootModal.jsx:51

• The close function is not defined in this component scope. It should be () => dispatch(setAutoPilotRebootModalOpen(false)) to match the modal's onClose handler.

                  onClick={close}

gcs/src/params.jsx:1

• The itemData prop is still being passed to FixedSizeList but the Row component no longer expects or uses this data. This should be removed since the component now uses Redux state directly.

/*

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

[Copilot]

Pull Request Overview

Copilot reviewed 13 out of 13 changed files in this pull request and generated 5 comments.

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

[1Blademaster]

LGTM, tested and fixed some bugs + talked on discord

[Copilot]

Pull Request Overview

Copilot reviewed 15 out of 15 changed files in this pull request and generated 4 comments.

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}