Add Eve Online integration by ronaldvdmeer · Pull Request #168143 · home-assistant/core

ronaldvdmeer · 2026-04-13T23:08:33Z

Proposed change

Adds a new Eve Online integration using OAuth2 (CCP ESI API). The integration provides three sensor entities per character:

Wallet balance: ISK balance (measurement sensor, 2 decimal precision)
Location: current solar system name (optional, degrades gracefully)
Ship: current ship type (optional, degrades gracefully)

Uses a single DataUpdateCoordinator polling at 60-second intervals. The wallet endpoint is the primary — a failure marks all entities unavailable. Location and ship failures are logged at debug level only.

Required OAuth2 scopes:

esi-location.read_location.v1
esi-location.read_ship_type.v1
esi-wallet.read_character_wallet.v1

Related PRs:

Brands: Add Eve Online brand brands#10133
Documentation: Add Eve Online integration documentation home-assistant.io#44762

Dependency libary changelog: https://github.com/ronaldvdmeer/python-eveonline/releases/tag/v0.4.0

Type of change

Dependency upgrade
Bugfix (non-breaking change which fixes an issue)
New integration (thank you!)
New feature (which adds functionality to an existing integration)
Deprecation (breaking change to happen in the future)
Breaking change (fix/feature causing existing functionality to break)
Code quality improvements to existing code or addition of tests

Additional information

This PR fixes or closes issue: fixes #
This PR is related to issue:
Link to documentation pull request: Add Eve Online integration documentation home-assistant.io#44762
Link to developer documentation pull request:
Link to frontend pull request:

Checklist

I understand the code I am submitting and can explain how it works.
The code change is tested and works locally.
Local tests pass. Your PR cannot be merged unless tests pass
There is no commented out code in this PR.
I have followed the development checklist
I have followed the perfect PR recommendations
The code has been formatted using Ruff (ruff format homeassistant tests)
Tests have been added to verify that the new code works.
Any generated code has been carefully reviewed for correctness and compliance with project standards.

If user exposed functionality or configuration variables are added/changed:

Documentation added/updated for www.home-assistant.io

If the code communicates with devices, web services, or third-party tools:

The manifest file has all fields filled out correctly.
Updated and included derived files by running: python3 -m script.hassfest.
New or updated dependencies have been added to requirements_all.txt.
Updated by running python3 -m script.gen_requirements_all.
For the updated dependencies a diff between library versions and ideally a link to the changelog/release notes is added to the PR description.

Adds an Eve Online integration using OAuth2 (CCP ESI API) that provides three sensor entities per character: - Status: online/offline (ENUM sensor with translated states) - Wallet balance: ISK balance (MEASUREMENT sensor, 2 decimal precision) - Location: current solar system name Uses a single DataUpdateCoordinator polling at 60-second intervals. Optional endpoints (wallet, location) degrade gracefully — the coordinator only fails hard when the character online status endpoint is unavailable. Required OAuth2 scopes: - esi-location.read_location.v1 - esi-location.read_online.v1 - esi-wallet.read_character_wallet.v1

- Strip to 3 sensors: wallet_balance (primary), location, ship - Remove PARALLEL_UPDATES (Silver-level) - Remove exception translations and icons.json (Gold-level) - Remove online sensor (would require binary_sensor platform) - Add ship sensor with esi-location.read_ship_type.v1 scope - Mark all Bronze quality scale rules as done

Copilot

Pull request overview

Adds a new Home Assistant eveonline integration that authenticates via OAuth2 (EVE SSO / CCP ESI) and exposes character wallet/location/ship as sensors.

Changes:

Introduces the eveonline integration (config flow, application credentials support, coordinator, entities, sensors, and translations).
Adds a full test suite for config flow, setup/unload behavior, coordinator/auth helpers, and sensor snapshots.
Registers the integration in generated metadata and adds the python-eveonline==0.4.0 dependency to requirements files.

