Skip to content

Speed up uninstall and restore live removal details - #1397

Merged
sridharkalaibala merged 3 commits into
developfrom
fix/fast-uninstall-develop
Oct 16, 2026
Merged

sridharkalaibala merged 3 commits into
developfrom
fix/fast-uninstall-develop

Conversation

@sridharkalaibala

Copy link
Copy Markdown
Contributor

Uninstall previously paused with no live details while copying and hashing the full local profile, including node_modules. Apply the three Keep choices directly, state that unchecked local data is permanently removed without an extra backup, restore visible streaming removal details, and run file traversal in compiled code with throttled status updates.

Validation: six focused Windows tests pass, including all eight Keep combinations, long/read-only paths, link rejection, running-app protection, and registration cleanup. GSHCL installer builds and package checks pass. A 6,000-file synthetic cleanup improved from 3992 ms to 1804 ms. User tested the optimized installer and confirmed faster uninstall. Attribution check passed.

Scope: local uninstall only. Cloud data and sync lanes are unchanged. Existing saved backups follow their independent Keep choice. No automatic recovery archive is created. Installer/new-shop reset behavior is unchanged.

@sridharkalaibala
sridharkalaibala merged commit bc834a5 into develop Oct 16, 2026
11 checks passed
@github-actions github-actions Bot added the ready for QA Merged to develop and live on develop.posnic.io - anyone can test it label Oct 16, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Merged to develop. Anyone can test this - you do not need write access.

Try it at https://develop.posnic.io, or run it yourself:

git fetch origin develop && git checkout develop
npm install && npm --prefix api install
npm run dev   # then http://localhost:3000

When you have tested it, say what you did and what happened, and set
QA passed or QA failed. If you cannot set labels, just comment -
a maintainer will.

Reporting that something is broken is as useful as fixing it. It is
better found here than by a shopkeeper.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready for QA Merged to develop and live on develop.posnic.io - anyone can test it

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant