@veloxts/cli 0.7.4 → 0.7.5

package/CHANGELOG.md

@@ -1,5 +1,17 @@
 # @veloxts/cli
 
+## 0.7.5
+
+### Patch Changes
+
+- fix(cli): address sync command review findings
+- Updated dependencies
+  - @veloxts/auth@0.7.5
+  - @veloxts/core@0.7.5
+  - @veloxts/orm@0.7.5
+  - @veloxts/router@0.7.5
+  - @veloxts/validation@0.7.5
+
 ## 0.7.4
 
 ### Patch Changes

package/dist/sync/analyzer.js

@@ -117,7 +117,7 @@ function findClosingBrace(content, openIndex) {
                 return i;
         }
     }
-    return content.length;
+    throw new Error('Prisma schema has unbalanced braces — model block not properly closed');
 }
 /**
  * Parse every field-like line from a model body into `RawFieldLine` objects.

package/dist/sync/index.js

@@ -52,7 +52,7 @@ export async function executeSync(projectRoot, options) {
     // ── Stage 3: Prompt ─────────────────────────────────────────
     let choices;
     if (options.force) {
-        choices = models.map((model) => buildDefaultChoices(model));
+        choices = models.map((model) => buildDefaultChoices(model, existing));
     }
     else {
         const prompted = await promptAllModels(models, existing);
@@ -152,6 +152,7 @@ async function generateFiles(snapshot, plan, projectRoot, options) {
     }
     // ── Register procedures in router ───────────────────────────
     if (!options.skipRegistration) {
+        let registrationFailures = 0;
         for (const reg of plan.registrations) {
             try {
                 const result = registerProcedures(projectRoot, reg.entityName, reg.procedureVarName, false);
@@ -159,14 +160,19 @@ async function generateFiles(snapshot, plan, projectRoot, options) {
                     registered.push(reg.procedureVarName);
                 }
                 else if (result.error) {
+                    registrationFailures++;
                     errors.push(`Registration ${reg.procedureVarName}: ${result.error}`);
                 }
             }
             catch (err) {
+                registrationFailures++;
                 const msg = err instanceof Error ? err.message : String(err);
                 errors.push(`Registration ${reg.procedureVarName}: ${msg}`);
             }
         }
+        if (registrationFailures > 0 && registrationFailures === plan.registrations.length) {
+            p.log.warn('All procedure registrations failed. You may need to manually register in src/index.ts.');
+        }
     }
     // ── Display results ─────────────────────────────────────────
     displayResults(created, overwritten, registered, errors, projectRoot);
@@ -237,11 +243,13 @@ function displayResults(created, overwritten, registered, errors, projectRoot) {
 /**
  * Build default ModelChoices for a model (used when --force is set).
  * Generates all CRUD, uses output strategy, includes no relations.
+ * Sets 'regenerate' if existing files are detected, so rollback works correctly.
  */
-function buildDefaultChoices(model) {
+function buildDefaultChoices(model, existing) {
+    const hasExisting = existing.procedures.has(model.name) || existing.schemas.has(model.name);
     return {
         model: model.name,
-        action: 'generate',
+        action: hasExisting ? 'regenerate' : 'generate',
         outputStrategy: 'output',
         crud: {
             get: true,

package/dist/sync/procedure-generator.js

@@ -152,18 +152,15 @@ function generateListProcedure(plan, names) {
     const lines = [];
     lines.push(`  ${procName}: procedure()`);
     lines.push(`    .input(List${names.pascalPlural}Schema)`);
-    if (plan.outputStrategy === 'output') {
-        lines.push(`    .query(async ({ input, ctx }) => {`);
-    }
-    else {
-        lines.push(`    .query(async ({ input, ctx }) => {`);
-    }
+    lines.push(`    .query(async ({ input, ctx }) => {`);
     lines.push(`      const { page, perPage } = input;`);
     lines.push(`      const [items, total] = await Promise.all([`);
     lines.push(`        ctx.db.${names.camel}.findMany({`);
     lines.push(`          skip: (page - 1) * perPage,`);
     lines.push(`          take: perPage,`);
-    lines.push(`          orderBy: { createdAt: 'desc' },`);
+    if (plan.model.hasTimestamps) {
+        lines.push(`          orderBy: { createdAt: 'desc' },`);
+    }
     if (hasIncludes) {
         lines.push(generateIncludeBlock(plan.includeRelations, 10));
     }
@@ -204,7 +201,7 @@ function generateUpdateProcedure(_plan, names) {
     const procName = `update${names.pascal}`;
     const lines = [];
     lines.push(`  ${procName}: procedure()`);
-    lines.push(`    .input(z.object({ id: z.string().uuid() }).merge(Update${names.pascal}Schema))`);
+    lines.push(`    .input(Update${names.pascal}Schema.extend({ id: z.string().uuid() }))`);
     lines.push(`    .mutation(async ({ input, ctx }) => {`);
     lines.push(`      const { id, ...data } = input;`);
     lines.push(`      return ctx.db.${names.camel}.update({ where: { id }, data });`);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@veloxts/cli",
-  "version": "0.7.4",
+  "version": "0.7.5",
   "description": "Developer tooling and CLI commands for VeloxTS framework",
   "type": "module",
   "main": "dist/index.js",
@@ -41,11 +41,11 @@
     "pluralize": "8.0.0",
     "tsx": "4.21.0",
     "yaml": "2.8.2",
-    "@veloxts/core": "0.7.4",
-    "@veloxts/auth": "0.7.4",
-    "@veloxts/router": "0.7.4",
-    "@veloxts/orm": "0.7.4",
-    "@veloxts/validation": "0.7.4"
+    "@veloxts/auth": "0.7.5",
+    "@veloxts/core": "0.7.5",
+    "@veloxts/router": "0.7.5",
+    "@veloxts/orm": "0.7.5",
+    "@veloxts/validation": "0.7.5"
   },
   "peerDependencies": {
     "@prisma/client": ">=7.0.0"