fix(Fix_unreachable_netservices_v2): v16 compatibility fixes

[Joeboyc2]

Summary

Two issues prevent Fix_unreachable_netservices_v2 from building under WLED v16:

• Missing libArchive: false in library.json — the v16 build system requires "build": {"libArchive": false} in every usermod's library.json. Without it the build fails immediately with ERROR: libArchive=false is missing on usermod(s).

• serializeConfig() no longer has a no-argument overload — v16 changed serializeConfig() to require a JsonObject parameter. The correct no-argument replacement for saving config to the filesystem is serializeConfigToFS().

Test plan

• Fix_unreachable_netservices_v2 compiles cleanly in an ESP8266 build environment with both fixes applied
• Verified serializeConfigToFS() is the correct equivalent — it allocates the JSON document internally and calls serializeConfig(root) + writes to LittleFS

🤖 Generated with Claude Code

Summary by CodeRabbit

• Bug Fixes

  • Fixed an issue where network service configuration updates were not being properly persisted to storage.

• Chores

  • Updated library build configuration settings.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: ccdd8ff5-ac87-42d1-857e-1f0d786c3a4e

📥 Commits

Reviewing files that changed from the base of the PR and between 37623ed and aba977b.

📒 Files selected for processing (2)

• usermods/Fix_unreachable_netservices_v2/library.json
• usermods/Fix_unreachable_netservices_v2/usermod_Fix_unreachable_netservices.cpp

---

Walkthrough

This PR fixes the FixUnreachableNetServices usermod by ensuring periodic configuration updates are persisted to the filesystem instead of kept in memory. The library manifest is also updated to control build archiving behavior.

Changes

Fix unreachable netservices v2

Layer / File(s)	Summary
Build Configuration usermods/Fix_unreachable_netservices_v2/library.json	Library manifest adds build.libArchive: false to control archiving behavior during the build process.
Runtime Config Persistence Fix usermods/Fix_unreachable_netservices_v2/usermod_Fix_unreachable_netservices.cpp	Config update handler switches from serializeConfig() to serializeConfigToFS() to persist settings to filesystem during periodic updates.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly and concisely summarizes the main change: v16 compatibility fixes for the Fix_unreachable_netservices_v2 usermod.
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

Tip

💬 Introducing Slack Agent: The best way for teams to turn conversations into code.

Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.

• Generate code and open pull requests
• Plan features and break down work
• Investigate incidents and troubleshoot customer tickets together
• Automate recurring tasks and respond to alerts with triggers
• Summarize progress and report instantly

Built for teams:

• Shared memory across your entire org—no repeating context
• Per-thread sandboxes to safely plan and execute work
• Governance built-in—scoped access, auditability, and budget controls

One agent for your entire SDLC. Right inside Slack.

👉 Get started

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[softhack007]

Looks good for me - I don't have any 8266 for testing, however i've checked that serializeConfigToFS() is a proper replacement. In fact, configNeedsWrite = true; would have been sufficient to trigger config writing in the main loop, but it should also work as proposed in this PR.

[Joeboyc2]

Thanks Frank, Ive deployed this to 3 of my esp8266s devices and all seems to working well

…

On Mon May 11, 2026, 06:03 PM GMT, Frank Möhle ***@***.***> wrote: @softhack007 approved this pull request. Looks good for me - I don't have any 8266 for testing, however i've checked that serializeConfigToFS() is a proper replacement. In fact, configNeedsWrite = true; would have been sufficient to trigger config writing in the main loop, but it should also work as proposed in this PR. — Reply to this email directly, view it on GitHub <#5590 (review)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AB6SOLW4CQJUSARXX4PHVAT42IIYTAVCNFSM6AAAAACYX434R2VHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHM2DENRWGE2TGMRSGM>. You are receiving this because you authored the thread.Message ID: ***@***.***>