Skip to content

chore: add policy markdown file to docker image#282

Merged
laggu merged 2 commits into
developfrom
FIENMEE-661
Oct 18, 2025
Merged

chore: add policy markdown file to docker image#282
laggu merged 2 commits into
developfrom
FIENMEE-661

Conversation

@laggu
Copy link
Copy Markdown
Member

@laggu laggu commented Oct 16, 2025

policy 관련 파일을 도커 이미지에 추가합니다

@notion-workspace
Copy link
Copy Markdown

@gemini-code-assist
Copy link
Copy Markdown
Contributor

Summary of Changes

Hello @laggu, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!

이 PR은 서버 Docker 이미지에 필요한 정책 관련 파일을 포함시키고, 개발 환경에서 사용되는 Firebase Admin SDK 파일명을 일반적인 이름으로 통일하여 배포 및 관리의 일관성을 높이는 것을 목표로 합니다.

Highlights

  • Docker 이미지에 정책 파일 추가: Dockerfile_SERVER에 public 디렉토리를 /fienmee/apps/server/dist/public 경로로 복사하는 COPY 명령어를 추가하여 정책 관련 파일이 Docker 이미지에 포함되도록 했습니다.
  • Firebase Admin SDK 파일명 변경: docker-compose.yaml에서 Firebase Admin SDK 파일 마운트 경로를 firebase-dev-adminsdk.json에서 firebase-adminsdk.json으로 변경하여 파일명을 표준화했습니다.
Using Gemini Code Assist

The full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips.

Invoking Gemini

You can request assistance from Gemini at any point by creating a comment using either /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands on the current page.

Feature Command Description
Code Review /gemini review Performs a code review for the current pull request in its current state.
Pull Request Summary /gemini summary Provides a summary of the current pull request in its current state.
Comment @gemini-code-assist Responds in comments when explicitly tagged, both in pull request comments and review comments.
Help /gemini help Displays a list of available commands.

Customization

To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a .gemini/ folder in the base of the repository. Detailed instructions can be found here.

Limitations & Feedback

Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here.

You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension.

Footnotes

  1. Review the Privacy Notices, Generative AI Prohibited Use Policy, Terms of Service, and learn how to configure Gemini Code Assist in GitHub here. Gemini can make mistakes, so double check it and use code with caution.

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

안녕하세요. policy 관련 파일을 Docker 이미지에 추가하는 변경 사항을 리뷰했습니다. Dockerfile_SERVERdocker-compose.yaml의 수정으로 PR의 목적이 명확하게 달성된 점이 좋습니다. 추가적으로, 향후 유지보수성과 이미지 최적화를 위해 Dockerfile_SERVER에 대한 작은 개선 아이디어를 코멘트로 남겼으니 확인 부탁드립니다.

Comment thread Dockerfile_SERVER

COPY --from=build /fienmee/apps/server/package.json /fienmee/apps/server/package.json
COPY --from=build /fienmee/apps/server/dist /fienmee/apps/server/dist
COPY --from=build /fienmee/apps/server/public /fienmee/apps/server/dist/public
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

medium

COPY 명령어는 최종 이미지에 새로운 레이어를 추가합니다. 현재 runner 단계에서 dist 디렉토리를 구성하기 위해 여러 COPY 명령(L30, L31, L32)이 사용되고 있습니다. Docker 이미지의 레이어 수를 줄이고 가독성을 높이기 위해, build 단계에서 dist 디렉토리를 완성하는 것을 고려해 보세요.

예를 들어, build 단계에서 pnpm build:server 실행 후에 관련 파일들을 dist로 복사할 수 있습니다:

# In build stage
...
RUN pnpm build:server
RUN cp -r /fienmee/apps/server/public /fienmee/apps/server/dist/
RUN cp /fienmee/apps/server/newrelic.cjs /fienmee/apps/server/dist/

이렇게 하면 runner 단계에서는 dist 디렉토리 전체를 단일 COPY 명령으로 복사할 수 있어 Dockerfile이 더 효율적이고 깔끔해집니다.

@laggu laggu merged commit 4a6af0b into develop Oct 18, 2025
2 checks passed
@laggu laggu deleted the FIENMEE-661 branch October 18, 2025 04:52
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.

2 participants