Skip to content

Make default profile help primary#6

Merged
akcorca merged 1 commit into
mainfrom
codex/profile-default-help
May 20, 2026
Merged

Make default profile help primary#6
akcorca merged 1 commit into
mainfrom
codex/profile-default-help

Conversation

@akcorca
Copy link
Copy Markdown
Contributor

@akcorca akcorca commented May 20, 2026

Summary

  • remove --profile from primary Go CLI help examples
  • document that default profile is used unless --profile or CRAKEN_PROFILE is set
  • bump VERSION to v0.1.7

Tests

  • go test ./...
  • ./scripts/check_coverage.sh 35
  • golangci-lint run ./...
  • go build ./cmd/craken

Summary by CodeRabbit

릴리스 노트

  • Documentation

    • 로그인 명령 사용 예시를 단순화했습니다. 기본 설정으로 진행되는 방식을 강조하며, 브라우저가 다른 머신에 있는 경우의 --no-open 옵션 사용법을 추가했습니다.
  • Chores

    • 버전을 v0.1.7로 업데이트했습니다.

Review Change Stack

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 20, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 2406c008-acc4-4a66-8215-a7862d500da1

📥 Commits

Reviewing files that changed from the base of the PR and between 17d43de and 9c6dbba.

📒 Files selected for processing (4)
  • README.md
  • VERSION
  • internal/craken/command.go
  • internal/craken/command_test.go
📜 Recent review details
🔇 Additional comments (4)
README.md (1)

23-23: LGTM!

Also applies to: 26-26, 31-38, 40-40

VERSION (1)

1-1: LGTM!

internal/craken/command.go (1)

186-187: LGTM!

Also applies to: 190-190

internal/craken/command_test.go (1)

61-61: LGTM!

Also applies to: 95-96, 108-109


Walkthrough

이 PR은 Craken CLI에서 기본 프로필(default)을 가정하도록 문서와 도움말 예시를 정리합니다. --profile 플래그를 명시적으로 표시하지 않도록 통일하며, 관련 테스트를 갱신하고 버전을 v0.1.7로 업데이트합니다.

Changes

프로필 플래그 제거 및 도움말 정리

Layer / File(s) Summary
문서 및 버전 업데이트
README.md, VERSION
README의 로그인 예시를 craken auth login --profile ak에서 craken auth login으로 변경하고, 기본 프로필에 저장되는 동작과 --no-open 옵션 사용 조건을 설명에 추가하며, 사용 예시 커맨드에서 --profile ak 플래그를 제거합니다. 버전을 v0.1.6에서 v0.1.7로 갱신합니다.
도움말 출력 및 테스트 정렬
internal/craken/command.go, internal/craken/command_test.go
printCatalogHelp의 예시 커맨드에서 --profile PROFILE 인자를 제거하고, 테스트에서 프로필 없는 기본 커맨드(craken auth login, craken commands --format text)가 출력에 포함되는지 검증하며, 프로필 기반 변형 커맨드(--profile PROFILE 포함)는 포함되지 않는지 확인하는 "stale" 검사 항목을 추가합니다.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

  • corca-ai/craken-cli#4: 이 PR과 동일하게 internal/craken/command.goprintCatalogHelp 로직과 internal/craken/command_test.go의 도움말 렌더링 테스트를 수정하는 서버 카탈로그 기반 도움말 렌더링 변경입니다.

  • corca-ai/craken-cli#5: printCatalogHelp 도움말 출력 예시와 서버 커맨드 카탈로그 렌더링 로직의 테스트를 동일하게 수정하므로 코드 레벨에서 직접 관련됩니다.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed PR 제목 '기본 프로필 도움말을 주요 내용으로'는 변경 사항의 주요 목적을 명확하게 요약합니다. 기본 프로필을 기본값으로 사용하고 --profile 플래그를 제거하는 변경의 핵심을 포착합니다.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch codex/profile-default-help

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@akcorca akcorca merged commit 22cfd35 into main May 20, 2026
2 checks passed
@akcorca akcorca deleted the codex/profile-default-help branch May 20, 2026 12:26
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