Skip to content

Add message limit option#15

Merged
akcorca merged 1 commit into
mainfrom
codex/message-limit
May 21, 2026
Merged

Add message limit option#15
akcorca merged 1 commit into
mainfrom
codex/message-limit

Conversation

@akcorca
Copy link
Copy Markdown
Contributor

@akcorca akcorca commented May 21, 2026

Summary

  • send --limit through channel and DM message page requests
  • show --limit in focused message-command help
  • document message limit examples and preserve service validation errors

Closes #10

Validation

  • mise exec -- go test ./...
  • CGO_ENABLED=1 mise exec -- go test -race ./...
  • mise exec -- ./scripts/check_coverage.sh 35

Summary by CodeRabbit

릴리스 노트

  • 새로운 기능

    • 메시지 관련 명령에 --limit 옵션 추가로 페이지네이션 제어 기능 개선
  • 문서

    • README.md 사용 예시 확장 (메시지 조회 명령 예제 추가)
    • 아키텍처 문서에 메시지 페이지네이션 동작 설명 추가
  • 테스트

    • 메시지 명령의 limit 쿼리 전송 검증 테스트 추가
    • 서버 검증 에러 처리 테스트 추가

Review Change Stack

@akcorca akcorca merged commit 82b2599 into main May 21, 2026
1 check passed
@akcorca akcorca deleted the codex/message-limit branch May 21, 2026 16:17
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 21, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: fe4a63b8-94a0-44fe-8632-0223acf1adc7

📥 Commits

Reviewing files that changed from the base of the PR and between 2664cfb and 872efde.

📒 Files selected for processing (5)
  • README.md
  • docs/architecture.md
  • internal/craken/command.go
  • internal/craken/command_test.go
  • internal/craken/resources.go

Walkthrough

메시지 조회 명령(channel messages, dm messages)에 --limit 옵션을 추가하여 페이지 크기 제어를 가능하게 합니다. 쿼리 파라미터 처리, 도움말 텍스트, 문서, 사용자 예시를 업데이트하고 쿼리 전달 및 서버 유효성 검사를 검증하는 테스트를 추가합니다.

Changes

메시지 조회 --limit 옵션 추가

Layer / File(s) Summary
아키텍처 및 사용자 예시 문서
docs/architecture.md, README.md
메시지 페이지 명령이 --limit과 cursor 옵션을 서비스로 전달하는 아키텍처 규칙이 추가되고, README 사용 예시에 craken channel messagescraken dm messages 조회 명령이 추가되어 전체 흐름(조회→대기→전송→조회)이 완성됩니다.
쿼리 파라미터 및 도움말 통합
internal/craken/resources.go, internal/craken/command.go
messagePageQuery의 허용 파라미터 목록에 limit을 추가하여 --limit 값이 쿼리 파라미터로 포함되도록 변경하고, channel.messages/dm.messages/dm.list 도움말 옵션 목록에 --limit N 안내를 추가합니다.
쿼리 전달 및 유효성 검사 테스트
internal/craken/command_test.go
도움말 출력에 --limit이 포함됨을 검증하고, --limit 10이 URL 쿼리 limit=10으로 channel messages와 dm messages 엔드포인트 모두에 전달되는지 확인하며, --limit 0 호출 시 서버 유효성 검사 에러("limit must be a positive integer")가 에러 메시지에 보존되는지 검증합니다.

🎯 2 (Simple) | ⏱️ ~12분

✨ 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/message-limit

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.

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.

Add --limit to channel and DM message commands once the API supports it

1 participant