Merge pull request 'fix: Exclude id field from noteData in update route' (#7) from develop into main

Reviewed-on: #7

Dockerfile

@@ -1,9 +1,7 @@
 # Stage 1: Dependencies
-FROM node:20-slim AS deps
+FROM node:20-alpine AS deps
 
-RUN apt-get update && apt-get install -y --no-install-recommends \
-    openssl \
-    && rm -rf /var/lib/apt/lists/*
+RUN apk add --no-cache libc6-compat
 
 WORKDIR /app
 
@@ -12,53 +10,41 @@ COPY package.json package-lock.json* ./
 RUN npm ci
 
 # Stage 2: Build
-FROM node:20-slim AS builder
+FROM node:20-alpine AS builder
 
 WORKDIR /app
 
-RUN apt-get update && apt-get install -y --no-install-recommends \
-    openssl \
-    && rm -rf /var/lib/apt/lists/*
-
 COPY --from=deps /app/node_modules ./node_modules
 COPY . .
 
-ENV DATABASE_URL="file:./dev.db"
-
 RUN npx prisma generate
 
 RUN npm run build
 
 # Stage 3: Production
-FROM node:20-slim AS runner
+FROM node:20-alpine AS runner
 
 WORKDIR /app
 
 ENV NODE_ENV=production
 
-RUN apt-get update && apt-get install -y --no-install-recommends \
-    openssl \
-    && rm -rf /var/lib/apt/lists/* \
-    && apt-get clean && rm -rf /var/lib/apt/lists/*
+RUN addgroup --system --gid 1001 nodejs
+RUN adduser --system --uid 1001 nextjs
 
 COPY --from=builder /app/public ./public
 COPY --from=builder /app/.next/standalone ./
 COPY --from=builder /app/.next/static ./.next/static
 COPY --from=builder /app/prisma ./prisma
 COPY --from=builder /app/node_modules/.prisma ./node_modules/.prisma
-COPY --from=builder /app/node_modules/prisma ./node_modules/prisma
-COPY --from=builder /app/node_modules/@prisma ./node_modules/@prisma
 
 COPY --from=builder /app/prisma/schema.prisma /app/schema.prisma
-COPY docker-entrypoint.sh /usr/local/bin/
-RUN chmod +x /usr/local/bin/docker-entrypoint.sh
+
+USER nextjs
 
 EXPOSE 3000
 
 ENV PORT=3000
 ENV HOSTNAME="0.0.0.0"
-ENV DATABASE_URL="file:/app/data/dev.db"
+ENV DATABASE_URL="file:./dev.db"
 
-ENTRYPOINT ["/usr/local/bin/docker-entrypoint.sh"]
 CMD ["node", "server.js"]
-

Jenkinsfile

@@ -85,13 +85,13 @@ services:
         - "traefik.http.routers.${NAME_SERVICE}.rule=Host(`recall.vodorod.cl`)"
     environment:
       - TZ=America/Santiago
-      - DATABASE_URL=file:/app/data/dev.db
+      - DATABASE_URL=file:./data/dev.db
     volumes:
         - ./data:/app/data
     networks:
       - homelab-net
-    mem_limit: 512m
-    mem_reservation: 256m
+    mem_limit: 32m
+    mem_reservation: 16m
     restart: unless-stopped
 networks:
   homelab-net:

README.md

@@ -248,7 +248,3 @@ src/
 └── types/
     └── note.ts
 ```
-
-
-
-nueva prueba

docker-compose.yml

@@ -8,5 +8,5 @@ services:
     volumes:
       - ./data:/app/data
     environment:
-      - DATABASE_URL=file:/app/data/dev.db
+      - DATABASE_URL=file:./data/dev.db
     restart: unless-stopped

docker-entrypoint.sh

@@ -1,25 +0,0 @@
-#!/bin/bash
-set -e
-
-cd /app
-
-echo "Checking database..."
-
-# Ensure data directory exists with proper permissions
-mkdir -p /app/data
-chmod 777 /app/data
-
-# Run db push (creates/updates database schema)
-# If it fails due to OOM but DB exists, continue anyway
-./node_modules/prisma/build/index.js db push --skip-generate || {
-    exit_code=$?
-    if [ -f /app/data/dev.db ]; then
-        echo "db push failed (code $exit_code) but database exists, continuing..."
-    else
-        echo "db push failed and database does not exist"
-        exit $exit_code
-    fi
-}
-
-echo "Starting application..."
-exec "$@"

src/app/page.tsx

@@ -1,8 +1,6 @@
 import { Dashboard } from '@/components/dashboard'
 import { getDashboardData } from '@/lib/dashboard'
 
-export const dynamic = 'force-dynamic'
-
 export default async function HomePage() {
   const data = await getDashboardData(6)