Compare commits
10 Commits
| Author | SHA1 | Date | |
|---|---|---|---|
| 7c1a91937a | |||
| 7ac3289bb7 | |||
| 19bdbd1b9a | |||
| 24363cd2ac | |||
| e19ca5bff1 | |||
| 05897d4370 | |||
| 7a3a28d87f | |||
| 240cded8a9 | |||
| 31e241f052 | |||
| de579682a0 |
30
CHANGELOG.md
30
CHANGELOG.md
@@ -11,6 +11,36 @@ Sections:
|
|||||||
|
|
||||||
---
|
---
|
||||||
|
|
||||||
|
## [0.6.5] - 2026-01-18
|
||||||
|
|
||||||
|
### Added
|
||||||
|
- Separate handling of HACS official repositories with an enable/disable toggle in the Store UI.
|
||||||
|
- HACS repositories are now loaded independently from the main store index.
|
||||||
|
|
||||||
|
### Changed
|
||||||
|
- Store index can remain minimal and curated; HACS repositories are no longer required in store.yaml.
|
||||||
|
- Improved Store performance and clarity by clearly separating repository sources.
|
||||||
|
|
||||||
|
### Fixed
|
||||||
|
- Browser cache issues resolved by proper panel cache-busting for UI updates.
|
||||||
|
|
||||||
|
### Internal
|
||||||
|
- No changes to install, update, backup, or restore logic.
|
||||||
|
- Fully backward compatible with existing installations and configurations.
|
||||||
|
|
||||||
|
## [0.6.4] - 2026-01-18
|
||||||
|
|
||||||
|
### Fixed
|
||||||
|
- Fixed long Home Assistant startup times caused by background repository enrichment running too early.
|
||||||
|
|
||||||
|
### Changed
|
||||||
|
- Background repository enrichment is now started only after Home Assistant has fully started.
|
||||||
|
- Repository cache updates now run fully asynchronous without blocking Home Assistant startup.
|
||||||
|
|
||||||
|
### Internal
|
||||||
|
- Improved alignment with Home Assistant startup lifecycle.
|
||||||
|
- No functional changes to store behavior or UI.
|
||||||
|
|
||||||
## [0.6.3] - 2026-01-18
|
## [0.6.3] - 2026-01-18
|
||||||
|
|
||||||
### Changed
|
### Changed
|
||||||
|
|||||||
@@ -5,7 +5,8 @@ from datetime import timedelta
|
|||||||
|
|
||||||
from homeassistant.core import HomeAssistant
|
from homeassistant.core import HomeAssistant
|
||||||
from homeassistant.components.panel_custom import async_register_panel
|
from homeassistant.components.panel_custom import async_register_panel
|
||||||
from homeassistant.helpers.event import async_track_time_interval
|
from homeassistant.helpers.event import async_track_time_interval, async_call_later
|
||||||
|
from homeassistant.const import EVENT_HOMEASSISTANT_STARTED
|
||||||
from homeassistant.helpers.discovery import async_load_platform
|
from homeassistant.helpers.discovery import async_load_platform
|
||||||
|
|
||||||
from .core import BCSCore, BCSConfig, BCSError
|
from .core import BCSCore, BCSConfig, BCSError
|
||||||
@@ -34,6 +35,7 @@ async def async_setup(hass: HomeAssistant, config: dict) -> bool:
|
|||||||
from .views import (
|
from .views import (
|
||||||
StaticAssetsView,
|
StaticAssetsView,
|
||||||
BCSApiView,
|
BCSApiView,
|
||||||
|
BCSSettingsView,
|
||||||
BCSReadmeView,
|
BCSReadmeView,
|
||||||
BCSVersionsView,
|
BCSVersionsView,
|
||||||
BCSRepoDetailView,
|
BCSRepoDetailView,
|
||||||
@@ -48,6 +50,7 @@ async def async_setup(hass: HomeAssistant, config: dict) -> bool:
|
|||||||
|
|
||||||
hass.http.register_view(StaticAssetsView())
|
hass.http.register_view(StaticAssetsView())
|
||||||
hass.http.register_view(BCSApiView(core))
|
hass.http.register_view(BCSApiView(core))
|
||||||
|
hass.http.register_view(BCSSettingsView(core))
|
||||||
hass.http.register_view(BCSReadmeView(core))
|
hass.http.register_view(BCSReadmeView(core))
|
||||||
hass.http.register_view(BCSVersionsView(core))
|
hass.http.register_view(BCSVersionsView(core))
|
||||||
hass.http.register_view(BCSRepoDetailView(core))
|
hass.http.register_view(BCSRepoDetailView(core))
|
||||||
@@ -64,17 +67,24 @@ async def async_setup(hass: HomeAssistant, config: dict) -> bool:
|
|||||||
frontend_url_path="bahmcloud-store",
|
frontend_url_path="bahmcloud-store",
|
||||||
webcomponent_name="bahmcloud-store-panel",
|
webcomponent_name="bahmcloud-store-panel",
|
||||||
# IMPORTANT: bump v to avoid caching old JS
|
# IMPORTANT: bump v to avoid caching old JS
|
||||||
module_url="/api/bahmcloud_store_static/panel.js?v=106",
|
module_url="/api/bahmcloud_store_static/panel.js?v=107",
|
||||||
sidebar_title="Bahmcloud Store",
|
sidebar_title="Bahmcloud Store",
|
||||||
sidebar_icon="mdi:store",
|
sidebar_icon="mdi:store",
|
||||||
require_admin=True,
|
require_admin=True,
|
||||||
config={},
|
config={},
|
||||||
)
|
)
|
||||||
|
|
||||||
try:
|
async def _do_startup_refresh(_now=None) -> None:
|
||||||
await core.full_refresh(source="startup")
|
try:
|
||||||
except BCSError as e:
|
await core.full_refresh(source="startup")
|
||||||
_LOGGER.error("Initial refresh failed: %s", e)
|
except BCSError as e:
|
||||||
|
_LOGGER.error("Initial refresh failed: %s", e)
|
||||||
|
|
||||||
|
# Do not block Home Assistant startup. Schedule the initial refresh after HA started.
|
||||||
|
def _on_ha_started(_event) -> None:
|
||||||
|
async_call_later(hass, 30, _do_startup_refresh)
|
||||||
|
|
||||||
|
hass.bus.async_listen_once(EVENT_HOMEASSISTANT_STARTED, _on_ha_started)
|
||||||
|
|
||||||
async def periodic(_now) -> None:
|
async def periodic(_now) -> None:
|
||||||
try:
|
try:
|
||||||
|
|||||||
@@ -32,6 +32,10 @@ RESTART_REQUIRED_ISSUE_ID = "restart_required"
|
|||||||
|
|
||||||
BACKUP_META_FILENAME = ".bcs_backup_meta.json"
|
BACKUP_META_FILENAME = ".bcs_backup_meta.json"
|
||||||
|
|
||||||
|
# Optional HACS integrations index (GitHub repositories only).
|
||||||
|
HACS_INTEGRATIONS_URL = "https://data-v2.hacs.xyz/integration/repositories.json"
|
||||||
|
HACS_DEFAULT_CATEGORY = "Integrations"
|
||||||
|
|
||||||
|
|
||||||
class BCSError(Exception):
|
class BCSError(Exception):
|
||||||
"""BCS core error."""
|
"""BCS core error."""
|
||||||
@@ -95,6 +99,9 @@ class BCSCore:
|
|||||||
self._install_lock = asyncio.Lock()
|
self._install_lock = asyncio.Lock()
|
||||||
self._installed_cache: dict[str, Any] = {}
|
self._installed_cache: dict[str, Any] = {}
|
||||||
|
|
||||||
|
# Persistent settings (UI toggles etc.)
|
||||||
|
self.settings: dict[str, Any] = {"hacs_enabled": False}
|
||||||
|
|
||||||
# Phase F2: backups before install/update
|
# Phase F2: backups before install/update
|
||||||
self._backup_root = Path(self.hass.config.path(".bcs_backups"))
|
self._backup_root = Path(self.hass.config.path(".bcs_backups"))
|
||||||
self._backup_keep_per_domain: int = 5
|
self._backup_keep_per_domain: int = 5
|
||||||
@@ -104,6 +111,14 @@ class BCSCore:
|
|||||||
self.version = await self._read_manifest_version_async()
|
self.version = await self._read_manifest_version_async()
|
||||||
await self._refresh_installed_cache()
|
await self._refresh_installed_cache()
|
||||||
|
|
||||||
|
# Load persistent settings (do not fail startup)
|
||||||
|
try:
|
||||||
|
s = await self.storage.get_settings()
|
||||||
|
if isinstance(s, dict):
|
||||||
|
self.settings.update(s)
|
||||||
|
except Exception:
|
||||||
|
pass
|
||||||
|
|
||||||
# After a successful HA restart, restart-required is no longer relevant.
|
# After a successful HA restart, restart-required is no longer relevant.
|
||||||
self._clear_restart_required_issue()
|
self._clear_restart_required_issue()
|
||||||
|
|
||||||
@@ -175,19 +190,48 @@ class BCSCore:
|
|||||||
data = (self._installed_cache or {}).get(repo_id)
|
data = (self._installed_cache or {}).get(repo_id)
|
||||||
return data if isinstance(data, dict) else None
|
return data if isinstance(data, dict) else None
|
||||||
|
|
||||||
|
def get_settings_public(self) -> dict[str, Any]:
|
||||||
|
"""Return UI-relevant settings (no I/O)."""
|
||||||
|
return {
|
||||||
|
"hacs_enabled": bool(self.settings.get("hacs_enabled", False)),
|
||||||
|
}
|
||||||
|
|
||||||
|
async def set_settings(self, updates: dict[str, Any]) -> dict[str, Any]:
|
||||||
|
"""Persist settings and apply them."""
|
||||||
|
safe_updates: dict[str, Any] = {}
|
||||||
|
if "hacs_enabled" in (updates or {}):
|
||||||
|
safe_updates["hacs_enabled"] = bool(updates.get("hacs_enabled"))
|
||||||
|
|
||||||
|
merged = await self.storage.set_settings(safe_updates)
|
||||||
|
if isinstance(merged, dict):
|
||||||
|
self.settings.update(merged)
|
||||||
|
|
||||||
|
# Reload repo list after changing settings.
|
||||||
|
await self.full_refresh(source="settings")
|
||||||
|
return self.get_settings_public()
|
||||||
|
|
||||||
async def refresh(self) -> None:
|
async def refresh(self) -> None:
|
||||||
index_repos, refresh_seconds = await self._load_index_repos()
|
index_repos, refresh_seconds = await self._load_index_repos()
|
||||||
self.refresh_seconds = refresh_seconds
|
self.refresh_seconds = refresh_seconds
|
||||||
|
|
||||||
|
hacs_enabled = bool(self.settings.get("hacs_enabled", False))
|
||||||
|
hacs_repos: list[RepoItem] = []
|
||||||
|
if hacs_enabled:
|
||||||
|
try:
|
||||||
|
hacs_repos = await self._load_hacs_repos()
|
||||||
|
except Exception as e:
|
||||||
|
_LOGGER.warning("BCS HACS index load failed: %s", e)
|
||||||
|
|
||||||
custom_repos = await self.storage.list_custom_repos()
|
custom_repos = await self.storage.list_custom_repos()
|
||||||
|
|
||||||
# Fast path: if index + custom repos did not change, skip expensive work.
|
# Fast path: if index + custom repos did not change, skip expensive work.
|
||||||
try:
|
try:
|
||||||
custom_sig = [(c.id, (c.url or '').strip(), (c.name or '').strip()) for c in (custom_repos or [])]
|
custom_sig = [(c.id, (c.url or '').strip(), (c.name or '').strip()) for c in (custom_repos or [])]
|
||||||
custom_sig.sort()
|
custom_sig.sort()
|
||||||
refresh_signature = json.dumps({"index_hash": self.last_index_hash, "custom": custom_sig}, sort_keys=True)
|
hacs_sig = len(hacs_repos) if hacs_enabled else 0
|
||||||
|
refresh_signature = json.dumps({"index_hash": self.last_index_hash, "custom": custom_sig, "hacs": hacs_sig, "hacs_enabled": hacs_enabled}, sort_keys=True)
|
||||||
except Exception:
|
except Exception:
|
||||||
refresh_signature = f"{self.last_index_hash}:{len(custom_repos or [])}"
|
refresh_signature = f"{self.last_index_hash}:{len(custom_repos or [])}:{'h' if hacs_enabled else 'n'}:{len(hacs_repos)}"
|
||||||
|
|
||||||
if self._last_refresh_signature and refresh_signature == self._last_refresh_signature and self.repos:
|
if self._last_refresh_signature and refresh_signature == self._last_refresh_signature and self.repos:
|
||||||
_LOGGER.debug("BCS refresh skipped (no changes detected)")
|
_LOGGER.debug("BCS refresh skipped (no changes detected)")
|
||||||
@@ -199,6 +243,9 @@ class BCSCore:
|
|||||||
for item in index_repos:
|
for item in index_repos:
|
||||||
merged[item.id] = item
|
merged[item.id] = item
|
||||||
|
|
||||||
|
for item in hacs_repos:
|
||||||
|
merged[item.id] = item
|
||||||
|
|
||||||
for c in custom_repos:
|
for c in custom_repos:
|
||||||
merged[c.id] = RepoItem(
|
merged[c.id] = RepoItem(
|
||||||
id=c.id,
|
id=c.id,
|
||||||
@@ -216,12 +263,52 @@ class BCSCore:
|
|||||||
self._last_refresh_signature = refresh_signature
|
self._last_refresh_signature = refresh_signature
|
||||||
|
|
||||||
_LOGGER.info(
|
_LOGGER.info(
|
||||||
"BCS refresh complete: repos=%s (index=%s, custom=%s)",
|
"BCS refresh complete: repos=%s (index=%s, hacs=%s, custom=%s)",
|
||||||
len(self.repos),
|
len(self.repos),
|
||||||
len([r for r in self.repos.values() if r.source == "index"]),
|
len([r for r in self.repos.values() if r.source == "index"]),
|
||||||
|
len([r for r in self.repos.values() if r.source == "hacs"]),
|
||||||
len([r for r in self.repos.values() if r.source == "custom"]),
|
len([r for r in self.repos.values() if r.source == "custom"]),
|
||||||
)
|
)
|
||||||
|
|
||||||
|
async def _load_hacs_repos(self) -> list[RepoItem]:
|
||||||
|
"""Load the official HACS integrations repository list.
|
||||||
|
|
||||||
|
This is used as an optional additional source to keep the local store index small.
|
||||||
|
We only parse owner/repo strings and map them to GitHub URLs.
|
||||||
|
"""
|
||||||
|
session = async_get_clientsession(self.hass)
|
||||||
|
headers = {
|
||||||
|
"User-Agent": "BahmcloudStore (Home Assistant)",
|
||||||
|
"Cache-Control": "no-cache, no-store, max-age=0",
|
||||||
|
"Pragma": "no-cache",
|
||||||
|
}
|
||||||
|
async with session.get(HACS_INTEGRATIONS_URL, timeout=60, headers=headers) as resp:
|
||||||
|
if resp.status != 200:
|
||||||
|
raise BCSError(f"HACS index returned {resp.status}")
|
||||||
|
data = await resp.json()
|
||||||
|
|
||||||
|
if not isinstance(data, list):
|
||||||
|
raise BCSError("HACS repositories.json must be a list")
|
||||||
|
|
||||||
|
items: list[RepoItem] = []
|
||||||
|
for entry in data:
|
||||||
|
if not isinstance(entry, str):
|
||||||
|
continue
|
||||||
|
full_name = entry.strip().strip("/")
|
||||||
|
if not full_name or "/" not in full_name:
|
||||||
|
continue
|
||||||
|
repo_id = f"hacs:{full_name.lower()}"
|
||||||
|
items.append(
|
||||||
|
RepoItem(
|
||||||
|
id=repo_id,
|
||||||
|
name=full_name,
|
||||||
|
url=f"https://github.com/{full_name}",
|
||||||
|
source="hacs",
|
||||||
|
meta_category=HACS_DEFAULT_CATEGORY,
|
||||||
|
)
|
||||||
|
)
|
||||||
|
return items
|
||||||
|
|
||||||
async def _enrich_and_resolve(self, merged: dict[str, RepoItem]) -> None:
|
async def _enrich_and_resolve(self, merged: dict[str, RepoItem]) -> None:
|
||||||
sem = asyncio.Semaphore(6)
|
sem = asyncio.Semaphore(6)
|
||||||
|
|
||||||
|
|||||||
@@ -1,7 +1,7 @@
|
|||||||
{
|
{
|
||||||
"domain": "bahmcloud_store",
|
"domain": "bahmcloud_store",
|
||||||
"name": "Bahmcloud Store",
|
"name": "Bahmcloud Store",
|
||||||
"version": "0.6.3",
|
"version": "0.6.5",
|
||||||
"documentation": "https://git.bahmcloud.de/bahmcloud/bahmcloud_store",
|
"documentation": "https://git.bahmcloud.de/bahmcloud/bahmcloud_store",
|
||||||
"platforms": ["update"],
|
"platforms": ["update"],
|
||||||
"requirements": [],
|
"requirements": [],
|
||||||
|
|||||||
@@ -36,6 +36,7 @@ class BCSStorage:
|
|||||||
Keys:
|
Keys:
|
||||||
- custom_repos: list of manually added repositories
|
- custom_repos: list of manually added repositories
|
||||||
- installed_repos: mapping repo_id -> installed metadata
|
- installed_repos: mapping repo_id -> installed metadata
|
||||||
|
- settings: persistent user settings (e.g. toggles in the UI)
|
||||||
"""
|
"""
|
||||||
|
|
||||||
def __init__(self, hass: HomeAssistant) -> None:
|
def __init__(self, hass: HomeAssistant) -> None:
|
||||||
@@ -53,8 +54,32 @@ class BCSStorage:
|
|||||||
if "installed_repos" not in data or not isinstance(data.get("installed_repos"), dict):
|
if "installed_repos" not in data or not isinstance(data.get("installed_repos"), dict):
|
||||||
data["installed_repos"] = {}
|
data["installed_repos"] = {}
|
||||||
|
|
||||||
|
if "settings" not in data or not isinstance(data.get("settings"), dict):
|
||||||
|
data["settings"] = {}
|
||||||
|
|
||||||
return data
|
return data
|
||||||
|
|
||||||
|
async def get_settings(self) -> dict[str, Any]:
|
||||||
|
"""Return persistent settings.
|
||||||
|
|
||||||
|
Currently used for UI/behavior toggles.
|
||||||
|
"""
|
||||||
|
data = await self._load()
|
||||||
|
settings = data.get("settings", {})
|
||||||
|
return settings if isinstance(settings, dict) else {}
|
||||||
|
|
||||||
|
async def set_settings(self, updates: dict[str, Any]) -> dict[str, Any]:
|
||||||
|
"""Update persistent settings and return the merged settings."""
|
||||||
|
data = await self._load()
|
||||||
|
settings = data.get("settings", {})
|
||||||
|
if not isinstance(settings, dict):
|
||||||
|
settings = {}
|
||||||
|
for k, v in (updates or {}).items():
|
||||||
|
settings[str(k)] = v
|
||||||
|
data["settings"] = settings
|
||||||
|
await self._save(data)
|
||||||
|
return settings
|
||||||
|
|
||||||
async def _save(self, data: dict[str, Any]) -> None:
|
async def _save(self, data: dict[str, Any]) -> None:
|
||||||
await self._store.async_save(data)
|
await self._store.async_save(data)
|
||||||
|
|
||||||
|
|||||||
@@ -215,7 +215,12 @@ class BCSApiView(HomeAssistantView):
|
|||||||
|
|
||||||
async def get(self, request: web.Request) -> web.Response:
|
async def get(self, request: web.Request) -> web.Response:
|
||||||
return web.json_response(
|
return web.json_response(
|
||||||
{"ok": True, "version": self.core.version, "repos": self.core.list_repos_public()}
|
{
|
||||||
|
"ok": True,
|
||||||
|
"version": self.core.version,
|
||||||
|
"settings": self.core.get_settings_public(),
|
||||||
|
"repos": self.core.list_repos_public(),
|
||||||
|
}
|
||||||
)
|
)
|
||||||
|
|
||||||
async def post(self, request: web.Request) -> web.Response:
|
async def post(self, request: web.Request) -> web.Response:
|
||||||
@@ -248,6 +253,37 @@ class BCSApiView(HomeAssistantView):
|
|||||||
return web.json_response({"ok": False, "message": "Unknown operation"}, status=400)
|
return web.json_response({"ok": False, "message": "Unknown operation"}, status=400)
|
||||||
|
|
||||||
|
|
||||||
|
class BCSSettingsView(HomeAssistantView):
|
||||||
|
"""Persistent UI settings (e.g. toggles)."""
|
||||||
|
|
||||||
|
url = "/api/bcs/settings"
|
||||||
|
name = "api:bcs_settings"
|
||||||
|
requires_auth = True
|
||||||
|
|
||||||
|
def __init__(self, core: Any) -> None:
|
||||||
|
self.core: BCSCore = core
|
||||||
|
|
||||||
|
async def get(self, request: web.Request) -> web.Response:
|
||||||
|
return web.json_response({"ok": True, "settings": self.core.get_settings_public()})
|
||||||
|
|
||||||
|
async def post(self, request: web.Request) -> web.Response:
|
||||||
|
try:
|
||||||
|
data = await request.json()
|
||||||
|
except Exception:
|
||||||
|
data = {}
|
||||||
|
|
||||||
|
updates: dict[str, Any] = {}
|
||||||
|
if "hacs_enabled" in data:
|
||||||
|
updates["hacs_enabled"] = bool(data.get("hacs_enabled"))
|
||||||
|
|
||||||
|
try:
|
||||||
|
settings = await self.core.set_settings(updates)
|
||||||
|
return web.json_response({"ok": True, "settings": settings})
|
||||||
|
except Exception as e:
|
||||||
|
_LOGGER.exception("BCS set settings failed: %s", e)
|
||||||
|
return web.json_response({"ok": False, "message": str(e) or "Failed"}, status=500)
|
||||||
|
|
||||||
|
|
||||||
class BCSCustomRepoView(HomeAssistantView):
|
class BCSCustomRepoView(HomeAssistantView):
|
||||||
url = "/api/bcs/custom_repo"
|
url = "/api/bcs/custom_repo"
|
||||||
name = "api:bcs_custom_repo"
|
name = "api:bcs_custom_repo"
|
||||||
|
|||||||
Reference in New Issue
Block a user