Sovran_Systems/Sovran_SystemsOS
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge pull request #201 from naturallaw777/copilot/fix-update-modal-message

Browse Source 
fix: check_for_updates() tri-state return to unblock updates on inconclusive checks
This commit is contained in:
Sovran_Systems
2026-04-12 07:59:45 -05:00
committed by GitHub
parent bd1acfa266 536b3bfa78
commit 37bc0c6192
1 changed files with 5 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
app/sovran_systemsos_web/server.py
View File

@@ -515,12 +515,12 @@ def _get_remote_rev(branch=None):
return None return None
def check_for_updates() -> bool: def check_for_updates() -> bool | None:
locked_rev, branch = _get_locked_info() locked_rev, branch = _get_locked_info()
remote_rev = _get_remote_rev(branch) remote_rev = _get_remote_rev(branch)
if locked_rev and remote_rev: if locked_rev and remote_rev:
return locked_rev != remote_rev return locked_rev != remote_rev
return False return None # inconclusive — couldn't read lock or reach remote
# ── IP helpers ─────────────────────────────────────────────────── # ── IP helpers ───────────────────────────────────────────────────
@@ -2360,7 +2360,8 @@ async def api_ports_health():
async def api_updates_check(): async def api_updates_check():
loop = asyncio.get_event_loop() loop = asyncio.get_event_loop()
available = await loop.run_in_executor(None, check_for_updates) available = await loop.run_in_executor(None, check_for_updates)
return {"available": available} # None means inconclusive (check failed) — report as available so the UI doesn't block
return {"available": available is not False}
@app.post("/api/reboot") @app.post("/api/reboot")
@@ -2396,7 +2397,7 @@ async def api_updates_run():
pass pass
available = await loop.run_in_executor(None, check_for_updates) available = await loop.run_in_executor(None, check_for_updates)
if not available: if available is False: # only block when positively confirmed no updates
# Clear stale status/log so they don't contaminate future modal opens. # Clear stale status/log so they don't contaminate future modal opens.
_write_update_status("IDLE") _write_update_status("IDLE")
try: try: