Skip to content

feat: add Dockerfile for self-hosted deployment #36

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
felipeelias wants to merge 2 commits into
base: main
Choose a base branch
from
Open

feat: add Dockerfile for self-hosted deployment #36

Changes from 1 commit
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
9842698
feat: add Dockerfile for self-hosted deployment
felipeelias Mar 29, 2026
7940880
fix: improve comment wording and pin tsx version
felipeelias Mar 29, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
66 changes: 66 additions & 0 deletions Dockerfile
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,66 @@
# ==============================================================
# Stage 1: Install dependencies and build frontend
# ==============================================================
FROM node:22-slim AS builder

RUN apt-get update && apt-get install -y --no-install-recommends \
python3 make g++ \
&& rm -rf /var/lib/apt/lists/*

WORKDIR /app

# Copy workspace package.json files first for layer caching
COPY package.json ./
COPY packages/agent-bridge/package.json ./packages/agent-bridge/
COPY packages/doc-core/package.json ./packages/doc-core/
COPY packages/doc-editor/package.json ./packages/doc-editor/
COPY packages/doc-server/package.json ./packages/doc-server/
COPY packages/doc-store-sqlite/package.json ./packages/doc-store-sqlite/
COPY apps/proof-example/package.json ./apps/proof-example/

RUN npm install

COPY . .

# Build frontend SPA (Vite → dist/), then merge into public/ so the
# Express static middleware can serve the built assets alongside existing
# static files, then prune dev dependencies.
# Single RUN so dev deps don't persist in any layer.
Comment thread
felipeelias marked this conversation as resolved.
Outdated
RUN npm run build \
&& cp -r dist/* public/ \
Comment on lines +25 to +31

Copilot AI Mar 29, 2026

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

cp -r dist/* public/ will copy dist/index.html into public/, and because express.static(.../public) is registered before app.get('/') (see server/index.ts), requests to / will start serving the SPA index.html instead of the server’s landing page route. To avoid this behavior change, only copy the built asset directory into public (e.g. dist/assets -> public/assets) while keeping dist/index.html in dist/ for share-web-routes to read.

Suggested change
# Build frontend SPA (Vite → dist/), then merge into public/ so the
# Express static middleware can serve the built assets alongside existing
# static files, then prune dev dependencies.
# Single RUN so dev deps don't persist in any layer.
RUN npm run build \
&& cp -r dist/* public/ \
# Build frontend SPA (Vite → dist/), then copy only built assets into
# public/ so Express can serve them, while keeping dist/index.html in dist/
# for the server routes to read, then prune dev dependencies.
# Single RUN so dev deps don't persist in any layer.
RUN npm run build \
&& mkdir -p public/assets \
&& cp -r dist/assets/* public/assets/ \

Copilot uses AI. Check for mistakes.

@felipeelias felipeelias Mar 29, 2026

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The copy is intentional. Serving the SPA at / via public/index.html is the expected behavior. share-web-routes reads dist/index.html separately for document pages (/d/... routes), not the root. Both paths work correctly with this setup.

&& npm prune --omit=dev

# ==============================================================
# Stage 2: Production runtime
# ==============================================================
FROM node:22-slim

RUN apt-get update && apt-get install -y --no-install-recommends \
dumb-init \
&& rm -rf /var/lib/apt/lists/*

RUN npm install -g tsx@4 && npm cache clean --force
Comment thread
felipeelias marked this conversation as resolved.
Outdated

ENV NODE_ENV=production

WORKDIR /app
RUN chown node:node /app

# Server source and shared modules it imports at runtime
COPY --from=builder --chown=node:node /app/server ./server
COPY --from=builder --chown=node:node /app/src ./src
COPY --from=builder --chown=node:node /app/node_modules ./node_modules
COPY --from=builder --chown=node:node /app/public ./public
COPY --from=builder --chown=node:node /app/dist ./dist
COPY --from=builder --chown=node:node /app/docs/agent-docs.md ./docs/
COPY --from=builder --chown=node:node /app/AGENT_CONTRACT.md ./
COPY --from=builder --chown=node:node /app/package.json ./

RUN mkdir -p /data && chown node:node /data

USER node

EXPOSE 4000

ENTRYPOINT ["/usr/bin/dumb-init", "--"]
CMD ["tsx", "server/index.ts"]
Loading