Skip to content

fix: scope uninstall guidance cleanup - #313

Merged
jlitola merged 1 commit into
mainfrom
jlitola/fix-uninstall-selected-tools
Aug 26, 2026
Merged

fix: scope uninstall guidance cleanup#313
jlitola merged 1 commit into
mainfrom
jlitola/fix-uninstall-selected-tools

Conversation

@jlitola

@jlitola jlitola commented Aug 26, 2026

Copy link
Copy Markdown
Contributor

Summary

  • scope interactive user guidance cleanup to the selected tools
  • preserve shared guidance used by any detected retained tool
  • preserve a selected tool's guidance when its MCP removal fails
  • keep --yes, project uninstall, and guidance-only cleanup scope-wide

Validation

  • bun run plugins:generate
  • bun run plugins:check
  • bun test — 3283 passed
  • bun run typecheck
  • bun run lint
  • bun run format
  • bun run build
  • bun run smoke:cli — 18 deterministic unauthenticated steps passed
  • bun run smoke:mcp — 7 registration steps passed
  • targeted uninstall regressions after review fixes — 3 passed
  • internal review and three-round Opus review loop — clean

Authenticated live CLI/MCP smoke reached the backend but the final JSON example request was rate-limited with HTTP 429. No agent-e2e workload currently covers interactive uninstall.

Respect interactive agent selection when removing guidance.\n\nPreserve shared targets used by detected hosts and retain guidance when MCP removal fails.
@jlitola jlitola added bug Something isn't working fix labels Aug 26, 2026
@jlitola
jlitola marked this pull request as ready for review August 26, 2026 20:44
@jlitola
jlitola merged commit cb50f9f into main Aug 26, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working fix

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant