From b0a4b2d9c35dc9ee24d66330431c4becc8b0d1e5 Mon Sep 17 00:00:00 2001 From: saravanakumardb1 Date: Tue, 24 Mar 2026 07:28:44 -0700 Subject: [PATCH] =?UTF-8?q?docs:=20add=20Docker=20build=20verification=20s?= =?UTF-8?q?ection=20(=C2=A717)=20=E2=80=94=20all=204=20images=20pass,=2015?= =?UTF-8?q?91=20tests=20green?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- docs/devops/GITEA_NPM_REGISTRY_MIGRATION.md | 31 +++++++++++++++++++++ 1 file changed, 31 insertions(+) diff --git a/docs/devops/GITEA_NPM_REGISTRY_MIGRATION.md b/docs/devops/GITEA_NPM_REGISTRY_MIGRATION.md index b9d0329c..1239eb7a 100644 --- a/docs/devops/GITEA_NPM_REGISTRY_MIGRATION.md +++ b/docs/devops/GITEA_NPM_REGISTRY_MIGRATION.md @@ -642,3 +642,34 @@ After the initial migration was marked complete, a systematic audit was run acro ### Lesson learned Mobile workspace members (FlowMonk, NomGap, NoteLett) that were previously insulated from the `file:` → registry change because their lockfiles still resolved transitively will break when the lockfile is regenerated. Always run `GITEA_NPM_TOKEN=... pnpm install --no-frozen-lockfile` after converting mobile `package.json` refs. + +--- + +## 17. Docker Build Verification (2026-03-24) + +After completing the migration and audit, all Dockerfiles were tested with actual `docker build` commands using the Gitea registry. + +### Bugs found and fixed during Docker verification + +| # | Finding | Severity | Fix | Repo | +|---|---------|----------|-----|------| +| 6 | MindLyst + LysnrAI Dockerfiles used shell syntax (`2>/dev/null \|\| true`) in COPY instructions | **BUG** | Removed — Docker COPY doesn't support shell redirects | both | +| 7 | MindLyst web Dockerfile used `pnpm run build -- --webpack` — pnpm passes `--webpack` as directory arg | **BUG** | Changed to `npx next build --webpack` | `learning_multimodal_memory_agents` | +| 8 | `@bytelyst/extraction` was missing from Gitea registry | **GAP** | Published (now 49 packages total) | common-plat | + +### Docker build results — all PASS + +| Repo | Image | Status | +|------|-------|--------| +| MindLyst | `mindlyst-backend:test` | ✅ | +| MindLyst | `mindlyst-web:test` | ✅ | +| LysnrAI | `lysnrai-backend:test` | ✅ | +| LysnrAI | `lysnrai-dashboard:test` | ✅ | + +(Standard repos were already Docker-verified during the initial migration.) + +### Full verification sweep + +- **Backend tests**: 1,591 tests passing across all 10 repos +- **Web typecheck**: 9/9 web apps typecheck clean +- **Backend typecheck**: 10/10 repos typecheck clean