Skip to content

Add back reboot button#168

Open
CoMPaTech wants to merge 4 commits intomainfrom
reboot
Open

Add back reboot button#168
CoMPaTech wants to merge 4 commits intomainfrom
reboot

Conversation

@CoMPaTech
Copy link
Owner

@CoMPaTech CoMPaTech commented Feb 13, 2026

Summary by CodeRabbit

  • New Features
    • Re-added device reboot capability accessible via the public API.
  • Documentation
    • README and changelog updated to document the reboot feature and the new release.

@CoMPaTech CoMPaTech requested a review from Copilot February 13, 2026 13:40
@dosubot dosubot bot added the size:S This PR changes 10-29 lines, ignoring generated files. label Feb 13, 2026
@coderabbitai
Copy link

coderabbitai bot commented Feb 13, 2026

Warning

Rate limit exceeded

@CoMPaTech has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 15 minutes and 11 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

Walkthrough

Adds a new AirOS.reboot() method that POSTs to a reboot.cgi endpoint and returns a boolean, reintroduces reboot functionality in CHANGELOG.md, bumps project version to 0.6.4, and documents the API change in README.md.

Changes

Cohort / File(s) Summary
Version & Changelog
CHANGELOG.md, pyproject.toml
Bump project version to 0.6.4 and add changelog entry documenting the re-added reboot functionality.
API & Implementation
airos/base.py
Add _reboot_cgi_url attribute and public reboot() method on AirOS that sends a POST to reboot.cgi with form data {"reboot":"yes"} and returns True when the response indicates success.
Docs
README.md
Document the new reboot() method in the device control API examples/documentation.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Poem

🐇 I nibble keys and tap the log,
A tiny hop to clear the fog.
Push the CGI, the bytes take flight,
Reboot returns — all systems bright.
Version bumped, I dance tonight ✨

🚥 Pre-merge checks | ✅ 4
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The PR title 'Add back reboot button' accurately describes the main change - reintroducing reboot functionality to the AirOS library, as evidenced by changes adding a reboot() method and updating documentation.
Docstring Coverage ✅ Passed Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.
Merge Conflict Detection ✅ Passed ✅ No merge conflicts detected when merging into main

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

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch reboot

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.

@codecov
Copy link

codecov bot commented Feb 13, 2026

Codecov Report

❌ Patch coverage is 33.33333% with 4 lines in your changes missing coverage. Please review.
✅ Project coverage is 86.64%. Comparing base (8c38230) to head (56b6bb1).

Files with missing lines Patch % Lines
airos/base.py 33.33% 4 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #168      +/-   ##
==========================================
- Coverage   87.07%   86.64%   -0.44%     
==========================================
  Files           8        8              
  Lines         743      749       +6     
==========================================
+ Hits          647      649       +2     
- Misses         96      100       +4     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@dosubot dosubot bot added the enhancement New feature or request label Feb 13, 2026
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR re-introduces device reboot support in the airos Python library and publishes it as a new patch release.

Changes:

  • Bump project version from 0.6.3 to 0.6.4
  • Add a new AirOS.reboot() API calling /reboot.cgi
  • Document the change in CHANGELOG.md

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.

File Description
pyproject.toml Version bump to 0.6.4 for the reboot re-addition release.
airos/base.py Adds reboot endpoint URL and new reboot() async method.
CHANGELOG.md Adds 0.6.4 entry noting reboot functionality is re-added.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 Fix all issues with AI agents
In `@README.md`:
- Line 44: The sentence "Device Control: Provides methods to perform actions,
such as reconnecting/kicking a connected wireless station and rebooting the
device itself.." has an extra period; remove the duplicate period so it ends
with a single period. Locate the exact string "Device Control: Provides methods
to perform actions, such as reconnecting/kicking a connected wireless station
and rebooting the device itself.." in README.md and change the trailing ".." to
"." ensuring no other punctuation is altered.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request size:S This PR changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant