-
Notifications
You must be signed in to change notification settings - Fork 23
V1.4 features: Docsum and Codegen enabled #61
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR adds support for document summarization (Docsum) and code generation (Codegen) features in V1.4, updates E2E tests to match new UI locators and statuses, and integrates new microservices (ASR, Docsum) into deployment manifests.
- Update Playwright tests and configuration for new status texts and element locators.
- Revise FlowListTable layout and add sandbox status indicators.
- Introduce new server nodes (llm_docsum, llm_codegen, asr), update exporter logic, and extend backend/frontend manifests and routing for new services.
Reviewed Changes
Copilot reviewed 150 out of 179 changed files in this pull request and generated no comments.
Show a summary per file
File | Description |
---|---|
tests/playwright/studio-e2e/002_test_sandbox_chatqna.spec.ts | Updated locators, status text, and upload flows for chat/QnA |
tests/playwright/studio-e2e/001_test_sandbox_deployment.spec.ts | Adjusted locators to V1.4 sandbox controls and observability |
tests/playwright/playwright.config.js | Set baseURL to local HTTPS endpoint |
studio-frontend/packages/ui/src/ui-component/table/FlowListTable.jsx | Swapped columns, adjusted widths, and added status spinner |
studio-frontend/packages/server/src/nodes/llm_docsum.js | Added LLM Document Summarization node |
studio-frontend/packages/server/src/nodes/llm_codegen.js | Added LLM Code Generation node |
studio-frontend/packages/server/src/nodes/asr.js | Added Audio/Speech Recognition node |
studio-backend/app/utils/exporter_utils.py | Extended mappings for new services and debug logging |
studio-backend/app/templates/microsvc-manifests/whisper.yaml | New Whisper microservice manifest |
studio-backend/app/templates/microsvc-manifests/llm-uservice.yaml | Updated image reference to use dynamic IMAGE_REPOSITORY |
studio-backend/app/templates/microsvc-manifests/asr-usvc.yaml | New ASR microservice manifest |
studio-backend/app/templates/app/app.manifest.yaml | Added chat history service and environment variables |
studio-backend/app/routers/clickdeploy_router.py | Added endpoints for click-deploy and WebSocket status checks |
app-frontend/react/src/redux/Conversation/ConversationSlice.ts | Major refactor to support summary, codegen, and data source |
app-frontend/react/src/pages/History/HistoryView.tsx | New history view with multi-select and delete functionality |
app-frontend/react/src/pages/DataSource/DataSourceManagement.tsx | New data source management page |
Comments suppressed due to low confidence (3)
studio-frontend/packages/ui/src/ui-component/table/FlowListTable.jsx:320
- Reused key '1b' (and '1a') across multiple elements can cause React key collisions. Use unique keys or more descriptive identifiers.
<StyledTableCell style={{ width: '15%' }} key='1b'>
studio-backend/app/routers/clickdeploy_router.py:156
- The code
await websocket.close()from fastapi import...
is missing a newline or separator. It will cause a syntax error. Split into two statements or insert a semicolon.
await websocket.close()from fastapi import APIRouter, HTTPException, WebSocket, WebSocketDisconnect
studio-backend/app/utils/exporter_utils.py:145
- [nitpick] Remove or guard debug print statements before production; use a structured logger if visibility into runtime data is needed.
print("process_opea_services: node_name", node_name, "node_info", node_info)
266382c
to
30c8ff3
Compare
Signed-off-by: wwanarif <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
No description provided.