@aurorajs.dev/catalyst-cli 1.1.3 → 2.0.0

package/dist/templates/backend/application/src/@aurora/modules/pdf/pdfmake/pdfmake-renderer.service.spec.ts

@@ -0,0 +1,58 @@
+import { PdfmakeRendererService } from './pdfmake-renderer.service.js';
+
+describe('PdfmakeRendererService', () => {
+  let service: PdfmakeRendererService;
+
+  beforeEach(() => {
+    service = new PdfmakeRendererService();
+  });
+
+  // The one assertion in this whole change made against a BINARY rather than a value (design D10):
+  // everything else asserts the document definition pdfmake receives, never pdfmake's own output.
+  it('renders a minimal document definition to a non-empty PDF buffer', async () => {
+    const buffer = await service.render({
+      content: [{ text: 'Hello world' }],
+    });
+
+    expect(buffer).toBeInstanceOf(Buffer);
+    expect(buffer.length).toBeGreaterThan(0);
+    expect(buffer.subarray(0, 4).toString()).toBe('%PDF');
+  });
+
+  it('embeds a referenced asset that exists under assets/pdf', async () => {
+    const buffer = await service.render({
+      content: [{ image: 'logo', width: 50 }],
+      images: { logo: 'pdf/bipdrive-logo.jpg' },
+    });
+
+    expect(buffer.subarray(0, 4).toString()).toBe('%PDF');
+  });
+
+  // Requirement "A missing asset fails loudly" (backend-pdf-rendering spec): the render must
+  // reject with an error naming the missing asset, never produce a document with a blank space.
+  it('fails loudly, naming the asset, when a referenced asset is absent from the artifact', async () => {
+    await expect(
+      service.render({
+        content: [{ image: 'missing', width: 50 }],
+        images: { missing: 'pdf/does-not-exist.jpg' },
+      }),
+    ).rejects.toThrow(/pdf\/does-not-exist\.jpg/);
+  });
+
+  it("renders header and footer callbacks fed by the engine's currentPage/pageCount", async () => {
+    const seenContexts: Array<{ currentPage: number; pageCount: number }> = [];
+
+    const buffer = await service.render({
+      content: [{ text: 'Body' }],
+      header: (context) => {
+        seenContexts.push(context);
+        return { text: `Pag. ${context.currentPage} / ${context.pageCount}` };
+      },
+      footer: (context) => ({ text: `Footer ${context.currentPage}` }),
+    });
+
+    expect(buffer.subarray(0, 4).toString()).toBe('%PDF');
+    expect(seenContexts.length).toBeGreaterThan(0);
+    expect(seenContexts[0]).toEqual({ currentPage: 1, pageCount: 1 });
+  });
+});

package/dist/templates/backend/application/src/@aurora/modules/pdf/pdfmake/pdfmake-renderer.service.ts

@@ -0,0 +1,123 @@
+import { Injectable } from '@nestjs/common';
+import { existsSync, readFileSync } from 'node:fs';
+import { extname, join } from 'node:path';
+// pdfmake's CJS build (`js/index.js`) exports `module.exports = new pdfmake()` — a default
+// export under Node's CJS/ESM interop, not a named one. This is the ONLY file below
+// `@aurora/modules/pdf` allowed to reach the library (design D2, HR-BARRELS-004).
+import pdfMake from 'pdfmake';
+
+import {
+  PdfContentNode,
+  PdfDocumentDefinition,
+  PdfHeaderFooterBuilder,
+} from '../pdf-document.types.js';
+import { PdfRendererService } from '../pdf-renderer.service.js';
+
+// pdfmake's Node build renders through PDFKit, whose 14 "standard" font names (Helvetica among
+// them) need no font FILE at all — no TTF to ship, no licensing question. Design D11: this stands
+// in for the PRD's unlicensed Century Gothic/Avenir until a licensed font file is supplied.
+const FONT_FAMILY = 'Helvetica';
+const ALLOWED_LOCAL_FONT_NAMES = new Set([
+  'Helvetica',
+  'Helvetica-Bold',
+  'Helvetica-Oblique',
+  'Helvetica-BoldOblique',
+]);
+const FONTS = {
+  [FONT_FAMILY]: {
+    normal: 'Helvetica',
+    bold: 'Helvetica-Bold',
+    italics: 'Helvetica-Oblique',
+    bolditalics: 'Helvetica-BoldOblique',
+  },
+};
+
+const CONTENT_TYPE_BY_EXTENSION: Record<string, string> = {
+  '.jpg': 'image/jpeg',
+  '.jpeg': 'image/jpeg',
+  '.png': 'image/png',
+};
+
+// Mirrors `MailerCLientModule`'s `join(import.meta.dirname, '..', '..', '..')` + `assets/...`
+// (design D2): resolves identically from `src/` under `nest start`/Vitest and from `dist/src/`
+// after a build, because `nest-cli.json`'s `assets/**/*` copy mirrors the `src/` tree one-to-one
+// into `dist/src/`. This adapter sits one folder deeper than the mailer module (`pdf/pdfmake/`,
+// not `mailer/`), hence the extra `..`.
+const ASSETS_ROOT = join(import.meta.dirname, '..', '..', '..', '..', 'assets');
+
+/**
+ * The `pdfmake`-backed implementation of `PdfRendererService` (design D1). It lives BESIDE the
+ * port, in `@aurora/modules/pdf/`, but deliberately OUTSIDE the port's own `index.ts` barrel —
+ * reachable only through this folder's own `pdfmake/index.ts` sub-barrel.
+ *
+ * This diverges from `@aurora/modules/mailer`, whose `index.ts` re-exports its concrete transport
+ * (`MailerSMTPTransportService`) directly. That is fine for mailer: `MAILER_ENABLED` gates the SDK
+ * behind a runtime flag, so a stray re-export costs nothing extra. It is NOT fine here: `pdfmake`
+ * has no such gate, `@aurora/modules/index.ts` re-exports the PDF barrel unconditionally, and
+ * `app.module.ts` imports from it — so a barrel that re-exported this class would drag `pdfmake`
+ * into the boot graph of every consumer of `@aurora/modules`, not just the one that renders a PDF.
+ * `pdf-isolation.spec.ts` guards this property by making `pdfmake` throw on `require` and asserting
+ * the port's barrel still imports clean; DO NOT "fix" the split back to the mailer's shape — it
+ * would silently reintroduce the dependency this test exists to catch.
+ */
+// `pdfMake` is now a properly typed import (previously an untyped `require`), so
+// `createPdf`'s first argument is pdfmake's own concrete `Content`-union AST — a type this
+// port deliberately does NOT mirror (`PdfContentNode` stays a loose bag, see
+// `pdf-document.types.ts`). Deriving the parameter type from `createPdf` itself, rather than
+// naming an unexported `@types/pdfmake` interface, keeps the cast anchored to the one function
+// signature this adapter actually calls.
+type PdfMakeDocumentDefinition = Parameters<typeof pdfMake.createPdf>[0];
+
+@Injectable()
+export class PdfmakeRendererService extends PdfRendererService {
+  async render(document: PdfDocumentDefinition): Promise<Buffer> {
+    pdfMake.setUrlAccessPolicy(() => false);
+    pdfMake.setLocalAccessPolicy((path: string) => ALLOWED_LOCAL_FONT_NAMES.has(path));
+    pdfMake.setFonts(FONTS);
+
+    const pdfDocGenerator = pdfMake.createPdf({
+      content: document.content,
+      images: this.resolveImages(document.images),
+      defaultStyle: { font: FONT_FAMILY, ...document.defaultStyle },
+      styles: document.styles,
+      header: document.header ? this.toDynamicContent(document.header) : undefined,
+      footer: document.footer ? this.toDynamicContent(document.footer) : undefined,
+      pageMargins: document.pageMargins,
+    } as unknown as PdfMakeDocumentDefinition);
+
+    return pdfDocGenerator.getBuffer();
+  }
+
+  // Requirement "Document assets ship with the build" / "A missing asset fails loudly"
+  // (backend-pdf-rendering spec): resolved at RENDER time, relative to the running code, and
+  // embedded as a base64 data URI — never a bare file path — so pdfmake's own file-path branch
+  // (and the local-access-policy check that guards it) is never exercised for images at all.
+  private resolveImages(images?: Record<string, string>): Record<string, string> | undefined {
+    if (!images) return undefined;
+
+    const resolved: Record<string, string> = {};
+    for (const [name, relativePath] of Object.entries(images)) {
+      const absolutePath = join(ASSETS_ROOT, relativePath);
+      if (!existsSync(absolutePath)) {
+        throw new Error(
+          `PdfmakeRendererService: missing document asset '${relativePath}' (referenced as '${name}')`,
+        );
+      }
+      const contentType =
+        CONTENT_TYPE_BY_EXTENSION[extname(absolutePath).toLowerCase()] ??
+        'application/octet-stream';
+      resolved[name] =
+        `data:${contentType};base64,${readFileSync(absolutePath).toString('base64')}`;
+    }
+    return resolved;
+  }
+
+  // Adapts the port's context-object callback to pdfmake's positional `(currentPage, pageCount,
+  // pageSize) => Content` signature — the ONE place that convention leaks into this file, kept out
+  // of the port and out of every caller of it.
+  private toDynamicContent(
+    builder: PdfHeaderFooterBuilder,
+  ): (currentPage: number, pageCount: number) => PdfContentNode {
+    return (currentPage: number, pageCount: number) => builder({ currentPage, pageCount });
+  }
+}

package/dist/templates/backend/application/src/@aurora/modules/report-engine/execute-report.service.spec.ts

@@ -13,8 +13,8 @@
  */
 import { ForbiddenException, NotFoundException } from '@nestjs/common';
 
-import { ExecuteReportService } from './execute-report.service';
-import type { ExecuteReportParams, ReportAccount, ReportCube } from './report-cube.types';
+import { ExecuteReportService } from './execute-report.service.js';
+import type { ExecuteReportParams, ReportAccount, ReportCube } from './report-cube.types.js';
 
 // ---------------------------------------------------------------------------
 // Fixture helpers
@@ -54,11 +54,11 @@ const defaultSelection = { dimensions: ['dim1'], measures: ['m1'] };
 function makeService(cubeOrUndefined: ReportCube | undefined): ExecuteReportService {
   return new ExecuteReportService(
     { findByKey: () => cubeOrUndefined } as any,
-    { validateReportSelection: jest.fn() },
-    { checkCardinality: jest.fn() },
-    { buildPredicates: jest.fn().mockReturnValue({ predicates: [], binds: {} }) } as any,
+    { validateReportSelection: vi.fn() },
+    { checkCardinality: vi.fn() },
+    { buildPredicates: vi.fn().mockReturnValue({ predicates: [], binds: {} }) } as any,
     {
-      execute: jest.fn().mockResolvedValue(stubClientResult),
+      execute: vi.fn().mockResolvedValue(stubClientResult),
     } as any,
     null,
     {} as any, // builder — not exercised in these tests

package/dist/templates/backend/application/src/@aurora/modules/report-engine/execute-report.service.ts

@@ -1,18 +1,18 @@
 import { ForbiddenException, Inject, Injectable, NotFoundException } from '@nestjs/common';
 
-import { checkCardinality } from './report-cardinality.guard';
+import { checkCardinality } from './report-cardinality.guard.js';
 import type {
   ExecuteReportParams,
   ReportAccount,
   ReportResult,
   ReportSelection,
-} from './report-cube.types';
-import { ReportQueryBuilder } from './report-query.builder';
-import { ReportRegistry } from './report-registry';
-import { validateReportSelection } from './report-selection.validator';
-import { ReportTenantScoper } from './report-tenant.scoper';
-import { ClientMode } from './result-modes/client-mode';
-import { ServerMode } from './result-modes/server-mode';
+} from './report-cube.types.js';
+import { ReportQueryBuilder } from './report-query.builder.js';
+import { ReportRegistry } from './report-registry.js';
+import { validateReportSelection } from './report-selection.validator.js';
+import { ReportTenantScoper } from './report-tenant.scoper.js';
+import { ClientMode } from './result-modes/client-mode.js';
+import { ServerMode } from './result-modes/server-mode.js';
 
 /** DI tokens for the duck-typed validator/guard dependencies. */
 export const SELECTION_VALIDATOR_TOKEN = 'REPORT_SELECTION_VALIDATOR';

package/dist/templates/backend/application/src/@aurora/modules/report-engine/index.ts

@@ -5,7 +5,7 @@
  */
 
 // Token
-export { AURORA_REPORT } from './aurora-report.token';
+export { AURORA_REPORT } from './aurora-report.token.js';
 
 // Core types
 export type {
@@ -32,7 +32,7 @@ export type {
   TenantColumnDeclaration,
   TimeGrain,
   TooLargeResult,
-} from './report-cube.types';
+} from './report-cube.types.js';
 
 // Const maps
 export {
@@ -42,41 +42,41 @@ export {
   JOIN_TYPE,
   RESULT_MODE,
   TIME_GRAIN,
-} from './report-cube.types';
+} from './report-cube.types.js';
 
 // Validators
-export { validateCubeRegistration } from './report-cube.validator';
-export { SelectionValidationError, validateReportSelection } from './report-selection.validator';
+export { validateCubeRegistration } from './report-cube.validator.js';
+export { SelectionValidationError, validateReportSelection } from './report-selection.validator.js';
 
 // Cardinality guard
-export { CardinalityError, checkCardinality } from './report-cardinality.guard';
+export { CardinalityError, checkCardinality } from './report-cardinality.guard.js';
 
 // Builder
-export { ReportQueryBuilder } from './report-query.builder';
-export type { BuildExecuteOptions, BuilderResult } from './report-query.builder';
+export { ReportQueryBuilder } from './report-query.builder.js';
+export type { BuildExecuteOptions, BuilderResult } from './report-query.builder.js';
 
 // Tenant scoper
-export { ReportTenantScoper } from './report-tenant.scoper';
-export type { TenantScopeResult } from './report-tenant.scoper';
+export { ReportTenantScoper } from './report-tenant.scoper.js';
+export type { TenantScopeResult } from './report-tenant.scoper.js';
 
 // Result modes
-export { ClientMode } from './result-modes/client-mode';
-export type { ClientModeConfig, ClientModeExecuteOptions } from './result-modes/client-mode';
-export { ServerMode } from './result-modes/server-mode';
-export type { ServerModeExecuteOptions } from './result-modes/server-mode';
+export { ClientMode } from './result-modes/client-mode.js';
+export type { ClientModeConfig, ClientModeExecuteOptions } from './result-modes/client-mode.js';
+export { ServerMode } from './result-modes/server-mode.js';
+export type { ServerModeExecuteOptions } from './result-modes/server-mode.js';
 
 // Registry + composite entry point
 export {
   CARDINALITY_GUARD_TOKEN,
   ExecuteReportService,
   SELECTION_VALIDATOR_TOKEN,
-} from './execute-report.service';
-export { ReportRegistry } from './report-registry';
+} from './execute-report.service.js';
+export { ReportRegistry } from './report-registry.js';
 
 // Read-only pool
