Skip to content

fix(skeleton): PYTHONPATH 없이 skeleton 스크립트 실행 가능하도록 개선#723

Merged
jk-kim0 merged 1 commit intomainfrom
feat/remove-pythonpath-requirement
Feb 14, 2026
Merged

fix(skeleton): PYTHONPATH 없이 skeleton 스크립트 실행 가능하도록 개선#723
jk-kim0 merged 1 commit intomainfrom
feat/remove-pythonpath-requirement

Conversation

@jk-kim0
Copy link
Copy Markdown
Contributor

@jk-kim0 jk-kim0 commented Feb 14, 2026

Summary

  • PYTHONPATH=bin 접두사 없이 bin/skeleton/cli.py를 직접 실행할 수 있도록 개선합니다
  • Python 스크립트가 자체적으로 sys.path를 설정하여 skeleton 패키지를 찾습니다

변경 내용

  • bin/skeleton/cli.py, bin/skeleton/diff.py: import 전에 bin/ 디렉토리를 sys.path에 자동 추가
  • bin/review-skeleton-diff.sh: PYTHONPATH=bin 접두사 제거
  • tests/run-tests.sh: PYTHONPATH export 제거
  • .claude/skills/mdx-skeleton-comparison.md: 실행 예시에서 PYTHONPATH=bin 제거
  • docs/translation.md: 실행 예시 업데이트

Before / After

# Before
PYTHONPATH=bin bin/skeleton/cli.py -r

# After
bin/skeleton/cli.py -r

Test plan

  • bin/skeleton/cli.py --help — PYTHONPATH 없이 정상 실행 확인
  • python -m pytest tests/test_mdx_to_skeleton.py -v — 58 passed
  • tests/run-tests.sh --type skeleton — 18 passed, 0 failed

🤖 Generated with Claude Code

- bin/skeleton/cli.py, diff.py에 sys.path 자동 설정을 추가합니다
- bin/review-skeleton-diff.sh에서 PYTHONPATH=bin 접두사를 제거합니다
- tests/run-tests.sh에서 PYTHONPATH export를 제거합니다
- Skill 문서, docs/translation.md에서 실행 예시를 업데이트합니다

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@vercel
Copy link
Copy Markdown

vercel Bot commented Feb 14, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
querypie-docs Ready Ready Preview, Comment Feb 14, 2026 10:18am

Request Review

@jk-kim0 jk-kim0 self-assigned this Feb 14, 2026
@jk-kim0 jk-kim0 merged commit e00d3b3 into main Feb 14, 2026
7 checks passed
@jk-kim0 jk-kim0 deleted the feat/remove-pythonpath-requirement branch February 14, 2026 10:19
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.

1 participant