supervisor/**/*.py: Use Python 3.13+ and modern Python features: type hints with typing module, f-strings, dataclasses, enum classes, async/await, and pattern matching where appropriate
Use Ruff for code formatting
Use PyLint and Ruff for linting
Use MyPy for type checking
Use constants from supervisor/const.py instead of hardcoding values; define new constants following existing patterns and group related constants together
Inherit from CoreSysAttributes for major classes to provide access to centralized system state via self.coresys and convenient sys_* properties
Implement a load() method for components that initialize from external sources (containers, files, D-Bus services)
Use Supervisor's Docker manager (self.sys_docker) for container management instead of direct Docker API usage
All I/O operations must be async: file operations, network calls, subprocess execution
Prefer asyncio.gather() over sequential awaits for concurrent async operations
Use self.sys_run_in_executor() for blocking operations
Use two-phase initialization: __init__ for sync setup, post_init() for async initialization
Use the from clause for exception chaining when propagating errors
Avoid direct Docker API usage; use Supervisor's Docker manager instead
Avoid blocking operations in async context; use asyncio alternatives
Avoid hardcoded values; use constants from const.py

📄 Source: CodeRabbit Inference Engine (.github/copilot-instructions.md)

List of files the instruction was applied to:

• supervisor/const.py
• supervisor/addons/manager.py
• supervisor/api/addons.py

*/**(html|markdown|md): - For instructional content in documentation, use a direct and authoritative tone. Avoid expressions of politeness such as 'may' or 'please', and ensure the goal of the instruction is fronted.

• Apply the Microsoft Style Guide to ensure documentation maintains clarity and conciseness.
• In step-by-step instructions, front the location phrase in the instructional sentence.
• In step-by-step instructions, front the 'goal' in the instructional sentence.
• In step-by-step instructions, if in doubt what to front, front the 'goal' before the location phrase in the instructional sentence.
• do not hyphenate terms like 'top-right' or 'bottom-left' with 'corner'

⚙️ Source: CodeRabbit Configuration File

List of files the instruction was applied to:

• supervisor/const.py

*/**(html|markdown|md): - Use bold to mark UI strings.

• If "" are used to mark UI strings, replace them by bold.

⚙️ Source: CodeRabbit Configuration File

List of files the instruction was applied to:

• supervisor/const.py

*/**(html|markdown|md): - Be brief in your replies and don't add fluff like "thank you for..." and "Please let me know if"

⚙️ Source: CodeRabbit Configuration File

List of files the instruction was applied to:

• supervisor/const.py

*/**(html|markdown|md): - Use sentence-style capitalization also in headings.

⚙️ Source: CodeRabbit Configuration File

List of files the instruction was applied to:

• supervisor/const.py

*/**(html|markdown|md): do not comment on HTML used for icons

⚙️ Source: CodeRabbit Configuration File

List of files the instruction was applied to:

• supervisor/const.py

*/**(html|markdown|md): Avoid flagging inline HTML for embedding videos in future reviews for this repository.

⚙️ Source: CodeRabbit Configuration File

List of files the instruction was applied to:

• supervisor/const.py

supervisor/addons/**/*.py: AppArmor profiles are mandatory for add-ons, with minimal capabilities

📄 Source: CodeRabbit Inference Engine (.github/copilot-instructions.md)

List of files the instruction was applied to:

• supervisor/addons/manager.py

supervisor/api/**/*.py: All REST API endpoints must be under /api/, use bearer token authentication, consistent response format, and voluptuous schemas with api_validate()
Use the @api_process decorator for API endpoints to handle standard error handling and response formatting
Use APIError with appropriate HTTP status codes for API errors
Use token-based API authentication with role-based access
Avoid manual error handling in API endpoints; let @api_process handle it

📄 Source: CodeRabbit Inference Engine (.github/copilot-instructions.md)

List of files the instruction was applied to:

• supervisor/api/addons.py

tests/**/*.py: Use pytest with asyncio support for testing
Place all tests in the tests/ directory with module mirroring
Use pytest fixtures extensively for CoreSys setup in tests
Mock external dependencies (Docker, D-Bus, network calls) in tests
Maintain minimum 90% test coverage, 100% for security-sensitive code

📄 Source: CodeRabbit Inference Engine (.github/copilot-instructions.md)

List of files the instruction was applied to:

• tests/api/test_addons.py

tests/conftest.py (7)

• api_client (514-537)
• coresys (332-415)
• install_addon_ssh (627-635)
• container (801-809)
• tmp_supervisor_data (430-449)
• path_extern (92-95)
• docker (106-136)

supervisor/addons/addon.py (5)

• Addon (132-1606)
• load (209-242)
• state (166-168)
• state (171-202)
• container_state_changed (1566-1584)

supervisor/const.py (2)

• AddonState (446-453)
• CpuArch (511-518)

supervisor/docker/const.py (1)

• ContainerState (30-38)

[error] 285-285: nonlocal state_changes is unused: name is never assigned in scope

(F824)