feat(autoresearch): verify RP2350W peer echo payload integrity

[zackees]

Summary

• make the RP2350W AutoResearch peer client POST a deterministic 4,096-byte payload to the C6 echo fixture
• require HTTP success, exact echoed byte count, and matching FNV-1a checksum before network validation passes
• retain bounded waits and feed the FastLED watchdog while receiving the response

Verification

• �ash lint --cpp --strict examples/AutoResearch/AutoResearchNet.cpp
• �ash compile rp2350w --examples AutoResearch --no-interactive

This proves the RP2350W-to-C6 payload direction in source/build; live HIL and C6-to-RP payload coverage remain follow-up gates.

Summary by CodeRabbit

• New Features

  • Added a 4,096-byte POST /echo network test for RP2350W.
  • The test validates HTTP status, response size, and payload integrity.

• Tests

  • Expanded the client test suite from six to seven tests.
  • Updated reported success and failure totals.

[coderabbitai]

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 715e7c27-f041-460a-9572-464a10438a79

📥 Commits

Reviewing files that changed from the base of the PR and between 49fb846 and c26b7ef.

📒 Files selected for processing (1)

• examples/AutoResearch/AutoResearchNet.cpp

---

📝 Walkthrough

Walkthrough

The RP2350W network client adds a deterministic 4,096-byte POST /echo test. It verifies the response status, length, and FNV-1a hash, then updates the suite to report seven tests.

Changes

RP2350W HTTP echo validation

Layer / File(s)	Summary
Payload generation and integrity validation examples/AutoResearch/AutoResearchNet.cpp	Adds fl::array storage, an FNV-1a helper, and a watchdog-safe /echo test for deterministic 4,096-byte payloads.
Network test suite integration examples/AutoResearch/AutoResearchNet.cpp	Runs the payload test and updates aggregate success and failure totals from six tests to seven.

Estimated code review effort: 3 (Moderate) | ~20 minutes

Sequence Diagram(s)

sequenceDiagram
participant RP2350WClient
participant EchoEndpoint
participant Watchdog
RP2350WClient->>RP2350WClient: Generate payload and compute hash
RP2350WClient->>EchoEndpoint: POST 4,096-byte payload to /echo
EchoEndpoint-->>RP2350WClient: Return HTTP response
RP2350WClient->>Watchdog: Feed watchdog during polling
RP2350WClient->>RP2350WClient: Validate status, length, and hash

Loading

Possibly related issues

• feat(rp2350w): complete pin-free AutoResearch and ESP32-C6 peer-network bring-up #3832 — Describes the deterministic 4 KiB HTTP echo validation implemented by this change.

Possibly related PRs

• FastLED/FastLED#3608 — Both changes modify HTTP endpoint tests in runNetClientTest.
• FastLED/FastLED#3843 — This change extends the RP2350W network client implementation from that PR.
• FastLED/FastLED#3853 — This change implements the planned 4 KiB payload and hash verification follow-up.

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch feat/rp2350w-net-payload-hash

---

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.}