Skip to content

refactor(settings): remove custom agents UI from LocalAgents#1956

Merged
piorpua merged 1 commit intomainfrom
zynx/refactor/hide-custom-agents-ui
Mar 30, 2026
Merged

refactor(settings): remove custom agents UI from LocalAgents#1956
piorpua merged 1 commit intomainfrom
zynx/refactor/hide-custom-agents-ui

Conversation

@piorpua
Copy link
Copy Markdown
Contributor

@piorpua piorpua commented Mar 30, 2026

Summary

  • Remove the Custom Agents section (header, add button, agent list, InlineAgentEditor) from the Agents settings page — the section was re-introduced in fix(settings): preserve preset agents when saving custom agents #1908
  • Delete all related state and handler logic (customAgents, showEditor, editingAgent, handleSaveAgent, handleDeleteAgent, handleToggleAgent, loadCustomAgents, refreshAgentDetection) from LocalAgents.tsx
  • Remove corresponding tests that covered the now-deleted UI

Test plan

  • Open the Agents settings page and verify the Custom Agents section no longer appears
  • Verify detected agents (Gemini CLI, Claude Code, etc.) still display correctly
  • bunx vitest run tests/unit/LocalAgents.dom.test.tsx passes (3 tests)
  • bunx tsc --noEmit reports no errors

Hide the Custom Agents section (header, add button, list, InlineAgentEditor)
and all related state/handlers that were introduced in #1908.
Update the LocalAgents test suite to remove tests covering the removed UI.
@piorpua piorpua enabled auto-merge (squash) March 30, 2026 15:07
@piorpua piorpua merged commit 34ad258 into main Mar 30, 2026
15 of 17 checks passed
@piorpua piorpua deleted the zynx/refactor/hide-custom-agents-ui branch March 30, 2026 15:14
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