polen 0.10.0-next.14 → 0.10.0-next.16

package/src/lib/graphql-change/change-types.ts

@@ -0,0 +1,203 @@
+// dprint-ignore-file
+import type { GraphqlInspector } from '#dep/graphql-inspector/index'
+
+/**
+ * Additional properties that Polen adds to GraphQL Inspector changes
+ */
+export interface ChangeExtensions {
+  message: string
+  path?: string
+  criticality: GraphqlInspector.Core.Criticality
+}
+
+// Type changes
+export interface TypeAddedChange extends GraphqlInspector.Core.TypeAddedChange, ChangeExtensions {}
+export interface TypeRemovedChange extends GraphqlInspector.Core.TypeRemovedChange, ChangeExtensions {}
+export interface TypeKindChangedChange extends GraphqlInspector.Core.TypeKindChangedChange, ChangeExtensions {}
+export interface TypeDescriptionChangedChange extends GraphqlInspector.Core.TypeDescriptionChangedChange, ChangeExtensions {}
+export interface TypeDescriptionAddedChange extends GraphqlInspector.Core.TypeDescriptionAddedChange, ChangeExtensions {}
+export interface TypeDescriptionRemovedChange extends GraphqlInspector.Core.TypeDescriptionRemovedChange, ChangeExtensions {}
+
+// Field changes
+export interface FieldAddedChange extends GraphqlInspector.Core.FieldAddedChange, ChangeExtensions {}
+export interface FieldRemovedChange extends GraphqlInspector.Core.FieldRemovedChange, ChangeExtensions {}
+export interface FieldTypeChangedChange extends GraphqlInspector.Core.FieldTypeChangedChange, ChangeExtensions {}
+// Note: GraphQL Inspector doesn't have FieldDeprecatedChange, it's handled by FieldDeprecationAddedChange
+export interface FieldDeprecationAddedChange extends GraphqlInspector.Core.FieldDeprecationAddedChange, ChangeExtensions {}
+export interface FieldDeprecationRemovedChange extends GraphqlInspector.Core.FieldDeprecationRemovedChange, ChangeExtensions {}
+export interface FieldDeprecationReasonChangedChange extends GraphqlInspector.Core.FieldDeprecationReasonChangedChange, ChangeExtensions {}
+export interface FieldDeprecationReasonAddedChange extends GraphqlInspector.Core.FieldDeprecationReasonAddedChange, ChangeExtensions {}
+export interface FieldDeprecationReasonRemovedChange extends GraphqlInspector.Core.FieldDeprecationReasonRemovedChange, ChangeExtensions {}
+export interface FieldDescriptionChangedChange extends GraphqlInspector.Core.FieldDescriptionChangedChange, ChangeExtensions {}
+export interface FieldDescriptionAddedChange extends GraphqlInspector.Core.FieldDescriptionAddedChange, ChangeExtensions {}
+export interface FieldDescriptionRemovedChange extends GraphqlInspector.Core.FieldDescriptionRemovedChange, ChangeExtensions {}
+
+// Field argument changes
+export interface FieldArgumentAddedChange extends GraphqlInspector.Core.FieldArgumentAddedChange, ChangeExtensions {}
+export interface FieldArgumentRemovedChange extends GraphqlInspector.Core.FieldArgumentRemovedChange, ChangeExtensions {}
+export interface FieldArgumentTypeChangedChange extends GraphqlInspector.Core.FieldArgumentTypeChangedChange, ChangeExtensions {}
+export interface FieldArgumentDescriptionChangedChange extends GraphqlInspector.Core.FieldArgumentDescriptionChangedChange, ChangeExtensions {}
+export interface FieldArgumentDefaultChangedChange extends GraphqlInspector.Core.FieldArgumentDefaultChangedChange, ChangeExtensions {}
+
+// Enum changes
+export interface EnumValueAddedChange extends GraphqlInspector.Core.EnumValueAddedChange, ChangeExtensions {}
+export interface EnumValueRemovedChange extends GraphqlInspector.Core.EnumValueRemovedChange, ChangeExtensions {}
+export interface EnumValueDescriptionChangedChange extends GraphqlInspector.Core.EnumValueDescriptionChangedChange, ChangeExtensions {}
+export interface EnumValueDeprecationReasonAddedChange extends GraphqlInspector.Core.EnumValueDeprecationReasonAddedChange, ChangeExtensions {}
+export interface EnumValueDeprecationReasonChangedChange extends GraphqlInspector.Core.EnumValueDeprecationReasonChangedChange, ChangeExtensions {}
+export interface EnumValueDeprecationReasonRemovedChange extends GraphqlInspector.Core.EnumValueDeprecationReasonRemovedChange, ChangeExtensions {}
+
+// Input field changes
+export interface InputFieldAddedChange extends GraphqlInspector.Core.InputFieldAddedChange, ChangeExtensions {}
+export interface InputFieldRemovedChange extends GraphqlInspector.Core.InputFieldRemovedChange, ChangeExtensions {}
+export interface InputFieldTypeChangedChange extends GraphqlInspector.Core.InputFieldTypeChangedChange, ChangeExtensions {}
+export interface InputFieldDescriptionAddedChange extends GraphqlInspector.Core.InputFieldDescriptionAddedChange, ChangeExtensions {}
+export interface InputFieldDescriptionRemovedChange extends GraphqlInspector.Core.InputFieldDescriptionRemovedChange, ChangeExtensions {}
+export interface InputFieldDescriptionChangedChange extends GraphqlInspector.Core.InputFieldDescriptionChangedChange, ChangeExtensions {}
+export interface InputFieldDefaultValueChangedChange extends GraphqlInspector.Core.InputFieldDefaultValueChangedChange, ChangeExtensions {}
+
+// Union changes
+export interface UnionMemberAddedChange extends GraphqlInspector.Core.UnionMemberAddedChange, ChangeExtensions {}
+export interface UnionMemberRemovedChange extends GraphqlInspector.Core.UnionMemberRemovedChange, ChangeExtensions {}
+
+// Interface changes
+export interface ObjectTypeInterfaceAddedChange extends GraphqlInspector.Core.ObjectTypeInterfaceAddedChange, ChangeExtensions {}
+export interface ObjectTypeInterfaceRemovedChange extends GraphqlInspector.Core.ObjectTypeInterfaceRemovedChange, ChangeExtensions {}
+
+// Directive changes
+export interface DirectiveAddedChange extends GraphqlInspector.Core.DirectiveAddedChange, ChangeExtensions {}
+export interface DirectiveRemovedChange extends GraphqlInspector.Core.DirectiveRemovedChange, ChangeExtensions {}
+export interface DirectiveDescriptionChangedChange extends GraphqlInspector.Core.DirectiveDescriptionChangedChange, ChangeExtensions {}
+export interface DirectiveLocationAddedChange extends GraphqlInspector.Core.DirectiveLocationAddedChange, ChangeExtensions {}
+export interface DirectiveLocationRemovedChange extends GraphqlInspector.Core.DirectiveLocationRemovedChange, ChangeExtensions {}
+export interface DirectiveArgumentAddedChange extends GraphqlInspector.Core.DirectiveArgumentAddedChange, ChangeExtensions {}
+export interface DirectiveArgumentRemovedChange extends GraphqlInspector.Core.DirectiveArgumentRemovedChange, ChangeExtensions {}
+export interface DirectiveArgumentDescriptionChangedChange extends GraphqlInspector.Core.DirectiveArgumentDescriptionChangedChange, ChangeExtensions {}
+export interface DirectiveArgumentDefaultValueChangedChange extends GraphqlInspector.Core.DirectiveArgumentDefaultValueChangedChange, ChangeExtensions {}
+export interface DirectiveArgumentTypeChangedChange extends GraphqlInspector.Core.DirectiveArgumentTypeChangedChange, ChangeExtensions {}
+
+// Schema changes
+export interface SchemaQueryTypeChangedChange extends GraphqlInspector.Core.SchemaQueryTypeChangedChange, ChangeExtensions {}
+export interface SchemaMutationTypeChangedChange extends GraphqlInspector.Core.SchemaMutationTypeChangedChange, ChangeExtensions {}
+export interface SchemaSubscriptionTypeChangedChange extends GraphqlInspector.Core.SchemaSubscriptionTypeChangedChange, ChangeExtensions {}
+
+// Directive usage changes
+export interface DirectiveUsageUnionMemberAddedChange extends GraphqlInspector.Core.DirectiveUsageUnionMemberAddedChange, ChangeExtensions {}
+export interface DirectiveUsageUnionMemberRemovedChange extends GraphqlInspector.Core.DirectiveUsageUnionMemberRemovedChange, ChangeExtensions {}
+export interface DirectiveUsageEnumAddedChange extends GraphqlInspector.Core.DirectiveUsageEnumAddedChange, ChangeExtensions {}
+export interface DirectiveUsageEnumRemovedChange extends GraphqlInspector.Core.DirectiveUsageEnumRemovedChange, ChangeExtensions {}
+export interface DirectiveUsageEnumValueAddedChange extends GraphqlInspector.Core.DirectiveUsageEnumValueAddedChange, ChangeExtensions {}
+export interface DirectiveUsageEnumValueRemovedChange extends GraphqlInspector.Core.DirectiveUsageEnumValueRemovedChange, ChangeExtensions {}
+export interface DirectiveUsageInputObjectAddedChange extends GraphqlInspector.Core.DirectiveUsageInputObjectAddedChange, ChangeExtensions {}
+export interface DirectiveUsageInputObjectRemovedChange extends GraphqlInspector.Core.DirectiveUsageInputObjectRemovedChange, ChangeExtensions {}
+export interface DirectiveUsageFieldAddedChange extends GraphqlInspector.Core.DirectiveUsageFieldAddedChange, ChangeExtensions {}
+export interface DirectiveUsageFieldRemovedChange extends GraphqlInspector.Core.DirectiveUsageFieldRemovedChange, ChangeExtensions {}
+export interface DirectiveUsageScalarAddedChange extends GraphqlInspector.Core.DirectiveUsageScalarAddedChange, ChangeExtensions {}
+export interface DirectiveUsageScalarRemovedChange extends GraphqlInspector.Core.DirectiveUsageScalarRemovedChange, ChangeExtensions {}
+export interface DirectiveUsageObjectAddedChange extends GraphqlInspector.Core.DirectiveUsageObjectAddedChange, ChangeExtensions {}
+export interface DirectiveUsageObjectRemovedChange extends GraphqlInspector.Core.DirectiveUsageObjectRemovedChange, ChangeExtensions {}
+export interface DirectiveUsageInterfaceAddedChange extends GraphqlInspector.Core.DirectiveUsageInterfaceAddedChange, ChangeExtensions {}
+export interface DirectiveUsageInterfaceRemovedChange extends GraphqlInspector.Core.DirectiveUsageInterfaceRemovedChange, ChangeExtensions {}
+export interface DirectiveUsageArgumentDefinitionAddedChange extends GraphqlInspector.Core.DirectiveUsageArgumentDefinitionAddedChange, ChangeExtensions {}
+export interface DirectiveUsageArgumentDefinitionRemovedChange extends GraphqlInspector.Core.DirectiveUsageArgumentDefinitionRemovedChange, ChangeExtensions {}
+export interface DirectiveUsageSchemaAddedChange extends GraphqlInspector.Core.DirectiveUsageSchemaAddedChange, ChangeExtensions {}
+export interface DirectiveUsageSchemaRemovedChange extends GraphqlInspector.Core.DirectiveUsageSchemaRemovedChange, ChangeExtensions {}
+export interface DirectiveUsageFieldDefinitionAddedChange extends GraphqlInspector.Core.DirectiveUsageFieldDefinitionAddedChange, ChangeExtensions {}
+export interface DirectiveUsageFieldDefinitionRemovedChange extends GraphqlInspector.Core.DirectiveUsageFieldDefinitionRemovedChange, ChangeExtensions {}
+export interface DirectiveUsageInputFieldDefinitionAddedChange extends GraphqlInspector.Core.DirectiveUsageInputFieldDefinitionAddedChange, ChangeExtensions {}
+export interface DirectiveUsageInputFieldDefinitionRemovedChange extends GraphqlInspector.Core.DirectiveUsageInputFieldDefinitionRemovedChange, ChangeExtensions {}
+
+/**
+ * Discriminated union of all possible change types
+ */
+export type Change =
+  | TypeAddedChange
+  | TypeRemovedChange
+  | TypeKindChangedChange
+  | TypeDescriptionChangedChange
+  | TypeDescriptionAddedChange
+  | TypeDescriptionRemovedChange
+  | FieldAddedChange
+  | FieldRemovedChange
+  | FieldTypeChangedChange
+  // | FieldDeprecatedChange
+  | FieldDeprecationAddedChange
+  | FieldDeprecationRemovedChange
+  | FieldDeprecationReasonChangedChange
+  | FieldDeprecationReasonAddedChange
+  | FieldDeprecationReasonRemovedChange
+  | FieldDescriptionChangedChange
+  | FieldDescriptionAddedChange
+  | FieldDescriptionRemovedChange
+  | FieldArgumentAddedChange
+  | FieldArgumentRemovedChange
+  | FieldArgumentTypeChangedChange
+  | FieldArgumentDescriptionChangedChange
+  | FieldArgumentDefaultChangedChange
+  | EnumValueAddedChange
+  | EnumValueRemovedChange
+  | EnumValueDescriptionChangedChange
+  | EnumValueDeprecationReasonAddedChange
+  | EnumValueDeprecationReasonChangedChange
+  | EnumValueDeprecationReasonRemovedChange
+  | InputFieldAddedChange
+  | InputFieldRemovedChange
+  | InputFieldTypeChangedChange
+  | InputFieldDescriptionAddedChange
+  | InputFieldDescriptionRemovedChange
+  | InputFieldDescriptionChangedChange
+  | InputFieldDefaultValueChangedChange
+  | UnionMemberAddedChange
+  | UnionMemberRemovedChange
+  | ObjectTypeInterfaceAddedChange
+  | ObjectTypeInterfaceRemovedChange
+  | DirectiveAddedChange
+  | DirectiveRemovedChange
+  | DirectiveDescriptionChangedChange
+  | DirectiveLocationAddedChange
+  | DirectiveLocationRemovedChange
+  | DirectiveArgumentAddedChange
+  | DirectiveArgumentRemovedChange
+  | DirectiveArgumentDescriptionChangedChange
+  | DirectiveArgumentDefaultValueChangedChange
+  | DirectiveArgumentTypeChangedChange
+  | SchemaQueryTypeChangedChange
+  | SchemaMutationTypeChangedChange
+  | SchemaSubscriptionTypeChangedChange
+  | DirectiveUsageUnionMemberAddedChange
+  | DirectiveUsageUnionMemberRemovedChange
+  | DirectiveUsageEnumAddedChange
+  | DirectiveUsageEnumRemovedChange
+  | DirectiveUsageEnumValueAddedChange
+  | DirectiveUsageEnumValueRemovedChange
+  | DirectiveUsageInputObjectAddedChange
+  | DirectiveUsageInputObjectRemovedChange
+  | DirectiveUsageFieldAddedChange
+  | DirectiveUsageFieldRemovedChange
+  | DirectiveUsageScalarAddedChange
+  | DirectiveUsageScalarRemovedChange
+  | DirectiveUsageObjectAddedChange
+  | DirectiveUsageObjectRemovedChange
+  | DirectiveUsageInterfaceAddedChange
+  | DirectiveUsageInterfaceRemovedChange
+  | DirectiveUsageArgumentDefinitionAddedChange
+  | DirectiveUsageArgumentDefinitionRemovedChange
+  | DirectiveUsageSchemaAddedChange
+  | DirectiveUsageSchemaRemovedChange
+  | DirectiveUsageFieldDefinitionAddedChange
+  | DirectiveUsageFieldDefinitionRemovedChange
+  | DirectiveUsageInputFieldDefinitionAddedChange
+  | DirectiveUsageInputFieldDefinitionRemovedChange
+
+/**
+ * Type guard to check if a value is a valid Change
+ */
+export const isChange = (value: unknown): value is Change => {
+  return (
+    typeof value === 'object' &&
+    value !== null &&
+    'type' in value &&
+    'criticality' in value &&
+    'message' in value
+  )
+}

package/src/lib/graphql-change/criticality.ts

@@ -0,0 +1,45 @@
+import { CriticalityLevel } from '@graphql-inspector/core'
+import type { BadgeProps } from '@radix-ui/themes'
+import type { Change } from './change-types.js'
+
+export { CriticalityLevel } from '@graphql-inspector/core'
+
+// Type guard functions
+export const isCriticalityBreaking = (change: Change): boolean => change.criticality.level === CriticalityLevel.Breaking
+
+export const isCriticalityDangerous = (change: Change): boolean =>
+  change.criticality.level === CriticalityLevel.Dangerous
+
+export const isCriticalitySafe = (change: Change): boolean => change.criticality.level === CriticalityLevel.NonBreaking
+
+/** Ordered array for iteration (from most to least critical) */
+export const CRITICALITY_LEVELS = [
+  CriticalityLevel.Breaking,
+  CriticalityLevel.Dangerous,
+  CriticalityLevel.NonBreaking,
+] as const
+
+// Config for UI display
+interface CriticalityConfig {
+  color: Exclude<BadgeProps['color'], undefined>
+  label: string
+  shortLabel: string
+}
+
+export const CRITICALITY_CONFIG = {
+  [CriticalityLevel.Breaking]: {
+    color: 'red',
+    label: 'Breaking Changes',
+    shortLabel: 'Breaking',
+  },
+  [CriticalityLevel.Dangerous]: {
+    color: 'yellow',
+    label: 'Dangerous Changes',
+    shortLabel: 'Dangerous',
+  },
+  [CriticalityLevel.NonBreaking]: {
+    color: 'green',
+    label: 'Safe Changes',
+    shortLabel: 'Safe',
+  },
+} satisfies Record<CriticalityLevel, CriticalityConfig>

