linting

Author: asoorm

protographic/src/operation-to-proto.ts

@@ -78,25 +78,20 @@ export function compileOperationsToProto(
   const document: DocumentNode = typeof operationSource === 'string' ? parse(operationSource) : operationSource;
 
   // Validate that only a single named operation is present
-  const namedOperations = document.definitions.filter(
-    (def) => def.kind === 'OperationDefinition' && def.name
-  );
+  const namedOperations = document.definitions.filter((def) => def.kind === 'OperationDefinition' && def.name);
 
   if (namedOperations.length === 0) {
     throw new Error(
-      'No named operations found in document. ' +
-      'At least one named operation is required for proto compilation.'
+      'No named operations found in document. ' + 'At least one named operation is required for proto compilation.',
     );
   }
 
   if (namedOperations.length > 1) {
-    const operationNames = namedOperations
-      .map((op: any) => op.name.value)
-      .join(', ');
+    const operationNames = namedOperations.map((op: any) => op.name.value).join(', ');
     throw new Error(
       `Multiple operations found in document: ${operationNames}. ` +
-      'Only a single named operation per document is supported for proto reversibility. ' +
-      'Please compile each operation separately.'
+        'Only a single named operation per document is supported for proto reversibility. ' +
+        'Please compile each operation separately.',
     );
   }
 
@@ -235,8 +230,8 @@ class OperationsToProtoVisitor {
         if (selection.kind === 'Field' && selection.alias) {
           throw new Error(
             `Root-level field alias "${selection.alias.value}: ${selection.name.value}" is not supported. ` +
-            'Field aliases at the root level break proto-to-GraphQL reversibility. ' +
-            'Please remove the alias or use it only on nested fields.'
+              'Field aliases at the root level break proto-to-GraphQL reversibility. ' +
+              'Please remove the alias or use it only on nested fields.',
           );
         }
       }

protographic/tests/operations/enum-support.test.ts

@@ -797,7 +797,7 @@ describe('Enum Support', () => {
       `;
 
       expect(() => compileOperationsToProto(operations, schema)).toThrow(
-        'Multiple operations found in document: GetUser, UpdateUser'
+        'Multiple operations found in document: GetUser, UpdateUser',
       );
     });
   });

protographic/tests/operations/field-ordering-stability.test.ts

@@ -826,7 +826,7 @@ describe('Operations Field Ordering Stability', () => {
       expectValidProto(result1.proto);
 
       const root1 = loadProtoFromText(result1.proto);
-      
+
       // Get field numbers at all nesting levels
       const criteriaFields1 = getFieldNumbersFromMessage(root1, 'SearchCriteria');
       const filterGroupFields1 = getFieldNumbersFromMessage(root1, 'FilterGroup');
@@ -837,17 +837,17 @@ describe('Operations Field Ordering Stability', () => {
       // Store original field numbers at each level
       const criteriaFiltersNumber = criteriaFields1['filters'];
       const criteriaSortingNumber = criteriaFields1['sorting'];
-      
+
       const filterGroupUserNumber = filterGroupFields1['user'];
       const filterGroupDateNumber = filterGroupFields1['date'];
-      
+
       const userFiltersNameNumber = userFiltersFields1['name'];
       const userFiltersEmailNumber = userFiltersFields1['email'];
       const userFiltersActiveNumber = userFiltersFields1['active'];
-      
+
       const dateFiltersFromNumber = dateFiltersFields1['from'];
       const dateFiltersToNumber = dateFiltersFields1['to'];
-      
+
       const sortFieldNumber = sortFields1['field'];
       const sortDirectionNumber = sortFields1['direction'];
 
@@ -869,7 +869,7 @@ describe('Operations Field Ordering Stability', () => {
       expectValidProto(result2.proto);
 
       const root2 = loadProtoFromText(result2.proto);
-      
+
       const criteriaFields2 = getFieldNumbersFromMessage(root2, 'SearchCriteria');
       const filterGroupFields2 = getFieldNumbersFromMessage(root2, 'FilterGroup');
       const userFiltersFields2 = getFieldNumbersFromMessage(root2, 'UserFilters');
@@ -879,17 +879,17 @@ describe('Operations Field Ordering Stability', () => {
       // Verify all field numbers are preserved at all nesting levels
       expect(criteriaFields2['filters']).toBe(criteriaFiltersNumber);
       expect(criteriaFields2['sorting']).toBe(criteriaSortingNumber);
-      
+
       expect(filterGroupFields2['user']).toBe(filterGroupUserNumber);
       expect(filterGroupFields2['date']).toBe(filterGroupDateNumber);
-      
+
       expect(userFiltersFields2['name']).toBe(userFiltersNameNumber);
       expect(userFiltersFields2['email']).toBe(userFiltersEmailNumber);
       expect(userFiltersFields2['active']).toBe(userFiltersActiveNumber);
-      
+
       expect(dateFiltersFields2['from']).toBe(dateFiltersFromNumber);
       expect(dateFiltersFields2['to']).toBe(dateFiltersToNumber);
-      
+
       expect(sortFields2['field']).toBe(sortFieldNumber);
       expect(sortFields2['direction']).toBe(sortDirectionNumber);
     });
@@ -1072,7 +1072,7 @@ describe('Operations Field Ordering Stability', () => {
       `;
 
       expect(() => compileOperationsToProto(operations, schema)).toThrow(
-        'Multiple operations found in document: GetUser, GetUsers'
+        'Multiple operations found in document: GetUser, GetUsers',
       );
     });
   });

