[10.10.5] 배포

[HeeMang-Lee]

🔎 작업 내용

-- 유지 관리(Chores)
런타임을 Node.js 22로 업데이트했습니다.
Brave 검색 MCP 서버를 공식 패키지로 교체하고 실행 방식을 스크립트로 표준화했습니다.
server-brave-search 명령어를 STDIO 전송 방식으로 실행하도록 보장하고, 가용성/도움말 확인을 추가했습니다.
불필요한 패키지 정리 절차를 조정해 빌드 정리를 유지했습니다.
문서화
설치 및 실행 방식 관련 주석을 최신 권장 사항에 맞게 업데이트했습니다.

---

Summary by CodeRabbit

• New Features
  • 컨테이너가 기본적으로 8080 포트에서 서비스되며, 시작 시 애플리케이션이 자동 실행되도록 설정했습니다.
  • Brave Search MCP 서버 제공 방식을 개선해 실행 일관성과 사용성을 높였습니다.
• Chores
  • 런타임 Node.js를 최신 버전으로 상향하고 패키지 구성을 정리해 보안과 유지보수를 개선했습니다.
  • 불필요한 캐시/의존성 정리로 이미지 경량화를 진행했습니다.
• Refactor
  • 시작 및 동작 확인 절차를 단순화해 안정성과 가시성을 향상했습니다.

[coderabbitai]

Walkthrough

cs25-service/Dockerfile에서 Node.js를 20.x에서 22.x로 변경하고, Brave Search MCP 서버 설치 방식을 기존 패키지/심볼릭 링크에서 @brave/brave-search-mcp-server + 래퍼 스크립트로 전환했습니다. 설치 검증 로직을 단순화하고, EXPOSE 8080 및 Java JAR ENTRYPOINT를 명시했습니다.

Changes

Cohort / File(s)

Change Summary

Dockerfile 런타임 및 MCP 서버 설정 cs25-service/Dockerfile

- Node.js: setup_20.x → setup_22.x - MCP 서버: @modelcontextprotocol/server-brave-search 제거, @brave/brave-search-mcp-server 글로벌 설치로 대체 - 글로벌 설치 경로 동적 해석 후 /usr/local/bin/server-brave-search 래퍼 스크립트 생성(노드로 dist/index.js 실행, --transport stdio) - 검증: which/--help 출력으로 단순화, 오류 시 실패 유지 - 클린업: npm cache/apt autoremove/clean 유지, gnupg purge 제거 - 빌드 아티팩트: 빌더에서 jar 복사 유지 - 런타임: EXPOSE 8080, ENTRYPOINT ["java","-jar","/apps/app.jar"] 추가

Sequence Diagram(s)

sequenceDiagram
  participant Dev as Developer
  participant D as Docker
  participant B as Builder Stage
  participant R as Runtime Stage
  participant App as Java App

  Dev->>D: docker build
  D->>B: Install deps, build JAR
  B-->>R: Copy app.jar
  D->>R: Install Node 22, npm global @brave/brave-search-mcp-server
  R->>R: Create wrapper /usr/local/bin/server-brave-search
  R->>R: Verify which + --help

  Dev->>D: docker run
  D->>R: Start container
  R->>App: ENTRYPOINT java -jar /apps/app.jar

Loading

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Possibly related PRs

• Refactor/402 래퍼스크립트를 만들어 커맨드명 유지하도록 도커파일 수정 #403: 동일한 Dockerfile 변경(Node 20→22, Brave MCP 패키지 교체, 래퍼 스크립트, 검증/클린업) 적용.
• 10차 배포 - 10 #375: 동일 파일에서 Node/npm 및 MCP 서버 설치 변경과 관련(이 PR은 패키지 교체와 래퍼 추가까지 포함).
• chore: MCP 서버 전역설치 및 심볼릭 링크 생성, 빌드타임 확인 명령어 추가 #398: Brave Search MCP 서버 설치 및 런타임 검증 로직을 다루는 유사 변경.

Suggested reviewers

• wannabeing
• jong-0126
• crocusia
• Kimyoonbeom

