Revert "[pg-protocol] use literals instead of const enum (#2490)"

This reverts commit 45fa27e.

Author: brianc

packages/pg-protocol/src/messages.ts

@@ -1,75 +1,76 @@
 export type Mode = 'text' | 'binary'
 
-export type MessageName =
-  | 'parseComplete'
-  | 'bindComplete'
-  | 'closeComplete'
-  | 'noData'
-  | 'portalSuspended'
-  | 'replicationStart'
-  | 'emptyQuery'
-  | 'copyDone'
-  | 'copyData'
-  | 'rowDescription'
-  | 'parameterStatus'
-  | 'backendKeyData'
-  | 'notification'
-  | 'readyForQuery'
-  | 'commandComplete'
-  | 'dataRow'
-  | 'copyInResponse'
-  | 'copyOutResponse'
-  | 'authenticationOk'
-  | 'authenticationMD5Password'
-  | 'authenticationCleartextPassword'
-  | 'authenticationSASL'
-  | 'authenticationSASLContinue'
-  | 'authenticationSASLFinal'
-  | 'error'
-  | 'notice'
+export const enum MessageName {
+  parseComplete = 'parseComplete',
+  bindComplete = 'bindComplete',
+  closeComplete = 'closeComplete',
+  noData = 'noData',
+  portalSuspended = 'portalSuspended',
+  replicationStart = 'replicationStart',
+  emptyQuery = 'emptyQuery',
+  copyDone = 'copyDone',
+  copyData = 'copyData',
+  rowDescription = 'rowDescription',
+  parameterStatus = 'parameterStatus',
+  backendKeyData = 'backendKeyData',
+  notification = 'notification',
+  readyForQuery = 'readyForQuery',
+  commandComplete = 'commandComplete',
+  dataRow = 'dataRow',
+  copyInResponse = 'copyInResponse',
+  copyOutResponse = 'copyOutResponse',
+  authenticationOk = 'authenticationOk',
+  authenticationMD5Password = 'authenticationMD5Password',
+  authenticationCleartextPassword = 'authenticationCleartextPassword',
+  authenticationSASL = 'authenticationSASL',
+  authenticationSASLContinue = 'authenticationSASLContinue',
+  authenticationSASLFinal = 'authenticationSASLFinal',
+  error = 'error',
+  notice = 'notice',
+}
 
 export interface BackendMessage {
   name: MessageName
   length: number
 }
 
 export const parseComplete: BackendMessage = {
-  name: 'parseComplete',
+  name: MessageName.parseComplete,
   length: 5,
 }
 
 export const bindComplete: BackendMessage = {
-  name: 'bindComplete',
+  name: MessageName.bindComplete,
   length: 5,
 }
 
 export const closeComplete: BackendMessage = {
-  name: 'closeComplete',
+  name: MessageName.closeComplete,
   length: 5,
 }
 
 export const noData: BackendMessage = {
-  name: 'noData',
+  name: MessageName.noData,
   length: 5,
 }
 
 export const portalSuspended: BackendMessage = {
-  name: 'portalSuspended',
+  name: MessageName.portalSuspended,
   length: 5,
 }
 
 export const replicationStart: BackendMessage = {
-  name: 'replicationStart',
+  name: MessageName.replicationStart,
   length: 4,
 }
 
 export const emptyQuery: BackendMessage = {
-  name: 'emptyQuery',
+  name: MessageName.emptyQuery,
   length: 4,
 }
 
 export const copyDone: BackendMessage = {
-  name: 'copyDone',
+  name: MessageName.copyDone,
   length: 4,
 }
 
@@ -116,7 +117,7 @@ export class DatabaseError extends Error implements NoticeOrError {
 }
 
 export class CopyDataMessage {
-  public readonly name = 'copyData'
+  public readonly name = MessageName.copyData
   constructor(public readonly length: number, public readonly chunk: Buffer) {}
 }
 
