Skip to content

Conversation

@ParkGyeongTae
Copy link
Contributor

#️⃣ Issue Number

📝 요약(Summary)

  • 기존 lang2sql.py에 집중되어 있던 실행/결과 표시/세션 초기화 로직을 모듈 단위로 분리했습니다.
  • UI 중심 코드와 내부 실행 로직을 구분하여 유지보수성과 확장성을 개선했습니다.
  • Streamlit 페이지는 UI 전용으로 단순화되었습니다.

💬 To Reviewers (선택)

  • 모듈 분리 구조(lang2sql_runner.py, result_renderer.py, session_utils.py)가 적절한지 확인 부탁드립니다.

PR Checklist

  • Lang2SQL 실행 로직 분리 (lang2sql_runner.py)
  • 결과 표시 모듈화 (result_renderer.py)
  • 세션 상태 초기화 분리 (session_utils.py)
  • 기존 lang2sql.py에서 UI 로직만 유지

reference) How to Code Review

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

@ParkGyeongTae ParkGyeongTae self-assigned this Oct 4, 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.

👍 잘 동작하는것을 확인했습니다! UI 부분도 좀 더 기여하기 나은 환경이 되어가는듯 합니다!

@ParkGyeongTae ParkGyeongTae merged commit 61b93e7 into master Oct 4, 2025
2 checks passed
@ehddnr301 ehddnr301 deleted the 182-lang2sql-modularization 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