Remove Integrations Procedure | Home Assistant | Page 1

cloud venture Mar 31, 2026, 4:19 PM

#

Wait we removed bmw completely?

vital wolf Mar 31, 2026, 4:20 PM

#

I think so?

cloud venture Mar 31, 2026, 4:24 PM

#

That should be similar to myq or Mazda

vital wolf Mar 31, 2026, 4:38 PM

#

So we can never fully remove an old integration?

#

Myq was removed 2 years ago and still has the skeleton in our codebase.

#

I think that's a bit odd IMO

cloud venture Mar 31, 2026, 4:39 PM

#

okay so my question here was more like, let's make sure BMW uses the skeleton

#

But euh, before that we just dropped it

#

YAML integrations we just drop, unless there's probably something we want the user to know I guess

#

for config flow ones we just agreed on keeping it around and having that skeleton

#

but we haven't really spoke about removing them altogether

#

I think we can after some time

vital wolf Mar 31, 2026, 4:42 PM

#

I think we should discuss that and write that down somewhere.

#

Should I create a PR for BMW then? To add the repair issue?

#

Tomorrow is the release, so today or never

verbal oriole Mar 31, 2026, 5:02 PM

#

The benefit of the skeleton + repair issue is that it allows us to customize the message for the user and explain what happened. This is nice. We have a default repair issue for config entries with missing integration, but that doesn't explain why the integration is gone.

#

I suggest we add a docs page for how to make the skeleton and repair issue for config entry based integrations. And how we treat YAML integrations when removing those.

vital wolf Mar 31, 2026, 5:06 PM

#

I added it to next weeks Tuesday agenda so we can discuss the details there (like after how many months/years a skeleton is fine to remove). I agree, a separate docs page would be nice.

#

Pr incoming

hallow obsidian Mar 31, 2026, 5:09 PM

#

cloud venture okay so my question here was more like, let's make sure BMW uses the skeleton

Regarding BMW I can create a skeleton integration tonight and ping you.

#

If not already done, still on the train

vital wolf Mar 31, 2026, 5:31 PM

#

Ah, sorry. I worked on this already: https://github.com/home-assistant/core/pull/166983

Feel free to comment on it though

#

It follows pretty closely to what myq did

verbal oriole Mar 31, 2026, 6:01 PM

#

Please add the same tests as for myq. Otherwise good to go.

hallow obsidian Mar 31, 2026, 6:07 PM

#

Thanks and no worries about being too fast 🤣

vital wolf Mar 31, 2026, 6:17 PM

#

Done

vital wolf Apr 1, 2026, 7:31 AM

#

cloud venture YAML integrations we just drop, unless there's probably something we want the us...

So for the PR I mentioned #166838, it's ok to just remove it completly since it is a yaml only integration. Did I get that right?

verbal oriole Apr 1, 2026, 7:44 AM

#

If we want to be consistent, I guess we should have a repair issue also for YAML integrations that explain the reason why the integration is gone. What do others think?

cloud venture Apr 1, 2026, 8:27 AM

#

I guess that's fine

#Remove Integrations Procedure