1 Commits

Include renames

Author	SHA1	Message	Date
Zack PollardandClaude Opus 5	10e376848a	chore(survey): update dependencies (#718) ... * fix(survey): import Migrator from kysely/migration kysely 0.29 moved Migrator, Migration and MigrationProvider out of the package root and into the kysely/migration subpath export, so the self-hosted Node entrypoint died at startup with "Migrator is not a constructor" and served every request against an unmigrated database. packages/sql-tools already imports from the subpath; this brings the survey backend in line. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_014aAfftndexMEN2j2os6yEv * chore(deps): update survey.immich.app dependencies to latest Brings every dependency of the survey app and its backend up to the newest release published on npm, and aligns the app with the TypeScript setup the rest of the monorepo already uses. Majors: - vite 7 -> 8 with @sveltejs/vite-plugin-svelte 6 -> 7 (the plugin's peer range requires vite 8 and svelte >= 5.46.4); every other app in the workspace was already on this pair - typescript ^5.9.2 -> the workspace convention of typescript -> npm:@typescript/typescript6 plus @typescript/native -> npm:typescript@7. @sveltejs/kit, svelte-check and typescript-eslint all cap their typescript peer below 7, so TS 6 stays the tooling compiler and TS 7 native is installed alongside it - @types/node 24 -> 26 - backend: @hono/node-server 1 -> 2 (drops Node 18, removes the Vercel adapter; neither is used here), better-sqlite3 12 -> 13 (Node-API rewrite, same JS API) and @types/better-sqlite3 7 -> 9 Everything else is a minor or patch bump: @sveltejs/kit 2.70.3, svelte 5.56.9, vitest 4.1.11, eslint 10.8.1, tailwindcss 4.3.3, wrangler 4.124.0, hono 4.13.3, playwright 1.62.1, and the rest. @cloudflare/workers-types and @sveltejs/kit were published inside pnpm's minimumReleaseAge window, so they need an explicit exemption for `pnpm install --frozen-lockfile` to accept the lockfile. The entries can be dropped once those versions age out. Only the two survey importers changed in the lockfile; the other 15 workspace projects resolve identically. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_014aAfftndexMEN2j2os6yEv * chore(survey): build the Docker image on node 24 better-sqlite3 13 declares engines node >= 22 and the repo sets engine-strict, so `pnpm install` in the image no longer resolves on node:20-slim. node 24 matches the version .mise/config.toml pins for CI and local development. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_014aAfftndexMEN2j2os6yEv * chore(deps): keep survey deps inside pnpm's minimumReleaseAge window @sveltejs/kit 2.70.3 and @cloudflare/workers-types 5.20260819.1 were published within pnpm's minimum release age, so accepting them required a minimumReleaseAgeExclude entry for `pnpm install --frozen-lockfile` to pass. Step both back one release instead and drop the exemption, leaving pnpm-workspace.yaml untouched. 2.70.2 is also what the other apps in the workspace resolve to. Also declares kysely and @immich/kysely-adapter-cloudflare on the backend. Both are imported by backend/src but were only ever resolved through the frontend's node_modules, so any install that did not also include the frontend could not resolve them. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_014aAfftndexMEN2j2os6yEv * fix(survey): repair the Docker build The image had not been buildable for a while. Each of these failed the build on its own: - `--filter survey...` matches no project; the app is survey.immich.app, so the frontend stage installed nothing and the build that followed had no node_modules - the manifest for futo-backups-survey was still copied in, but that app was removed in 7d9d2d3, so COPY aborted on a missing path. Neither it nor the datasets manifest is needed: pnpm --filter installs fine without unrelated workspace manifests, so both are gone along with the comment that justified them - the backend stage never received apps/survey.immich.app/shared, which backend/src imports, so esbuild could not resolve it - the esbuild bundle is ESM but inlines CommonJS dependencies, and esbuild's ESM output replaces require with a stub that throws. ws hit it on `require("events")` and the server died on startup. build:node now restores a real require via createRequire, and the Dockerfile calls that script instead of repeating the flags Adds a .dockerignore so COPY stops dragging the host's pnpm symlink farm and stale build output into the context, and marks the backend package as ESM, which is what it already is and what its bundle needs to avoid being reparsed at every start. Verified by replaying every stage's COPY and RUN outside Docker: both builders complete, and the resulting payload boots, applies migrations, serves the built frontend and records a survey response through better-sqlite3. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_014aAfftndexMEN2j2os6yEv * fix(lint): pin tsconfigRootDir in both eslint configs The survey app is the only workspace project with its own flat config, and ESLint 10 resolves a config per linted file, so a root `pnpm lint` loads that config alongside this one. typescript-eslint records a candidate tsconfigRootDir for every config file it is loaded from and throws rather than guess once there is more than one, failing 215 files across the survey app, ui.immich.app, bin and packages with "No tsconfigRootDir was set, and multiple candidate TSConfigRootDirs are present". Aliasing the survey app's typescript to @typescript/typescript6 is what surfaced it: the app used to resolve its own typescript-eslint instance against typescript 5.9, so the two configs registered their candidates in separate module state. Now both configs share one instance and see each other's candidate. Pinning tsconfigRootDir explicitly in each config is what the error asks for and keeps each config anchored to its own directory. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_014aAfftndexMEN2j2os6yEv --------- Co-authored-by: Claude <noreply@anthropic.com>	2026-08-19 16:59:51 -04:00