Reviewed changes

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

Show a summary per file

File	Description
`homeassistant/components/eveonline/__init__.py`	Sets up OAuth2 session, client, coordinator, and forwards sensor platform setup.
`homeassistant/components/eveonline/api.py`	Adds config-entry-bound auth helper for the upstream client.
`homeassistant/components/eveonline/application_credentials.py`	Declares the EVE SSO authorization server endpoints for Application Credentials.
`homeassistant/components/eveonline/config_flow.py`	Implements OAuth2 config flow and derives character identity from the JWT access token.
`homeassistant/components/eveonline/const.py`	Defines domain constants, OAuth endpoints, and required scopes.
`homeassistant/components/eveonline/coordinator.py`	Polls wallet/location/ship via a `DataUpdateCoordinator`, treating wallet as the primary endpoint.
`homeassistant/components/eveonline/entity.py`	Provides shared device/unique-id behavior for character entities.
`homeassistant/components/eveonline/sensor.py`	Adds three sensor entities backed by coordinator data.
`homeassistant/components/eveonline/strings.json`	Adds config flow and entity name strings.
`homeassistant/components/eveonline/manifest.json`	Declares the new integration and dependency.
`homeassistant/components/eveonline/quality_scale.yaml`	Sets initial quality scale statuses for the new integration.
`tests/components/eveonline/*` + snapshots	Adds fixtures and tests for config flow, setup, auth helper, and sensor behavior.
`requirements_all.txt` / `requirements_test_all.txt`	Adds `python-eveonline==0.4.0`.
`homeassistant/generated/*`	Registers the new integration for config flows/application credentials/integrations metadata.
`CODEOWNERS`	Assigns ownership for the new integration and its tests.

…cale - Add freezer.tick(61) in test_sensor_wallet_updated so the coordinator update interval is exceeded before firing the time changed event - Add PR links to brands and docs quality scale entries to clarify that both PRs are already submitted

- Combine solar system and ship type resolve_names calls into one API call for efficiency - Show ship type name (e.g. 'Flycatcher') instead of the player-assigned ship name (e.g. 'i.probe')

Copilot

Pull request overview

Copilot reviewed 21 out of 24 changed files in this pull request and generated 2 comments.

async_unload_entry is implemented and tested in test_init.py

…en-unavailable as done - integration-owner: codeowners already set in manifest.json - entity-unavailable: CoordinatorEntity handles this automatically - log-when-unavailable: DataUpdateCoordinator logs this automatically

Copilot

Pull request overview

Copilot reviewed 21 out of 24 changed files in this pull request and generated 2 comments.

- test_sensor_location_unavailable_when_endpoint_fails -> test_sensor_location_shows_unknown_when_endpoint_fails - test_sensor_ship_unavailable_when_endpoint_fails -> test_sensor_ship_shows_unknown_when_endpoint_fails

Copilot

Pull request overview

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

erwindouna

Thanks for adding a new integration, @ronaldvdmeer. Here's an initial review.

…onUnavailableError, recovery test

erwindouna

I think you're progressing very well and we're almost nearing completion here, if you'd ask me. Just a few minor details. :)

Per review by erwindouna: reuse result rather than introducing result2/state2 in tests that run two flows sequentially.

Per review by erwindouna: use the STATE_UNAVAILABLE constant instead of the hardcoded string literal.

Per review by erwindouna: use the STATE_UNKNOWN constant instead of the hardcoded string literal.

Copilot

Pull request overview

Copilot reviewed 20 out of 23 changed files in this pull request and generated no new comments.

- model: 'Character' — consistent label for all character devices - configuration_url: evewho.com profile link for the character

ronaldvdmeer · 2026-04-19T21:47:25Z

I noticed the device_info was not yet fully populated. I took the liberty of completing it in c5bea38

ronaldvdmeer · 2026-04-19T22:04:46Z

