Skip to content

[Fix] Show all pending tools during bulk updates - #694

Merged
Power-Maverick merged 2 commits into
devfrom
copilot/fix-multi-update-blocking
Oct 1, 2026
Merged

Power-Maverick merged 2 commits into
devfrom
copilot/fix-multi-update-blocking

Conversation

Copilot AI commented Sep 30, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Bulk updates showed only the currently updating tool as in progress, leaving other update actions available. Mark all queued tools as updating immediately and disable “Update All” until the batch finishes.

Type of change

  • New feature
  • Bug fix
  • Refactor (no functional change)
  • Documentation
  • Chore / maintenance (dependency update, build, config)
  • Test addition / improvement

Changes

  • toolsSidebarManagement.ts: mark every update target as in progress before starting the batch; clear its state after each tool completes.
  • toolUpdateState.ts: track queued and completed tool IDs.
  • Added unit coverage for bulk update state tracking.

Architecture checklist

Packages (types & validation)

  • Not applicable — no changes to packages/

  • @pptb/types (types): type definitions updated and version bumped in packages/types/package.json

  • @pptb/validate (validation): validation rules updated and version bumped in packages/validation/package.json

Code quality

  • pnpm run typecheck passes with 0 errors (warnings are acceptable)
  • pnpm run lint passes with 0 errors (warnings are acceptable)
  • pnpm run build completes successfully

Testing

  • pnpm run test:unit passes (for changes to src/main/, src/common/, or src/renderer/ utilities)
  • pnpm run test:e2e passes (for UI / navigation / end-to-end flows)
  • Manually tested in the running app (pnpm run dev)

Scenario tested: Unit coverage verifies all queued tools are marked updating and each can be cleared independently.

Screenshots / recordings

Not applicable.

Breaking changes

  • No breaking changes
  • Yes — describe impact and migration path below:

Reviewer notes

  • I have added appropriate unit and/or e2e tests for this change
  • I have resolved all GitHub Copilot review comments
  • I have followed the guidelines in CONTRIBUTING.md

Co-authored-by: Power-Maverick <36135520+Power-Maverick@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix multi update to block all updateable tools in list [Fix] Show all pending tools during bulk updates Sep 30, 2026
@github-actions

github-actions Bot commented Oct 1, 2026 •

Copy link
Copy Markdown

✅ PR Checklist

All required checklist items are complete. This PR is ready for review.

@Power-Maverick
Power-Maverick marked this pull request as ready for review October 1, 2026 17:40
@PowerPlatformToolBox PowerPlatformToolBox deleted a comment from github-actions Bot Oct 1, 2026
@github-actions

github-actions Bot commented Oct 1, 2026

Copy link
Copy Markdown

Bundle Size Report 📦

Bundle Size
Main Process 2.06 MB
Renderer JS 1.15 MB
Renderer CSS 118.26 KB
Total 3.33 MB

Bundle Analysis Reports

The detailed bundle analysis reports are available in the workflow artifacts:

  • 📊 Main Process: stats-main.html
  • 📊 Renderer Process: stats-renderer.html

Download the artifacts from the workflow run to view interactive visualizations.


Bundle size tracking is now active! This helps prevent bundle bloat.

@PowerPlatformToolBox PowerPlatformToolBox deleted a comment from github-actions Bot Oct 1, 2026
@Power-Maverick
Power-Maverick merged commit 8348e53 into dev Oct 1, 2026
12 of 18 checks passed
@Power-Maverick
Power-Maverick deleted the copilot/fix-multi-update-blocking branch October 1, 2026 17:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: Multi Update does not block all updateable tools in list

2 participants