TypeScript Dockerfiles (filesystem, memory, sequentialthinking) ignore the workspace lockfile and overrides
维护者通常 1 天内回复
还没有人认领这个 Issue。
评估
- 难度
- 2/5
- 预计耗时
- 1-3 小时
- 新手友好度
- 78/100
- Issue 类型
- 缺陷
- 描述清晰度
- 描述清楚
- 活跃度
- 活跃
- 技术栈
- docker, node.js, typescript
- 领域
- build-system, devops
调研方向
比较 src/everything/Dockerfile 及其 Docker 忽略文件与 filesystem、memory 和 sequentialthinking 的 Dockerfile。首先检查现有镜像如何使用根目录锁定文件、workspace 安装和非 root 用户，然后将相同的方法应用到指定的三个镜像。从仓库根目录构建每个镜像，并确认安装过程使用锁定文件、正确省略开发依赖项，且以非 root 用户运行。
由索引模型根据 Issue 内容生成。
描述
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
overridesare ignored. Each builder stage copiessrc/<server>alone and runs a lockfile-lessnpm install, which resolves every dependency fresh on the day of the build. The rootpackage-lock.jsonand the rootpackage.jsonoverrides(the pins for transitive advisories:qs,hono,proxy-addr,ip-address, …) never reach the image. Example from theeverythingimage built the same way: it installed[email protected], where the lockfile has6.16.0. --omit-devis not an npm flag.npm ci --ignore-scripts --omit-devprintsnpm warn Unknown cli config "--omit-dev"; dev dependencies are left out only becauseNODE_ENV=productionis 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.
- 主要语言
- TypeScript
- 星标
- 91.6k
- 派生
- 11.9k
- 平均合并
- 4 小时 31 分钟
- 30 天内合并 PR
- 236
环境准备
- 没有 Dockerfile 或 Docker Compose 文件
- 有 Pull Request 模板
- 阅读贡献指南
从这里开始
- 先读完整个 Issue，再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库，在一个分支上完成修改。
- 提交 Pull Request，并在描述里引用这个 Issue 编号。
modelcontextprotocol/servers 的其他 Issue
-
enhancement server-everything v2
难度 2/5 1-3 小时 新手友好度 72/100
modelcontextprotocol/servers#5371 · 1 条评论 ·
维护者通常 1 天内回复
-
docs(git): clarify unconfined default scope and repository confinement options in README and CLI未关闭documentation server-git v2
难度 1/5 1 小时以内 新手友好度 74/100
modelcontextprotocol/servers#5230 · 1 条评论 ·
维护者通常 1 天内回复
-
bug server-filesystem v2
难度 2/5 1-3 小时 新手友好度 72/100
modelcontextprotocol/servers#5216 · 3 条评论 ·
维护者通常 1 天内回复
-
documentation server-git server-time v2
难度 2/5 1-3 小时 新手友好度 72/100
modelcontextprotocol/servers#5213 · 2 条评论 ·
维护者通常 1 天内回复
-
documentation v2
难度 1/5 1-3 小时 新手友好度 78/100
modelcontextprotocol/servers#5212 · 1 条评论 ·
维护者通常 1 天内回复
查看 modelcontextprotocol/servers 的全部 Issue
相似的 Issue
-
area:desktop area:ui bug platform:macos
难度 2/5 1-3 小时 新手友好度 67/100
anthropics/claude-code#106183 ·
维护者通常 1 天内回复
-
难度 1/5 1-3 小时 新手友好度 76/100
维护者通常 1 天内回复
-
feature:App-Tour feature:Chat ready Web
难度 2/5 1-3 小时 新手友好度 62/100
digitalfabrik/integreat-app#4523 · 2 个 reaction ·
维护者通常 3 天内回复
-
难度 2/5 1-3 小时 新手友好度 78/100
维护者通常 1 天内回复
-
Required Lexical field accepts two empty paragraphs可能已有人在做 @lazerg 今天认领。 未关闭Bug
难度 2/5 1-3 小时 新手友好度 76/100
payloadcms/payload#19222 ·
维护者通常 1 天内回复