@pattern-stack/codegen 0.14.0 → 0.14.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pattern-stack/codegen",
-  "version": "0.14.0",
+  "version": "0.14.2",
   "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/bridge/bridge.module.ts

@@ -70,6 +70,7 @@ import { BridgeOutboxDrainHook } from './bridge-outbox-drain-hook';
 import { EventFlowService } from './event-flow.service';
 import { BridgeDeliveryHandler } from './bridge-delivery-handler';
 import { bridgeRegistry } from './generated/registry';
+import type { BridgeRegistry } from './bridge.protocol';
 import { BRIDGE_RESERVED_POOLS } from './reserved-pools';
 
 export interface BridgeModuleOptions {
@@ -87,6 +88,26 @@ export interface BridgeModuleOptions {
    * `null` always passes (cross-tenant work). Defaults to `false`.
    */
   multiTenant?: boolean;
+  /**
+   * The codegen-emitted `Record<EventTypeName, BridgeTriggerEntry[]>` that
+   * drives outbox-drain trigger lookup (BRIDGE-4) and the facade's Case B
+   * dedup (BRIDGE-7).
+   *
+   * **Package mode (ADR-037).** When the runtime is imported from
+   * `@pattern-stack/codegen` (not vendored), the bundled
+   * `./generated/registry` is a frozen empty placeholder (`{}`) — a
+   * consumer's `@JobHandler.triggers` are scanned into a registry that lives
+   * in THEIR `src/generated/bridge-registry.ts`, which the package can't
+   * import. The generated subsystem barrel therefore threads that registry in
+   * here: `BridgeModule.forRoot({ ..., registry: bridgeRegistry })`. Omitted
+   * (vendored mode / tests) ⇒ falls back to the bundled `./generated/registry`,
+   * which in vendored mode IS the consumer's freshly-generated file.
+   *
+   * Without this, package-mode consumers' triggers never bind and the bridge
+   * routes nothing (the "wrappers sit pending" footgun's silent twin —
+   * nothing is ever enqueued in the first place).
+   */
+  registry?: BridgeRegistry;
 }
 
 @Module({})
@@ -108,7 +129,11 @@ export class BridgeModule implements OnModuleInit {
       providers: [
         { provide: BRIDGE_MODULE_OPTIONS, useValue: opts },
         { provide: BRIDGE_MULTI_TENANT, useValue: opts.multiTenant ?? false },
-        { provide: BRIDGE_REGISTRY, useValue: bridgeRegistry },
+        // Package mode threads the consumer's generated registry through
+        // `opts.registry`; vendored mode omits it and we fall back to the
+        // bundled `./generated/registry` (which IS the consumer's generated
+        // file in a vendored tree). See `BridgeModuleOptions.registry`.
+        { provide: BRIDGE_REGISTRY, useValue: opts.registry ?? bridgeRegistry },
         repoProvider,
         // Drain hook — always wired; `DrizzleEventBus` consumes it via
         // `@Optional()`, so non-bridge mounts simply see `undefined`.

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()`