I think you're progressing very well and we're almost nearing completion here, if you'd ask me. Just a few minor details. :)

Thanks a lot, really appreciate that. Glad to hear we’re getting close now. I’ve taken care of the remaining minor details so we can hopefully get this over the finish line soon. 👍

Copilot

Pull request overview

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

OAuth2-only integration; the EVE SSO token exchange is the only pre-configure step possible. No traditional connectivity test can be performed before the OAuth round-trip completes.

Copilot

Pull request overview

Copilot reviewed 20 out of 23 changed files in this pull request and generated no new comments.

ronaldvdmeer · 2026-05-02T19:51:00Z

The PR was still green on the 26 of April, so this looks more like newer core/CI behavior exposing the missing reauth path than a new review requirement. The integration already raises ConfigEntryAuthFailed, so core starts reauth automatically, and that now fails because the flow does not implement reauth yet.

ronaldvdmeer · 2026-05-02T20:03:39Z

I’ve now implemented the missing reauth path instead of broadening the scope: the OAuth flow now supports reauth, updates the existing config entry on success, and aborts if a different character is returned.

That matches the existing HA OAuth pattern and fixes the current failure mode directly, since the integration was already raising ConfigEntryAuthFailed and core was already trying to start reauth. I also added focused flow tests for both successful reauth and character mismatch.

Copilot

Pull request overview

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

ronaldvdmeer · 2026-05-02T20:14:51Z

@erwindouna See my 2 comments and the commit. Triggered by a dev branch merge.

#168143 (comment) and #168143 (comment) and as an result: 50e32ce

ronaldvdmeer added 2 commits April 13, 2026 23:50

Copilot AI review requested due to automatic review settings April 13, 2026 23:08

home-assistant Bot added cla-signed has-tests integration: eveonline new-integration quality-scale labels Apr 13, 2026

This was referenced Apr 13, 2026

Add Eve Online brand home-assistant/brands#10133

Open

Add Eve Online integration documentation home-assistant/home-assistant.io#44762

Open

Copilot started reviewing on behalf of ronaldvdmeer April 13, 2026 23:09 View session

Copilot AI reviewed Apr 13, 2026

View reviewed changes

Comment thread tests/components/eveonline/test_sensor.py

Comment thread homeassistant/components/eveonline/quality_scale.yaml

Comment thread homeassistant/components/eveonline/quality_scale.yaml Outdated

ronaldvdmeer added 2 commits April 14, 2026 01:21

Fix ship sensor to show ship type name instead of player-assigned name

18a2f87

- Combine solar system and ship type resolve_names calls into one API call for efficiency - Show ship type name (e.g. 'Flycatcher') instead of the player-assigned ship name (e.g. 'i.probe')

Copilot AI review requested due to automatic review settings April 13, 2026 23:34

Copilot started reviewing on behalf of ronaldvdmeer April 13, 2026 23:34 View session

Copilot AI reviewed Apr 13, 2026

View reviewed changes

Comment thread homeassistant/components/eveonline/config_flow.py

Comment thread homeassistant/components/eveonline/quality_scale.yaml Outdated

ronaldvdmeer added 2 commits April 14, 2026 01:42

Mark config-entry-unloading as done in quality scale

e057b84

async_unload_entry is implemented and tested in test_init.py

Copilot AI review requested due to automatic review settings April 14, 2026 05:55

Copilot started reviewing on behalf of ronaldvdmeer April 14, 2026 05:56 View session

Copilot AI reviewed Apr 14, 2026

View reviewed changes

Comment thread tests/components/eveonline/test_sensor.py Outdated

Comment thread tests/components/eveonline/test_sensor.py Outdated

ronaldvdmeer marked this pull request as ready for review April 14, 2026 06:47

Copilot AI review requested due to automatic review settings April 14, 2026 06:47

Copilot started reviewing on behalf of ronaldvdmeer April 14, 2026 06:48 View session

