fix: remove createCRUDBuilder

Author: 0xcadams

README.md

@@ -92,7 +92,6 @@ You can also control optional outputs from the generator:
 
 - **--skip-types**: Skip generating table `Row[]` type exports.
 - **--skip-builder**: Skip generating the query `createBuilder` export.
-- **--skip-crud**: Skip generating the CRUD mutator `createCRUDBuilder` export.
 - **--skip-declare**: Skip generating the module augmentation for default types in Zero.
 - **--enable-legacy-mutators**: Enable legacy CRUD mutators (sets `enableLegacyMutators` to `true` in the generated schema).
 - **--enable-legacy-queries**: Enable legacy CRUD queries (sets `enableLegacyQueries` to `true` in the generated schema).

package.json

@@ -1,6 +1,6 @@
 {
   "name": "drizzle-zero",
-  "version": "0.17.0-canary.1",
+  "version": "0.17.0-canary.2",
   "description": "Generate Zero schemas from Drizzle ORM schemas",
   "type": "module",
   "scripts": {

src/cli/index.ts

@@ -61,7 +61,6 @@ export interface GeneratorOptions {
   jsFileExtension?: boolean;
   skipTypes?: boolean;
   skipBuilder?: boolean;
-  skipCrud?: boolean;
   skipDeclare?: boolean;
   enableLegacyMutators?: boolean;
   enableLegacyQueries?: boolean;
@@ -79,7 +78,6 @@ async function main(opts: GeneratorOptions = {}) {
     jsFileExtension,
     skipTypes,
     skipBuilder,
-    skipCrud,
     skipDeclare,
     enableLegacyMutators,
     enableLegacyQueries,
@@ -152,7 +150,6 @@ async function main(opts: GeneratorOptions = {}) {
     jsExtensionOverride: jsFileExtension ? 'force' : 'auto',
     skipTypes: Boolean(skipTypes),
     skipBuilder: Boolean(skipBuilder),
-    skipCrud: Boolean(skipCrud),
     skipDeclare: Boolean(skipDeclare),
     enableLegacyMutators: Boolean(enableLegacyMutators),
     enableLegacyQueries: Boolean(enableLegacyQueries),
@@ -204,7 +201,6 @@ function cli() {
     )
     .option('--skip-types', 'Skip generating table Row[] type exports', false)
     .option('--skip-builder', 'Skip generating the builder export', false)
-    .option('--skip-crud', 'Skip generating the CRUD builder export', false)
     .option(
       '--skip-declare',
       'Skip generating the module augmentation for default types in Zero',
@@ -234,7 +230,6 @@ function cli() {
         jsFileExtension: command.jsFileExtension,
         skipTypes: command.skipTypes,
         skipBuilder: command.skipBuilder,
-        skipCrud: command.skipCrud,
         skipDeclare: command.skipDeclare,
         enableLegacyMutators: command.enableLegacyMutators,
         enableLegacyQueries: command.enableLegacyQueries,

src/cli/shared.ts

@@ -17,7 +17,6 @@ export function getGeneratedSchema({
   jsExtensionOverride = 'auto',
   skipTypes = false,
   skipBuilder = false,
-  skipCrud = false,
   skipDeclare = false,
   enableLegacyMutators = false,
   enableLegacyQueries = false,
@@ -31,7 +30,6 @@ export function getGeneratedSchema({
   jsExtensionOverride?: 'auto' | 'force' | 'none';
   skipTypes?: boolean;
   skipBuilder?: boolean;
-  skipCrud?: boolean;
   skipDeclare?: boolean;
   enableLegacyMutators?: boolean;
   enableLegacyQueries?: boolean;
@@ -568,30 +566,6 @@ export function getGeneratedSchema({
     });
   }
 
-  // Add CRUD builder export
-  if (!skipCrud) {
-    zeroSchemaGenerated.addImportDeclaration({
-      moduleSpecifier: '@rocicorp/zero',
-      namedImports: [{name: 'createCRUDBuilder'}],
-    });
-
-    const crudVariable = zeroSchemaGenerated.addVariableStatement({
-      declarationKind: VariableDeclarationKind.Const,
-      isExported: true,
-      declarations: [
-        {
-          name: 'crud',
-          initializer: `createCRUDBuilder(${schemaObjectName})`,
-        },
-      ],
-    });
-
-    crudVariable.addJsDoc({
-      description:
-        '\nRepresents the CRUD builder.\nThis type is auto-generated from your Drizzle schema definition.',
-    });
-  }
-
   // Add module augmentation for default types
   if (!skipDeclare) {
     zeroSchemaGenerated.addStatements(writer => {

tests/cli.test.ts

@@ -137,7 +137,7 @@ describe('getGeneratedSchema', () => {
       // You should NOT make any changes in this file as it will be overwritten.
 
       import type { Row } from "@rocicorp/zero";
-      import { createBuilder, createCRUDBuilder } from "@rocicorp/zero";
+      import { createBuilder } from "@rocicorp/zero";
       import type { ZeroCustomType } from "drizzle-zero";
       import type { schema as zeroSchema } from "./tests/schemas/one-to-one.zero";
 
@@ -1048,7 +1048,7 @@ describe('getGeneratedSchema', () => {
 
     // Check for @rocicorp/zero imports
     expect(generatedSchema).toContain(
-      'import { createBuilder, createCRUDBuilder } from "@rocicorp/zero";',
+      'import { createBuilder } from "@rocicorp/zero";',
     );
     expect(generatedSchema).toContain(
       'import type { Row } from "@rocicorp/zero";',
@@ -1213,7 +1213,7 @@ describe('getGeneratedSchema', () => {
 
     // Check for all the new imports
     expect(generatedSchema).toContain(
-      'import { createBuilder, createCRUDBuilder } from "@rocicorp/zero";',
+      'import { createBuilder } from "@rocicorp/zero";',
     );
     expect(generatedSchema).toContain(
       'import type { Row } from "@rocicorp/zero";',
@@ -1255,7 +1255,7 @@ describe('getGeneratedSchema', () => {
 
     // Should still include builder export but not Row import when no tables
     expect(generatedSchema).toContain(
-      'import { createBuilder, createCRUDBuilder } from "@rocicorp/zero";',
+      'import { createBuilder } from "@rocicorp/zero";',
     );
     expect(generatedSchema).not.toContain(
       'import type { Row } from "@rocicorp/zero";',
@@ -1307,7 +1307,7 @@ describe('getGeneratedSchema', () => {
 
     // Builder should still be present
     expect(generatedSchema).toContain(
-      'import { createBuilder, createCRUDBuilder } from "@rocicorp/zero";',
+      'import { createBuilder } from "@rocicorp/zero";',
     );
     expect(generatedSchema).toContain(
       'export const zql = createBuilder(schema);',