RockteMQ-AI commented on issue #1706: URL: https://github.com/apache/rocketmq-dashboard/issues/1706#issuecomment-5251750866
**Issue Evaluation** Category: `type/bug` (frontend) | Severity: **Low** | Status: **Confirmed** `ConsumerPage` has React hooks lifecycle issues: synchronous state resets in effects trigger cascading renders, and the instance lifecycle state transitions are split across independent effects. When the selected instance changes, prior instance-local UI state (details, selection, loading, group data) should be discarded before loading the next group list, but the current implementation causes intermediate render states. **Root Cause:** State updates for instance lifecycle are performed synchronously inside effects rather than being consolidated, violating React best practices and triggering the exhaustive-deps lint rule. **Impact:** Visual flicker during instance switching, potential stale state display, and lint rule violations that make the code harder to maintain. **Proposed Scope:** Consolidating the instance lifecycle state management and adding focused frontend regression tests. This is well-scoped and improves code quality. This is a valid bug. A fix PR is welcome. --- *Automated evaluation by github-manager* -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
