feat: add kakao channel, template, and brand template management#5
feat: add kakao channel, template, and brand template management#5Palbahngmiyine merged 3 commits intomainfrom
Conversation
…ands Add comprehensive Kakao business channel inquiry, alimtalk template management, and brand template management to the CLI. This enables users to manage their Kakao messaging resources directly from solactl. New command groups: - `solactl kakao channel` (list/get/categories) — channel inquiry - `solactl kakao channel-group` (list/get) — channel group inquiry - `solactl kakao template` (list/get/create/update/delete/inspect/ cancel-inspect/sendable/categories/release-dormant) — full alimtalk template lifecycle - `solactl kakao brand-template` (list/create/update/delete/sendable) — brand template CRUD All list commands support cursor-based pagination with --start-key and --limit flags. Sensitive fields (senderKeys, groupKeys) are excluded from response types. 83 tests covering success/failure/pagination/XOR validation/delete confirmation paths. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request implements a comprehensive suite of CLI commands for managing KakaoTalk channels, channel groups, alimtalk templates, and brand templates, including full CRUD support and inspection workflows. Feedback focuses on resolving compilation errors caused by an undefined stdinReader variable in deletion commands, ensuring consistent output behavior by removing redundant JSON printing in creation commands, and refining the brand template update logic to correctly handle optional JSON fields.
|
|
||
| if !kakaoBtplDeleteFlagYes { | ||
| _, _ = fmt.Fprintf(out(), "\u26a0 브랜드 템플릿 %s을 삭제하시겠습니까? (y/N): ", brandTemplateID) | ||
|
|
There was a problem hiding this comment.
stdinReader 변수가 cmd 패키지 내의 비-테스트 파일 어디에도 정의되어 있지 않습니다. 이로 인해 컴파일 오류("undefined: stdinReader")가 발생합니다. 테스트 시 입력을 모킹하기 위한 용도라면 cmd/root.go 등에 패키지 수준 변수로 선언해야 합니다.
| func runKakaoTemplateDelete(cmd *cobra.Command, args []string) error { | ||
| templateID := args[0] | ||
|
|
||
| if !kakaoTplDeleteFlagYes { |
There was a problem hiding this comment.
stdinReader 변수가 cmd 패키지 내의 비-테스트 파일 어디에도 정의되어 있지 않습니다. 이로 인해 컴파일 오류("undefined: stdinReader")가 발생합니다. 테스트 시 입력을 모킹하기 위한 용도라면 cmd/root.go 등에 패키지 수준 변수로 선언해야 합니다.
| _, _ = fmt.Fprintln(out(), "브랜드 템플릿이 생성되었습니다.") | ||
| return p.PrintJSON(raw) |
There was a problem hiding this comment.
--json 플래그가 지정되지 않은 경우에도 생성된 리소스의 전체 JSON을 출력하고 있습니다. 이는 다른 커맨드들과 일관성이 없으며, 일반적인 텍스트 모드 출력으로는 부적절합니다. 성공 메시지만 출력하거나 p.PrintResult를 사용하여 플래그에 따라 출력을 제어하는 것이 좋습니다.
| _, _ = fmt.Fprintln(out(), "브랜드 템플릿이 생성되었습니다.") | |
| return p.PrintJSON(raw) | |
| _, _ = fmt.Fprintln(out(), "브랜드 템플릿이 생성되었습니다.") | |
| return nil |
| jsonFields := map[string]string{ | ||
| "carousel": kakaoBtplUpdateFlagCarousel, | ||
| "mainWideItem": kakaoBtplUpdateFlagMainWideItem, | ||
| "subWideItemList": kakaoBtplUpdateFlagSubWideItemList, | ||
| "video": kakaoBtplUpdateFlagVideo, | ||
| "commerce": kakaoBtplUpdateFlagCommerce, | ||
| "buttons": kakaoBtplUpdateFlagButtons, | ||
| "coupon": kakaoBtplUpdateFlagCoupon, | ||
| } | ||
| for key, value := range jsonFields { | ||
| if err := setJSONField(body, key, value); err != nil { | ||
| return err | ||
| } | ||
| } |
There was a problem hiding this comment.
JSON 필드(carousel, video 등)를 업데이트할 때 cmd.Flags().Changed()를 확인하지 않고 루프를 통해 처리하고 있습니다. 또한 main-wide-item과 같이 플래그 이름과 JSON 키가 다른 경우에 대한 처리가 누락되어 있어 의도한 대로 동작하지 않을 수 있습니다. kakao_template_update.go와 같이 각 필드별로 Changed() 여부를 확인하여 명시적으로 처리하는 것이 안전합니다.
| return p.PrintJSON(raw) | ||
| } |
There was a problem hiding this comment.
--json 플래그가 지정되지 않은 경우에도 생성된 리소스의 전체 JSON을 출력하고 있습니다. 이는 다른 커맨드들과 일관성이 없으며, 일반적인 텍스트 모드 출력으로는 부적절합니다. 성공 메시지만 출력하거나 p.PrintResult를 사용하여 플래그에 따라 출력을 제어하는 것이 좋습니다.
_, _ = fmt.Fprintln(out(), "알림톡 템플릿이 생성되었습니다.")
return nil- Fix double output in create commands (success message + raw JSON)
- Remove dead --channel-id/--channel-group-id flags from template
update (API does not support changing channel via update)
- Gate brand-template update JSON fields on Changed state to prevent
sending empty/default values for unspecified flags
- Replace variadic interface{} setIfChanged with typed
setStringIfChanged and setJSONIfChanged helpers
- Replace custom contains/searchString with strings.Contains in tests
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
- Replace all interface{} with any in kakao command and test files
- Add resetPflagChanged helper to clear pflag Changed bits between
tests, preventing order-dependent test failures in partial update
scenarios
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 participant
Summary
kakao channel— list/get/categories)kakao channel-group— list/get, 소속 채널 포함)kakao template— list/get/create/update/delete/inspect/cancel-inspect/sendable/categories/release-dormant)kakao brand-template— list/create/update/delete/sendable)--start-key,--limit) 및 다음 페이지 안내Test plan
go test ./cmd/ -run TestKakaoChannel -count=1— 채널/채널 그룹 테스트go test ./cmd/ -run TestKakaoTemplate -count=1— 알림톡 템플릿 테스트go test ./cmd/ -run TestKakaoBrandTemplate -count=1— 브랜드 템플릿 테스트go test ./pkg/types/ -run TestKakao -count=1— 타입 테스트go test ./cmd/ -run TestKakao -race -count=1— race detectiongo build ./...— 빌드 성공go vet ./...— 정적 분석 통과🤖 Generated with Claude Code