256b8347e7
This is a fix to pnpm deployment which needs testing as the final part of origin/master release No functional changes * fix(docs): remove stray tool-call tags breaking the MDX build platform/docs/docs/migration-guide/3p12-to-3p13/build-tooling.md ended with two orphan closing tags (leftover tool-call serialization artifacts): </content> </invoke> Docusaurus compiles Markdown as MDX (JSX-aware), so the orphan closing tag failed the docs build: MDX compilation failed ... Unexpected closing slash in tag, expected an open tag first (build-tooling.md line 402) This was the remaining blocker for build-and-deploy-docs once the --no-frozen-lockfile change let the install step succeed. A scan of the docs tree found no other such artifacts. Verified locally: docusaurus build now generates static files with no MDX errors. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> * Update lockfile and avoid freshness check on every command * fix(release): keep workspace:* in the repo, concretize only at publish The release flow rewrote internal @ohif/* dependency specifiers to the concrete version and committed them, so pnpm-lock.yaml (which records workspace links) drifted from the manifests on every version bump. The resulting ERR_PNPM_OUTDATED_LOCKFILE broke every frozen install: Netlify (viewer-dev), the docs deploy, pnpm's pre-run deps check, and post-merge installs. Keep workspace:* everywhere in the committed repo and move the concrete-version substitution to publish time only: - publish-version.mjs: bump each package's own `version` field only; stop rewriting @ohif/* dependency/peerDependency specifiers. - publish-package.mjs: publish with `pnpm publish --no-git-checks` instead of `npm publish`. pnpm rewrites workspace:* to the exact version in the published tarball; npm would publish the literal "workspace:*", which npm/yarn consumers cannot resolve. - One-time: revert the 25 workspace manifests' @ohif/* specifiers to workspace:* (version fields untouched) and regenerate pnpm-lock.yaml to match. Because internal deps are workspace:* (links, not versions in the lockfile), version bumps no longer change pnpm-lock.yaml, so it stays in sync and frozen installs keep working. Verified: `pnpm install --frozen-lockfile` passes, and `pnpm pack` of @ohif/core emits a tarball whose @ohif/ui dependency is the exact version (3.13.0-beta.92), not workspace:*. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> * docs(ci): correct build-docs install comment for workspace:* release flow publish-version.mjs no longer rewrites @ohif/* deps to concrete versions, so the old comment was stale. Internal deps stay workspace:* and the lockfile stays consistent; pnpm publish concretizes only the published tarball. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> * ci(docs): use --frozen-lockfile now that the lockfile no longer drifts With internal deps as workspace:* the lockfile stays in sync across version bumps, so the docs deploy can install frozen -- failing fast on genuine lockfile drift instead of silently reconciling. The --no-frozen-lockfile workaround is no longer needed. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> * ci: use --frozen-lockfile in CI install steps now that the lockfile is stable Internal @ohif/* deps are workspace:* so pnpm-lock.yaml no longer drifts; the UNIT_TESTS/BUILD/NPM_PUBLISH installs can run frozen and fail fast on genuine drift. Kept --no-frozen-lockfile only where it is still required: the Dockerfile (platform/docs is excluded from the build context) and the playwright CS3D-version step (mutates @cornerstonejs versions before installing). Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> * test: stability of seg load mpr test * Better drag fix for crosshairs --------- Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
90 lines
3.3 KiB
Docker
90 lines
3.3 KiB
Docker
# syntax=docker/dockerfile:1.7-labs
|
|
# This dockerfile is used to publish the `ohif/app` image on dockerhub.
|
|
#
|
|
# It's a good example of how to build our static application and package it
|
|
# with a web server capable of hosting it as static content.
|
|
#
|
|
# docker build
|
|
# --------------
|
|
# If you would like to use this dockerfile to build and tag an image, make sure
|
|
# you set the context to the project's root directory:
|
|
# https://docs.docker.com/engine/reference/commandline/build/
|
|
#
|
|
#
|
|
# SUMMARY
|
|
# --------------
|
|
# This dockerfile has two stages:
|
|
#
|
|
# 1. Building the React application for production
|
|
# 2. Setting up our Nginx (Alpine Linux) image w/ step one's output
|
|
#
|
|
|
|
|
|
# Stage 1: Build the application
|
|
# docker build -t ohif/viewer:latest .
|
|
# Copy Files
|
|
FROM node:24-slim as builder
|
|
|
|
RUN apt-get update && apt-get install -y --no-install-recommends build-essential python3 \
|
|
&& rm -rf /var/lib/apt/lists/*
|
|
|
|
RUN npm install -g pnpm@11
|
|
|
|
RUN mkdir /usr/src/app
|
|
WORKDIR /usr/src/app
|
|
ENV PATH=/usr/src/app/node_modules/.bin:$PATH
|
|
|
|
# Copy package manifests for install caching. preinstall.js is included because
|
|
# the root package.json's "preinstall" lifecycle script (node preinstall.js)
|
|
# runs during `pnpm install` below -- before the full source is copied -- so the
|
|
# script file must already be present or install fails with MODULE_NOT_FOUND.
|
|
COPY package.json pnpm-lock.yaml pnpm-workspace.yaml .npmrc preinstall.js ./
|
|
COPY --parents ./extensions/*/package.json ./modes/*/package.json ./platform/*/package.json ./
|
|
# Run the install before copying the rest of the files.
|
|
# Keep --no-frozen-lockfile here (unlike CI): .dockerignore excludes
|
|
# platform/docs, so the lockfile's docs importer has no manifest in the build
|
|
# context and a frozen install would fail. pnpm reconciles (drops docs) instead.
|
|
RUN pnpm install --no-frozen-lockfile
|
|
# Copy the local directory
|
|
COPY --link --exclude=pnpm-lock.yaml --exclude=package.json --exclude=Dockerfile . .
|
|
|
|
# Build here
|
|
# After install it should hopefully be stable until the local directory changes
|
|
ENV QUICK_BUILD true
|
|
# ENV GENERATE_SOURCEMAP=false
|
|
ARG APP_CONFIG=config/default.js
|
|
ARG PUBLIC_URL=/
|
|
ENV PUBLIC_URL=${PUBLIC_URL}
|
|
|
|
RUN pnpm run show:config
|
|
RUN pnpm run build
|
|
|
|
# Precompress files
|
|
RUN chmod u+x .docker/compressDist.sh
|
|
RUN ./.docker/compressDist.sh
|
|
|
|
# Stage 2: Bundle the built application into a Docker container
|
|
# which runs Nginx using Alpine Linux
|
|
FROM nginxinc/nginx-unprivileged:1.27-alpine as final
|
|
#RUN apk add --no-cache bash
|
|
ARG PUBLIC_URL=/
|
|
ENV PUBLIC_URL=${PUBLIC_URL}
|
|
ARG PORT=80
|
|
ENV PORT=${PORT}
|
|
RUN rm /etc/nginx/conf.d/default.conf
|
|
USER nginx
|
|
COPY --chown=nginx:nginx .docker/Viewer-v3.x /usr/src
|
|
RUN chmod 777 /usr/src/entrypoint.sh
|
|
COPY --from=builder /usr/src/app/platform/app/dist /usr/share/nginx/html${PUBLIC_URL}
|
|
# Copy paths that are renamed/redirected generally
|
|
# Microscopy libraries depend on root level include, so must be copied
|
|
COPY --from=builder /usr/src/app/platform/app/dist/dicom-microscopy-viewer /usr/share/nginx/html/dicom-microscopy-viewer
|
|
|
|
# In entrypoint.sh, app-config.js might be overwritten, so chmod it to be writeable.
|
|
# The nginx user cannot chmod it, so change to root.
|
|
USER root
|
|
RUN chown -R nginx:nginx /usr/share/nginx/html && chmod -R 777 /usr/share/nginx/html
|
|
USER nginx
|
|
ENTRYPOINT ["/usr/src/entrypoint.sh"]
|
|
CMD ["nginx", "-g", "daemon off;"]
|