Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Sep 13, 2025

This PR fixes a critical issue where August Smart Lock Pro + Gen 3 devices are detected by the plugin but don't appear in HomeKit, leaving users confused with no visible error messages.

Problem

When August locks have homeKitEnabled: true (common for Pro models), they won't be registered with Homebridge unless the user explicitly sets overrideHomeKitEnabled: true in their configuration. However, the error message explaining this requirement was only logged at debug level (debugErrorLog), making it invisible to most users who don't have debug logging enabled.

Users would see:

[August] Total August Locks Found: 1

But no locks would appear in HomeKit, with no indication of why.

Solution

  1. Improved error visibility: Changed debugErrorLog to errorLog for homeKitEnabled devices without override, making the error visible to all users regardless of logging level.

  2. Clearer error message: Replaced the technical error message with a user-friendly one:

    Device: [Lock Name] already has HomeKit enabled. To register with Homebridge, add "overrideHomeKitEnabled": true to your device config.
    
  3. Better general error handling: Changed generic device registration failures from debugErrorLog to warnLog for improved visibility.

  4. Comprehensive testing: Added 4 new test cases covering all registerDevice scenarios to prevent regressions.

  5. Documentation: Added a troubleshooting section to the README with clear configuration examples.

Impact

Users will now immediately see why their August lock isn't appearing in HomeKit and know exactly how to fix it. No more silent failures or hunting through debug logs to understand configuration issues.

Fixes #178.


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI changed the title [WIP] Support Request: Fix August lock not appearing in HomeKit due to invisible error messages Sep 13, 2025
Copilot AI requested a review from donavanbecker September 13, 2025 02:34
Copilot finished work on behalf of donavanbecker September 13, 2025 02:34
@donavanbecker donavanbecker changed the base branch from latest to beta-3.0.4 September 13, 2025 02:58
@donavanbecker donavanbecker marked this pull request as ready for review September 13, 2025 02:58
@donavanbecker donavanbecker merged commit 6724506 into beta-3.0.4 Sep 13, 2025
3 checks passed
@donavanbecker donavanbecker deleted the copilot/fix-178 branch September 13, 2025 02:58
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.

Support Request:

2 participants