Skip to content

fix: update shaper command API parameters, track status, and bump to 0.3.3#571

Merged
firstof9 merged 2 commits into
mainfrom
version-bump
May 16, 2026
Merged

fix: update shaper command API parameters, track status, and bump to 0.3.3#571
firstof9 merged 2 commits into
mainfrom
version-bump

Conversation

@firstof9
Copy link
Copy Markdown
Owner

@firstof9 firstof9 commented May 16, 2026

This PR includes the latest updates for python-openevse-http:

Highlights

  • Fix: Updated set_shaper command parameters to use {"shaper": mode} instead of {"mode": mode} and process the request via RAPI parameters (rapi=data) to resolve shaper activation issues on OpenEVSE chargers.
  • Feature: Added status tracking to automatically sync the local client state (divertmode and shaper) when issuing mode-change commands.
  • Chore: Bumped the library version in setup.py to 0.3.3.

Details

  • Updates openevsehttp/commands.py set_shaper to send a POST request with RAPI data.
  • Automatically stores updated state on set_shaper and set_divert_mode.
  • Suppresses noise by lowering warnings for non-JSON responses to DEBUG level.
  • Adds mock tests in tests/test_shaper.py to cover non-JSON status updates.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 16, 2026

📝 Walkthrough

Walkthrough

The package version constant in setup.py is incremented from 0.3.2 to 0.3.3, updating the version metadata used by the package distribution mechanism.

Changes

Package Version Update

Layer / File(s) Summary
Version constant update
setup.py
VERSION constant incremented from "0.3.2" to "0.3.3" for the package release.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~1 minute

Possibly related PRs

Poem

🐰 A tiny bump, from two to three,
The version hops along with glee,
No breaking change, just increments small,
Zero-three-three rises tall!

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Title check ⚠️ Warning The PR title mentions updating shaper command API parameters, tracking status, and bumping to 0.3.3, but the changeset only contains a version bump from 0.3.2 to 0.3.3 in setup.py with no other changes present. Update the title to accurately reflect the actual changes: 'Bump version to 0.3.3' or clarify if other changes are missing from this PR.
✅ Passed checks (4 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.

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

✨ 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 version-bump

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

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

@firstof9 firstof9 changed the title Bump version to 0.3.3 fix: update shaper command API parameters, track status, and bump to 0.3.3 May 16, 2026
@firstof9
Copy link
Copy Markdown
Owner Author

PR has been updated to include the fix for OpenEVSE shaper command parameters and status tracking updates alongside the package version bump to 0.3.3.

Summary of latest updates included:

  • Shaper Activation Fix: Corrected command payload ({"shaper": mode}) and routed the post requests through the rapi parameters in set_shaper.
  • Status Syncing: Local client status is now updated automatically on both set_shaper and set_divert_mode.
  • Test Coverage: Added regression tests in tests/test_shaper.py targeting non-JSON response handling.
  • Log Noise Reduction: Shifted warning messages for non-JSON responses to debug-level logging.

@firstof9 firstof9 merged commit 347ac8b into main May 16, 2026
11 checks passed
@firstof9 firstof9 deleted the version-bump branch May 16, 2026 21:43
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.

1 participant