Skip to content

feat: Add companion WiFi CLI#2217

Open
dz0ny wants to merge 1 commit intomeshcore-dev:devfrom
dz0ny:feat/configure-wifi-via-cli
Open

feat: Add companion WiFi CLI#2217
dz0ny wants to merge 1 commit intomeshcore-dev:devfrom
dz0ny:feat/configure-wifi-via-cli

Conversation

@dz0ny
Copy link
Copy Markdown

@dz0ny dz0ny commented Apr 1, 2026

This is a small PR that enables configuring wifi over CLI.

ref: #917

Copilot AI review requested due to automatic review settings April 1, 2026 08:01
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds persisted WiFi credentials to the companion_radio example and exposes them via the CLI rescue interface so WiFi can be reconfigured without recompiling firmware.

Changes:

  • Extend NodePrefs with persisted wifi_ssid/wifi_pwd fields.
  • Persist/load WiFi credentials in DataStore and use them during ESP32 WiFi startup.
  • Add CLI rescue commands to set/get WiFi SSID/password and restart WiFi after updates.

Reviewed changes

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

File Description
examples/companion_radio/NodePrefs.h Adds persisted fields for WiFi SSID/password.
examples/companion_radio/DataStore.cpp Saves/loads the new WiFi fields as part of prefs storage.
examples/companion_radio/MyMesh.cpp Adds “effective WiFi creds” helpers, WiFi restart helper, and CLI rescue set/get commands.
examples/companion_radio/main.cpp Uses persisted WiFi credentials (fallback to build-time) when starting WiFi on ESP32.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@dz0ny dz0ny force-pushed the feat/configure-wifi-via-cli branch 9 times, most recently from df9280d to 5e23630 Compare April 1, 2026 08:39
@dz0ny dz0ny force-pushed the feat/configure-wifi-via-cli branch from 5e23630 to e43c584 Compare April 1, 2026 08:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants