@webpieces/dev-config 0.2.81 → 0.2.83

package/architecture/executors/validate-dtos/executor.ts

@@ -40,6 +40,7 @@ export interface ValidateDtosOptions {
     mode?: ValidateDtosMode;
     prismaSchemaPath?: string;
     dtoSourcePaths?: string[];
+    ignoreModifiedUntilEpoch?: number;
 }
 
 export interface ExecutorResult {
@@ -393,29 +394,92 @@ function findViolations(
 }
 
 /**
- * Report violations to console.
+ * Compute similarity between two strings using longest common subsequence ratio.
+ * Returns a value between 0 and 1, where 1 is an exact match.
+ */
+function similarity(a: string, b: string): number {
+    const al = a.toLowerCase();
+    const bl = b.toLowerCase();
+    if (al === bl) return 1;
+
+    const m = al.length;
+    const n = bl.length;
+    const prev = new Array<number>(n + 1).fill(0);
+    const curr = new Array<number>(n + 1).fill(0);
+
+    for (let i = 1; i <= m; i++) {
+        for (let j = 1; j <= n; j++) {
+            if (al[i - 1] === bl[j - 1]) {
+                curr[j] = prev[j - 1] + 1;
+            } else {
+                curr[j] = Math.max(prev[j], curr[j - 1]);
+            }
+        }
+        for (let j = 0; j <= n; j++) {
+            prev[j] = curr[j];
+            curr[j] = 0;
+        }
+    }
+
+    const lcsLen = prev[n];
+    return (2 * lcsLen) / (m + n);
+}
+
+/**
+ * Find the closest matching Dbo field for a given Dto field name.
+ * Returns the best match or null if nothing scores above the threshold.
+ */
+function suggestClosestField(fieldName: string, availableFields: string[]): string | null {
+    let bestMatch: string | null = null;
+    let bestScore = 0.4; // minimum threshold
+
+    for (const candidate of availableFields) {
+        const score = similarity(fieldName, candidate);
+        if (score > bestScore) {
+            bestScore = score;
+            bestMatch = candidate;
+        }
+    }
+
+    return bestMatch;
+}
+
+/**
+ * Report violations to console with prominent AI warning and rename suggestions.
  */
 function reportViolations(violations: DtoViolation[]): void {
     console.error('');
-    console.error('❌ DTO fields don\'t match Prisma Dbo models!');
+    console.error('❌❌❌ DTO FIELD MISMATCH - AI AGENT: READ THIS CAREFULLY ❌❌❌');
+    console.error('');
+    console.error('🛑 DO NOT attempt find/replace or manual renaming. Instead:');
+    console.error('   1. IDE rename is 10x faster than AI renaming PLUS 100% correct vs. AI\'s 80%');
+    console.error('   2. Based on errors below, suggest renames to the user');
+    console.error('   3. Ask the human to use IDE rename-refactoring (F2 / right-click → Rename Symbol)');
     console.error('');
     console.error('📚 Every non-deprecated field in a Dto must exist in the corresponding Dbo.');
-    console.error('   This prevents AI from inventing field names that don\'t match the database schema.');
-    console.error('   Dbo can have extra fields (e.g., password) - Dto cannot.');
+    console.error('   Dbo can have extra fields (e.g., password) — Dto fields cannot be invented.');
+    console.error('');
+    console.error('   When needing fields from multiple tables (e.g., a join), use a XxxJoinDto that');
+    console.error('   contains YYDto and ZZDto fields from the other tables instead of flattening.');
+    console.error('   Fields marked @deprecated in the Dto are exempt from this check.');
     console.error('');
 
     for (const v of violations) {
         console.error(`  ❌ ${v.file}:${v.line}`);
         console.error(`     ${v.dtoName}.${v.fieldName} does not exist in ${v.dboName}`);
-        console.error(`     Available Dbo fields: ${v.availableFields.join(', ')}`);
+
+        const suggestion = suggestClosestField(v.fieldName, v.availableFields);
+        if (suggestion) {
+            console.error(`     💡 Suggested rename: ${v.fieldName} → ${suggestion}   (Use IDE rename on ${v.dtoName}.${v.fieldName})`);
+        } else {
+            const preview = v.availableFields.slice(0, 8).join(', ');
+            const ellipsis = v.availableFields.length > 8 ? ', ...' : '';
+            console.error(`     💡 No close match found. Available: ${preview}${ellipsis}`);
+        }
     }
     console.error('');
 
-    console.error('   Dto fields must be a subset of Dbo fields (matching camelCase field names).');
-    console.error('   Fields marked @deprecated in the Dto are exempt from this check.');
-    console.error('');
-    console.error('   When needing fields from multiple tables (e.g., a join), use a XxxJoinDto that');
-    console.error('   contains YYDto and ZZDto fields from the other tables instead of flattening.');
+    console.error('❌❌❌ AI AGENT: Suggest to the human which are the most likely renames needed and list them ❌❌❌');
     console.error('');
 }
 
@@ -559,12 +623,32 @@ function validateDtoFiles(
     return { success: false };
 }
 
+/**
+ * Resolve mode considering ignoreModifiedUntilEpoch override.
+ * When active, downgrades to OFF. When expired, logs a warning.
+ */
+function resolveMode(normalMode: ValidateDtosMode, epoch: number | undefined): ValidateDtosMode {
+    if (epoch === undefined || normalMode === 'OFF') {
+        return normalMode;
+    }
+    const nowSeconds = Date.now() / 1000;
+    if (nowSeconds < epoch) {
+        const expiresDate = new Date(epoch * 1000).toISOString().split('T')[0];
+        console.log(`\n⏭️  Skipping validate-dtos (ignoreModifiedUntilEpoch active, expires: ${expiresDate})`);
+        console.log('');
+        return 'OFF';
+    }
+    const expiresDate = new Date(epoch * 1000).toISOString().split('T')[0];
+    console.log(`\n⚠️  validateDtos.ignoreModifiedUntilEpoch (${epoch}) has expired (${expiresDate}). Remove it from nx.json. Using normal mode: ${normalMode}\n`);
+    return normalMode;
+}
+
 export default async function runExecutor(
     options: ValidateDtosOptions,
     context: ExecutorContext
 ): Promise<ExecutorResult> {
     const workspaceRoot = context.root;
-    const mode: ValidateDtosMode = options.mode ?? 'OFF';
+    const mode = resolveMode(options.mode ?? 'OFF', options.ignoreModifiedUntilEpoch);
 
     if (mode === 'OFF') {
         console.log('\n⏭️  Skipping validate-dtos (mode: OFF)');

package/architecture/executors/validate-dtos/schema.json

@@ -18,6 +18,10 @@
             "type": "array",
             "items": { "type": "string" },
             "description": "Array of directories (relative to workspace root) containing Dto files"
+        },
+        "ignoreModifiedUntilEpoch": {
+            "type": "number",
+            "description": "Epoch seconds. Until this time, skip DTO validation entirely. When expired, normal mode resumes. Omit when not needed."
         }
     },
     "required": []

package/architecture/executors/validate-prisma-converters/executor.d.ts

@@ -0,0 +1,57 @@
+/**
+ * Validate Prisma Converters Executor
+ *
+ * Validates that Prisma converter methods follow a scalable pattern:
+ * methods returning XxxDto (where XxxDbo exists in schema.prisma) must
+ * accept that exact XxxDbo as the first parameter. This keeps single-table
+ * converters clean and forces join converters to compose them.
+ *
+ * ============================================================================
+ * RULES
+ * ============================================================================
+ *
+ * 1. First param must be exact Dbo:
+ *    If method returns XxxDto and XxxDbo exists in schema.prisma,
+ *    the first parameter must be of type XxxDbo.
+ *
+ * 2. Extra params must be booleans:
+ *    Additional parameters beyond the Dbo are allowed but must be boolean
+ *    (used for filtering payloads / security info).
+ *
+ * 3. No async converters:
+ *    Methods returning Promise<XxxDto> are flagged — converters should be
+ *    pure data mapping, no async work.
+ *
+ * 4. No standalone functions:
+ *    Standalone functions in converter files are flagged — must be class
+ *    methods so the converter class can be injected (dependency tree tracing).
+ *
+ * 5. Dto creation outside converters directory:
+ *    Files outside the configured convertersPath that create `new XxxDto(...)`
+ *    where XxxDbo exists in schema.prisma are flagged — Dto instances tied to
+ *    a Dbo must only be created via a converter class.
+ *
+ * ============================================================================
+ * SKIP CONDITIONS
+ * ============================================================================
+ * - Methods with @deprecated decorator or JSDoc tag
+ * - Lines with: // webpieces-disable prisma-converter -- [reason]
+ *
+ * ============================================================================
+ * MODES
+ * ============================================================================
+ * - OFF:            Skip validation entirely
+ * - MODIFIED_FILES: Validate converter files that were modified in the diff
+ */
+import type { ExecutorContext } from '@nx/devkit';
+export type PrismaConverterMode = 'OFF' | 'MODIFIED_FILES';
+export interface ValidatePrismaConvertersOptions {
+    mode?: PrismaConverterMode;
+    schemaPath?: string;
+    convertersPaths?: string[];
+    ignoreModifiedUntilEpoch?: number;
+}
+export interface ExecutorResult {
+    success: boolean;
+}
+export default function runExecutor(options: ValidatePrismaConvertersOptions, context: ExecutorContext): Promise<ExecutorResult>;