protographic/tests/operations/operation-to-proto.test.ts

@@ -232,7 +232,7 @@ describe('Operation to Proto - Integration Tests', () => {
       `;
 
       expect(() => compileOperationsToProto(operations, schema)).toThrow(
-        'Multiple operations found in document: GetUser, GetPosts'
+        'Multiple operations found in document: GetUser, GetPosts',
       );
     });
   });
@@ -385,7 +385,7 @@ describe('Operation to Proto - Integration Tests', () => {
       `;
 
       expect(() => compileOperationsToProto(operations, schema)).toThrow(
-        'Multiple operations found in document: OnMessageAdded, OnUserStatusChanged'
+        'Multiple operations found in document: OnMessageAdded, OnUserStatusChanged',
       );
     });
 
@@ -556,7 +556,7 @@ describe('Operation to Proto - Integration Tests', () => {
       `;
 
       expect(() => compileOperationsToProto(operations, schema)).toThrow(
-        'Multiple operations found in document: GetMessages, AddMessage, OnMessageAdded'
+        'Multiple operations found in document: GetMessages, AddMessage, OnMessageAdded',
       );
     });
 
@@ -606,7 +606,7 @@ describe('Operation to Proto - Integration Tests', () => {
       expect(() =>
         compileOperationsToProto(operations, schema, {
           queryIdempotency: 'NO_SIDE_EFFECTS',
-        })
+        }),
       ).toThrow('Multiple operations found in document: GetMessages, AddMessage, OnMessageAdded');
     });
   });
@@ -1066,7 +1066,7 @@ describe('Operation to Proto - Integration Tests', () => {
       expect(() =>
         compileOperationsToProto(operation, schema, {
           queryIdempotency: 'NO_SIDE_EFFECTS',
-        })
+        }),
       ).toThrow('Multiple operations found in document: GetHello, GetUser');
     });
 
@@ -1142,7 +1142,7 @@ describe('Operation to Proto - Integration Tests', () => {
       expect(() =>
         compileOperationsToProto(operations, schema, {
           queryIdempotency: 'NO_SIDE_EFFECTS',
-        })
+        }),
       ).toThrow('Multiple operations found in document: CreateUser, UpdateUser');
     });
 
@@ -1181,7 +1181,7 @@ describe('Operation to Proto - Integration Tests', () => {
       expect(() =>
         compileOperationsToProto(operations, schema, {
           queryIdempotency: 'NO_SIDE_EFFECTS',
-        })
+        }),
       ).toThrow('Multiple operations found in document: GetUser, CreateUser');
     });
 
@@ -1396,7 +1396,7 @@ describe('Operation to Proto - Integration Tests', () => {
       `;
 
       expect(() => compileOperationsToProto(operations, schema)).toThrow(
-        'Multiple operations found in document: GetUser, CreateUser'
+        'Multiple operations found in document: GetUser, CreateUser',
       );
     });
 
@@ -1447,9 +1447,7 @@ describe('Operation to Proto - Integration Tests', () => {
         }
       `;
 
-      expect(() => compileOperationsToProto(operation, schema)).toThrow(
-        'No named operations found in document'
-      );
+      expect(() => compileOperationsToProto(operation, schema)).toThrow('No named operations found in document');
     });
 
     test('should handle empty selection sets', () => {
@@ -1510,7 +1508,7 @@ describe('Operation to Proto - Integration Tests', () => {
       `;
 
       expect(() => compileOperationsToProto(operation, schema)).toThrow(
-        'Root-level field alias "currentUser: user" is not supported'
+        'Root-level field alias "currentUser: user" is not supported',
       );
     });
   });

protographic/tests/operations/operation-validation.test.ts

@@ -190,7 +190,7 @@ describe('Operation Validation', () => {
       `;
 
       const result = compileOperationsToProto(operation, schema);
-      
+
       // Verify the proto can be generated
       expect(result.proto).toContain('rpc GetUser');
       expect(result.proto).toContain('message GetUserRequest');
@@ -285,4 +285,4 @@ describe('Operation Validation', () => {
       }).toThrow(/Root-level field alias "myUser: user" is not supported/);
     });
   });
-});
+});

protographic/tests/util.ts

@@ -62,10 +62,7 @@ export function getFieldNumbersFromMessage(root: protobufjs.Root, messagePath: s
   } catch (error) {
     // Provide helpful error message with available types
     const availableTypes = getAllNestedTypeNames(root);
-    throw new Error(
-      `Could not find message "${messagePath}". ` +
-      `Available types: ${availableTypes.join(', ')}`
-    );
+    throw new Error(`Could not find message "${messagePath}". ` + `Available types: ${availableTypes.join(', ')}`);
   }
 }
 
@@ -74,7 +71,7 @@ export function getFieldNumbersFromMessage(root: protobufjs.Root, messagePath: s
  */
 function getAllNestedTypeNames(root: protobufjs.Root): string[] {
   const names: string[] = [];
-  
+
   function collectNames(obj: any, prefix: string = '') {
     if (obj.nested) {
       for (const [name, nested] of Object.entries(obj.nested)) {
@@ -84,7 +81,7 @@ function getAllNestedTypeNames(root: protobufjs.Root): string[] {
       }
     }
   }
-  
+
   collectNames(root);
   return names;
 }