Commit 424e563
LLM Chat Interface with MCP Enabled Tool Orchestration (#1202)
* MCP client implemented as new tab LLM Chat
Signed-off-by: Keval Mahajan <[email protected]>
* Improvements
Signed-off-by: Keval Mahajan <[email protected]>
* experience changes
Signed-off-by: Keval Mahajan <[email protected]>
* Improved MCP client backend
Signed-off-by: Keval Mahajan <[email protected]>
* Fix scrolling to the latest message
Signed-off-by: Keval Mahajan <[email protected]>
* Fixed jwt token for servers
Signed-off-by: Keval Mahajan <[email protected]>
* require auth token for team and private level servers
Signed-off-by: Keval Mahajan <[email protected]>
* proper error handling
Signed-off-by: Keval Mahajan <[email protected]>
* Thinking functionality display
Signed-off-by: Keval Mahajan <[email protected]>
* responsive tabs
Signed-off-by: Keval Mahajan <[email protected]>
* code standard fixes
Signed-off-by: Keval Mahajan <[email protected]>
* updated env example
Signed-off-by: Keval Mahajan <[email protected]>
* added new optional dependencies
Signed-off-by: Keval Mahajan <[email protected]>
* Fixed web linting
Signed-off-by: Keval Mahajan <[email protected]>
* Fix linting issues for PR #1202
- Fix bandit B105 false positive in llmchat_router.py by adding nosec comment for empty string check
- Fix CSS linting issues in admin.css:
- Rename keyframes to kebab-case (messageSlideIn -> message-slide-in, fadeIn -> fade-in, thinkingStepIn -> thinking-step-in)
- Replace deprecated word-break: break-word with overflow-wrap: break-word
- Split single-line declaration blocks into multi-line format
- Remove duplicate @Keyframes fade-in and .animate-fade-in selectors
- Merge duplicate .thinking-step selector with animation property
All pytest tests pass (3392 passed, 45 skipped)
All quality checks pass (flake8, bandit, interrogate, pylint, verify)
All web linting passes (stylelint, htmlhint)
* Fix doctest failures by adding import guards for optional LLM dependencies
Wrapped langchain imports in try/except block to handle environments where
the optional LLM chat dependencies are not installed. This allows doctest
to run successfully without requiring these dependencies.
- Added _LLMCHAT_AVAILABLE flag to track availability
- Set imports to None with type ignore when unavailable
- Prevents ModuleNotFoundError during doctest runs
* Linting and test fixes
Signed-off-by: Mihai Criveti <[email protected]>
* Web lint
Signed-off-by: Mihai Criveti <[email protected]>
---------
Signed-off-by: Keval Mahajan <[email protected]>
Signed-off-by: Mihai Criveti <[email protected]>
Co-authored-by: Mihai Criveti <[email protected]>1 parent c7330e0 commit 424e563
File tree
34 files changed
+11017
-4802
lines changed- docs/docs/using/servers/third-party
- mcp-servers/go/system-monitor-server
- cmd/server
- internal
- config
- metrics
- monitor
- pkg/types
- mcpgateway
- alembic/versions
- middleware
- routers
- services
- static
- templates
- plugins/external/opa/tests
34 files changed
+11017
-4802
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
840 | 840 | | |
841 | 841 | | |
842 | 842 | | |
| 843 | + | |
| 844 | + | |
| 845 | + | |
| 846 | + | |
| 847 | + | |
| 848 | + | |
| 849 | + | |
| 850 | + | |
| 851 | + | |
| 852 | + | |
| 853 | + | |
| 854 | + | |
| 855 | + | |
| 856 | + | |
| 857 | + | |
| 858 | + | |
| 859 | + | |
| 860 | + | |
| 861 | + | |
| 862 | + | |
| 863 | + | |
| 864 | + | |
| 865 | + | |
| 866 | + | |
| 867 | + | |
| 868 | + | |
| 869 | + | |
| 870 | + | |
| 871 | + | |
| 872 | + | |
| 873 | + | |
| 874 | + | |
| 875 | + | |
| 876 | + | |
| 877 | + | |
| 878 | + | |
| 879 | + | |
| 880 | + | |
| 881 | + | |
| 882 | + | |
| 883 | + | |
| 884 | + | |
| 885 | + | |
| 886 | + | |
| 887 | + | |
| 888 | + | |
| 889 | + | |
| 890 | + | |
| 891 | + | |
| 892 | + | |
| 893 | + | |
| 894 | + | |
| 895 | + | |
| 896 | + | |
| 897 | + | |
| 898 | + | |
| 899 | + | |
| 900 | + | |
| 901 | + | |
| 902 | + | |
| 903 | + | |
| 904 | + | |
| 905 | + | |
| 906 | + | |
| 907 | + | |
| 908 | + | |
| 909 | + | |
| 910 | + | |
| 911 | + | |
| 912 | + | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1182 | 1182 | | |
1183 | 1183 | | |
1184 | 1184 | | |
| 1185 | + | |
| 1186 | + | |
| 1187 | + | |
| 1188 | + | |
| 1189 | + | |
| 1190 | + | |
| 1191 | + | |
| 1192 | + | |
| 1193 | + | |
| 1194 | + | |
| 1195 | + | |
| 1196 | + | |
| 1197 | + | |
| 1198 | + | |
| 1199 | + | |
| 1200 | + | |
| 1201 | + | |
| 1202 | + | |
| 1203 | + | |
| 1204 | + | |
| 1205 | + | |
| 1206 | + | |
| 1207 | + | |
| 1208 | + | |
| 1209 | + | |
| 1210 | + | |
| 1211 | + | |
| 1212 | + | |
| 1213 | + | |
| 1214 | + | |
| 1215 | + | |
| 1216 | + | |
| 1217 | + | |
| 1218 | + | |
| 1219 | + | |
| 1220 | + | |
| 1221 | + | |
| 1222 | + | |
| 1223 | + | |
| 1224 | + | |
| 1225 | + | |
| 1226 | + | |
| 1227 | + | |
| 1228 | + | |
| 1229 | + | |
| 1230 | + | |
| 1231 | + | |
| 1232 | + | |
| 1233 | + | |
| 1234 | + | |
| 1235 | + | |
| 1236 | + | |
| 1237 | + | |
| 1238 | + | |
| 1239 | + | |
| 1240 | + | |
| 1241 | + | |
| 1242 | + | |
| 1243 | + | |
| 1244 | + | |
| 1245 | + | |
| 1246 | + | |
| 1247 | + | |
| 1248 | + | |
| 1249 | + | |
| 1250 | + | |
| 1251 | + | |
| 1252 | + | |
| 1253 | + | |
| 1254 | + | |
| 1255 | + | |
| 1256 | + | |
| 1257 | + | |
| 1258 | + | |
| 1259 | + | |
| 1260 | + | |
| 1261 | + | |
| 1262 | + | |
| 1263 | + | |
1185 | 1264 | | |
1186 | 1265 | | |
1187 | 1266 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | 1 | | |
2 | | - | |
| 2 | + | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
753 | 753 | | |
754 | 754 | | |
755 | 755 | | |
756 | | - | |
| 756 | + | |
0 commit comments