Poem

(\/)
( ••) 토끼의 일기: 도커가 새 옷 입었지!
/ >🍃 Node는 스물둘, Brave는 래퍼로 춤추네.
포트는 8080, 자바 심장 쿵쾅—
server-brave-search, 길도 잘 냈다네.
오늘의 빌드, 발끝에서 하옵니다.

✨ Finishing Touches

🧪 Generate unit tests

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch dev

---

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

• X
• Mastodon
• Reddit
• LinkedIn

🪧 Tips

Chat

There are 3 ways to chat with CodeRabbit:

• Review comments: Directly reply to a review comment made by CodeRabbit. Example:
  • I pushed a fix in commit <commit_id>, please review it.
  • Open a follow-up GitHub issue for this discussion.
• Files and specific lines of code (under the "Files changed" tab): Tag @coderabbitai in a new review comment at the desired location with your query.
• PR comments: Tag @coderabbitai in a new PR comment to ask questions about the PR branch. For the best results, please provide a very specific query, as very limited context is provided in this mode. Examples:
  • @coderabbitai gather interesting stats about this repository and render them as a table. Additionally, render a pie chart showing the language distribution in the codebase.
  • @coderabbitai read the files in the src/scheduler package and generate a class diagram using mermaid and a README in the markdown format.

Support

Need help? Create a ticket on our support page for assistance with any issues or questions.

CodeRabbit Commands (Invoked using PR/Issue comments)

Type @coderabbitai help to get the list of available commands.

Other keywords and placeholders

• Add @coderabbitai ignore anywhere in the PR description to prevent this PR from being reviewed.
• Add @coderabbitai summary to generate the high-level summary at a specific location in the PR description.
• Add @coderabbitai anywhere in the PR title to generate the title automatically.

CodeRabbit Configuration File (.coderabbit.yaml)

• You can programmatically configure CodeRabbit by adding a .coderabbit.yaml file to the root of your repository.
• Please see the configuration documentation for more information.
• If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json

Status, Documentation and Community

• Visit our Status Page to check the current availability of CodeRabbit.
• Visit our Documentation for detailed information on how to use CodeRabbit.
• Join our Discord Community to get help, request features, and share feedback.
• Follow us on X/Twitter for updates and announcements.

[coderabbitai]

Actionable comments posted: 1

🧹 Nitpick comments (5)

cs25-service/Dockerfile (5)

27-29: bash 설치 불필요 — 의존성 최소화로 이미지 슬림화 제안

래퍼 스크립트가 bash에 의존하지 않도록 하면 bash 설치를 생략해 이미지 크기를 줄일 수 있습니다.

아래처럼 bash 제거를 권장합니다.

- && apt-get install -y --no-install-recommends curl ca-certificates gnupg bash \
+ && apt-get install -y --no-install-recommends curl ca-certificates gnupg \

---

28-29: NodeSource 파이프 설치 스크립트 사용 — 보안/재현성 검토 권장

curl | bash 방식과 최신 nodejs 패키지 설치는 재현성에 취약합니다(빌드 시점마다 버전이 달라질 수 있음). 배포 안정성을 위해 버전 핀(PIN) 또는 서명 검증을 고려해주세요.

• apt로 nodejs 버전을 핀(예: nodejs=22.x.y-*)하고 apt-mark hold 적용
• 또는 공식 tarball + 체크섬 검증으로 설치(더 엄격한 보안 요구 시)
• RUN 스텝 시작에 set -eux 추가해 조기 실패 유도

---

31-33: @brave/brave-search-mcp-server 전역 설치 버전 핀 필요

전역 설치가 latest로 고정되지 않아 재현성이 떨어집니다. 특정 버전으로 핀하고 주기적으로 갱신하는 편이 안전합니다.

예시(빌드 ARG로 버전 관리):

- && npm install -g @brave/brave-search-mcp-server \
+ && npm install -g @brave/brave-search-mcp-server@${BRAVE_MCP_VERSION} \

파일 바깥 변경(참고):

ARG BRAVE_MCP_VERSION=1.0.0