-export { reportDbPoolProvider } from './report-db-pool.provider';
-export { REPORT_DB_POOL } from './report-db-pool.token';
-export { ReportPreflightService } from './report-preflight.service';
+export { reportDbPoolProvider } from './report-db-pool.provider.js';
+export { REPORT_DB_POOL } from './report-db-pool.token.js';
+export { ReportPreflightService } from './report-preflight.service.js';
 
 // Module (imported by consumer BCs)
-export { ReportEngineModule } from './report-engine.module';
+export { ReportEngineModule } from './report-engine.module.js';

package/dist/templates/backend/application/src/@aurora/modules/report-engine/report-cardinality.guard.spec.ts

@@ -1,5 +1,5 @@
-import { checkCardinality } from './report-cardinality.guard';
-import type { ReportCube, ReportSelection } from './report-cube.types';
+import { checkCardinality } from './report-cardinality.guard.js';
+import type { ReportCube, ReportSelection } from './report-cube.types.js';
 
 // ---------------------------------------------------------------------------
 // Fixture cubes

package/dist/templates/backend/application/src/@aurora/modules/report-engine/report-cardinality.guard.ts

@@ -1,4 +1,4 @@
-import type { CubeMeasure, ReportCube, ReportSelection } from './report-cube.types';
+import type { CubeMeasure, ReportCube, ReportSelection } from './report-cube.types.js';
 
 /**
  * Aggregate functions that are unsafe when a measure's source table is on the

package/dist/templates/backend/application/src/@aurora/modules/report-engine/report-cube.validator.spec.ts

@@ -1,7 +1,7 @@
-import { CardinalityError } from './report-cardinality.guard';
-import type { ReportCube } from './report-cube.types';
-import { validateCubeRegistration } from './report-cube.validator';
-import { SelectionValidationError } from './report-selection.validator';
+import { CardinalityError } from './report-cardinality.guard.js';
+import type { ReportCube } from './report-cube.types.js';
+import { validateCubeRegistration } from './report-cube.validator.js';
+import { SelectionValidationError } from './report-selection.validator.js';
 
 // ---------------------------------------------------------------------------
 // Fixture cubes

package/dist/templates/backend/application/src/@aurora/modules/report-engine/report-cube.validator.ts

@@ -1,6 +1,6 @@
-import { checkCardinality } from './report-cardinality.guard';
-import type { ReportCube } from './report-cube.types';
-import { validateReportSelection } from './report-selection.validator';
+import { checkCardinality } from './report-cardinality.guard.js';
+import type { ReportCube } from './report-cube.types.js';
+import { validateReportSelection } from './report-selection.validator.js';
 
 /**
  * Validates a cube at registration time:

package/dist/templates/backend/application/src/@aurora/modules/report-engine/report-db-pool.preflight.spec.ts

@@ -1,6 +1,6 @@
 import { InternalServerErrorException } from '@nestjs/common';
 
-import { ReportPreflightService } from './report-preflight.service';
+import { ReportPreflightService } from './report-preflight.service.js';
 
 // ---------------------------------------------------------------------------
 // Helpers
@@ -22,8 +22,8 @@ import { ReportPreflightService } from './report-preflight.service';
  */
 function makePgPool(): any {
   return {
-    query: jest.fn(),
-    end: jest.fn(),
+    query: vi.fn(),
+    end: vi.fn(),
   };
 }
 
