From 173ee9c053ea8f5447dc4ffb0195b1ddbf327b9c Mon Sep 17 00:00:00 2001 From: Shanu Date: Sun, 23 Aug 2026 22:28:05 +0530 Subject: [PATCH] Pin an npm that can resolve the AgentMemory image MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit `AgentMemory E2E` has been red on every branch since this afternoon, including `main`, where the commit it fails on touches no JavaScript. It went green at 10:47 and red at 15:39 with nothing between. The image ships npm 10.9.8, whose arborist dereferences null on this package.json: npm error Cannot read properties of null (reading 'edgesOut') It is the npm version, not the dependencies. npm 11.12.1 resolves the same file — copied out of the pinned tag — to 276 packages and exits 0. So the build installs an npm that works before using it. Pinned exactly rather than floated, because the reason this broke at all is that the build has no lockfile and resolves against a registry that moves underneath it; a floating `npm@11` would leave half of that non-determinism in place. The upstream tag ships no lockfile, so `npm ci` is not available here. That is the real fix and it is not ours to make. --- integration/remote-engines/docker-compose.yml | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/integration/remote-engines/docker-compose.yml b/integration/remote-engines/docker-compose.yml index 2078e3d..7ac6e0c 100644 --- a/integration/remote-engines/docker-compose.yml +++ b/integration/remote-engines/docker-compose.yml @@ -9,7 +9,14 @@ services: FROM node:22-bookworm-slim WORKDIR /app COPY package.json . - RUN npm install + # The image's bundled npm (10.9.8) fails resolving this package.json + # with "Cannot read properties of null (reading 'edgesOut')" — an + # arborist bug, not a bad dependency: npm 11.12.1 resolves the exact + # same file to 276 packages. It surfaced without any change here, on a + # day when this job went from green at 10:47 to red at 15:39, because + # the build has no lockfile and resolves against a registry that moves. + # Pinned rather than floated for the same reason. + RUN npm install -g npm@11.12.1 && npm install COPY . . RUN npm run build CMD ["node", "dist/index.mjs"]