app frontend for v1.4#16
Conversation
fixed llm-tracer tracer not found issue Signed-off-by: wwanarif <wan.abdul.hakim.b.wan.arif@intel.com>
…kflow (opea-project#54) Signed-off-by: Sun, Xuehao <xuehao.sun@intel.com>
There was a problem hiding this comment.
Pull Request Overview
This pull request implements the v1.4 frontend update. Key changes include a major refactor of the ConversationSlice (especially the event stream handling logic), updates to build configuration (including tsconfig changes targeting ES5), and numerous component and style adjustments across the app.
Reviewed Changes
Copilot reviewed 117 out of 135 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| app-frontend/react/tsconfig.json | Updated compiler options including target downgrade from ES2020 to ES5 and additional paths and lib modifications. |
| app-frontend/react/src/redux/User/userSlice.ts | Modified user slice with changes to action payload and slice naming from "user" to "init user". |
| app-frontend/react/src/redux/Conversation/ConversationSlice.ts | Extensive refactor of conversation state management and streaming event handling functions. |
| Multiple component/scss files | Various component, layout, and style changes for enhanced user interface and theming consistency. |
| config.ts | Updated environment configuration and added additional logging of service endpoint URLs. |
Comments suppressed due to low confidence (2)
app-frontend/react/tsconfig.json:3
- Changing the target from ES2020 to es5 may affect performance and modern feature availability; please confirm that targeting ES5 is an intentional requirement.
"target": "es5",
app-frontend/react/src/redux/User/userSlice.ts:15
- [nitpick] The slice name 'init user' is ambiguous; consider renaming it to 'user' to better reflect that it handles the current user's state.
name: "init user",
| eventStream(type, body, conversationId); | ||
| }; | ||
|
|
||
| const eventStream = (type: string, body: any, conversationId: string = "") => { |
There was a problem hiding this comment.
[nitpick] The eventStream function is very long and handles several complex tasks; refactoring it into smaller helper functions could improve readability and ease future maintenance.
|
reraise PR after fixing conflicts locally |
No description provided.