feat: implement MQTT publishing for lap data in LapController and SolarMQTTClient

[dastine0308]

Summary by CodeRabbit

• New Features
  • Lap data is now broadcast in real time, published over MQTT for external integration, and persisted to the database for reliable storage and later analysis.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
helios-telemetry	Ready	Preview, Comment	Jan 31, 2026 6:58pm

[coderabbitai]

📝 Walkthrough

Walkthrough

Adds MQTT lap-data publishing: a new topic and publish method in the MQTT client, and an MQTT publish step in the LapController's handleLapData flow before database insertion.

Changes

Cohort / File(s)	Summary
MQTT Topic Definition packages/server/src/datasources/SolarMQTTClient/SolarMQTTClient.types.ts	Added lapdataTopic: "lapdata" to the exported topics object.
MQTT Client Publisher packages/server/src/datasources/SolarMQTTClient/SolarMQTTClient.ts	Imported ILapData and added publishLapData(lapdata: ILapData) which publishes JSON.stringify(lapdata) to the lapdataTopic with error logging.
Lap Controller Integration packages/server/src/controllers/LapController/LapController.ts	Inserted a call to the MQTT client's publishLapData in handleLapData so lap data is published over MQTT before being inserted into DynamoDB.

Sequence Diagram

sequenceDiagram
    participant LapController as LapController
    participant MQTTClient as SolarMQTTClient
    participant Broker as MQTT_Broker
    participant DB as DynamoDB

    LapController->>MQTTClient: publishLapData(lapdata)
    MQTTClient->>MQTTClient: JSON.stringify(lapdata)
    MQTTClient->>Broker: publish to "lapdata" topic
    Broker-->>MQTTClient: ack
    MQTTClient-->>LapController: return
    LapController->>DB: insert lap data
    DB-->>LapController: ack

Loading

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 I hopped a small packet into the night,
Published lap-times with a soft little byte,
Over MQTT the moments flew,
Now laps sing loud and records grew,
A rabbit cheered — "Data looks right!"

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately describes the main change: implementing MQTT publishing for lap data across LapController and SolarMQTTClient, which aligns with the file modifications shown in the summary.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch TEL-318-Send-Lap-data-through-MQTT

---

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

[justin-phxm]

Clean code 🔥

[promatty]

lgtm

[justin-phxm]

lgtm