Hacktoberfest 2026: những issue maintainer đã đánh dấu cho tháng Mười, đang mở và phù hợp người mới. Xem issue Hacktoberfest

TypeScript Dockerfiles (filesystem, memory, sequentialthinking) ignore the workspace lockfile and overrides

Đang mở Phù hợp với người mới
  • Có gì không ổn?
#5,345 0 bình luận 0 reaction 0 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 1 ngày

Chưa có ai nhận issue này.

Đánh giá

Độ khó
2/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
Sôi nổi
Công nghệ
docker, node.js, typescript
Lĩnh vực
build-system, devops

Hướng nghiên cứu

So sánh src/everything/Dockerfile và tệp loại trừ Docker của nó với các Dockerfile của filesystem, memory và sequentialthinking. Trước tiên, kiểm tra cách image hiện có sử dụng tệp khóa ở thư mục gốc, cài đặt workspace và người dùng không phải root, sau đó áp dụng cách làm tương tự cho ba image được nêu. Build từng image từ thư mục gốc của repository và xác nhận rằng quá trình cài đặt sử dụng tệp khóa, loại bỏ đúng các dependency phát triển và chạy dưới quyền người dùng không phải root.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

chore server-filesystem server-memory server-sequentialthinking v2

Problem

The Dockerfiles for filesystem, memory and sequentialthinking build (since #5147 moved them to node:24-alpine, see #4782), but they do not install what this repository tested:

  • The workspace lockfile and overrides are ignored. Each builder stage copies src/<server> alone and runs a lockfile-less npm install, which resolves every dependency fresh on the day of the build. The root package-lock.json and the root package.json overrides (the pins for transitive advisories: qs, hono, proxy-addr, ip-address, …) never reach the image. Example from the everything image built the same way: it installed [email protected], where the lockfile has 6.16.0.
  • --omit-dev is not an npm flag. npm ci --ignore-scripts --omit-dev prints npm warn Unknown cli config "--omit-dev"; dev dependencies are left out only because NODE_ENV=production is set. The flag is --omit=dev.
  • They run as root (the TypeScript half of #4741, which covers the Python images).

Reproduce

docker build -f src/filesystem/Dockerfile -t fs-check . 2>&1 | grep -E 'omit-dev|added'

Expected

Each image installs from the root lockfile with npm ci --workspace src/<server>, building from the repository root as context, with a src/<server>/Dockerfile.dockerignore allow-list. #3309's PR does this for everything (src/everything/Dockerfile); the same shape applies to the other three. Out of scope for #3309, which concerns everything only.

Ngôn ngữ chính
TypeScript
Star
91.6k
Fork
11.9k
Merge trung bình
4 giờ 38 phút
Pull request đã merge (30 ngày)
241

Chuẩn bị môi trường

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. 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.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của modelcontextprotocol/servers

Tất cả issue của modelcontextprotocol/servers

Issue tương tự

Thêm issue về TypeScript

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.