Refactor agent form UX: add section rail layout and Memory placeholder#228
Draft
esafwan wants to merge 1 commit into
Draft
Refactor agent form UX: add section rail layout and Memory placeholder#228esafwan wants to merge 1 commit into
esafwan wants to merge 1 commit into
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Motivation
Description
SectionRailLayoutandSectionBlockto render a left-side section rail with status dots and anchor navigation (frontend/src/components/agent/form-base/SectionRailLayout.tsx).GeneralTabto the new section-rail layout and split it intoIdentity,Model, andInstructionssections while keeping provider/model filtering, sliders and the instructions textarea (frontend/src/components/agent/GeneralTab.tsx).BehaviorTabto the section-rail pattern and split it intoChat Access,History, andExecutionsections while keeping the existing cross-field logic and toasts (frontend/src/components/agent/BehaviorTab.tsx).Memorytab placeholder marked “Coming Soon” so the structure is visible but inputs are disabled until the feature is merged (frontend/src/components/agent/MemoryTab.tsx) and wired the tab intoAgentFormPage(updated tab config and tab bar columns) (frontend/src/pages/AgentFormPage.tsx).Testing
npx eslint src/components/agent/GeneralTab.tsx src/components/agent/BehaviorTab.tsx src/components/agent/MemoryTab.tsx src/components/agent/form-base/SectionRailLayout.tsxand the checked files passed without errors.npm run typecheckand it failed due to a pre-existing, unrelated TypeScript type mismatch insrc/components/agent/TriggerModal.tsx; the type error is not introduced by these changes and requires a separate fix.Codex Task