Add external node for e-meter on VCAN

[PatMcN7]

Resolves https://app.notion.com/p/EMeter-CAN-VCAN-3a73b28e58b680c99728f6b1d9c604af

[gitar-bot]

CI failed: GitHub Actions labeler workflow failed due to insufficient token permissions, which is unrelated to the PR changes.

Overview

One failure pattern was found across 2 logs related to a GitHub Actions workflow configuration permission issue.

Failures

GitHub Actions Labeler Permission Error (confidence: high)

• Type: configuration
• Affected jobs: 104595530171, 104615782724
• Related to change: no
• Root cause: The GitHub Actions workflow lacks the necessary pull-requests or issues write permissions required to set or modify labels via the GitHub API.
• Suggested fix: Update the workflow file to include appropriate permissions, specifically granting pull-requests: write permissions for the labeler job.

Summary

• Change-related failures: 0 failures
• Infrastructure/flaky/workflow failures: 2 failures due to missing workflow permissions
• Recommended action: Update the repository workflow permissions or the workflow YAML file to grant the necessary integration permissions.

Code Review ✅ Approved 1 closed / 1 findings

Adds external node configuration for the e-meter on VCAN with measurement, status, and temperature messages. The E_METER_STATUS energy float byte-alignment issue has been resolved. No issues remain.

✅ 1 closed

✅ Bug: E_METER_STATUS energy float no longer byte-aligned

📄 firmware/generators/configs/external_nodes/E_METER.json:43-51 📄 firmware/generators/configs/external_nodes/E_METER.json:52-66
The old 6-bit reserved field kept the first byte full (violation+logging+reserved=8 bits), so energy started at a byte boundary. Replacing it with only two 1-bit bools (fault_active, fault_prev_active) leaves just 4 bits before energy, so the 32-bit float is now packed at bit offset 4 and straddles bytes, and the following reserved bytes shift too. Since E_METER is an external node describing a device we don't generate firmware for, this layout must match the real E-Meter frame — if the device byte-aligns energy, every decode will be wrong. Add a 4-bit reserved field after fault_prev_active to restore byte alignment (violation+logging+fault_active+fault_prev_active+reserved(4) = 8 bits).

Review coverage

Rules No rules evaluated

Functional validation Not enabled · Set up

Tip

Comment Gitar fix CI or enable auto-apply: gitar auto-apply:on

Options

Auto-apply is off → Gitar will not commit updates to this branch.
Display: compact → Counting what did not apply, without listing it.

Comment with these commands to change the behavior for this request:

Auto-apply	Compact
gitar auto-apply:on	gitar display:verbose

_{Was this helpful? React with 👍 / 👎 | Powered by Gitar — free for open source}