@pattern-stack/codegen 0.14.0 → 0.14.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pattern-stack/codegen",
-  "version": "0.14.0",
+  "version": "0.14.1",
   "description": "Entity-driven code generation for full-stack TypeScript applications",
   "license": "MIT",
   "type": "module",

package/runtime/subsystems/bridge/bridge-delivery.memory-backend.ts

@@ -116,12 +116,19 @@ export class MemoryBridgeDeliveryRepo implements IJobBridge {
   async getStatusHistogram(
     windowHours: number,
     tenantId?: string | null,
+    // Reference instant for the window cutoff. Defaults to the wall clock; exposed
+    // as a test-injection seam so a caller can pin the cutoff to the same `now` it
+    // positioned a boundary row from. Without it the cutoff is re-sampled here a few
+    // ms later, pushing an "exactly at the boundary" row just below the window
+    // (the OBS-3 boundary test flake). The drizzle backend has no analogue — it
+    // evaluates the cutoff once in SQL via `now()`.
+    nowMs: number = Date.now(),
   ): Promise<StatusHistogram> {
     if (!Number.isFinite(windowHours) || windowHours <= 0) {
       throw new RangeError('windowHours must be positive');
     }
 
-    const cutoffMs = Date.now() - windowHours * 3_600_000;
+    const cutoffMs = nowMs - windowHours * 3_600_000;
     const histogram: StatusHistogram = {
       pending: 0,
       delivered: 0,

package/runtime/subsystems/jobs/job-orchestrator.memory-backend.ts

@@ -137,10 +137,15 @@ export class MemoryJobOrchestrator implements IJobOrchestrator {
   private readonly queueBlockers = new Map<string, string[]>();
 
   constructor(
-    private readonly store: MemoryJobStore,
-    private readonly stepService: MemoryJobStepService,
+    // ADR-037 (package-mode DI): explicit `@Inject` tokens on every param —
+    // the published bundle has no `design:paramtypes` metadata (built without
+    // `emitDecoratorMetadata`), so by-type injection would resolve to
+    // `undefined` in package mode. Class tokens (`MemoryJobStore`,
+    // `MemoryJobStepService`, `ModuleRef`) are passed to `@Inject` explicitly.
+    @Inject(MemoryJobStore) private readonly store: MemoryJobStore,
+    @Inject(MemoryJobStepService) private readonly stepService: MemoryJobStepService,
     @Inject(JOBS_MULTI_TENANT) private readonly multiTenant: boolean,
-    @Optional() private readonly moduleRef?: ModuleRef,
+    @Optional() @Inject(ModuleRef) private readonly moduleRef?: ModuleRef,
   ) {}
 
   /**

package/runtime/subsystems/jobs/job-run-service.memory-backend.ts

@@ -39,7 +39,10 @@ const NON_TERMINAL_STATUSES: JobRunRow['status'][] = [
 @Injectable()
 export class MemoryJobRunService implements IJobRunService {
   constructor(
-    private readonly store: MemoryJobStore,
+    // ADR-037 (package-mode DI): explicit `@Inject(MemoryJobStore)` — the
+    // published bundle carries no `design:paramtypes`, so a by-type inject
+    // would resolve to `undefined` in package mode.
+    @Inject(MemoryJobStore) private readonly store: MemoryJobStore,
     @Inject(JOB_ORCHESTRATOR) private readonly orchestrator: IJobOrchestrator,
     @Inject(JOBS_MULTI_TENANT) private readonly multiTenant: boolean,
   ) {}

package/runtime/subsystems/jobs/job-step-service.memory-backend.ts

@@ -8,7 +8,7 @@
  * the Drizzle backend (which deletes non-completed rows on replay).
  */
 import { randomUUID } from 'node:crypto';
-import { Injectable } from '@nestjs/common';
+import { Inject, Injectable } from '@nestjs/common';
 import type { JobStepRow } from './job-orchestration.schema';
 import type {
   IJobStepService,
@@ -19,7 +19,12 @@ import { MemoryJobStore } from './memory-job-store';
 
 @Injectable()
 export class MemoryJobStepService implements IJobStepService {
-  constructor(private readonly store: MemoryJobStore) {}
+  // ADR-037 (package-mode DI): explicit `@Inject(MemoryJobStore)` — the
+  // published bundle carries no `design:paramtypes`, so a by-type inject
+  // would resolve to `undefined` in package mode.
+  constructor(
+    @Inject(MemoryJobStore) private readonly store: MemoryJobStore,
+  ) {}
 
   async findStep(runId: string, stepId: string): Promise<JobStep | null> {
     const rows = this.store.steps.get(runId);

package/runtime/subsystems/jobs/job-worker.module.ts

@@ -158,7 +158,19 @@ export class JobWorkerOrchestrator implements OnModuleInit, OnModuleDestroy {
      * without supplying a `DRIZZLE` provider.
      */
     @Optional() @Inject(DRIZZLE) private readonly db: DrizzleClient | null = null,
-    private readonly moduleRef?: ModuleRef,
+    /**
+     * ADR-037 (package-mode DI): inject `ModuleRef` EXPLICITLY via `@Inject`
+     * rather than relying on `design:paramtypes` reflection. The published
+     * package bundle is built without `emitDecoratorMetadata` (tsup/esbuild
+     * default), so a by-type injection here would resolve to `undefined` at
+     * boot in package mode — breaking the worker entirely (the
+     * `ModuleRef not available` throw). Vendored mode happened to work only
+     * because the consumer's own `tsc` (emitDecoratorMetadata: true)
+     * recompiled the source and emitted the metadata. The explicit token is
+     * mode-agnostic. `ModuleRef` is always provided by `@nestjs/core`, so no
+     * `@Optional()` is needed (it's a hard dependency of the worker path).
+     */
+    @Inject(ModuleRef) private readonly moduleRef?: ModuleRef,
     /**
      * BULLMQ-1 — resolved BullMQ connection + config, only bound when the
      * inner `JobsDomainModule` was booted with `backend: 'bullmq'`. `@Optional()`