Skip to content

Comments

Add back reboot button#168

Merged
CoMPaTech merged 4 commits intomainfrom
reboot
Feb 21, 2026
Merged

Add back reboot button#168
CoMPaTech merged 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).
⚠️ Report is 7 commits behind head on main.

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.

@CoMPaTech CoMPaTech merged commit fbd3aed into main Feb 21, 2026
17 of 19 checks passed
@CoMPaTech CoMPaTech deleted the reboot branch February 21, 2026 15:45
@dosubot
Copy link

dosubot bot commented Feb 21, 2026

Related Documentation

Checked 0 published document(s) in 1 knowledge base(s). No updates required.

How did I do? Any feedback?  Join Discord

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