Skip to content

fix(rebalance): show cleanup warnings and progress state#132

Merged
cxymds merged 1 commit into
mainfrom
codex/rebalance-cleanup-warnings-ui
Jun 14, 2026
Merged

fix(rebalance): show cleanup warnings and progress state#132
cxymds merged 1 commit into
mainfrom
codex/rebalance-cleanup-warnings-ui

Conversation

@cxymds

@cxymds cxymds commented Jun 14, 2026

Copy link
Copy Markdown
Member

Pull Request

Description

Display backend rebalance cleanup warnings in the rebalance pool table and avoid showing mixed completed/idle pool states as fully completed. This makes failed or partially inactive pools easier to diagnose after a rebalance run.

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update
  • Code refactoring
  • Performance improvement
  • Test improvements
  • Security fix

Testing

  • Unit tests added/updated
  • Manual testing completed
node --test tests/lib/pool-operations.test.js tests/lib/pool-overview-source.test.js
node_modules/.bin/eslint lib/pool-operations.ts 'app/(dashboard)/rebalance/page.tsx' tests/lib/pool-operations.test.js tests/lib/pool-overview-source.test.js
node_modules/.bin/prettier --check lib/pool-operations.ts 'app/(dashboard)/rebalance/page.tsx' tests/lib/pool-operations.test.js tests/lib/pool-overview-source.test.js

pnpm commands could not be executed in the current environment because Corepack rejects the existing devEngines.packageManager.version value ^11.4.0 with Invalid package manager specification in package.json (pnpm@^11.4.0); expected a semver version. Direct full tsc --noEmit is also blocked by existing repository-wide TypeScript configuration/source errors outside this change.

Checklist

  • Code follows the project's style guidelines
  • Self-review completed
  • TypeScript types are properly defined
  • All commit messages are in English (Conventional Commits)
  • All existing tests pass
  • No new dependencies added, or they are justified

Related Issues

Closes N/A

Screenshots (if applicable)

N/A

Additional Notes

The branch intentionally includes only rebalance UI/model/test files. Other pre-existing local workspace changes were not staged.

@cxymds cxymds merged commit c34ef44 into main Jun 14, 2026
6 checks passed
@cxymds cxymds deleted the codex/rebalance-cleanup-warnings-ui branch June 14, 2026 11:18
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.

1 participant