@neohm/nh-cli 1.1.1 → 1.1.3

package/README.md

@@ -90,13 +90,14 @@ created spreading `baseJobConstants` / `baseEntityConstants` /
 Generated artifacts extend the `@neohm/nestend` base classes:
 
 - Entity → `CommonEntity` (with an empty `attributes` jsonb column)
-- DTO → `CommonPayloadDto`
+- DTO → `CommonDataDto`
 - Data processor → `CommonDataProcessor` (`process()` → `validate()` → `set()`)
 - Listing processor → `CommonListProcessor` (column allowlist config, `id` default sort). It takes `CommonListFilterDto` directly — no per-list DTO is generated, since paging, sorting, filters and free-text search all live on the base DTO and the filterable columns are declared in the processor's `columns` config.
 
 When both a controller and a listing processor are generated, the controller
 ships with a `POST list` endpoint that runs the listing processor (with
 `SqlService` injected via the constructor).
+
 - Migration → `MigrationUtility`
 - Seed → `SeederUtility`
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@neohm/nh-cli",
-  "version": "1.1.1",
+  "version": "1.1.3",
   "description": "nh — Command Line Interface for neohm. NestJS projects",
   "main": "src/index.js",
   "bin": {

package/src/commands/gen.js

@@ -222,6 +222,28 @@ async function gen() {
     );
   }
 
+  // 7. Optional: migration -------------------------------------------------
+  const { migration } = await inquirer.prompt([
+    {
+      type: "confirm",
+      name: "migration",
+      message: `Create migration (M-${n.migrationName})?`,
+      default: true,
+    },
+  ]);
+
+  if (migration) {
+    const ts = Date.now();
+    // Same filename/class rule as `nh migration`.
+    const file = naming.migrationFileName(n.migrationName, ts);
+    const migrationPath = path.join(paths.migrationsDir, file);
+    report(
+      writeFile(migrationPath, tpl.genMigration(entityName, tableName, ts)),
+      migrationPath,
+      paths.root,
+    );
+  }
+
   // Auto-wire: rebuild this module's barrel and register it in app.module.ts.
   log.info("\nWiring imports...");
   const mod = sync.scanModule(

package/src/commands/migration.js

@@ -27,7 +27,7 @@ async function migration() {
   ]);
 
   const ts = Date.now();
-  const file = `${ts}M-${naming.toPascalCase(migrationName)}.ts`;
+  const file = naming.migrationFileName(migrationName, ts);
   const p = path.join(paths.migrationsDir, file);
   report(
     writeFile(p, tpl.migration(migrationName, tableName, ts)),

package/src/lib/naming.js

@@ -42,10 +42,25 @@ function toKebabCase(input) {
   return toWords(input).join('-');
 }
 
+/**
+ * `<timestamp>M-<PascalName>.ts` — the single migration filename rule, shared by
+ * `nh migration` and the migration `nh gen` scaffolds alongside an entity.
+ */
+function migrationFileName(migrationName, timestamp) {
+  return `${timestamp}M-${toPascalCase(migrationName)}.ts`;
+}
+
+/** `<PascalName><timestamp>` — the class name matching migrationFileName(). */
+function migrationClassName(migrationName, timestamp) {
+  return `${toPascalCase(migrationName)}${timestamp}`;
+}
+
 module.exports = {
   toWords,
   toDotCase,
   toPascalCase,
   toCamelCase,
   toKebabCase,
+  migrationFileName,
+  migrationClassName,
 };

package/src/lib/templates.js

@@ -1,11 +1,17 @@
-'use strict';
+"use strict";
 
-const { toDotCase, toPascalCase, toKebabCase, toCamelCase } = require('./naming');
+const {
+  toDotCase,
+  toPascalCase,
+  toKebabCase,
+  toCamelCase,
+  migrationClassName,
+} = require("./naming");
 
 /**
  * Code templates. All generated code imports shared base classes from the
  * published library `@neohm/nestend` (verified exports: CommonEntity,
- * CommonPayloadDto, CommonDataProcessor, MigrationUtility, SeederUtility, etc.)
+ * CommonDataDto, CommonDataProcessor, MigrationUtility, SeederUtility, etc.)
  * and matches the dot.separated filename / PascalCase class conventions used
  * across nestjs-boilerplate.
  */
@@ -36,6 +42,9 @@ function names(base) {
     subscriberFile: `${dot}.subscriber.ts`,
     jobClass: `${pascal}Job`,
     jobFile: `${dot}.job.ts`,
+    // Fed through naming.migrationFileName/migrationClassName so the migration
+    // `nh gen` writes is named exactly like one from `nh migration`.
+    migrationName: `Add${pascal}Table`,
   };
 }
 
@@ -94,9 +103,9 @@ export class ${n.controllerClass} {
 
 function dto(base) {
   const n = names(base);
-  return `import { CommonPayloadDto } from '@neohm/nestend';
+  return `import { CommonDataDto } from '@neohm/nestend';
 
-export class ${n.dtoClass} extends CommonPayloadDto {
+export class ${n.dtoClass} extends CommonDataDto {
   // TODO: declare validated payload fields for ${n.pascal}.
 }
 `;
@@ -120,7 +129,7 @@ export class ${n.processorClass} extends CommonDataProcessor {
 
   private async validate() {
     // TODO: collect validation errors via this.addErrors({ column: 'message' }).
-    this.throwExceptionOnError();
+    this.raiseExceptionOnValidationFailure();
   }
 
   private async set() {
@@ -206,7 +215,7 @@ export class ${n.jobClass} extends CommonJob {
     super(JobConstants.${n.camel}Job);
   }
 
-  async handle(event: any): Promise<void> {
+  async handle(event: DataBaseEventDto<${n.entityClass}>): Promise<void> {
     return;
   }
 }
@@ -220,7 +229,7 @@ export class ${n.jobClass} extends CommonJob {
  */
 function genMigration(base, tableName, timestamp) {
   const n = names(base);
-  const className = `Add${n.pascal}Table${timestamp}`;
+  const className = migrationClassName(n.migrationName, timestamp);
   return `import { MigrationUtility } from '@neohm/nestend';
 
 export class ${className} extends MigrationUtility {
@@ -242,7 +251,7 @@ export class ${className} extends MigrationUtility {
 
 /** Standalone migration (`nh migration`). */
 function migration(migrationName, tableName, timestamp) {
-  const className = `${toPascalCase(migrationName)}${timestamp}`;
+  const className = migrationClassName(migrationName, timestamp);
   return `import { MigrationUtility } from '@neohm/nestend';
 
 export class ${className} extends MigrationUtility {