From f4e454b7278c054d576db2c7cb9ac8a4f81a9577 Mon Sep 17 00:00:00 2001 From: chiguyong Date: Sat, 13 Jun 2026 09:50:56 +0800 Subject: [PATCH] fix(review): address code review findings - QuadrantPanel: persist activeTab and collapsed state to localStorage via watch - TopNav: remove v-if="false" dead code for TaskSelector, remove unused ASelect import - SplitPane: add null guard before containerRef.value access in onMouseDown - Router: fix /agent/code route loading ChatView instead of WorkflowView - Router: add default redirect for /legacy to /legacy/chat --- .../frontend/src/components/layout/QuadrantPanel.vue | 12 ++++++++++-- .../frontend/src/components/layout/SplitPane.vue | 5 +++-- .../server/frontend/src/components/layout/TopNav.vue | 11 +---------- src/agentkit/server/frontend/src/router/index.ts | 6 +++++- .../server/static/assets/AgentLayout-4LUhAboc.css | 1 - .../server/static/assets/AgentLayout-BTmu8UQm.js | 1 - .../server/static/assets/AgentLayout-C3jirk5z.js | 1 + .../server/static/assets/AgentLayout-CFKQRBqo.css | 1 + .../{AppLayout-ChVEHFqE.js => AppLayout-BSXe3ZUc.js} | 2 +- ...ined-BuRUmkq3.js => AppstoreOutlined-fAjHecyI.js} | 2 +- .../{ChatView-BQlsnIIs.js => ChatView-VBSqbtQ5.js} | 2 +- .../{Checkbox-C1b034xJ.js => Checkbox-BKRvy9FI.js} | 2 +- ...eView-CNxrLPiM.js => ComputerUseView-CW9s7NtE.js} | 2 +- ...tlined-BPP2kbR8.js => DeleteOutlined-BZTTXnow.js} | 2 +- ...lined-CoQCwQyg.js => DesktopOutlined-DajX3wuP.js} | 2 +- .../{Dropdown-BUKifQVF.js => Dropdown-XbNR4d_F.js} | 4 ++-- ...ionView-BhOD-ngQ.js => EvolutionView-Bf98iUCx.js} | 2 +- ...ed-DV9WKc_3.js => FolderOpenOutlined-DjFpnogK.js} | 2 +- ...ntext-D_7H_KA_.js => FormItemContext-C7YeCp9s.js} | 2 +- ...iew-DM5c3M3U.js => KnowledgeBaseView-DF8sIvYT.js} | 2 +- ...Outlined-CXpCfAZF.js => LeftOutlined-CBUjbbRo.js} | 2 +- ...lined-CL42n2eQ.js => SettingOutlined-BgDlD6Oj.js} | 2 +- ...ingsView-C1a5n3Uj.js => SettingsView-CNi_QVj1.js} | 2 +- ...SkillsView-q_2-8csR.js => SkillsView-vANXhvRv.js} | 2 +- ...inalView-BibxhXp4.js => TerminalView-B49qvo-w.js} | 2 +- ...Outlined-BMs4GNfR.js => UserOutlined-CvCrh-WB.js} | 2 +- ...flowView-D3Pe6eXM.js => WorkflowView-Dv5wD9j5.js} | 2 +- ...7-XR.js => _plugin-vue_export-helper-Da7nhmaa.js} | 2 +- .../assets/{base-B4siOKZE.js => base-DPp00EV9.js} | 2 +- .../assets/{chat-dMZvRo2f.js => chat-pmO9UsHL.js} | 2 +- src/agentkit/server/static/assets/index-3crJqV8H.js | 1 - .../assets/{index-B5q-1V92.js => index-BEjLgAar.js} | 2 +- src/agentkit/server/static/assets/index-BMkziFFN.js | 1 - .../assets/{index-BLB5C8KY.js => index-Bh3tEcch.js} | 2 +- .../assets/{index-CIzBtwkp.js => index-C60lOYOg.js} | 2 +- .../assets/{index-DJ0mAqy8.js => index-CWabtNoR.js} | 2 +- .../assets/{index-CIdKTsyN.js => index-D4-AezQY.js} | 4 ++-- .../assets/{index-Da8dBU05.js => index-D4egOf7T.js} | 6 +++--- .../assets/{index-DBR_iMr9.js => index-DDQQQHWy.js} | 2 +- .../assets/{index-Cec7QIaL.js => index-Dd95JYlE.js} | 2 +- .../assets/{index-CzM1ezFC.js => index-DoEPe2JM.js} | 2 +- src/agentkit/server/static/assets/index-DuhbuVXy.js | 1 + src/agentkit/server/static/assets/index-DwzKHBT7.js | 1 + .../assets/{index-D6JhFblJ.js => index-KxFPh2gJ.js} | 2 +- .../assets/{index-DaJ9bCD1.js => index-RLKibtJr.js} | 2 +- .../assets/{index-Dr_Qcbdk.js => index-YXX8X6VB.js} | 2 +- src/agentkit/server/static/assets/index-fKiBikrc.js | 1 + .../assets/{index-CuVGmFKn.js => index-gcNyH-By.js} | 4 ++-- .../assets/{index-Ci55MVrK.js => index-gqpFY6Qt.js} | 4 ++-- .../assets/{index-CKNOcsSD.js => index-ldpn1699.js} | 4 ++-- src/agentkit/server/static/assets/index-yRXoO2C0.js | 1 - .../{pickAttrs-Crnv5AEc.js => pickAttrs-Ko_UXWn6.js} | 2 +- .../static/assets/responsiveObserve-CDxZiPRN.js | 1 - .../static/assets/responsiveObserve-CWMCg4ra.js | 1 + ...eChecker-CUnokkun.js => styleChecker-B3mut-jB.js} | 2 +- .../assets/{zoom-C2fVs05p.js => zoom-BFNOWbJv.js} | 4 ++-- src/agentkit/server/static/index.html | 2 +- 57 files changed, 74 insertions(+), 70 deletions(-) delete mode 100644 src/agentkit/server/static/assets/AgentLayout-4LUhAboc.css delete mode 100644 src/agentkit/server/static/assets/AgentLayout-BTmu8UQm.js create mode 100644 src/agentkit/server/static/assets/AgentLayout-C3jirk5z.js create mode 100644 src/agentkit/server/static/assets/AgentLayout-CFKQRBqo.css rename src/agentkit/server/static/assets/{AppLayout-ChVEHFqE.js => AppLayout-BSXe3ZUc.js} (79%) rename src/agentkit/server/static/assets/{AppstoreOutlined-BuRUmkq3.js => AppstoreOutlined-fAjHecyI.js} (95%) rename src/agentkit/server/static/assets/{ChatView-BQlsnIIs.js => ChatView-VBSqbtQ5.js} (99%) rename src/agentkit/server/static/assets/{Checkbox-C1b034xJ.js => Checkbox-BKRvy9FI.js} (72%) rename src/agentkit/server/static/assets/{ComputerUseView-CNxrLPiM.js => ComputerUseView-CW9s7NtE.js} (99%) rename src/agentkit/server/static/assets/{DeleteOutlined-BPP2kbR8.js => DeleteOutlined-BZTTXnow.js} (99%) rename src/agentkit/server/static/assets/{DesktopOutlined-CoQCwQyg.js => DesktopOutlined-DajX3wuP.js} (93%) rename src/agentkit/server/static/assets/{Dropdown-BUKifQVF.js => Dropdown-XbNR4d_F.js} (98%) rename src/agentkit/server/static/assets/{EvolutionView-BhOD-ngQ.js => EvolutionView-Bf98iUCx.js} (99%) rename src/agentkit/server/static/assets/{FolderOpenOutlined-DV9WKc_3.js => FolderOpenOutlined-DjFpnogK.js} (92%) rename src/agentkit/server/static/assets/{FormItemContext-D_7H_KA_.js => FormItemContext-C7YeCp9s.js} (84%) rename src/agentkit/server/static/assets/{KnowledgeBaseView-DM5c3M3U.js => KnowledgeBaseView-DF8sIvYT.js} (97%) rename src/agentkit/server/static/assets/{LeftOutlined-CXpCfAZF.js => LeftOutlined-CBUjbbRo.js} (96%) rename src/agentkit/server/static/assets/{SettingOutlined-CL42n2eQ.js => SettingOutlined-BgDlD6Oj.js} (98%) rename src/agentkit/server/static/assets/{SettingsView-C1a5n3Uj.js => SettingsView-CNi_QVj1.js} (94%) rename src/agentkit/server/static/assets/{SkillsView-q_2-8csR.js => SkillsView-vANXhvRv.js} (96%) rename src/agentkit/server/static/assets/{TerminalView-BibxhXp4.js => TerminalView-B49qvo-w.js} (77%) rename src/agentkit/server/static/assets/{UserOutlined-BMs4GNfR.js => UserOutlined-CvCrh-WB.js} (97%) rename src/agentkit/server/static/assets/{WorkflowView-D3Pe6eXM.js => WorkflowView-Dv5wD9j5.js} (99%) rename src/agentkit/server/static/assets/{_plugin-vue_export-helper-CBXJ7-XR.js => _plugin-vue_export-helper-Da7nhmaa.js} (98%) rename src/agentkit/server/static/assets/{base-B4siOKZE.js => base-DPp00EV9.js} (98%) rename src/agentkit/server/static/assets/{chat-dMZvRo2f.js => chat-pmO9UsHL.js} (98%) delete mode 100644 src/agentkit/server/static/assets/index-3crJqV8H.js rename src/agentkit/server/static/assets/{index-B5q-1V92.js => index-BEjLgAar.js} (98%) delete mode 100644 src/agentkit/server/static/assets/index-BMkziFFN.js rename src/agentkit/server/static/assets/{index-BLB5C8KY.js => index-Bh3tEcch.js} (97%) rename src/agentkit/server/static/assets/{index-CIzBtwkp.js => index-C60lOYOg.js} (97%) rename src/agentkit/server/static/assets/{index-DJ0mAqy8.js => index-CWabtNoR.js} (83%) rename src/agentkit/server/static/assets/{index-CIdKTsyN.js => index-D4-AezQY.js} (71%) rename src/agentkit/server/static/assets/{index-Da8dBU05.js => index-D4egOf7T.js} (87%) rename src/agentkit/server/static/assets/{index-DBR_iMr9.js => index-DDQQQHWy.js} (97%) rename src/agentkit/server/static/assets/{index-Cec7QIaL.js => index-Dd95JYlE.js} (98%) rename src/agentkit/server/static/assets/{index-CzM1ezFC.js => index-DoEPe2JM.js} (63%) create mode 100644 src/agentkit/server/static/assets/index-DuhbuVXy.js create mode 100644 src/agentkit/server/static/assets/index-DwzKHBT7.js rename src/agentkit/server/static/assets/{index-D6JhFblJ.js => index-KxFPh2gJ.js} (98%) rename src/agentkit/server/static/assets/{index-DaJ9bCD1.js => index-RLKibtJr.js} (83%) rename src/agentkit/server/static/assets/{index-Dr_Qcbdk.js => index-YXX8X6VB.js} (93%) create mode 100644 src/agentkit/server/static/assets/index-fKiBikrc.js rename src/agentkit/server/static/assets/{index-CuVGmFKn.js => index-gcNyH-By.js} (94%) rename src/agentkit/server/static/assets/{index-Ci55MVrK.js => index-gqpFY6Qt.js} (97%) rename src/agentkit/server/static/assets/{index-CKNOcsSD.js => index-ldpn1699.js} (71%) delete mode 100644 src/agentkit/server/static/assets/index-yRXoO2C0.js rename src/agentkit/server/static/assets/{pickAttrs-Crnv5AEc.js => pickAttrs-Ko_UXWn6.js} (97%) delete mode 100644 src/agentkit/server/static/assets/responsiveObserve-CDxZiPRN.js create mode 100644 src/agentkit/server/static/assets/responsiveObserve-CWMCg4ra.js rename src/agentkit/server/static/assets/{styleChecker-CUnokkun.js => styleChecker-B3mut-jB.js} (92%) rename src/agentkit/server/static/assets/{zoom-C2fVs05p.js => zoom-BFNOWbJv.js} (90%) diff --git a/src/agentkit/server/frontend/src/components/layout/QuadrantPanel.vue b/src/agentkit/server/frontend/src/components/layout/QuadrantPanel.vue index 84887f7..5d255c5 100644 --- a/src/agentkit/server/frontend/src/components/layout/QuadrantPanel.vue +++ b/src/agentkit/server/frontend/src/components/layout/QuadrantPanel.vue @@ -30,7 +30,7 @@