@@ -166,7 +166,7 @@ describe('ReportPreflightService', () => {
 
   describe('Scenario: Reports use the read-only pool, not the main pool', () => {
     it('ReportPreflightService receives the read-only pg.Pool instance via DI token', () => {
-      const mockPool: any = { query: jest.fn(), end: jest.fn() };
+      const mockPool: any = { query: vi.fn(), end: vi.fn() };
       const svc = new ReportPreflightService(mockPool);
       expect(svc).toBeDefined();
     });

package/dist/templates/backend/application/src/@aurora/modules/report-engine/report-db-pool.provider.spec.ts

@@ -1,7 +1,7 @@
 import { InternalServerErrorException } from '@nestjs/common';
 import { Pool } from 'pg';
 
-import { reportDbPoolProvider } from './report-db-pool.provider';
+import { reportDbPoolProvider } from './report-db-pool.provider.js';
 
 const EXPECTED_POOL_CLASS = Pool;
 

package/dist/templates/backend/application/src/@aurora/modules/report-engine/report-db-pool.provider.ts

@@ -2,7 +2,7 @@ import type { Provider } from '@nestjs/common';
 import { InternalServerErrorException } from '@nestjs/common';
 import { Pool } from 'pg';
 
-import { REPORT_DB_POOL } from './report-db-pool.token';
+import { REPORT_DB_POOL } from './report-db-pool.token.js';
 
 /**
  * Creates the read-only `pg.Pool` connection pool for report execution.

package/dist/templates/backend/application/src/@aurora/modules/report-engine/report-engine.module.spec.ts

@@ -17,9 +17,9 @@
 import { Module } from '@nestjs/common';
 import { Test } from '@nestjs/testing';
 
-import type { ReportCube } from './report-cube.types';
-import { ReportEngineModule } from './report-engine.module';
-import { ReportRegistry } from './report-registry';
+import type { ReportCube } from './report-cube.types.js';
+import { ReportEngineModule } from './report-engine.module.js';
+import { ReportRegistry } from './report-registry.js';
 
 // ---------------------------------------------------------------------------
 // Minimal valid cube (primaryTableAlias must appear in tenantColumns)
@@ -96,12 +96,11 @@ describe('ReportEngineModule.forFeature — cross-module DI scope', () => {
     );
   });
 
-  it('ends the report pg.Pool on application shutdown (jest exits cleanly, no leaked sockets)', async () => {
+  it('ends the report pg.Pool on application shutdown (exits cleanly, no leaked sockets)', async () => {
     // The report pool is the engine's OWN connection (separate read-only
     // credentials); without teardown its sockets keep the Node event loop
-    // alive after app.close() — the "Jest did not exit one second after the
-    // test run has completed" hang. Mirrors DrizzleConfigModule's teardown.
-    const fakePool = { end: jest.fn().mockResolvedValue(undefined), ended: false };
+    // alive after app.close(), hanging the test run. Mirrors DrizzleConfigModule's teardown.
+    const fakePool = { end: vi.fn().mockResolvedValue(undefined), ended: false };
     const moduleInstance = new ReportEngineModule(fakePool as never);
 
     await moduleInstance.onApplicationShutdown();
@@ -110,7 +109,7 @@ describe('ReportEngineModule.forFeature — cross-module DI scope', () => {
   });
 
   it('shutdown is a no-op when the pool is already ended (no double-end crash)', async () => {
-    const fakePool = { end: jest.fn().mockResolvedValue(undefined), ended: true };
+    const fakePool = { end: vi.fn().mockResolvedValue(undefined), ended: true };
     const moduleInstance = new ReportEngineModule(fakePool as never);
 
     await moduleInstance.onApplicationShutdown();

package/dist/templates/backend/application/src/@aurora/modules/report-engine/report-engine.module.ts

@@ -1,24 +1,24 @@
 import { DynamicModule, Inject, Module, OnApplicationShutdown, Optional } from '@nestjs/common';
 import type { Pool } from 'pg';
 
-import { AURORA_REPORT } from './aurora-report.token';
+import { AURORA_REPORT } from './aurora-report.token.js';
 import {
   CARDINALITY_GUARD_TOKEN,
   ExecuteReportService,
   SELECTION_VALIDATOR_TOKEN,
-} from './execute-report.service';
-import { checkCardinality } from './report-cardinality.guard';
-import type { ReportCube } from './report-cube.types';
-import { validateCubeRegistration } from './report-cube.validator';
-import { reportDbPoolProvider } from './report-db-pool.provider';
-import { REPORT_DB_POOL } from './report-db-pool.token';
-import { ReportPreflightService } from './report-preflight.service';
-import { ReportQueryBuilder } from './report-query.builder';
-import { ReportRegistry } from './report-registry';
-import { validateReportSelection } from './report-selection.validator';
-import { ReportTenantScoper } from './report-tenant.scoper';
-import { ClientMode } from './result-modes/client-mode';
-import { ServerMode } from './result-modes/server-mode';
+} from './execute-report.service.js';
+import { checkCardinality } from './report-cardinality.guard.js';
+import type { ReportCube } from './report-cube.types.js';
+import { validateCubeRegistration } from './report-cube.validator.js';
+import { reportDbPoolProvider } from './report-db-pool.provider.js';
+import { REPORT_DB_POOL } from './report-db-pool.token.js';
+import { ReportPreflightService } from './report-preflight.service.js';
+import { ReportQueryBuilder } from './report-query.builder.js';
+import { ReportRegistry } from './report-registry.js';
+import { validateReportSelection } from './report-selection.validator.js';
+import { ReportTenantScoper } from './report-tenant.scoper.js';
+import { ClientMode } from './result-modes/client-mode.js';
+import { ServerMode } from './result-modes/server-mode.js';
 
 /**
  * NestJS module for the report engine framework primitive.
@@ -53,9 +53,8 @@ export class ReportEngineModule implements OnApplicationShutdown {
 
   /**
    * Ends the report pool on shutdown — without it, the pool's open sockets
-   * keep the Node event loop alive after `app.close()`, hanging every Jest
-   * e2e run ("Jest did not exit one second after the test run has
-   * completed"). Mirrors `DrizzleConfigModule`'s teardown (core-back).
+   * keep the Node event loop alive after `app.close()`, hanging every e2e run.
+   * Mirrors `DrizzleConfigModule`'s teardown (core-back).
    */
   async onApplicationShutdown(): Promise<void> {
     if (this.reportDbPool && !this.reportDbPool.ended) {

package/dist/templates/backend/application/src/@aurora/modules/report-engine/report-preflight.service.ts

@@ -7,7 +7,7 @@ import {
 } from '@nestjs/common';
 import type { Pool } from 'pg';
 
-import { REPORT_DB_POOL } from './report-db-pool.token';
+import { REPORT_DB_POOL } from './report-db-pool.token.js';
 
 /**
  * Boot-time preflight for the read-only report database pool.

package/dist/templates/backend/application/src/@aurora/modules/report-engine/report-query.builder.spec.ts

@@ -1,5 +1,6 @@
-import type { ColumnMeta, ReportCube, ReportSelection } from './report-cube.types';
-import { ReportQueryBuilder } from './report-query.builder';
+import type { Mock } from 'vitest';
+import type { ColumnMeta, ReportCube, ReportSelection } from './report-cube.types.js';
+import { ReportQueryBuilder } from './report-query.builder.js';
 
 // ---------------------------------------------------------------------------
 // Fixture cube
@@ -42,9 +43,9 @@ const salesSelection: ReportSelection = {
  * bare array Sequelize's `QueryTypes.SELECT` returned. Objects are keyed by
  * the column alias used in the SELECT clause, same as before.
  */
-function makeMockPgPool(rows: Record<string, unknown>[] = []): { query: jest.Mock } {
+function makeMockPgPool(rows: Record<string, unknown>[] = []): { query: Mock } {
   return {
-    query: jest.fn().mockResolvedValue({ rows }),
+    query: vi.fn().mockResolvedValue({ rows }),
   };
 }
 
@@ -315,7 +316,9 @@ describe('ReportQueryBuilder', () => {
       expect(result.rows).toHaveLength(1);
       expect(result.rows[0]).toHaveLength(1);
       // Must be parseable as 42, not undefined/NaN/0
-      expect(parseInt(String(result.rows[0][0]), 10)).toBe(42);
+      const cell = result.rows[0][0];
+      expect(typeof cell === 'string' || typeof cell === 'number').toBe(true);
+      expect(parseInt(String(cell as string | number), 10)).toBe(42);
     });
 
     it('baseFilter predicates appear in the generated WHERE clause', async () => {

package/dist/templates/backend/application/src/@aurora/modules/report-engine/report-query.builder.ts

@@ -1,8 +1,8 @@
 import { Inject, Injectable } from '@nestjs/common';
 import type { Pool } from 'pg';
 
-import type { ColumnMeta, ReportCube, ReportSelection } from './report-cube.types';
-import { REPORT_DB_POOL } from './report-db-pool.token';
+import type { ColumnMeta, ReportCube, ReportSelection } from './report-cube.types.js';
+import { REPORT_DB_POOL } from './report-db-pool.token.js';
 
 /** Options passed to `buildAndExecute` for extra binds (e.g. tenant values). */
 export interface BuildExecuteOptions {
@@ -159,7 +159,7 @@ export class ReportQueryBuilder {
     tenantPredicates: string[],
     options: BuildExecuteOptions = {},
   ): { sql: string; bind: Record<string, unknown> } {
-    const bind: Record<string, unknown> = { ...(options.binds ?? {}) };
+    const bind: Record<string, unknown> = { ...options.binds };
 
     const selectParts = this.buildSelectParts(cube, selection);
     const fromClause = cube.primaryTableSql;

package/dist/templates/backend/application/src/@aurora/modules/report-engine/report-registry.spec.ts

@@ -1,9 +1,11 @@
 import { Test } from '@nestjs/testing';
+import * as fs from 'node:fs';
+import * as path from 'node:path';
 
-import { AURORA_REPORT } from './aurora-report.token';
-import { ExecuteReportService } from './execute-report.service';
-import type { ExecuteReportParams, ReportAccount, ReportCube } from './report-cube.types';
-import { ReportRegistry } from './report-registry';
+import { AURORA_REPORT } from './aurora-report.token.js';
+import { ExecuteReportService } from './execute-report.service.js';
+import type { ExecuteReportParams, ReportAccount, ReportCube } from './report-cube.types.js';
+import { ReportRegistry } from './report-registry.js';
 
 // ---------------------------------------------------------------------------
 // Fixture cubes
@@ -47,7 +49,7 @@ const account: ReportAccount = {
 };
 
 const mockBuilder = {
-  buildAndExecute: jest.fn().mockResolvedValue({
+  buildAndExecute: vi.fn().mockResolvedValue({
     columns: [{ key: 'region', type: 'dimension' }],
     rows: [['north', '100']],
     meta: { mode: 'client', total: null },
@@ -62,7 +64,7 @@ describe('ReportRegistry', () => {
   let registry: ReportRegistry;
 
   beforeEach(async () => {
-    jest.clearAllMocks();
+    vi.clearAllMocks();
 
     // Provide AURORA_REPORT directly — both cubes in a single useValue array.
     // This mirrors what ReportEngineModule.forFeature() does internally.
@@ -99,13 +101,13 @@ describe('ExecuteReportService', () => {
         {
           findByKey: (key: string) => (key === 'sales-by-region' ? clientCube : undefined),
         } as any,
-        { validateReportSelection: jest.fn() },
-        { checkCardinality: jest.fn() },
+        { validateReportSelection: vi.fn() },
+        { checkCardinality: vi.fn() },
         {
-          buildPredicates: jest.fn().mockReturnValue({ predicates: [], binds: {} }),
+          buildPredicates: vi.fn().mockReturnValue({ predicates: [], binds: {} }),
         } as any,
         {
-          execute: jest.fn().mockResolvedValue({
+          execute: vi.fn().mockResolvedValue({
             columns: [{ key: 'region', type: 'dimension' }],
             rows: [['north', '100']],
             meta: { mode: 'client', total: null },
@@ -128,10 +130,10 @@ describe('ExecuteReportService', () => {
     it('throws NotFoundException when the cube key is not registered', async () => {
       const service = new ExecuteReportService(
         { findByKey: () => undefined } as any,
-        { validateReportSelection: jest.fn() },
-        { checkCardinality: jest.fn() },
-        { buildPredicates: jest.fn().mockReturnValue({ predicates: [], binds: {} }) } as any,
-        { execute: jest.fn() } as any,
+        { validateReportSelection: vi.fn() },
+        { checkCardinality: vi.fn() },
+        { buildPredicates: vi.fn().mockReturnValue({ predicates: [], binds: {} }) } as any,
+        { execute: vi.fn() } as any,
         null,
         mockBuilder as any,
       );
@@ -144,12 +146,7 @@ describe('ExecuteReportService', () => {
 
   describe('Scenario: Engine does not import @app cube modules directly', () => {
     it('has no @app imports in engine source files (static assertion)', () => {
-      // eslint-disable-next-line @typescript-eslint/no-require-imports
-      const fs = require('fs');
-      // eslint-disable-next-line @typescript-eslint/no-require-imports
-      const path = require('path');
-
-      const engineDir = path.join(__dirname);
+      const engineDir = path.join(import.meta.dirname);
       const resultModesDir = path.join(engineDir, 'result-modes');
 
       const rootFiles = fs