@@ -145,15 +146,15 @@ export class Field {
 }
 
 export class RowDescriptionMessage {
-  public readonly name: MessageName = 'rowDescription'
+  public readonly name: MessageName = MessageName.rowDescription
   public readonly fields: Field[]
   constructor(public readonly length: number, public readonly fieldCount: number) {
     this.fields = new Array(this.fieldCount)
   }
 }
 
 export class ParameterStatusMessage {
-  public readonly name: MessageName = 'parameterStatus'
+  public readonly name: MessageName = MessageName.parameterStatus
   constructor(
     public readonly length: number,
     public readonly parameterName: string,
@@ -162,17 +163,17 @@ export class ParameterStatusMessage {
 }
 
 export class AuthenticationMD5Password implements BackendMessage {
-  public readonly name: MessageName = 'authenticationMD5Password'
+  public readonly name: MessageName = MessageName.authenticationMD5Password
   constructor(public readonly length: number, public readonly salt: Buffer) {}
 }
 
 export class BackendKeyDataMessage {
-  public readonly name: MessageName = 'backendKeyData'
+  public readonly name: MessageName = MessageName.backendKeyData
   constructor(public readonly length: number, public readonly processID: number, public readonly secretKey: number) {}
 }
 
 export class NotificationResponseMessage {
-  public readonly name: MessageName = 'notification'
+  public readonly name: MessageName = MessageName.notification
   constructor(
     public readonly length: number,
     public readonly processId: number,
@@ -182,26 +183,26 @@ export class NotificationResponseMessage {
 }
 
 export class ReadyForQueryMessage {
-  public readonly name: MessageName = 'readyForQuery'
+  public readonly name: MessageName = MessageName.readyForQuery
   constructor(public readonly length: number, public readonly status: string) {}
 }
 
 export class CommandCompleteMessage {
-  public readonly name: MessageName = 'commandComplete'
+  public readonly name: MessageName = MessageName.commandComplete
   constructor(public readonly length: number, public readonly text: string) {}
 }
 
 export class DataRowMessage {
   public readonly fieldCount: number
-  public readonly name: MessageName = 'dataRow'
+  public readonly name: MessageName = MessageName.dataRow
   constructor(public length: number, public fields: any[]) {
     this.fieldCount = fields.length
   }
 }
 
 export class NoticeMessage implements BackendMessage, NoticeOrError {
   constructor(public readonly length: number, public readonly message: string | undefined) {}
-  public readonly name = 'notice'
+  public readonly name = MessageName.notice
   public severity: string | undefined
   public code: string | undefined
   public detail: string | undefined

packages/pg-protocol/src/parser.ts

@@ -183,9 +183,9 @@ export class Parser {
       case MessageCodes.BackendKeyData:
         return this.parseBackendKeyData(offset, length, bytes)
       case MessageCodes.ErrorMessage:
-        return this.parseErrorMessage(offset, length, bytes, 'error')
+        return this.parseErrorMessage(offset, length, bytes, MessageName.error)
       case MessageCodes.NoticeMessage:
-        return this.parseErrorMessage(offset, length, bytes, 'notice')
+        return this.parseErrorMessage(offset, length, bytes, MessageName.notice)
       case MessageCodes.RowDescriptionMessage:
         return this.parseRowDescriptionMessage(offset, length, bytes)
       case MessageCodes.CopyIn:
@@ -217,11 +217,11 @@ export class Parser {
   }
 
   private parseCopyInMessage(offset: number, length: number, bytes: Buffer) {
-    return this.parseCopyMessage(offset, length, bytes, 'copyInResponse')
+    return this.parseCopyMessage(offset, length, bytes, MessageName.copyInResponse)
   }
 
   private parseCopyOutMessage(offset: number, length: number, bytes: Buffer) {
-    return this.parseCopyMessage(offset, length, bytes, 'copyOutResponse')
+    return this.parseCopyMessage(offset, length, bytes, MessageName.copyOutResponse)
   }
 
   private parseCopyMessage(offset: number, length: number, bytes: Buffer, messageName: MessageName) {
@@ -295,7 +295,7 @@ export class Parser {
     const code = this.reader.int32()
     // TODO(bmc): maybe better types here
     const message: BackendMessage & any = {
-      name: 'authenticationOk',
+      name: MessageName.authenticationOk,
       length,
     }
 
@@ -304,18 +304,18 @@ export class Parser {
         break
       case 3: // AuthenticationCleartextPassword
         if (message.length === 8) {
-          message.name = 'authenticationCleartextPassword'
+          message.name = MessageName.authenticationCleartextPassword
         }
         break
       case 5: // AuthenticationMD5Password
         if (message.length === 12) {
-          message.name = 'authenticationMD5Password'
+          message.name = MessageName.authenticationMD5Password
           const salt = this.reader.bytes(4)
           return new AuthenticationMD5Password(length, salt)
         }
         break
       case 10: // AuthenticationSASL
-        message.name = 'authenticationSASL'
+        message.name = MessageName.authenticationSASL
         message.mechanisms = []
         let mechanism: string
         do {
@@ -327,11 +327,11 @@ export class Parser {
         } while (mechanism)
         break
       case 11: // AuthenticationSASLContinue
-        message.name = 'authenticationSASLContinue'
+        message.name = MessageName.authenticationSASLContinue
         message.data = this.reader.string(length - 8)
         break
       case 12: // AuthenticationSASLFinal
-        message.name = 'authenticationSASLFinal'
+        message.name = MessageName.authenticationSASLFinal
         message.data = this.reader.string(length - 8)
         break
       default:
@@ -352,7 +352,9 @@ export class Parser {
     const messageValue = fields.M
 
     const message =
-      name === 'notice' ? new NoticeMessage(length, messageValue) : new DatabaseError(messageValue, length, name)
+      name === MessageName.notice
+        ? new NoticeMessage(length, messageValue)
+        : new DatabaseError(messageValue, length, name)
 
     message.severity = fields.S
     message.code = fields.C