@neohm/nh-cli 1.1.0 → 1.1.2

package/README.md

@@ -67,7 +67,6 @@ shape/
   es6.classes.ts                     # barrel of controllers/services/jobs/subscribers
   controllers/shape.controller.ts
   dtos/add.shape.data.dto.ts
-  dtos/shape.list.filter.dto.ts
   entities/shape.entity.ts
   libraries/shape.data.processor.ts
   libraries/shape.list.processor.ts
@@ -91,14 +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()`)
-- List filter DTO → `CommonListFilterDto` (plus a `custom_filter` example field)
-- Listing processor → `CommonListProcessor` (column allowlist config, `id` default sort, and an example custom filter)
+- 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.0",
+  "version": "1.1.2",
   "description": "nh — Command Line Interface for neohm. NestJS projects",
   "main": "src/index.js",
   "bin": {

package/src/commands/gen.js

@@ -194,7 +194,7 @@ async function gen() {
     report(writeFile(procPath, tpl.processor(entityName)), procPath, paths.root);
   }
 
-  // 6. Optional: listing processor (includes *ListFilterDto) -----------------
+  // 6. Optional: listing processor (takes CommonListFilterDto directly) ------
   const { listProcessor } = await inquirer.prompt([
     {
       type: "confirm",
@@ -205,13 +205,6 @@ async function gen() {
   ]);
 
   if (listProcessor) {
-    const listDtoPath = path.join(moduleDir, "dtos", n.listFilterDtoFile);
-    report(
-      writeFile(listDtoPath, tpl.listFilterDto(entityName)),
-      listDtoPath,
-      paths.root,
-    );
-
     const listProcPath = path.join(moduleDir, "libraries", n.listProcessorFile);
     report(
       writeFile(listProcPath, tpl.listProcessor(entityName, tableName)),

package/src/lib/templates.js

@@ -1,11 +1,16 @@
-'use strict';
+"use strict";
 
-const { toDotCase, toPascalCase, toKebabCase, toCamelCase } = require('./naming');
+const {
+  toDotCase,
+  toPascalCase,
+  toKebabCase,
+  toCamelCase,
+} = 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.
  */
@@ -30,8 +35,6 @@ function names(base) {
     dtoFile: `add.${dot}.data.dto.ts`,
     processorClass: `${pascal}DataProcessor`,
     processorFile: `${dot}.data.processor.ts`,
-    listFilterDtoClass: `${pascal}ListFilterDto`,
-    listFilterDtoFile: `${dot}.list.filter.dto.ts`,
     listProcessorClass: `${pascal}ListProcessor`,
     listProcessorFile: `${dot}.list.processor.ts`,
     subscriberClass: `${pascal}Subscriber`,
@@ -78,8 +81,7 @@ export class ${n.controllerClass} {
   }
   return `import { Body, Controller, Post } from '@nestjs/common';
 import { ApiTags } from '@nestjs/swagger';
-import { SqlService } from '@neohm/nestend';
-import { ${n.listFilterDtoClass} } from '../dtos/${n.dot}.list.filter.dto';
+import { CommonListFilterDto, SqlService } from '@neohm/nestend';
 import { ${n.listProcessorClass} } from '../libraries/${n.dot}.list.processor';
 
 @ApiTags('${n.kebab}')
@@ -88,7 +90,7 @@ export class ${n.controllerClass} {
   constructor(private readonly sqlService: SqlService) {}
 
   @Post('list')
-  async list(@Body() body: ${n.listFilterDtoClass}) {
+  async list(@Body() body: CommonListFilterDto) {
     return new ${n.listProcessorClass}(this.sqlService).process(body);
   }
 }
@@ -97,9 +99,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}.
 }
 `;
@@ -123,7 +125,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() {
@@ -133,31 +135,21 @@ export class ${n.processorClass} extends CommonDataProcessor {
 `;
 }
 
-function listFilterDto(base) {
-  const n = names(base);
-  return `import { CommonListFilterDto } from '@neohm/nestend';
-import { Expose } from 'class-transformer';
-import { IsOptional, IsString } from 'class-validator';
-
-export class ${n.listFilterDtoClass} extends CommonListFilterDto {
-  @Expose()
-  @IsOptional()
-  @IsString()
-  custom_filter?: string;
-}
-`;
-}
-
 function listProcessor(base, tableName) {
   const n = names(base);
   return `import {
   CommonListFilterConfig,
+  CommonListFilterDto,
   CommonListProcessor,
   SortDirection,
   SqlService,
 } from '@neohm/nestend';
-import { ${n.listFilterDtoClass} } from '../dtos/${n.dot}.list.filter.dto';
 
+/**
+ * Lists / filters ${n.pascal}. The \`columns\` config below is the allowlist:
+ * an identifier must appear here before a caller can filter or sort on it.
+ * Mark a column \`searchable: true\` to include it in the free-text \`str\` sweep.
+ */
 export class ${n.listProcessorClass} extends CommonListProcessor {
   protected config: CommonListFilterConfig = {
     query: '${tableName} a',
@@ -173,25 +165,11 @@ export class ${n.listProcessorClass} extends CommonListProcessor {
     super();
   }
 
-  async process(payload: ${n.listFilterDtoClass}) {
+  async process(payload: CommonListFilterDto) {
     this.payload = payload;
 
-    this.applyCustomFilters(payload);
-
     return this.handle();
   }
-
-  private async applyCustomFilters(payload: ${n.listFilterDtoClass}) {
-    this.applyCustomFilter(payload.custom_filter);
-  }
-
-  private async applyCustomFilter(custom_filter: string) {
-    if (!custom_filter) return;
-
-    // Example of a hand-rolled condition beyond the config-driven filters.
-    // TODO: replace test_column with a real filterable column of ${n.pascal}.
-    this.conditions.push(\`test_column ilike '%\${custom_filter}%'\`);
-  }
 }
 `;
 }
@@ -403,7 +381,6 @@ module.exports = {
   controller,
   dto,
   processor,
-  listFilterDto,
   listProcessor,
   subscriber,
   job,