Skip to content

Conversation

@ParkGyeongTae
Copy link
Contributor

#️⃣ Issue Number

📝 요약(Summary)

  • 기존 단일 파일에 포함된 CLI 로직을 기능별 모듈(core, commands, utils)로 분리했습니다.
  • run_streamlit, query, environment, streamlit_runner 등 각 역할에 맞는 파일로 나눠 유지보수성을 개선했습니다.
  • CLI 엔트리포인트(cli/__init__.py)는 명령어 등록 및 실행만 담당하도록 단순화했습니다.

💬 To Reviewers (선택)

  • 모듈 분리된 구조(cli/core, cli/commands, cli/utils)가 적절한지 확인 부탁드립니다.

PR Checklist

  • CLI 모듈 분리 및 코드 리팩토링
  • 기존 CLI 동작 (run-streamlit, query) 정상 실행 확인

reference) How to Code Review

  • 따봉(👍): 리뷰어가 리뷰이의 코드에서 칭찬의 의견을 남기고 싶을 때 사용합니다.
  • 느낌표(❗): 리뷰어가 리뷰이에게 필수적으로 코드 수정을 요청할 때 사용합니다.
  • 물음표 (❓): 리뷰어가 리뷰이에게 의견을 물어보고 싶을 때 사용합니다.
  • 알약 (💊): 리뷰어가 리뷰이의 코드에서 개선된 방법을 제안하지만 그것의 반영이 필수까지는 아닐 때 사용합니다.

@ParkGyeongTae ParkGyeongTae self-assigned this Sep 24, 2025
Copy link
Collaborator

@ehddnr301 ehddnr301 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 잘 동작합니다아ㅏㅏ 앞으로 테스트나 기여가 좀 더 쉬워지겠네요오!!

저는 아래 명령어들로 테스트해봤습니다! 잘 응답오거나 실행되는것을 확인했습니다!

uv run lang2sql --datahub_server http://34.16.99.175:8080 run-streamlit -p 30859

uv run lang2sql --vectordb-type faiss --vectordb-location ./dev/table_info_db run-streamlit -p 30859

uv run lang2sql --datahub_server http://34.16.99.175:8080 --env-file-path .env.dev --prompt-dir-path ./prompts run-streamlit -p 30859

uv run lang2sql query "신규 유저" --top-n 5 --database-env clickhouse --device cpu 

uv run lang2sql query "신규 유저" --vectordb-type faiss --vectordb-location ./dev/table_info_db --top-n 8 --use-enriched-graph 

uv run lang2sql query "신규 유저" --vectordb-type pgvector --vectordb-location "postgresql://postgres:postgres@localhost:5431/postgres" 

💬 앞으로는 이런 테스트를 자동으로 돌려도 좋겠네요! 늘 좋은 인사이트 주셔서 감사합니다!

@ParkGyeongTae ParkGyeongTae merged commit 1275902 into master Sep 28, 2025
2 checks passed
@ehddnr301 ehddnr301 deleted the 167-refactor-cli-split-modules branch October 11, 2025 11:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants