Skip to content

Fix React hook memoization lint blockers#86

Open
bmendonca3 wants to merge 650 commits intogrp06:mainfrom
bmendonca3:bm/lint-react-compiler-deps
Open

Fix React hook memoization lint blockers#86
bmendonca3 wants to merge 650 commits intogrp06:mainfrom
bmendonca3:bm/lint-react-compiler-deps

Conversation

@bmendonca3
Copy link

Summary

  • fix react-hooks/preserve-manual-memoization lint blockers in controller hooks by destructuring stable inputs from params
  • align hook dependency arrays to the destructured values in:
    • useGatewayConfigSyncController
    • useRuntimeSyncController
    • useSettingsRouteController
  • remove remaining lint warnings from unused destructured variables (AgentInspectPanels, livePatchQueue) and the delete-restart polling effect in useAgentSettingsMutationController

Testing

  • npm run lint
  • npm run typecheck
  • npm test -- --run
  • npm run e2e (known disconnected-flow failures on main; tracked separately)

Related

grp06 and others added 30 commits February 10, 2026 20:31
This reverts commit e7b6589, reversing
changes made to 767dff8.
When the gateway is busy processing cron jobs (~60% of the time), the
initial WebSocket handshake times out and the Studio shows "No agents
available" with no recovery. Add exponential backoff retry (2s → 30s,
up to 20 attempts) that automatically reconnects when the gateway
becomes available. Auth errors are not retried. Manual disconnect
suppresses auto-retry.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
(cherry picked from commit 4eb381e)
grp06 and others added 28 commits February 24, 2026 11:57
…. Update HeaderBar and GatewayConnectScreen for better responsiveness. Adjust theme toggle button size and icon dimensions. Enhance ui-topbar styling with new color mixes and shadow properties.
- Implemented `resolveAssistantTerminalLine` to generate appropriate terminal messages for aborted or errored assistant runs.
- Updated `buildHistoryLines` to include terminal messages when assistant content is empty.
- Added unit tests to verify behavior for handling aborted messages and clearing stale running states in history sync operations.
- Adjusted styles for the skill eligibility indicator to enhance visibility and consistency.
- Changed font size and added border styles for eligible and blocked states.
- Implemented conditional styling for blocked skills to improve user experience.
sioakim pushed a commit to sioakim/openclaw-studio that referenced this pull request Mar 2, 2026
- Wrap root page in Suspense boundary to fix useSearchParams() build error
- Fix React hooks memoization lint blockers (destructure stable params)
- Stabilize ThemeToggle localStorage unit tests with full Storage mock

Cherry-picked from upstream PRs grp06#88, grp06#86, grp06#20
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.

3 participants