fix: fix awilix injection in production builds

Author: theosiemensrhodes

Dockerfile

@@ -42,6 +42,7 @@ WORKDIR /app
 # Set production environment
 ENV NODE_ENV=production
 ENV NEXT_TELEMETRY_DISABLED=1
+ENV NODE_OPTIONS=--enable-source-maps
 
 # Create a non-root user
 RUN addgroup --system --gid 1001 nodejs
@@ -51,6 +52,7 @@ 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/.next/server ./.next/server
 
 # Set correct permissions
 RUN chown -R nextjs:nodejs /app

docker-compose.yaml

@@ -11,6 +11,7 @@ services:
       REDIS_URL: "redis://:${REDIS_PASSWORD}@host.docker.internal:6379/0"
       MINIO_HOST: "host.docker.internal"
       FILES_BASE_URL: "http://host.docker.internal:9000"
+      NODE_OPTIONS: "--enable-source-maps"
     extra_hosts:
       - "host.docker.internal:host-gateway"
     ports:

gen-pass.sh

@@ -1 +1 @@
-openssl rand -base64 12 | tr '+/' '-_'
+openssl rand -base64 12 | tr '+/' '-_'

next.config.js

@@ -8,11 +8,16 @@ import "./src/env.js";
 const nextConfig = {
   output: "standalone",
 
-  webpack(config) {
+  webpack(config, { isServer, dev }) {
     const fileLoaderRule = config.module.rules.find((rule) =>
       rule.test?.test?.(".svg"),
     );
 
+    // Server-side sourcemaps for production Node stack traces
+    if (!dev) {
+      config.devtool = isServer ? "source-map" : "hidden-source-map";
+    }
+
     // Exclude svg from the default loader
     fileLoaderRule.exclude = /\.svg$/;
 

src/server/api/di-container.ts

@@ -4,6 +4,7 @@ import {
   asClass,
   asValue,
   createContainer,
+  InjectionMode,
   type AwilixContainer,
 } from "awilix";
 import type { Sql } from "postgres";
@@ -58,7 +59,9 @@ export type NeuronContainer = AwilixContainer<NeuronCradle>;
 
 const createRootContainer = (): NeuronContainer => {
   const container = createContainer<NeuronCradle>({
-    injectionMode: "CLASSIC",
+    // PROXY is resilient to build-time minification (CLASSIC breaks when
+    // constructor/function parameter names get mangled).
+    injectionMode: InjectionMode.PROXY,
     strict: true,
   });
 

src/server/db/cache.ts

@@ -8,7 +8,7 @@ export type CacheClient = RedisClientType;
 export function registerCacheClient(container: NeuronContainer) {
   container.register({
     cacheClient: asFunction<CacheClient>(
-      (env: typeof environment): CacheClient => {
+      ({ env }: { env: typeof environment }): CacheClient => {
         const g = globalThis as unknown as { __redis?: CacheClient };
 
         // Reuse the dev client across HMR updates

src/server/services/cacheService.ts

@@ -32,7 +32,7 @@ export class CacheService implements ICacheService {
   private singleObjectMemoryCache?: Map<string, string>; // key -> serialized string
   private groupObjectMemoryCache?: Map<string, Map<string, string>>; // group -> (key -> serialized string)
 
-  constructor(cacheClient: CacheClient) {
+  constructor({ cacheClient }: { cacheClient: CacheClient }) {
     this.cacheClient = cacheClient;
   }
 

src/server/services/emailService.ts

@@ -20,7 +20,7 @@ export class EmailService implements IEmailService {
   private readonly transporter: Transporter;
   private readonly env: typeof environment;
 
-  constructor(env: typeof environment) {
+  constructor({ env }: { env: typeof environment }) {
     this.env = env;
     this.transporter = nodemailer.createTransport({
       host: env.SMTP_HOST,

src/server/services/entity/classService.ts

@@ -62,14 +62,21 @@ export class ClassService implements IClassService {
   private readonly shiftService: IShiftService;
   private readonly imageService: IImageService;
 
-  constructor(
-    db: Drizzle,
-    userService: IUserService,
-    volunteerService: IVolunteerService,
-    termService: ITermService,
-    shiftService: IShiftService,
-    imageService: IImageService,
-  ) {
+  constructor({
+    db,
+    userService,
+    volunteerService,
+    termService,
+    shiftService,
+    imageService,
+  }: {
+    db: Drizzle;
+    userService: IUserService;
+    volunteerService: IVolunteerService;
+    termService: ITermService;
+    shiftService: IShiftService;
+    imageService: IImageService;
+  }) {
     this.db = db;
     this.userService = userService;
     this.volunteerService = volunteerService;

src/server/services/entity/coverageService.ts

@@ -62,11 +62,15 @@ export class CoverageService implements ICoverageService {
   private readonly volunteerService: IVolunteerService;
   private readonly shiftService: IShiftService;
 
-  constructor(
-    db: Drizzle,
-    volunteerService: IVolunteerService,
-    shiftService: IShiftService,
-  ) {
+  constructor({
+    db,
+    volunteerService,
+    shiftService,
+  }: {
+    db: Drizzle;
+    volunteerService: IVolunteerService;
+    shiftService: IShiftService;
+  }) {
     this.db = db;
     this.volunteerService = volunteerService;
     this.shiftService = shiftService;