kubeflow / kubeflow/docs-agent

Deprecate duplicated code in WebSocket and HTTP servers

Open
#60 1 comment 0 reactions 0 assignees View on GitHub
Dominant language
Python
Stars
42
Forks
111
Avg merge
6d 23m
Merged PRs (30d)
2

Description

Based on feedback from PR #49, the duplicated code in `server/app.py` and `server-https/app.py` will be deprecated in the future. The current code duplication is intentional to ensure appropriate deployment and ease of switching between socket and HTTP servers.

This issue is to track the future cleanup and deprecation of the current architecture once a unified or improved approach is adopted.

**Context:**
- The extraction of the shared RAG module (in PR #49) was reverted to maintain the duplication for now.
- Moving forward, this duplicated architecture should be revisited for deprecation.

Contributor guide

Open the contributing guide

Research direction

Start by comparing server/app.py and server-https/app.py, then review PR #49 and its reverted shared RAG module extraction. The issue does not define the replacement architecture; completion requires an agreed unified approach followed by deprecating the duplicated server code.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
backend
Issue type
Refactor
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.