최신 안정 버전 확인이 필요하면 알려주세요. 조회 스크립트/웹검색으로 확인해 드리겠습니다.

---

43-45: 헬스체크 명령 개선: --help 대신 --version 사용

래퍼가 항상 --transport stdio를 주입하므로 일부 CLI가 도움말 파싱 실패 시 비정상 종료할 수 있습니다. 설치 검증은 --version이 더 안전합니다.

- && echo "=== server-brave-search --help ===" && server-brave-search --help || (echo "[ERROR] server-brave-search 실행 실패" && exit 1) \
+ && echo "=== server-brave-search --version ===" && server-brave-search --version >/dev/null || (echo "[ERROR] server-brave-search 실행 실패" && exit 1) \

---

47-50: 정리 단계 보완: 불필요 캐시/패키지 추가 제거로 이미지 축소

이미지 크기 축소를 위해 npm/apt 캐시 외에도 다음 제거를 추천합니다.

• gnupg는 설치/키 추가 이후 상시 필요하지 않으면 purge
• root npm 캐시 디렉토리 제거

  && npm cache clean --force \
- && apt-get autoremove -y --purge \
+ && apt-get autoremove -y --purge \
+ && apt-get purge -y gnupg \
  && apt-get clean \
+ && rm -rf /root/.npm /tmp/* \
  && rm -rf /var/lib/apt/lists/*

📜 Review details

Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 9ebb3cc and 8531558.

📒 Files selected for processing (1)

• cs25-service/Dockerfile (2 hunks)

🔇 Additional comments (1)

cs25-service/Dockerfile (1)

56-60: 런타임 포트/엔트리포인트 명시 — LGTM

EXPOSE 8080 및 Java 엔트리포인트 명시는 명확하고 일관적입니다.

[coderabbitai]

🛠️ Refactor suggestion

래퍼가 dist/index.js에 직접 의존 — 패키지 bin 엔트리 사용으로 결합도 낮추기

패키지 내부 레이아웃(dist/index.js)이 변경되면 즉시 깨집니다. npm이 제공하는 bin 엔트리(글로벌 .bin 또는 prefix/bin 경로)를 사용하면 내구성이 훨씬 좋아집니다. 또한 sh로 충분하므로 bash 의존 제거 가능.

아래처럼 bin 경로를 사용하고 쉘을 sh로 바꾸는 걸 권장합니다.

  # 전역 모듈 경로 계산
  && NPM_PREFIX="$(npm prefix -g)" \
- && SRCDIR="${NPM_PREFIX}/lib/node_modules/@brave/brave-search-mcp-server" \
+ && SRCDIR="${NPM_PREFIX}/lib/node_modules/@brave/brave-search-mcp-server" \
+ && BIN_DIR="${NPM_PREFIX}/bin" \
  \
  # 실행 래퍼 스크립트 생성: server-brave-search (STDIO 고정)
- && printf '#!/usr/bin/env bash\nexec node "%s/dist/index.js" --transport stdio "$@"\n' "$SRCDIR" > /usr/local/bin/server-brave-search \
+ && printf '#!/usr/bin/env sh\nexec "%s/server-brave-search" --transport stdio "$@"\n' "$BIN_DIR" > /usr/local/bin/server-brave-search \
  && chmod +x /usr/local/bin/server-brave-search \

이렇게 하면 패키지의 공식 bin 엔트리를 통해 실행하므로 내부 파일 구조 변경에도 안전합니다. 동시에 bash 설치를 제거할 수 있습니다(위 코멘트 참조).

🤖 Prompt for AI Agents

In cs25-service/Dockerfile around lines 35 to 41, the wrapper script hard-codes
dist/index.js and depends on bash; change it to use the package's published bin
entry from the global prefix bin directory (e.g. "$(npm prefix
-g)/bin/server-brave-search" or "$(npm bin -g)/server-brave-search") so the
wrapper invokes the package's official executable instead of an internal path,
and make the script use /bin/sh in the shebang to remove the bash dependency;
keep creating the wrapper and chmod +x but point to the bin path and use sh.