Skip to content

revert: desktop parity PR #57441 (breaks update/install) - #57638

Closed
teknium1 wants to merge 1 commit into
mainfrom
revert/desktop-parity-57441
Closed

revert: desktop parity PR #57441 (breaks update/install)#57638
teknium1 wants to merge 1 commit into
mainfrom
revert/desktop-parity-57441

Conversation

@teknium1

@teknium1 teknium1 commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

Summary

Reverts #57441 (squash commit c7103c6) — desktop CLI/dashboard parity (skills hub, MCP test/toggle/catalog, maintenance ops, log filters, toolset model catalogs). Teknium reports updating and install no longer work after the merge.

Clean git revert of the squash commit; restores all 15 files to their pre-merge state.

The parity work will be re-landed after the update/install breakage is diagnosed and fixed.

Infographic

revert

…ggle/catalog, maintenance ops, log filters (#57441)"

This reverts commit c7103c6.
@teknium1

teknium1 commented Jul 3, 2026

Copy link
Copy Markdown
Contributor Author

Closing unmerged — wrong premise. The update/install breakage was diagnosed as the Windows venv-lock / half-update race: a running Hermes process (desktop backend / gateway) keeps native .pyd files mapped while the installer or hermes update mutates the venv, producing a half-updated environment (No module named 'annotated_doc', locked _brotlicffi.pyd / _sodium.pyd). PR #57441 is renderer code plus two REST endpoints and does not touch the update path.

A dedicated hardening fix for the update/install coordination is being prepared instead.

@teknium1 teknium1 closed this Jul 3, 2026
@teknium1
teknium1 deleted the revert/desktop-parity-57441 branch July 3, 2026 09:49
@alt-glitch alt-glitch added type/bug Something isn't working comp/desktop Electron desktop app (apps/desktop/*) comp/dashboard Web dashboard / control panel UI (dashboard/, landing) sweeper:risk-compatibility Sweeper risk: may break existing users, config, migrations, defaults, or upgrades P2 Medium — degraded but workaround exists labels Jul 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp/dashboard Web dashboard / control panel UI (dashboard/, landing) comp/desktop Electron desktop app (apps/desktop/*) P2 Medium — degraded but workaround exists sweeper:risk-compatibility Sweeper risk: may break existing users, config, migrations, defaults, or upgrades type/bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants