Skip to content

docs. Remote Config 필수 키 안내 추가#20

Open
swift-man wants to merge 1 commit into
mainfrom
docs/remote-config-keys
Open

docs. Remote Config 필수 키 안내 추가#20
swift-man wants to merge 1 commit into
mainfrom
docs/remote-config-keys

Conversation

@swift-man

Copy link
Copy Markdown
Owner

변경 사항

  • Firebase Remote Config에 설정해야 할 키를 강제 업데이트, 차단 버전, 선택 업데이트, 공지 기능별 표로 추가했습니다.
  • 강제 업데이트와 차단 버전이 forceUpdateAlertDoneLinkURLKey를 함께 사용한다는 점을 명시했습니다.
  • 공개 API와 패키지 동작은 변경하지 않았습니다.

검증

  • swift test: Swift Testing 12개 테스트 통과
  • ./GeneratingDocumentationSite: DocC 정적 사이트 생성 성공
  • git diff --check: 통과

강제 업데이트, 차단 버전, 선택 업데이트, 공지에 필요한 Firebase Remote Config 키를 기능별 표로 정리했습니다. 강제 업데이트와 차단 버전이 동일한 이동 URL 키를 공유한다는 점도 명시했습니다.

@gemini-pr-review-bot gemini-pr-review-bot Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Firebase Remote Config 필수 키에 대한 표 및 세부 설명이 README.md에 추가되어 사용자의 연동 편의성이 대폭 향상되었습니다. 커스텀 키를 의존성에 등록하는 예시 코드에 DependencyValues 확장 부분이 생략되어 있는데, 이를 추가해주면 더욱 완성도 높은 문서가 될 것입니다.

좋은 점

  • Firebase Remote Config 연동 시 각 기능에 매핑되어야 하는 키 값들을 표 형태로 명확히 구조화하고, 강제 업데이트와 차단 버전이 동일한 URL 키를 사용한다는 동작 방식을 구체적으로 기술한 점이 훌륭합니다.

(주: 다음 1개 코멘트는 PR diff 범위 밖이라 본문에 모았습니다.)

드롭된 라인 지적

  • README.md:165 — [Suggestion] 커스텀 DependencyKey 구현 예제 코드에서 DependencyValues에 프로퍼티를 등록하는 확장(extension DependencyValues) 코드도 함께 안내되면 사용자가 더 편리하게 연동할 수 있습니다.

리뷰 생성 모델: Gemini 3.5 Flash (High) #3

@claudecode-pr-review-bot

Copy link
Copy Markdown

⚠️ Claude Review — 리뷰 엔진 실패

이 PR 은 자동 리뷰를 완료하지 못했습니다 (full 모드 실패 → diff-only 모드 재시도까지 실패).

  • 마지막 시도 모드: diff
  • 컨텍스트 파일 수: 1
    모델 한도 해제 정보
  • claude-opus-4-8: 2:20am (Asia/Seoul)
  • Claude Opus 4.6 (Thinking) #1: 132h24m53s
  • Claude Opus 4.6 (Thinking) #2: 25h6m42s
  • Claude Opus 4.6 (Thinking) #3: 57h40m41s

조치 제안

  1. CLAUDE_MAX_INPUT_TOKENS 를 모델 실제 윈도우보다 작게 조정 (예: 150000) → 큰 PR 은 자동 diff 모드로 떨어집니다.
  2. 더 큰 컨텍스트 윈도우의 모델로 CLAUDE_MODEL 변경.
  3. 서버 로그(stderr 전체) 를 확인해 모델/CLI 측 메시지 검증.

@codex-review-bot codex-review-bot Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

기능별 최소 활성 키 조합과 공용 URL 키 설명이 실제 의존성 동작과 일치합니다. 문서 변경 범위에서 부정확한 안내나 병합을 막을 문제는 확인되지 않았습니다.

좋은 점

  • 기능별 필수 키를 표로 구분해 설정 누락 여부를 빠르게 확인할 수 있게 한 점
  • 강제 업데이트와 차단 버전이 동일한 이동 URL 키를 공유한다는 동작을 명시한 점

리뷰 모델: gpt-5.6-sol

@mlx-review-bot mlx-review-bot Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

MLX 리뷰

이 PR은 README.md에 Firebase Remote Config의 기본 키 설정에 대한 문서를 추가하여, 앱 개발자가 원격 설정을 올바르게 구성할 수 있도록 안내합니다. 변경된 내용은 문서의 명확성과 사용자 가이드의 완전성을 높입니다.

라인 단위 코멘트

  • 라인 단위로 남길 개선 사항은 발견되지 않았습니다.

사용된 모델: mlx-community/Qwen3-30B-A3B-Instruct-2507-8bit

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