[Audit] Docker/production build uses `npm install` instead of the CI-tested `npm ci`
Maintainer thường phản hồi trong vòng 7 ngày
Đánh giá
- Độ khó
- 1/5
- Thời gian dự kiến
- 1-3 giờ
- Mức phù hợp với người mới
- 78/100
- Loại issue
- Lỗi
- Độ rõ ràng
- Đặc tả rõ ràng
- Mức độ hoạt động
- Ít trao đổi
- Công nghệ
- docker, node.js, typescript
- Lĩnh vực
- build-system
Hướng nghiên cứu
Bắt đầu với backend/Dockerfile:14 và render.yaml:8, sau đó so sánh các lệnh cài đặt của chúng với các đường dẫn cài đặt CI được đề cập trong issue. Thay đổi cả hai đường dẫn build production để dùng npm ci và chạy bước kiểm tra CI liên quan; hoàn tất khi cả hai đều dùng npm ci và package-lock.json vẫn không thay đổi.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Filed from the second-wave repository audit (issue 54/100). See the audit summary for full category context.
- Location:
backend/Dockerfile:14,render.yaml:8 - Problem: Both the backend's Docker builder stage and the Render.com production build command use
npm install, while every CI install path usesnpm cito pin exact lockfile versions. The image/deployment that actually ships to production can resolve different dependency versions than what CI tested. - Evidence:
RUN npm installinDockerfile:14;npm install && npm run prisma:generate && npm run buildinrender.yaml:8. - Suggested implementation: Change both to
npm cifor deterministic, lockfile-pinned installs matching CI. - Acceptance criteria: Docker build and Render build both use
npm ci; a CI check confirmspackage-lock.jsonisn't modified as a result. - Difficulty: S
- Expected impact: Eliminates a real "works in CI, different in prod" dependency-drift risk for a financial application.
- Ngôn ngữ chính
- TypeScript
- Star
- 18
- Fork
- 250
- Merge trung bình
- 5 ngày 6 giờ
- Pull request đã merge (30 ngày)
- 82
Chuẩn bị môi trường
- Có Dockerfile hoặc tệp Docker Compose
- Có mẫu pull request
- Đọc hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của LabsCrypt/flowfi
-
backend points: 150 testing
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
Maintainer thường phản hồi trong vòng 7 ngày
-
backend points: 150 testing
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
Maintainer thường phản hồi trong vòng 7 ngày
-
backend points: 150 testing
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 76/100
Maintainer thường phản hồi trong vòng 7 ngày
-
backend points: 150 testing
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
Maintainer thường phản hồi trong vòng 7 ngày
-
backend points: 150 testing
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
Maintainer thường phản hồi trong vòng 7 ngày
Tất cả issue của LabsCrypt/flowfi
Issue tương tự
-
bug skills
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 77/100
equinor/fusion-framework#5846 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 74/100
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 85/100
Eyevinn/open-videocore#1256 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Frontend build fails with Frappe version-16: DataImport is no longer exported by @framework/uiĐang mởbug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
Maintainer thường phản hồi trong vòng 1 ngày
-
[Bug]: Theme preset geist points at undefined --font-geistCó thể đã có người làm @claude đã nhận hôm nay. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 73/100
Maintainer thường phản hồi trong vòng 1 ngày