Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Pylint plugin for Home Assistant
Custom pylint checkers for the Home Assistant codebase. These checkers enforce coding standards, quality scale compliance, and common review patterns specific to Home Assistant integrations.
This plugin extends pylint with checks that cannot be expressed as
Ruff rules because they require type
inference, cross-file analysis (reading manifest.json, quality_scale.yaml),
or AST patterns that go beyond what a linter operating on a single file
can detect.
Setup
This plugin is designed for internal use by Home Assistant Core's CI/CD pipeline. It is not intended for external use, such as linting custom integration repositories.
Plugins are loaded via the load-plugins setting in pyproject.toml.
The init-hook adds pylint/plugins to sys.path so these modules
are importable.
Why not (just) Ruff?
Home Assistant uses both Ruff and pylint. Ruff handles fast, single-file linting (import sorting, formatting, common Python issues). These pylint checkers cover patterns that Ruff cannot:
- Cross-file analysis: reading
manifest.jsonto checkintegration_type, readingquality_scale.yamlto verify claims. - Type inference: resolving decorator names like
_pytest.fixtures.FixtureFunctionMarkerthrough imports. - Complex AST patterns: tracing variable assignments to API calls
(e.g., detecting that a variable assigned from
data[CONF_HOST]later flows intoasync_set_unique_id()).
Disabling checks
Always use the rule name (e.g., home-assistant-logger-period) rather than the
error code (e.g., C7401) for readability.
Single line -- add the disable comment at the end of the line:
hass.data[DOMAIN] = data # pylint: disable=home-assistant-use-runtime-data
Next line only -- if the inline comment would make the line too long,
use disable-next on the line above:
# pylint: disable-next=home-assistant-use-runtime-data
hass.data[DOMAIN] = data
Entire module -- place the disable comment at the top of the file, after the module docstring:
"""My integration setup."""
# pylint: disable=home-assistant-use-runtime-data
Automated code analysis
Every check has a code following the pylint convention:
{C,W,E,R}74{00-99}, where74is the base ID for Home Assistant.C= Convention,W= Warning,E= Error,R= Refactor.
| Code | Rule | Description |
|---|---|---|
C7401 |
home-assistant-logger-period |
Logger messages must not end with a period |
C7402 |
home-assistant-logger-capital |
Logger messages must start with a capital letter or use debug level |
E7401 |
home-assistant-invalid-inheritance |
Invalid entity class inheritance chain |
C7403 |
home-assistant-relative-import |
Use relative imports within an integration |
W7401 |
home-assistant-deprecated-import |
Import uses a deprecated path |
C7404 |
home-assistant-absolute-import |
Use absolute imports for cross-integration references |
C7405 |
home-assistant-component-root-import |
Do not import from another integration's internals |
C7406 |
home-assistant-helper-namespace-import |
Use the helper namespace import pattern |
C7407 |
home-assistant-import-constant-alias |
Aliased DOMAIN import needs a descriptive alias |
C7408 |
home-assistant-import-constant-unnecessary-alias |
Unnecessary alias when importing DOMAIN within the same integration |
E7402 |
home-assistant-argument-type |
Function argument should have the specified type hint |
E7403 |
home-assistant-return-type |
Function should have the specified return type hint |
R7401 |
home-assistant-consider-usefixtures-decorator |
Use @pytest.mark.usefixtures for unused fixtures |
E7404 |
home-assistant-missing-super-call |
Method must call its parent via super() |
C7409 |
home-assistant-enforce-sorted-platforms |
PLATFORMS list must be sorted alphabetically |
C7410 |
home-assistant-enforce-greek-micro-char |
Use Greek mu (U+03BC), not ANSI micro sign (U+00B5) |
C7411 |
home-assistant-enforce-class-module |
Entity class should be in the correct platform module |
W7402 |
home-assistant-async-callback-decorator |
Coroutine should not be decorated with @callback |
W7403 |
home-assistant-pytest-fixture-decorator |
Pytest fixture has invalid scope or autouse config |
W7404 |
home-assistant-async-load-fixtures |
Test fixture files should be loaded asynchronously |
W7405 |
home-assistant-use-runtime-data |
Use entry.runtime_data instead of hass.data[DOMAIN] |
W7406 |
home-assistant-unique-id-ip-based |
Unique ID should not be based on IP/hostname |
W7407 |
home-assistant-config-flow-polling-field |
Config flow should not include polling interval fields |
W7408 |
home-assistant-config-flow-name-field |
Config flow should not include name fields |
R7402 |
home-assistant-unused-test-fixture-argument |
Unused test function argument should use @pytest.mark.usefixtures |
home_assistant_logger checker
Enforces consistent formatting of logger messages across the codebase.
C7401: home-assistant-logger-period
User-visible logger messages must not end with a period. Log messages in Home Assistant follow a convention of not using trailing punctuation.
C7402: home-assistant-logger-capital
Logger messages must start with a capital letter. Debug-level messages are exempt from this rule. If a message does not warrant capitalization, consider downgrading it to debug level.
home_assistant_imports checker
Enforces import conventions for Home Assistant integrations. Integrations should use relative imports for their own modules and follow specific patterns for cross-integration references.
C7403: home-assistant-relative-import
Use relative imports within an integration (e.g., from .const import DOMAIN
instead of from homeassistant.components.myintegration.const import DOMAIN).
W7401: home-assistant-deprecated-import
Import uses a deprecated path that has been moved or renamed.
C7404: home-assistant-absolute-import
Use absolute imports when referencing modules outside the current integration.
C7405: home-assistant-component-root-import
Do not import from another integration's internal modules. Only import from the integration's top-level public API.
C7406: home-assistant-helper-namespace-import
Use the helper namespace import pattern for helper modules.
C7407: home-assistant-import-constant-alias
Aliased DOMAIN import from another integration should use a descriptive alias.
C7408: home-assistant-import-constant-unnecessary-alias
Unnecessary alias when importing DOMAIN from within the same integration.
home_assistant_enforce_type_hints checker
Enforces type hints on platform functions, config flow methods, and test functions. Checks both argument types and return types against the expected signatures defined by Home Assistant's platform interfaces.
E7402: home-assistant-argument-type
Function argument should have the specified type hint. Platform functions
like async_setup_entry have well-defined signatures that must be followed.
E7403: home-assistant-return-type
Function should have the specified return type hint.
R7401: home-assistant-consider-usefixtures-decorator
Test function should use @pytest.mark.usefixtures("fixture_name") instead
of accepting an unused fixture as a parameter.
home_assistant_decorator checker
Validates decorator usage on functions and fixtures.
W7402: home-assistant-async-callback-decorator
A coroutine function (async def) should not be decorated with @callback.
The @callback decorator is only for synchronous functions that should be
called from the event loop without scheduling.
W7403: home-assistant-pytest-fixture-decorator
Pytest fixture has invalid scope or autouse configuration. For example,
session-scoped fixtures in component tests should use package scope
or lower.
home_assistant_inheritance checker
Validates that entity platform modules only use entity classes from their
own platform (e.g., a sensor.py module should not inherit from
BinarySensorEntity).
E7401: home-assistant-invalid-inheritance
A platform module uses an entity class from a different platform. For
example, a sensor.py file should not define classes inheriting from
BinarySensorEntity.
home_assistant_enforce_super_call checker
Ensures methods call their parent implementation when required.
E7404: home-assistant-missing-super-call
Method must call its parent implementation via super(). Certain entity
methods require calling the parent to maintain correct behavior.
home_assistant_enforce_sorted_platforms checker
Ensures platform lists are maintained in alphabetical order.
C7409: home-assistant-enforce-sorted-platforms
The PLATFORMS (or _PLATFORMS) list must be sorted alphabetically. This
makes it easier to review and prevents merge conflicts.
home_assistant_enforce_greek_micro_char checker
Ensures correct Unicode character for the micro prefix.
C7410: home-assistant-enforce-greek-micro-char
Constants with a micro unit prefix (e.g., "μg/m³") must use the Greek
small letter mu (U+03BC μ), not the ANSI micro sign (U+00B5 µ). The
two characters look identical but are different Unicode code points.
home_assistant_enforce_class_module checker
Ensures entity classes are placed in the correct module.
C7411: home-assistant-enforce-class-module
A class deriving from a platform entity (e.g., SensorEntity) should be
placed in the corresponding platform module (e.g., sensor.py), not in
__init__.py or an unrelated module.
home_assistant_enforce_runtime_data checker
Enforces the modern entry.runtime_data pattern over the legacy
hass.data[DOMAIN] dictionary pattern. Only flags integrations that
have a config flow (YAML-only integrations are skipped).
W7405: home-assistant-use-runtime-data
Use entry.runtime_data instead of hass.data[DOMAIN]. The runtime_data
approach is type-safe (via ConfigEntry[T]), automatically cleaned up on
entry unload, and avoids key collisions in the shared hass.data dictionary.
See the runtime-data quality scale rule for migration guidance.
home_assistant_async_load_fixtures checker
Ensures test fixture files are loaded asynchronously.
W7404: home-assistant-async-load-fixtures
Test fixture files should be loaded using async I/O, not synchronous file reads. This prevents blocking the event loop during tests.
home_assistant_enforce_config_entry_unique_id_no_ip checker
Detects async_set_unique_id calls where the argument is an IP address
or hostname. IP addresses change when devices get new DHCP leases, breaking
the config entry. Uses variable tracking to catch indirect usage (e.g.,
uid = data[CONF_HOST]; await self.async_set_unique_id(uid)).
W7406: home-assistant-unique-id-ip-based
async_set_unique_id should not use an IP address or hostname. Use a
stable hardware identifier instead: a MAC address (via format_mac),
serial number, or device-provided unique ID.
See the unique-config-entry quality scale rule.
home_assistant_enforce_config_flow_no_polling checker
Detects polling interval fields in config flow schemas. Polling intervals should be fixed by the integration author, not exposed as user-configurable fields.
W7407: home-assistant-config-flow-polling-field
Config flow should not include polling interval fields like
CONF_SCAN_INTERVAL, update_interval, or refresh_interval. The
integration author determines the appropriate polling frequency based on
API rate limits, device capabilities, and data freshness needs.
See the appropriate-polling quality scale rule.
home_assistant_enforce_config_flow_no_name checker
Detects name fields (CONF_NAME, "name", CONF_DEVICE_NAME,
"device_name") in config flow schemas. Config flows should not ask
users to provide a name -- the name is automatically derived from the
device (via discovery) or set by the integration code itself.
Helper integrations (integration_type: helper in manifest.json) and
subentry flows (ConfigSubentryFlow subclasses) are excluded.
W7408: home-assistant-config-flow-name-field
Config flow should not include a name field. Users should not set names in config flows; they come automatically from the device or are set by the integration.
home_assistant_unused_test_fixture_args checker
Disabled by default while existing violations are being cleaned up.
R7402: home-assistant-unused-test-fixture-argument
Test functions that receive a fixture argument but never reference it in
the function body should use @pytest.mark.usefixtures("name") instead.
This keeps the function signature clean and makes it clear the fixture is
only needed for its side effects.
This rule only applies to test_* functions, not to fixture functions.