Minor cleanup: fix small bugs and style issues in connect.sh

[Copilot]

Applies a small set of targeted fixes to connect.sh without altering its overall structure or behaviour.

Changes Made

• Missing newline after password prompt: Added echo after read -rs so subsequent output is not printed on the same line as the password input
• Removed dead code: Deleted the misleading commented-out .env block that served no purpose
• Consistent redirection: Changed >/dev/null to &>/dev/null on the docker ps check in the setup section to match the cleanup section
• Standard function declaration: Changed function cleanup() to the POSIX-compatible cleanup() form

Testing

• bash -n connect.sh — syntax check passes
• shellcheck connect.sh — no warnings

---

💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.

[DaTiC0]

This refactoring is overkill for this type of script @copilot

[Copilot]

This refactoring is overkill for this type of script @copilot

You're right. Rolled back to the original structure and applied only four minimal fixes in e7816be:

• Added echo after the silent password read (missing newline was a real UX bug)
• Removed the dead/misleading .env comment block
• Made docker ps &>/dev/null consistent between the setup and cleanup sections
• Changed function cleanup() to the standard cleanup() declaration form

No new functions, no logging infrastructure, no input validation loops.