fix: don't write enum/service options when they're empty

stephenh · Sep 2, 2024 · ba318c7 · ba318c7
1 parent c10e4f4
commit ba318c7
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 8 deletions.
diff --git a/integration/meta-typings-as-const/simple.ts b/integration/meta-typings-as-const/simple.ts
diff --git a/src/schema.ts b/src/schema.ts
@@ -140,7 +140,7 @@ export function generateSchema(ctx: Context, fileDesc: FileDescriptorProto, sour
     if (methodsOptions.length > 0 || serviceOptions) {
       servicesOptions.push(code`
         '${service.name}': {
-          options: ${serviceOptions},
+          ${serviceOptions ? code`options: ${serviceOptions},` : ""}
           methods: {${joinCode(methodsOptions, { on: "," })}}
         }
       `);
@@ -173,7 +173,7 @@ export function generateSchema(ctx: Context, fileDesc: FileDescriptorProto, sour
     if (valuesOptions.length > 0 || enumOptions) {
       enumsOptions.push(code`
         '${Enum.name}': {
-          options: ${enumOptions},
+          ${enumOptions ? code`options: ${enumOptions},` : ""}
           values: {${joinCode(valuesOptions, { on: "," })}}
         }
       `);