refactor: simplify cmd package and consolidate documentation#39
Merged
refactor: simplify cmd package and consolidate documentation#39
Conversation
- --targets 플래그 제거, 언어 기반 라우팅으로 단순화
- --register-mcp, --setup-llm 플래그 제거 - .sym/config.json 생성 추가
- Symphony 섹션 마커 추가 (idempotent 업데이트) - claude.md → CLAUDE.md 네이밍 규칙 적용 - 도움말 텍스트 업데이트
- bufio.ReadString → survey.Select로 변경
- sym init --setup-llm → sym llm setup 안내 수정
- CONVERT_FEATURE.md, CONVERT_USAGE.md 삭제 - LINTER_VALIDATION.md, LLM_VALIDATOR.md 삭제 - command.md 추가 (통합 CLI 레퍼런스)
- Remove cmd/git.go, use internal/git.GetRepoRoot() instead - Update init and validate commands to use git package - Apply verbose flag consistently in convert/validate
- Remove auto-dashboard launch from mcp command - Remove claude-desktop and cline support from mcp register - Remove unused fileExists and launchDashboard functions
- Remove saveAPIKeyToEnv wrapper, call envutil directly - Inline handleError in my_role command
- Change appendMode to isSharedFile for clarity - Remove unnecessary .bak backup for Symphony-dedicated files - Consistent logic with MCP config handling
- internal/ui/colors.go → internal/cmd/colors.go - 모든 exported 함수/상수를 private으로 전환 - 미사용 함수 삭제 (Info, PrintInfo, Title, PrintIndent) - 8개 의존 파일에서 import 및 호출 업데이트
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Changes
패키지 구조 개선
internal/ui/colors.go→internal/cmd/colors.go로 이동 및 private 함수로 전환internal/cmd/git.go삭제,internal/git패키지 직접 사용으로 변경saveAPIKeyToEnv,handleError등)명령어 인터페이스 단순화
--register-mcp,--setup-llm플래그 제거,.sym/config.json생성 추가--targets플래그 제거, 언어 기반 라우팅으로 자동 결정UX 개선
bufio.ReadString→survey.Select로 변경하여 선택 UX 개선claude.md→CLAUDE.md네이밍 규칙 적용, 불필요한 백업 생성 제거문서 정리
CONVERT_FEATURE.md,CONVERT_USAGE.md,LINTER_VALIDATION.md,LLM_VALIDATOR.md삭제docs/command.md추가 (통합 CLI 레퍼런스)