Copilot AI reviewed Apr 14, 2026

View reviewed changes

Comment thread homeassistant/components/eveonline/__init__.py Outdated

erwindouna requested changes Apr 14, 2026

View reviewed changes

ronaldvdmeer added 2 commits April 14, 2026 10:40

Address erwindouna review: parallel location+ship fetch, Implementati…

fdb811a

…onUnavailableError, recovery test

Remove test_api.py and move auth failure test to test_init.py

58e326a

erwindouna requested changes Apr 17, 2026

View reviewed changes

Comment thread tests/components/eveonline/test_config_flow.py Outdated

Comment thread tests/components/eveonline/test_sensor.py Outdated

Comment thread tests/components/eveonline/test_sensor.py Outdated

home-assistant Bot marked this pull request as draft April 17, 2026 20:06

Use result variable consistently in multi-flow config flow tests

6271516

Per review by erwindouna: reuse result rather than introducing result2/state2 in tests that run two flows sequentially.

Copilot AI review requested due to automatic review settings April 17, 2026 20:17

Use STATE_UNAVAILABLE constant in sensor test

fb26507

Per review by erwindouna: use the STATE_UNAVAILABLE constant instead of the hardcoded string literal.

Copilot started reviewing on behalf of ronaldvdmeer April 17, 2026 20:18 View session

Use STATE_UNKNOWN constant in sensor tests

0209c66

Per review by erwindouna: use the STATE_UNKNOWN constant instead of the hardcoded string literal.

ronaldvdmeer marked this pull request as ready for review April 17, 2026 20:18

home-assistant Bot requested a review from erwindouna April 17, 2026 20:19

Copilot AI reviewed Apr 17, 2026

View reviewed changes

Add model and configuration_url to DeviceInfo

c5bea38

- model: 'Character' — consistent label for all character devices - configuration_url: evewho.com profile link for the character

Merge branch 'dev' into feat/eveonline-core-sensors

8bc0bb0

Copilot AI review requested due to automatic review settings April 19, 2026 22:01

Copilot started reviewing on behalf of ronaldvdmeer April 19, 2026 22:01 View session

Copilot AI reviewed Apr 19, 2026

View reviewed changes

Comment thread homeassistant/components/eveonline/quality_scale.yaml Outdated

ronaldvdmeer added 2 commits April 20, 2026 07:47

Fix quality_scale: mark test-before-configure as exempt

d99bbbb

OAuth2-only integration; the EVE SSO token exchange is the only pre-configure step possible. No traditional connectivity test can be performed before the OAuth round-trip completes.

Merge branch 'dev' into feat/eveonline-core-sensors

bc00048

Copilot AI review requested due to automatic review settings April 26, 2026 00:23

Copilot started reviewing on behalf of ronaldvdmeer April 26, 2026 00:23 View session

Copilot AI reviewed Apr 26, 2026

View reviewed changes

Merge branch 'dev' into feat/eveonline-core-sensors

e5226ac

Add Eve Online reauth flow

50e32ce

Copilot AI review requested due to automatic review settings May 2, 2026 20:02

Copilot started reviewing on behalf of ronaldvdmeer May 2, 2026 20:03 View session

Copilot AI reviewed May 2, 2026

View reviewed changes

Comment thread homeassistant/components/eveonline/sensor.py

Uh oh!

Conversation

ronaldvdmeer commented Apr 13, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Proposed change

Type of change

Additional information

Checklist

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

erwindouna left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

erwindouna left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

ronaldvdmeer commented Apr 19, 2026

Uh oh!

ronaldvdmeer commented Apr 19, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

ronaldvdmeer commented May 2, 2026

Uh oh!

ronaldvdmeer commented May 2, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

ronaldvdmeer commented May 2, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

ronaldvdmeer commented Apr 13, 2026 •

edited

Loading

ronaldvdmeer commented May 2, 2026 •

edited

Loading