Use '[[' instead of '[' for conditional tests. The '[[' construct is safer and more feature-rich.

[ziagham]

Description

In the following shell conditional, the test uses both [[ and [ constructs.
To improve consistency and robustness, we should replace [ with [[ throughout the script.

Affected File: install/install.sh

lines: 32, 43, 49, 67, 89, 102, 123, 129, 142, 144, 150, 161, 166, 173, 185, 211

Example

Line 32

if [[ "$OS" == "linux" || "$OS" == "darwin" ]] && [ "$FLOWCTL_INSTALL_DIR" == "/usr/local/bin" ]; then

Suggested change

Use [[ consistently for both conditions:

if [[ ("$OS" == "linux" || "$OS" == "darwin") && "$FLOWCTL_INSTALL_DIR" == "/usr/local/bin" ]]; then

Action Items

• Replace all instances of [ with [[ in conditional expressions.
• Test script compatibility on supported platforms (Linux, macOS).

[ziagham]

@Sakeeb91
FYI: I’ve created a few issues for this repo. If you’re looking for something to work on, feel free to pick one of them up. I’ll also be creating some issues on other FlowSynx-related repositories soon. Additionally, I have a few ideas for AI/ML plugins that we’ll be starting in the near future — I’ll be getting your help to implement those AI-based plugins as well.

[Sakeeb91]

Thank you so much @ziagham ! Looking forward to working!