package/src/lib/graphql-change/graphql-change.ts

@@ -1,11 +1,13 @@
 import { GraphqlInspector } from '#dep/graphql-inspector/index'
 import type { GrafaidOld } from '#lib/grafaid-old/index'
+import type { Change } from './change-types.js'
 
-export type Change = GraphqlInspector.Core.SerializableChange & {
-  message: string
-  path?: string
-  criticality: GraphqlInspector.Core.Criticality
-}
+export * from './change-types.js'
+
+export * from './criticality.js'
+
+import * as Group from './change-groups.js'
+export { Group }
 
 export const calcChangeset = async (parameters: {
   after: GrafaidOld.Schema.Schema

package/src/lib/graphql-change/index.ts

@@ -1 +1 @@
-export * as GraphqlChange from './graphql-change.ts'
+export * as GraphqlChange from './graphql-change.js'

package/src/lib/graphql-changeset/index.ts

@@ -1 +1 @@
-export * as GraphqlChangeset from './graphql-changeset.ts'
+export * as GraphqlChangeset from './graphql-changeset.js'

package/src/lib/graphql-document/$$.ts

@@ -1,4 +1,4 @@
-export * from './analysis.ts'
-export * from './positioning-simple.ts'
-export * from './schema-integration.ts'
-export * from './types.ts'
+export * from './analysis.js'
+export * from './positioning-simple.js'
+export * from './schema-integration.js'
+export * from './types.js'

package/src/lib/graphql-document/$.test.ts

@@ -1,5 +1,5 @@
 import { describe, expect, it } from 'vitest'
-import { GraphQLDocument } from './$.ts'
+import { GraphQLDocument } from './$.js'
 
 describe('GraphQLDocument', () => {
   describe('analyze', () => {
@@ -97,7 +97,7 @@ describe('GraphQLDocument', () => {
 
   describe('extractIdentifiers', () => {
     it('should extract identifiers with positions', () => {
-      const source = `query { user { name } }`
+      const source = 'query { user { name } }'
 
       const identifiers = GraphQLDocument.extractIdentifiers(source)
 
@@ -113,7 +113,7 @@ describe('GraphQLDocument', () => {
     })
 
     it('should create proper index maps', () => {
-      const source = `query { user { name } }`
+      const source = 'query { user { name } }'
 
       const identifiers = GraphQLDocument.extractIdentifiers(source)
 

package/src/lib/graphql-document/$.ts

@@ -1 +1 @@
-export * as GraphQLDocument from './graphql-document.ts'
+export * as GraphQLDocument from './graphql-document.js'

package/src/lib/graphql-document/analysis.ts

@@ -21,7 +21,7 @@ import type {
   Identifier,
   IdentifierContext,
   IdentifierMap,
-} from './types.ts'
+} from './types.js'
 
 /**
  * Default GraphQL document analyzer implementation
@@ -36,7 +36,7 @@ export class DefaultGraphQLAnalyzer implements GraphQLAnalyzer {
         noLocation: false, // We need location info for positioning
       })
     } catch (error) {
-      throw new Error(`Failed to parse GraphQL document: ${error instanceof Error ? error.message : 'Unknown error'}`)
+      throw new Error(`Failed to parse GraphQL document: ${error instanceof Error ? error.message : `Unknown error`}`)
     }
   }
 
@@ -56,7 +56,7 @@ export class DefaultGraphQLAnalyzer implements GraphQLAnalyzer {
     const schema = config.schema
 
     // Context tracking during traversal
-    let currentOperationType: 'query' | 'mutation' | 'subscription' | undefined
+    let currentOperationType: `query` | `mutation` | `subscription` | undefined
     let currentOperationName: string | undefined
     let currentFragment: string | undefined
     let selectionPath: string[] = []
@@ -81,23 +81,23 @@ export class DefaultGraphQLAnalyzer implements GraphQLAnalyzer {
 
           // Set the root type based on operation type
           if (schema) {
-            if (node.operation === 'query' && schema.getQueryType()) {
+            if (node.operation === `query` && schema.getQueryType()) {
               parentTypes = [schema.getQueryType()!.name]
-            } else if (node.operation === 'mutation' && schema.getMutationType()) {
+            } else if (node.operation === `mutation` && schema.getMutationType()) {
               parentTypes = [schema.getMutationType()!.name]
-            } else if (node.operation === 'subscription' && schema.getSubscriptionType()) {
+            } else if (node.operation === `subscription` && schema.getSubscriptionType()) {
               parentTypes = [schema.getSubscriptionType()!.name]
             } else {
               parentTypes = []
             }
           } else {
             // Fallback to default root type names
-            if (node.operation === 'query') {
-              parentTypes = ['Query']
-            } else if (node.operation === 'mutation') {
-              parentTypes = ['Mutation']
-            } else if (node.operation === 'subscription') {
-              parentTypes = ['Subscription']
+            if (node.operation === `query`) {
+              parentTypes = [`Query`]
+            } else if (node.operation === `mutation`) {
+              parentTypes = [`Mutation`]
+            } else if (node.operation === `subscription`) {
+              parentTypes = [`Subscription`]
             } else {
               parentTypes = []
             }
@@ -119,7 +119,7 @@ export class DefaultGraphQLAnalyzer implements GraphQLAnalyzer {
           // Add fragment name as identifier
           this.addIdentifier(identifiers, {
             name: node.name.value,
-            kind: 'Fragment',
+            kind: `Fragment`,
             position: this.getPosition(node.name),
             schemaPath: [node.name.value],
             context: createContext(),
@@ -128,7 +128,7 @@ export class DefaultGraphQLAnalyzer implements GraphQLAnalyzer {
           // Add type condition as identifier
           this.addIdentifier(identifiers, {
             name: node.typeCondition.name.value,
-            kind: 'Type',
+            kind: `Type`,
             position: this.getPosition(node.typeCondition.name),
             schemaPath: [node.typeCondition.name.value],
             context: createContext(),
@@ -149,7 +149,7 @@ export class DefaultGraphQLAnalyzer implements GraphQLAnalyzer {
 
           this.addIdentifier(identifiers, {
             name: fieldName,
-            kind: 'Field',
+            kind: `Field`,
             position: this.getPosition(node.name),
             parentType,
             schemaPath: parentType ? [parentType, fieldName] : [fieldName],
@@ -161,7 +161,7 @@ export class DefaultGraphQLAnalyzer implements GraphQLAnalyzer {
           let pushedType = false
           if (schema && parentType) {
             const type = schema.getType(parentType)
-            if (type && ('getFields' in type)) {
+            if (type && (`getFields` in type)) {
               const fields = (type as any).getFields()
               const field = fields[fieldName]
               if (field) {
@@ -203,7 +203,7 @@ export class DefaultGraphQLAnalyzer implements GraphQLAnalyzer {
 
           this.addIdentifier(identifiers, {
             name: argName,
-            kind: 'Argument',
+            kind: `Argument`,
             position: this.getPosition(node.name),
             parentType,
             schemaPath: parentType && fieldName
@@ -218,7 +218,7 @@ export class DefaultGraphQLAnalyzer implements GraphQLAnalyzer {
         enter: (node: VariableDefinitionNode) => {
           this.addIdentifier(identifiers, {
             name: node.variable.name.value,
-            kind: 'Variable',
+            kind: `Variable`,
             position: this.getPosition(node.variable.name),
             schemaPath: [node.variable.name.value],
             context: createContext(),
@@ -229,7 +229,7 @@ export class DefaultGraphQLAnalyzer implements GraphQLAnalyzer {
           if (typeName) {
             this.addIdentifier(identifiers, {
               name: typeName,
-              kind: 'Type',
+              kind: `Type`,
               position: this.getTypePosition(node.type),
               schemaPath: [typeName],
               context: createContext(),
@@ -242,7 +242,7 @@ export class DefaultGraphQLAnalyzer implements GraphQLAnalyzer {
         enter: (node: DirectiveNode) => {
           this.addIdentifier(identifiers, {
             name: node.name.value,
-            kind: 'Directive',
+            kind: `Directive`,
             position: this.getPosition(node.name),
             schemaPath: [node.name.value],
             context: createContext(),
@@ -275,10 +275,10 @@ export class DefaultGraphQLAnalyzer implements GraphQLAnalyzer {
       }
     } catch (error) {
       return {
-        ast: { kind: 'Document', definitions: [] } as DocumentNode,
+        ast: { kind: `Document`, definitions: [] } as DocumentNode,
         identifiers: this.createIdentifierMap([], [{
-          message: error instanceof Error ? error.message : 'Unknown parsing error',
-          severity: 'error',
+          message: error instanceof Error ? error.message : `Unknown parsing error`,
+          severity: `error`,
         }]),
         isValid: false,
         errors: [],
@@ -292,7 +292,7 @@ export class DefaultGraphQLAnalyzer implements GraphQLAnalyzer {
     identifiers.push(identifier)
   }
 
-  private getPosition(node: ASTNode): Identifier['position'] {
+  private getPosition(node: ASTNode): Identifier[`position`] {
     const loc = node.loc
     if (!loc) {
       return { start: 0, end: 0, line: 1, column: 1 }
@@ -306,7 +306,7 @@ export class DefaultGraphQLAnalyzer implements GraphQLAnalyzer {
     }
   }
 
-  private getTypePosition(node: ASTNode): Identifier['position'] {
+  private getTypePosition(node: ASTNode): Identifier[`position`] {
     // For type nodes, we need to extract the base type name position
     // This is a simplified implementation
     return this.getPosition(node)
@@ -314,10 +314,10 @@ export class DefaultGraphQLAnalyzer implements GraphQLAnalyzer {
 
   private extractTypeName(typeNode: any): string | null {
     // Recursively extract the base type name from wrapped types (NonNull, List)
-    if (typeNode.kind === 'NamedType') {
+    if (typeNode.kind === `NamedType`) {
       return typeNode.name.value
     }
-    if (typeNode.kind === 'NonNullType' || typeNode.kind === 'ListType') {
+    if (typeNode.kind === `NonNullType` || typeNode.kind === `ListType`) {
       return this.extractTypeName(typeNode.type)
     }
     return null
@@ -327,37 +327,37 @@ export class DefaultGraphQLAnalyzer implements GraphQLAnalyzer {
     // This is only used as a fallback when no schema is available
     // Common patterns for object-returning fields
     const objectFieldPatterns = [
-      'user',
-      'users',
-      'post',
-      'posts',
-      'comment',
-      'comments',
-      'profile',
-      'settings',
-      'organization',
-      'project',
-      'target',
-      'member',
-      'members',
-      'node',
-      'nodes',
-      'edge',
-      'edges',
+      `user`,
+      `users`,
+      `post`,
+      `posts`,
+      `comment`,
+      `comments`,
+      `profile`,
+      `settings`,
+      `organization`,
+      `project`,
+      `target`,
+      `member`,
+      `members`,
+      `node`,
+      `nodes`,
+      `edge`,
+      `edges`,
     ]
     return objectFieldPatterns.some(pattern => fieldName.toLowerCase().includes(pattern))
   }
 
   private inferReturnType(parentType: string | undefined, fieldName: string): string {
     // Simplified type inference - in real implementation would use schema
-    if (fieldName === 'user') return 'User'
-    if (fieldName === 'posts') return 'Post'
-    if (fieldName === 'comments') return 'Comment'
+    if (fieldName === `user`) return `User`
+    if (fieldName === `posts`) return `Post`
+    if (fieldName === `comments`) return `Comment`
     return fieldName.charAt(0).toUpperCase() + fieldName.slice(1)
   }
 
   private createContext(
-    operationType?: 'query' | 'mutation' | 'subscription',
+    operationType?: `query` | `mutation` | `subscription`,
     operationName?: string,
     inFragment?: string,
     selectionPath: string[] = [],
@@ -372,7 +372,7 @@ export class DefaultGraphQLAnalyzer implements GraphQLAnalyzer {
 
   private createIdentifierMap(identifiers: Identifier[], errors: AnalysisError[]): IdentifierMap {
     const byPosition = new Map<number, Identifier>()
-    const byKind = new Map<Identifier['kind'], Identifier[]>()
+    const byKind = new Map<Identifier[`kind`], Identifier[]>()
 
     for (const identifier of identifiers) {
       // Index by position